Merge pull request #1127 from objectstack-ai/copilot/fix-i18n-object-format

service-ai: complete I18nLabelSchema migration for Setup nav labels

Author: xuyushun441-sys

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 - **CI: Replace `pnpm/action-setup@v6` with corepack** — Switched all GitHub Actions workflows (`ci.yml`, `lint.yml`, `release.yml`, `validate-deps.yml`, `pr-automation.yml`) from `pnpm/action-setup@v6` to `corepack enable` to fix persistent `ERR_PNPM_BROKEN_LOCKFILE` errors. Corepack reads the exact `packageManager` field from `package.json` (including SHA verification), ensuring the correct pnpm version is used in CI. Also bumped pnpm store cache keys to v3 and added a pnpm version verification step.
 - **Broken pnpm lockfile** — Regenerated `pnpm-lock.yaml` from scratch to fix `ERR_PNPM_BROKEN_LOCKFILE` ("expected a single document in the stream, but found more") that was causing all CI jobs to fail. The previous merge of PR #1117 only included workflow cache key changes but did not carry over the regenerated lockfile.
+- **service-ai: Fix navigation item labels using deprecated i18n object format** — Replaced `{ key, defaultValue }` i18n objects with plain string labels in `AIServicePlugin`'s Setup App navigation contributions, completing the `I18nLabelSchema` migration from [#1054](https://github.com/objectstack-ai/framework/issues/1054).
 
 ### Added
 - **MCP Runtime Server Plugin (`plugin-mcp-server`)** — New kernel plugin that exposes ObjectStack

packages/services/service-ai/src/__tests__/ai-service.test.ts

@@ -818,6 +818,23 @@ describe('AIServicePlugin', () => {
     expect(ctx.replaceService).toHaveBeenCalledWith('ai', expect.any(Object));
   });
 
+  it('should contribute Setup navigation labels as plain strings', async () => {
+    const plugin = new AIServicePlugin();
+    const ctx = createMockContext();
+    const contribute = vi.fn();
+    ctx.registerService('setupNav', { contribute });
+
+    await plugin.init(ctx);
+
+    expect(contribute).toHaveBeenCalledWith({
+      areaId: 'area_ai',
+      items: [
+        expect.objectContaining({ id: 'nav_ai_conversations', label: 'Conversations' }),
+        expect.objectContaining({ id: 'nav_ai_messages', label: 'Messages' }),
+      ],
+    });
+  });
+
   it('should clean up on destroy', async () => {
     const plugin = new AIServicePlugin();
     const ctx = createMockContext();

packages/services/service-ai/src/plugin.ts

@@ -231,8 +231,8 @@ export class AIServicePlugin implements Plugin {
         setupNav.contribute({
           areaId: 'area_ai',
           items: [
-            { id: 'nav_ai_conversations', type: 'object', label: { key: 'setup.nav.ai_conversations', defaultValue: 'Conversations' }, objectName: 'conversations', icon: 'message-square', order: 10 },
-            { id: 'nav_ai_messages', type: 'object', label: { key: 'setup.nav.ai_messages', defaultValue: 'Messages' }, objectName: 'messages', icon: 'messages-square', order: 20 },
+            { id: 'nav_ai_conversations', type: 'object', label: 'Conversations', objectName: 'conversations', icon: 'message-square', order: 10 },
+            { id: 'nav_ai_messages', type: 'object', label: 'Messages', objectName: 'messages', icon: 'messages-square', order: 20 },
           ],
         });
         ctx.logger.info('[AI] Navigation items contributed to Setup App');