Skip to content

fix(kiloclaw): update dev-start.sh paths for monorepo restructure#2040

Merged
jeanduplessis merged 1 commit intomainfrom
catrielmuller/fix-kiloclaw-dev-start
Apr 6, 2026
Merged

fix(kiloclaw): update dev-start.sh paths for monorepo restructure#2040
jeanduplessis merged 1 commit intomainfrom
catrielmuller/fix-kiloclaw-dev-start

Conversation

@catrielmuller
Copy link
Copy Markdown
Contributor

Summary

  • Fixes dev-start.sh paths that broke after the monorepo restructure (apps/services/packages layout in refactor: restructure repo into apps/services/packages monorepo #1315).
  • MONOREPO_ROOT was calculated one level too shallow (services/kiloclaw/.. instead of ../..).
  • .env.local, docker-compose.yml, and the Next.js dev command now correctly reference their new locations under apps/web/.

Verification

  • Reviewed all path references in dev-start.sh against the new monorepo directory structure
  • Confirmed the only changed file is services/kiloclaw/scripts/dev-start.sh

Visual Changes

N/A

Reviewer Notes

  • All changes are path corrections — no logic changes.
  • The vercel env pull now pulls into both apps/web/.env.local (for the web app) and the monorepo root (for shared usage), matching the restructured layout.
  • Pre-push hook has pre-existing failures in apps/mobile/ and apps/web/ TypeScript files unrelated to this shell script change.

After the monorepo restructure into apps/services/packages, the
dev-start script still referenced old root-level paths. This fixes:
- MONOREPO_ROOT calculation (services/kiloclaw is now two levels deep)
- .env.local pulled into apps/web/.env.local
- docker-compose path updated to apps/web/dev/docker-compose.yml
- Next.js dev command runs from apps/web/
- POSTGRES_REPLICA_EU_URL commenting targets correct .env.local
@catrielmuller catrielmuller self-assigned this Apr 5, 2026
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot bot commented Apr 5, 2026

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 2
SUGGESTION 0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

N/A

Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File Line Issue
services/kiloclaw/scripts/dev-start.sh 714 STRIPE_CMD still runs pnpm run stripe from the repo root, but the stripe script now exists in apps/web/package.json, so the webhook listener will fail to start after the monorepo move.
services/kiloclaw/scripts/dev-start.sh 755 The Stripe webhook secret is still written to "$MONOREPO_ROOT/.env.development.local", while the app now runs from apps/web and its local env file lives at apps/web/.env.development.local, so the webhook route will not see the captured secret.
Files Reviewed (1 files)
  • services/kiloclaw/scripts/dev-start.sh - 2 issues

Reviewed by gpt-5.4-20260305 · 654,926 tokens

@jeanduplessis jeanduplessis merged commit b1398fb into main Apr 6, 2026
15 checks passed
@jeanduplessis jeanduplessis deleted the catrielmuller/fix-kiloclaw-dev-start branch April 6, 2026 05:08
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