Repository navigation
Remove tracked agent instruction files - #1252
Merged
Merged
Conversation
Agent skills and rules now live in the untracked tstack set. Delete the .agents/ and .cursor/skills/ trees, docs/agents/, rust/scripts/AGENTS.md and the autopilot pr-state test, and stop .gitignore from re-allowing .cursor/skills/. CI still runs three of the old skill helpers, so they move instead of going: the Rust gateway verify.py, test_launch.py and their fixture move to scripts/verify-rust-gateway/, and the web control-print-partner helper and its test move to scripts/verify-print-partner/. Only their path references change. Workflows, the native path filter and its test, and web/package.json point at the new paths. Dead links in README.md, docs/README.md, docs/development/building.md and one code comment are dropped. Co-Authored-By: Claude Opus 5.5 <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
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.
Agent skills and rules now come from the local tstack set, which isn't tracked. This PR removes the 31 tracked agent-instruction files, plus
scripts/autopilot-pr-state.test.mjs, which only tested the removed autopilot helper.CI depended on some of the removed files
Three CI proofs ran helpers inside the skill directories:
desktop-runtime.ymlranverify.pyend to end and thetest_launch.pyForceKillTestagainst the builtpp-server.desktop-native-macos.ymlran the sameForceKillTest.test:runtimerancontrol-print-partner.test.mjs.Deleting them would have dropped those checks. Instead the helpers move out of the agent paths, and no check is removed or skipped:
.cursor/skills/verify-print-partner-rust/helpers/{verify.py,test_launch.py,test_verify_repo_root.py,health-runtime-shape.json}move toscripts/verify-rust-gateway/with no content change.verify.pyfinds the repo root withgit rev-parse..cursor/skills/verify-print-partner/helpers/{control-print-partner.mjs,control-print-partner.test.mjs,compose.verify.yml}move toscripts/verify-print-partner/. Only the repo-root and compose paths change.nativeprefix inscripts/ci-path-filter.pyand its test, andweb/package.jsonpoint at the new paths. Job selection stays the same: a helper-only change still selectsnativeand notrust. Web CI's old.cursorfilter lines are removed becausescripts/**already covers the new path.Other fallout
.gitignoreignores all of.cursor/again, and no longer re-allows.cursor/skills/.AGENTS.mdstays ignored.README.md,docs/README.md(the Agent workflows section) anddocs/development/building.md. A comment inplansList.tsthat pointed atCONTEXT.mdis reworded.CHANGELOG.md,docs/audits/,docs/triage/,cursor-plugin/,docs/playbooks/andrust/rules/are unchanged.No Linear ticket covers this chore.
Evidence
git diff --stat -M origin/main...HEAD: 24 deletions from the list plus 7 moves out of it (31 files), andscripts/autopilot-pr-state.test.mjsdeleted.No references to the removed paths remain in CI or scripts:
Local runs at e06955b:
CI on e06955b: 23 pass, 13 skipping. All 5 required checks pass: actionlint, zizmor, rust (linux), rust (windows), Lint Evidence block. The checks that run the moved helpers pass too:
rust-verify(verify.py end to end),runtime(test_launch ForceKillTest) andweb(control-print-partner test).🤖 Generated with Claude Code