Skip to content

Conversation

Copy link

Copilot AI commented Nov 7, 2025

The typecheck command failed on Windows with "Invalid option schema: missing at least one property from 'all', 'recursive', 'since', or 'worktree'". Yarn's workspaces foreach requires a base selector when using --include filters.

Changes

  • Added --all flag to typecheck command to satisfy Yarn's requirement
  • typecheck:all already had the flag and continues to work correctly
- "typecheck": "yarn workspaces foreach --parallel --include main --include router-api --include tests run typecheck",
+ "typecheck": "yarn workspaces foreach --parallel --all --include main --include router-api --include tests run typecheck",

The command now tells Yarn to consider all workspaces, then filter to only the specified ones.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@vercel
Copy link

vercel bot commented Nov 7, 2025

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

Project Deployment Preview Comments Updated (UTC)
curve-dapp Ready Ready Preview Comment Nov 7, 2025 3:24pm
curve-dapp-storybook Ready Ready Preview Comment Nov 7, 2025 3:24pm

Co-authored-by: DanielSchiavini <2369243+DanielSchiavini@users.noreply.github.com>
Copy link
Author

Copilot AI commented Nov 7, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • download.cypress.io
  • www-api.ibm.com
    • Triggering command: /usr/local/bin/node /home/REDACTED/work/curve-frontend/curve-frontend/node_modules/@ibm/telemetry-js/dist/REDACTED-process.js --log=/tmp/ibmtelemetry-20251107T151829188Z-385014.log --config=telemetry.yml (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Fix typecheck process for apps and tests fix: add missing --all flag to typecheck command Nov 7, 2025
Copilot AI requested a review from DanielSchiavini November 7, 2025 15:24
Copilot finished work on behalf of DanielSchiavini November 7, 2025 15:24
Copy link
Collaborator

@DanielSchiavini DanielSchiavini left a comment

Choose a reason for hiding this comment

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

Works!

image

@DanielSchiavini DanielSchiavini marked this pull request as ready for review November 10, 2025 08:56
@DanielSchiavini DanielSchiavini merged commit 9494d12 into chore/typecheck Nov 10, 2025
13 checks passed
@DanielSchiavini DanielSchiavini deleted the copilot/sub-pr-1609 branch November 10, 2025 08:57
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