@theholocron/cli 5.0.0-alpha.182 → 5.0.0-alpha.184

package/dist/cli.mjs

@@ -2410,6 +2410,7 @@ async function runDeployFromFiles(input) {
 *   errors   → `whoami()` (org reachability) or `describe()`
 *   logs     → `whoami()` (dataset reachability) or `describe()`
 *   wiki     → `whoami()` (token presence only — not a live check, see holocron#1079)
+*   notifications → `whoami()` when the provider provisions (bot token + server access); otherwise just "loaded"
 *   others   → just "loaded" (no smoke endpoint defined yet)
 *
 * Output is plain text via a passed-in `print` function so tests can
@@ -2435,15 +2436,13 @@ async function runDoctor(input) {
 		const cardinality = CARDINALITY[key];
 		const entry = config.providers[key];
 		const providers = entry?.cardinality === "many" ? entry.tuples.map((t) => t.provider) : entry?.cardinality === "single" ? [entry.tuple.provider] : [];
-		if (cardinality === "many") loader.get(key).forEach((_impl, idx) => {
-			rows.push({
-				capability: key,
-				provider: providers[idx] ?? "?",
-				status: "skip",
-				message: "loaded (no smoke check defined for this capability)"
-			});
-		});
-		else {
+		if (cardinality === "many") {
+			const impls = loader.get(key);
+			for (const [idx, impl] of impls.entries()) {
+				const name = impl.providerName ?? providers[idx];
+				rows.push(await smokeCheck(key, name, impl));
+			}
+		} else {
 			const impl = loader.get(key);
 			const row = await smokeCheck(key, providers[0] ?? "?", impl);
 			rows.push(row);
@@ -2499,11 +2498,11 @@ async function smokeCheck(key, provider, impl) {
 }
 /** Vendor response bodies land in the terminal — cap their length so one
 *  oversized error can't flood the report, and strip control characters
-*  (newlines included) so a crafted body can't forge extra report lines
-*  or inject terminal escape sequences. */
+*  (newlines and the C1 range, whose 0x9b is an 8-bit CSI) so a crafted body
+*  can't forge extra report lines or inject terminal escape sequences. */
 const MAX_DETAIL_LENGTH = 200;
 function sanitizeDetail(text) {
-	const stripped = text.replace(/[\x00-\x1f\x7f]/g, " ").replace(/\s+/g, " ").trim();
+	const stripped = text.replace(/[\x00-\x1f\x7f-\x9f]/g, " ").replace(/\s+/g, " ").trim();
 	return stripped.length > MAX_DETAIL_LENGTH ? `${stripped.slice(0, MAX_DETAIL_LENGTH)}…` : stripped;
 }
 /**
@@ -2529,19 +2528,19 @@ async function runSmokeCheck(key, provider, impl) {
 			const report = await impl.doctor();
 			return mk(key, provider, "ok", `${report.authedAs} / ${report.projectLabel} (${countResolvedLifecycle(report)})`);
 		}
-		case "secrets": return mk(key, provider, "ok", `repo secrets: ${(await impl.listSecrets({ kind: "repo" })).length} configured`);
+		case "secrets": return mk(key, provider, "ok", `secrets: ${(await impl.listSecrets({ kind: "repo" })).length} configured`);
 		case "ci": {
 			const runs = await impl.listRuns({ limit: 1 });
 			return mk(key, provider, "ok", runs.length === 0 ? "no recent runs" : `most recent: ${runs[0].workflowName} (${runs[0].status})`);
 		}
-		case "vault": return mk(key, provider, "ok", `${(await impl.list()).length} keys available`);
-		case "deployment": return mk(key, provider, "ok", `${(await impl.listProjects()).length} project(s) visible`);
-		case "environments": return mk(key, provider, "ok", `${(await impl.listEnvironments()).length} environment(s) configured`);
+		case "vault": return mk(key, provider, "ok", `keys: ${(await impl.list()).length} visible`);
+		case "deployment": return mk(key, provider, "ok", `projects: ${(await impl.listProjects()).length} visible`);
+		case "environments": return mk(key, provider, "ok", `environments: ${(await impl.listEnvironments()).length} configured`);
 		case "dns": {
 			const dns = impl;
 			if (dns.whoami) {
 				const who = await dns.whoami();
-				return mk(key, provider, who.ok ? "ok" : "fail", `${who.zones} zone(s) visible`);
+				return mk(key, provider, who.ok ? "ok" : "fail", `zones: ${who.zones} visible`);
 			}
 			return mk(key, provider, "skip", "loaded (no smoke check defined for this capability)");
 		}
@@ -2549,7 +2548,7 @@ async function runSmokeCheck(key, provider, impl) {
 			const workers = impl;
 			if (workers.whoami) {
 				const who = await workers.whoami();
-				return mk(key, provider, who.ok ? "ok" : "fail", `${who.scripts} script(s) visible`);
+				return mk(key, provider, who.ok ? "ok" : "fail", `scripts: ${who.scripts} visible`);
 			}
 			return mk(key, provider, "skip", "loaded (no smoke check defined for this capability)");
 		}
@@ -2582,6 +2581,12 @@ async function runSmokeCheck(key, provider, impl) {
 			}
 			return mk(key, provider, "skip", "loaded (no smoke check defined for this capability)");
 		}
+		case "notifications": {
+			const notifications = impl;
+			const who = notifications.whoami ? await notifications.whoami() : null;
+			if (who) return mk(key, provider, "ok", `server: ${sanitizeDetail(who.target)}`);
+			return mk(key, provider, "skip", "loaded (no smoke check defined for this capability)");
+		}
 		case "wiki": {
 			const wiki = impl;
 			if (wiki.whoami) {