[Copilot]

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]

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

[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]

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]

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 } = await import('../plugin.js');
	const plugin = new FreshPlugin();
	// 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.
	const plugin = new AIServicePlugin();