Skip to content

Conversation

@MaxKless
Copy link
Collaborator

@MaxKless MaxKless commented Oct 22, 2025

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

image

@vercel
Copy link

vercel bot commented Oct 22, 2025

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

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Oct 23, 2025 1:28pm

@netlify
Copy link

netlify bot commented Oct 22, 2025

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 6decf1d
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/68fa2afc0e964500082359d4
😎 Deploy Preview https://deploy-preview-33182--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link
Contributor

nx-cloud bot commented Oct 22, 2025

View your CI Pipeline Execution ↗ for commit 6decf1d

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 45m 43s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 2m 48s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 3s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-10-23 14:08:59 UTC

})),
footer: () =>
chalk.dim(
"Multiple selections possible. If you don't want any agents, just hit enter."
Copy link
Member

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.

Copy link
Contributor

@nx-cloud nx-cloud bot left a 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:

  1. The local npm registry used for e2e testing was not properly populated with all necessary packages
  2. The missing package (is-negative-zero) is not directly related to any of the changes in the PR
  3. The package is a deep transitive dependency that should be available in the test environment's registry
  4. 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.

Nx CloudView in Nx Cloud ↗


🎓 To learn more about Self Healing CI, please visit nx.dev

@MaxKless MaxKless merged commit 44187bd into master Oct 23, 2025
19 checks passed
@MaxKless MaxKless deleted the footer-in-cnxw-ai branch October 23, 2025 16:08
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.

4 participants