Skip to content

Add an AGENTS.md - #414

Open
bjester wants to merge 1 commit into
learningequality:release-v0.9.xfrom
bjester:agents-md
Open

bjester wants to merge 1 commit into
learningequality:release-v0.9.xfrom
bjester:agents-md

Conversation

@bjester

@bjester bjester commented Nov 2, 2026

Copy link
Copy Markdown
Member

Summary

I used Claude to craft an AGENTS.md file. I've edited to remove overly specific guidance, and other guidance that doesn't make sense. It feels like a good starting point.

AI usage

Claude generated it (a couple weeks ago), I modified it, then recently modified it further because it had guidance to avoid some pitfalls that had existed in the codebase which I subsequently removed.

@bjester
bjester requested review from a team, leq-bot and rtibblesbot November 2, 2026 22:51
@rtibblesbot

rtibblesbot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

🟡 Waiting for changes

Outcome: reviewed 9099aa5

Last updated: 2026-11-03 16:09 UTC

@rtibblesbot rtibblesbot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PR #414: two claims in AGENTS.md describe code that no longer exists. The rest check out against the branch.

CI passing.

  • suggestion: the private-import example was removed (line 7)
  • suggestion: the circular-import note has the direction reversed (line 52)
  • nitpick: Buffer also yields on a partition change (line 50)

Next step: This is a low-risk change; it needs a light-touch code review by a human engineer.


Written by rtibblesbot, an LLM-based coding agent.

Comment thread AGENTS.md Resolved
Comment thread AGENTS.md Resolved
Comment thread AGENTS.md Resolved

@rtibblesbot rtibblesbot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PR #414: all 3 prior findings are resolved. No new findings. CI is passing.

Prior-finding status

RESOLVED — AGENTS.md:7 — removed _deserialize_from_store example
RESOLVED — AGENTS.md:52 — reversed import direction
RESOLVED — AGENTS.md:50 — Buffer partition trigger missing

Next step: This is a low-risk change and can merge as is, with no further human review.


Written by rtibblesbot, an LLM-based coding agent.

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.

2 participants