Refactor/gatsby node modularization #7622
Refactor/gatsby node modularization #7622Manishnemade12 wants to merge 11 commits intolayer5io:masterfrom
Conversation
Signed-off-by: Manishnemade12 <mnemade140@gmail.com>
Signed-off-by: Manishnemade12 <mnemade140@gmail.com>
Signed-off-by: Manishnemade12 <mnemade140@gmail.com>
Signed-off-by: Manishnemade12 <mnemade140@gmail.com>
5cc1776 to
536c075
Compare
|
@rishiraj38 can you please review this pr |
|
@rishiraj38 actually i forgot to checkout master during creating current branch , but i reverted that part . |
|
Preview deployment for PR #7622 removed. This PR preview was automatically pruned because we keep only the 6 most recently updated previews on GitHub Pages to stay within deployment size limits. If needed, push a new commit to this PR to generate a fresh preview. |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR modularizes parts of gatsby-node.js by extracting page-creation logic for Blog, Kanvas Labs, and Sistent components into dedicated handler modules, aiming to reduce merge conflicts and improve maintainability.
Changes:
- Added three new
src/node-api/*handlers to create Blog, Kanvas Lab, and Sistent component pages. - Updated
gatsby-node.jsto orchestrate these handlers via dependency injection. - Removed the extracted GraphQL sections and page-creation loops from the monolithic
createPagesimplementation.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
gatsby-node.js |
Calls newly extracted handlers and removes inlined GraphQL/page-creation logic. |
src/node-api/createBlogPages.js |
Encapsulates blog post + category/tag list page creation. |
src/node-api/createKanvasLabPages.js |
Encapsulates Kanvas lab page creation. |
src/node-api/createSistentComponentPages.js |
Encapsulates Sistent component (tabbed) page creation via grouping. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Manish Nemade <166635137+Manishnemade12@users.noreply.github.com>
Signed-off-by: Manishnemade12 <mnemade140@gmail.com>
375ac83 to
5a572b3
Compare
|
@Manishnemade12 Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂 |


Description
refactor/gatsby-node-modularization-issue-5 --title "[gatsby-node] Modularize createPages handlers for blog, Kanvas labs, and Sistent" --body "Description
Closes : #7615
This PR modularizes targeted parts of
gatsby-node.jsto reduce merge conflicts and improve maintainability.What changed
src/node-api/createBlogPages.jsfor blog post, category, and tag page creation.src/node-api/createKanvasLabPages.jsfor Kanvas lab page creation.src/node-api/createSistentComponentPages.jsfor Sistent component page creation.gatsby-node.jsto orchestrate these handlers through dependency injection (graphql,createPage,reporter, templates/helpers).createPagesexport.Scope control
Validation
npx eslint gatsby-node.js src/node-api/createBlogPages.js src/node-api/createKanvasLabPages.js src/node-api/createSistentComponentPages.jsCannot find module 'globals'fromeslint.config.js.Notes for Reviewers
gatsby-node.jsnow acts as a cleaner orchestrator for these concerns.Signed commits