ai_ml_operations/chat_web_app
2
0
Fork 0
Code Issues 16 Pull Requests Actions Packages Projects Releases Wiki Activity

Fix brace-expansion lock for npm 10 beta deploy #70

Merged
westfarn merged 1 commits from fix/brace-expansion-npm10-ci into master 2026-07-30 16:57:17 -07:00
Conversation 0 Commits 1 Files Changed 2
+23 -37
westfarn commented 2026-07-30 11:04:31 -07:00
Owner
Copy Link
Copy Source

Summary

  • Beta deploy after #69 failed: Cannot find module 'brace-expansion' during npm run build:beta
  • Root cause: package-lock.json linked brace-expansion to empty phantom paths (minimatch/vendor/brace-expansion-compat). npm 11 remapped them locally; deploy Node 22 / npm 10 followed the lock literally
  • Declare vendored shim as a direct dependency and rewrite lock links to vendor/brace-expansion-compat

Test plan

  • npm ci under Node 22.23.1 / npm 10.9.8 resolves brace-expansion from fork-ts-checker nested minimatch
  • npx react-scripts build with .env.beta succeeds
  • npm audit still 4 moderate (0 critical / 0 high)
  • Redeploy chat_web_app beta after merge
## Summary - Beta deploy after #69 failed: `Cannot find module 'brace-expansion'` during `npm run build:beta` - Root cause: `package-lock.json` linked `brace-expansion` to empty phantom paths (`minimatch/vendor/brace-expansion-compat`). npm 11 remapped them locally; deploy Node 22 / npm 10 followed the lock literally - Declare vendored shim as a direct dependency and rewrite lock links to `vendor/brace-expansion-compat` ## Test plan - [x] `npm ci` under Node 22.23.1 / npm 10.9.8 resolves `brace-expansion` from fork-ts-checker nested minimatch - [x] `npx react-scripts build` with `.env.beta` succeeds - [x] `npm audit` still 4 moderate (0 critical / 0 high) - [ ] Redeploy `chat_web_app` beta after merge
westfarn added 1 commit 2026-07-30 11:04:31 -07:00
Fix brace-expansion lock so npm 10 can build on beta deploy
Unit Tests / test (pull_request) Successful in 10s
Details
587fe0a858 
Deploy hosts use Node 22 / npm 10, which followed broken package-lock links to empty phantom paths and failed with Cannot find module brace-expansion. Point overrides at the vendored shim and declare it as a direct dependency so npm ci resolves correctly.
westfarn merged commit 553d526862 into master 2026-07-30 16:57:17 -07:00
westfarn deleted branch fix/brace-expansion-npm10-ci 2026-07-30 16:57:18 -07:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
twestfall westfarn (Ryan Westfall)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ai_ml_operations/chat_web_app#70