Skip to content

test(project): rescue non-Hono InstanceBootstrap boundary tests#26453

Merged
kitlangton merged 1 commit into
devfrom
kit/test-rescue-instance-bootstrap
May 9, 2026
Merged

test(project): rescue non-Hono InstanceBootstrap boundary tests#26453
kitlangton merged 1 commit into
devfrom
kit/test-rescue-instance-bootstrap

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • `test/project/instance-bootstrap-regression.test.ts` covers four boundaries: `WithInstance.provide`, the CLI bootstrap, the legacy Hono `InstanceMiddleware`, and `InstanceRuntime.reloadInstance`. PR Hono → Effect HttpApi #25667 deletes the entire file with Hono.
  • The first, second, and fourth boundaries are backend-agnostic and need to keep being covered.
  • Adds `test/project/instance-bootstrap.test.ts` with the three surviving checks. Original file can stay until Hono → Effect HttpApi #25667 lands; once that merges, only this file remains.

Test plan

  • `bun run test test/project/instance-bootstrap.test.ts` (3 pass)

@kitlangton kitlangton force-pushed the kit/httpapi-test-script-effect-mode branch from 3986e17 to 6b13e18 Compare May 9, 2026 02:56
@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch from bc957f9 to 02e21fc Compare May 9, 2026 02:57
@kitlangton kitlangton force-pushed the kit/httpapi-test-script-effect-mode branch from 6b13e18 to dfb49e5 Compare May 9, 2026 03:02
@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch from 02e21fc to 800992e Compare May 9, 2026 03:02
@kitlangton kitlangton force-pushed the kit/httpapi-test-script-effect-mode branch from dfb49e5 to 1df6d83 Compare May 9, 2026 03:07
@kitlangton kitlangton closed this May 9, 2026
@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch from 800992e to 1df6d83 Compare May 9, 2026 03:07
@kitlangton kitlangton reopened this May 9, 2026
@kitlangton kitlangton force-pushed the kit/httpapi-test-script-effect-mode branch from 31eded7 to 81681f2 Compare May 9, 2026 03:13
@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch from a0dbe07 to 5e84331 Compare May 9, 2026 03:13
@kitlangton kitlangton force-pushed the kit/httpapi-test-script-effect-mode branch from 81681f2 to a605023 Compare May 9, 2026 03:19
@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch from 5e84331 to d1d42db Compare May 9, 2026 03:19
@kitlangton kitlangton force-pushed the kit/httpapi-test-script-effect-mode branch from a605023 to 897eb7f Compare May 9, 2026 03:40
@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch from d1d42db to 20e338c Compare May 9, 2026 03:40
@kitlangton kitlangton force-pushed the kit/httpapi-test-script-effect-mode branch from 897eb7f to 940633e Compare May 9, 2026 03:41
@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch from 20e338c to b7425e3 Compare May 9, 2026 03:41
@kitlangton kitlangton force-pushed the kit/httpapi-test-script-effect-mode branch from 940633e to d086513 Compare May 9, 2026 03:45
@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch from b7425e3 to d26a17b Compare May 9, 2026 03:45
@kitlangton kitlangton changed the base branch from kit/httpapi-test-script-effect-mode to dev May 9, 2026 03:45
The existing test/project/instance-bootstrap-regression.test.ts covers
four boundaries: WithInstance.provide, the CLI bootstrap, the legacy
Hono InstanceMiddleware, and InstanceRuntime.reloadInstance. The Hono
deletion PR (#25667) drops the entire file because the Hono variant
goes away, taking the three backend-agnostic boundary checks with it.

Move the three surviving boundary checks into a new
instance-bootstrap.test.ts so the InstanceBootstrap-runs-at-boundary
invariant stays covered after Hono is deleted. The original file can
remain in place until #25667 lands; once the deletion lands, only this
file remains.
@kitlangton kitlangton force-pushed the kit/test-rescue-instance-bootstrap branch from d26a17b to 5e288fa Compare May 9, 2026 03:49
@kitlangton kitlangton merged commit aab82cc into dev May 9, 2026
6 of 9 checks passed
@kitlangton kitlangton deleted the kit/test-rescue-instance-bootstrap branch May 9, 2026 03:53
katosun2 pushed a commit to katosun2/opencode that referenced this pull request May 10, 2026
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