Conversation
…trol-plane - Rename @r_masseater/ops-harbor-core → @repo/ops-harbor-core (private) - Rename @r_masseater/ops-harbor-control-plane → @repo/ops-harbor-control-plane (private) - Move ops-harbor-core from dependencies to devDependencies (bun build inlines it) - Bundle control-plane into dist/control-plane.js via build:server - Add localtunnel to ops-harbor dependencies (external in control-plane bundle) - Add control-plane auto-start logic to production cli.ts Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add localtunnel and @repo/ops-harbor-control-plane to ops-harbor ignoreDependencies (used in build:server script only) - Exclude .agents/** from knip file scanning (workspace artifacts, not runtime code) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (21)
📝 WalkthroughWalkthroughThis PR renames package scopes across the monorepo from Changes
Sequence DiagramsequenceDiagram
participant CLI as ops-harbor CLI
participant Config as Config File<br/>(~/.config/...)
participant Health as Control Plane<br/>Health Endpoint
participant Spawn as Process Spawn
participant CP as Control Plane<br/>Service
participant Signals as Signal Handler
CLI->>Config: Read control-plane.json
Config-->>CLI: Port (or default 4130)
CLI->>Health: GET /api/health<br/>(700ms timeout)
alt Control Plane Reachable
Health-->>CLI: 200 OK
CLI->>Signals: Register SIGINT/SIGTERM handlers
else Control Plane Unreachable
Health-->>CLI: Timeout/Error
CLI->>Spawn: Spawn control-plane.js via bun
Spawn-->>CP: Process started
CP-->>CLI: Service running
CLI->>Signals: Register SIGINT/SIGTERM handlers
end
Signals->>CP: kill(15) on signal
CP-->>Signals: Process terminated
Signals->>CLI: Exit code 0
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
@repoスコープにリネーム:@r_masseater/ops-harbor-core→@repo/ops-harbor-core,@r_masseater/ops-harbor-control-plane→@repo/ops-harbor-control-planenpm installで private パッケージを解決しようとする問題を解消dist/control-plane.js(122KB) として同梱。production CLI で自動起動するロジックを追加.agents/workspace ファイルを除外Test plan
bun install— lockfile 再生成bun run build— 全パッケージビルド成功、control-plane.js バンドル確認bun run typecheck— 型エラーなしbun run test— 全テストパスbun knip— unused dependency/file なしnpm publish --dry-run— tarball にdist/control-plane.js含む、dependencies に@repo/*なし🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Chores
@r_masseaterto@repoworkspace-wide.