@minniexcode/codex-switch 0.1.5 → 0.2.2

package/dist/domain/runtime-state.js

@@ -1,94 +1,6 @@
 "use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getStorageRoles = getStorageRoles;
 exports.inspectLiveStateDrift = inspectLiveStateDrift;
-const path = __importStar(require("node:path"));
-/**
- * Returns the stable storage contract used by the CLI.
- */
-function getStorageRoles(args) {
-    const toolHomeDir = path.dirname(path.resolve(args.providersPath));
-    const backupsDir = path.join(toolHomeDir, "backups");
-    const runtimeDir = args.runtimeDir ? path.resolve(args.runtimeDir) : path.join(toolHomeDir, "runtime");
-    const runtimesDir = args.runtimesDir ? path.resolve(args.runtimesDir) : path.join(toolHomeDir, "runtimes");
-    return {
-        toolHome: {
-            root: toolHomeDir,
-            toolConfig: path.join(toolHomeDir, "codex-switch.json"),
-            providers: path.resolve(args.providersPath),
-            backupsDir,
-            latestBackup: path.join(backupsDir, "latest.json"),
-            runtimeStateDir: runtimeDir,
-            runtimeInstallDir: runtimesDir,
-        },
-        targetRuntime: {
-            root: path.resolve(args.codexDir),
-            config: path.resolve(args.configPath),
-            auth: path.resolve(args.authPath),
-        },
-        managementSSOT: {
-            scope: "toolHome",
-            path: path.resolve(args.providersPath),
-        },
-        runtimeMirrors: [
-            {
-                scope: "targetRuntime",
-                path: path.resolve(args.configPath),
-            },
-        ],
-        authStateFile: {
-            scope: "targetRuntime",
-            path: path.resolve(args.authPath),
-        },
-        rollbackState: {
-            scope: "toolHome",
-            path: path.join(backupsDir, "latest.json"),
-        },
-        runtimeState: {
-            scope: "toolHome",
-            path: runtimeDir,
-            managedBackup: false,
-        },
-        runtimeInstall: {
-            scope: "toolHome",
-            path: runtimesDir,
-            managedBackup: false,
-        },
-    };
-}
 /**
  * Compares the live active model_provider against managed providers to detect drift.
  */

package/dist/interaction/add-interactive.js

@@ -2,7 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.COMMON_TAG_CHOICES = void 0;
 exports.collectAddInput = collectAddInput;
-exports.collectCopilotAddInput = collectCopilotAddInput;
 exports.createNonInteractiveAddError = createNonInteractiveAddError;
 exports.promptTags = promptTags;
 const errors_1 = require("../domain/errors");
@@ -38,51 +37,10 @@ async function collectAddInput(runtime, defaults, providerExists, profileExists)
         tags,
     };
 }
-/**
- * Collects Copilot add command inputs interactively when required values are missing.
- */
-async function collectCopilotAddInput(runtime, defaults, providerExists, profileExists, options) {
-    runtime.writeLine("Interactive add mode");
-    runtime.writeLine("Provide the missing Copilot provider fields. Press Enter to keep optional values empty.");
-    const providerName = defaults.providerName
-        ? normalizeRequiredValue(defaults.providerName)
-        : await promptProviderName(runtime, providerExists);
-    const profile = defaults.profile ? normalizeRequiredValue(defaults.profile) : await promptRequiredValue(runtime, "Profile");
-    const createProfile = !profileExists(profile);
-    const model = createProfile
-        ? defaults.model
-            ? normalizeRequiredValue(defaults.model)
-            : await promptRequiredValue(runtime, `Model for new profile "${profile}"`)
-        : defaults.model ?? null;
-    const note = defaults.note ?? normalizeOptionalValue(await runtime.inputText("Note (optional)"));
-    const tags = defaults.tags.length > 0 ? defaults.tags : await promptTags(runtime);
-    const bridgeHost = options?.bridgeHost ?? normalizeOptionalValue(await runtime.inputText("Bridge host (optional)", { defaultValue: "127.0.0.1" }));
-    const bridgePortText = options?.bridgePort !== undefined && options.bridgePort !== null
-        ? String(options.bridgePort)
-        : await runtime.inputText("Bridge port (optional)", { defaultValue: "41415" });
-    const bridgePort = normalizeBridgePort(runtime, bridgePortText);
-    const bridgeApiKey = options?.bridgeApiKey ?? normalizeOptionalValue(await runtime.inputSecret("Bridge API key (optional)"));
-    return {
-        providerName,
-        profile,
-        createProfile,
-        model,
-        note,
-        tags,
-        bridgeApiKey,
-        bridgeHost,
-        bridgePort,
-    };
-}
 /**
  * Throws a consistent error when interactive add is unavailable.
  */
-function createNonInteractiveAddError(options) {
-    if (options?.copilot) {
-        return (0, errors_1.cliError)("INVALID_ARGUMENT", "add --copilot requires <provider> and --profile when running without an interactive TTY.", {
-            suggestion: "Run in a terminal TTY or pass <provider>, --profile, and any optional Copilot bridge flags explicitly.",
-        });
-    }
+function createNonInteractiveAddError() {
     return (0, errors_1.cliError)("INVALID_ARGUMENT", "add requires <provider>, --profile, and --api-key when running without an interactive TTY.", {
         suggestion: "Run in a terminal TTY or pass all required values explicitly.",
     });
@@ -132,18 +90,6 @@ function normalizeOptionalValue(value) {
     const normalized = value.trim();
     return normalized === "" ? null : normalized;
 }
-function normalizeBridgePort(runtime, value) {
-    const normalized = value.trim();
-    if (normalized === "") {
-        return null;
-    }
-    const parsed = Number(normalized);
-    if (Number.isInteger(parsed) && parsed > 0) {
-        return parsed;
-    }
-    runtime.writeLine("Bridge port must be a positive integer. Falling back to the default port.");
-    return null;
-}
 async function promptTags(runtime, defaults = []) {
     const defaultPresetTags = defaults.filter(isCommonTag);
     return runtime.selectMany("Select tags (optional)", exports.COMMON_TAG_CHOICES.map((tag) => ({ value: tag, label: tag })), { defaultValues: defaultPresetTags });

package/dist/interaction/interactive.js

@@ -52,7 +52,6 @@ const fs = __importStar(require("node:fs"));
 const path = __importStar(require("node:path"));
 const errors_1 = require("../domain/errors");
 const backups_1 = require("../domain/backups");
-const providers_1 = require("../domain/providers");
 const runtime_state_1 = require("../domain/runtime-state");
 const codex_paths_1 = require("../storage/codex-paths");
 const config_repo_1 = require("../storage/config-repo");
@@ -81,14 +80,13 @@ async function promptForProviderSelection(runtime, providersPath, configPath, me
     const choices = Object.entries(providers.providers)
         .sort(([left], [right]) => left.localeCompare(right))
         .map(([providerName, provider]) => {
-        const providerType = (0, providers_1.isCopilotBridgeProvider)(provider) ? "copilot" : "direct";
         const currentMarker = liveState.providerResolvable && liveState.mappedProvider === providerName ? " | current" : "";
         const legacyMarker = !currentMarker && legacyCurrentProvider === providerName ? " | current" : "";
         const ambiguousMarker = !liveState.providerResolvable && liveState.mappedProviders.includes(providerName) ? " | current=ambiguous" : "";
         return {
             value: providerName,
             label: providerName,
-            hint: `profile=${provider.profile} | type=${providerType}${currentMarker}${legacyMarker}${ambiguousMarker}`,
+            hint: `profile=${provider.profile}${currentMarker}${legacyMarker}${ambiguousMarker}`,
         };
     });
     if (choices.length === 0) {

package/dist/runtime/codex-probe.js

@@ -1,19 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.codexRuntimeProbe = void 0;
 exports.probeCodexRuntime = probeCodexRuntime;
 const codex_cli_1 = require("./codex-cli");
-/**
- * Default dependency probe implementation for the local codex CLI runtime.
- */
-exports.codexRuntimeProbe = {
-    probe(options) {
-        if (options?.minVersion) {
-            return probeCodexRuntime(options.minVersion);
-        }
-        return probeCodexRuntime();
-    },
-};
 /**
  * Checks whether the codex CLI is installed and, optionally, satisfies a minimum version.
  */

package/dist/storage/codex-paths.js

@@ -88,8 +88,6 @@ function createCodexPaths(args) {
         backupsDir: path.join(toolHomeDir, "backups"),
         latestBackupPath: path.join(toolHomeDir, "backups", "latest.json"),
         lockPath: path.join(toolHomeDir, ".codex-switch.lock"),
-        runtimeDir: path.join(toolHomeDir, "runtime"),
-        runtimesDir: path.join(toolHomeDir, "runtimes"),
         codexDir,
         configPath: path.join(codexDir, "config.toml"),
         authPath: path.join(codexDir, "auth.json"),

package/docs/Design/codex-switch-v0.2.0-design.md

@@ -0,0 +1,56 @@
+# codex-switch v0.2.0 Design
+
+`0.2.0` is a major architecture release that replaces the Copilot SDK-based authentication and runtime model with a direct GitHub device-flow token exchange and HTTP proxy bridge.
+
+## Architecture Changes
+
+### Authentication: SDK-based to Device-Flow Token
+
+- The Copilot SDK (`@github/copilot-sdk`) is no longer required for authentication or session management.
+- Authentication is now handled via GitHub OAuth Device Flow (`login copilot`), which produces a GitHub personal access token stored at `<toolHomeDir>/github-token`.
+- The GitHub PAT is exchanged for a short-lived Copilot API token via `POST /copilot_internal/v2/token` before every bridge start or switch operation.
+- `TokenManager` handles background token refresh and expiry-aware caching with a new `invalidate()` method for forced refresh on upstream 401 responses.
+
+### Bridge: SDK Session to HTTP Proxy
+
+- The bridge worker (`copilot-http-bridge-worker.ts`) is now a pure HTTP reverse proxy between the local OpenAI-compatible surface and `api.githubcopilot.com`.
+- Copilot token lifecycle (exchange, refresh, invalidation) is managed inside the worker via `createTokenManager`.
+- On upstream 401, the worker invalidates its cached token and retries once with a freshly exchanged token.
+- A `createStaticTokenManager` variant supports test scenarios where the exchange should be bypassed (`CODEX_SWITCH_BRIDGE_COPILOT_TOKEN` env var).
+
+### Path Resolution: toolHomeDir Propagation
+
+- `toolHomeDir` is now explicitly threaded through all functions that read the GitHub token: `switchProvider`, `startBridge`, `getStatus`, `runDoctor`, and the bridge worker spawn.
+- The bridge worker receives the correct `toolHomeDir` (not `runtimeDir`) via `CODEX_SWITCH_TOOL_HOME_DIR`, ensuring the token is found at `<toolHomeDir>/github-token` rather than a nested runtime subdirectory.
+- `readGithubToken(toolHomeDir?)` resolves the home directory in a consistent order: explicit arg, `CODEXS_HOME` env var, then `~/.config/codex-switch`.
+
+### Provider Runtime Kind
+
+- New providers created via `add --copilot` now use `kind: "copilot-http-proxy"` instead of `kind: "copilot-sdk-bridge"`.
+- Both kinds are accepted by `isCopilotBridgeProvider` for backward compatibility with existing provider files.
+- The distinction is cosmetic; both route through the same HTTP bridge worker.
+
+### Status Contract
+
+- `getStatus` output `copilotSdk` field is simplified to `{ installed: boolean, source: string }` reflecting the presence of a GitHub token rather than an SDK install.
+- `copilotAuth` reflects the token exchange readiness rather than SDK session health.
+- Legacy fields (`installDir`, `packageName`, `packageVersion`) are removed from the status contract.
+
+## Test Infrastructure
+
+- `setCopilotTokenExchangeImplementation` / `resetCopilotTokenExchangeImplementation` provide an in-process mock for the Copilot token exchange, enabling offline test execution.
+- The bridge worker supports `CODEX_SWITCH_BRIDGE_COPILOT_TOKEN` env var to skip the real exchange in spawned child processes during tests.
+- Integration tests that previously relied on fake SDK mock responses now verify state and configuration correctness without making HTTP requests through the bridge (bridge request handling is covered by `copilot-bridge-contract.spec.js`).
+
+## Breaking Changes
+
+- `copilotSdk` status output no longer includes `installDir`, `packageName`, or `packageVersion`.
+- Provider `runtimeKind` for newly-created Copilot providers is `"copilot-http-proxy"` instead of `"copilot-sdk-bridge"`.
+- `switchProvider` requires `toolHomeDir` to be passed explicitly when operating outside the default env-var-resolved home.
+- The Copilot SDK (`@github/copilot-sdk`) is no longer a runtime dependency for authentication flows.
+
+## Non-Goals
+
+- The Copilot SDK is not removed from the repository; existing workflows that depend on it for non-auth purposes remain unchanged.
+- No migration of existing `copilot-sdk-bridge` provider records to `copilot-http-proxy`; both are accepted.
+- No changes to direct (non-Copilot) provider workflows.

package/docs/Design/codex-switch-v0.2.1-design.md

@@ -0,0 +1,77 @@
+# codex-switch v0.2.1 Design
+
+## Summary
+
+`0.2.1` narrows the current implementation to provider/model-provider management for Codex. The design removes current-facing bridge/account runtime paths and keeps a direct local-state architecture.
+
+## Command Registry
+
+The command registry contains only provider-management, config inspection, backup, rollback, and diagnostics commands. Removed command ids such as `login` and `bridge-*` are not part of the internal `CommandId` union.
+
+`setup` remains registered only as a deprecated pointer to `init` and `migrate`.
+
+## Provider Records
+
+`providers.json` remains the managed source of provider records:
+
+```json
+{
+  "providers": {
+    "packycode": {
+      "profile": "packycode",
+      "apiKey": "sk-...",
+      "model": "gpt-5",
+      "baseUrl": "https://api.example/v1"
+    }
+  }
+}
+```
+
+The stored `profile` field is the managed Codex `model_provider` id. Current user-facing docs explain `--profile` as this alias.
+
+## Codex Projection
+
+Mutating commands project provider state into the target Codex directory:
+
+```toml
+model = "gpt-5"
+model_provider = "packycode"
+
+[model_providers.packycode]
+name = "packycode"
+base_url = "https://api.example/v1"
+wire_api = "responses"
+requires_openai_auth = true
+```
+
+Auth projection writes API-key mode into `auth.json`. Managed projection does not create legacy `[profiles.*]` sections for new providers.
+
+## Output Contract
+
+Human output is provider-management-only:
+
+- `list` shows provider name, current marker, model-provider id, model, tags, and note.
+- `status` shows target Codex directory, tool home, current route, mapping state, auth/config health, warnings, and next step.
+- `doctor` shows local config/provider/auth/Codex CLI issues.
+
+JSON output keeps the standard envelope:
+
+```json
+{
+  "ok": true,
+  "command": "status",
+  "data": {},
+  "warnings": [],
+  "error": null
+}
+```
+
+## Diagnostics
+
+`status` uses route mapping and consistency checks for a lightweight operator view. `doctor` performs issue-first checks across config existence, providers existence, consistency issues, auth projection, route drift, and Codex CLI availability.
+
+No optional bridge, SDK, upstream-account, or background-service diagnostics are part of the current design.
+
+## Migration Policy
+
+Because this repository is still treated as development-version software, `0.2.1` does not add automatic migration shims for old experimental provider or bridge state. Users can manually clean old state or re-add providers.

package/docs/PRD/codex-switch-prd-v0.2.1.md

@@ -0,0 +1,82 @@
+# codex-switch v0.2.1 PRD
+
+## Summary
+
+`0.2.1` is a provider-management-only consolidation release for `@minniexcode/codex-switch`.
+
+The product is a local-first CLI that manages Codex provider/model-provider routing state. It stores local provider records, projects Codex `model_provider` definitions, switches the active top-level `model` / `model_provider` route, and provides backups, diagnostics, import/export, and rollback.
+
+## Version
+
+- Version line: `0.2.1`
+- Target package version: `0.2.1`
+- Status: current repository development line
+
+## Goals
+
+- Make the current product positioning explicit: local-first provider/model-provider management.
+- Remove current command and documentation promises for account-login and bridge runtime experiments.
+- Keep direct OpenAI-compatible provider workflows simple and inspectable.
+- Preserve current route-first Codex config projection for Codex `0.134.0+`.
+- Keep `migrate` as an advanced adopt helper for existing Codex state.
+- Keep `setup` only as a deprecated pointer.
+
+## Current Command Surface
+
+Current public commands are:
+
+- `init`
+- `migrate`
+- `list`
+- `show`
+- `current`
+- `status`
+- `config show`
+- `config list-profiles`
+- `add`
+- `edit`
+- `switch`
+- `remove`
+- `import`
+- `export`
+- `backups list`
+- `rollback`
+- `doctor`
+- `setup` as deprecated pointer
+
+## Primary Workflow
+
+```bash
+codexs init
+codexs add <provider> --profile <model-provider-id> --model <model> --api-key <key> --base-url <url>
+codexs switch <provider>
+codexs status
+codexs doctor
+```
+
+## Non-Goals
+
+`0.2.1` does not implement or reserve runtime code paths for:
+
+- GitHub Copilot SDK integration.
+- GitHub device-flow login.
+- `login copilot`.
+- `add --copilot`.
+- HTTP proxy bridge or local bridge worker commands.
+- `bridge start`, `bridge status`, or `bridge stop`.
+- Background runtime services, bridge logs, or bridge runtime state.
+- Built-in third-party router packaging.
+- Account systems or cloud sync.
+- Automatic migration of old Copilot or bridge state.
+
+A future release may introduce a third-party router-like integration, but that is outside `0.2.1` and must not be represented as current workflow, schema, or runtime behavior.
+
+## Acceptance Criteria
+
+- Package metadata reports `0.2.1`.
+- Current docs point to this PRD and the `0.2.1` design doc as fact sources.
+- Help and command registry do not expose removed command ids.
+- Human `list` output does not display provider type.
+- Human `status` and `doctor` output do not report bridge, SDK, upstream auth, or bridge-log state.
+- Interactive add collects only provider-management fields.
+- Tests cover the provider-management-only release contract.

package/docs/Tests/testing.md

@@ -1,34 +1,32 @@
-# codex-switch Testing Guide
-
-This guide records the current `0.1.x` verification contract for release and review work.
-
-The current repository line is `0.1.5` and remains an unreleased development line until an explicit release task says otherwise.
-
-## Required checks
-
-Run these commands before publishing or reviewing release changes:
-
-```bash
-npm run build
-npm test
-npx tsc --noEmit
-npm pack --dry-run
-node dist/cli.js --help
-node dist/cli.js --version
-```
-
-## Release scenarios
-
-- Fresh direct-provider flow: `init -> add -> switch -> status -> doctor`
-- Fresh Copilot flow: `init -> login copilot -> add --copilot -> switch -> status -> doctor`
-- Ambiguous active profile: `list`, `status`, and provider pickers must surface ambiguity instead of inventing a unique current provider
-- `--json` envelope: top-level `ok`, `command`, `data`, `warnings`, and `error` must remain stable
-- `migrate`: advanced adopt helper only
-- `setup`: deprecated entry only
-- Release hygiene: `package.json`, `package-lock.json`, current-line docs, changelog top entry, and current PRD/Design fact sources must agree on the `0.1.5` development line
-
-## Fixture guidance
-
-- Prefer isolated temp directories for mutating tests.
-- Keep fixtures small and realistic.
-- Use `dev-codex/local-sandbox` only when a test needs a representative Codex directory layout.
+# Testing
+
+Current version: `0.2.1`
+
+The test suite is plain Node.js. `npm test` rebuilds the CLI and runs `tests/run-tests.js`, which discovers `tests/*.spec.js` files.
+
+## Commands
+
+```bash
+npm run build
+npx tsc --noEmit
+npm test
+node dist/cli.js --help
+node dist/cli.js --version
+npm pack --dry-run
+```
+
+## Required Coverage For 0.2.1
+
+Focus on the provider-management-only contract:
+
+- Version metadata is `0.2.1` in `package.json` and `package-lock.json`.
+- Current docs point to `docs/PRD/codex-switch-prd-v0.2.1.md` and `docs/Design/codex-switch-v0.2.1-design.md`.
+- Help exposes only current commands: `init`, `migrate`, `list`, `show`, `current`, `status`, `config show`, `config list-profiles`, `add`, `edit`, `switch`, `remove`, `import`, `export`, `backups list`, `rollback`, `doctor`, and deprecated `setup`.
+- Fresh provider flow: `init -> add -> switch -> status -> doctor`.
+- Base URL drift diagnostics.
+- Ambiguous active provider mapping.
+- `migrate` remains an advanced adopt helper.
+- `setup` remains a deprecated pointer.
+- JSON output uses the stable envelope.
+
+Do not add tests for removed `0.2.1` runtime experiments such as Copilot SDK integration, GitHub login, `add --copilot`, or bridge commands.