ymmv-cli 0.11.1 → 0.11.2

package/README.md

@@ -58,8 +58,8 @@ via npm Trusted Publishing, with provenance.
   the commands to run by hand)
 - `ymmv version` prints the CLI version (and notes a newer release when one is known)
 
-Values are capped at 256 characters and extra labels at 64; a profile holds up
-to 32 extras.
+Values are capped at 256 characters and extra labels at 64, and each needs at
+least one visible character; a profile holds up to 32 extras.
 
 Every profile is open JSON too: `GET https://ymmv.fyi/api/v1/u/<handle>`. Full contract
 (shape, statuses, caching, CORS):

package/dist/cli.js

@@ -393,6 +393,12 @@ function isValidHandle(handle) {
   return handle.length >= 1 && handle.length <= 39 && HANDLE_RE.test(handle);
 }
 
+// ../shared/dist/visible.js
+var VISIBLE_RE = /[^\s\p{Default_Ignorable_Code_Point}\p{Cc}]/u;
+function hasVisibleContent(s) {
+  return VISIBLE_RE.test(s);
+}
+
 // src/render.ts
 var ESC = String.fromCharCode(27);
 var CSI = `${ESC}[`;
@@ -425,6 +431,9 @@ var BIDI_RE = new RegExp("\\p{Bidi_Control}", "gu");
 function sanitizeValue(value) {
   return value.replace(ANSI_RE, "").replace(CTRL_RE, "").replace(BIDI_RE, "");
 }
+function showsVisibleText(value) {
+  return hasVisibleContent(sanitizeValue(value));
+}
 function useColor(env, isTTY) {
   if (env.NO_COLOR !== void 0) return false;
   if (env.FORCE_COLOR !== void 0) {
@@ -597,10 +606,16 @@ function causeText(err) {
   return sanitizeValue(text);
 }
 function wireText(text) {
-  const clean = sanitizeValue(String(text));
-  return clean.length > 200 ? `${clean.slice(0, 200)}\u2026` : clean;
+  return capped(sanitizeValue(String(text)));
+}
+var WIRE_TEXT_CAP = 200;
+function head(clean) {
+  return [...clean.slice(0, WIRE_TEXT_CAP * 2)].slice(0, WIRE_TEXT_CAP).join("");
+}
+function capped(clean) {
+  const kept = head(clean);
+  return kept.length < clean.length ? `${kept}\u2026` : clean;
 }
-var INVISIBLE_RE = new RegExp("\\p{Default_Ignorable_Code_Point}", "gu");
 async function wireBody(res) {
   try {
     return await res.text();
@@ -615,8 +630,8 @@ function wireErrorBody(raw) {
     const out = {};
     if (typeof body?.error === "string") out.slug = body.error;
     if (typeof body?.message === "string") {
-      const clean = wireText(body.message).trim();
-      if (clean.replace(INVISIBLE_RE, "").trim()) out.message = clean;
+      const clean = sanitizeValue(body.message);
+      if (hasVisibleContent(head(clean))) out.message = capped(clean).trim();
     }
     return out;
   } catch {
@@ -1712,7 +1727,31 @@ function pagePointer(handle) {
   const c = palette(color);
   return ` ${c.faint}\u2192 ${color ? displayUrl(BASE) : BASE}/${handle}${c.reset}`;
 }
-async function promptEntries(defaults, prompter) {
+function writeRuleRefusal(entries, existing) {
+  const saved = savedKeys(existing);
+  for (const { key, value } of entries) {
+    if (!isCuratedKey(key)) continue;
+    const problem = valueProblem(value);
+    if (problem === void 0) continue;
+    const remedy = problem === "invisible" ? "Set one" : "Set a shorter one";
+    const source = saved.has(key) ? "saved" : "detected";
+    const remove = problem === "invisible" && source === "saved" ? `, or remove it: ymmv unset ${key}` : "";
+    return `The ${source} ${KEY_LABELS[key]} value ${ruleClause(problem, value)}. ${remedy}: ymmv set ${key} <value>${remove}.`;
+  }
+  return void 0;
+}
+function valueProblem(value) {
+  if (!showsVisibleText(value)) return "invisible";
+  if (value.length > MAX_VALUE) return "over-cap";
+  return void 0;
+}
+function ruleClause(problem, value) {
+  return problem === "invisible" ? "has no visible text" : `is ${value.length} characters; the cap is ${MAX_VALUE}`;
+}
+function savedKeys(existing) {
+  return new Set((existing?.entries ?? []).map((e) => e.key));
+}
+async function promptEntries(defaults, saved, prompter) {
   const c = palette(colorEnabled());
   console.log(message(`${c.faint}Enter to keep, "-" to clear${c.reset}`));
   const chosen = /* @__PURE__ */ new Map();
@@ -1720,13 +1759,15 @@ async function promptEntries(defaults, prompter) {
     for (; ; ) {
       const answer = (await prompter.ask(KEY_LABELS[key], defaults.get(key))).trim();
       const value = answer === "-" ? "" : answer;
-      if (value.length > MAX_VALUE) {
-        const isDefault = value === (defaults.get(key) ?? "").trim();
-        console.log(
-          message(
-            `${c.faint}${isDefault ? `the saved value is ${value.length} characters; the cap is ${MAX_VALUE}. Type a shorter value or - to clear` : `that value is ${value.length} characters; the cap is ${MAX_VALUE}`}${c.reset}`
-          )
-        );
+      const rawDefault = defaults.get(key) ?? "";
+      const isDefault = value === sanitizeValue(rawDefault).trim();
+      const emptiedDefault = answer === "" && rawDefault !== "";
+      const problem = emptiedDefault ? "invisible" : value === "" ? void 0 : valueProblem(value);
+      if (problem !== void 0) {
+        const which = `the ${saved.has(key) ? "saved" : "detected"} value`;
+        const clause = ruleClause(problem, value);
+        const note = isDefault ? `${which} ${clause}. Type a ${problem === "invisible" ? "value" : "shorter value"} or - to clear` : `that value ${clause}`;
+        console.log(message(`${c.faint}${note}${c.reset}`));
         continue;
       }
       if (value) chosen.set(key, value);
@@ -1788,26 +1829,23 @@ async function publish(io) {
   let values = defaults;
   const assemble = () => [...entriesFromMap(values), ...carried];
   const edits = /* @__PURE__ */ new Map();
+  let saved = savedKeys(existing);
   const prompt = async (prompter) => {
     const before = values;
-    values = await promptEntries(values, prompter);
+    values = await promptEntries(values, saved, prompter);
     for (const key of CURATED_KEYS) {
       const after = values.get(key);
       if (after !== before.get(key)) edits.set(key, after);
     }
   };
   if (!io.interactive || !io.prompter || io.yes) {
-    const over = [...values].find(([, v]) => v.length > MAX_VALUE);
-    if (over) {
-      console.error(
-        message(
-          `The detected ${KEY_LABELS[over[0]]} value is ${over[1].length} characters; the cap is ${MAX_VALUE}. Set a shorter one: ymmv set ${over[0]} <value>.`
-        )
-      );
+    const entries = assemble();
+    const refusal = writeRuleRefusal(entries, existing);
+    if (refusal !== void 0) {
+      console.error(message(refusal));
       process.exitCode = 1;
       return;
     }
-    const entries = assemble();
     showCard(entries);
     printPublished(
       await publishProfile(newProfile(handle, entries, extras), cred, { ifMatch }),
@@ -1817,7 +1855,9 @@ async function publish(io) {
   }
   try {
     if (!existing) await prompt(io.prompter);
+    const failsRule = ([, v]) => valueProblem(v) !== void 0;
     for (; ; ) {
+      if ([...values].some(failsRule)) await prompt(io.prompter);
       const entries = assemble();
       showCard(entries);
       const ans = await io.prompter.choice(
@@ -1854,6 +1894,7 @@ async function publish(io) {
               else rebased.set(key, value);
             }
             values = rebased;
+            saved = savedKeys(fresh.profile);
             carried = unknownEntries(fresh.profile);
             extras = fresh.profile.extras;
             ifMatch = fresh.etag;
@@ -1954,18 +1995,24 @@ async function runSet(target) {
     process.exitCode = 1;
     return;
   }
+  const refusal = writeRuleRefusal(entries, existing);
+  if (refusal !== void 0) {
+    console.error(message(refusal));
+    process.exitCode = 1;
+    return;
+  }
   const res = await publishProfile(newProfile(handle, entries, extras), cred, {
     ifMatch: own?.etag
   });
-  const line = target.kind === "curated" ? `Set ${KEY_LABELS[target.key]} = ${target.value}.` : `Set extra ${target.label} = ${target.value}.`;
+  const line = target.kind === "curated" ? `Set ${KEY_LABELS[target.key]} = ${sanitizeValue(target.value)}.` : `Set extra ${sanitizeValue(target.label)} = ${sanitizeValue(target.value)}.`;
   console.log(message(`${line}${pagePointer(res.handle)}`));
 }
 var PASTE_SAFE_LABEL = /^[\p{L}\p{N} _.+/:-]+$/u;
 function extraHint(existing, label) {
   const eq = label.indexOf("=");
-  const head = eq > 0 ? label.slice(0, eq).trim() : "";
-  if (!head || !applyUnset(existing, { kind: "extra", label: head }).removed) return "";
-  const shown = PASTE_SAFE_LABEL.test(head) ? head : "Label";
+  const head2 = eq > 0 ? label.slice(0, eq).trim() : "";
+  if (!head2 || !applyUnset(existing, { kind: "extra", label: head2 }).removed) return "";
+  const shown = PASTE_SAFE_LABEL.test(head2) ? head2 : "Label";
   return `
 (unset takes just the label: ymmv unset --extra "${shown}")`;
 }
@@ -1986,6 +2033,12 @@ async function runUnset(target) {
     console.log(message(line2));
     return;
   }
+  const refusal = writeRuleRefusal(entries, existing);
+  if (refusal !== void 0) {
+    console.error(message(refusal));
+    process.exitCode = 1;
+    return;
+  }
   const res = await publishProfile(newProfile(handle, entries, extras), cred, {
     ifMatch: own?.etag
   });
@@ -2034,10 +2087,10 @@ var SET_USAGE = `usage: ymmv set <key> <value>  |  ${SET_EXTRA}`;
 var EXTRA_USAGE = `usage: ${SET_EXTRA}`;
 var UNSET_USAGE = `usage: ymmv unset <key>  |  ${UNSET_EXTRA}`;
 var VIEW_USAGE = "usage: ymmv view <handle>";
-function invalidKeyError(head, hint) {
+function invalidKeyError(head2, hint) {
   return {
     kind: "error",
-    message: `"${sanitizeValue(head)}" is not a curated key. Valid keys: ${CURATED_KEYS.join(", ")}.
+    message: `"${sanitizeValue(head2)}" is not a curated key. Valid keys: ${CURATED_KEYS.join(", ")}.
 For anything else, use: ${hint}.`
   };
 }
@@ -2061,9 +2114,15 @@ function valueCapError(value) {
     message: `That value is ${value.length} characters; the cap is ${MAX_VALUE}.`
   };
 }
+function labelInvisibleError() {
+  return { kind: "error", message: "That label has no visible text." };
+}
+function valueInvisibleError() {
+  return { kind: "error", message: "That value has no visible text." };
+}
 function parseSet(rest) {
-  const head = rest[0];
-  if (head === "--extra" || head === "-e") {
+  const head2 = rest[0];
+  if (head2 === "--extra" || head2 === "-e") {
     const spec = rest.slice(1).join(" ").trim();
     const eq = spec.indexOf("=");
     if (eq <= 0) return { kind: "error", message: EXTRA_USAGE };
@@ -2071,30 +2130,33 @@ function parseSet(rest) {
     const value2 = spec.slice(eq + 1).trim();
     if (!label || !value2) return { kind: "error", message: EXTRA_USAGE };
     if (value2 === "-") return { kind: "unset", target: { kind: "extra", label } };
+    if (!showsVisibleText(label)) return labelInvisibleError();
+    if (!showsVisibleText(value2)) return valueInvisibleError();
     if (label.length > MAX_LABEL) return labelCapError(label);
     if (value2.length > MAX_VALUE) return valueCapError(value2);
     return { kind: "set", target: { kind: "extra", label, value: value2 } };
   }
-  if (!head) return { kind: "error", message: SET_USAGE };
-  if (!isCuratedKey(head)) return invalidKeyError(head, SET_EXTRA);
+  if (!head2) return { kind: "error", message: SET_USAGE };
+  if (!isCuratedKey(head2)) return invalidKeyError(head2, SET_EXTRA);
   const value = rest.slice(1).join(" ").trim();
-  if (!value) return { kind: "error", message: `usage: ymmv set ${head} <value>` };
-  if (value === "-") return { kind: "unset", target: { kind: "curated", key: head } };
+  if (!value) return { kind: "error", message: `usage: ymmv set ${head2} <value>` };
+  if (value === "-") return { kind: "unset", target: { kind: "curated", key: head2 } };
+  if (!showsVisibleText(value)) return valueInvisibleError();
   if (value.length > MAX_VALUE) return valueCapError(value);
-  return { kind: "set", target: { kind: "curated", key: head, value } };
+  return { kind: "set", target: { kind: "curated", key: head2, value } };
 }
 function parseUnset(rest) {
-  const head = rest[0];
-  if (head === "--extra" || head === "-e") {
+  const head2 = rest[0];
+  if (head2 === "--extra" || head2 === "-e") {
     const label = rest.slice(1).join(" ").trim();
     if (!label) return { kind: "error", message: `usage: ${UNSET_EXTRA}` };
     if (label.length > MAX_LABEL) return labelCapError(label);
     return { kind: "unset", target: { kind: "extra", label } };
   }
-  if (!head) return { kind: "error", message: UNSET_USAGE };
-  if (!isCuratedKey(head)) return invalidKeyError(head, UNSET_EXTRA);
-  if (rest.length > 1) return { kind: "error", message: `usage: ymmv unset ${head}` };
-  return { kind: "unset", target: { kind: "curated", key: head } };
+  if (!head2) return { kind: "error", message: UNSET_USAGE };
+  if (!isCuratedKey(head2)) return invalidKeyError(head2, UNSET_EXTRA);
+  if (rest.length > 1) return { kind: "error", message: `usage: ymmv unset ${head2}` };
+  return { kind: "unset", target: { kind: "curated", key: head2 } };
 }
 function resolveArg(argv) {
   const first = argv[0];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ymmv-cli",
-  "version": "0.11.1",
+  "version": "0.11.2",
   "description": "Publish and diff terminal-native developer tool-stack profiles at ymmv.fyi.",
   "type": "module",
   "bin": {