Skip to content

docs: add CONTRIBUTING.md and rewrite AGENTS.md#3

Merged
Ethan-Arrowood merged 5 commits into
mainfrom
feat/agents-doc
Apr 29, 2026
Merged

docs: add CONTRIBUTING.md and rewrite AGENTS.md#3
Ethan-Arrowood merged 5 commits into
mainfrom
feat/agents-doc

Conversation

@Ethan-Arrowood

@Ethan-Arrowood Ethan-Arrowood commented Apr 27, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds CONTRIBUTING.md with code organization, module system conventions (ESM/.ts imports, erasableSyntaxOnly), tsconfig explanation, scripts, dev setup, and release process
  • Rewrites AGENTS.md to focus on three things only: behavioral contracts (what's safe to run autonomously), operational grounding (things agents need stated explicitly), and failure surface (footguns and constraints)
  • AGENTS.md links to README and CONTRIBUTING rather than duplicating content

🤖 Generated with Claude Code

Ethan-Arrowood and others added 2 commits April 27, 2026 16:55
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
CONTRIBUTING.md covers code organization, module system conventions,
TypeScript config, scripts, dev setup, and release process.

AGENTS.md is rewritten to focus on behavioral contracts, operational
grounding, and known failure surfaces rather than describing the project.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Ethan-Arrowood Ethan-Arrowood changed the title docs: add AGENTS.md docs: add AGENTS.md and CONTRIBUTING.md Apr 28, 2026
@Ethan-Arrowood Ethan-Arrowood changed the title docs: add AGENTS.md and CONTRIBUTING.md docs: add CONTRIBUTING.md and rewrite AGENTS.md Apr 28, 2026
Comment thread AGENTS.md
- Do not edit files in `dist/`; it is compiled output and gitignored.

## Code Style
- Use ESM and TypeScript

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm curious, have you seen examples of AI not picking up on the code style? Or noticing the erasableSyntaxOnly in the tsconfig.json?
I have been thinking that the main value of AGENTS.md:

  • Giving information that isn't accessible in the context.
  • Accelerating the comprehension of the code base without having to spend a bunch of tokens reading a lot of source code on every new task.

But definitely curious where we find other values.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I generally agree with that. Here is the definition I've come up with for AGENTS.md:

The AGENTS.md is a minimal, agent-specific document complementary to the README.md and CONTRIBUTING.md. It answers what does an agent need to know that it cannot reliably infer from existing documentation?. Furthermore, it does not replace or duplicate existing documentation.

We can reasonably assume an agent can read the tsconfig and discern facts from it, but that then requires it reading the tsconfig and interpreting those rules every new session. By putting the important stuff in the AGENTS.md, it can skip reading tsconfig unless it has to (to say debug a TS issue).

I think its very arbitrary what we do or don't include in an agents.md file. The important bits is that it should be factual and succinct. I think if we find the agent constantly looking for additional information about the project when its tasked with doing something, we may want to add that information to the agents.md document.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Btw I incorporated some of your thinking in some general guidelines for this doc: HarperFast/code-guidelines#9 want to iterate on it there?

@Ethan-Arrowood Ethan-Arrowood merged commit c776cd0 into main Apr 29, 2026
Ethan-Arrowood added a commit that referenced this pull request Apr 29, 2026
#3

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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