From a8123ab5580fbc8045aca0096c5f22e620f56275 Mon Sep 17 00:00:00 2001 From: Jeremy Date: Mon, 13 Apr 2026 06:30:21 -0500 Subject: [PATCH] fix(cli): resolve duplicate validateConfiguredModel and missing getPiDefaultModelAndProvider import Commit 110c01b8c added an inline `validateConfiguredModel` function in `src/cli.ts` while leaving the prior import from `./startup-model-validation.js` in place, producing TS2440 (import declaration conflicts with local declaration). The same commit added a call to `getPiDefaultModelAndProvider()` without importing it, producing TS2304 (cannot find name). Both errors block `npm run build` and every CI job on main. Drop the stale import and add `getPiDefaultModelAndProvider` to the existing `./pi-migration.js` import where the symbol is actually exported. The local `validateConfiguredModel` function (lines 139-174) becomes the sole definition in scope. `./startup-model-validation.js` is still consumed by its dedicated test files so the module stays. --- src/cli.ts | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/cli.ts b/src/cli.ts index 29b7c804b..ba0f204ef 100644 --- a/src/cli.ts +++ b/src/cli.ts @@ -16,8 +16,7 @@ import { agentDir, sessionsDir, authFilePath } from './app-paths.js' import { initResources, buildResourceLoader, getNewerManagedResourceVersion } from './resource-loader.js' import { ensureManagedTools } from './tool-bootstrap.js' import { loadStoredEnvKeys } from './wizard.js' -import { migratePiCredentials } from './pi-migration.js' -import { validateConfiguredModel } from './startup-model-validation.js' +import { migratePiCredentials, getPiDefaultModelAndProvider } from './pi-migration.js' import { shouldMigrateAnthropicToClaudeCode } from './provider-migrations.js' import { shouldRunOnboarding, runOnboarding } from './onboarding.js' import chalk from 'chalk'