-
Notifications
You must be signed in to change notification settings - Fork 2.6k
fix(misc): add explanatory footer to ai agents prompts #33182
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Deploy Preview for nx-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
View your CI Pipeline Execution ↗ for commit 6decf1d
☁️ Nx Cloud last updated this comment at |
| })), | ||
| footer: () => | ||
| chalk.dim( | ||
| "Multiple selections possible. If you don't want any agents, just hit enter." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about Select multiple or press Enter to skip.? Shorter and says the same thing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nx Cloud has identified a possible root cause for your failed CI:
The test failure occurred during an e2e test that generates a React Module Federation host application using webpack. The failure happened when pnpm attempted to install dependencies and could not find the is-negative-zero package in the local npm registry at http://localhost:4873.
The pull request introduced changes to:
- AI agent prompt configuration to add footer text for better user guidance
- Changelog renderer logic to properly handle conventional commits configuration when using version plans
- Package dependencies including adding
nx-mcp@^0.1.0
The error indicates that when the generator ran pnpm install --no-frozen-lockfile, it attempted to fetch is-negative-zero (a transitive dependency through eslint-plugin-import → array-includes → es-abstract) from the local registry, but the package was not available (404 Not Found).
This is classified as an environment_state issue because:
- The local npm registry used for e2e testing was not properly populated with all necessary packages
- The missing package (
is-negative-zero) is not directly related to any of the changes in the PR - The package is a deep transitive dependency that should be available in the test environment's registry
- The code changes do not affect package resolution or registry configuration
The e2e test infrastructure relies on a local npm registry that mirrors required packages for isolated testing. When this registry is missing packages, tests fail despite the code being correct. The registry needs to be properly seeded with all packages before tests run.
A code change would likely not resolve this issue, so no action was taken.
🎓 To learn more about Self Healing CI, please visit nx.dev
Current Behavior
it can be confusing to have the multi select in a long list of just single selects
Expected Behavior
We try to help people by adding an explanatory footer