argos-harness 0.1.0 → 0.2.0

package/dist/lib/assets.d.ts

@@ -1,9 +1,11 @@
 /**
- * Ordered ids of the 5 managed CLAUDE.md blocks. Order defines the order the
+ * Ordered ids of the 6 managed CLAUDE.md blocks. Order defines the order the
  * blocks appear in the rendered file. Ids are the basenames (without `.md`)
- * of `assets/managed/*.md`.
+ * of `assets/managed/*.md`. This array is the single shared source for both
+ * `init.ts` (installs/injects each block, in this order) and `doctor.ts`
+ * (audits presence/version drift for each block) — never duplicate it.
  */
-export declare const MANAGED_BLOCK_IDS: readonly ["identidad", "formato-respuesta", "aterrizaje", "orquestacion", "operaciones-seguras"];
+export declare const MANAGED_BLOCK_IDS: readonly ["identidad", "formato-respuesta", "disciplina-skills", "aterrizaje", "orquestacion", "operaciones-seguras"];
 /**
  * Resolve the Argos assets directory (`managed/`, `agents/`, `skills/`,
  * `output-styles/`) relative to the currently running module. See

package/dist/lib/assets.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"assets.d.ts","sourceRoot":"","sources":["../../src/lib/assets.ts"],"names":[],"mappings":"AAIA;;;;GAIG;AACH,eAAO,MAAM,iBAAiB,kGAMpB,CAAC;AAEX;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,GAAE,MAAwB,GAAG,MAAM,CAE1E;AAED,+DAA+D;AAC/D,wBAAgB,SAAS,CAAC,SAAS,EAAE,MAAM,EAAE,GAAG,OAAO,EAAE,MAAM,EAAE,GAAG,MAAM,CAEzE;AAED,6EAA6E;AAC7E,wBAAgB,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,EAAE,CAKxD;AAED,qEAAqE;AACrE,wBAAgB,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,EAAE,CAKxD;AAED;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,SAAS,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAY3E"}
+{"version":3,"file":"assets.d.ts","sourceRoot":"","sources":["../../src/lib/assets.ts"],"names":[],"mappings":"AAIA;;;;;;GAMG;AACH,eAAO,MAAM,iBAAiB,uHAOpB,CAAC;AAEX;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,GAAE,MAAwB,GAAG,MAAM,CAE1E;AAED,+DAA+D;AAC/D,wBAAgB,SAAS,CAAC,SAAS,EAAE,MAAM,EAAE,GAAG,OAAO,EAAE,MAAM,EAAE,GAAG,MAAM,CAEzE;AAED,6EAA6E;AAC7E,wBAAgB,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,EAAE,CAKxD;AAED,qEAAqE;AACrE,wBAAgB,YAAY,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,EAAE,CAKxD;AAED;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,SAAS,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAY3E"}

package/dist/lib/assets.js

@@ -2,12 +2,15 @@ import { readdirSync, readFileSync } from "node:fs";
 import { join } from "node:path";
 import { resolvePackageRoot } from "./package-root.js";
 /**
- * Ordered ids of the 5 managed CLAUDE.md blocks. Order defines the order the
+ * Ordered ids of the 6 managed CLAUDE.md blocks. Order defines the order the
  * blocks appear in the rendered file. Ids are the basenames (without `.md`)
- * of `assets/managed/*.md`.
+ * of `assets/managed/*.md`. This array is the single shared source for both
+ * `init.ts` (installs/injects each block, in this order) and `doctor.ts`
+ * (audits presence/version drift for each block) — never duplicate it.
  */ export const MANAGED_BLOCK_IDS = [
     "identidad",
     "formato-respuesta",
+    "disciplina-skills",
     "aterrizaje",
     "orquestacion",
     "operaciones-seguras"

package/dist/lib/assets.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../../src/lib/assets.ts"],"sourcesContent":["import { readdirSync, readFileSync } from \"node:fs\";\nimport { join } from \"node:path\";\nimport { resolvePackageRoot } from \"./package-root.js\";\n\n/**\n * Ordered ids of the 5 managed CLAUDE.md blocks. Order defines the order the\n * blocks appear in the rendered file. Ids are the basenames (without `.md`)\n * of `assets/managed/*.md`.\n */\nexport const MANAGED_BLOCK_IDS = [\n  \"identidad\",\n  \"formato-respuesta\",\n  \"aterrizaje\",\n  \"orquestacion\",\n  \"operaciones-seguras\",\n] as const;\n\n/**\n * Resolve the Argos assets directory (`managed/`, `agents/`, `skills/`,\n * `output-styles/`) relative to the currently running module. See\n * `resolvePackageRoot` for the resolution strategy.\n */\nexport function resolveAssetsDir(fromUrl: string = import.meta.url): string {\n  return join(resolvePackageRoot(fromUrl), \"assets\");\n}\n\n/** Read a text asset file relative to the assets directory. */\nexport function readAsset(assetsDir: string, ...relPath: string[]): string {\n  return readFileSync(join(assetsDir, ...relPath), \"utf-8\");\n}\n\n/** List agent ids (basenames without `.md`) found under `assets/agents/`. */\nexport function listAgentIds(assetsDir: string): string[] {\n  return readdirSync(join(assetsDir, \"agents\"))\n    .filter((f) => f.endsWith(\".md\"))\n    .map((f) => f.slice(0, -3))\n    .sort();\n}\n\n/** List skill ids (directory names) found under `assets/skills/`. */\nexport function listSkillIds(assetsDir: string): string[] {\n  return readdirSync(join(assetsDir, \"skills\"), { withFileTypes: true })\n    .filter((entry) => entry.isDirectory())\n    .map((entry) => entry.name)\n    .sort();\n}\n\n/**\n * Recursively list every file shipped under a skill's asset directory\n * (`assets/skills/<skillId>/`), as relative paths from that directory with\n * forward slashes regardless of platform (e.g. `SKILL.md`,\n * `references/core.md`, `phases/0-product.md`). Sorted for deterministic\n * output. Some skills are a single `SKILL.md`; ~15 also ship load-bearing\n * subdirectories (`references/`, `phases/`, `assets/`) that this walks too —\n * without it, only `SKILL.md` would ever reach the user's `~/.claude/skills`.\n */\nexport function listSkillFiles(assetsDir: string, skillId: string): string[] {\n  const root = join(assetsDir, \"skills\", skillId);\n  const out: string[] = [];\n  const walk = (dir: string, prefix: string) => {\n    for (const entry of readdirSync(dir, { withFileTypes: true })) {\n      const rel = prefix ? `${prefix}/${entry.name}` : entry.name;\n      if (entry.isDirectory()) walk(join(dir, entry.name), rel);\n      else out.push(rel);\n    }\n  };\n  walk(root, \"\");\n  return out.sort();\n}\n"],"names":["readdirSync","readFileSync","join","resolvePackageRoot","MANAGED_BLOCK_IDS","resolveAssetsDir","fromUrl","url","readAsset","assetsDir","relPath","listAgentIds","filter","f","endsWith","map","slice","sort","listSkillIds","withFileTypes","entry","isDirectory","name","listSkillFiles","skillId","root","out","walk","dir","prefix","rel","push"],"mappings":"AAAA,SAASA,WAAW,EAAEC,YAAY,QAAQ,UAAU;AACpD,SAASC,IAAI,QAAQ,YAAY;AACjC,SAASC,kBAAkB,QAAQ,oBAAoB;AAEvD;;;;CAIC,GACD,OAAO,MAAMC,oBAAoB;IAC/B;IACA;IACA;IACA;IACA;CACD,CAAU;AAEX;;;;CAIC,GACD,OAAO,SAASC,iBAAiBC,UAAkB,YAAYC,GAAG;IAChE,OAAOL,KAAKC,mBAAmBG,UAAU;AAC3C;AAEA,6DAA6D,GAC7D,OAAO,SAASE,UAAUC,SAAiB,EAAE,GAAGC,OAAiB;IAC/D,OAAOT,aAAaC,KAAKO,cAAcC,UAAU;AACnD;AAEA,2EAA2E,GAC3E,OAAO,SAASC,aAAaF,SAAiB;IAC5C,OAAOT,YAAYE,KAAKO,WAAW,WAChCG,MAAM,CAAC,CAACC,IAAMA,EAAEC,QAAQ,CAAC,QACzBC,GAAG,CAAC,CAACF,IAAMA,EAAEG,KAAK,CAAC,GAAG,CAAC,IACvBC,IAAI;AACT;AAEA,mEAAmE,GACnE,OAAO,SAASC,aAAaT,SAAiB;IAC5C,OAAOT,YAAYE,KAAKO,WAAW,WAAW;QAAEU,eAAe;IAAK,GACjEP,MAAM,CAAC,CAACQ,QAAUA,MAAMC,WAAW,IACnCN,GAAG,CAAC,CAACK,QAAUA,MAAME,IAAI,EACzBL,IAAI;AACT;AAEA;;;;;;;;CAQC,GACD,OAAO,SAASM,eAAed,SAAiB,EAAEe,OAAe;IAC/D,MAAMC,OAAOvB,KAAKO,WAAW,UAAUe;IACvC,MAAME,MAAgB,EAAE;IACxB,MAAMC,OAAO,CAACC,KAAaC;QACzB,KAAK,MAAMT,SAASpB,YAAY4B,KAAK;YAAET,eAAe;QAAK,GAAI;YAC7D,MAAMW,MAAMD,SAAS,GAAGA,OAAO,CAAC,EAAET,MAAME,IAAI,EAAE,GAAGF,MAAME,IAAI;YAC3D,IAAIF,MAAMC,WAAW,IAAIM,KAAKzB,KAAK0B,KAAKR,MAAME,IAAI,GAAGQ;iBAChDJ,IAAIK,IAAI,CAACD;QAChB;IACF;IACAH,KAAKF,MAAM;IACX,OAAOC,IAAIT,IAAI;AACjB"}
+{"version":3,"sources":["../../src/lib/assets.ts"],"sourcesContent":["import { readdirSync, readFileSync } from \"node:fs\";\nimport { join } from \"node:path\";\nimport { resolvePackageRoot } from \"./package-root.js\";\n\n/**\n * Ordered ids of the 6 managed CLAUDE.md blocks. Order defines the order the\n * blocks appear in the rendered file. Ids are the basenames (without `.md`)\n * of `assets/managed/*.md`. This array is the single shared source for both\n * `init.ts` (installs/injects each block, in this order) and `doctor.ts`\n * (audits presence/version drift for each block) — never duplicate it.\n */\nexport const MANAGED_BLOCK_IDS = [\n  \"identidad\",\n  \"formato-respuesta\",\n  \"disciplina-skills\",\n  \"aterrizaje\",\n  \"orquestacion\",\n  \"operaciones-seguras\",\n] as const;\n\n/**\n * Resolve the Argos assets directory (`managed/`, `agents/`, `skills/`,\n * `output-styles/`) relative to the currently running module. See\n * `resolvePackageRoot` for the resolution strategy.\n */\nexport function resolveAssetsDir(fromUrl: string = import.meta.url): string {\n  return join(resolvePackageRoot(fromUrl), \"assets\");\n}\n\n/** Read a text asset file relative to the assets directory. */\nexport function readAsset(assetsDir: string, ...relPath: string[]): string {\n  return readFileSync(join(assetsDir, ...relPath), \"utf-8\");\n}\n\n/** List agent ids (basenames without `.md`) found under `assets/agents/`. */\nexport function listAgentIds(assetsDir: string): string[] {\n  return readdirSync(join(assetsDir, \"agents\"))\n    .filter((f) => f.endsWith(\".md\"))\n    .map((f) => f.slice(0, -3))\n    .sort();\n}\n\n/** List skill ids (directory names) found under `assets/skills/`. */\nexport function listSkillIds(assetsDir: string): string[] {\n  return readdirSync(join(assetsDir, \"skills\"), { withFileTypes: true })\n    .filter((entry) => entry.isDirectory())\n    .map((entry) => entry.name)\n    .sort();\n}\n\n/**\n * Recursively list every file shipped under a skill's asset directory\n * (`assets/skills/<skillId>/`), as relative paths from that directory with\n * forward slashes regardless of platform (e.g. `SKILL.md`,\n * `references/core.md`, `phases/0-product.md`). Sorted for deterministic\n * output. Some skills are a single `SKILL.md`; ~15 also ship load-bearing\n * subdirectories (`references/`, `phases/`, `assets/`) that this walks too —\n * without it, only `SKILL.md` would ever reach the user's `~/.claude/skills`.\n */\nexport function listSkillFiles(assetsDir: string, skillId: string): string[] {\n  const root = join(assetsDir, \"skills\", skillId);\n  const out: string[] = [];\n  const walk = (dir: string, prefix: string) => {\n    for (const entry of readdirSync(dir, { withFileTypes: true })) {\n      const rel = prefix ? `${prefix}/${entry.name}` : entry.name;\n      if (entry.isDirectory()) walk(join(dir, entry.name), rel);\n      else out.push(rel);\n    }\n  };\n  walk(root, \"\");\n  return out.sort();\n}\n"],"names":["readdirSync","readFileSync","join","resolvePackageRoot","MANAGED_BLOCK_IDS","resolveAssetsDir","fromUrl","url","readAsset","assetsDir","relPath","listAgentIds","filter","f","endsWith","map","slice","sort","listSkillIds","withFileTypes","entry","isDirectory","name","listSkillFiles","skillId","root","out","walk","dir","prefix","rel","push"],"mappings":"AAAA,SAASA,WAAW,EAAEC,YAAY,QAAQ,UAAU;AACpD,SAASC,IAAI,QAAQ,YAAY;AACjC,SAASC,kBAAkB,QAAQ,oBAAoB;AAEvD;;;;;;CAMC,GACD,OAAO,MAAMC,oBAAoB;IAC/B;IACA;IACA;IACA;IACA;IACA;CACD,CAAU;AAEX;;;;CAIC,GACD,OAAO,SAASC,iBAAiBC,UAAkB,YAAYC,GAAG;IAChE,OAAOL,KAAKC,mBAAmBG,UAAU;AAC3C;AAEA,6DAA6D,GAC7D,OAAO,SAASE,UAAUC,SAAiB,EAAE,GAAGC,OAAiB;IAC/D,OAAOT,aAAaC,KAAKO,cAAcC,UAAU;AACnD;AAEA,2EAA2E,GAC3E,OAAO,SAASC,aAAaF,SAAiB;IAC5C,OAAOT,YAAYE,KAAKO,WAAW,WAChCG,MAAM,CAAC,CAACC,IAAMA,EAAEC,QAAQ,CAAC,QACzBC,GAAG,CAAC,CAACF,IAAMA,EAAEG,KAAK,CAAC,GAAG,CAAC,IACvBC,IAAI;AACT;AAEA,mEAAmE,GACnE,OAAO,SAASC,aAAaT,SAAiB;IAC5C,OAAOT,YAAYE,KAAKO,WAAW,WAAW;QAAEU,eAAe;IAAK,GACjEP,MAAM,CAAC,CAACQ,QAAUA,MAAMC,WAAW,IACnCN,GAAG,CAAC,CAACK,QAAUA,MAAME,IAAI,EACzBL,IAAI;AACT;AAEA;;;;;;;;CAQC,GACD,OAAO,SAASM,eAAed,SAAiB,EAAEe,OAAe;IAC/D,MAAMC,OAAOvB,KAAKO,WAAW,UAAUe;IACvC,MAAME,MAAgB,EAAE;IACxB,MAAMC,OAAO,CAACC,KAAaC;QACzB,KAAK,MAAMT,SAASpB,YAAY4B,KAAK;YAAET,eAAe;QAAK,GAAI;YAC7D,MAAMW,MAAMD,SAAS,GAAGA,OAAO,CAAC,EAAET,MAAME,IAAI,EAAE,GAAGF,MAAME,IAAI;YAC3D,IAAIF,MAAMC,WAAW,IAAIM,KAAKzB,KAAK0B,KAAKR,MAAME,IAAI,GAAGQ;iBAChDJ,IAAIK,IAAI,CAACD;QAChB;IACF;IACAH,KAAKF,MAAM;IACX,OAAOC,IAAIT,IAAI;AACjB"}

package/dist/lib/prompter.d.ts

@@ -0,0 +1,63 @@
+/**
+ * Interactive CLI layer (spec 0004, F5). The `Prompter` interface is the
+ * injectable seam every interactive surface (`argos init`, `argos adopt`,
+ * `argos remove`, `argos workspace link`) depends on instead of importing
+ * `@clack/prompts` directly — same idiom as `OpenclawRunner` in
+ * `lib/openclaw-agents.ts`: a typed interface, a real default implementation
+ * that's a thin pass-through over the real library, and every surface takes
+ * an optional `prompter?: Prompter` that defaults to it. Tests then inject a
+ * trivial plain-object fake instead of depending on `@clack` internals.
+ *
+ * Shaped to mirror `@clack/prompts`' actual API 1:1 (same option names,
+ * same `Value | symbol` return convention for cancellable prompts) so the
+ * default implementation below is a pure pass-through.
+ */
+export interface SelectOption<Value extends string> {
+    value: Value;
+    label?: string;
+    hint?: string;
+}
+export interface SelectPromptOptions<Value extends string> {
+    message: string;
+    options: SelectOption<Value>[];
+    initialValue?: Value;
+}
+export interface ConfirmPromptOptions {
+    message: string;
+    initialValue?: boolean;
+}
+export interface TextPromptOptions {
+    message: string;
+    placeholder?: string;
+    defaultValue?: string;
+    initialValue?: string;
+    /**
+     * Signature matches `@clack/prompts`' own `Validate<string>` (accepts
+     * `undefined` too — the prompt hasn't been submitted yet on first render)
+     * so the default implementation below is a pure pass-through.
+     */
+    validate?: (value: string | undefined) => string | Error | undefined;
+}
+export interface Prompter {
+    select<Value extends string>(opts: SelectPromptOptions<Value>): Promise<Value | symbol>;
+    confirm(opts: ConfirmPromptOptions): Promise<boolean | symbol>;
+    text(opts: TextPromptOptions): Promise<string | symbol>;
+    isCancel(value: unknown): value is symbol;
+    cancel(message?: string): void;
+    note(message?: string, title?: string): void;
+    intro(title?: string): void;
+    outro(message?: string): void;
+}
+/** Real default implementation, backed by `@clack/prompts`. */
+export declare const clackPrompter: Prompter;
+/**
+ * TTY gate every interactive surface checks independently before layering
+ * prompts on top of its own flag-driven defaults (spec 0004's hard rule):
+ * both stdout and stdin must be a real TTY, AND `--yes` must not have been
+ * passed — `--yes` forces non-interactive behavior even under a real TTY,
+ * since that's how a human operator opts out of the wizard on demand.
+ */
+export declare function isInteractive(options: {
+    yes?: boolean;
+}): boolean;
+//# sourceMappingURL=prompter.d.ts.map

package/dist/lib/prompter.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prompter.d.ts","sourceRoot":"","sources":["../../src/lib/prompter.ts"],"names":[],"mappings":"AAEA;;;;;;;;;;;;;GAaG;AAQH,MAAM,WAAW,YAAY,CAAC,KAAK,SAAS,MAAM;IAChD,KAAK,EAAE,KAAK,CAAC;IACb,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,mBAAmB,CAAC,KAAK,SAAS,MAAM;IACvD,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,YAAY,CAAC,KAAK,CAAC,EAAE,CAAC;IAC/B,YAAY,CAAC,EAAE,KAAK,CAAC;CACtB;AAED,MAAM,WAAW,oBAAoB;IACnC,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,CAAC,EAAE,OAAO,CAAC;CACxB;AAED,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB;;;;OAIG;IACH,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,GAAG,SAAS,KAAK,MAAM,GAAG,KAAK,GAAG,SAAS,CAAC;CACtE;AAED,MAAM,WAAW,QAAQ;IACvB,MAAM,CAAC,KAAK,SAAS,MAAM,EAAE,IAAI,EAAE,mBAAmB,CAAC,KAAK,CAAC,GAAG,OAAO,CAAC,KAAK,GAAG,MAAM,CAAC,CAAC;IACxF,OAAO,CAAC,IAAI,EAAE,oBAAoB,GAAG,OAAO,CAAC,OAAO,GAAG,MAAM,CAAC,CAAC;IAC/D,IAAI,CAAC,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,MAAM,GAAG,MAAM,CAAC,CAAC;IACxD,QAAQ,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,MAAM,CAAC;IAC1C,MAAM,CAAC,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,IAAI,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7C,KAAK,CAAC,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,KAAK,CAAC,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC/B;AAED,+DAA+D;AAC/D,eAAO,MAAM,aAAa,EAAE,QAkB3B,CAAC;AAEF;;;;;;GAMG;AACH,wBAAgB,aAAa,CAAC,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,OAAO,CAAA;CAAE,GAAG,OAAO,CAEjE"}

package/dist/lib/prompter.js

@@ -0,0 +1,30 @@
+import { cancel, confirm, intro, isCancel, note, outro, select, text } from "@clack/prompts";
+/** Real default implementation, backed by `@clack/prompts`. */ export const clackPrompter = {
+    // `@clack/prompts`' `Option<Value>` is a conditional type keyed on
+    // `Value extends Primitive` — TS can't reduce that conditional for our
+    // still-generic `Value` type parameter here (it only resolves once a
+    // caller picks a concrete `Value`), so it refuses to unify
+    // `SelectOption<Value>[]` with `Option<Value>[]` even though every
+    // concrete instantiation (`Value extends string`) is structurally
+    // identical. Narrow, single-purpose cast at this one boundary — not `any`
+    // on the whole function — since `opts` is otherwise fully typed above.
+    select: (opts)=>select(opts),
+    confirm: (opts)=>confirm(opts),
+    text: (opts)=>text(opts),
+    isCancel: (value)=>isCancel(value),
+    cancel: (message)=>cancel(message),
+    note: (message, title)=>note(message, title),
+    intro: (title)=>intro(title),
+    outro: (message)=>outro(message)
+};
+/**
+ * TTY gate every interactive surface checks independently before layering
+ * prompts on top of its own flag-driven defaults (spec 0004's hard rule):
+ * both stdout and stdin must be a real TTY, AND `--yes` must not have been
+ * passed — `--yes` forces non-interactive behavior even under a real TTY,
+ * since that's how a human operator opts out of the wizard on demand.
+ */ export function isInteractive(options) {
+    return process.stdout.isTTY === true && process.stdin.isTTY === true && !options.yes;
+}
+
+//# sourceMappingURL=prompter.js.map

package/dist/lib/prompter.js.map

@@ -0,0 +1 @@
+{"version":3,"sources":["../../src/lib/prompter.ts"],"sourcesContent":["import { cancel, confirm, intro, isCancel, note, outro, select, text } from \"@clack/prompts\";\n\n/**\n * Interactive CLI layer (spec 0004, F5). The `Prompter` interface is the\n * injectable seam every interactive surface (`argos init`, `argos adopt`,\n * `argos remove`, `argos workspace link`) depends on instead of importing\n * `@clack/prompts` directly — same idiom as `OpenclawRunner` in\n * `lib/openclaw-agents.ts`: a typed interface, a real default implementation\n * that's a thin pass-through over the real library, and every surface takes\n * an optional `prompter?: Prompter` that defaults to it. Tests then inject a\n * trivial plain-object fake instead of depending on `@clack` internals.\n *\n * Shaped to mirror `@clack/prompts`' actual API 1:1 (same option names,\n * same `Value | symbol` return convention for cancellable prompts) so the\n * default implementation below is a pure pass-through.\n */\n\n// `Value extends string` (rather than unconstrained) deliberately mirrors\n// every actual use of `select` in this codebase (language codes, workspace\n// names, match candidates) — it's also what makes `SelectOption` line up\n// with `@clack/prompts`' own `Option<Value>` (whose `label` is optional only\n// for `Value extends Readonly<string | boolean | number>`), so the default\n// implementation below can pass options straight through with no adapter.\nexport interface SelectOption<Value extends string> {\n  value: Value;\n  label?: string;\n  hint?: string;\n}\n\nexport interface SelectPromptOptions<Value extends string> {\n  message: string;\n  options: SelectOption<Value>[];\n  initialValue?: Value;\n}\n\nexport interface ConfirmPromptOptions {\n  message: string;\n  initialValue?: boolean;\n}\n\nexport interface TextPromptOptions {\n  message: string;\n  placeholder?: string;\n  defaultValue?: string;\n  initialValue?: string;\n  /**\n   * Signature matches `@clack/prompts`' own `Validate<string>` (accepts\n   * `undefined` too — the prompt hasn't been submitted yet on first render)\n   * so the default implementation below is a pure pass-through.\n   */\n  validate?: (value: string | undefined) => string | Error | undefined;\n}\n\nexport interface Prompter {\n  select<Value extends string>(opts: SelectPromptOptions<Value>): Promise<Value | symbol>;\n  confirm(opts: ConfirmPromptOptions): Promise<boolean | symbol>;\n  text(opts: TextPromptOptions): Promise<string | symbol>;\n  isCancel(value: unknown): value is symbol;\n  cancel(message?: string): void;\n  note(message?: string, title?: string): void;\n  intro(title?: string): void;\n  outro(message?: string): void;\n}\n\n/** Real default implementation, backed by `@clack/prompts`. */\nexport const clackPrompter: Prompter = {\n  // `@clack/prompts`' `Option<Value>` is a conditional type keyed on\n  // `Value extends Primitive` — TS can't reduce that conditional for our\n  // still-generic `Value` type parameter here (it only resolves once a\n  // caller picks a concrete `Value`), so it refuses to unify\n  // `SelectOption<Value>[]` with `Option<Value>[]` even though every\n  // concrete instantiation (`Value extends string`) is structurally\n  // identical. Narrow, single-purpose cast at this one boundary — not `any`\n  // on the whole function — since `opts` is otherwise fully typed above.\n  select: <Value extends string>(opts: SelectPromptOptions<Value>) =>\n    select(opts as Parameters<typeof select>[0]) as Promise<Value | symbol>,\n  confirm: (opts) => confirm(opts),\n  text: (opts) => text(opts),\n  isCancel: (value): value is symbol => isCancel(value),\n  cancel: (message) => cancel(message),\n  note: (message, title) => note(message, title),\n  intro: (title) => intro(title),\n  outro: (message) => outro(message),\n};\n\n/**\n * TTY gate every interactive surface checks independently before layering\n * prompts on top of its own flag-driven defaults (spec 0004's hard rule):\n * both stdout and stdin must be a real TTY, AND `--yes` must not have been\n * passed — `--yes` forces non-interactive behavior even under a real TTY,\n * since that's how a human operator opts out of the wizard on demand.\n */\nexport function isInteractive(options: { yes?: boolean }): boolean {\n  return process.stdout.isTTY === true && process.stdin.isTTY === true && !options.yes;\n}\n"],"names":["cancel","confirm","intro","isCancel","note","outro","select","text","clackPrompter","opts","value","message","title","isInteractive","options","process","stdout","isTTY","stdin","yes"],"mappings":"AAAA,SAASA,MAAM,EAAEC,OAAO,EAAEC,KAAK,EAAEC,QAAQ,EAAEC,IAAI,EAAEC,KAAK,EAAEC,MAAM,EAAEC,IAAI,QAAQ,iBAAiB;AAgE7F,6DAA6D,GAC7D,OAAO,MAAMC,gBAA0B;IACrC,mEAAmE;IACnE,uEAAuE;IACvE,qEAAqE;IACrE,2DAA2D;IAC3D,mEAAmE;IACnE,kEAAkE;IAClE,0EAA0E;IAC1E,uEAAuE;IACvEF,QAAQ,CAAuBG,OAC7BH,OAAOG;IACTR,SAAS,CAACQ,OAASR,QAAQQ;IAC3BF,MAAM,CAACE,OAASF,KAAKE;IACrBN,UAAU,CAACO,QAA2BP,SAASO;IAC/CV,QAAQ,CAACW,UAAYX,OAAOW;IAC5BP,MAAM,CAACO,SAASC,QAAUR,KAAKO,SAASC;IACxCV,OAAO,CAACU,QAAUV,MAAMU;IACxBP,OAAO,CAACM,UAAYN,MAAMM;AAC5B,EAAE;AAEF;;;;;;CAMC,GACD,OAAO,SAASE,cAAcC,OAA0B;IACtD,OAAOC,QAAQC,MAAM,CAACC,KAAK,KAAK,QAAQF,QAAQG,KAAK,CAACD,KAAK,KAAK,QAAQ,CAACH,QAAQK,GAAG;AACtF"}

package/dist/lib/settings-merge.d.ts

@@ -75,6 +75,84 @@ export interface MergeHooksOptions {
  * content, not necessarily byte-identical bytes).
  */
 export declare function mergeHooksIntoSettings(settingsPath: string, specs: ArgosHookSpec[], options?: MergeHooksOptions): SettingsMergeResult;
+/**
+ * True when `value` (a `settings.json.outputStyle` value) points at the
+ * predecessor harness's voice (`navori`) — matched EXACTLY, never by
+ * substring. Claude Code custom output styles are typically just the bare
+ * style name (e.g. `"navori"`, backed by `~/.claude/output-styles/navori.md`),
+ * but a hand-edited value could plausibly carry a path-ish form instead
+ * (`.../output-styles/navori.md`) — this also matches that shape, but only
+ * when the FINAL path segment, split on `/` or `\`, is exactly `navori.md`
+ * (case-insensitive extension). A bare `.includes("navori.md")` or
+ * `.includes("output-styles/navori")` substring check would false-positive
+ * on a user's own unrelated voice whose name merely starts with "navori"
+ * (e.g. `navori-fork.md`, `navori-team-voice`) — and since
+ * `applyOutputStylePolicy` defaults to taking this over unconditionally on
+ * the non-interactive path (`--yes`/no-TTY, the common CI/agent path), a
+ * false match there would silently clobber an unrelated user setting with
+ * zero confirmation. Anchoring to exact equality closes that hole.
+ */
+export declare function isNavoriOutputStyle(value: unknown): boolean;
+export type OutputStyleStatus = "created" | "updated" | "unchanged" | "untouched" | "error";
+export interface ApplyOutputStyleResult {
+    status: OutputStyleStatus;
+    detail?: string;
+}
+export interface ApplyOutputStyleOptions {
+    /**
+     * Whether to take over a `navori`-matched value into `"Argos"`. Only
+     * consulted when the current value matches `isNavoriOutputStyle` — ignored
+     * when the key is absent (always set) or matches some other foreign voice
+     * (never touched, regardless of this flag). Default `true`, matching the
+     * `--yes`/no-TTY "replace unconditionally" behavior from spec 0004;
+     * callers backing an interactive confirm prompt pass `false` when the user
+     * declines.
+     */
+    takeoverNavori?: boolean;
+    /** Same test-only race seam as `MergeHooksOptions.onBeforeWrite` above. */
+    onBeforeWrite?: () => void;
+}
+/**
+ * Surgical policy for `settings.json.outputStyle` (spec 0004 "Activación de
+ * la voz"), same mechanics as `mergeHooksIntoSettings`: read (missing file →
+ * `{}`), mtime captured at read time, corrupt/unexpected JSON shape → writes
+ * nothing and returns `status: "error"`, mutate in memory, re-stat right
+ * before an atomic write and bail if the file moved underneath us.
+ *
+ * - Key absent → set to `"Argos"` (`created`/`updated` depending on whether
+ *   the file itself existed).
+ * - Key already `"Argos"` → `unchanged`, no write.
+ * - Key matches the predecessor voice (`isNavoriOutputStyle`) → takeover
+ *   governed by `options.takeoverNavori` (default `true`): accepted →
+ *   overwritten to `"Argos"`, `status: "updated"` with an explicit detail
+ *   naming the takeover; declined → `status: "untouched"`, nothing written.
+ * - Key matches any other value → NEVER touched, `status: "untouched"`.
+ *
+ * Every other top-level key in `settings.json` is left exactly as found —
+ * same surgical contract as the hooks merge.
+ */
+export declare function applyOutputStylePolicy(settingsPath: string, options?: ApplyOutputStyleOptions): ApplyOutputStyleResult;
+export type RemoveOutputStyleStatus = "removed" | "unchanged" | "error";
+export interface RemoveOutputStyleResult {
+    status: RemoveOutputStyleStatus;
+    detail?: string;
+}
+export interface RemoveOutputStyleOptions {
+    /** Same `argos remove` preview-mode seam as `RemoveHooksOptions.dryRun` above. */
+    dryRun?: boolean;
+    /** Same test-only race seam as `MergeHooksOptions.onBeforeWrite` above. */
+    onBeforeWrite?: () => void;
+}
+/**
+ * `argos remove`'s mirror of `applyOutputStylePolicy`: removes
+ * `settings.json.outputStyle` ONLY when it's exactly `"Argos"` — Argos's own
+ * value. Any other value (including a foreign voice `argos init` never
+ * touched, or the key simply absent) is left exactly as found; this command
+ * only ever cleans up after itself, never after the user's own choice of
+ * voice. Same missing-file/corrupt-JSON/mtime-guard/atomic-write mechanics
+ * as the rest of this module.
+ */
+export declare function removeOutputStyleIfArgos(settingsPath: string, options?: RemoveOutputStyleOptions): RemoveOutputStyleResult;
 export type RemoveHooksStatus = "removed" | "unchanged" | "error";
 export interface RemoveHooksResult {
     status: RemoveHooksStatus;

package/dist/lib/settings-merge.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"settings-merge.d.ts","sourceRoot":"","sources":["../../src/lib/settings-merge.ts"],"names":[],"mappings":"AAsBA;;;;;;;;;GASG;AACH,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,IAAI,MAAM,CAEtE;AAqCD,MAAM,WAAW,aAAa;IAC5B,uGAAuG;IACvG,UAAU,EAAE,MAAM,CAAC;IACnB,6CAA6C;IAC7C,OAAO,EAAE,MAAM,CAAC;IAChB,mFAAmF;IACnF,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,MAAM,mBAAmB,GAAG,SAAS,GAAG,SAAS,GAAG,WAAW,GAAG,OAAO,CAAC;AAEhF,MAAM,WAAW,mBAAmB;IAClC,MAAM,EAAE,mBAAmB,CAAC;IAC5B,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,iBAAiB;IAChC;;;;;;;OAOG;IACH,iBAAiB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC7B;;;;;;OAMG;IACH,aAAa,CAAC,EAAE,MAAM,IAAI,CAAC;CAC5B;AAiBD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,wBAAgB,sBAAsB,CACpC,YAAY,EAAE,MAAM,EACpB,KAAK,EAAE,aAAa,EAAE,EACtB,OAAO,GAAE,iBAAsB,GAC9B,mBAAmB,CA8HrB;AAED,MAAM,MAAM,iBAAiB,GAAG,SAAS,GAAG,WAAW,GAAG,OAAO,CAAC;AAElE,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,iBAAiB,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,sFAAsF;IACtF,YAAY,EAAE,MAAM,CAAC;CACtB;AAED,MAAM,WAAW,kBAAkB;IACjC;;;;;OAKG;IACH,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,sEAAsE;IACtE,aAAa,CAAC,EAAE,MAAM,IAAI,CAAC;CAC5B;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH,wBAAgB,+BAA+B,CAC7C,YAAY,EAAE,MAAM,EACpB,QAAQ,EAAE,MAAM,EAChB,OAAO,GAAE,kBAAuB,GAC/B,iBAAiB,CAmGnB"}
+{"version":3,"file":"settings-merge.d.ts","sourceRoot":"","sources":["../../src/lib/settings-merge.ts"],"names":[],"mappings":"AAsBA;;;;;;;;;GASG;AACH,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,IAAI,MAAM,CAEtE;AAqCD,MAAM,WAAW,aAAa;IAC5B,uGAAuG;IACvG,UAAU,EAAE,MAAM,CAAC;IACnB,6CAA6C;IAC7C,OAAO,EAAE,MAAM,CAAC;IAChB,mFAAmF;IACnF,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,MAAM,mBAAmB,GAAG,SAAS,GAAG,SAAS,GAAG,WAAW,GAAG,OAAO,CAAC;AAEhF,MAAM,WAAW,mBAAmB;IAClC,MAAM,EAAE,mBAAmB,CAAC;IAC5B,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,iBAAiB;IAChC;;;;;;;OAOG;IACH,iBAAiB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC7B;;;;;;OAMG;IACH,aAAa,CAAC,EAAE,MAAM,IAAI,CAAC;CAC5B;AAiBD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,wBAAgB,sBAAsB,CACpC,YAAY,EAAE,MAAM,EACpB,KAAK,EAAE,aAAa,EAAE,EACtB,OAAO,GAAE,iBAAsB,GAC9B,mBAAmB,CA8HrB;AAID;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,OAAO,GAAG,OAAO,CAK3D;AAED,MAAM,MAAM,iBAAiB,GAAG,SAAS,GAAG,SAAS,GAAG,WAAW,GAAG,WAAW,GAAG,OAAO,CAAC;AAE5F,MAAM,WAAW,sBAAsB;IACrC,MAAM,EAAE,iBAAiB,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,uBAAuB;IACtC;;;;;;;;OAQG;IACH,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,2EAA2E;IAC3E,aAAa,CAAC,EAAE,MAAM,IAAI,CAAC;CAC5B;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,sBAAsB,CACpC,YAAY,EAAE,MAAM,EACpB,OAAO,GAAE,uBAA4B,GACpC,sBAAsB,CAkFxB;AAED,MAAM,MAAM,uBAAuB,GAAG,SAAS,GAAG,WAAW,GAAG,OAAO,CAAC;AAExE,MAAM,WAAW,uBAAuB;IACtC,MAAM,EAAE,uBAAuB,CAAC;IAChC,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,wBAAwB;IACvC,kFAAkF;IAClF,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,2EAA2E;IAC3E,aAAa,CAAC,EAAE,MAAM,IAAI,CAAC;CAC5B;AAED;;;;;;;;GAQG;AACH,wBAAgB,wBAAwB,CACtC,YAAY,EAAE,MAAM,EACpB,OAAO,GAAE,wBAA6B,GACrC,uBAAuB,CAkDzB;AAED,MAAM,MAAM,iBAAiB,GAAG,SAAS,GAAG,WAAW,GAAG,OAAO,CAAC;AAElE,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,iBAAiB,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,sFAAsF;IACtF,YAAY,EAAE,MAAM,CAAC;CACtB;AAED,MAAM,WAAW,kBAAkB;IACjC;;;;;OAKG;IACH,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,sEAAsE;IACtE,aAAa,CAAC,EAAE,MAAM,IAAI,CAAC;CAC5B;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH,wBAAgB,+BAA+B,CAC7C,YAAY,EAAE,MAAM,EACpB,QAAQ,EAAE,MAAM,EAChB,OAAO,GAAE,kBAAuB,GAC/B,iBAAiB,CAmGnB"}

package/dist/lib/settings-merge.js

@@ -226,6 +226,210 @@ function errorMessage(err) {
         status: existed ? "updated" : "created"
     };
 }
+// --- output-style ("voice activation") — spec 0004 "Activación de la voz" ---
+/**
+ * True when `value` (a `settings.json.outputStyle` value) points at the
+ * predecessor harness's voice (`navori`) — matched EXACTLY, never by
+ * substring. Claude Code custom output styles are typically just the bare
+ * style name (e.g. `"navori"`, backed by `~/.claude/output-styles/navori.md`),
+ * but a hand-edited value could plausibly carry a path-ish form instead
+ * (`.../output-styles/navori.md`) — this also matches that shape, but only
+ * when the FINAL path segment, split on `/` or `\`, is exactly `navori.md`
+ * (case-insensitive extension). A bare `.includes("navori.md")` or
+ * `.includes("output-styles/navori")` substring check would false-positive
+ * on a user's own unrelated voice whose name merely starts with "navori"
+ * (e.g. `navori-fork.md`, `navori-team-voice`) — and since
+ * `applyOutputStylePolicy` defaults to taking this over unconditionally on
+ * the non-interactive path (`--yes`/no-TTY, the common CI/agent path), a
+ * false match there would silently clobber an unrelated user setting with
+ * zero confirmation. Anchoring to exact equality closes that hole.
+ */ export function isNavoriOutputStyle(value) {
+    if (typeof value !== "string") return false;
+    if (value === "navori") return true;
+    const lastSegment = value.split(/[/\\]/).pop() ?? "";
+    return lastSegment.toLowerCase() === "navori.md";
+}
+/**
+ * Surgical policy for `settings.json.outputStyle` (spec 0004 "Activación de
+ * la voz"), same mechanics as `mergeHooksIntoSettings`: read (missing file →
+ * `{}`), mtime captured at read time, corrupt/unexpected JSON shape → writes
+ * nothing and returns `status: "error"`, mutate in memory, re-stat right
+ * before an atomic write and bail if the file moved underneath us.
+ *
+ * - Key absent → set to `"Argos"` (`created`/`updated` depending on whether
+ *   the file itself existed).
+ * - Key already `"Argos"` → `unchanged`, no write.
+ * - Key matches the predecessor voice (`isNavoriOutputStyle`) → takeover
+ *   governed by `options.takeoverNavori` (default `true`): accepted →
+ *   overwritten to `"Argos"`, `status: "updated"` with an explicit detail
+ *   naming the takeover; declined → `status: "untouched"`, nothing written.
+ * - Key matches any other value → NEVER touched, `status: "untouched"`.
+ *
+ * Every other top-level key in `settings.json` is left exactly as found —
+ * same surgical contract as the hooks merge.
+ */ export function applyOutputStylePolicy(settingsPath, options = {}) {
+    const existed = existsSync(settingsPath);
+    let raw = "{}";
+    let mtimeAtRead;
+    if (existed) {
+        try {
+            raw = readFileSync(settingsPath, "utf-8");
+            mtimeAtRead = statSync(settingsPath).mtimeMs;
+        } catch (err) {
+            return {
+                status: "error",
+                detail: errorMessage(err)
+            };
+        }
+    }
+    let settings;
+    try {
+        const parsed = raw.trim().length === 0 ? {} : JSON.parse(raw);
+        if (!isPlainObject(parsed)) {
+            return {
+                status: "error",
+                detail: "settings.json existente no es un objeto JSON — arreglalo a mano."
+            };
+        }
+        settings = parsed;
+    } catch (err) {
+        return {
+            status: "error",
+            detail: `settings.json existente tiene JSON inválido (${errorMessage(err)}) — arreglalo a mano y volvé a correr argos init.`
+        };
+    }
+    const current = settings.outputStyle;
+    if (current === "Argos") {
+        return {
+            status: "unchanged"
+        };
+    }
+    if (current !== undefined && !isNavoriOutputStyle(current)) {
+        return {
+            status: "untouched",
+            detail: `outputStyle ya apunta a otra voz ('${String(current)}') — no se toca`
+        };
+    }
+    if (current !== undefined && isNavoriOutputStyle(current)) {
+        const takeover = options.takeoverNavori ?? true;
+        if (!takeover) {
+            return {
+                status: "untouched",
+                detail: `outputStyle sigue en '${String(current)}' (voz del harness predecesor) — takeover declinado`
+            };
+        }
+    }
+    const previous = current;
+    settings.outputStyle = "Argos";
+    options.onBeforeWrite?.();
+    if (existed && mtimeAtRead !== undefined) {
+        let mtimeNow;
+        try {
+            mtimeNow = statSync(settingsPath).mtimeMs;
+        } catch  {
+            mtimeNow = undefined;
+        }
+        if (mtimeNow !== undefined && mtimeNow !== mtimeAtRead) {
+            return {
+                status: "error",
+                detail: "settings.json cambió durante el merge — reintenta."
+            };
+        }
+    }
+    try {
+        writeFileAtomic(settingsPath, `${JSON.stringify(settings, null, 2)}\n`);
+    } catch (err) {
+        return {
+            status: "error",
+            detail: errorMessage(err)
+        };
+    }
+    if (previous !== undefined) {
+        return {
+            status: "updated",
+            detail: `reemplazó ${String(previous)} → Argos`
+        };
+    }
+    // `previous === undefined` here means the KEY was absent (not that the
+    // file itself was absent) — always "created" regardless of `existed`,
+    // since another writer (e.g. `mergeHooksIntoSettings`, in the same
+    // `runInit` call) may have already created the file moments earlier for
+    // an unrelated reason.
+    return {
+        status: "created"
+    };
+}
+/**
+ * `argos remove`'s mirror of `applyOutputStylePolicy`: removes
+ * `settings.json.outputStyle` ONLY when it's exactly `"Argos"` — Argos's own
+ * value. Any other value (including a foreign voice `argos init` never
+ * touched, or the key simply absent) is left exactly as found; this command
+ * only ever cleans up after itself, never after the user's own choice of
+ * voice. Same missing-file/corrupt-JSON/mtime-guard/atomic-write mechanics
+ * as the rest of this module.
+ */ export function removeOutputStyleIfArgos(settingsPath, options = {}) {
+    if (!existsSync(settingsPath)) return {
+        status: "unchanged"
+    };
+    let raw;
+    let mtimeAtRead;
+    try {
+        raw = readFileSync(settingsPath, "utf-8");
+        mtimeAtRead = statSync(settingsPath).mtimeMs;
+    } catch (err) {
+        return {
+            status: "error",
+            detail: errorMessage(err)
+        };
+    }
+    let settings;
+    try {
+        const parsed = raw.trim().length === 0 ? {} : JSON.parse(raw);
+        if (!isPlainObject(parsed)) {
+            return {
+                status: "error",
+                detail: "settings.json existente no es un objeto JSON — arreglalo a mano."
+            };
+        }
+        settings = parsed;
+    } catch (err) {
+        return {
+            status: "error",
+            detail: `settings.json existente tiene JSON inválido (${errorMessage(err)}) — arreglalo a mano.`
+        };
+    }
+    if (settings.outputStyle !== "Argos") return {
+        status: "unchanged"
+    };
+    if (options.dryRun) return {
+        status: "removed"
+    };
+    delete settings.outputStyle;
+    options.onBeforeWrite?.();
+    let mtimeNow;
+    try {
+        mtimeNow = statSync(settingsPath).mtimeMs;
+    } catch  {
+        mtimeNow = undefined;
+    }
+    if (mtimeNow !== undefined && mtimeNow !== mtimeAtRead) {
+        return {
+            status: "error",
+            detail: "settings.json cambió durante el remove — reintenta."
+        };
+    }
+    try {
+        writeFileAtomic(settingsPath, `${JSON.stringify(settings, null, 2)}\n`);
+    } catch (err) {
+        return {
+            status: "error",
+            detail: errorMessage(err)
+        };
+    }
+    return {
+        status: "removed"
+    };
+}
 /**
  * Symmetric counterpart to `mergeHooksIntoSettings`: strips every Argos-owned
  * hook entry (any command matched by `isArgosOwnedHookCommand` against