Repository navigation
chore(specs): tighten the agnostic-ai specs - #1426
Draft
Chemaclass wants to merge 1 commit into
Draft
Chemaclass wants to merge 1 commit into
Chemaclass wants to merge 1 commit into
Conversation
Claude startup drops from 709 to 492 words, a src/ edit loads 1,050 words of rules instead of 1,973, and the Codex root AGENTS.md drops from 4,173 to 2,505 bytes. - Gate runs make test, make test/strict, make sa, make lint, as CI does - Rename the issue skills to bashunit-gh-issue and bashunit-gh-issues - Move the module table to a fork-census reference and fix stale rows - Move return slots out of the tests/** scope into their own rule - One home each for the gate, commit types and the shfmt warning - Fix the pr labels and the fix-test fixtures path - Ask before editing acceptance snapshots - Add lint budgets; drop the session-wide model pin - Hook falls back to agnostic-ai on PATH and exits 0 without it Refs #1396
This branch has not been deployed
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.
Background
Related #1396
The agnostic-ai specs had drifted from CI and the repo, loaded a stale module table on every
src/edit, and the issue skills were shadowed by global skills of the same name. This PR does the parts that work on the pinned agnostic-ai 0.87.0. Three items still need work, listed below.Changes
make test,make test/strict,make sa,make lint, matching CI's strict job. The gate, commit types and theshfmt -wwarning each live in one spec file.bashunit-gh-issueandbashunit-gh-issues;doctor --scope projectreports no findings.skills/fork-census/references/module-map.md, with stale and missing rows fixed (reports/, Cobertura, awk rules, diff coverage, sandbox, suites, benchmark files). Return slots moved torules/return-slots.md, so test edits no longer load them.prlabels andfix-testfixture path now exist. Snapshot edits ask first. Lint budgets added. Session-widemodel: balancedpin removed; per-agent tiers stay. The after-edit hook falls back toagnostic-aion PATH and exits 0 without it.Sizes from
agnostic-ai explain --context, before and after:src/edit, extra rulesAGENTS.mdsrc/chainValidation:
npm run agents:checkpasses,lint --strictis clean, anddoctor --scope projecthas no findings. The hook exits 0 in a clone withoutnode_modulesand withoutagnostic-aion PATH.make saandmake lintpass. I stoppedmake test/strictbefore it finished; this PR does not touchsrc/ortests/, so CI's run decides.Still missing
min-release-age=5blocks 0.89.0, published Oct 27, until Nov 1.requiresstays0.87.0.builtins: [memory]adds a hook with the eventTaskStart, andvalidaterejects that event. This needs the upgrade first. The three facts (Two wrong claims in the rules files: subshellRANDOMinheritance, and theextdebugside effect on Bash 5.2+ #1354, Investigate intermittent cancellation regression failure in parallel CI #1385, Count parallel results from test files under hidden directories #1393) are drafted and ready.AGENTS.mdpast 3 KB. The checklist stays in thecode-revieweragent.denyonCLAUDE.md,AGENTS.md,.claude/rules/**and.codex/**triggers LINT022, because sync writes those files, and that failslint --strict. Only the snapshotaskrule is in this PR.Checklist
CHANGELOG.mdto reflect the new feature or fix