refactor(playbooks): rehost model-adaptation chapters to plugin level (ADR-0007)

[kyle-sexton]

Summary

Implements the adopted RA-3b decision (doc-corpus campaign, Sitting 4): rehosts the per-version model-adaptation chapters out of the fable-5 skill's private surface to plugin level.

• Moves plugins/playbooks/skills/fable-5/context/model-adaptation/{opus-4-8,opus-5}.md to plugins/playbooks/reference/model-adaptation/ (100%-similarity renames, content untouched).
• Re-points five fable-5/SKILL.md references at the new host via ${CLAUDE_PLUGIN_ROOT}/reference/model-adaptation/ — the full argument's EXCEPT clause is rewritten semantically (the old exclusion had nothing left to exclude once the directory left context/), preserving the load-only-the-selected-chapter fence.
• Adds ADR-0007, superseding ADR-0006 on the seam's address and nothing else; ADR-0006's decision (model-scoped by default, promotion gate, version-not-family routing) is preserved verbatim, and two of its three live private-surface cites survive untouched by design.
• Bumps playbooks 0.6.4 → 0.6.5 with CHANGELOG entry.

Gate 1 (${CLAUDE_PLUGIN_ROOT} interpolation in SKILL bodies) was probed empirically on Claude Code 2.1.220 and independently reproduced by the verifier; the four-part verification record (Claim/Basis/As-of/Recheck) is in ADR-0007 and the CHANGELOG.

Second commit applies the independent verifier's two wording notes (CHANGELOG reference-count taxonomy; ADR-0007 cite-site enumeration).

No linked issue

Related

• ADR-0006 (docs/adr/0006-scope-model-doctrine-per-version-behind-a-promotion-gate.md) — superseded on the hosting seam only
• feat(knowledge): land the harness-surface definition and bare-name boundary in the docs profile #1888 — prior doc-corpus campaign PR touching the knowledge profile (same campaign, adjacent lane)

🤖 Generated with Claude Code

https://claude.ai/code/session_011UarawwEnZQu7cB6i7WatJ

[claude]

Claude finished @kyle-sexton's task in 3m 14s —— View job

---

Review: PR #1891 — rehost model-adaptation chapters (ADR-0007)

• Gather context (diff, CLAUDE.md, REVIEW.md)
• Review renamed reference files
• Review SKILL.md path re-pointing
• Review ADR-0007 and ADR-0006 supersession
• Review CHANGELOG entry and version bump
• Post findings

Summary: This is a clean, well-verified refactor. I checked every factual claim the PR makes against the actual repo state and found no discrepancies.

Verified correct:

• The two renamed files (opus-4-8.md, opus-5.md) are 100%-similarity git renames with zero content diff; the old skills/fable-5/context/model-adaptation/ directory is fully gone, and the skill's other twelve context/ chapters are untouched.
• All five ${CLAUDE_PLUGIN_ROOT}/reference/model-adaptation/ re-points in fable-5/SKILL.md land correctly (meta-rule 3, the full argument clause, the routing-table preamble, the routing-table row, and the "not model-version documentation" fence) — no stale context/model-adaptation references remain anywhere in the file.
• The full argument's semantic rewrite is sound: the old EXCEPT context/model-adaptation/ exclusion had nothing left to exclude once the directory left context/; the new wording ("only the adaptation chapter meta-rule 3 selects, never the directory as a whole") preserves the original fence against loading conflicting sibling-version chapters in one session.
• ADR-0007's private-surface characterization checks out against audit-encapsulation's actual contract: the "any subdirectory except scripts/" rule (context/public-surface-contract.md:49) and the "Public surface matrix" (SKILL.md:20-27, enumerating exactly Skill/Rule file/Scheduled-automation prompt with no plugin-level-directory row) both match the ADR's claims verbatim.
• The reference-count taxonomy (ADR-0006 carries three live private-surface cites at :41-42, :48, :56-58; this PR cures only :41-42) matches the actual line content of ADR-0006 exactly — this is the wording the second commit fixed, and it's now accurate.
• The CHANGELOG's relative-path fallback citation (plugins/autonomy/skills/setup/templates/isolation-probe.md:6) does contain a genuine relative-path link, supporting the claim as a valid in-repo precedent.
• plugin.json version bump (0.6.4 → 0.6.5) is consistent with this plugin's changelog-driven versioning discipline; marketplace.json doesn't pin a per-plugin version, so no companion update is needed there.
• Per CLAUDE.md's fresh-docs mandate scope note, this change is a SKILL.md prose-body edit (not frontmatter/contract surface), so the mandate doesn't strictly apply — but the PR goes further anyway, empirically verifying the undocumented ${CLAUDE_PLUGIN_ROOT} body-interpolation behavior and recording a proper four-part Claim/Basis/As-of/Recheck verification block in ADR-0007, which is good practice for a claim upstream doesn't document.

Nitpick (non-blocking): plugins/playbooks/CHANGELOG.md line 33 has a mid-paragraph hard line-break wider than the ~85-char wrap used by the surrounding lines in that bullet. MD013 (line length) is disabled repo-wide, so this isn't a lint failure — purely cosmetic, not worth a follow-up commit on its own.

No correctness, architecture, or maintainability issues found. Nothing here needs a security-lane review — no code execution, no new egress, no config secrets.

[claude]

Claude finished @kyle-sexton's task in 53s —— View job

---

Security review complete

• Gather context (PR diff, changed files)
• Review changed files for security issues
• Post findings

Scope reviewed: all 7 changed files (gh pr diff / git diff origin/main...HEAD) — two ADRs (0006 status edit, new 0007), plugins/playbooks/CHANGELOG.md, plugins/playbooks/.claude-plugin/plugin.json (version bump only), plugins/playbooks/skills/fable-5/SKILL.md, and two 100%-similarity file renames (opus-4-8.md, opus-5.md, content byte-identical per diff stat — 0 insertions/deletions).

No security issues found.

Rationale:

• This is a pure documentation/config rehost: markdown content (ADRs, CHANGELOG) and a semver bump in plugin.json. No executable code, no scripts, no .mcp.json, and no .github/workflows/* files are touched.
• The only behavioral surface is SKILL.md's prose body, where five path references change from context/model-adaptation/ to ${CLAUDE_PLUGIN_ROOT}/reference/model-adaptation/. ${CLAUDE_PLUGIN_ROOT} is a harness-substituted path token resolved to the installed plugin's own root — it can't be influenced by external/untrusted input in this diff, so it isn't an injection or path-traversal vector. It only changes where the model is told to read a static, repo-authored file from, not how any command is constructed or executed.
• SKILL.md's frontmatter (allowed-tools, user-invocable, disable-model-invocation) is unchanged — no permission widening.
• No secrets, tokens, or credentials appear in the diff (the two renamed files carry zero content changes, confirmed by the 0/-0 diff stat).
• No GitHub Actions triggers (pull_request_target, workflow_run), action pins, or permissions: blocks are touched — this lane's remit (zizmor covers that surface statically) doesn't apply since no workflow file changed.

[github-actions]

Claude has reviewed this PR 1 time. The lane skips further automatic reviews after 5; deleting this comment resets the count.