@inspecto-dev/cli 0.3.3 → 0.3.4

package/tests/integration-install.test.ts

@@ -1,6 +1,8 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 const writeFileMock = vi.fn()
+const writeJSONMock = vi.fn()
+const readJSONMock = vi.fn()
 const existsMock = vi.fn()
 const chmodMock = vi.fn()
 const logMock = {
@@ -14,9 +16,13 @@ const logMock = {
   blank: vi.fn(),
 }
 const runIntegrationAutomationMock = vi.fn()
+const resolveIntegrationHostIdeMock = vi.fn()
+const resolveIntegrationDispatchModeMock = vi.fn()
 
 vi.mock('../src/utils/fs.js', () => ({
   writeFile: writeFileMock,
+  writeJSON: writeJSONMock,
+  readJSON: readJSONMock,
   exists: existsMock,
 }))
 
@@ -38,11 +44,21 @@ vi.mock('../src/commands/integration-automation.js', () => ({
   runIntegrationAutomation: runIntegrationAutomationMock,
 }))
 
+vi.mock('../src/commands/integration-host-ide.js', () => ({
+  resolveIntegrationHostIde: resolveIntegrationHostIdeMock,
+}))
+
+vi.mock('../src/commands/integration-dispatch-mode.js', () => ({
+  resolveIntegrationDispatchMode: resolveIntegrationDispatchModeMock,
+}))
+
 describe('integration install', () => {
   beforeEach(() => {
     vi.resetModules()
     vi.clearAllMocks()
+    vi.spyOn(process, 'cwd').mockReturnValue('/repo')
     existsMock.mockResolvedValue(false)
+    readJSONMock.mockResolvedValue(null)
     vi.stubGlobal(
       'fetch',
       vi.fn().mockResolvedValue({
@@ -56,6 +72,17 @@ describe('integration install', () => {
       status: 'launched',
       message: 'Onboarding launched in Cursor for Codex.',
     })
+    resolveIntegrationHostIdeMock.mockResolvedValue({
+      ide: 'cursor',
+      source: 'explicit',
+      confidence: 'high',
+      candidates: ['cursor'],
+    })
+    resolveIntegrationDispatchModeMock.mockResolvedValue({
+      mode: 'extension',
+      ready: true,
+      reason: 'cursor_codex_extension',
+    })
   })
 
   it('installs the Claude Code skill into the user skill directory', async () => {
@@ -145,6 +172,92 @@ describe('integration install', () => {
     )
   })
 
+  it('installs the Trae skill with a launcher script', async () => {
+    const { installIntegration } = await import('../src/commands/integration-install.js')
+
+    await installIntegration('trae')
+
+    expect(writeFileMock).toHaveBeenCalledWith(
+      '.trae/skills/inspecto-onboarding/SKILL.md',
+      '# mock asset',
+    )
+    expect(writeFileMock).toHaveBeenCalledWith(
+      '.trae/skills/inspecto-onboarding/scripts/run-inspecto.sh',
+      '# mock asset',
+    )
+    expect(chmodMock).toHaveBeenCalledWith(
+      '.trae/skills/inspecto-onboarding/scripts/run-inspecto.sh',
+      0o755,
+    )
+  })
+
+  it('persists an explicit project host ide into .inspecto/settings.local.json before onboarding launch', async () => {
+    const { installIntegration } = await import('../src/commands/integration-install.js')
+
+    await installIntegration('gemini', { ide: 'trae-cn' })
+
+    expect(writeJSONMock).toHaveBeenCalledWith('/repo/.inspecto/settings.local.json', {
+      ide: 'trae-cn',
+      'provider.default': 'gemini.extension',
+    })
+  })
+
+  it('updates the project host ide and provider.default to the selected assistant without dropping other keys', async () => {
+    readJSONMock.mockResolvedValue({
+      ide: 'trae',
+      'provider.default': 'coco.cli',
+      'prompt.autoSend': true,
+    })
+
+    const { installIntegration } = await import('../src/commands/integration-install.js')
+
+    await installIntegration('gemini', { ide: 'trae-cn' })
+
+    expect(writeJSONMock).toHaveBeenCalledWith('/repo/.inspecto/settings.local.json', {
+      ide: 'trae-cn',
+      'provider.default': 'gemini.extension',
+      'prompt.autoSend': true,
+    })
+  })
+
+  it('persists the selected assistant as provider.default so onboarding does not fall back to another detected tool', async () => {
+    const { installIntegration } = await import('../src/commands/integration-install.js')
+
+    await installIntegration('codex', { ide: 'cursor' })
+
+    expect(writeJSONMock).toHaveBeenCalledWith('/repo/.inspecto/settings.local.json', {
+      ide: 'cursor',
+      'provider.default': 'codex.extension',
+    })
+  })
+
+  it('does not persist host ide settings for user-level installs', async () => {
+    const { installIntegration } = await import('../src/commands/integration-install.js')
+
+    await installIntegration('codex', { scope: 'user', ide: 'cursor' })
+
+    expect(writeJSONMock).not.toHaveBeenCalled()
+  })
+
+  it('installs the Coco skill with a launcher script', async () => {
+    const { installIntegration } = await import('../src/commands/integration-install.js')
+
+    await installIntegration('coco')
+
+    expect(writeFileMock).toHaveBeenCalledWith(
+      '.trae/skills/inspecto-onboarding/SKILL.md',
+      '# mock asset',
+    )
+    expect(writeFileMock).toHaveBeenCalledWith(
+      '.trae/skills/inspecto-onboarding/scripts/run-inspecto.sh',
+      '# mock asset',
+    )
+    expect(chmodMock).toHaveBeenCalledWith(
+      '.trae/skills/inspecto-onboarding/scripts/run-inspecto.sh',
+      0o755,
+    )
+  })
+
   it('surfaces a blocked automation outcome as a warning with next-step guidance', async () => {
     runIntegrationAutomationMock.mockResolvedValue({
       status: 'blocked',
@@ -207,6 +320,7 @@ describe('integration install', () => {
     await installIntegration('codex', { preview: true, ide: 'cursor' })
 
     expect(writeFileMock).not.toHaveBeenCalled()
+    expect(writeJSONMock).not.toHaveBeenCalled()
     expect(chmodMock).not.toHaveBeenCalled()
     expect(fetchMock).not.toHaveBeenCalled()
     expect(logMock.info).toHaveBeenCalledWith('Step 1/6: Previewing Codex integration assets')
@@ -398,6 +512,20 @@ describe('integration install', () => {
     })
   })
 
+  it('describes coco using the trae skill install target', async () => {
+    const { describeIntegration } = await import('../src/commands/integration-install.js')
+
+    expect(describeIntegration('coco')).toMatchObject({
+      assistant: 'coco',
+      targets: [
+        '.trae/skills/inspecto-onboarding/SKILL.md',
+        '.trae/skills/inspecto-onboarding/scripts/run-inspecto.sh',
+      ],
+      preferredInstall:
+        'npx @inspecto-dev/cli integrations install coco --host-ide <vscode|cursor|trae|trae-cn>',
+    })
+  })
+
   it('prints integration paths without using post-install hints', async () => {
     const { printIntegrationPath } = await import('../src/commands/integration-install.js')
 

package/tests/onboard.test.ts

@@ -68,6 +68,101 @@ describe('target resolution', () => {
     expect(resolution.status).toBe('resolved')
     expect(resolution.selected?.packagePath).toBe('apps/web')
   })
+
+  it('requires explicit selection when one package has multiple supported build configs', () => {
+    const resolution = resolveOnboardingTarget({
+      repoRoot: '/repo',
+      buildTools: [
+        {
+          tool: 'vite',
+          configPath: 'finder/vite.config.mts',
+          label: 'Vite (finder/vite.config.mts)',
+          packagePath: 'finder',
+        },
+        {
+          tool: 'rspack',
+          configPath: 'finder/rspack-config/rspack.config.dev.ts',
+          label: 'Rspack (finder/rspack-config/rspack.config.dev.ts)',
+          packagePath: 'finder',
+          isLegacyRspack: true,
+        },
+      ],
+      frameworkSupportByPackage: {
+        finder: ['react'],
+      },
+    })
+
+    expect(resolution.status).toBe('needs_selection')
+    expect(resolution.candidates).toHaveLength(2)
+    expect(new Set(resolution.candidates.map(item => item.id)).size).toBe(2)
+    expect(new Set(resolution.candidates.map(item => item.candidateId)).size).toBe(2)
+    expect(resolution.candidates.map(item => item.configPath)).toEqual([
+      'finder/vite.config.mts',
+      'finder/rspack-config/rspack.config.dev.ts',
+    ])
+    expect(resolution.selectionPurpose).toContain('build target')
+    expect(resolution.selectionInstructions).toContain('--target <candidateId>')
+  })
+
+  it('resolves an explicitly selected build target by candidate id', () => {
+    const resolution = resolveOnboardingTarget({
+      repoRoot: '/repo',
+      buildTools: [
+        {
+          tool: 'vite',
+          configPath: 'finder/vite.config.mts',
+          label: 'Vite (finder/vite.config.mts)',
+          packagePath: 'finder',
+        },
+        {
+          tool: 'rspack',
+          configPath: 'finder/rspack-config/rspack.config.dev.ts',
+          label: 'Rspack (finder/rspack-config/rspack.config.dev.ts)',
+          packagePath: 'finder',
+          isLegacyRspack: true,
+        },
+      ],
+      frameworkSupportByPackage: {
+        finder: ['react'],
+      },
+      selectedPackagePath: 'finder:rspack:finder/rspack-config/rspack.config.dev.ts',
+    })
+
+    expect(resolution.status).toBe('resolved')
+    expect(resolution.selected?.buildTool).toBe('rspack')
+    expect(resolution.selected?.configPath).toBe('finder/rspack-config/rspack.config.dev.ts')
+    expect(resolution.selected?.isLegacyRspack).toBe(true)
+  })
+
+  it('resolves an explicitly selected build target by config path for root-level candidates', () => {
+    const resolution = resolveOnboardingTarget({
+      repoRoot: '/repo',
+      buildTools: [
+        {
+          tool: 'webpack',
+          configPath: 'webpack.config.common.js',
+          label: 'Webpack (webpack.config.common.js)',
+          packagePath: '',
+          isLegacyWebpack: true,
+        },
+        {
+          tool: 'webpack',
+          configPath: 'webpack.dll.config.js',
+          label: 'Webpack (webpack.dll.config.js)',
+          packagePath: '',
+          isLegacyWebpack: true,
+        },
+      ],
+      frameworkSupportByPackage: {
+        '': ['react'],
+      },
+      selectedPackagePath: 'webpack.config.common.js',
+    })
+
+    expect(resolution.status).toBe('resolved')
+    expect(resolution.selected?.configPath).toBe('webpack.config.common.js')
+    expect(resolution.selected?.buildTool).toBe('webpack')
+  })
 })
 
 describe('onboard command', () => {

package/tests/plan.test.ts

@@ -269,6 +269,108 @@ describe('planner orchestration', () => {
     ])
   })
 
+  it('returns a legacy rspack partial-manual strategy when the selected build target is legacy rspack', () => {
+    const context: OnboardingContext = {
+      root: '/repo/finder',
+      packageManager: 'pnpm',
+      buildTools: {
+        supported: [
+          {
+            tool: 'rspack',
+            configPath: 'finder/rspack-config/rspack.config.dev.ts',
+            label: 'Rspack (finder/rspack-config/rspack.config.dev.ts) [Legacy]',
+            packagePath: 'finder',
+            isLegacyRspack: true,
+          },
+        ],
+        unsupported: [],
+      },
+      frameworks: {
+        supported: ['react'],
+        unsupported: [],
+      },
+      ides: [{ ide: 'trae', supported: true }],
+      providers: [{ id: 'coco', label: 'Coco CLI', supported: true, preferredMode: 'cli' }],
+    }
+
+    const result = planner.createPlanResult(context)
+
+    expect(result.status).toBe('warning')
+    expect(result.strategy).toBe('manual')
+    expect(result.blockers).toEqual([])
+    expect(result.warnings).toEqual([
+      {
+        code: 'legacy-rspack-requires-manual-config',
+        message:
+          'Legacy Rspack detected at finder/rspack-config/rspack.config.dev.ts. Inspecto must use the legacy Rspack plugin entry and manual config steps.',
+      },
+    ])
+    expect(result.actions).toEqual([
+      {
+        type: 'install_dependency',
+        target: '@inspecto-dev/plugin @inspecto-dev/core',
+        description: 'Install the Inspecto runtime packages with pnpm.',
+      },
+      {
+        type: 'manual_step',
+        target: 'finder/rspack-config/rspack.config.dev.ts',
+        description:
+          'Update finder/rspack-config/rspack.config.dev.ts to import `rspackPlugin` from `@inspecto-dev/plugin/legacy/rspack` and add it to the Rspack plugins array.',
+      },
+    ])
+  })
+
+  it('returns a webpack 4 partial-manual strategy when the selected build target is legacy webpack', () => {
+    const context: OnboardingContext = {
+      root: '/repo/app',
+      packageManager: 'pnpm',
+      buildTools: {
+        supported: [
+          {
+            tool: 'webpack',
+            configPath: 'app/webpack.config.js',
+            label: 'Webpack (app/webpack.config.js) [Webpack 4]',
+            packagePath: 'app',
+            isLegacyWebpack: true,
+          },
+        ],
+        unsupported: [],
+      },
+      frameworks: {
+        supported: ['react'],
+        unsupported: [],
+      },
+      ides: [{ ide: 'cursor', supported: true }],
+      providers: [{ id: 'copilot', label: 'Copilot', supported: true, preferredMode: 'extension' }],
+    }
+
+    const result = planner.createPlanResult(context)
+
+    expect(result.status).toBe('warning')
+    expect(result.strategy).toBe('manual')
+    expect(result.blockers).toEqual([])
+    expect(result.warnings).toEqual([
+      {
+        code: 'legacy-webpack4-requires-manual-config',
+        message:
+          'Webpack 4 detected at app/webpack.config.js. Inspecto must use the legacy Webpack 4 plugin entry and manual config steps.',
+      },
+    ])
+    expect(result.actions).toEqual([
+      {
+        type: 'install_dependency',
+        target: '@inspecto-dev/plugin @inspecto-dev/core',
+        description: 'Install the Inspecto runtime packages with pnpm.',
+      },
+      {
+        type: 'manual_step',
+        target: 'app/webpack.config.js',
+        description:
+          'Update app/webpack.config.js to import `webpackPlugin` from `@inspecto-dev/plugin/legacy/webpack4` and add it to the Webpack plugins array.',
+      },
+    ])
+  })
+
   it('blocks root-level apply planning when multiple supported build targets are detected', () => {
     const context: OnboardingContext = {
       root: '/repo',