Skip to content

fix(dev-cli): address review comments — env banner, logs, status, log dirs#1296

Open
evan-claw wants to merge 1 commit intoKilo-Org:improvement/kiloclaw-dev-clifrom
evan-claw:improvement/kiloclaw-dev-cli
Open

fix(dev-cli): address review comments — env banner, logs, status, log dirs#1296
evan-claw wants to merge 1 commit intoKilo-Org:improvement/kiloclaw-dev-clifrom
evan-claw:improvement/kiloclaw-dev-cli

Conversation

@evan-claw
Copy link

Fixes all 5 review comments from kilo-code-bot on #1142:

  1. env.tsallGood now tracks .env.local and Vercel project checks, not just the per-service loop. Final banner no longer lies.
  2. logs.tsmigrations (non-docker infra) no longer crashes with docker compose logs. Shows helpful message instead.
  3. status.ts — Services sharing a port (auto-fix/db-proxy on 8792, kiloclaw/git-token on 8795) are grouped with a shared-port note. Port only probed once.
  4. code-review.ts — Added mkdir(logDir, { recursive: true }) before Bun.write() so it works on fresh checkouts.
  5. auto-fix.ts — Same mkdir fix as AppBuilder - Fix model selector not showing saved model on reload #4.

All 15/15 unit tests pass.

Addresses review comments on #1142.

@kilo-code-bot
Copy link
Contributor

kilo-code-bot bot commented Mar 19, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (5 files)
  • dev/cli/src/commands/env.ts
  • dev/cli/src/commands/logs.ts
  • dev/cli/src/commands/status.ts
  • dev/cli/src/projects/auto-fix.ts
  • dev/cli/src/projects/code-review.ts

Reviewed by gpt-5.4-20260305 · 301,977 tokens

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.

1 participant