Turn adapter-less deferred modules into non-user-invocable skills + cut self-referential contract rhetoric#457
Open
clkao wants to merge 1 commit into
Conversation
…ble skills + trim self-referential rhetoric Turn the adapter-less deferred FO modules (fo-status-viewer, fo-write-core) into non-user-invocable skills whose SKILL.md description carries the when-to-load, and cut the self-referential contract rhetoric the lean-contract deferrals accreted. Part A — skill conversion: - Add skills/fo-status-viewer/SKILL.md + skills/fo-write-core/SKILL.md (user-invocable:false, description carries the load-point, section bodies verbatim), mirroring present-gate / feedback-rejection-flow. - Delete references/fo-status-viewer.md + references/fo-write-core.md. - Collapse the four-entry `## Deferred Modules (registry)` into a terse `## Deferred load points` block: two Skill(...) invocations + two references/*.md, one shared greet-guard, no arrow-taxonomy. - Re-point the three cross-refs off the deleted paths (feedback-rejection-flow, claude runtime adapter, migrated status-viewer body). Part B — rhetoric trim: cut the `(host-neutral)` header labels, the self-describing "Lazily loaded ... never reads it" headers, and the `(named by the boot-resident core)` parentheticals across the shared core, the Claude runtime adapter, and the dispatch/merge references. Test wiring: - journeymetrics: ClaudeTurn.SkillNames extracted parallel to ReadTargets (per-block + later-delta merge + fresh-turn init). - ensigncycle: retire the Read-based greet oracle for assertGreetInvokesNoDeferredFOSkill (keys on the skill argument; present-gate allowed pre-greet), with a dedicated later-delta RED control and an augmented GREEN fixture carrying a pre-greet Skill(present-gate). - contractlint: rewrite both registry-coupled tests for the new shape (deferredSkillCores + spacedock:<name> skill-anchor check; `## Deferred load points` fold), add the unguarded-re-point grep gate. - docs/runtime-support.md: state the refined reference-vs-skill discriminator (host binding location, not adapter-loaded-alongside). AC-1 net byte delta over the affected set vs origin/main: -1924 B. go build ./... and go test ./... green. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Slim the first-officer contract: convert two adapter-less deferred references into non-user-invocable skills and cut self-referential rhetoric, netting a leaner boot core.
What changed
fo-status-viewer+fo-write-corereferences intouser-invocable:falseskills carrying their when-to-load in frontmatter.## Deferred load pointsblock.ClaudeTurn.SkillNamesparallel toReadTargetsand re-point the AC-2 greet oracle to skill invocations.Evidence
contractlint+ensigncycle+journeymetricsgreen; fullgo test ./...green.origin/main(independently re-confirmed byte-for-byte).nt