Found by the 2026-09-10 coverage review. Each of these tells the next person to read
it that something is unimplemented when it has since landed — the kind of comment that
causes work to be done twice, or a gap to be reported that does not exist.
nobilis/src/backend/discord/messages.rs:422 — "Voting is Discord's own interactive
machinery and is not implemented". Poll voting is discord/polls.rs:158.
nobilis/src/backend/kick/chat.rs:277 — "Announced rather than made interactive:
voting goes through the player". Kick poll voting is kick/api.rs:1455 and the
votable card is emitted at chat.rs:293.
nobilis/src/backend/discord/gateway.rs:1088 — "Reported rather than used: this
build establishes the session only". It calls voice::note_voice_server nine lines
below and Songbird opens a real connection.
nobilis/src/backend/matrix/verification.rs:1-4 — "Self-verification only - no
cross-user identity verification". start_user_verification exists at verification.rs:210.
A related class worth the same pass: doc comments still citing backend/discord.rs:NNNN
and backend/irc.rs:NNNN, which have not existed since the backends were split into
folders.
Found by the 2026-09-10 coverage review. Each of these tells the next person to read
it that something is unimplemented when it has since landed — the kind of comment that
causes work to be done twice, or a gap to be reported that does not exist.
- `nobilis/src/backend/discord/messages.rs:422` — "Voting is Discord's own interactive
machinery and is not implemented". Poll voting is `discord/polls.rs:158`.
- `nobilis/src/backend/kick/chat.rs:277` — "Announced rather than made interactive:
voting goes through the player". Kick poll voting is `kick/api.rs:1455` and the
votable card is emitted at `chat.rs:293`.
- `nobilis/src/backend/discord/gateway.rs:1088` — "Reported rather than used: this
build establishes the session only". It calls `voice::note_voice_server` nine lines
below and Songbird opens a real connection.
- `nobilis/src/backend/matrix/verification.rs:1-4` — "Self-verification only - no
cross-user identity verification". `start_user_verification` exists at
`verification.rs:210`.
A related class worth the same pass: doc comments still citing `backend/discord.rs:NNNN`
and `backend/irc.rs:NNNN`, which have not existed since the backends were split into
folders.
Finished in nobilis f6b3696. The four comments themselves were fixed earlier in e10fe48; this closes the related sweep the ticket names.
33 doc comments named a file that stopped existing when the backends were split into folders. Each was resolved to where the named thing actually lives rather than to the folder it probably moved to — extract_reactions to discord/messages.rs, run_qr_login to discord/login.rs, backfill_channel_history to discord/history.rs, emit_presence to irc/presence.rs. A pointer that is merely close is the same failure one step smaller.
Two were not path renames at all, and would have been mangled by a find-and-replace:
irc/dcc.rs opened by describing itself as sitting "beside irc.rs the way discord_voice.rs sits beside discord.rs" — an analogy to a layout that no longer exists. Rewritten to the one that does.
A runtime field described its roster as the one "userlist.rs builds from", naming a file this repository has never had. It means roomstate.rs's emit_matrix_presence, which the same comment goes on to name two lines later.
Ten references survive the sweep and should. The Sneedchat backend cites auth/totp.rs, chat/json.rs, net/kiwiflare.rs and others from sneedchat-rs — a different project, each with its URL beside it. Those are attributions rather than directions, and rewriting them to local paths would have been the exact mistake this ticket exists to undo.
489 tests pass.
Finished in nobilis [f6b3696](https://git.salastil.com/Salastil/nobilis/commit/f6b3696). The four comments themselves were fixed earlier in [e10fe48](https://git.salastil.com/Salastil/nobilis/commit/e10fe48); this closes the related sweep the ticket names.
**33 doc comments** named a file that stopped existing when the backends were split into folders. Each was resolved to where the named thing actually lives rather than to the folder it probably moved to — `extract_reactions` to `discord/messages.rs`, `run_qr_login` to `discord/login.rs`, `backfill_channel_history` to `discord/history.rs`, `emit_presence` to `irc/presence.rs`. A pointer that is merely close is the same failure one step smaller.
**Two were not path renames at all**, and would have been mangled by a find-and-replace:
- `irc/dcc.rs` opened by describing itself as sitting "beside `irc.rs` the way `discord_voice.rs` sits beside `discord.rs`" — an analogy to a layout that no longer exists. Rewritten to the one that does.
- A runtime field described its roster as the one "`userlist.rs` builds from", naming a file this repository has **never had**. It means `roomstate.rs`'s `emit_matrix_presence`, which the same comment goes on to name two lines later.
**Ten references survive the sweep and should.** The Sneedchat backend cites `auth/totp.rs`, `chat/json.rs`, `net/kiwiflare.rs` and others from **sneedchat-rs** — a different project, each with its URL beside it. Those are attributions rather than directions, and rewriting them to local paths would have been the exact mistake this ticket exists to undo.
489 tests pass.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Found by the 2026-09-10 coverage review. Each of these tells the next person to read
it that something is unimplemented when it has since landed — the kind of comment that
causes work to be done twice, or a gap to be reported that does not exist.
nobilis/src/backend/discord/messages.rs:422— "Voting is Discord's own interactivemachinery and is not implemented". Poll voting is
discord/polls.rs:158.nobilis/src/backend/kick/chat.rs:277— "Announced rather than made interactive:voting goes through the player". Kick poll voting is
kick/api.rs:1455and thevotable card is emitted at
chat.rs:293.nobilis/src/backend/discord/gateway.rs:1088— "Reported rather than used: thisbuild establishes the session only". It calls
voice::note_voice_servernine linesbelow and Songbird opens a real connection.
nobilis/src/backend/matrix/verification.rs:1-4— "Self-verification only - nocross-user identity verification".
start_user_verificationexists atverification.rs:210.A related class worth the same pass: doc comments still citing
backend/discord.rs:NNNNand
backend/irc.rs:NNNN, which have not existed since the backends were split intofolders.
Finished in nobilis f6b3696. The four comments themselves were fixed earlier in e10fe48; this closes the related sweep the ticket names.
33 doc comments named a file that stopped existing when the backends were split into folders. Each was resolved to where the named thing actually lives rather than to the folder it probably moved to —
extract_reactionstodiscord/messages.rs,run_qr_logintodiscord/login.rs,backfill_channel_historytodiscord/history.rs,emit_presencetoirc/presence.rs. A pointer that is merely close is the same failure one step smaller.Two were not path renames at all, and would have been mangled by a find-and-replace:
irc/dcc.rsopened by describing itself as sitting "besideirc.rsthe waydiscord_voice.rssits besidediscord.rs" — an analogy to a layout that no longer exists. Rewritten to the one that does.userlist.rsbuilds from", naming a file this repository has never had. It meansroomstate.rs'semit_matrix_presence, which the same comment goes on to name two lines later.Ten references survive the sweep and should. The Sneedchat backend cites
auth/totp.rs,chat/json.rs,net/kiwiflare.rsand others from sneedchat-rs — a different project, each with its URL beside it. Those are attributions rather than directions, and rewriting them to local paths would have been the exact mistake this ticket exists to undo.489 tests pass.