docs: retire audit snapshots, reclassify depth-variant convention (#204 follow-ups)#205
Merged
Merged
Conversation
Post-#204 reorg follow-ups. Retire audit snapshots (#1). Delete docs/evals-coverage.md and docs/extensibility-contract-grading.md — point-in-time records whose durable policy already lives in the migration playbook (warrant rule + consumer-verify recipe; four-seams contract). Generating would derive only eval presence (a glob) while class, contract grades, and medley program state stay hand-stamped, so retire beats a half-generated hybrid. The two warranted-but-unowned gaps recorded only in the snapshot are captured first in melodic-software/medley#1537 (context7/setup) and #1538 (planning/wayfind). Inbound pointers in README and the playbook are updated. Reclassify depth-variant convention (#2). The playbook said a depth/intensity variant takes an argument, but discovery's explore-deep/research-deep ship as siblings. They are structural, not depth: explore-deep is a `context: fork` execution topology and research-deep a heavier dispatch tier, both fixed in frontmatter and not expressible as a runtime argument. Amend the convention to classify execution tier as a structural axis and bind the `-deep` suffix to it. README revisit-condition dedup (#3) assessed and closed no-op: the coupling is a documented, directional SSOT+index (CATALOG-TAXONOMY.md owns only pointers; each README owns its condition). Verification: scripts/validate-plugins.sh pass; scripts/run-plugin-tests.sh 7 passed, 0 failed; markdownlint-cli2 0 errors on touched markdown. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0163dXPBdRYFpx4wTtS648tE
kyle-sexton
added a commit
that referenced
this pull request
Jul 16, 2026
Resolves a modify/delete conflict on docs/evals-coverage.md: #205 deliberately retired that file (decision-gated with the maintainer), so the deletion wins over this branch's now-moot one-line correction to it.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The three follow-ups deliberately deferred out of #204 (recorded in its PR body), each decision-gated with the maintainer before editing. Net −341 lines; no plugin manifest, schema, or skill behavior touched.
#1 — Retire the audit snapshots
Delete
docs/evals-coverage.mdanddocs/extensibility-contract-grading.md. They were hand-stamped, point-in-time records that drift the moment a plugin changes (the reorg already forced a two-layer temporal re-stamp).Retire over generate. The durable policy already lives in
docs/MIGRATION-PLAYBOOK.md— the warrant rule + consumer-verify recipe ("Evals — warrant policy and consumer-verify recipe") and the four-seams contract ("Extensibility contract v2.1"). A generator could derive only eval presence (a live glob) — the minority of one doc and none of the other; class, contract grades, and medley program state stay irreducibly hand-graded or externally-owned. Generating only the presence slice yields a half-generated hybrid that still drifts on its biggest surface. Retire is the clean fit.Gaps captured first. Two warranted-but-unowned eval gaps were recorded only in the retired snapshot (verified still live against the tree):
context7/setupandplanning/wayfind. Filed durably before deletion:melodic-software/medley#1537— evals(context7/setup)melodic-software/medley#1538— evals(planning/wayfind)Inbound pointers updated: README doc-index (2 bullets removed) and the playbook's evals section (now states on-demand verification instead of pointing at a deleted snapshot). No dangling references remain (repo-wide grep clean).
#2 — Reclassify the depth-variant convention tension
The playbook stated a depth/intensity variant "takes an argument, never a sibling," yet
discovery'sexplore-deep/research-deepship as sibling skills. Inspection shows they are structural, not depth:explore-deep—context: fork+agent: general-purpose: a distinct execution topology (isolated forked subagent), declared in frontmatter, not expressible as a runtime argument.research-deep— a dispatcher to the heaviest isolated tier (workflow engine, forked subagent, or inline fallback), with its own body and auto-invocation description.The convention needed no exception — it needed a sharper structural/depth test. Amended to classify execution tier as a structural axis (sibling) and bind the
-deepsuffix to that axis, so the naming no longer reads as a violation and a future reader does not re-flag it. A true effort knob on one execution path still takes an argument. No skills renamed.#3 — README revisit-condition dedup
Assessed, closed no-op. The coupling flagged in #204 is a documented, directional SSOT+index:
docs/CATALOG-TAXONOMY.md(L71–73) declares plugin-scoped revisit conditions are owned by each plugin's README and the taxonomy "Trigger register" lists them "only as pointers." The READMEs are the single owners of genuine plugin-specific rationale; the register summarizes them. Nothing to dedup. Confirmed none of the three sections cross-names the taxonomy doc.Verification
scripts/validate-plugins.sh: pass (all manifests + strict catalog sync)scripts/run-plugin-tests.sh: 7 passed, 0 failedmarkdownlint-cli2: 0 errors on both touched markdown filesNote (not in this PR)
docs/hook-migration-audit.mdis the same "point-in-time audit" genus as the two retired here. Left untouched; flagged for a consistency decision in a later pass.🤖 Generated with Claude Code
https://claude.ai/code/session_0163dXPBdRYFpx4wTtS648tE