Skip to content

test: include server/session/tmux in coverage gate (#1616)#1628

Closed
OneStepAt4time wants to merge 1 commit intodevelopfrom
fix/1616-core-coverage-gate-v2
Closed

test: include server/session/tmux in coverage gate (#1616)#1628
OneStepAt4time wants to merge 1 commit intodevelopfrom
fix/1616-core-coverage-gate-v2

Conversation

@OneStepAt4time
Copy link
Copy Markdown
Owner

Summary

  • remove src/server.ts, src/session.ts, and src/tmux.ts from Vitest coverage exclusions so they are enforced by the global quality gate
  • add high-signal integration-style coverage suites for SessionManager and TmuxManager, including discovery polling, permission flows, transcript access paths, retry/timeout behavior, and platform-specific env injection branches
  • keep existing global thresholds unchanged (70/60/70/70) and raise core-module coverage enough for the gate to pass

Validation

  • npm ci
  • npm run lint
  • npm run build
  • npx tsc --noEmit
  • npm test -- --coverage

All commands pass in this worktree.

Closes #1616

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@OneStepAt4time
Copy link
Copy Markdown
Owner Author

Superseded by #1627, which already merged the #1616 coverage-gate solution into develop.

@OneStepAt4time OneStepAt4time deleted the fix/1616-core-coverage-gate-v2 branch April 11, 2026 11:00
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