-
Notifications
You must be signed in to change notification settings - Fork 931
[docs]: review copilotkit docs #5855
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
[docs]: review copilotkit docs #5855
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
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.
PR Summary
Comprehensive restructuring of CopilotKit integration documentation in docs/src/content/en/docs/frameworks/agentic-uis/copilotkit.mdx
to improve clarity and organization.
- Reorganized frontend setup to appear first, providing a more intuitive implementation flow
- Added separate integration paths for Mastra server and Next.js implementations with clear distinctions
- Enhanced code examples with proper TypeScript type definitions and filename indicators
- Improved typed runtime context examples with clearer type definitions
- Streamlined content by removing redundant sections while maintaining comprehensive coverage
1 file reviewed, 1 comment
Edit PR Review Bot Settings | Greptile
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Braintrust eval reportNo experiments to report |
a6001d5
to
e3bf360
Compare
The only other thing i'm not sure about is the tab titles: I don't think we've really finalised terminology yet but. In other areas of the docs we refer to:
Next.js Remote is a little misleading as it suggests you need to use Next.js, when really, it's just React with Mastra Client, right? --- it could also easily be Astro, with a React client directive on the CopilotKit component. Maybe these tab headings can be: |
be57789
to
7ae23b5
Compare
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.
Superb man! Great job!!
Description
Review the copilotkit docs.
Type of Change
Checklist