fix(fight): a commented-out signature header silently deletes a scanner

[hyperpolymath]

What is wrong

parse_signatures splits the directive on the bare literal:

for chunk in raw.split(SIGNATURE_TABLE).skip(1) {

SIGNATURE_TABLE is [[gate-triage.detection.signatures]]. The split matches that
text anywhere — in prose, and inside a # comment.

The directive this parser reads documents its own table name in a comment:
.machine_readable/bot_directives/gate_triage.a2ml:174, inside the
[gate-triage.detection] section that opens at line 171 — well above the three real
headers at 258 / 265 / 272.

Two failures, both now covered by tests that are RED against the old parser

Grafting the two new tests onto the current parser, unchanged:

test gate_triage::tests::a_prose_mention_of_the_table_name_does_not_open_a_scanner ... FAILED
  left: 2
 right: 1

test gate_triage::tests::a_commented_out_header_neither_opens_nor_closes_a_table ... FAILED
  left: 0
 right: 1

test result: FAILED. 43 passed; 2 failed

1. A prose or commented mention opens a phantom scanner — left: 2, right: 1.
2. A commented-out header deletes a real scanner — left: 0, right: 1. This is the
   serious one. The stray match closes the live table above it and opens a body which
   the section-header scan then discards, so both halves are dropped and the scanner
   vanishes. It does not add noise; it silently removes enforcement.

The fix

Line-oriented. signature_bodies walks split_inclusive('\n'), strips each line's
trailing comment via code_of, and treats a line as a header only when the code
part is exactly the table name. A comment can then neither open nor close a table.

until_next_header is retained — section_of still calls it, so removing it would
trip unused_imports/dead-code under -D warnings.

Two pre-existing assert_eq! calls in this file are reflowed by rustfmt; that is the
only churn outside the new code.

Not fixed here, but adjacent

extract_array in crates/squabble-fight/src/context.rs:152 has the same
comment-blindness: once collecting, it pushes quoted strings from every subsequent
line, comments included, until ]. Out of scope for this PR; worth a follow-up.

Verification

cargo test --workspace --all-features                                 130 passed, 0 failed
cargo clippy --workspace --all-features --all-targets -- -D warnings  rc=0
cargo fmt --all --check                                               gate_triage.rs clean

cargo fmt --all --check still exits 1 for three diffs in
crates/squabble-core/src/polarity.rs — those are pre-existing on main, unrelated
to this change, and deliberately untouched.

Two things a reviewer should know

• Stacked on fix(cli): remove duplicate Evidence import that broke the workspace build #78. main does not compile (duplicate Evidence import), so this
  branch is based on the build repair. Base retargets to main automatically when fix(cli): remove duplicate Evidence import that broke the workspace build #78
  merges.
• CI will not produce a green Rust check. The Rust CI workflow in this repo is
  startup-dead — runs are created with jobs.total_count == 0 and emit no check run at
  all, so the required context never reports. The evidence above is local.

🤖 Generated with Claude Code

https://claude.ai/code/session_0178nN4Nm3neFRy5K9StZKnB

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: ac593e2d-662e-4c6d-9d4e-e932875d4bfb

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud