diff --git a/packages/extension/src/credential_scanner.ts b/packages/extension/src/credential_scanner.ts index 5601a7f2..c79d4e53 100644 --- a/packages/extension/src/credential_scanner.ts +++ b/packages/extension/src/credential_scanner.ts @@ -14,7 +14,7 @@ import * as fs from "node:fs"; import * as path from "node:path"; import * as os from "node:os"; -import { PROVIDER_MODELS, writeOnboardingConfig } from "./onboarding_panel"; +import { PROVIDER_MODELS } from "./onboarding_panel"; // ─── Types ─────────────────────────────────────────────────────────────────── diff --git a/packages/extension/src/extension.ts b/packages/extension/src/extension.ts index 05b681fd..c370b24f 100644 --- a/packages/extension/src/extension.ts +++ b/packages/extension/src/extension.ts @@ -40,7 +40,7 @@ import { watchSolverMode, applyEntitlementForMode, readSolverModeState } from ". import { runSetCloudKeyCommand } from "./cloud_key"; import { amicodeOpsDir } from "./substrate/vault_store"; import { registerOnboardingPanel, onOnboardingComplete, onOnboardingCancelled } from "./onboarding_panel"; -import { isModelConfigured, writeWelcomeShown } from "./onboarding_routing"; +import { isModelConfigured } from "./onboarding_routing"; import { stagePasqalConnector } from "./pasqal_assets"; import { needsProvision, pasqalVenvDir, provisionPasqalPython } from "./pasqal_python"; import { createLocalPersonalVault, sanitizeVaultName, suggestVaultName } from "./substrate/vault_setup"; diff --git a/packages/extension/src/onboarding_webview.ts b/packages/extension/src/onboarding_webview.ts index 9484ee6e..1807224c 100644 --- a/packages/extension/src/onboarding_webview.ts +++ b/packages/extension/src/onboarding_webview.ts @@ -624,7 +624,6 @@ function buildForm(): void { const importLink = document.getElementById("import-link") as HTMLAnchorElement; const importStatus = document.getElementById("import-status") as HTMLDivElement; const importPreview = document.getElementById("import-preview") as HTMLDivElement; - const importSection = document.getElementById("import-section") as HTMLDivElement; importLink.addEventListener("click", (e) => { e.preventDefault(); diff --git a/packages/extension/test/credential_scanner.test.ts b/packages/extension/test/credential_scanner.test.ts index 3b670761..464e0b45 100644 --- a/packages/extension/test/credential_scanner.test.ts +++ b/packages/extension/test/credential_scanner.test.ts @@ -473,6 +473,14 @@ describe("scanCredentials — end-to-end with real default paths", () => { console.log(` ${c.provider} (from ${c.source}) — key ${c.key.slice(0, 6)}...`); } + // A CI runner has no real opencode install — the e2e assertion only makes + // sense on a dev machine with credentials (same skip idiom as the + // testConnection case below; #450 shipped this unguarded and CI went red). + if (result.credentials.length === 0) { + console.log(" [e2e] No credentials on this machine — skipping live assertion"); + return; + } + expect(result.credentials.length).toBeGreaterThan(0); // Should find opencode since account.json has opencode-go / opencode entries