Skip to content

Reads sixteen where the count correction had not reached - #83

Merged
johnnyt merged 1 commit into
mainfrom
sp-74k-telemetry-count-index-row
Sep 6, 2026
Merged

Reads sixteen where the count correction had not reached#83
johnnyt merged 1 commit into
mainfrom
sp-74k-telemetry-count-index-row

Conversation

@johnnyt

@johnnyt johnnyt commented Sep 6, 2026

Copy link
Copy Markdown
Member

What

Settles the two count sites that sp-8wv's ADR-0009 amendment (PR 78, main
1015654) corrected by reference but did not reach in text.

The amendment moved decision 8's frozen telemetry list from fourteen event
names to sixteen, by addition, and named two places still reading the old
count: "the Consequences section above and docs/adr/README.md's index
row".

  1. docs/adr/README.md's ADR-0009 index row now reads sixteen, edited
    in place. An index-row count correction is an in-place edit by the
    operator's standing word, recorded on sp-74k's dated SCHEDULE note:
    "the index row edit is in-place by the operator's standing word for
    count corrections". This is the one removed line in the diff, and it is
    authorized.
  2. A dated Note in ADR-0009, amend-by-addition. Reading the record
    turned up a third site the amendment named nowhere: decision 3's
    inventory heading, "Fourteen events, listed with their measurements
    and metadata in docs/telemetry.md". The Note corrects it by addition
    and moves no decision.

The count is verified, not carried

@events in lib/statifier_persistence/telemetry.ex holds sixteen event
names and StatifierPersistence.Telemetry.events/0 returns them. The two
the amendment added are [:statifier_persistence, :child, :recorded] and
[:statifier_persistence, :child, :settled]. Sixteen is the code's count.

Removed lines

git diff origin/main -- docs/adr/0009-telemetry-events-for-the-durable-stepper.md
shows zero removed lines. The single removed line under docs/adr/ is
the README index row, which is the authorized in-place edit above.

Out of scope

  • PR 78's own body says "all four precedent amendment sections" and then
    lists five. That is a merged PR body: historical, body-only, not
    editable here, and deliberately left alone.
  • docs/adr/0010-...'s "fourteen callbacks" is a different count (adapter
    callbacks) and is correct as written.
  • CHANGELOG.md's 0.6.0 entry names the fourteen events shipped in that
    release. Released changelog text is historical and stays as written.

Gate

Docs-only: the touched paths are outside build_paths and
also_gated_paths is empty in .claude/wurk.json, so the repo's docs-only
exemption applies. CI runs the gate.

Refs: sp-74k

sp-8wv corrected decision 8's frozen telemetry list from fourteen event
names to sixteen, by addition, and named two sites still carrying the old
count. Both are settled here.

docs/adr/README.md's ADR-0009 index row now reads sixteen, edited in
place: an index-row count correction is an in-place edit by the
operator's standing word, recorded on this bead's dated SCHEDULE note.

The record itself gains a dated Note. The amendment named the Consequences
section and the index row; decision 3's "Fourteen events" inventory
heading it did not name, and that line is corrected by addition, zero
removed lines under docs/adr/.

The count is checked against the code, not carried: @events in
lib/statifier_persistence/telemetry.ex holds sixteen names and
StatifierPersistence.Telemetry.events/0 returns them.

Refs: sp-74k
@johnnyt
johnnyt force-pushed the sp-74k-telemetry-count-index-row branch from c7728e3 to 8000d3e Compare September 6, 2026 23:01
@johnnyt

johnnyt commented Sep 6, 2026

Copy link
Copy Markdown
Member Author

Direction review, pass 1: VERDICT UNQUALIFIED, zero findings, zero NEW-SURFACE. Sixteen verified against @events in lib/statifier_persistence/telemetry.ex at the PR head; Note pattern precedent cited at docs/adr/0008-durable-subchart-child-runs.md:627; zero removed lines in the record file, the single removed line under docs/adr/ being the README index row authorized by sp-74k's dated SCHEDULE note. Three non-qualifying tidy-up notes recorded on the bead. Rebased onto 2ef58e2 without conflict; CI green on the rebased head.

@johnnyt
johnnyt merged commit 6d8e71f into main Sep 6, 2026
1 check passed
@johnnyt
johnnyt deleted the sp-74k-telemetry-count-index-row branch September 6, 2026 23:03
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