@letterstory/cli 0.29.0 → 0.29.2

package/README.md

@@ -142,9 +142,10 @@ letterstory phantom domain connect <phantom-id> myblog.com      # attach one you
 ```bash
 letterstory posts list [--limit <n>] [--collection <uuid>]
 letterstory posts show <article-id>
-letterstory posts new --title "…" --collection <uuid> --body "…"     # or --file <path|->
+letterstory posts new --title "…" --collection <uuid> --body "…" [--article-id <uuid>]   # or --file <path|->; --article-id updates that article instead of creating one
 letterstory posts publish <article-id>
 letterstory posts unpublish <article-id>
+letterstory posts delete --id <uuid> [--id <uuid> ...]                # permanent; content, history, and any published copy are all removed
 
 letterstory published list [--limit <n>] [--collection <uuid>] [--format html|md]
 letterstory published show --slug my-post-slug [--format html|md]    # or --id <uuid>

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.0";
+export const VERSION = "0.29.2";
 
 // Flags that never take a value. Listing them explicitly means `deploy get --json <id>`
 // can't accidentally swallow the id as --json's value.
@@ -205,9 +206,10 @@ Phantomstory (a collection Letterstory hosts as an independent publication):
 Content:
   posts list [--limit <n>] [--collection <uuid>]     List your articles
   posts show <article-id>                            Show one article
-  posts new --title <t> --collection <uuid> (--body <text> | --file <path|->)
+  posts new --title <t> --collection <uuid> (--body <text> | --file <path|->) [--article-id <uuid>]
   posts publish <article-id>                         Publish an article
   posts unpublish <article-id>                       Unpublish an article
+  posts delete --id <uuid> [--id <uuid> ...]         Permanently delete article(s)
   published list [--limit <n>] [--collection <uuid>] [--format html|md]
   published show (--id <uuid> | --slug <slug>) [--format html|md]
 
@@ -580,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
@@ -644,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/posts.mjs

@@ -20,6 +20,9 @@ export const ARGS = {
 	publish_article: {
 		article_id: { positional: 0, label: "article-id", required: true },
 	},
+	delete_article: {
+		article_ids: { flag: "id", type: "list", required: true, label: "id" },
+	},
 };
 
 export async function cmdPosts(ctx) {
@@ -37,9 +40,12 @@ export async function cmdPosts(ctx) {
 			return postsPublish(rest);
 		case "unpublish":
 			return postsUnpublish(rest);
+		case "delete":
+		case "rm":
+			return postsDelete(rest);
 		default:
 			throw new CliError(
-				`Unknown posts subcommand: ${sub ?? "(none)"}. Try: list, show, new, publish, unpublish`
+				`Unknown posts subcommand: ${sub ?? "(none)"}. Try: list, show, new, publish, unpublish, delete`
 			);
 	}
 }
@@ -66,8 +72,10 @@ async function postsNew(ctx) {
 	const collection = requireFlag(flags, "collection");
 	const body = readBodyInput(flags);
 	if (!body) throw new CliError("Provide --body or --file <path>.");
-	const result = await client.callTool("ingest_article", { title, body, collection_id: collection });
-	ok(ctx, `Created draft "${title}".`);
+	const articleId = flagStr(flags["article-id"]);
+	const args = compact({ title, body, collection_id: collection, article_id: articleId });
+	const result = await client.callTool("ingest_article", args);
+	ok(ctx, articleId ? `Updated "${title}".` : `Created draft "${title}".`);
 	printResult(io, flags, result);
 	return 0;
 }
@@ -89,6 +97,17 @@ async function postsUnpublish(ctx) {
 	return 0;
 }
 
+// Permanent: no confirmation prompt here — the CLI is a scripting surface, and the
+// tool result reports exactly which ids were deleted vs not found.
+async function postsDelete(ctx) {
+	const { client, flags, io, positionals } = ctx;
+	const args = argsFromFlags(ARGS.delete_article, flags, positionals);
+	const result = await client.callTool("delete_article", args);
+	ok(ctx, `Deleted ${result?.deleted_ids?.length ?? 0} article(s).`);
+	printResult(io, flags, result);
+	return 0;
+}
+
 // --- published (frozen public view) -----------------------------------------
 
 export async function cmdPublished(ctx) {

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.0",
+	"version": "0.29.2",
 	"description": "Spin up and manage Letterstory phantom blogs from your terminal.",
 	"type": "module",
 	"license": "UNLICENSED",