Merge pull request #1154 from objectstack-ai/copilot/fix-object-list-change

fix(objectql): respect packageId filter when merging MetadataService items

Author: xuyushun441-sys

CHANGELOG.md

@@ -26,6 +26,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
     - Preserved skill independence — Each skill remains independently installable/referenceable (no global routing required)
 
 ### Fixed
+- **Studio: Package switcher not filtering object list** — Fixed a bug where switching packages in the Studio left sidebar did not change the displayed object list. The root cause was in `ObjectStackProtocolImplementation.getMetaItems()`: after filtering items by `packageId` via `SchemaRegistry.listItems()`, the code merged in ALL runtime items from MetadataService without respecting the `packageId` filter, effectively overriding the filtered results. The same issue existed in `HttpDispatcher.handleMetadata()` where the MetadataService fallback path also ignored `packageId`. Both paths now correctly filter MetadataService items by `_packageId` when a package scope is requested.
 - **MetadataPlugin driver bridging fallback** — Fixed `MetadataPlugin.start()` so the driver service scan fallback (`driver.*`) is reached when ObjectQL returns `null` (not just when it throws). Previously, `setDatabaseDriver` was never called in environments where ObjectQL was not loaded.
 - **Auth trustedOrigins test alignment** — Updated `plugin-auth` tests to match the auto-default `http://localhost:*` behavior added in PR #1152 for better-auth CORS support. When no `trustedOrigins` are configured, the implementation correctly defaults to trusting all localhost ports for development convenience.
 - **Docs build: lucide-react module resolution** — Added Turbopack `resolveAlias` in `apps/docs/next.config.mjs` so MDX content files in `content/docs/` (outside the app directory) can resolve `lucide-react`. Turbopack starts module resolution from the file's directory, which doesn't have access to the app's `node_modules/`.

packages/objectql/src/protocol.ts

@@ -253,7 +253,14 @@ export class ObjectStackProtocolImplementation implements ObjectStackProtocol {
             const services = this.getServicesRegistry?.();
             const metadataService = services?.get('metadata');
             if (metadataService && typeof metadataService.list === 'function') {
-                const runtimeItems = await metadataService.list(request.type);
+                let runtimeItems = await metadataService.list(request.type);
+                // When filtering by packageId, only include runtime items that
+                // belong to the requested package. MetadataService.list() returns
+                // items from ALL packages, so we must filter here to respect the
+                // package scope requested by the caller (e.g., Studio sidebar).
+                if (packageId && runtimeItems && runtimeItems.length > 0) {
+                    runtimeItems = runtimeItems.filter((item: any) => item?._packageId === packageId);
+                }
                 if (runtimeItems && runtimeItems.length > 0) {
                     // Merge, avoiding duplicates by name
                     const itemMap = new Map<string, any>();

packages/runtime/src/http-dispatcher.ts

@@ -534,7 +534,12 @@ export class HttpDispatcher {
             const metadataService = await this.getService(CoreServiceName.enum.metadata);
             if (metadataService && typeof (metadataService as any).list === 'function') {
                 try {
-                    const items = await (metadataService as any).list(typeOrName);
+                    let items = await (metadataService as any).list(typeOrName);
+                    // Respect package filter: MetadataService.list() returns ALL items,
+                    // so filter by _packageId when a specific package is requested.
+                    if (packageId && items && items.length > 0) {
+                        items = items.filter((item: any) => item?._packageId === packageId);
+                    }
                     if (items && items.length > 0) {
                         return { handled: true, response: this.success({ type: typeOrName, items }) };
                     }