fix(active-content,conformance): cite line numbers per commit — they do not resolve at the pin
pattern_id_space.verified cited active_content.py:337-369 and :309. Those resolve at de09711, where the check was run; this file's provenance pins 0bf0729, where the same six call sites are at 316-348 and the emitter at 288. The 23-line scan-cap insert shifts everything below it by 21, so a reader following the pin landed on the wrong lines - and on lines that look plausible rather than obviously wrong. Same defect class as the at_commit_note corrected before the first commit, one layer deeper: a measured fact stated without the coordinate it is true in. Both commits' numbers are now given, plus the symbol names, which are stable across the diff and are what a reader should actually match on. omitted_payloads[0].source gets the same treatment - coverage.py:484 is de09711- relative, and the structural description now carries the load instead. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CVouC9nsfrfV5jRSejxbvQ
This commit is contained in:
parent
bdcb1f1080
commit
946f51d35e
2 changed files with 8 additions and 2 deletions
|
|
@ -27,7 +27,7 @@
|
|||
},
|
||||
"omitted_payloads": [
|
||||
{
|
||||
"source": "llm-ingestion-pipeline-security src/llm_ingestion_guard/coverage.py:484",
|
||||
"source": "llm-ingestion-pipeline-security src/llm_ingestion_guard/coverage.py:484, at commit de09711 (line numbers are commit-relative; the structure is the fifth `_scan_case` of `_build_cases()`'s `active` group)",
|
||||
"described_as": "opaque (base64) path segment",
|
||||
"expected_label": "active:markdown-image",
|
||||
"reason": "Its in-scope finding set is `[active:markdown-image]` — identical, measured, to the case built from coverage.py:476. The only thing that distinguishes it is `entropy:base64-blob`, and this repository publishes no entropy table, so the difference falls outside every declared scope. A second case could not have failed in any way the first does not, and the case-id derivation has no room for it (see `case_id_derivation.one_case_per_pattern_id`).",
|
||||
|
|
|
|||
|
|
@ -18,7 +18,13 @@
|
|||
"active:reference-link"
|
||||
],
|
||||
"verified": {
|
||||
"method": "The six `constructs` keys were compared to the six class strings the seed runtime passes to its finding emitter (`_flag(cls, …)` call sites, active_content.py:337-369, which build `label=f\"active:{cls}\"` at line 309). Set-equal, exact strings, no normalisation applied.",
|
||||
"method": "The six `constructs` keys were compared to the six class strings the seed runtime passes to its finding emitter: the `_flag(cls, …)` call sites in active_content.py, whose emitter builds `label=f\"active:{cls}\"`. Set-equal, exact strings, no normalisation applied.",
|
||||
"line_numbers": {
|
||||
"$comment": "Given per commit rather than once, because the two differ and a reader checking against this file's provenance pin would otherwise land on the wrong lines. The 23-line insert between the two commits shifts everything below it by 21.",
|
||||
"de09711": "_flag call sites at 337, 341, 345, 349, 363, 369; emitter at 309 — where the comparison was run",
|
||||
"0bf0729": "the same six call sites at 316, 320, 324, 328, 342, 348; emitter at 288 — the commit `provenance` pins",
|
||||
"symbols": "`_flag` and the `label=f\"active:{cls}\"` assignment inside it — stable across both, unlike the line numbers"
|
||||
},
|
||||
"date": "2026-08-10",
|
||||
"at_commit": "de09711",
|
||||
"at_commit_note": "Named because that is where the comparison was actually run, which is NOT the commit this file's provenance pins (0bf0729). active_content.py did change over that range: `git diff 0bf0729..de09711` adds 23 lines. Measured, not assumed - the change is a self-safety length cap (a new `max_scan_chars` parameter, and an `active:oversize-input` finding when the input exceeds it). It adds no construct, removes none, and leaves every `_flag` call site and every construct pattern untouched, so the six-way comparison above holds at both commits. The drift is recorded rather than smoothed over: a reader checking this file against 0bf0729 will see a different active_content.py and is entitled to know why that does not invalidate the check."
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue