argsbarg 5.1.15 → 5.1.16

package/CHANGELOG.md

@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [5.1.16] - 2026-07-07
+
+
 ## [5.1.15] - 2026-07-06
 
 
@@ -20,6 +23,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **`configure --remove-all`** — skill uninstall actions no longer reinstall skills after printing "Removing …".
 - **Interactive `configure`** — choosing `n` on an installed skill now removes the skill directory (same bug as `--remove-all`, different code path).
 - **Configure closing summary** — counts artifacts (not per-file paths) and uses Removed/Installed/Synced instead of always "Updated N file(s)."
+- **Interactive `configure` app config** — re-prompts every config entry; Enter keeps the current value.
 
 ### Changed
 
@@ -681,7 +685,8 @@ const cli = { ... } satisfies CliProgram;  // or : CliProgram
 - Migrate schemas: rename every `children` property to **`commands`**; move positional definitions to **`CliPositional`** objects on `positionals` and strip `positional` / `argMin` / `argMax` from flag definitions under `options` (flags only carry `name`, `description`, `kind`, and optional `shortName`).
 - Imports: use `CliPositional` where needed; replace `CliOptionDef` with `CliOption` or `CliPositional` as appropriate.
 
-[Unreleased]: https://github.com/bdombro/bun-argsbarg/compare/v5.1.15...HEAD
+[Unreleased]: https://github.com/bdombro/bun-argsbarg/compare/v5.1.16...HEAD
+[5.1.16]: https://github.com/bdombro/bun-argsbarg/releases/tag/v5.1.16
 [5.1.15]: https://github.com/bdombro/bun-argsbarg/releases/tag/v5.1.15
 [5.1.14]: https://github.com/bdombro/bun-argsbarg/releases/tag/v5.1.14
 [5.1.13]: https://github.com/bdombro/bun-argsbarg/releases/tag/v5.1.13

package/docs/configure.md

@@ -118,12 +118,12 @@ When `program.appConfig` is set, ArgsBarg manages schema-driven values in that f
 | Mode | Description |
 | --- | --- |
 | `configure --sync` | Bootstraps `config.json` as `{}` when missing |
-| Interactive `configure` | Config wizard when `entries` is non-empty; writes only when values or `_bindings` change |
+| Interactive `configure` | Config wizard when `entries` is non-empty; re-prompts every entry (Enter keeps current); writes only when values or `_bindings` change |
 | `--status` | Shows config path, required keys (`set` / `missing`), and binding hints (`env`, `file`, `skip`) |
 | `--remove-config --yes` | Removes the config directory |
 | `configure set --from-env` | Bind a key to its mapped env var (stores `_bindings`, no literal secret) |
 
-Per-key intent is stored under the reserved `_bindings` object (e.g. `"apiToken": "env"`). Env still wins at resolve time when set; bindings record user choice and suppress re-prompts.
+Per-key intent is stored under the reserved `_bindings` object (e.g. `"apiToken": "env"`). Env still wins at resolve time when set; bindings record user choice. CLI startup only prompts for missing required keys; interactive `configure` re-prompts all entries.
 
 Export helpers from `argsbarg`: `resolveAppConfigPath`, `displayAppConfigPath`.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "argsbarg",
-  "version": "5.1.15",
+  "version": "5.1.16",
   "type": "module",
   "engines": {
     "bun": ">=1.3"

package/src/config/bootstrap.test.ts

@@ -0,0 +1,46 @@
+import { describe, expect, test } from "bun:test";
+import type { CliAppConfigEntry } from "../types.ts";
+import { shouldWizardPromptConfigKey } from "./bootstrap.ts";
+
+const requiredEntry: CliAppConfigEntry = {
+  description: "API token.",
+  required: true,
+};
+
+describe("shouldWizardPromptConfigKey", () => {
+  test("skips addressed keys by default", () => {
+    expect(
+      shouldWizardPromptConfigKey(
+        "apiToken",
+        { apiToken: "tok", _bindings: { apiToken: "file" } },
+        requiredEntry,
+        { apiToken: "tok" },
+        {},
+      ),
+    ).toBe(false);
+  });
+
+  test("rePromptAll prompts addressed keys", () => {
+    expect(
+      shouldWizardPromptConfigKey(
+        "apiToken",
+        { apiToken: "tok", _bindings: { apiToken: "file" } },
+        requiredEntry,
+        { apiToken: "tok" },
+        { rePromptAll: true },
+      ),
+    ).toBe(true);
+  });
+
+  test("prompts when env binding is broken", () => {
+    expect(
+      shouldWizardPromptConfigKey(
+        "apiToken",
+        { _bindings: { apiToken: "env" } },
+        { ...requiredEntry, env: "API_TOKEN" },
+        {},
+        {},
+      ),
+    ).toBe(true);
+  });
+});

package/src/config/bootstrap.ts

@@ -176,6 +176,8 @@ export interface RunInstallConfigureOpts {
   context?: "standalone" | "after-install";
   /** When false, omit the "Configuration Setup" banner (e.g. inside interactive `configure`). */
   showHeading?: boolean;
+  /** When true, prompt every entry (Enter keeps the current value). */
+  rePromptAll?: boolean;
 }
 
 /** Print the "Configuration Setup" heading before the configure prompts. */
@@ -199,6 +201,24 @@ function bindingsDiffer(a: Record<string, unknown>, b: Record<string, unknown>):
   return JSON.stringify(readBindings(a)) !== JSON.stringify(readBindings(b));
 }
 
+/** Whether the configure wizard should prompt for a config key. */
+export function shouldWizardPromptConfigKey(
+  key: string,
+  fileData: Record<string, unknown>,
+  entry: CliAppConfigEntry,
+  resolved: Record<string, unknown>,
+  opts: Pick<RunInstallConfigureOpts, "rePromptAll">,
+): boolean {
+  const bindings = readBindings(fileData);
+  if (bindings[key] === "env" && !isPresent(resolved[key])) {
+    return true;
+  }
+  if (opts.rePromptAll) {
+    return true;
+  }
+  return !isKeyAddressed(key, fileData, entry);
+}
+
 /** Ask the user for each required setting that is still empty; returns updates to save to the config file. */
 function promptMissingRequired(program: CliProgram): Record<string, unknown> {
   const appConfig = program.appConfig;
@@ -270,10 +290,7 @@ export function runConfigure(
   }
 
   for (const [key, entry] of Object.entries(program.appConfig.entries)) {
-    const bindings = readBindings(existing);
-    if (bindings[key] === "env" && !isPresent(resolved[key])) {
-      // Re-prompt when env binding is broken.
-    } else if (isKeyAddressed(key, existing, entry)) {
+    if (!shouldWizardPromptConfigKey(key, existing, entry, resolved, opts)) {
       continue;
     }
 
@@ -383,7 +400,7 @@ export function ensureAppConfig(program: CliProgram, opts: EnsureAppConfigOpts):
 
   if (opts.interactive && process.stdin.isTTY) {
     if (opts.configure) {
-      runConfigure(program, { context: "standalone" });
+      runConfigure(program, { context: "standalone", rePromptAll: true });
       fileData = readAppConfigFileRaw(resolveAppConfigPath(program));
       resolved = resolveAppConfig(program, fileData, hostEnv);
       exportConfigToEnv(program, resolved, hostEnv);

package/src/configure/index.ts

@@ -272,7 +272,7 @@ async function runInteractiveConfigure(root: CliProgram, opts: ConfigureOpts): P
     if (target.key === "configure") {
       if (!appConfigHasEntries(root)) continue;
 
-      const result = runConfigure(root, { context: "standalone", showHeading: false });
+      const result = runConfigure(root, { context: "standalone", showHeading: false, rePromptAll: true });
       if (result.changed) {
         installOut(`Wrote config: ${displayAppConfigPath(root)}`, mutationOpts);
         recordArtifactMutation(summary, [displayAppConfigPath(root)], "configured");