Skip to content

docs: correct stocktake tally and normalize impeccable license string - #20

Merged
kuker24 merged 1 commit into
mainfrom
docs/stocktake-tally-fix
Sep 14, 2026
Merged

kuker24 merged 1 commit into
mainfrom
docs/stocktake-tally-fix

Conversation

@kuker24

@kuker24 kuker24 commented Sep 14, 2026

Copy link
Copy Markdown
Owner

Docs consistency only. No behavior change, no new skills, no version bump.

1. Stocktake tally

adhd was recorded as KEEP in the verdict table even though its body was
edited in the same pass to stop naming Claude Code and GrokBuild as if
either were this runtime. The row's evidence cell already described that edit,
so the table contradicted itself, and the CHANGELOG described four host-name
bodies while the tally showed three UPDATE.

adhd moves to UPDATE (applied) with a writing-for-agents handoff.
Recounted programmatically from the table rows rather than by hand:

Verdict Before After
KEEP 57 56
COMPRESS (applied) 2 2
UPDATE (applied) 3 4
MERGE 0 0
RETIRE 0 0
Total 62 62

The CHANGELOG Unreleased tally string is updated to match. Its GrokBuild
bullet already said "four skill bodies" and needed no edit, which is what
made the mismatch visible.

2. impeccable license string

Frontmatter read Apache 2.0; vendor/license-audit.json records
Apache-2.0. tests/test_skills.py normalizes the form, so this was
cosmetic.

One extra edit beyond the literal request: the audit entry's evidence string
was "SKILL.md frontmatter (Apache 2.0)". That parenthetical existed only to
flag the unhyphenated frontmatter. Normalizing the frontmatter alone would
have left the citation quoting a string no longer present in the file, so both
move together and the entry now matches its siblings' plain
"SKILL.md frontmatter". The recorded license and redistribution values are
untouched; no test asserts the evidence text.

Follow-up 3 in docs/stocktake-1.8.3.md is struck through as resolved.
Follow-ups 1, 2, and 4 stay open.

Verification

  • python3 -m unittest tests.test_skills — 10 tests, OK
  • python3 -m unittest tests.test_license_audit — 2 tests, OK
  • python3 -m unittest discover -s tests — 315 tests, OK (skipped=2)
  • git diff --check — clean
  • json.load on vendor/license-audit.json — parses, entry verified
  • Repo-wide grep for stale 57 KEEP / 3 UPDATE / Apache 2.0 — no hits
    outside a third-party citation in full-performance-audit/references/sources.md

Not touched

VERSION stays 1.8.2, no 1.8.3 heading, no tag, no release. No skill body
changed apart from that one license string. GrokBuild image_gen tool-family
names and the A/B baseline are untouched.

The stocktake table recorded adhd as KEEP even though its body was
edited in the same pass to drop Claude Code and GrokBuild as runtime
names. The evidence cell already described the edit, so the row and the
tally disagreed with each other and with the CHANGELOG, which counted
four host-name bodies while the tally showed three UPDATE. adhd moves to
UPDATE (applied) with a writing-for-agents handoff. Recount from the
table rows: 56 KEEP, 2 COMPRESS, 4 UPDATE, 0 MERGE, 0 RETIRE, total 62.

impeccable frontmatter declared "Apache 2.0" while
vendor/license-audit.json records "Apache-2.0". The test normalizes the
form, so this was cosmetic, but the audit entry carried an evidence
string of "SKILL.md frontmatter (Apache 2.0)" whose parenthetical only
existed to flag the mismatch. Normalizing the frontmatter without it
would have left the citation quoting a string no longer in the file, so
both move together and the entry now matches its siblings.

Closes follow-up 3 in docs/stocktake-1.8.3.md. Follow-ups 1, 2, and 4
stay open: the A/B baseline, the image_gen tool-family naming decision,
and the 28 intentionally license-free frontmatters.

No skill bodies changed apart from that one license string. VERSION
stays 1.8.2, no 1.8.3 heading, no tag.
@kuker24
kuker24 merged commit 8d4c29d into main Sep 14, 2026
10 checks passed
@kuker24
kuker24 deleted the docs/stocktake-tally-fix branch September 14, 2026 11:28
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