Conversation
Move detailed agent guidance out of AGENTS.md into concise policy, playbook, and spec docs. Add a local CLI testing playbook and replace the old Cursor rule files with repo-native policies. Co-Authored-By: OpenAI Codex <codex@openai.com>
|
need to make changes here as it wiped a lot from AGENTS.md and im not sure it retained the things that we'd 100% want to keep, at minimum |
Restore compact project context and high-risk operational guidance that was lost when the root agent instructions were thinned down. Keep the policy, playbook, spec, and file lookup indexes intact while avoiding the old long-form lore dump. Co-Authored-By: OpenAI Codex <codex@openai.com>
Remove the duplicated policy and playbook index from the root agent instructions. Keep the read-first table focused on files agents should actually consult for each kind of change. Co-Authored-By: OpenAI Codex <codex@openai.com>
Convert command, reference, location, and long-term concern sections away from tables. Group related guidance to keep the root instructions under the target length while preserving the major operational warnings. Co-Authored-By: OpenAI Codex <codex@openai.com>
Keep AGENTS.md as a compact router and move edge-case notes into a task-scoped policy. Add Grit rules for command wrapper invariants that previously lived only in agent guidance. Co-Authored-By: OpenAI Codex <codex@openai.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 495be6a. Configure here.
| "@stricli/core": { | ||
| "importNames": ["buildCommand", "buildRouteMap"], | ||
| "message": "Import buildCommand from '../lib/command.js' and buildRouteMap from '../lib/route-map.js' instead. The wrappers inject telemetry and standard subcommand aliases." | ||
| "message": "Command files must use the repo command wrappers from lib/command.js, lib/list-command.js, lib/mutate-command.js, and lib/route-map.js instead of importing @stricli/core directly." |
There was a problem hiding this comment.
Restricted imports rule now blocks all @stricli/core usage
Low Severity
The noRestrictedImports override for src/commands/**/*.ts removed the importNames: ["buildCommand", "buildRouteMap"] field. Without importNames, Biome blocks all imports from @stricli/core in command files, including any future legitimate type imports or other non-wrapper utilities. The global rule at the top of the file still keeps importNames scoped, so this override silently broadens the restriction beyond what the global rule enforces.
Reviewed by Cursor Bugbot for commit 495be6a. Configure here.
Keep AGENTS.md readable in raw markdown by using bullets for commands, references, and file paths. Co-Authored-By: OpenAI Codex <codex@openai.com>
Make AGENTS.md section names match the referenced content and keep the smoke-test workflow under playbooks. Co-Authored-By: OpenAI Codex <codex@openai.com>
BYK
left a comment
BYK
left a comment
There was a problem hiding this comment.
I want to merge this but I think it is losing a lot of fidelity and adding too much boilerplate for questionable wins.
I'm fine making the AGENTS.md file smaller by splitting into parts. It's just this particular PR doesn't cut it for me due to the reasons I mentioned above.
| ## Intent | ||
| Commands should follow `gh`-style ergonomics while preserving consistent telemetry, JSON output, and error handling. | ||
|
|
||
| ## Command Wrappers |
There was a problem hiding this comment.
Seems like this missed the crucial buildListCommand helper
| # CLI Command Design | ||
|
|
||
| ## Intent | ||
| Commands should follow `gh`-style ergonomics while preserving consistent telemetry, JSON output, and error handling. |
There was a problem hiding this comment.
We should probably mention curl-style ergonomics too, especially for sentry api command
| AI commits MUST include: | ||
|
|
||
| ```text | ||
| Co-Authored-By: OpenAI Codex <codex@openai.com> |
2 participants
Move agent guidance out of the oversized AGENTS.md into repo-native policy, playbook, and spec docs. AGENTS.md now acts as a task index, while the new policy files carry durable defaults for comments, runtime APIs, command design, output, pagination, testing, and generated artifacts.
Local CLI Playbook
Add a local CLI testing playbook with dev, env-file, built-binary, output, and targeted-test commands for agent smoke checks.
Cursor Rule Migration
Remove
.cursor/rulesdocs after preserving their repo-specific guidance inpolicies/runtime-and-deps.mdand other focused policies.Validated with
bun run lint; it completed with an existing Biome warning about an unused suppression insrc/lib/formatters/markdown.ts.