Skip to content

fix: add search infrastructure readiness gate for CI stability#105

Merged
joshmu merged 1 commit intomasterfrom
feat/search-buffers
Feb 6, 2026
Merged

fix: add search infrastructure readiness gate for CI stability#105
joshmu merged 1 commit intomasterfrom
feat/search-buffers

Conversation

@joshmu
Copy link
Owner

@joshmu joshmu commented Feb 6, 2026

Summary

  • Adds a workspace readiness gate in verifyWorkspace() that polls vscode.workspace.findFiles() up to 5 times before tests begin, ensuring the search infrastructure is operational
  • Increases macOS CI timeout multiplier from 1.5x to 2x to account for slower virtualised I/O
  • Sets fail-fast: false in the test matrix so all platform results are collected even when one fails

Context

The Publish workflow failed on macOS with 12/139 tests failing — all due to ripgrep returning 0 results. The same commit passed in the standalone Tests workflow. Root cause: tests started before VSCode's search infrastructure finished indexing the fixture workspace.

Test plan

  • All 182 tests pass locally
  • CI passes on all 3 platforms (ubuntu, macOS, windows)
  • Publish workflow succeeds on merge to master

Publish workflow failed on macOS because tests ran before VSCode's
search infrastructure was ready, causing ripgrep to return 0 results.
Adds a polling loop in verifyWorkspace() that confirms findFiles works
before tests begin, increases macOS CI timeout multiplier to 2x, and
sets fail-fast: false for better cross-platform diagnostics.
@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

✅ Tests passed on ubuntu-latest

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

✅ Tests passed on macos-latest

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

✅ Tests passed on windows-latest

@joshmu joshmu merged commit e4540ee into master Feb 6, 2026
3 checks passed
@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

🎉 This PR is included in version 1.16.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant