fix: resolve CI test failures across 6 packages by Copilot · Pull Request #1077 · objectstack-ai/objectstack

Copilot · 2026-04-07T05:43:03Z

Tests were out of sync with production code after recent changes to HttpDispatcher.dispatch() signature, AuthPlugin dependencies, and workspace dependency graph.

`dispatch()` prefix argument — nuxt, nextjs, fastify, sveltekit

HttpDispatcher.dispatch() now takes a 6th prefix argument, but adapter test assertions only checked 5 args. Added '/api' to all toHaveBeenCalledWith calls:

expect(mockDispatcher.dispatch).toHaveBeenCalledWith('GET','/meta/objects',undefined,expect.any(Object),expect.objectContaining({request: expect.anything()}),'/api',// ← was missing);

`service-ai` SDK fallback test

@ai-sdk/openai is now resolvable as a transitive workspace dependency (via apps/studio), so the "SDK not installed" fallback test was hitting the success path instead. Fixed by mocking the dynamic imports with vi.doMock().

`plugin-auth` mock context

AuthPlugin.dependencies changed to ['com.objectstack.engine.objectql'] — updated assertion
AuthPlugin.init() now calls ctx.getService('manifest').register() — added mock manifest service to test context

vercel · 2026-04-07T05:43:10Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
objectstack-play	Ready	Preview, Comment	Apr 7, 2026 6:20am
spec	Ready	Preview, Comment	Apr 7, 2026 6:20am

…tests Add the 6th 'prefix' argument ('/api') to all toHaveBeenCalledWith assertions for dispatcher.dispatch() in fastify, nextjs, and sveltekit adapter test files. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

…veltekit, and plugin-auth - service-ai: mock @ai-sdk/openai dynamic import for SDK fallback test - nuxt, nextjs, fastify, sveltekit: add missing prefix arg to dispatch assertions - plugin-auth: update dependencies assertion and mock manifest service Agent-Logs-Url: https://github.com/objectstack-ai/framework/sessions/faa63a9f-127e-42d7-8e7f-a2a058ddde10 Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Agent-Logs-Url: https://github.com/objectstack-ai/framework/sessions/faa63a9f-127e-42d7-8e7f-a2a058ddde10 Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Copilot

Pull request overview

Fixes CI test failures by aligning multiple package test suites with recent runtime/plugin changes (notably HttpDispatcher.dispatch()’s 6th prefix arg, AuthPlugin dependency updates, and service-ai provider SDK fallback behavior) to keep the monorepo’s adapter/plugin/service tests consistent with production code.

Changes:

Updated adapter tests (Nuxt/Next.js/Fastify/SvelteKit) to assert the new 6th dispatch() argument (prefix, e.g. '/api').
Adjusted service-ai SDK-missing fallback test to reliably simulate missing provider SDKs using vi.doMock() and cleanup via vi.doUnmock().
Updated plugin-auth tests to reflect new dependencies and to provide a manifest service mock for init().

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
packages/services/service-ai/src/tests/ai-service.test.ts	Mocks provider SDK modules to force the “SDK missing” fallback path in auto-detection tests.
packages/plugins/plugin-auth/src/auth-plugin.test.ts	Updates expected dependencies and adds `manifest` service mocking in the test context.
packages/adapters/sveltekit/src/sveltekit.test.ts	Adds `'/api'` prefix argument to `dispatch()` assertion expectations.
packages/adapters/nuxt/src/nuxt.test.ts	Adds `'/api'` prefix argument to `dispatch()` assertion expectations.
packages/adapters/nextjs/src/nextjs.test.ts	Adds `'/api'` prefix argument to `dispatch()` assertion expectations.
packages/adapters/nextjs/src/metadata-api.test.ts	Adds `'/api'` prefix argument to `dispatch()` assertion expectations across integration cases.
packages/adapters/fastify/src/fastify.test.ts	Adds `'/api'` prefix argument to `dispatch()` assertion expectations.
CHANGELOG.md	Documents the CI test fixes under “Unreleased → Fixed”.

Copilot · 2026-04-07T06:29:53Z

+ getService: vi.fn((name: string) => {
+ if (name === 'manifest') return { register: vi.fn() };
+ if (name === 'data') return undefined;
+ return undefined;
+ }),


PluginContext.getService() is specified to throw when a service is missing (and the real kernel implementations do). This mock currently returns undefined for unknown services (including 'data'), which can mask incorrect service lookups and makes the test context diverge from production behavior. Consider having the mock throw by default and explicitly return only the services this test intends to provide (e.g. manifest), and if AuthPlugin is meant to support “no data engine” mode, simulate that by throwing for 'data' (and update the implementation accordingly).

Copilot · 2026-04-07T06:29:54Z

+ // Re-import the plugin module so it picks up the mocked imports
+ const { AIServicePlugin: FreshPlugin } = await import('../plugin.js');
+ const plugin = new FreshPlugin();


The comment says the module is re-imported “so it picks up the mocked imports”, but import('../plugin.js') will still return the cached module unless vi.resetModules() (or similar) is used. Either reset the module graph before re-importing, or drop the “fresh import” and just instantiate AIServicePlugin after setting vi.doMock(...) (the dynamic provider imports are what need mocking, not the plugin module load).
Suggested change
// Re-import the plugin module so it picks up the mocked imports
const{AIServicePlugin: FreshPlugin}=awaitimport('../plugin.js');
constplugin=newFreshPlugin();
// The provider SDKs are dynamically imported during plugin initialization,
// so mocking those modules is sufficient; the plugin module itself does
// not need to be re-imported here.
constplugin=newAIServicePlugin();

Initial plan
9df9d77

CopilotAI assigned Copilot and hotlong Apr 7, 2026

Copilotstarted work on behalf of hotlong April 7, 2026 05:43 View session

vercelBotdeployed to Preview – objectstack-play April 7, 2026 05:54 View deployment

vercelBotdeployed to Preview – spec April 7, 2026 05:56 View deployment

CopilotAIand others added 2 commits April 7, 2026 06:01

vercelBotdeployed to Preview – spec April 7, 2026 06:11 View deployment

docs: update CHANGELOG.md with CI test fix details
b98cac2
Agent-Logs-Url: https://github.com/objectstack-ai/framework/sessions/faa63a9f-127e-42d7-8e7f-a2a058ddde10 Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

CopilotAI changed the title ~~[WIP] Fix all CI build and test errors~~fix: resolve CI test failures across 6 packagesApr 7, 2026

CopilotAI requested a review from hotlong April 7, 2026 06:14

Copilotfinished work on behalf of hotlong April 7, 2026 06:14

vercelBotdeployed to Preview – objectstack-play April 7, 2026 06:18 View deployment

vercelBotdeployed to Preview – spec April 7, 2026 06:20 View deployment

hotlong marked this pull request as ready for review April 7, 2026 06:24

CopilotAI review requested due to automatic review settings April 7, 2026 06:24

hotlong merged commit db30cd1 into mainApr 7, 2026
3 checks passed

Copilotstarted reviewing on behalf of hotlong April 7, 2026 06:25 View session

CopilotAI reviewed Apr 7, 2026
View reviewed changes

claudeBot mentioned this pull request Aug 6, 2026
[PM] 分片分工登记表 — 谁在管哪个仓库的队列 #4604
Closed

os-zhuang mentioned this pull request Aug 6, 2026
fix(driver-turso): RemoteTransport 拒收条件层的 $-算子键 —— 不再编译成静默空集/全表写 (#5769) #5921
Merged

Navigation Menu

Uh oh!

fix: resolve CI test failures across 6 packages - #1077

fix: resolve CI test failures across 6 packages#1077
hotlong merged 4 commits into
mainfrom
copilot/fix-ci-build-test-errors-another-one

CopilotAI commented Apr 7, 2026•
edited
Loading

Uh oh!

vercelBot commented Apr 7, 2026•
edited
Loading

Uh oh!

Uh oh!

CopilotAI left a comment

Uh oh!

CopilotAIApr 7, 2026

Uh oh!

CopilotAIApr 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

-// Re-import the plugin module so it picks up the mocked imports
-const{AIServicePlugin: FreshPlugin}=awaitimport('../plugin.js');
-constplugin=newFreshPlugin();
+// The provider SDKs are dynamically imported during plugin initialization,
+// so mocking those modules is sufficient; the plugin module itself does
+// not need to be re-imported here.
+constplugin=newAIServicePlugin();

Uh oh!

Conversation

CopilotAI commented Apr 7, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

dispatch() prefix argument — nuxt, nextjs, fastify, sveltekit

service-ai SDK fallback test

plugin-auth mock context

Uh oh!

vercelBot commented Apr 7, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

CopilotAIApr 7, 2026

Choose a reason for hiding this comment

Uh oh!

CopilotAIApr 7, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

CopilotAI commented Apr 7, 2026•
edited
Loading

`dispatch()` prefix argument — nuxt, nextjs, fastify, sveltekit

`service-ai` SDK fallback test

`plugin-auth` mock context

vercelBot commented Apr 7, 2026•
edited
Loading