@letterstory/cli 0.100.2 → 0.100.5

package/lib/cli.mjs

@@ -59,7 +59,7 @@ import {
 } from "./commands.mjs";
 
 // Keep in sync with cli/package.json.
-export const VERSION = "0.100.2";
+export const VERSION = "0.100.5";
 
 // Flags that never take a value. Listing them explicitly means `deploy get --json <id>`
 // can't accidentally swallow the id as --json's value. Every `type: "bool"` flag (and its
@@ -77,6 +77,7 @@ export const BOOLEAN_FLAGS = new Set([
 	"no-control",
 	"own-beat",
 	"refresh",
+	"preview",
 	"accept",
 	"help",
 	"version",
@@ -1120,10 +1121,10 @@ Repo changes (Claude Code edits your repo in a sandbox; the Letterstory GitHub A
   repo repos                                           Repos the Letterstory GitHub App can reach
   repo branches --repo owner/name                      Branches a change can start from (default first)
   repo start --repo <owner/name> --instructions <text>|--instructions-file <path|->
-             [--branch <base>] [--attach <path>]... [--attach-url <name>=<https url>]... [--wait]
+             [--branch <base>] [--attach <path>]... [--attach-url <name>=<https url>]... [--preview] [--wait]
                                                        Start a change (attached files go to the sandbox);
                                                        --wait follows it to the PR
-  repo fix-audit --issue <issue-type> [--audit <id>] [--branch <base>] [--wait]
+  repo fix-audit --issue <issue-type> [--audit <id>] [--branch <base>] [--preview] [--wait]
                                                        One PR fixing a site-audit issue on every affected page,
                                                        in the repo verified to build the site
   repo sites [--detect [--force]]                      Which website each repo builds, and the evidence
@@ -1132,7 +1133,8 @@ Repo changes (Claude Code edits your repo in a sandbox; the Letterstory GitHub A
   repo reset-site --repo <owner/name> --site <host>    Back to automatic detection
   repo status <run-id>                                 Phase, spend, PR link and Claude's summary
   repo list [--status running|succeeded|no_changes|failed|cancelled] [--limit n] [--audit <id>]
-  repo cancel <run-id>                                 Stop it and delete the sandbox; no PR
+  repo publish <run-id>                                Make PR on a preview in review
+  repo cancel <run-id>                                 Stop it and delete the sandbox; no PR (Discard a preview)
 
 Feedback (tell us what's broken, missing, unclear, or wanted):
   feedback submit --category bug|missing_feature|unclear|feature_request|praise|other

package/lib/commands/org.mjs

@@ -220,9 +220,17 @@ export function formatApply(result) {
 	for (const a of result.skipped) lines.push(`  ${SYMBOL[a.op]} ${a.section}/${a.name} — ${a.reason}`);
 	if (result.failed.length) lines.push("Failed:");
 	for (const a of result.failed) lines.push(`  ${SYMBOL[a.op]} ${a.section}/${a.name} — ${a.error}`);
-	if (result.gated?.length) {
+	// `gated` lists every gated change, allowed or not; after a real apply, show only those still held.
+	// A field-gated record can land in `applied` with its gated field skipped, so check `skipped` too.
+	const key = (a) => `${a.op} ${a.section}/${a.name}`;
+	const done = new Set(result.dryRun ? [] : result.applied.map(key));
+	const held = (result.gated ?? []).filter(
+		(a) =>
+			!done.has(key(a)) || result.skipped.some((s) => key(s) === key(a) && s.reason.includes(`gated (${a.gate})`))
+	);
+	if (held.length) {
 		lines.push(`Gated (allow with --allow; confirm ${result.confirm}):`);
-		for (const a of result.gated) lines.push(`  ${SYMBOL[a.op]} ${a.section}/${a.name} — ${a.gate}`);
+		for (const a of held) lines.push(`  ${SYMBOL[a.op]} ${a.section}/${a.name} — ${a.gate}`);
 	}
 	for (const w of result.plan.warnings) lines.push(`warning: ${w}`);
 	return lines.join("\n");

package/lib/commands/repo.mjs

@@ -2,6 +2,7 @@
 // pull request by the Letterstory GitHub App (src/lib/repo-changes). One subcommand per tool:
 //   repo start   → start_repo_change     repo status → get_repo_change
 //   repo list    → list_repo_changes     repo cancel → cancel_repo_change
+//   repo publish → publish_repo_change (Make PR on a preview)
 //   repo repos   → list_github_app_repos  repo branches → list_repo_branches
 
 import { readFileSync } from "node:fs";
@@ -36,6 +37,7 @@ const ARGS = {
 		instructions: { value: (flags) => readTextInput(flags, "instructions", "instructions-file") },
 		// --attach <path> (uploaded first) and --attach-url <name>=<https url>, both repeatable.
 		files: { value: (_flags, _positionals, extra) => (extra.files?.length ? extra.files : undefined) },
+		preview: { flag: "preview", type: "bool" },
 	},
 	create_repo_change_upload: { file_names: { internal: "the basenames of the --attach files" } },
 	list_repo_sites: {},
@@ -55,9 +57,11 @@ const ARGS = {
 		issue_type: { flag: "issue", required: true },
 		audit_id: "audit",
 		base_branch: "branch",
+		preview: { flag: "preview", type: "bool" },
 	},
 	get_repo_change: { run_id: { positional: 0, required: true, label: "run-id" } },
 	cancel_repo_change: { run_id: { positional: 0, required: true, label: "run-id" } },
+	publish_repo_change: { run_id: { positional: 0, required: true, label: "run-id" } },
 	list_repo_changes: {
 		status: { flag: "status", enum: ["running", "succeeded", "no_changes", "failed", "cancelled"] },
 		limit: { flag: "limit", type: "num" },
@@ -76,7 +80,10 @@ function describe(run) {
 	const lines = [
 		`${run.id}  ${run.repo}@${run.baseBranch}  ${run.status} (${run.phase})  $${run.costUsd.toFixed(2)}`,
 	];
-	if (run.prUrl) lines.push(`  PR: ${run.prUrl}`);
+	if (run.preview?.url) lines.push(`  Preview: ${run.preview.url}`);
+	if (run.preview?.state === "failed") lines.push(`  Preview failed: ${run.preview.detail ?? ""}`);
+	if (run.phase === "review") lines.push(`  Make PR: letterstory repo publish ${run.id}`);
+	if (run.prUrl) lines.push(`  PR: ${run.prUrl} (${run.prState ?? "state unknown"})`);
 	if (run.errorCode) lines.push(`  Error (${run.errorCode}): ${run.errorDetail ?? ""}`);
 	if (run.summary) lines.push(...run.summary.split("\n").map((l) => `  ${l}`));
 	return lines.join("\n");
@@ -102,13 +109,15 @@ export async function cmdRepo(ctx) {
 			return repoList(rest);
 		case "cancel":
 			return repoCancel(rest);
+		case "publish":
+			return repoPublish(rest);
 		case "repos":
 			return repoRepos(rest);
 		case "branches":
 			return repoBranches(rest);
 		default:
 			throw new CliError(
-				`Unknown repo subcommand: ${sub ?? "(none)"}. Try: start, fix-audit, sites, set-site, reset-site, status, list, cancel, repos, branches`
+				`Unknown repo subcommand: ${sub ?? "(none)"}. Try: start, fix-audit, sites, set-site, reset-site, status, list, publish, cancel, repos, branches`
 			);
 	}
 }
@@ -162,6 +171,7 @@ async function repoStart(ctx) {
 		"instructions-file",
 		"attach",
 		"attach-url",
+		"preview",
 		"wait",
 	]);
 	const files = [
@@ -183,7 +193,7 @@ async function repoStart(ctx) {
 		async () => {
 			const polled = await client.callTool("get_repo_change", { run_id: started.run.id });
 			ok(ctx, `… ${polled.run.status} (${polled.run.phase})`);
-			return { done: TERMINAL.has(polled.run.status), value: polled.run };
+			return { done: settledFor(polled.run), value: polled.run };
 		},
 		{ intervalMs: 30000, ceilingMs: 60 * 60 * 1000 }
 	);
@@ -192,6 +202,11 @@ async function repoStart(ctx) {
 	return value.status === "failed" ? 1 : 0;
 }
 
+/** --wait stops at a terminal state, or at review: a preview waits on a person, not on time. */
+function settledFor(run) {
+	return TERMINAL.has(run.status) || run.phase === "review";
+}
+
 async function followRun(ctx, run) {
 	const { client, flags, io } = ctx;
 	const { settled, value } = await followJob(
@@ -199,7 +214,7 @@ async function followRun(ctx, run) {
 		async () => {
 			const polled = await client.callTool("get_repo_change", { run_id: run.id });
 			ok(ctx, `… ${polled.run.status} (${polled.run.phase})`);
-			return { done: TERMINAL.has(polled.run.status), value: polled.run };
+			return { done: settledFor(polled.run), value: polled.run };
 		},
 		{ intervalMs: 30000, ceilingMs: 60 * 60 * 1000 }
 	);
@@ -211,7 +226,7 @@ async function followRun(ctx, run) {
 /** `repo fix-audit --issue <type> [--audit <id>] [--wait]`: one audit issue → one PR, in the repo verified to build the site. */
 async function repoFixAudit(ctx) {
 	const { client, flags, io } = ctx;
-	refuseUnknownFlags(ctx, {}, ["issue", "audit", "branch", "wait"]);
+	refuseUnknownFlags(ctx, {}, ["issue", "audit", "branch", "preview", "wait"]);
 	const result = await client.callTool(
 		"fix_audit_issue",
 		argsFromFlags(ARGS.fix_audit_issue, flags, ctx.positionals)
@@ -297,6 +312,18 @@ async function repoCancel(ctx) {
 	return 0;
 }
 
+/** `repo publish <run-id>`: Make PR on a preview in review. */
+async function repoPublish(ctx) {
+	const { client, flags, io } = ctx;
+	const result = await client.callTool(
+		"publish_repo_change",
+		argsFromFlags(ARGS.publish_repo_change, flags, ctx.positionals)
+	);
+	ok(ctx, result.message);
+	printResult(io, flags, result.run, describe);
+	return 0;
+}
+
 async function repoRepos(ctx) {
 	const { client, flags, io } = ctx;
 	const result = await client.callTool("list_github_app_repos", argsFromFlags(ARGS.list_github_app_repos, flags));

package/package.json

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