Skip to content

feat: Implement Uplink remote management system#6

Open
MiniCodeMonkey wants to merge 55 commits intomainfrom
feat/uplink
Open

feat: Implement Uplink remote management system#6
MiniCodeMonkey wants to merge 55 commits intomainfrom
feat/uplink

Conversation

@MiniCodeMonkey
Copy link
Owner

@MiniCodeMonkey MiniCodeMonkey commented Feb 16, 2026

Summary

Implements the Uplink feature — a complete remote management and control system for Chief. This enables a central server to manage Chief instances running on remote machines via WebSocket, supporting remote PRD sessions, run control, file watching, project management, and automated deployment.

Changes

CLI Foundation

  • Migrated CLI to Cobra framework for extensible command structure (US-001)
  • Extracted shared engine from TUI for reuse across command modes (US-005)

Authentication

  • Credential storage with secure keyring integration (US-002)
  • chief login / chief logout with automatic token refresh (US-003, US-004)
  • One-time setup token for automated provisioning (US-025)

WebSocket Communication Layer

  • WebSocket client with automatic reconnection and exponential backoff (US-006)
  • Protocol handshake with version negotiation (US-007)
  • Typed message serialization with 30+ message types (US-008)
  • Rate limiting to prevent message flooding (US-026)

Server Mode (chief serve)

  • Headless server command for remote-managed operation (US-009)
  • Workspace scanner for project discovery (US-010)
  • Selective file watcher with gitignore-aware filtering (US-011)
  • State snapshots and project handlers (US-012)
  • Graceful shutdown with connection draining (US-027)
  • Configurable WebSocket URL for local development (US-028)

Remote Operations

  • Interactive PRD sessions over WebSocket (US-013)
  • Session timeout with idle warnings (US-014)
  • Run control — start, pause, resume, cancel (US-015)
  • Quota detection and automatic pause (US-016)
  • Real-time run progress streaming (US-017)
  • Project settings management (US-018)
  • Per-story logging and retrieval (US-019)
  • Per-story diff generation (US-020)
  • Git clone and project creation (US-021)
  • Remote chief update triggered via WebSocket (US-022, US-023)

Deployment

  • Systemd service unit file (US-024)
  • Cloud-init script for automated VM provisioning (US-024)

Stats

  • 58 files changed
  • ~19,550 lines added
  • 28 user stories implemented (US-001 through US-028)
  • Comprehensive test coverage across all new packages

MiniCodeMonkey and others added 30 commits February 15, 2026 21:34
@MiniCodeMonkey @claude
feat: US-001 - Migrate CLI to Cobra Framework
629968a 
Replace manual flag parsing with Cobra command tree. All existing
commands (new, edit, status, list, wiggum) and flags (--max-iterations,
--no-sound, --no-retry, --verbose, --merge, --force, --help, --version)
are preserved. TUI remains the default when no subcommand is given.
Clean, organized help output with command grouping.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-002 - Implement Credential Storage
c6fc23e 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-003 - Implement chief login
51b8ff2 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-004 - Implement chief logout and Automatic Token Refresh
2b0f712 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-005 - Extract Shared Engine from TUI
c45524f 
Create internal/engine package that wraps loop.Manager with fan-out
event subscription, enabling multiple concurrent consumers (TUI and
future WebSocket handler). Refactor TUI to consume events from engine
instead of directly from loop.Manager.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-006 - Implement WebSocket Client with Reconnection
268fe32 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-007 - Implement Protocol Handshake
c37d074 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-008 - Implement Message Types and Serialization
45f0591 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-009 - Implement Basic chief serve Command
01006a8 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-009 - Implement Basic chief serve Command
424db4c 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-010 - Implement Workspace Scanner
5835b69 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-011 - Implement Selective File Watching
4a4a82d 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-012 - Implement State Snapshot and Project Handlers
2d9255f 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-012 - Implement State Snapshot and Project Handlers
fe13e4b 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-013 - Implement Interactive Claude PRD Sessions over WebSocket
55055b2 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-014 - Implement Session Timeout with Warnings
14ac03a 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-015 - Implement Run Control via WebSocket
eedddc7 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-016 - Implement Quota Detection and Auto-Pause
540c120 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-017 - Implement Run Progress Streaming
417825e 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-018 - Implement Project Settings via WebSocket
5619507 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-019 - Implement Per-Story Logging
fdfae76 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-020 - Implement Per-Story Diff Generation and Retrieval
6da6867 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-021 - Implement Git Clone and Project Creation via WebSocket
a6a6ed6 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-022 - Implement Version Check and chief update Command
df28929 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-023 - Implement Remote Update via WebSocket
572587f 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-024 - Create Systemd Service and Cloud-Init Script
c168eba 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-025 - Implement One-Time Setup Token for Automated Chief Auth
050576d 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-026 - Implement WebSocket Rate Limiting
14432b0 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-027 - Implement Graceful Shutdown for chief serve
76f510c 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey @claude
feat: US-028 - Implement WebSocket URL Configuration for Local Develo…
d06835b 
…pment

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@MiniCodeMonkey
fix(update): don't flag dev builds as outdated when based on latest r…
072b5bd 
…elease

Dev builds from git describe (e.g. v0.4.0-61-gd06835b) were compared
as plain strings against the latest release tag, causing a false
"update available" prompt. Extract the base semver from git-describe
suffixes so builds ahead of a tag are recognized as current.
@MiniCodeMonkey
build: output binary to bin/ directory instead of project root
9c17e79
@MiniCodeMonkey
fix(auth): add /api prefix to OAuth endpoint paths
9f1de29
@MiniCodeMonkey
fix(auth): correct device authorization URL path
eb8af16 
The CLI was directing users to /device but the web app serves the
device code entry page at /oauth/device.
@MiniCodeMonkey
fix(serve): default --workspace to current directory
62e799d 
Running `chief serve` without --workspace produced a confusing error
with an empty path. Now defaults to the current directory and resolves
to an absolute path before validation.
@MiniCodeMonkey
feat(ws): discover WebSocket URL from server during login and token r…
fcfcfc5 
…efresh

The Go client previously hardcoded the WS URL, but on Laravel Cloud the
Reverb WebSocket server runs at a separate hostname. Include ws_url in
OAuth token responses so the client discovers it automatically during
login and refresh, storing it in credentials.yaml.

WS URL precedence: flag > env > user config > credentials > default.
@MiniCodeMonkey
fix(ws): round retry delay to nearest millisecond in log output
ff40884
@MiniCodeMonkey
fix(prd): add --verbose flag required by stream-json output format
9cfb2ee
@MiniCodeMonkey
docs: Websocket refactor spec
dc972fe
@MiniCodeMonkey
feat: US-014 - HTTP uplink client — connect and disconnect
ed190c9 
Add internal/uplink package with HTTP client for the device API.
Replaces WebSocket-based connect/disconnect with HTTP POST endpoints.

- Client struct with functional options pattern (matching ws.Client style)
- Connect(ctx) calls POST /api/device/connect, returns WelcomeResponse with Reverb config
- Disconnect(ctx) calls POST /api/device/disconnect
- HTTPS enforcement with localhost/127.0.0.1 exception
- Thread-safe SetAccessToken() for token refresh
- ConnectWithRetry() with exponential backoff + jitter (same pattern as ws.Client)
- Auth error classification (401/403 not retried)
- 20 unit tests with httptest mock server
@MiniCodeMonkey
feat: US-015 - HTTP uplink client — message sending
bcffa3c
@MiniCodeMonkey
feat: US-016 - Message batcher
4ab3986
@MiniCodeMonkey
feat: US-017 - Pusher client for receiving commands
ef16f05
@MiniCodeMonkey
feat: US-018 - Uplink unified interface
dc45cdb
@MiniCodeMonkey
feat: US-019 - Heartbeat goroutine
1bb1fe0
@MiniCodeMonkey
feat: US-020 - Reconnection logic
936774e
@MiniCodeMonkey
feat: US-021 - Refactor serve.go to use uplink
ddb3c07
@MiniCodeMonkey
feat: US-022 - Graceful shutdown updates
6fc386d
@MiniCodeMonkey
feat: US-023 - End-to-end integration testing
d2d19b8
@MiniCodeMonkey
feat: US-027 - Remove old WebSocket client code
1d1ebf3
@MiniCodeMonkey
fix(uplink): correct Pusher protocol handling for origin and data enc…
b69c621 
…oding

Set explicit Origin header on WebSocket dial to satisfy Reverb's origin
check. Double-unmarshal the connection_established data field to match
the real Pusher wire format (JSON string containing JSON). Update test
helper to match the real protocol.
@MiniCodeMonkey
fix(update): handle -dirty suffix in git-describe version strings
42cdf39 
Strip the -dirty suffix before parsing so dev builds like
"0.4.0-81-g1d1ebf3-dirty" correctly resolve to base version "0.4.0"
instead of being treated as a different version.
@MiniCodeMonkey
fix(auth): send grant_type and Accept header in token refresh request
c6e5d15 
The server requires grant_type=refresh_token in the request body and
returns HTML error pages without an Accept: application/json header,
causing JSON parse failures on refresh.
@MiniCodeMonkey
fix(uplink): unwrap payload envelope from CommandRelayController mess…
e5e7417 
…ages

The server's CommandRelayController sends commands wrapped in
{"type": ..., "payload": {...}} but handlers expected fields at the
top level. Extract the payload before dispatching to handlers.
@MiniCodeMonkey
feat: add contract testing between chief CLI and chief-uplink
de10961 
Add shared JSON fixtures and contract tests that verify both sides
agree on the wire format. Catches serialization mismatches (like
port-as-string, name-vs-project_slug, payload wrapper) with zero
infrastructure — just unit tests against canonical fixture files.

Also fixes mock Pusher server to double-encode connection_established
data, matching the real Pusher protocol.

- contract/fixtures/ synced from chief-uplink (source of truth)
- internal/contract/contract_test.go: 9 contract tests
- Makefile: sync-fixtures and test-contract targets
- .gitignore: exclude synced fixtures
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MiniCodeMonkey