Update CLAUDE.md with build/test commands and doc clarifications - #9
Update CLAUDE.md with build/test commands and doc clarifications#9thetigerone888 wants to merge 1 commit into
Conversation
- Add build:web, server build/start, and qr commands to the workflow list - Document how to run a single test file or test by name with Vitest - Note that root README.md duplicates server/README.md - Note that .github/workflows are stock GitHub samples, not project CI Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WE4dzSSfapDKjt5xACz9pd
Reviewer's guide (collapsed on small PRs)Reviewer's GuideDocumentation-only PR updating CLAUDE.md with clarified project guidance, additional build/test commands, and notes about repo structure and CI workflows to better support AI/code-assistant usage and human readers. Flow diagram for updated development workflow commands in CLAUDE.mdflowchart TD
A[Start development] --> B[pnpm dev]
A --> C[pnpm check]
A --> D[pnpm lint]
A --> E[pnpm format]
A --> F[pnpm test]
F --> G[pnpm test tests/analytics.test.ts]
F --> H[pnpm vitest run -t calculateNPS]
A --> I[pnpm build:web]
A --> J[pnpm build && pnpm start]
A --> K[pnpm qr exps_url]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Code Review
This pull request updates CLAUDE.md to clarify guidance for Claude Code, document additional files in the directory structure, and add CLI command examples for testing, building, and running the application. The review feedback suggests improving consistency in the test commands by leveraging the existing pnpm test script instead of calling vitest directly.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| pnpm test # vitest run | ||
| pnpm test # vitest run (all tests) | ||
| pnpm test tests/analytics.test.ts # run a single test file | ||
| pnpm vitest run -t "calculateNPS" # run tests matching a name |
There was a problem hiding this comment.
Instead of invoking vitest run directly via pnpm vitest run, you can use the existing pnpm test script which is mapped to vitest run. This keeps the commands consistent and leverages the defined package scripts.
| pnpm vitest run -t "calculateNPS" # run tests matching a name | |
| pnpm test -t "calculateNPS" # run tests matching a name |
Summary
Reviewed the existing
CLAUDE.mdagainst the current state of the repository. Its core content (architecture notes, directory map, conventions) was verified accurate, so this PR makes only targeted additions:pnpm build:web(static web export used by the Vercel deploy),pnpm build && pnpm start(server production bundle), andpnpm qrto the development workflow list.pnpm test tests/analytics.test.ts) or tests by name (pnpm vitest run -t "calculateNPS"). Verified the file-level command passes (21/21 tests).README.mdis a duplicate ofserver/README.md(the backend guide), not a project overview, so future readers don't get misled..github/workflows/contains stock GitHub-suggested workflows (CodeQL, stale bot, Jekyll Pages, npm-publish samples) not tailored to this pnpm/Expo project.Verification
server/routers.tshas only system/auth routers;drizzle/schema.tshas onlyusers).DEPLOY.md/metro.config.jsafter the recentforceWriteFileSystemchange.pnpm test tests/analytics.test.ts— all 21 tests pass.🤖 Generated with Claude Code
https://claude.ai/code/session_01WE4dzSSfapDKjt5xACz9pd
Generated by Claude Code
Summary by Sourcery
Update CLAUDE.md with additional project guidance and development workflow details.
Documentation: