@garygentry/system1 0.4.0 → 0.4.1

package/dist/bundle/chunks/{bin-TIEUYIP3.mjs → bin-NJIZOMS4.mjs}

@@ -6,7 +6,7 @@ import {
   VERSION,
   emit,
   extractFormat
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // src/bin.ts
 import { readFileSync } from "node:fs";
@@ -66,25 +66,25 @@ async function main(argv, io) {
   }
   switch (command) {
     case "ask":
-      return (await import("./decide-HI5CUH3C.mjs")).runAskCommand(rest, io, format);
+      return (await import("./decide-QXDMZDNT.mjs")).runAskCommand(rest, io, format);
     case "many":
-      return (await import("./decide-HI5CUH3C.mjs")).runManyCommand(rest, io, format);
+      return (await import("./decide-QXDMZDNT.mjs")).runManyCommand(rest, io, format);
     case "usage":
-      return (await import("./misc-6SDKH2PI.mjs")).runUsageCommand(rest, io, format);
+      return (await import("./misc-2SNIZHU6.mjs")).runUsageCommand(rest, io, format);
     case "config":
-      return (await import("./config-KXGUNAJR.mjs")).runConfigCommand(rest, io, format);
+      return (await import("./config-7OESN6WC.mjs")).runConfigCommand(rest, io, format);
     case "spec":
-      return (await import("./spec-GLTPEEON.mjs")).runSpecCommand(rest, io, format);
+      return (await import("./spec-56ZXLEYW.mjs")).runSpecCommand(rest, io, format);
     case "opportunities":
-      return (await import("./opportunities-ONEIYAG3.mjs")).runOpportunitiesCommand(rest, io, format);
+      return (await import("./opportunities-I6VON6X7.mjs")).runOpportunitiesCommand(rest, io, format);
     case "schema":
-      return (await import("./misc-6SDKH2PI.mjs")).runSchemaCommand(rest, io, format);
+      return (await import("./misc-2SNIZHU6.mjs")).runSchemaCommand(rest, io, format);
     case "route":
-      return (await import("./route-BF5J2NLU.mjs")).runRouteCommand(rest, io, format);
+      return (await import("./route-JQO5EDAN.mjs")).runRouteCommand(rest, io, format);
     case "ping":
-      return (await import("./misc-6SDKH2PI.mjs")).runPingCommand(io, format);
+      return (await import("./misc-2SNIZHU6.mjs")).runPingCommand(io, format);
     case "doctor":
-      return (await import("./doctor-NDV3Y6T7.mjs")).runDoctorCommand(io, format);
+      return (await import("./doctor-EXJQQ7NY.mjs")).runDoctorCommand(io, format);
     case "version":
     case "--version":
     case "-v":

package/dist/bundle/chunks/{chunk-XNRV73UE.mjs → chunk-A7OWKSHE.mjs}

@@ -2,10 +2,10 @@ import { createRequire as __decideRequire } from "node:module"; const require =
 import {
   parseFileRef,
   parseQuestionSet
-} from "./chunk-QYUSJUFX.mjs";
+} from "./chunk-CUKOJ7OM.mjs";
 import {
   DecisionsError
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // src/args.ts
 import { readFileSync } from "node:fs";

package/dist/bundle/chunks/{chunk-WZIVOY5Q.mjs → chunk-B62DSXC3.mjs}

@@ -151,7 +151,7 @@ function extractFormat(argv, fallback) {
 }
 
 // ../core/dist/version.js
-var VERSION = "0.4.0";
+var VERSION = "0.4.1";
 var CLI_PACKAGE = "@garygentry/system1";
 
 export {

package/dist/bundle/chunks/{chunk-QYUSJUFX.mjs → chunk-CUKOJ7OM.mjs}

@@ -9,7 +9,7 @@ import {
   __require,
   __toESM,
   isDecisionsError
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // ../../node_modules/.pnpm/yaml@2.9.1/node_modules/yaml/dist/nodes/identity.js
 var require_identity = __commonJS({
@@ -9598,6 +9598,7 @@ function loadConfig(options = {}) {
     repoRoot,
     apiKey: key?.value,
     apiKeySource: key?.source,
+    apiKeyQuoted: key?.quoted ?? false,
     replay: /^(1|true|yes)$/i.test(env.SYSTEM1_REPLAY ?? ""),
     session: session?.id,
     sessionOrigin: session?.origin,
@@ -9841,18 +9842,36 @@ function triggerList(value, file) {
   });
 }
 function resolveApiKey(env, userDir) {
-  const fromEnv = nonEmpty2(env.OPENROUTER_API_KEY);
+  const fromEnv = unquoteKey(env.OPENROUTER_API_KEY);
   if (fromEnv)
-    return { value: fromEnv, source: "env" };
+    return { ...fromEnv, source: "env" };
   const file = join(userDir, "credentials");
   if (!existsSync(file))
     return void 0;
   if (process.platform !== "win32" && (statSync(file).mode & 63) !== 0) {
     throw new DecisionsError("config-error", `${file} is readable by other users. Run \`chmod 600 ${file}\`.`, { file });
   }
-  const parsed = readLayer(file);
-  const value = nonEmpty2(typeof parsed?.openrouter_api_key === "string" ? parsed.openrouter_api_key : void 0);
-  return value ? { value, source: "credentials", file } : void 0;
+  let parsed;
+  try {
+    parsed = readLayer(file);
+  } catch {
+    throw new DecisionsError("config-error", `${file} is not a valid YAML mapping; it should hold one line, \`openrouter_api_key: <key>\``, { file });
+  }
+  const key = unquoteKey(typeof parsed?.openrouter_api_key === "string" ? parsed.openrouter_api_key : void 0, `openrouter_api_key in ${file}`);
+  return key ? { ...key, source: "credentials", file } : void 0;
+}
+function unquoteKey(raw, source = "OPENROUTER_API_KEY") {
+  const value = nonEmpty2(raw);
+  if (!value)
+    return void 0;
+  const quoted = /^(["']).*\1$/s.test(value);
+  const key = quoted ? nonEmpty2(value.slice(1, -1)) : value;
+  if (!key)
+    return void 0;
+  if (/[^\x21-\x7e]/.test(key)) {
+    throw new DecisionsError("config-error", `${source} contains a space, line break or other character an API key can't have; set it again`);
+  }
+  return { value: key, quoted };
 }
 function nonEmpty2(value) {
   return value === void 0 || value.trim() === "" ? void 0 : value.trim();
@@ -21082,12 +21101,23 @@ async function runDoctor(options) {
       fix: "correct or remove these keys (docs/configuration.md lists every key)"
     });
   }
-  checks.push(config.apiKey ? { name: "key", status: "ok", detail: `OPENROUTER_API_KEY present (${config.apiKeySource})` } : {
+  const credentials = join6(userConfigDir(env, options.home), "credentials");
+  checks.push(config.apiKey && config.apiKeyQuoted ? {
+    name: "key",
+    status: "warn",
+    detail: `OPENROUTER_API_KEY present (${config.apiKeySource}), but wrapped in quotes; they were removed`,
+    fix: config.apiKeySource === "env" ? "set OPENROUTER_API_KEY without the surrounding quotes" : `remove the extra quotes around openrouter_api_key in ${credentials}`,
+    advisory: true
+  } : config.apiKey ? {
+    name: "key",
+    status: "ok",
+    detail: `OPENROUTER_API_KEY present (${config.apiKeySource})`
+  } : {
     name: "key",
     status: "warn",
     detail: "no API key: only replay works",
     // The file this shell would read: XDG_CONFIG_HOME moves it.
-    fix: `set OPENROUTER_API_KEY, or write \`openrouter_api_key: <key>\` to ${join6(userConfigDir(env, options.home), "credentials")} (create its directory first; chmod 600)`
+    fix: `set OPENROUTER_API_KEY, or write \`openrouter_api_key: <key>\` to ${credentials} (create its directory first; chmod 600)`
   });
   checks.push(config.egress.consent.granted ? { name: "consent", status: "ok", detail: `egress consent granted for ${config.repoRoot}` } : {
     name: "consent",
@@ -21141,8 +21171,8 @@ function pathCheck(harness, env) {
     return {
       name: "path",
       status: "warn",
-      detail: "decide is not on PATH, so the decisions plugin's bin/ is not loaded",
-      fix: "install the decisions plugin from its marketplace, or start Claude Code with `--plugin-dir <checkout>/plugins/system1`"
+      detail: "decide is not on PATH, so the System 1 plugin's bin/ is not loaded",
+      fix: `npm i -g ${CLI_PACKAGE}@${VERSION} (also puts decide in your own terminal), or install the System 1 plugin from its marketplace`
     };
   }
   return {
@@ -21374,7 +21404,6 @@ function lintSpec(spec) {
 var NUMERIC = /* @__PURE__ */ new Set([">=", ">", "<=", "<"]);
 function lintQuestion(name, q) {
   const out = [];
-  const texts = [q.instructions, ...criteriaTexts(q)];
   if (q.type === "choice") {
     const options = Object.entries(q.criteria);
     const wayOut = ([key, desc]) => key.split(/[_\-\s/]+/).some((t) => WAY_OUT_KEY.test(t)) || WAY_OUT_TEXT.test(desc);
@@ -21636,10 +21665,17 @@ function resolveConnection(env = process.env) {
   return {
     endpoint: nonEmpty3(env.SYSTEM1_ENDPOINT) ?? DEFAULT_ENDPOINT,
     model: nonEmpty3(env.SYSTEM1_MODEL) ?? DEFAULT_MODEL,
-    apiKey: nonEmpty3(env.OPENROUTER_API_KEY),
+    apiKey: envKey(env),
     replay: /^(1|true|yes)$/i.test(env.SYSTEM1_REPLAY ?? "")
   };
 }
+function envKey(env) {
+  try {
+    return unquoteKey(env.OPENROUTER_API_KEY)?.value;
+  } catch {
+    return void 0;
+  }
+}
 function nonEmpty3(value) {
   return value === void 0 || value.trim() === "" ? void 0 : value.trim();
 }

package/dist/bundle/chunks/{config-KXGUNAJR.mjs → config-7OESN6WC.mjs}

@@ -1,7 +1,7 @@
 import { createRequire as __decideRequire } from "node:module"; const require = __decideRequire(import.meta.url);
 import {
   typedParse
-} from "./chunk-XNRV73UE.mjs";
+} from "./chunk-A7OWKSHE.mjs";
 import {
   DEFAULT_EXCLUDES,
   activeTriggers,
@@ -10,11 +10,11 @@ import {
   loadConfig,
   repoConfigPath,
   setConsent
-} from "./chunk-QYUSJUFX.mjs";
+} from "./chunk-CUKOJ7OM.mjs";
 import {
   DecisionsError,
   emit
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // src/commands/config.ts
 function runConfigCommand(argv, io, format) {

package/dist/bundle/chunks/{decide-HI5CUH3C.mjs → decide-QXDMZDNT.mjs}

@@ -6,17 +6,17 @@ import {
 import {
   buildInput,
   parseDecideFlags
-} from "./chunk-XNRV73UE.mjs";
+} from "./chunk-A7OWKSHE.mjs";
 import {
   createContext,
   runAsk,
   runMany
-} from "./chunk-QYUSJUFX.mjs";
+} from "./chunk-CUKOJ7OM.mjs";
 import {
   DecisionsError,
   ENVELOPE_VERSION,
   emit
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // src/commands/decide.ts
 import { readFileSync } from "node:fs";

package/dist/bundle/chunks/{doctor-NDV3Y6T7.mjs → doctor-EXJQQ7NY.mjs}

@@ -1,10 +1,10 @@
 import { createRequire as __decideRequire } from "node:module"; const require = __decideRequire(import.meta.url);
 import {
   runDoctor
-} from "./chunk-QYUSJUFX.mjs";
+} from "./chunk-CUKOJ7OM.mjs";
 import {
   emit
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // src/commands/doctor.ts
 import { execFile } from "node:child_process";
@@ -35,7 +35,7 @@ function brief(r) {
 }
 function headline(r) {
   if (!r.healthy) return "PROBLEMS FOUND";
-  const missing = r.checks.filter((c) => c.status === "warn").map((c) => c.name);
+  const missing = r.checks.filter((c) => c.status === "warn" && !c.advisory).map((c) => c.name);
   return !r.live && missing.length > 0 ? `SETUP NEEDED (${missing.join(", ")})` : "healthy";
 }
 function probeVersion(path, env, timeoutMs = 2e3) {

package/dist/bundle/chunks/{misc-6SDKH2PI.mjs → misc-2SNIZHU6.mjs}

@@ -1,7 +1,7 @@
 import { createRequire as __decideRequire } from "node:module"; const require = __decideRequire(import.meta.url);
 import {
   typedParse
-} from "./chunk-XNRV73UE.mjs";
+} from "./chunk-A7OWKSHE.mjs";
 import {
   TOOL_SCHEMAS,
   createContext,
@@ -10,13 +10,13 @@ import {
   probeUrl,
   resolveConnection,
   runUsage
-} from "./chunk-QYUSJUFX.mjs";
+} from "./chunk-CUKOJ7OM.mjs";
 import {
   DecisionsError,
   EXIT,
   emit,
   isDecisionsError
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // src/commands/misc.ts
 function runSchemaCommand(argv, io, format) {

package/dist/bundle/chunks/{opportunities-ONEIYAG3.mjs → opportunities-I6VON6X7.mjs}

@@ -1,17 +1,17 @@
 import { createRequire as __decideRequire } from "node:module"; const require = __decideRequire(import.meta.url);
 import {
   typedParse
-} from "./chunk-XNRV73UE.mjs";
+} from "./chunk-A7OWKSHE.mjs";
 import {
   createContext,
   runOpportunitiesAdd,
   runOpportunitiesCheck,
   runOpportunitiesList
-} from "./chunk-QYUSJUFX.mjs";
+} from "./chunk-CUKOJ7OM.mjs";
 import {
   DecisionsError,
   emit
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // src/commands/opportunities.ts
 import { readFileSync } from "node:fs";

package/dist/bundle/chunks/{route-BF5J2NLU.mjs → route-JQO5EDAN.mjs}

@@ -1,14 +1,14 @@
 import { createRequire as __decideRequire } from "node:module"; const require = __decideRequire(import.meta.url);
 import {
   typedParse
-} from "./chunk-XNRV73UE.mjs";
+} from "./chunk-A7OWKSHE.mjs";
 import {
   runRoute
-} from "./chunk-QYUSJUFX.mjs";
+} from "./chunk-CUKOJ7OM.mjs";
 import {
   DecisionsError,
   emit
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // src/commands/route.ts
 import { readFileSync } from "node:fs";

package/dist/bundle/chunks/{spec-GLTPEEON.mjs → spec-56ZXLEYW.mjs}

@@ -5,7 +5,7 @@ import {
 } from "./chunk-FU3D2MBS.mjs";
 import {
   typedParse
-} from "./chunk-XNRV73UE.mjs";
+} from "./chunk-A7OWKSHE.mjs";
 import {
   assertExamples,
   createContext,
@@ -14,12 +14,12 @@ import {
   loadSpec,
   runSpecCheck,
   runSpecLint
-} from "./chunk-QYUSJUFX.mjs";
+} from "./chunk-CUKOJ7OM.mjs";
 import {
   DecisionsError,
   EXIT,
   emit
-} from "./chunk-WZIVOY5Q.mjs";
+} from "./chunk-B62DSXC3.mjs";
 
 // src/commands/spec.ts
 function runSpecCommand(argv, io, format) {

package/dist/bundle/decide.mjs

@@ -4,4 +4,4 @@ import { createRequire as __decideRequire } from "node:module"; const require =
 // src/entry.ts
 import module from "node:module";
 module.enableCompileCache?.();
-await import("./chunks/bin-TIEUYIP3.mjs");
+await import("./chunks/bin-NJIZOMS4.mjs");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@garygentry/system1",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "The `decide` CLI: typed, calibrated decisions for coding agents, hooks and CI.",
   "type": "module",
   "license": "MIT",