Repository navigation
docs: drop stamps from pointer records (playbooks) - #15461
Conversation
Pointer records in the playbooks plugin (skills, reference, model-adaptation chapters) no longer carry an as-of date or recheck trigger, per the upstream-drift contract 5.0.0. Stored values (model behaviors, benchmarks, probe results, observations, version-pinned claims) keep their stamps; recorded decisions with a named trigger keep the trigger without a date. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VcupojQwC9uJAfmSt4EHZm
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
claude-security-review reviewed Security review (incremental): PR #15461, docs: drop stamps from pointer records (playbooks)I found no security issues in this push. This lane already reviewed Scope. I read every hunk of all 3 files listed in
None of them matches a What changed. The push restores the version-probe stamps that the PR description lists under "After version-probe restore":
In Instruction-surface lens. These files are a skill body and skill references. Nothing in this push deletes, narrows or softens a rule. Every line it adds is a date, a version or a recheck condition, and none of them allows, forbids or gates an action. I didn't consult the protected-class register because no rule was removed. Other surfaces. This push touches no executable code, manifest ( Prior findings and settled threads. This lane's only earlier post is the summary comment from the last review, which reported 0 findings, so nothing carries forward. This push doesn't touch the settled thread on Embedded text. The PR context contains no instructions aimed at the review lane. critical-findings: 0 |
|
claude-review reviewed Code review: PR #15461, incremental review (0b084b9 → 5884049)Scope: I read every hunk of all 3 files in Result: no blocking findings. This increment puts back the version-probe stamps that the PR description lists under "After version-probe restore":
I checked the list structure at the head commit around each edit, and the bullets still render correctly. The description also lists two restored observations in Settled thread: the owner answered the thread on Closing issue: the PR closes none (it is batch 3 of the Q11 sweep), so there were no issue criteria to check. I found no embedded instructions in the PR context, the settled threads or the prior findings. Manifests: this increment changes no Verification: the markdownlint, check-skill, sync-copies, changelog and lychee results in the PR description are the author's claims. I did not run them. |
Owner decision D26 narrows a stored value to a recorded number or string (default, limit, version gate, price, error string, measured figure, hash, count, pinned SHA). Described model behavior, system-card findings without a figure, "no docs page covers X" notes and chapter-wide triggers with no value under them lose their stamp; restated prose is kept and listed in the PR body for a later cut. The model-adaptation AGENTS.md page-conflict convention no longer asks for a date. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VcupojQwC9uJAfmSt4EHZm
Probe and read records that carry the Claude Code version they were measured on are stored values under D26, so their as-of date and recheck trigger come back: three precompute-context probe records (2.1.296, 2.1.296, 2.1.316), two rewrite-a-plugin observations (2.1.289), the prompt-audit source read at 2.1.292, and the repo-sweep isolation record read through 2.1.296. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VcupojQwC9uJAfmSt4EHZm
No related issue: batch 3 of the Q11 sweep, which applies the pointer-without-stamps policy from #15449 and owner decision D26; the sweep has no tracking issue.
Summary
This PR applies upstream-drift contract 5.0.0 ("Required parts"), as narrowed by owner decision D26, to
plugins/playbooks/: the skills,reference/and the model-adaptation chapters. CHANGELOG.md andvendor/copies are not touched.How each kind of record was handled:
6fd9479calibration.md(first_party, one non-null id beside seven null)ultracode,/fastandcost-optimizestrings~/.claude/keybindings.jsonpath--no-buildand--no-install-localmanaged-agents-onboardcommand stringAGENTS.md: the page-conflict convention no longer asks for an as-of date.Fix
Counts were taken with a script matching
**As of**,As of:,Recheck triggerand inlineas of YYYY-MM-DD:Seven probe or read records that name the Claude Code version they were measured on keep their stamps, which were restored after the D26 pass:
precompute-context.md: three probe records (2.1.296, 2.1.296 and 2.1.316)rewrite-a-plugin.md: two observations (2.1.289)skill-criteria.md: the prompt-audit source (read at 2.1.292)repo-sweep/SKILL.md: the isolation record (read through 2.1.296)Restated behavior to cut
The prose below is kept and its stamp removed. A later change should cut it.
skills/boris/reference/orchestration.md~L53, ~L60, ~L68, ~L75: paraphrase the blog's failure modes, the dynamic-versus-static argument, the six patterns and the use cases.skills/boris/reference/autonomy.md~L354: paraphrases how/goalchecks its condition.skills/boris/reference/loops.md:17,:30: paraphrase the goal docs (no turn cap; a bare/goalshows spend).skills/boris/reference/foundations.md~L193: paraphrases whenPostToolBatchfires.skills/boris/reference/customization.md~L18: paraphrases which terminals take Shift+Enter.skills/boris/reference/adoption.md:10: paraphrases the thread's guardrails.skills/boris/SKILL.md:82: restates the Ultracode toggle on/effort.skills/skill-authoring/reference/precompute-context.md~L60-61: paraphrases the permission-checks section (a refused command aborts the skill load).skills/skill-authoring/reference/skill-criteria.md:13-15: paraphrases the prompt audit's cruft criterion.reference/model-adaptation/opus-4-8.md:148,sonnet-5.md:158: paraphrase the Fable guidance on over-prescription.reference/model-adaptation/sonnet-5.md:51-52: paraphrases the effort page saying the scale is set per model.skills/fable-5/context/verification.md(the Managed Agents automation record): restates the blog's re-check and delivery checks. It keeps its stamp for the command string.skills/fable-5/context/context-economy.md:22: restates thekeep:"all"behavior.skills/repo-sweep/scripts/state.sh:39,tick.sh:22: describe jq.exe text-mode stdout on Windows.Kept, unsure
skills/skill-authoring/SKILL.md:34: the post's 9 category names and tip titles are strings copied from the post. Whether a whole-skill restatement counts as a stored value is unsettled.opus-4-8.md~L150-152haiku-5-5.md:244,:246sonnet-5-5.md~L286, ~L289sonnet-5.md:183-186opus-5.md:185Kept, no anchor checked
skill-criteria.md:387prompt-caching.md:43haiku-5-5.md:208sonnet-5-5.md:264Out of scope
lib/prerequisites.{sh,mjs}andscripts/fetch-docs.share shared copies of the rootlib/canonicals.docs/contracts/upstream-drift/README.md:152-156still asks a "no docs page covers" record to keep its date. That text predates D26, which overrides it, so the contract needs its own follow-up.Verification
markdownlint-cli2on the changed markdown: 0 issues.check-skill.shonplugins/playbooks/skills: 29 WARN before and after both passes; 5 passed, 0 failed.sync-shared-copies.sh --check: all 802 copies match.check-changelog-fragments.sh --check: valid.lychee --offlineoverplugins/playbooks/{reference,skills}: 0 errors.fable-5/context/orchestration.md:86(asking to restore a "no docs page covers" date): answered with D26 and resolved.Related
🤖 Generated with Claude Code
https://claude.ai/code/session_01VcupojQwC9uJAfmSt4EHZm