Skip to content

chore(agents): sync the shared review-rules block - #130

Open
bymax-agents-sync[bot] wants to merge 1 commit into
mainfrom
chore/agents-sync
Open

bymax-agents-sync[bot] wants to merge 1 commit into
mainfrom
chore/agents-sync

Conversation

@bymax-agents-sync

Copy link
Copy Markdown
Contributor

The shared block in AGENTS.md is behind bymaxone/.github@v1.

Only the content between the shared:begin and shared:end markers is rewritten. Rules this repository wrote for itself live outside the block and are untouched.

Edit the canonical copy in bymaxone/.github rather than this file — a local edit inside the block is reverted on the next run.

That includes an edit nobody made on purpose. A repository whose commit hooks format Markdown — prettier --write through lint-staged, for example — can reformat the block on commit, and the next run reverts the churn silently in a file nobody watches. The canonical copy is kept stable against prettier's defaults, so this bites where a repository's own formatter configuration differs from them. If yours does, check the block against the canonical copy after any commit that touches this file.

Codex reads this file up to project_doc_max_bytes, 32 KiB by default, and truncates the rest silently — no error, no failed check. The block is spliced above this repository's own rules, so what a repository loses when it goes over is its own material, from the end of the file backwards. Keep the rules that invert a reviewer's default near the top: without one of those a reviewer does not merely miss something, it files the finding the rule exists to prevent.

Opened by the agents-sync reusable workflow.

Taken from bymaxone/.github@v1, agents/code-review-rules.md.
Only the block between the shared markers is rewritten; this repository's
own rules are outside it and untouched.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant