oh-my-harness 0.16.0 → 0.17.0

package/README.md

@@ -473,6 +473,7 @@ oh-my-harness/
 - [x] Pi ([pi.dev](https://pi.dev)) emitter — bridge extension (`.pi/extensions/omh-harness.ts`) reusing the same `.omh/hooks/*.sh`
 - [x] `ask` mode — request approval before executing risky tools (Claude native prompt / Pi `ctx.ui.select`; Codex falls back to block)
 - [x] `omh uninstall` — remove generated artifacts while preserving user content
+- [x] `omh config` — view, reconfigure, or reset the saved AI provider (rotate expired keys, switch provider/model)
 - [ ] Community harness.yaml registry — share and reuse configs
 - [ ] `omh modify "change X"` — NL config editing
 

package/dist/cli/commands/config.d.ts

@@ -0,0 +1,26 @@
+import { type ProviderConfig } from "../../nl/config-store.js";
+export interface ConfigOptions {
+    /** Print the current provider config (masked) without changing anything. */
+    show?: boolean;
+    /** Delete the saved provider config. */
+    reset?: boolean;
+    /** Skip confirmation prompts. */
+    yes?: boolean;
+    /**
+     * Interactive provider setup. Injectable for tests; defaults to the clack TUI.
+     * Returns the saved config, or undefined if the user cancelled.
+     */
+    setupRunner?: () => Promise<ProviderConfig | undefined>;
+    /** Confirmation prompt. Injectable for tests; defaults to the clack TUI. */
+    confirm?: (message: string) => Promise<boolean>;
+}
+export interface ConfigResult {
+    exitCode: number;
+}
+/**
+ * `omh config`: view, reconfigure, or reset the saved AI provider used for
+ * natural-language mode (~/.omh/config.json). Without flags it shows the current
+ * config and launches the provider setup flow so a user can rotate an expired
+ * key or switch providers.
+ */
+export declare function configCommand(options?: ConfigOptions): Promise<ConfigResult>;

package/dist/cli/commands/config.js

@@ -0,0 +1,81 @@
+import chalk from "chalk";
+import { loadProviderConfig, deleteProviderConfig, maskApiKey, } from "../../nl/config-store.js";
+const NO_CONFIG_MESSAGE = "No AI provider configured yet. Run `omh config` (or `omh init`) to set one up.";
+function printSummary(config) {
+    console.log(chalk.bold("Current AI provider configuration:"));
+    console.log(`  provider: ${chalk.cyan(config.provider)}`);
+    console.log(`  method:   ${chalk.cyan(config.method)}`);
+    if (config.method === "api") {
+        console.log(`  model:    ${chalk.cyan(config.model ?? "(default)")}`);
+        console.log(`  api key:  ${chalk.dim(maskApiKey(config.apiKey))}`);
+    }
+    else {
+        console.log(`  command:  ${chalk.cyan(config.cliCommand ?? config.provider)}`);
+    }
+}
+async function defaultConfirm(message) {
+    const p = await import("@clack/prompts");
+    const answer = await p.confirm({ message });
+    if (p.isCancel(answer))
+        return false;
+    return answer === true;
+}
+async function defaultSetupRunner() {
+    const { runProviderSetup } = await import("../tui/provider-setup.js");
+    return runProviderSetup();
+}
+/**
+ * `omh config`: view, reconfigure, or reset the saved AI provider used for
+ * natural-language mode (~/.omh/config.json). Without flags it shows the current
+ * config and launches the provider setup flow so a user can rotate an expired
+ * key or switch providers.
+ */
+export async function configCommand(options = {}) {
+    // --show and --reset express contradictory intent; fail loudly rather than
+    // silently letting one win.
+    if (options.show && options.reset) {
+        console.error("`--show` and `--reset` cannot be used together.");
+        return { exitCode: 1 };
+    }
+    const existing = await loadProviderConfig();
+    // --show: read-only summary.
+    if (options.show) {
+        if (!existing) {
+            console.log(NO_CONFIG_MESSAGE);
+            return { exitCode: 0 };
+        }
+        printSummary(existing);
+        return { exitCode: 0 };
+    }
+    // --reset: delete the saved config.
+    if (options.reset) {
+        if (!existing) {
+            console.log(NO_CONFIG_MESSAGE);
+            return { exitCode: 0 };
+        }
+        const confirm = options.confirm ?? defaultConfirm;
+        const ok = options.yes ? true : await confirm("Delete the saved AI provider configuration?");
+        if (!ok) {
+            console.log("Aborted. Configuration left unchanged.");
+            return { exitCode: 0 };
+        }
+        await deleteProviderConfig();
+        console.log(chalk.green("Removed ~/.omh/config.json"));
+        return { exitCode: 0 };
+    }
+    // Default: show current config (if any), then reconfigure.
+    if (existing) {
+        printSummary(existing);
+        console.log("");
+    }
+    else {
+        console.log("No AI provider configured yet. Let's set one up.\n");
+    }
+    const setupRunner = options.setupRunner ?? defaultSetupRunner;
+    const updated = await setupRunner();
+    if (!updated) {
+        // Setup was cancelled; nothing changed.
+        return { exitCode: 0 };
+    }
+    return { exitCode: 0 };
+}

package/dist/cli/commands/doctor.d.ts

@@ -21,6 +21,13 @@ export interface DoctorResult {
      * warning unless `strict` is set.
      */
     inSync?: boolean;
+    /**
+     * Whether an AI provider is configured for natural-language mode
+     * (~/.omh/config.json). Informational only — it never affects health, since
+     * the provider is global and optional. Surfaced so users can discover
+     * `omh config` to rotate an expired key or switch provider/model.
+     */
+    providerConfigured: boolean;
     messages: string[];
 }
 export declare function doctorCommand(options?: DoctorOptions): Promise<DoctorResult>;

package/dist/cli/commands/doctor.js

@@ -3,6 +3,7 @@ import path from "node:path";
 import { parse } from "smol-toml";
 import { OMH_HOOKS_DIR } from "../../utils/paths.js";
 import { computeDrift, HarnessNotFoundError } from "../../core/drift.js";
+import { loadProviderConfig } from "../../nl/config-store.js";
 export async function doctorCommand(options = {}) {
     const projectDir = options.projectDir ?? process.cwd();
     const messages = [];
@@ -154,6 +155,10 @@ export async function doctorCommand(options = {}) {
         }
         // No harness.yaml → leave inSync undefined (drift not applicable).
     }
+    // 9. AI provider config (~/.omh/config.json). Global and optional, so this is
+    // purely informational (INFO) and never affects health — it just surfaces
+    // `omh config` so users can rotate an expired key or switch provider/model.
+    const providerConfigured = await checkProviderConfig(messages);
     const checksHealthy = Object.values(checks).every(Boolean);
     const healthy = checksHealthy && (!options.strict || inSync !== false);
     const exitCode = healthy ? 0 : 1;
@@ -169,5 +174,25 @@ export async function doctorCommand(options = {}) {
             console.log(`  ${msg}`);
         }
     }
-    return { healthy, exitCode, checks, inSync, messages };
+    return { healthy, exitCode, checks, inSync, providerConfigured, messages };
+}
+/**
+ * Inspects the global AI provider config and appends an INFO hint to `messages`.
+ * Returns whether a provider is configured. Never reads or echoes the API key.
+ */
+async function checkProviderConfig(messages) {
+    const config = await loadProviderConfig();
+    if (!config) {
+        messages.push("INFO: No AI provider configured for natural-language mode — run `omh config` to set one up.");
+        return false;
+    }
+    if (config.method === "api") {
+        const model = config.model ? `, ${config.model}` : "";
+        messages.push(`INFO: AI provider: ${config.provider} (api${model}). ` +
+            "If your API key expired, run `omh config` to update it or switch provider.");
+    }
+    else {
+        messages.push(`INFO: AI provider: ${config.provider} (cli). Run \`omh config\` to switch provider or model.`);
+    }
+    return true;
 }

package/dist/cli/index.js

@@ -87,6 +87,19 @@ export function createCli() {
             process.exitCode = result.exitCode;
         }
     });
+    program
+        .command("config")
+        .description("View, reconfigure, or reset the saved AI provider")
+        .option("--show", "Show the current provider config (API key masked)")
+        .option("--reset", "Delete the saved provider config")
+        .option("-y, --yes", "Skip confirmation prompts")
+        .action(async (options) => {
+        const { configCommand } = await import("./commands/config.js");
+        const result = await configCommand(options);
+        if (result.exitCode !== 0) {
+            process.exitCode = result.exitCode;
+        }
+    });
     program
         .command("stats")
         .description("TUI dashboard for harness analytics")

package/dist/nl/config-store.d.ts

@@ -9,3 +9,10 @@ export declare function getConfigDir(): string;
 export declare function hasProviderConfig(): Promise<boolean>;
 export declare function loadProviderConfig(): Promise<ProviderConfig | undefined>;
 export declare function saveProviderConfig(config: ProviderConfig): Promise<void>;
+export declare function deleteProviderConfig(): Promise<void>;
+/**
+ * Masks an API key for display, keeping a short prefix and suffix so the user
+ * can recognize which key is stored without revealing it. Keys short enough
+ * that a prefix/suffix would overlap are masked entirely.
+ */
+export declare function maskApiKey(apiKey: string | undefined): string;

package/dist/nl/config-store.js

@@ -34,3 +34,17 @@ export async function saveProviderConfig(config) {
     await fs.writeFile(configPath, payload, { encoding: "utf-8", mode: 0o600 });
     await fs.chmod(configPath, 0o600);
 }
+export async function deleteProviderConfig() {
+    await fs.rm(getConfigPath(), { force: true });
+}
+/**
+ * Masks an API key for display, keeping a short prefix and suffix so the user
+ * can recognize which key is stored without revealing it. Keys short enough
+ * that a prefix/suffix would overlap are masked entirely.
+ */
+export function maskApiKey(apiKey) {
+    const key = apiKey?.trim() ?? "";
+    if (key.length <= 8)
+        return "****";
+    return `${key.slice(0, 3)}…${key.slice(-4)}`;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-harness",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "description": "Tame your AI coding agents with natural language",
   "type": "module",
   "bin": {