Skip to content

Comments

fix: update CLAUDE.md with accurate codebase info and soften pnpm enforcement#213

Merged
rubenmarcus merged 3 commits intomainfrom
fix/claude-md-update-207
Feb 23, 2026
Merged

fix: update CLAUDE.md with accurate codebase info and soften pnpm enforcement#213
rubenmarcus merged 3 commits intomainfrom
fix/claude-md-update-207

Conversation

@rubenmarcus
Copy link
Member

Summary

  • Softens the aggressive "ALWAYS use pnpm, NEVER use npm or yarn" instruction that caused AI agents to refuse npm/yarn in user projects
  • Fixes outdated agents list: 4 implemented (Claude Code, Cursor, Codex CLI, OpenCode), removed 4 that don't exist
  • Fixes executor.ts line count: 1400+ (was "900+")
  • Adds Todoist to integrations, MCP Server to key concepts
  • Fixes repo structure: removes nonexistent projects/, adds content/ and specs/
  • Expands Important Files table with 9 missing entries
  • Adds full src/ directory overview (16 directories)
  • Adds missing quick commands (format, check, typecheck, test:run)
  • Updates current priorities to match active work

Closes #207

Test plan

  • Verify CLAUDE.md content matches actual codebase structure
  • Run pnpm build to ensure no accidental breakage

🤖 Generated with Claude Code

…orcement

Resolves #207. The aggressive "ALWAYS use pnpm, NEVER use npm or yarn"
instruction caused AI agents to refuse npm/yarn in user projects. Softened
to clarify pnpm is for repo development only. Also fixed outdated info:
agents list (4 not 8), executor line count (1400+ not 900+), added Todoist
integration, fixed repo structure, expanded important files table, added
src/ directory overview, and updated current priorities.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added bug Something isn't working documentation labels Feb 21, 2026
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

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: c453a9df31

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

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 21, 2026

Greptile Summary

Updates CLAUDE.md documentation to accurately reflect the current codebase state and softens the overly strict pnpm requirement that was causing AI agents to incorrectly refuse npm/yarn in user projects.

Key improvements:

  • Softens "ALWAYS use pnpm, NEVER use npm or yarn" to clarify pnpm is only for ralph-starter development, not user projects
  • Fixes agent list from 8 claimed to 4 actually implemented (Claude Code, Cursor, Codex CLI, OpenCode)
  • Corrects executor.ts line count from "900+" to "1400+" (actual: 1398 lines)
  • Adds Todoist to integrations list (OAuth provider and source exist)
  • Expands Important Files table from 5 to 13 entries
  • Adds comprehensive src/ directory overview (16 directories)
  • Adds missing quick commands (format, check, typecheck, test:run)
  • Updates current priorities to match active development work

Issues found:

  • AGENTS.md still lists 8 agents while CLAUDE.md correctly shows 4 - consider updating for consistency
  • content/ directory listed in repo structure doesn't exist

Confidence Score: 4/5

  • Safe to merge with minor documentation inconsistencies that should be addressed
  • This is a documentation-only change that significantly improves accuracy and fixes a real problem (overly strict pnpm enforcement). The changes are well-researched and mostly accurate. Score is 4 instead of 5 due to: (1) content/ directory doesn't exist but is listed in repo structure, and (2) AGENTS.md inconsistency where it still lists 8 agents while CLAUDE.md correctly shows 4 implemented agents.
  • CLAUDE.md needs minor corrections (remove nonexistent content/ directory reference), and AGENTS.md should be updated for consistency

Important Files Changed

Filename Overview
CLAUDE.md Documentation update that softens pnpm enforcement, fixes outdated agent list (4 actually implemented vs 8 claimed), corrects executor.ts line count (1400+ vs 900+), adds Todoist integration, removes nonexistent projects/ directory, and updates repo structure and priorities to match actual codebase

Last reviewed commit: c453a9d

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 3 comments

Edit Code Review Agent Settings | Greptile

rubenmarcus and others added 2 commits February 23, 2026 12:22
Address PR review comments:
- Remove content/ from repo structure (directory doesn't exist)
- List all 8 supported agents to match AGENTS.md

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rubenmarcus rubenmarcus merged commit 2043eee into main Feb 23, 2026
8 checks passed
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

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: af1581945c

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


| Directory | Purpose |
|-----------|---------|
| `auth/` | OAuth providers (Anthropic, OpenAI, Linear, Notion, Todoist) |

Choose a reason for hiding this comment

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

P2 Badge Correct OAuth provider list in auth directory summary

The new auth/ row claims OAuth providers for Anthropic and OpenAI, but the implemented provider registry only includes Linear, Notion, and Todoist (src/auth/providers/index.ts). Because CLAUDE.md is used as execution guidance for agents, this mismatch can direct implementation/debugging work toward non-existent auth modules and lead to incorrect changes.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: ALWAYS use pnpm, NEVER use npm or yarn.

1 participant