Skip to content

Add CI pipeline and improve pool-degraded diagnostics#3

Merged
trentmcnitt merged 4 commits into
mainfrom
ci-and-pool-degraded-reasons
Mar 20, 2026
Merged

Add CI pipeline and improve pool-degraded diagnostics#3
trentmcnitt merged 4 commits into
mainfrom
ci-and-pool-degraded-reasons

Conversation

@trentmcnitt
Copy link
Copy Markdown
Owner

Summary

  • Add GitHub Actions CI workflow with two parallel jobs: Quality (lint + type-check + format) and Test (unit tests)
  • Add CI spec document (docs/CI.md)
  • Propagate reason strings through the pool-degraded event chain for specific error messages (warmup failure vs circuit breaker vs auth)
  • Add 30s warmup timeout to detect unresponsive CLI subprocesses
  • Improve restart flow: delegate to restartPools command, replace "Open Terminal" with "Open Log"

Test plan

  • CI jobs pass on this PR (first run!)
  • Pool degradation shows specific reason in error notification
  • "Open Log" button opens the output channel
  • "Restart Pools" recovers from degraded state

🤖 Generated with Claude Code

Add GitHub Actions CI workflow (lint, type-check, format, unit tests)
with branch protection ruleset on main. Propagate reason strings through
the pool-degraded event chain for better error messages and diagnostics.
Add warmup timeout (30s) to detect unresponsive CLI subprocesses.
tree-sitter-html and tree-sitter-rust declare peer deps on older
tree-sitter versions. npm ci enforces strict resolution, so these
overrides tell npm to use the root project's tree-sitter@0.25.
Vitest 4 requires Node 20+. The esbuild target (node18) controls output
syntax compatibility, not the CI runtime. Also ran prettier to fix 16
files with pre-existing formatting drift.
@trentmcnitt trentmcnitt merged commit 0ffbebb into main Mar 20, 2026
2 checks passed
@trentmcnitt trentmcnitt deleted the ci-and-pool-degraded-reasons branch March 20, 2026 00:29
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