nik/bitchat
1
0
Fork 0
mirror of https://github.com/permissionlesstech/bitchat.git synced 2026-08-29 07:27:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

Drop the periphery ignore the alias sweep made obsolete

Browse Source 
`PeerMessageKey.peerID` carried a `periphery:ignore` because it was read only
through the synthesized `Hashable` conformance, which the indexer cannot
attribute. The alias-scoped sweep in `flushOutbox(forAliases:)` now reads it
directly, so the suppression is superfluous — and Periphery flags a
superfluous ignore as an issue in its own right, which is what turned the
Dead Code check red on this PR.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
ecgang 2026-07-26 13:53:08 -07:00
parent 7ba946b734
commit a4502046ff
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
bitchat/Services/MessageRouter.swift
View File

@ -35,10 +35,10 @@ final class MessageRouter {
typealias QueuedMessage = MessageOutboxStore.QueuedMessage
private struct PeerMessageKey: Hashable {
// periphery:ignore - read only via the synthesized Hashable
// conformance (dictionary-key identity), which the indexer
// cannot attribute; see retain_codable_properties in .periphery.yml
// for the same class of false positive.
// Both properties are read directly now — `peerID` by the alias-scoped
// sweep in `flushOutbox(forAliases:)`, `messageID` throughout — so the
// ignore directive this once carried (for reads visible only through
// the synthesized Hashable conformance) would itself be flagged.
let peerID: PeerID
let messageID: String
}