Repository navigation
feat: rebuild the shipped AGENTS.md and preset skills - #24
Merged
Merged
Conversation
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.
Replaces the AGENTS.md content ClaudeX ships, splits it by whether a rule is portable or personal, trims the base skill set, and rebuilds the preset's skills as single-subject skills.
AGENTS.md
internal/embedded/agents/AGENTS.base.mdcarries Prose/Output Principles only, so everyclaudex applygets the portable half.developmentpreset is renamedprivate, and itsAGENTS.partial.mdcarries Development Principles, Pull Request Principles, and Operating Principles.$HOME/shell/extensions/, theuvandfnmenvironment variables) that are true on one machine only.Skills
cross-aiis removed. The base set isskill-creatorandsession-summary.review-codewhere a sub-agent is handed the path directly. Set size drops from ~426,000 to ~292,000 characters, and the floor cost of any Go task halves.go-foundations,go-cli,go-backend,go-frontend,go-concurrency,node-foundations,node-backend, andproject-ci-cdare split by subject.dockerizeis new and language-agnostic, covering multi-stage builds, base image choice, runtime packaging, cache clearing, and a non-root user.develop's Skill Map andreview-code's six domains are re-cut to the new set.go-oauth-cliat ~3,420.Standardizations applied across the set
make -s version, so CI and a laptop produce the same number. Tag-push triggering is documented nowhere.makedownloads them and CI calls the same target.review-codenow states the rule that replaces it: the expected pattern lives in the skill the domain names, and a finding cites the skill section it comes from.Nuances
chrome-extension-basicsis renamedchrome-extension, andgo-dockerfilenever lands because its material went todockerize.go-embedded-frontendmoves to Tailwind v4. The v3 CDN is frozen and the v4 browser build throws on a JS config, so the oldtailwind.configglobal produced no CSS.AGENTS.mdis Prose/Output, then Development, Pull Request, Operating.developmentkeeps a<!-- claudex:preset:development -->block.internal/flavorspackage, the--flavorand--no-flavorlaunch flags, the launch-time picker, and the~/.config/claudex/flavorsdirectoryconfigureused to create. An existing flavors directory is left on disk untouched.skills_gate_test.goandutils/paths_test.goare removed.internal/workspacekeeps the only test file, covering the AGENTS.md marker states and the git exclude anchoring..github/workflows/notifications.yamlis deleted.release.yamlis the only workflow left.🤖 Generated with Claude Code