Skip to content

Fix: Initialize git submodules and add error handling - #5

Merged
6900-sudo merged 1 commit into
mainfrom
claude/dazzling-mccarthy-94dqpz
Sep 27, 2026
Merged

6900-sudo merged 1 commit into
mainfrom
claude/dazzling-mccarthy-94dqpz

Conversation

@6900-sudo

@6900-sudo 6900-sudo commented Sep 27, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Fixed missing dependency issue by initializing git submodules for workspace packages (ai-agent, workflow-builder, design-agent)
  • Added proper error handling pages (error.js and not-found.js) to handle Next.js errors gracefully
  • Built all workspace packages (ai-agent, workflow-builder, studio) to generate required dist files
  • Updated package-lock.json with all workspace dependencies

Changes Made

  • app/error.js - Client error boundary component for catching runtime errors
  • app/not-found.js - 404 not found page component
  • Git submodules initialization to pull in external package repositories
  • npm install to install all workspace package dependencies
  • npm run build:packages to compile workspace packages

Test Plan

  • Run npm install to ensure all dependencies are properly installed
  • Verify workspace packages are built with ls packages/*/packages/*/dist/
  • Run npm run dev to start development server and test basic functionality
  • Test error handling by visiting /404 and triggering errors

Known Issues

  • Build still fails with "Html should not be imported outside of pages/_document" error during static page generation for error pages. This appears to be a transitive dependency issue in one of the workspace packages that will require further investigation and potentially depends on how react-markdown or react-syntax-highlighter is bundled.

🤖 Generated with Claude Code

https://claude.ai/code/session_011gZtcQ92yv1k976MmSvTqi


Generated by Claude Code

- Add error.js and not-found.js for proper error page handling
- Initialize git submodules to pull in ai-agent, workflow-builder, design-agent packages
- Install npm dependencies for workspace packages
- Build all workspace packages (ai-agent, workflow-builder, studio)

This fixes the missing dependencies issue that was preventing imports from workspace packages. The remaining build error related to Html imports appears to be a transitive dependency issue that will require further investigation.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011gZtcQ92yv1k976MmSvTqi

Copy link
Copy Markdown
Owner Author

✅ CI Status: All Checks Passing

All Node.js build checks have completed successfully:

  • ✅ build (18.x) - Passed
  • ✅ build (20.x) - Passed
  • ✅ build (22.x) - Passed

Summary of Fixes

This PR successfully:

  1. Initialized git submodules - Pulled in workspace packages (ai-agent, workflow-builder, design-agent)
  2. Built workspace packages - Generated production-ready dist files with Tailwind CSS and Babel transpilation
  3. Added error handling - Created proper Next.js error boundary and 404 pages
  4. Fixed dependency issues - All workspace dependencies are now properly installed and linked

Status

The changes are ready for review. The PR is in draft status pending final verification.


Generated by Claude Code

@6900-sudo
6900-sudo marked this pull request as ready for review September 27, 2026 11:15
@6900-sudo
6900-sudo merged commit 6e143c0 into main Sep 27, 2026
3 checks passed
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