fix(win32): fix plugin resolution with createRequire fallback

[Hona]

Summary

• Remove 3s sleep after writing package.json in installDependencies — was unnecessary and caused test timeouts.
• Add createRequire fallback when import.meta.resolve fails for plugins in freshly created node_modules — import.meta.resolve sometimes can't find packages that were just installed on Windows.
• Use static expected path in config test assertion instead of mirroring production fallback logic.

Fixes 4 Windows unit test failures. Split out from #14742.

[Copilot]

Pull request overview

This PR fixes Windows-specific test failures by improving plugin resolution reliability and bun install stability in CI environments.

Changes:

• Removed unnecessary 3-second sleep after writing package.json that caused test timeouts
• Added --no-cache flag to bun install in CI and E2E environments to prevent concurrent global cache lock contention
• Implemented createRequire fallback for plugin resolution when import.meta.resolve fails on Windows with freshly created node_modules
• Mirrored the createRequire fallback logic in config tests

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
packages/opencode/src/config/config.ts	Added createRequire import and fallback logic for plugin resolution; removed 3s sleep; expanded --no-cache conditions to include CI and E2E environments
packages/opencode/test/config/config.test.ts	Added createRequire fallback in test to mirror production behavior for Windows compatibility

Comments suppressed due to low confidence (2)

packages/opencode/src/config/config.ts:3

• The fileURLToPath import is not used anywhere in this file. Consider removing it to keep imports clean.

import { pathToFileURL, fileURLToPath } from "url"

packages/opencode/test/config/config.test.ts:697

• For consistency with the production code in config.ts (which imports createRequire at the top), consider importing createRequire at the top of the test file instead of using a dynamic import. This would also match the pattern used for other imports like pathToFileURL.

 const { createRequire } = await import("module")

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Inconsistency with production code: The test passes tmp.path + "/" to createRequire, but the production code in config.ts line 1340 passes options.path (the full path to opencode.json file). For consistency and to accurately test the production behavior, this should be createRequire(path.join(tmp.path, "opencode.json")). Additionally, using string concatenation with "/" could cause issues on Windows where paths use backslashes.

Suggested change

	constrequire=createRequire(tmp.path+"/")
	constrequire=createRequire(path.join(tmp.path,"opencode.json"))