Skip to content

refactor: remove dead install progress streaming - #158

Merged
fullstackjam merged 2 commits into
mainfrom
codex/remove-dead-progress
Aug 1, 2026
Merged

refactor: remove dead install progress streaming#158
fullstackjam merged 2 commits into
mainfrom
codex/remove-dead-progress

Conversation

@fullstackjam

Copy link
Copy Markdown
Member

What does this PR do?

Removes the obsolete install-progress event bridge and makes the existing linear console progress path unconditional.

Why?

The v5 wizard install screen was intentionally removed in #150, leaving internal/progress, the brew/npm SetProgressSink hooks, and every progressSink.Emit branch without a production caller. The informational deadcode job reports the drift but does not block merges.

Testing

  • go vet ./... passes
  • Relevant tests added or updated — obsolete streaming-only tests were removed; brew, npm, installer, and archtest packages pass
  • Interactive dry-run not run — this removes an unreachable TUI path and preserves the existing linear console behavior

Cross-repo checklist

  • No docs/content update is needed in openboot.dev
  • No CLI ↔ server API contract change

Notes for reviewer

The no-direct-exec baseline only changes because deleting branches moved two existing, intentional call sites to new line numbers.

@github-actions github-actions Bot added installer Package installation logic brew Homebrew related tests Tests only docs labels Aug 1, 2026
@fullstackjam
fullstackjam merged commit 4a17158 into main Aug 1, 2026
14 checks passed
@fullstackjam
fullstackjam deleted the codex/remove-dead-progress branch August 1, 2026 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

brew Homebrew related docs installer Package installation logic tests Tests only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant