Skip to content

fix(cli): simplify service registration lease#37576

Merged
thdxr merged 1 commit into
v2from
lease-monitor
Jul 17, 2026
Merged

fix(cli): simplify service registration lease#37576
thdxr merged 1 commit into
v2from
lease-monitor

Conversation

@opencode-agent

Copy link
Copy Markdown
Contributor

Summary

  • publish the managed-service registration exactly once after the exact-port bind
  • shut down the server when its registration is missing, corrupt, or no longer matches, without repairing it
  • close the listener before conditionally removing only the instance-owned registration
  • remove the unused startedAt registration generation from client Info

Validation

  • bun test test/service.test.ts --timeout 120000 (packages/cli, 15 passed)
  • bun test test/service.test.ts test/promise-service.test.ts --timeout 30000 (packages/client, 15 passed)
  • GOMAXPROCS=2 bun typecheck (packages/server)
  • GOMAXPROCS=2 bun typecheck (packages/cli)
  • GOMAXPROCS=2 bun typecheck (packages/client)
  • Prettier check on all changed files

Requested by: @thdxr (Dax Raad via Slack)

@thdxr
thdxr merged commit 60ce33c into v2 Jul 17, 2026
10 checks passed
@thdxr
thdxr deleted the lease-monitor branch July 17, 2026 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant