Skip to content

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

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

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
Copy Markdown
Contributor

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
Copy Markdown
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

@github-actions
Copy link
Copy Markdown

This Pull Request has been inactive for 30 days and will be closed in 7 days if no further activity occurs. Please update or close this PR if it is no longer relevant.

@github-actions github-actions Bot added the stale label Apr 19, 2026
@github-actions
Copy link
Copy Markdown

This Pull Request has been closed due to inactivity.

@github-actions github-actions Bot closed this Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant