Skip to content

test(core): stabilize shell progress test#37643

Merged
rekram1-node merged 2 commits into
v2from
shell-progress-test
Jul 18, 2026
Merged

test(core): stabilize shell progress test#37643
rekram1-node merged 2 commits into
v2from
shell-progress-test

Conversation

@opencode-agent

Copy link
Copy Markdown
Contributor

Summary

  • keep the real shell progress integration coverage while replacing the fixed 1.5-second race with a sentinel-file handshake
  • release the command only after observing a truncated text progress update
  • use a 15-second timeout only as a deadlock guard and always release the command during cleanup

Tests

  • bun test test/tool-shell.test.ts
  • targeted progress test passed 10/10 repeated runs
  • bun typecheck

Fixes the flaky Windows failure observed in https://github.com/anomalyco/opencode/actions/runs/29630441415/job/88042842174.

Requested by: @rekram1-node (Aiden Cline via Slack)

@rekram1-node
rekram1-node merged commit 7d4496e into v2 Jul 18, 2026
8 checks passed
@rekram1-node
rekram1-node deleted the shell-progress-test branch July 18, 2026 16:59
github-actions Bot pushed a commit to ReStranger/opencode that referenced this pull request Jul 18, 2026
* upstream/v2:
  feat(core): publish pending MCP status during connect (anomalyco#37605)
  feat(codemode): native coercion parity in interpreter (anomalyco#37608)
  test(core): stabilize shell progress test (anomalyco#37643)
  fix(ai): parse compatible reasoning deltas (anomalyco#37558)
  fix(core): remove session import cycle (anomalyco#37596)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant