Skip to content

Conversation

@MaxKless
Copy link
Collaborator

@MaxKless MaxKless commented Dec 15, 2025

this will start setting us up for a future where programmatic tool calling etc are the default.

@nx-cloud
Copy link
Contributor

nx-cloud bot commented Dec 15, 2025

View your CI Pipeline Execution ↗ for commit 9d6a421

Command Status Duration Result
nx affected --targets=lint,test,build,e2e-ci,ty... ❌ Failed 6m 24s View ↗
nx-cloud record -- yarn nx sync:check ✅ Succeeded 4s View ↗
nx-cloud record -- yarn nx run-many -t ktfmtFormat ✅ Succeeded 2m 25s View ↗
nx run-many -t ktfmtFormat ✅ Succeeded 2m 16s View ↗
nx-cloud record -- yarn nx format:check --verbose ✅ Succeeded 1m 18s View ↗

☁️ Nx Cloud last updated this comment at 2025-12-15 15:39:01 UTC

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 is proposing a fix for your failed CI:

These changes fix the test failures caused by migrating to the new registerTool API. We updated the test mocks to include the registerTool method and modified the output schemas to use .passthrough() on validators, allowing the structured output to accept additional properties from Nx plugins while maintaining type safety for core fields.

We verified this fix by re-running nx-mcp-server:test, nx-mcp-e2e:e2e-ci--src/nx-project-details-pagination.test.ts.

Warning

The suggested diff is too large to display here, but you can view it on Nx Cloud ↗


✅ The fix was applied locally; push changes to this branch.

View interactive diff ↗


🎓 Learn more about Self-Healing CI on nx.dev

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