Skip to content

refactor: share Go control-plane client#38

Merged
steipete merged 1 commit into
mainfrom
codex/deduplicate-go-control-plane
Jun 15, 2026
Merged

refactor: share Go control-plane client#38
steipete merged 1 commit into
mainfrom
codex/deduplicate-go-control-plane

Conversation

@steipete

Copy link
Copy Markdown
Contributor

Summary

  • move shared Crabfleet API models, authentication, lifecycle semantics, HTTP calls, and terminal operations into internal/fleetapi
  • move terminal-safe user, session-tree, log, status, and summary rendering into internal/fleettext
  • keep CLI and SSH gateway entry points focused on command-specific behavior
  • make Docker and GitHub Actions track the complete internal/** Go package boundary

Validation

  • go test ./...
  • go test -race ./...
  • go vet ./...
  • go build ./cmd/crabfleet ./cmd/crabbox-ssh-gateway
  • pnpm check
  • pnpm test
  • pnpm build
  • docker build -f Dockerfile.ssh-gateway .
  • autoreview clean after fixing its large JSON response finding

@steipete steipete requested a review from a team as a code owner June 15, 2026 08:34
@clawsweeper

clawsweeper Bot commented Jun 15, 2026

Copy link
Copy Markdown

ClawSweeper status: review started.

I am starting a fresh review of this pull request: refactor: share Go control-plane client This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@steipete steipete merged commit 3849a62 into main Jun 15, 2026
1 check passed
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