Bug type
Regression (worked before, now fails)
Summary
Commit 8e4a1d8 (#45824) switched tts-core.ts from resolveModel (sync) to resolveModelAsync but did not update the vi.mock in tts.test.ts. The mock only exports resolveModel, so vitest throws:
No "resolveModelAsync" export is defined on the "../agents/pi-embedded-runner/model.js" mock.
This causes Windows CI shard 3/6 to fail on any PR rebased onto latest main.
Steps to reproduce
- Rebase any PR onto current
main (includes 8e4a1d8)
- Run
pnpm test — src/tts/tts.test.ts fails
Fix
Add resolveModelAsync to the mock factory and update the two summarizeText test expectations from resolveModel → resolveModelAsync.
PR incoming.
Bug type
Regression (worked before, now fails)
Summary
Commit 8e4a1d8 (#45824) switched
tts-core.tsfromresolveModel(sync) toresolveModelAsyncbut did not update thevi.mockintts.test.ts. The mock only exportsresolveModel, so vitest throws:This causes Windows CI shard 3/6 to fail on any PR rebased onto latest main.
Steps to reproduce
main(includes 8e4a1d8)pnpm test—src/tts/tts.test.tsfailsFix
Add
resolveModelAsyncto the mock factory and update the twosummarizeTexttest expectations fromresolveModel→resolveModelAsync.PR incoming.