@taurgis/bonsai 3.1.0 → 3.2.2

package/dist/commands/config/set.js

@@ -1,18 +1,19 @@
 import { Args, Flags } from '@oclif/core';
 import { ConfigCommand, configScopeFlags } from './base.js';
-import { KEY_META, writeUserConfig, writeProjectConfig, validKeysHint, } from '../../lib/config/index.js';
+import { KEY_META, validKeysHint } from '../../lib/config/index.js';
+import { persistConfigPatch } from '../../lib/config-persist.js';
 export default class ConfigSet extends ConfigCommand {
     static id = 'config set';
-    static summary = 'Set a research configuration key.';
-    static description = 'Persist a configuration value. Writes user-level config by default; pass --local to write the project-level config (.bonsai.json in cwd).\n\n' +
+    static summary = 'Set a research configuration key';
+    static description = 'Persist a configuration value. Writes user-level config by default; pass --local for project-level config (.bonsai.json in cwd).\n\n' +
         `Valid keys: ${validKeysHint()}. The inline form \`<key>=<value>\` is also accepted.`;
     static examples = [
         {
-            description: 'store research cache inside the current project',
+            description: 'store research cache in the current project',
             command: '<%= config.bin %> config set storage project --local',
         },
         {
-            description: 'set the user-level default using the inline key=value form',
+            description: 'set the user-level default with key=value',
             command: '<%= config.bin %> config set storage=global',
         },
     ];
@@ -22,11 +23,11 @@ export default class ConfigSet extends ConfigCommand {
     };
     static flags = {
         ...configScopeFlags({
-            global: 'Write to user-level config (default).',
-            local: 'Write to project-level config (.bonsai.json in cwd).',
+            global: 'write user-level config (default)',
+            local: 'write project-level config (.bonsai.json in cwd)',
         }),
         'dry-run': Flags.boolean({
-            description: 'show what would be written without saving',
+            description: 'preview write without saving',
             default: false,
         }),
     };
@@ -39,6 +40,11 @@ export default class ConfigSet extends ConfigCommand {
             this.error(`Missing required argument: value for key "${keyArg}"`, {
                 exit: 2,
                 code: 'MISSING_ARGUMENT',
+                suggestions: [
+                    meta.values
+                        ? `Use one of: ${meta.values.join(', ')}`
+                        : `Provide a value for ${keyArg}: ${meta.description}`,
+                ],
             });
         }
         const parsed = meta.parseValue(valueArg);
@@ -47,35 +53,36 @@ export default class ConfigSet extends ConfigCommand {
             this.error(`Invalid value "${valueArg}" for "${keyArg}". ${guidance}`, {
                 exit: 2,
                 code: 'INVALID_VALUE',
+                suggestions: meta.values?.map((value) => `Set ${keyArg} to "${value}": ${this.config.bin} config set ${keyArg} ${value}`),
             });
         }
         const scope = this.writeScope(this.flags.local);
         const formatted = meta.format(parsed);
-        if (this.flags['dry-run']) {
+        const dryRun = this.effectiveDryRun(this.flags['dry-run']);
+        if (dryRun) {
             if (!this.jsonEnabled())
                 this.log(`[dry-run] Would set ${keyArg} = ${formatted} (${scope})`);
-            return { key: keyArg, value: parsed, scope, dryRun: true };
+            return { key: keyArg, value: parsed, scope, dryRun: true, status: 'would_set' };
         }
-        const patch = { [keyArg]: parsed };
-        if (scope === 'project') {
-            writeProjectConfig(process.cwd(), patch);
-        }
-        else {
-            const configDir = this.config.configDir;
-            if (!configDir) {
-                this.error('Could not determine user config directory. Use --local to write project config.', {
-                    exit: 1,
-                    code: 'CONFIG_DIR_UNAVAILABLE',
-                    suggestions: [
-                        `Write project config instead: ${this.config.bin} config set <key> <value> --local`,
-                    ],
-                });
-            }
-            writeUserConfig(configDir, patch);
+        const persisted = persistConfigPatch({
+            scope,
+            cwd: process.cwd(),
+            configDir: this.config.configDir,
+            patch: { [keyArg]: parsed },
+            action: 'set',
+            bin: this.config.bin,
+            key: keyArg,
+        });
+        if (!persisted.ok) {
+            this.error(persisted.message, {
+                exit: 1,
+                code: persisted.code,
+                suggestions: persisted.suggestions,
+            });
         }
         if (!this.jsonEnabled())
             this.log(`Set ${keyArg} = ${formatted} (${scope})`);
-        return { key: keyArg, value: parsed, scope, dryRun: false };
+        return { key: keyArg, value: parsed, scope, dryRun: false, status: 'set' };
     }
 }
 /**

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

@@ -1,4 +1,5 @@
 import { ConfigCommand } from './base.js';
+import type { ConfigWriteResult } from '../../lib/cli-result-types.js';
 export default class ConfigUnset extends ConfigCommand<typeof ConfigUnset> {
     static id: string;
     static summary: string;
@@ -16,5 +17,5 @@ export default class ConfigUnset extends ConfigCommand<typeof ConfigUnset> {
         local: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     static stdoutIsPrimaryData: boolean;
-    run(): Promise<unknown>;
+    run(): Promise<ConfigWriteResult>;
 }

package/dist/commands/config/unset.js

@@ -1,9 +1,10 @@
 import { Args, Flags } from '@oclif/core';
 import { ConfigCommand, configScopeFlags } from './base.js';
-import { writeUserConfig, writeProjectConfig, validKeysHint } from '../../lib/config/index.js';
+import { validKeysHint } from '../../lib/config/index.js';
+import { persistConfigPatch } from '../../lib/config-persist.js';
 export default class ConfigUnset extends ConfigCommand {
     static id = 'config unset';
-    static summary = 'Remove a research configuration key.';
+    static summary = 'Remove a research configuration key';
     static description = `Delete a key from user-level config (default) or project-level config (--local), restoring the built-in default.\n\nValid keys: ${validKeysHint()}.`;
     static examples = [
         {
@@ -20,11 +21,11 @@ export default class ConfigUnset extends ConfigCommand {
     };
     static flags = {
         ...configScopeFlags({
-            global: 'Remove from user-level config (default).',
-            local: 'Remove from project-level config (.bonsai.json in cwd).',
+            global: 'remove from user-level config (default)',
+            local: 'remove from project-level config (.bonsai.json in cwd)',
         }),
         'dry-run': Flags.boolean({
-            description: 'show what would be removed without saving',
+            description: 'preview removal without saving',
             default: false,
         }),
     };
@@ -33,30 +34,30 @@ export default class ConfigUnset extends ConfigCommand {
         const key = this.args.key;
         this.validateConfigKeyAndScope(key, this.flags.global, this.flags.local);
         const scope = this.writeScope(this.flags.local);
-        if (this.flags['dry-run']) {
+        const dryRun = this.effectiveDryRun(this.flags['dry-run']);
+        if (dryRun) {
             if (!this.jsonEnabled())
                 this.log(`[dry-run] Would unset ${key} (${scope})`);
-            return { key, scope, dryRun: true };
+            return { key, scope, dryRun: true, status: 'would_unset' };
         }
-        const patch = { [key]: undefined };
-        if (scope === 'project') {
-            writeProjectConfig(process.cwd(), patch);
-        }
-        else {
-            const configDir = this.config.configDir;
-            if (!configDir) {
-                this.error('Could not determine user config directory. Use --local to remove project config.', {
-                    exit: 1,
-                    code: 'CONFIG_DIR_UNAVAILABLE',
-                    suggestions: [
-                        `Remove project config instead: ${this.config.bin} config unset ${key} --local`,
-                    ],
-                });
-            }
-            writeUserConfig(configDir, patch);
+        const persisted = persistConfigPatch({
+            scope,
+            cwd: process.cwd(),
+            configDir: this.config.configDir,
+            patch: { [key]: undefined },
+            action: 'unset',
+            bin: this.config.bin,
+            key,
+        });
+        if (!persisted.ok) {
+            this.error(persisted.message, {
+                exit: 1,
+                code: persisted.code,
+                suggestions: persisted.suggestions,
+            });
         }
         if (!this.jsonEnabled())
             this.log(`Unset ${key} (${scope})`);
-        return { key, scope, dryRun: false };
+        return { key, scope, dryRun: false, status: 'unset' };
     }
 }

package/dist/commands/fetch.d.ts

@@ -1,4 +1,5 @@
 import { BaseCommand } from '../base-command.js';
+/** Hidden named `fetch` command; usually invoked as `bonsai <url>`. */
 export default class FetchCommand extends BaseCommand<typeof FetchCommand> {
     static id: string;
     static hidden: boolean;
@@ -26,20 +27,11 @@ export default class FetchCommand extends BaseCommand<typeof FetchCommand> {
         storage: import("@oclif/core/interfaces").OptionFlag<"global" | "project" | undefined, import("@oclif/core/interfaces").CustomOptions>;
     };
     static stdoutIsPrimaryData: boolean;
-    protected envelopeCommandId(): string;
-    private executeCacheHit;
-    private executeCacheMiss;
-    private buildResultData;
-    private validateDurationFlags;
-    private emitFetchError;
-    private resolveArtifact;
-    private persistFreshArtifact;
-    private persistSectionsIfFresh;
+    /**
+     * When a multi-URL batch has any per-URL failure, keep the result array (including hits) and
+     * surface FETCH_FAILED on the envelope — same batch contract as status/inspect CACHE_MISS.
+     */
+    protected toSuccessJson(data: unknown): Record<string, unknown>;
+    private serviceFlags;
     run(): Promise<unknown>;
-    private fetchSingleTarget;
 }
-export declare function describeError(err: unknown): string;
-export declare function fetchFailureGuidance(message: string, url: string): {
-    suggestions: string[];
-    ref?: string;
-} | undefined;