Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
3 Skipped Deployments
|
31305de to
5ba2856
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR adds an AGENTS.md file to document the BigCommerce Catalyst codebase architecture for AI-assisted development tools. The documentation explains the custom middleware-based routing system that overrides Next.js defaults, data fetching patterns using Partial Prerendering (PPR), and the separated UI design system.
- Added comprehensive documentation for the custom routing middleware that resolves URLs via GraphQL instead of file-based routing
- Documented data fetching patterns including the Streamable utility and PPR configuration
- Outlined the UI design system architecture with clear separation between business logic and presentation components
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
72ce4b9 to
bc9e3f0
Compare
matthewvolk
left a comment
There was a problem hiding this comment.
I'm cool with merging so we can all collaborate and add to it, IMO it's already very valuable so far
|
|
||
| 1. **Use React's `cache()` function** for server-side data fetching to ensure deduplication. | ||
|
|
||
| 2. **Implement proper cache strategies** based on whether user authentication is present. |
There was a problem hiding this comment.
🍹 Do examples help here?
Co-Authored-By: Copilot <[email protected]>
Co-authored-by: Matthew Volk <[email protected]>
Co-authored-by: Matthew Volk <[email protected]>
e7a6fc4 to
0f00e77
Compare
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
* Create AGENTS.md Co-Authored-By: Copilot <[email protected]> * Apply suggestion from @matthewvolk Co-authored-by: Matthew Volk <[email protected]> * Apply suggestion from @matthewvolk Co-authored-by: Matthew Volk <[email protected]> * Update AGENTS.md * Apply suggestions from code review Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Matthew Volk <[email protected]> Co-authored-by: Chancellor Clark <[email protected]>
* Create AGENTS.md Co-Authored-By: Copilot <[email protected]> * Apply suggestion from @matthewvolk Co-authored-by: Matthew Volk <[email protected]> * Apply suggestion from @matthewvolk Co-authored-by: Matthew Volk <[email protected]> * Update AGENTS.md * Apply suggestions from code review Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Matthew Volk <[email protected]> Co-authored-by: Chancellor Clark <[email protected]>
What/Why?
Add an AGENTS.md file to help AI-assisted development tools in understand the structure of Catalyst as well as the best practices for extending it.
Testing
TBD - need to test various development scenarios
Migration
N/A