feat: add navigation visibility option and fix workspace-wide lint errors#55
feat: add navigation visibility option and fix workspace-wide lint errors#55sarthak03dot wants to merge 2 commits intoYourGPT:mainfrom
Conversation
|
@sarthak03dot is attempting to deploy a commit to the Delta4 Infotech Team on Vercel. A member of the Team first needs to authorize it. |
|
Hey @sarthak03dot, thanks for the contribution! 🙏 We've reviewed the PR and here's our plan: ✅ ESLint Changes - We'll AcceptThe 🔄 Navigation Feature - Different ApproachWe love the idea of hiding tool messages (valid use case from #54), but we're planning a different implementation that fits better with the SDK architecture: Instead of a global useTools({
navigate: {
hidden: true, // Won't show in UI
description: 'Navigate to a page',
handler: async ({ path }) => { ... }
}
});This gives developers explicit control over which tools to hide, without false positives on tools like 📝 Changeset FixSmall note: Our plan: We'll handle the If you'd prefer to implement the Thanks again for contributing 🚀 |
|
Thank you for the detailed review, @Sahil5963! 🙌 This PR is complete from my side. I'm glad the ESLint fixes were helpful across the workspace. |
Summary
This PR introduces a new feature to control the visibility of navigation-related tool messages and resolves several ESLint configuration issues across the workspace to ensure compatibility with React 19/Next.js and ESLint 9.
Changes
showNavigationMessagesprop toCopilotProviderand chat components.react-compiler/react-compiler,react-hooks/set-state-in-effect, and others causing issues in the current state).anytypes andjsx-a11yerrors in demo packages.🦋 major/minorfor the feature and🦋 patchfor the lint fixes).Verification
pnpm buildacross the workspace.pnpm lintpasses for all 13 workspace tasks (13 successful).Resolves #54