Skip to content

chore: convert instruction files to AGENTS shape - #642

Merged
EtanHey merged 1 commit into
mainfrom
chore/agents-md-shape-conversion
Aug 3, 2026
Merged

chore: convert instruction files to AGENTS shape#642
EtanHey merged 1 commit into
mainfrom
chore/agents-md-shape-conversion

Conversation

@EtanHey

@EtanHey EtanHey commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep the existing 43-line AGENTS.md body byte-for-byte
  • move all shared BrainLayer architecture, workflows, interfaces, safety rules, paths, and memory guidance from CLAUDE.md into AGENTS.md
  • reduce CLAUDE.md to the one-line @AGENTS.md import because no remaining line is genuinely Claude-Code-specific
  • preserve the IDENTITY header verbatim; no wording edits or symlinks

Before / after line counts

File Before After
AGENTS.md 43 191
CLAUDE.md 149 1

TDD RED

FAIL /Users/etanheyman/Gits/brainlayer/.worktrees/agents-md-conversion — CLAUDE.md first nonblank line must be exactly @AGENTS.md
checker_exit=1

GREEN

PASS /Users/etanheyman/Gits/brainlayer/.worktrees/agents-md-conversion — instruction-file shape conforms
checker_exit=0

Zero-loss accounting

  • New AGENTS.md lines 1–43 are byte-identical to the original AGENTS.md.
  • Line 44 is a new Markdown separator blank line.
  • New AGENTS.md lines 45–191 are byte-identical to the original CLAUDE.md after the two deliberate meaning-level dedupes below.
  • CLAUDE.md is exactly @AGENTS.md plus its trailing newline.
  • Deliberate dedupes:
    • original CLAUDE.md:47 (- Tests: pytest) duplicates existing AGENTS.md:34
    • original CLAUDE.md:124 (canonical DB path) duplicates existing AGENTS.md:16
  • Zero unexplained deletions.
  • git diff --check origin/main..HEAD exits 0.
  • Changed files: only AGENTS.md and CLAUDE.md.

Verification

  • Repository documented BRAINLAYER_PREPUSH_SCOPE=changed-only gate passed:
    • MCP registration: 3 passed
    • isolated eval/hook routing: 40 passed
    • Bun regression: 1 passed
    • shell FTS5 determinism regression: passed
  • A prior full-gate attempt reached ~55% with no assertion failures, then the pytest process hit Errno 24: Too many open files during session cleanup. The shell soft limit was 256 while kern.maxfilesperproc was 138240. The remaining isolated/Bun/shell gates passed. This branch changes no Python, TypeScript, or shell code.

Review

  • Local CodeRabbit: SKIPPED — rate limited (recoverable CLI response; reset wait 19 minutes).
  • Fallback red-team/blue-team self-review found no unexplained loss or Claude-only content left in the shared file.
  • Remote exact-head review is required before merge.

Note

Low Risk
Documentation-only reshuffle of markdown instruction files; no application, test, or runtime code changes.

Overview
Consolidates agent-facing documentation so AGENTS.md is the single source of truth for both generic agent notes and the full BrainLayer architecture/runbook.

The existing 43-line AGENTS.md header is unchanged; the former body of CLAUDE.md (fleet memory purpose, stack, pipeline, MCP/daemon APIs, watcher, chunk lifecycle, paths, bulk DB safety, memory-tool guidance) is appended after a separator. Two intentional dedupes drop a redundant pytest bullet and a duplicate canonical DB path line already in Key Paths.

CLAUDE.md is reduced to @AGENTS.md so Claude Code still loads instructions via import with no Claude-only content left in a separate file.

Reviewed by Cursor Bugbot for commit 410d99c. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move BrainLayer documentation from CLAUDE.md to AGENTS.md

Relocates the BrainLayer (זיכרון) fleet organizational memory documentation from CLAUDE.md to AGENTS.md, replacing the original content in CLAUDE.md with a single @AGENTS.md reference line.

Macroscope summarized 410d99c.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@cursor

cursor Bot commented Aug 3, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_1b76caac-728c-40ee-bbce-d0bc9fb1959e)

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@EtanHey, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 12 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: c93392ce-23ed-4595-91fa-391153909568

📥 Commits

Reviewing files that changed from the base of the PR and between d4acb3a and 410d99c.

📒 Files selected for processing (2)
  • AGENTS.md
  • CLAUDE.md

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@EtanHey

EtanHey commented Aug 3, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@EtanHey

EtanHey commented Aug 3, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. You're on a roll.

Reviewed commit: 410d99c4a9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@EtanHey

EtanHey commented Aug 3, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: 410d99c4a9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@EtanHey
EtanHey merged commit 17c0247 into main Aug 3, 2026
9 checks passed
@EtanHey
EtanHey deleted the chore/agents-md-shape-conversion branch August 3, 2026 13:40
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