refactor: migrate from ERB to TurboRepo monorepo structure #3705
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remove legacy Electron React Boilerplate (ERB) configuration files, webpack configs, build scripts, and assets to prepare for monorepo migration to TurboRepo structure. - Remove .erb/ configuration directory and webpack configs - Remove old assets/ directory with icons and images - Remove legacy src/ directory structure - Remove package-lock.json files from old structure 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Transform package.json from ERB single-app configuration to TurboRepo monorepo with pnpm workspace support. - Add TurboRepo and workspace configuration - Update scripts for monorepo development workflow - Configure build and development commands for multiple apps - Set up proper dependency management for workspace packages 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Implement complete monorepo architecture with TurboRepo and pnpm workspaces: - Add apps/ directory with desktop, dashboard, and github-notes applications - Add services/ directory with api and github-service backend services - Add packages/ directory for shared libraries and components - Add gutenberg/ documentation site with Docusaurus - Configure pnpm workspace with workspace.yaml - Add TurboRepo configuration with turbo.json - Include proper dependency management with pnpm-lock.yaml 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Update gitignore patterns to support TurboRepo monorepo with multiple applications and services: - Add TurboRepo cache directory (.turbo/) - Add pnpm and workspace-specific patterns - Add Next.js and Node.js build artifacts - Add Electron build directories and output files - Add IDE and editor configuration patterns - Remove ERB-specific patterns 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Add comprehensive development documentation and AI code editor configuration: - Add CLAUDE.md with project structure and development workflow - Add .claude/ directory with project-specific configurations - Add ai-code-editors-files/ with development rules and guidelines - Include TypeScript, Python, and architecture guidelines - Add command templates for common development tasks 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
There was a problem hiding this comment.
Pull Request Overview
Complete migration from an Electron React Boilerplate to a TurboRepo-based monorepo with pnpm workspaces, reorganizing applications, services, and shared packages.
- Removed legacy ERB boilerplate and app entry point.
- Added
turbo.jsonandpnpm-workspace.yamlto orchestrate builds, dev, lint, test, and clean across workspaces. - Introduced new services (
api,github-service) and shared UI/utility packages underpackages/ui, along with monorepo apps inapps/.
Reviewed Changes
Copilot reviewed 166 out of 244 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| turbo.json | Defines TurboRepo tasks for build, dev, lint, start |
| pnpm-workspace.yaml | Lists workspace packages and onlyBuiltDependencies |
| src/renderer/App.tsx | Removed legacy Electron React Boilerplate UI |
| services/github-service/* | New Hono-based GitHub integration service |
| services/api/* | API client package with shared-logic import |
| packages/ui/* | Shared UI components, hooks, and utilities |
Files not reviewed (1)
- release/app/package-lock.json: Language not supported
Comments suppressed due to low confidence (1)
services/github-service/package.json:4
- [nitpick] Only a
devscript is defined here. For consistency and clarity in production workflows, consider addingbuildandstartscripts to document how to package and launch the service.
"dev": "bun run --hot src/index.ts"
| @@ -0,0 +1,9 @@ | |||
| import { Hono } from 'hono' | |||
There was a problem hiding this comment.
The service defines a Hono app but never starts listening on any port. Add an app.listen(PORT) call or export a handler that your runtime can invoke for requests.
Copilot uses AI. Check for mistakes.
Comment on lines
+13
to
+16
| "dependsOn": ["^build"] | ||
| }, | ||
| "lint:fix": { | ||
| "dependsOn": ["^build"] |
There was a problem hiding this comment.
[nitpick] The lint and lint:fix tasks depend on build, which forces a full build before linting. Consider removing that dependency so linting runs independently and faster.
Suggested change
| "dependsOn": ["^build"] | |
| }, | |
| "lint:fix": { | |
| "dependsOn": ["^build"] | |
| }, | |
| "lint:fix": { |
Copilot uses AI. Check for mistakes.
| @@ -0,0 +1,726 @@ | |||
| "use client" | |||
There was a problem hiding this comment.
[nitpick] This Sidebar component file is over 700 lines long, making it difficult to navigate. Consider splitting it into smaller subcomponents or modules for improved readability and maintainability.
Copilot uses AI. Check for mistakes.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Complete architectural transformation from Electron React Boilerplate (ERB) to a modern TurboRepo monorepo structure with pnpm workspaces.
Architecture Changes
New Structure
Key Features
pnpm devTest Plan
pnpm installpnpm devpnpm buildpnpm desktop:package🤖 Generated with Claude Code