Skip to content

fix hanging summary tty in local tests#968

Merged
computermode merged 1 commit intomainfrom
soph/fix-hanging-summary
Apr 16, 2026
Merged

fix hanging summary tty in local tests#968
computermode merged 1 commit intomainfrom
soph/fix-hanging-summary

Conversation

@Soph
Copy link
Copy Markdown
Collaborator

@Soph Soph commented Apr 16, 2026

[test] $ go test ./...
?       github.com/entireio/cli/cmd/entire      [no test files]
^CError: not a git repository
┃ Choose a summary provider
┃ This choice will be saved. Use `entire configure` to change it later.
┃ > Claude Code
┃   Codex
┃   Copilot CLI
┃   Cursor
┃   Gemini CLI

This worked in CI because there is no tty available and also no agents installed but it triggered on my local machine


Note

Low Risk
Test-only change that sets an env var to avoid interactive prompts; no production behavior changes.

Overview
Prevents local go test runs from hanging by forcing summary-provider selection into non-interactive mode in explain checkpoint generation tests.

Adds ENTIRE_TEST_TTY=0 to the v2 --generate-related tests in explain_test.go so they don’t prompt for a TTY-based provider choice when run on a developer machine.

Reviewed by Cursor Bugbot for commit 978a2b7. Configure here.

Entire-Checkpoint: fa6d38be106a
@Soph Soph requested a review from a team as a code owner April 16, 2026 14:28
Copilot AI review requested due to automatic review settings April 16, 2026 14:28
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes local go test runs hanging due to an interactive “Choose a summary provider” prompt being triggered when a TTY is available and summary-capable agent CLIs are installed.

Changes:

  • Forces non-interactive mode in three --generate explain-checkpoint unit tests by setting ENTIRE_TEST_TTY=0 to avoid the TUI prompt.

Comment thread cmd/entire/cli/explain_test.go
Comment thread cmd/entire/cli/explain_test.go
Comment thread cmd/entire/cli/explain_test.go
@computermode
Copy link
Copy Markdown
Contributor

Shared some thoughts outside of GH but stamping to unblock local testing ^

@computermode computermode merged commit 0d9591e into main Apr 16, 2026
14 checks passed
@computermode computermode deleted the soph/fix-hanging-summary branch April 16, 2026 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants