Repository navigation
Description
Problem
After #323 ports the pr skill to Go and its activation ships, the frozen pr shell scripts and their generated copies remain as the fallback for older binaries. They still source lib.sh, so #997 can't delete the library. #323 leaves deleting the frozen shell to a later cutover, and no issue tracks it for pr.
Acceptance criteria
- AC1: Given the retirement release recorded in AC3, each
prentry script contains only the probe and exec of the Go command, with one actionable stderr message and one documented non-zero status when the binary or subcommand is absent. No script of the skill sourceslib.sh. - AC2: The frozen fallback bodies and their comparison fixtures are removed from the canonical source,
packages/agents/claude/.apm/skills,packages/agents/codex/.apm/skills, and the deployed.claude/skillsand.agents/skills. Each Bats case that exercised the removed shell maps to a named Go test that runs and passes. - AC3: Before implementation,
docs/compatibility.mdanddocs/agent-skills-migration.mdrecord the release that first shipped the Move the pr, write-pr-description and agent-pr scripts into agentrepo pr #323 wrapper (the warning start), the retirement release (at least two releases later), and the evidence:git tag --contains <#323 merge commit>and the tag list. - AC4: Non-functional: the regenerated skills, token records and lock are committed in the order
mise run generate-skills,mise run skill-tokens,apm install.mise run check,mise run cover-go-checkandmise run cover-py-checkpass, no gate is loosened, and the diff stays under 800 code lines.
Scope and non-goals
In scope: removing the frozen pr shell and its fixtures. Non-goals: changing the Go command, deleting lib.sh (#997), and shortening the window.
Approach
Resolve the retirement release from the AC3 procedure first, and stop if the current release is earlier. List the frozen paths from the merged #323 change, then delete them together and regenerate.
Dependencies
Blocked (native links) by #323, by the activation change for pr, and by the end of the window per AC3. Blocks #997. Native parent: #588. Part of #588.
Size
S.
Verify plan
- AC1: with stand-in binaries, assert pass-through of arguments, stdout, stderr and status for a supporting binary, and the refusal status and stderr message for none and for one lacking the subcommand;
git grep -n 'lib\.sh' -- <skill source>prints nothing. - AC2: take the path list from the merged Move the pr, write-pr-description and agent-pr scripts into agentrepo pr #323 change, assert none exists, and run the mapped Go tests.
- AC3: read both documents and assert the release evidence and that the current release is not earlier than the retirement release.
- AC4: run the named tasks and the generation sequence, then assert exit 0 and no uncommitted change.
Risks
Removing the fallback early breaks consumers on an older binary, so the change waits for the AC3 evidence. Reverting the commit restores the fallback.
Operator authority
None.
Metadata
Metadata
Assignees
Labels
Type
Fields
Priority
Work type
Projects
- StatusShow more project fieldsBacklog
Activity