Skip to content

status --ac-scan: enumerate ACs annotated inside the bold (e.g. AC-1 (VALUE))#447

Merged
clkao merged 1 commit into
mainfrom
spacedock-ensign/ac-scan-value-annotation-skip
Jun 30, 2026
Merged

status --ac-scan: enumerate ACs annotated inside the bold (e.g. AC-1 (VALUE))#447
clkao merged 1 commit into
mainfrom
spacedock-ensign/ac-scan-value-annotation-skip

Conversation

@clkao

@clkao clkao commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Make the deterministic gate AC cross-check see value-annotated acceptance criteria, so the value AC can no longer hide from status --ac-scan.

What changed

  • Broaden the --ac-scan heading matcher so a label inside the bold (**AC-1 (VALUE)**) enumerates as its bare id.
  • Use [^*]* so a match can never span a ** boundary — no over-match.
  • Add TestACScanEnumeratesAnnotatedAC + annotated-ac-fixture.md (RED on the bare matcher, GREEN after).
  • Sync the dev README validation cross-check prose at :131.

Evidence

  • go test ./internal/status/... — all passed; TestACScanEnumeratesAnnotatedAC PASS.
  • Measured on rebuilt baseline+fixed binaries: real fn-binding-refinements.md 7→8 ACs (**AC-4 (value guardrail)** surfaces); fixture id set exactly {AC-1, AC-2}.

48g

Broaden acHeadingRe so an asterisk-free trailing label inside the bold
span is allowed and discarded — **AC-1 (VALUE)** enumerates as AC-1,
exactly as bare **AC-1**, so the value AC the README ideation policy
recommends is no longer silently dropped from the gate AC cross-check.
The [^*]* label excludes *, so it cannot span a ** boundary: two
headings on one line still enumerate separately.

TestACScanEnumeratesAnnotatedAC + annotated-ac-fixture.md: RED on the
bare matcher (AC-1 absent), GREEN after; enumerated id set is exactly
{AC-1, AC-2} (no over-match, prose 'see AC-3' not enumerated). README
validation cross-check prose gains the value-annotation clause.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@clkao
clkao merged commit 87db28e into main Jun 30, 2026
4 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant