Skip to content

Proposed AGENTS.md rule changes split from #3891 (for owner review) - #3892

Closed
BigSimmo wants to merge 1 commit into
mainfrom
docs/agents-md-document-handoff-rules
Closed

BigSimmo wants to merge 1 commit into
mainfrom
docs/agents-md-document-handoff-rules

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Nov 3, 2026 •

Copy link
Copy Markdown
Owner

Owner review required — do not merge, mark ready, or arm auto-merge without Josh's explicit approval.

This draft holds only the AGENTS.md changes from #3891, split out so they can be reviewed separately. The diff is copied exactly from #3891. Nothing else is included, and #3891 itself is unchanged.

Note: the new rule 2 text refers to docs/agents/shared-context-and-memory.md for claims, reconciliation and exclusions. The document-draft guidance #3891 adds to that file is not part of this PR.

Rule 2 (replaced)

Before:

Establish scope, branch, ownership and existing changes before editing. Preserve concurrent work and use an isolated checkout when needed.

After:

Check scope, branch, owners and changes; preserve existing work. Prefer task-owned document drafts; copying needs no takeover. Routine exact-file handoffs and isolated takeovers within the authorised project, purpose and audience need no renewed approval. Follow docs/agents/shared-context-and-memory.md for claims, reconciliation and exclusions.

Rule 8 (sentence added before "Every publication names and verifies its repository.")

The current request can supply that authority; ask again only when it is missing or scope, target, exposure, cost or material risk changes.

Rule 11 (clause appended to the save sentence)

…save nothing for duplicates, chatter or no eligible fact; after a long conversation (about 20+ exchanges or a context compaction) the final response may instead save a digest of up to five separate settled facts, each with its own topic key.


Summary by cubic

Splits the AGENTS.md rule changes from #3891 into a standalone draft for owner review; #3891 itself is unchanged.

  • Rule 2 now covers task-owned document drafts, and says routine exact-file handoffs and isolated takeovers within the authorised scope need no renewed approval, pointing to docs/agents/shared-context-and-memory.md for claims, reconciliation and exclusions.
  • Rule 8 adds that the current request can supply publication authority, requiring a fresh ask only when authority is missing or scope, target, exposure, cost or material risk changes.
  • Rule 11 allows the final response of a long conversation (about 20+ exchanges or after a context compaction) to save a digest of up to five separate settled facts, each with its own topic key.

The referenced docs/agents/shared-context-and-memory.md guidance from #3891 is not part of this PR.

Written for commit 65761d1. Summary will update on new commits.

View guided diff

@supabase

supabase Bot commented Nov 3, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

coderabbitai Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

@gitar-bot

gitar-bot Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

BigSimmo commented Nov 6, 2026

Copy link
Copy Markdown
Owner Author

Closing at the owner's request: this PR is redundant. Its AGENTS.md diff against current main (a64d237) is empty, because #3891 already landed the same rule 2, 8 and 11 wording.


Generated by Claude Code

@BigSimmo BigSimmo closed this Nov 6, 2026
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