@letterstory/cli 0.29.1 → 0.29.3

package/lib/cli.mjs

@@ -40,10 +40,11 @@ import {
 	cmdLede,
 	cmdLandings,
 	cmdAdmin,
+	cmdFeedback,
 } from "./commands.mjs";
 
 // Keep in sync with cli/package.json.
-export const VERSION = "0.29.1";
+export const VERSION = "0.29.3";
 
 // Flags that never take a value. Listing them explicitly means `deploy get --json <id>`
 // can't accidentally swallow the id as --json's value.
@@ -581,6 +582,11 @@ Insights:
   insights post <article-id> [--period 14d|30d|90d]
   insights top [--period 14d|30d|90d] [--collection <uuid>] [--limit <n>] [--sort clicks|impressions]
 
+Feedback (tell us what's broken, missing, unclear, or wanted):
+  feedback submit --category bug|missing_feature|unclear|feature_request|praise|other
+                  [--message <text>|--body <text>|--file <path>|--file -] [--rating 1-5]
+                  [--context '<json>']
+
 Anything else:
   tools list                         List every tool this server exposes, by name
   tools show <name>                  Show one tool's capability, argument schema, and return shape
@@ -645,6 +651,7 @@ const CLIENT_COMMANDS = {
 	lede: cmdLede,
 	ledes: cmdLede,
 	landings: cmdLandings,
+	feedback: cmdFeedback,
 	// Platform-scoped (staff key) — the only commands that act BEFORE an org exists.
 	admin: cmdAdmin,
 };

package/lib/commands/feedback.mjs

@@ -0,0 +1,57 @@
+// `feedback` — submit product feedback about Letterstory itself: a bug, something
+// missing, something unclear, a feature request, or praise. One subcommand, one tool
+// (submit_feedback) — there is no read/list surface yet, staff triage this table directly.
+
+import { CliError } from "../client.mjs";
+import { flagStr, readBodyInput, printResult, ok, argsFromFlags } from "./shared.mjs";
+
+function readJsonObjectFlag(flags, name) {
+	const inline = flagStr(flags[name]);
+	if (inline === undefined) return undefined;
+	let parsed;
+	try {
+		parsed = JSON.parse(inline);
+	} catch (err) {
+		throw new CliError(`--${name} is not valid JSON: ${err.message}`);
+	}
+	if (parsed === null || typeof parsed !== "object" || Array.isArray(parsed))
+		throw new CliError(`--${name} must be a JSON object`);
+	return parsed;
+}
+
+const ARGS = {
+	submit_feedback: {
+		category: { flag: "category", required: true },
+		// --message, or the larger-blob trio --body/--file/stdin (same convention as
+		// `landings create`'s --context).
+		message: { value: (flags) => flagStr(flags.message) ?? readBodyInput(flags) },
+		rating: { flag: "rating", type: "num" },
+		context: { value: (flags) => readJsonObjectFlag(flags, "context") },
+	},
+};
+
+export async function cmdFeedback(ctx) {
+	const sub = ctx.positionals[0];
+	const rest = { ...ctx, positionals: ctx.positionals.slice(1) };
+	switch (sub) {
+		case "submit":
+			return feedbackSubmit(rest);
+		default:
+			throw new CliError(`Unknown feedback subcommand: ${sub ?? "(none)"}. Try: submit`);
+	}
+}
+
+async function feedbackSubmit(ctx) {
+	const { client, flags, io } = ctx;
+	// ARGS.category has required: true, so a missing --category throws before we get here.
+	const args = argsFromFlags(ARGS.submit_feedback, flags, ctx.positionals);
+	if (!args.message) {
+		throw new CliError("Missing feedback message — pass --message, --body, --file <path>, or pipe it on stdin.");
+	}
+	const result = await client.callTool("submit_feedback", args);
+	ok(ctx, `Submitted ${args.category} feedback.`);
+	printResult(io, flags, result);
+	return 0;
+}
+
+export { ARGS };

package/lib/commands/onboard.mjs

@@ -15,8 +15,8 @@
 //
 // IMPORTANT: the masthead name is committed when the site is created (it seeds the URL),
 // so to use a name other than the inferred one, pass --name BEFORE applying. The first
-// posts are written by background jobs (20-40min each); they're reported as job ids and
-// this command does not block on them — poll with `phantom-job status <job-id>`.
+// posts are written by background runs (20-60min each); they're reported as run ids and
+// this command does not block on them — poll with `collection run-status <run-id>`.
 //
 // `--accept-plan <path|->` closes the review loop: the apply PINS the plan instead of
 // recomputing it, so the masthead that was reviewed is the masthead that is committed. In one

package/lib/commands.mjs

@@ -31,3 +31,4 @@ export * from "./commands/seers.mjs";
 export * from "./commands/shredder.mjs";
 export * from "./commands/landings.mjs";
 export * from "./commands/admin.mjs";
+export * from "./commands/feedback.mjs";

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@letterstory/cli",
-	"version": "0.29.1",
+	"version": "0.29.3",
 	"description": "Spin up and manage Letterstory phantom blogs from your terminal.",
 	"type": "module",
 	"license": "UNLICENSED",