Skip to content

docs: add CLAUDE.md with codebase guide for AI assistants#9

Open
jaakla wants to merge 2 commits into
mainfrom
claude/claude-md-docs-YW4lK
Open

docs: add CLAUDE.md with codebase guide for AI assistants#9
jaakla wants to merge 2 commits into
mainfrom
claude/claude-md-docs-YW4lK

Conversation

@jaakla

@jaakla jaakla commented Jun 7, 2026

Copy link
Copy Markdown
Owner

Covers uv workspace commands, SDK architecture (HMAC signing, namespace
pattern, business-error handling), MCP preview/confirm write flow,
registry extension pattern, and sales invoice field gotchas.

https://claude.ai/code/session_016MYUtehiW9q7BzBsUsqXzM

claude added 2 commits June 2, 2026 04:07
Covers uv workspace commands, SDK architecture (HMAC signing, namespace
pattern, business-error handling), MCP preview/confirm write flow,
registry extension pattern, and sales invoice field gotchas.

https://claude.ai/code/session_016MYUtehiW9q7BzBsUsqXzM
AGENTS.md holds the general codebase guide readable by any AI coding
agent (Codex CLI, Gemini CLI, etc.). CLAUDE.md keeps its required
header and pulls in AGENTS.md via the @-import syntax so Claude Code
gets the same content without duplication.

https://claude.ai/code/session_016MYUtehiW9q7BzBsUsqXzM

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5a9d74f229

ℹ️ 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".

Comment thread AGENTS.md
uv sync --all-extras

# Run SDK tests
uv run --package merit-api pytest

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Point pytest at the SDK tests

When this command is run from the repository root in a fresh workspace, uv run --package merit-api selects the package environment but does not change the current directory (the uv run --help flag for that is --directory), so bare pytest discovers mcp/tests as well. Because the merit-api package environment does not include MCP-only dependencies like fastmcp, following the documented SDK test command can fail before exercising the SDK tests; use pytest merit_api/tests or run with --directory merit_api instead.

Useful? React with 👍 / 👎.

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