docs: add AGENTS.md and CLAUDE.md#36
Conversation
chrip
left a comment
chrip
left a comment
There was a problem hiding this comment.
It just needs a few minor improvements.
| - **Packaging:** Uses `pkg`. Adding native modules (`.node`) requires updating `pkg` config. | ||
|
|
||
| ## Rules | ||
| - **Never** modify `SpellChecker/` (pre-compiled binary). |
There was a problem hiding this comment.
It isn't a pre-compiled binary. JS sources are in SpellChecker/sources.
chrip
left a comment
chrip
left a comment
There was a problem hiding this comment.
The AGENTS.md is a useful concept and mostly correct. Two factual errors need fixing:
-
SpellChecker is source code, not a pre-compiled binary.
SpellChecker/sources/server.js,spellCheck.js,languages.jsare all Node.js source files. The rule should not call it a "pre-compiled binary." -
Issue #51 does not exist.
GET /issues/51returns 404. Either create the issue first, or remove the reference.
The rest of the guidance (async patterns, storage abstraction, schema backends, pkg config) is correct and verified against the codebase.
|
Sorry for the duplicate. I posted the first two comments manually. Then, somewhere along the line, a sub-agent finished up and decided on its own to post the same review again. At least we seem to agree :) |
AGENTS.md is the canonical guidance file for AI agents working in this repository — covering architecture constraints, conventions, and rules. CLAUDE.md contains only `@AGENTS.md`, making Claude Code load this file automatically alongside other AI tools. Assisted-by: ClaudeCode:claude-sonnet-4-6 Signed-off-by: Alex Hernández <ahmartin@arsys.es>
Alex-Arsys
force-pushed
the
docs/add-claude-md
branch
from
28a85b2 to
7a4fbcb
Compare
chrip
left a comment
chrip
left a comment
There was a problem hiding this comment.
Both issues from my last pass are fixed. Looks good.
2 participants
AGENTS.md is the canonical guidance file for AI agents working in this repository — covering architecture constraints, conventions, and rules. CLAUDE.md contains only
@AGENTS.md, making Claude Code load this file automatically alongside other AI tools.Assisted-by: ClaudeCode:claude-sonnet-4-6