Skip to content

fix(mcp-server): remove yarn prefix from scripts to prevent state file crash#778

Merged
JeremyDev87 merged 1 commit intomasterfrom
fix/lint-crash-765
Mar 21, 2026
Merged

fix(mcp-server): remove yarn prefix from scripts to prevent state file crash#778
JeremyDev87 merged 1 commit intomasterfrom
fix/lint-crash-765

Conversation

@JeremyDev87
Copy link
Owner

Summary

  • Remove redundant yarn prefix from binary invocations (eslint, prettier, tsc, vitest, madge, ts-node) in apps/mcp-server/package.json scripts
  • Inline rm -rf dist/ in build script instead of yarn clean
  • Replace nested yarn <script> calls in validate with direct binary invocations
  • Root cause: nested Yarn calls require .yarn-state.yml which may not exist in worktrees or fresh clones, causing "Couldn't find the node_modules state file" crash

Test plan

  • yarn workspace codingbuddy lint runs successfully
  • yarn workspace codingbuddy typecheck passes
  • yarn workspace codingbuddy format:check passes
  • yarn workspace codingbuddy test:coverage — 4841 tests pass
  • yarn workspace codingbuddy circular — no circular deps
  • yarn workspace codingbuddy build — builds successfully

Closes #765

…e crash

- Remove redundant `yarn` prefix from binary invocations (eslint, prettier,
  tsc, vitest, madge, ts-node) in package.json scripts
- Inline `rm -rf dist/` in build script instead of `yarn clean`
- Replace `yarn <script>` calls in validate with direct binary invocations
- Prevents "Couldn't find the node_modules state file" crash when
  nested Yarn calls can't locate .yarn-state.yml

Closes #765
@JeremyDev87 JeremyDev87 added bug Something isn't working fix mcp-server apps/mcp-server labels Mar 21, 2026
@vercel
Copy link

vercel bot commented Mar 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
codingbuddy-landing Ready Ready Preview, Comment Mar 21, 2026 3:40pm

@JeremyDev87 JeremyDev87 self-assigned this Mar 21, 2026
@JeremyDev87 JeremyDev87 merged commit 670eafa into master Mar 21, 2026
25 checks passed
@JeremyDev87 JeremyDev87 deleted the fix/lint-crash-765 branch March 21, 2026 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working fix mcp-server apps/mcp-server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: Yarn workspace lint crash kills Claude Code process

1 participant