Conversation
| - `"private": true` | ||
| - `"packageManager": "pnpm@10.27.0"` | ||
| - `"engines": { "node": "^22" }` | ||
| - `"scripts"`: only workspace-wide scripts: |
There was a problem hiding this comment.
WARNING: Root workflow commands are missing from the new script list
Today the repo still relies on root format:check, dependency-cycle-check, drizzle, and test:e2e commands (ci.yml, deploy-production.yml, and chromatic.yml all call them). If the root package is reduced to only the scripts listed here, the first restructure PR will leave those workflows red unless the plan also preserves root aliases or explicitly rewrites each caller.
| - `"preinstall": "npx only-allow pnpm"` | ||
| - `"typecheck": "pnpm -r typecheck"` (or delegate to turbo) | ||
| - `"build": "turbo run build"` | ||
| - `"test": "turbo run test"` |
There was a problem hiding this comment.
WARNING: turbo run test changes CI semantics in a risky way
The current top-level test job only runs the web Jest suite; several worker packages are tested in separate jobs because they need extra setup (for example cloud-agent and cloud-agent-next build their wrappers first, and Playwright runs in chromatic.yml). Replacing the root script with turbo run test would make the generic CI job execute every workspace test script on every PR, which duplicates work and can break packages that are not prepared in that job.
Code Review SummaryStatus: 2 Issues Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Other Observations (not in diff)N/A Files Reviewed (1 files)
Reviewed by gpt-5.4-20260305 · 1,207,881 tokens |
Summary
Adds a detailed plan for restructuring the monorepo from the current flat layout (where the root serves as both pnpm workspace root and the Next.js app) into a standard
apps/+services/+packages/structure.Key decisions captured in the plan:
apps/web/apps/storybook/services/, stripping thecloudflare-prefix from directory nameskiloclaw-secret-catalogtopackages/(shared across web app and kiloclaw)package.jsoninto lean workspace root +apps/web/package.jsonThe plan covers 12 implementation phases, CI/CD workflow updates, config file migrations, risk mitigations, and post-merge manual steps (Vercel dashboard).
No code changes — this is a plan document for team review before implementation.
Verification
Visual Changes
N/A
Reviewer Notes
Please review
plans/monorepo-restructure.mdand comment on any decisions you disagree with or risks you see. Key areas to weigh in on:apps/vsservices/split for deployablescloudflare-prefix from service directory names