@mytegroupinc/myte-core 0.0.43 → 0.0.45

package/cli.js

@@ -208,10 +208,10 @@ function printHelp() {
     "  myte feedback reviews [--status open|terminal|all] [--request-id <id>] [--json]",
     "  myte feedback review --request-id <id> --action approve|reject|request_changes|cancel [--reason \"...\"] [--json]",
     "  myte feedback review --request-ids \"<id1,id2>\" --action approve|reject|request_changes|cancel [--reason \"...\"] [--json]",
-    "  myte feedback move --feedback-id <id> --to-state in_progress --from-state todo --reason \"...\" [--json]",
-    "  myte feedback move --feedback-ids \"<id1,id2>\" --to-state in_progress --reason \"...\" [--json]",
-    "  myte feedback comment --feedback-id <id> --body \"...\" [--body-file ./comment.md] [--json]",
-    "  myte feedback undo --feedback-id <id> --event-id <id> --reason \"...\" [--json]",
+    "  myte feedback move --feedback-id <id> --to-state in_progress --from-state todo --reason \"...\" [--json]",
+    "  myte feedback move --feedback-ids \"<id1,id2>\" --to-state in_progress --reason \"...\" [--json]",
+    "  myte feedback comment --feedback-id <id> --body \"...\" [--body-file ./comment.md] [--json]",
+    "  myte feedback undo --feedback-id <id> --event-id <id> --reason \"...\" [--json]",
     "  myte feedback prd-versions --feedback-id <id> [--json]",
     "  myte feedback prd-diff --feedback-id <id> --version-id <id> [--compare-to <id>] [--json]",
     "  myte feedback validate --file ./MyteCommandCenter/reviews/feedback/<id>-status.yml [--json]",
@@ -235,6 +235,12 @@ function printHelp() {
     "  - Set MYTE_API_KEY in a workspace .env (or env var)",
     "  - Set MYTEAI_API_KEY in a workspace .env (or env var) for `myte ai`",
     "",
+    "Live write approval:",
+    "  - Live project-key mutations require --confirm-write --approval-artifact <path>",
+    "  - The artifact must be a local .md, .yml/.yaml, or .json file shown to and approved by the user before the command is rerun",
+    "  - Batch artifacts must enumerate every feedback id, mission id, suggestion id, request id, or PRD item",
+    "  - Read/sync/query/validate, --print-context, and --dry-run commands do not require write approval",
+    "",
     "bootstrap contract:",
     "  - Run from any workspace where you want local MyteCommandCenter data written",
     "  - Writes MyteCommandCenter/data/project.yml plus phases, epics, stories, and missions locally",
@@ -313,7 +319,7 @@ function printHelp() {
     "  - Archived feedback is intentionally excluded from normal sync; returning it to active work is web UI only",
     "  - Writes project feedback metadata and conversation turns into MyteCommandCenter/data/feedback.yml",
     "  - Stores full PRD context in MyteCommandCenter/PRD/feedback-sync/*.md and points to those files from feedback.yml",
-    "  - Reads existing feedback comments and can create text-only feedback-specific comments through the project-key API",
+    "  - Reads existing feedback comments and can create text-only feedback-specific comments through the project-key API",
     "",
     "feedback review contract:",
     "  - Draft commands write review artifacts under MyteCommandCenter/reviews/feedback/*.yml for local IDE diff review",
@@ -322,9 +328,9 @@ function printHelp() {
     "  - reviews/review expose owner/delegate approval, rejection, request-changes, and cancel decisions",
     "  - review with --request-ids sends one batch review request; it does not loop single-review calls",
     "  - live lifecycle moves return pending_feedback_review while an active linked review request exists",
-    "  - move/undo operate safe board states directly; governed states remain backend-authorized",
-    "  - move with --feedback-ids sends one batch board move request; it supports active-state moves and authorized archive",
-    "  - comment creates text-only feedback-specific comments; attachments remain web UI only for this endpoint",
+    "  - move/undo operate safe board states directly; governed states remain backend-authorized",
+    "  - move with --feedback-ids sends one batch board move request; it supports active-state moves and authorized archive",
+    "  - comment creates text-only feedback-specific comments; attachments remain web UI only for this endpoint",
     "  - project-key API/CLI cannot unarchive archived Feedback; use the web archived Feedback view",
     "  - get/history read current feedback snapshots and audited event history through the project-key API",
     "  - prd-versions/prd-diff expose retained PRD versions and backend-generated text diffs",
@@ -340,9 +346,9 @@ function printHelp() {
     "  - revise: resubmit the original submitter's request after request_changes",
     "  - reviews: list open/terminal/all requests or inspect one request",
     "  - review: approve, reject, request_changes, or cancel; backend restricts this to owner/delegate capability",
-    "  - move: direct audited board movement for allowed states; governed transitions remain backend-authorized",
-    "  - comment: create a text-only comment on one Feedback item",
-    "  - undo: reverse an audited board event when conflict checks allow it",
+    "  - move: direct audited board movement for allowed states; governed transitions remain backend-authorized",
+    "  - comment: create a text-only comment on one Feedback item",
+    "  - undo: reverse an audited board event when conflict checks allow it",
     "  - prd-versions: list retained PRD baselines/revisions for a feedback item",
     "  - prd-diff: fetch backend-generated text diff between PRD versions",
     "  - history: list board/refinement events for audit review",
@@ -363,11 +369,11 @@ function printHelp() {
     "  --stdin               Read supported command content from stdin instead of inline text or a file path",
     "  --title <text>        Override PRD title for raw markdown uploads",
     "  --description <text>  Set feedback description/card summary for raw markdown uploads",
-    "  --content <text>      Team update content for update-team",
-    "  --body <text>         Feedback comment body for feedback comment",
-    "  --subject <text>      Subject for update-owner or update-client",
+    "  --content <text>      Team update content for update-team",
+    "  --body <text>         Feedback comment body for feedback comment",
+    "  --subject <text>      Subject for update-owner or update-client",
     "  --body-markdown <md>  Markdown body for update-owner or update-client",
-    "  --body-file <path>    Read feedback comment, update-owner, or update-client markdown body from a file",
+    "  --body-file <path>    Read feedback comment, update-owner, or update-client markdown body from a file",
     "  --target-contact-id   Add one client contact ObjectId (repeatable)",
     "  --target-contact-ids  Comma-separated client contact ObjectIds",
     "  --feedback-id <id>    Feedback ObjectId for feedback review commands",
@@ -396,6 +402,8 @@ function printHelp() {
     "  --wait                Poll batch status until terminal completion for run-qaqc",
     "  --sync                After run-qaqc completes, refresh local QAQC file",
     "  --force               Allow run-qaqc to bypass stale-state protection when supported",
+    "  --confirm-write       Confirm the user explicitly approved a live Myte project-key mutation",
+    "  --approval-artifact   Local .md/.yml/.json approval artifact shown to and approved by the user",
     "  --no-sync             Skip automatic post-mutation sync for suggestions, missions, and feedback mutations",
     "  --print-context       Print JSON payload and exit (no query call)",
     "  --no-fetch            Don't git fetch origin main/master before diff",
@@ -421,10 +429,10 @@ function printHelp() {
     "  myte feedback status --feedback-id 507f1f77bcf86cd799439011 --status in_review --reason \"Ready for owner review\"",
     "  myte feedback submit --file ./MyteCommandCenter/reviews/feedback/507f1f77bcf86cd799439011-edit.yml --json",
     "  myte feedback reviews --status open --json",
-    "  myte feedback review --request-id 507f1f77bcf86cd799439022 --action approve --reason \"Looks correct\" --json",
-    "  myte feedback move --feedback-id 507f1f77bcf86cd799439011 --to-state in_progress --reason \"Started\" --json",
-    "  myte feedback comment --feedback-id 507f1f77bcf86cd799439011 --body \"Implementation notes are ready for review.\" --json",
-    "  myte feedback validate --file ./MyteCommandCenter/reviews/feedback/507f1f77bcf86cd799439011-status.yml --json",
+    "  myte feedback review --request-id 507f1f77bcf86cd799439022 --action approve --reason \"Looks correct\" --json",
+    "  myte feedback move --feedback-id 507f1f77bcf86cd799439011 --to-state in_progress --reason \"Started\" --json",
+    "  myte feedback comment --feedback-id 507f1f77bcf86cd799439011 --body \"Implementation notes are ready for review.\" --json",
+    "  myte feedback validate --file ./MyteCommandCenter/reviews/feedback/507f1f77bcf86cd799439011-status.yml --json",
     "  myte feedback apply --file ./MyteCommandCenter/reviews/feedback/507f1f77bcf86cd799439011-status.yml --json",
     "  myte suggestions create --file ./suggestions/create.yml",
     "  myte suggestions revise",
@@ -693,6 +701,164 @@ async function fetchJsonWithTimeout(fetchFn, url, options, timeoutMs) {
   }
 }
 
+const WRITE_APPROVAL_TRANSPORT_KEYS = new Set(["write_approval", "client_session_id"]);
+
+function canonicalizeWritePayload(value) {
+  if (Array.isArray(value)) {
+    return value.map((item) => canonicalizeWritePayload(item));
+  }
+  if (value && typeof value === "object") {
+    return Object.fromEntries(
+      Object.keys(value)
+        .filter((key) => !WRITE_APPROVAL_TRANSPORT_KEYS.has(String(key)))
+        .sort()
+        .map((key) => [key, canonicalizeWritePayload(value[key])])
+    );
+  }
+  return value;
+}
+
+function projectApiWritePayloadHash(payload) {
+  return createHash("sha256")
+    .update(pythonJsonDumpsCompatible(canonicalizeWritePayload(payload || {})))
+    .digest("hex");
+}
+
+function isConfirmWrite(args) {
+  return Boolean(args["confirm-write"] || args.confirmWrite || args.confirm_write);
+}
+
+function resolveWriteApprovalArtifactPath(args) {
+  const explicit = firstNonEmptyString(args["approval-artifact"], args.approvalArtifact, args.approval_artifact);
+  if (!explicit) return "";
+  const resolved = path.resolve(process.cwd(), explicit);
+  if (!fs.existsSync(resolved) || !fs.statSync(resolved).isFile()) {
+    console.error(`Approval artifact not found: ${resolved}`);
+    process.exit(1);
+  }
+  return resolved;
+}
+
+function inferApprovalArtifactKind(filePath, fallbackKind) {
+  const ext = path.extname(String(filePath || "")).toLowerCase();
+  if (ext === ".md" || ext === ".markdown") return "markdown";
+  if (ext === ".yml" || ext === ".yaml") return "yaml";
+  if (ext === ".json") return "json";
+  return fallbackKind || "artifact";
+}
+
+function normalizeApprovalTargets(targets) {
+  return toStringArray(targets).map((item) => item.trim()).filter(Boolean);
+}
+
+function writePayloadItems(payload) {
+  if (Array.isArray(payload?.items)) return payload.items;
+  if (Array.isArray(payload)) return payload;
+  return payload && typeof payload === "object" ? [payload] : [];
+}
+
+function suggestionApprovalTargets(payload) {
+  return writePayloadItems(payload).map((item, index) => {
+    if (!item || typeof item !== "object") return `index:${index}`;
+    return firstNonEmptyString(
+      item.suggestion_id,
+      item.id,
+      item._id,
+      item.mission_id,
+      item.target_mission_id,
+      item.title ? `${item.change_type || "suggestion"}:${item.title}` : ""
+    ) || `index:${index}`;
+  });
+}
+
+function prdApprovalTargets(payload) {
+  return writePayloadItems(payload).map((item, index) => {
+    if (!item || typeof item !== "object") return `index:${index}`;
+    return firstNonEmptyString(item.client_ref, item.title) || `index:${index}`;
+  });
+}
+
+function requireWriteApproval(args, payload, { operation, artifactKind, targets = [], batchCount = null } = {}) {
+  const payloadSha256 = projectApiWritePayloadHash(payload || {});
+  const targetValues = normalizeApprovalTargets(targets);
+  const expectedBatchCount = batchCount !== null && batchCount !== undefined
+    ? Number(batchCount)
+    : targetValues.length > 1
+      ? targetValues.length
+      : null;
+
+  if (!isConfirmWrite(args)) {
+    console.error("Refusing live Myte write without explicit human approval.");
+    console.error(`Operation: ${operation || "unknown"}`);
+    console.error(`Payload SHA256: ${payloadSha256}`);
+    if (targetValues.length) console.error(`Targets: ${targetValues.join(", ")}`);
+    if (expectedBatchCount !== null) console.error(`Batch count: ${expectedBatchCount}`);
+    console.error("Create a local .md/.yml/.json approval artifact that lists the exact operation, payload summary, and every target.");
+    console.error("Show that artifact to the user and ask for explicit approval.");
+    console.error("Then rerun with --confirm-write --approval-artifact <path>.");
+    process.exit(1);
+  }
+
+  const artifactPath = resolveWriteApprovalArtifactPath(args);
+  if (!artifactPath) {
+    console.error("Missing --approval-artifact <path> for approved Myte write.");
+    console.error(`Payload SHA256: ${payloadSha256}`);
+    console.error("The artifact must be a local .md/.yml/.json file shown to and explicitly approved by the user.");
+    process.exit(1);
+  }
+
+  return {
+    approved: true,
+    source: "explicit_user_approval",
+    artifact_path: artifactPath,
+    artifact_kind: inferApprovalArtifactKind(artifactPath, artifactKind),
+    payload_sha256: payloadSha256,
+    targets: targetValues,
+    batch_count: expectedBatchCount,
+  };
+}
+
+function writeApprovalHeaders(writeApproval) {
+  if (!writeApproval) return {};
+  const headers = {
+    "X-Myte-Write-Approval": "explicit_user_approval",
+    "X-Myte-Approval-Artifact": String(writeApproval.artifact_path || ""),
+    "X-Myte-Approval-Artifact-Kind": String(writeApproval.artifact_kind || ""),
+    "X-Myte-Approval-Payload-SHA256": String(writeApproval.payload_sha256 || ""),
+  };
+  if (writeApproval.batch_count !== null && writeApproval.batch_count !== undefined) {
+    headers["X-Myte-Approval-Batch-Count"] = String(writeApproval.batch_count);
+  }
+  if (Array.isArray(writeApproval.targets) && writeApproval.targets.length) {
+    headers["X-Myte-Approval-Targets"] = writeApproval.targets.join(",");
+  }
+  return headers;
+}
+
+function formatProjectApiErrorMessage(body, fallback) {
+  const message = body?.message || fallback;
+  const data = body?.data || {};
+  const code = String(data?.code || "");
+  if (!code.startsWith("myte_write_approval")) {
+    return message;
+  }
+  const lines = [
+    message,
+    `Approval error: ${code}`,
+  ];
+  if (data.operation) lines.push(`Operation: ${data.operation}`);
+  if (data.payload_sha256) lines.push(`Payload SHA256: ${data.payload_sha256}`);
+  if (Array.isArray(data.targets) && data.targets.length) lines.push(`Targets: ${data.targets.join(", ")}`);
+  if (data.batch_count !== undefined && data.batch_count !== null) lines.push(`Batch count: ${data.batch_count}`);
+  if (Array.isArray(data.required_agent_steps) && data.required_agent_steps.length) {
+    lines.push("Required agent steps:");
+    for (const step of data.required_agent_steps) {
+      lines.push(`- ${step}`);
+    }
+  }
+  return lines.join("\n");
+}
+
 function summarizeDiffDiagnosticsForContext(diagnostics) {
   if (!diagnostics) return null;
   const repos = Array.isArray(diagnostics.repo_summaries)
@@ -1756,7 +1922,7 @@ async function fetchFeedbackHistory({ apiBase, key, timeoutMs, feedbackId }) {
   return body.data || {};
 }
 
-async function postFeedbackRefinement({ apiBase, key, timeoutMs, feedbackId, mode, payload, idempotencyKey, clientSessionId }) {
+async function postFeedbackRefinement({ apiBase, key, timeoutMs, feedbackId, mode, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const action = String(mode || "").trim();
   const url = `${apiBase}/project-assistant/feedback/${encodeURIComponent(String(feedbackId || ""))}/refinement/${action}`;
@@ -1764,6 +1930,7 @@ async function postFeedbackRefinement({ apiBase, key, timeoutMs, feedbackId, mod
     "Content-Type": "application/json",
     Authorization: `Bearer ${key}`,
     ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+    ...writeApprovalHeaders(writeApproval),
   };
   if (idempotencyKey) {
     headers["X-Idempotency-Key"] = String(idempotencyKey).trim();
@@ -1780,7 +1947,7 @@ async function postFeedbackRefinement({ apiBase, key, timeoutMs, feedbackId, mod
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Feedback refinement ${action} failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Feedback refinement ${action} failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
     err.data = body?.data;
@@ -1859,7 +2026,7 @@ async function fetchFeedbackEvents({ apiBase, key, timeoutMs, feedbackId, limit
   return body.data || {};
 }
 
-async function postFeedbackReviewSubmission({ apiBase, key, timeoutMs, feedbackId, payload, idempotencyKey, clientSessionId }) {
+async function postFeedbackReviewSubmission({ apiBase, key, timeoutMs, feedbackId, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}/project-assistant/feedback/${encodeURIComponent(String(feedbackId || ""))}/refinement/requests`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -1872,6 +2039,7 @@ async function postFeedbackReviewSubmission({ apiBase, key, timeoutMs, feedbackI
         Authorization: `Bearer ${key}`,
         "X-Idempotency-Key": String(idempotencyKey || "").trim(),
         ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
       },
       body: JSON.stringify(payload || {}),
     },
@@ -1879,7 +2047,7 @@ async function postFeedbackReviewSubmission({ apiBase, key, timeoutMs, feedbackI
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Feedback review submission failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Feedback review submission failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
     err.data = body?.data;
@@ -1888,7 +2056,7 @@ async function postFeedbackReviewSubmission({ apiBase, key, timeoutMs, feedbackI
   return body.data || {};
 }
 
-async function postFeedbackReviewRequestMutation({ apiBase, key, timeoutMs, requestId, endpointAction, payload, idempotencyKey, clientSessionId }) {
+async function postFeedbackReviewRequestMutation({ apiBase, key, timeoutMs, requestId, endpointAction, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}/project-assistant/feedback-review-requests/${encodeURIComponent(String(requestId || ""))}/${endpointAction}`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -1901,6 +2069,7 @@ async function postFeedbackReviewRequestMutation({ apiBase, key, timeoutMs, requ
         Authorization: `Bearer ${key}`,
         "X-Idempotency-Key": String(idempotencyKey || "").trim(),
         ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
       },
       body: JSON.stringify(payload || {}),
     },
@@ -1908,7 +2077,7 @@ async function postFeedbackReviewRequestMutation({ apiBase, key, timeoutMs, requ
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Feedback review ${endpointAction} failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Feedback review ${endpointAction} failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
     err.data = body?.data;
@@ -1917,7 +2086,7 @@ async function postFeedbackReviewRequestMutation({ apiBase, key, timeoutMs, requ
   return body.data || {};
 }
 
-async function postFeedbackBatchReviewMutation({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId }) {
+async function postFeedbackBatchReviewMutation({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}/project-assistant/feedback-review-requests/batch-review`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -1930,6 +2099,7 @@ async function postFeedbackBatchReviewMutation({ apiBase, key, timeoutMs, payloa
         Authorization: `Bearer ${key}`,
         "X-Idempotency-Key": String(idempotencyKey || "").trim(),
         ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
       },
       body: JSON.stringify(payload || {}),
     },
@@ -1937,7 +2107,7 @@ async function postFeedbackBatchReviewMutation({ apiBase, key, timeoutMs, payloa
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Feedback batch review failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Feedback batch review failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
     err.data = body?.data;
@@ -1946,7 +2116,7 @@ async function postFeedbackBatchReviewMutation({ apiBase, key, timeoutMs, payloa
   return body.data || {};
 }
 
-async function postFeedbackBoardMutation({ apiBase, key, timeoutMs, feedbackId, endpoint, payload, idempotencyKey, clientSessionId }) {
+async function postFeedbackBoardMutation({ apiBase, key, timeoutMs, feedbackId, endpoint, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}/project-assistant/feedback/${encodeURIComponent(String(feedbackId || ""))}/${endpoint}`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -1959,6 +2129,7 @@ async function postFeedbackBoardMutation({ apiBase, key, timeoutMs, feedbackId,
         Authorization: `Bearer ${key}`,
         "X-Idempotency-Key": String(idempotencyKey || "").trim(),
         ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
       },
       body: JSON.stringify(payload || {}),
     },
@@ -1966,7 +2137,7 @@ async function postFeedbackBoardMutation({ apiBase, key, timeoutMs, feedbackId,
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Feedback board mutation failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Feedback board mutation failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
     err.data = body?.data;
@@ -1975,7 +2146,7 @@ async function postFeedbackBoardMutation({ apiBase, key, timeoutMs, feedbackId,
   return body.data || {};
 }
 
-async function postFeedbackBatchBoardMutation({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId }) {
+async function postFeedbackBatchBoardMutation({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}/project-assistant/feedback/batch-board-move`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -1988,6 +2159,7 @@ async function postFeedbackBatchBoardMutation({ apiBase, key, timeoutMs, payload
         Authorization: `Bearer ${key}`,
         "X-Idempotency-Key": String(idempotencyKey || "").trim(),
         ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
       },
       body: JSON.stringify(payload || {}),
     },
@@ -1995,45 +2167,46 @@ async function postFeedbackBatchBoardMutation({ apiBase, key, timeoutMs, payload
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Feedback batch board mutation failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Feedback batch board mutation failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
     err.data = body?.data;
     throw err;
   }
-  return body.data || {};
-}
-
-async function postFeedbackComment({ apiBase, key, timeoutMs, feedbackId, payload, idempotencyKey, clientSessionId }) {
-  const fetchFn = await getFetch();
-  const url = `${apiBase}/project-assistant/feedback/${encodeURIComponent(String(feedbackId || ""))}/comments`;
-  const { resp, body } = await fetchJsonWithTimeout(
-    fetchFn,
-    url,
-    {
-      method: "POST",
-      headers: {
-        "Content-Type": "application/json",
-        Authorization: `Bearer ${key}`,
-        "X-Idempotency-Key": String(idempotencyKey || "").trim(),
-        ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
-      },
-      body: JSON.stringify(payload || {}),
-    },
-    timeoutMs
-  );
-
-  if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Feedback comment failed (${resp.status})`;
-    const err = new Error(msg);
-    err.status = resp.status;
-    err.data = body?.data;
-    throw err;
-  }
-  return body.data || {};
-}
-
-async function fetchFeedbackPrdVersions({ apiBase, key, timeoutMs, feedbackId }) {
+  return body.data || {};
+}
+
+async function postFeedbackComment({ apiBase, key, timeoutMs, feedbackId, payload, idempotencyKey, clientSessionId, writeApproval }) {
+  const fetchFn = await getFetch();
+  const url = `${apiBase}/project-assistant/feedback/${encodeURIComponent(String(feedbackId || ""))}/comments`;
+  const { resp, body } = await fetchJsonWithTimeout(
+    fetchFn,
+    url,
+    {
+      method: "POST",
+      headers: {
+        "Content-Type": "application/json",
+        Authorization: `Bearer ${key}`,
+        "X-Idempotency-Key": String(idempotencyKey || "").trim(),
+        ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
+      },
+      body: JSON.stringify(payload || {}),
+    },
+    timeoutMs
+  );
+
+  if (!resp.ok || body.status !== "success") {
+    const msg = formatProjectApiErrorMessage(body, `Feedback comment failed (${resp.status})`);
+    const err = new Error(msg);
+    err.status = resp.status;
+    err.data = body?.data;
+    throw err;
+  }
+  return body.data || {};
+}
+
+async function fetchFeedbackPrdVersions({ apiBase, key, timeoutMs, feedbackId }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}/project-assistant/feedback/${encodeURIComponent(String(feedbackId || ""))}/prd/versions`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -2130,7 +2303,7 @@ function resolveProjectMutationIdempotencyKey({ args, operation, payload }) {
   return createHash("sha256").update(fingerprint).digest("hex");
 }
 
-async function postSuggestionsMutation({ apiBase, key, timeoutMs, endpoint, payload, idempotencyKey, clientSessionId }) {
+async function postSuggestionsMutation({ apiBase, key, timeoutMs, endpoint, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}${endpoint}`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -2143,6 +2316,7 @@ async function postSuggestionsMutation({ apiBase, key, timeoutMs, endpoint, payl
         Authorization: `Bearer ${key}`,
         "X-Idempotency-Key": String(idempotencyKey || "").trim(),
         ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
       },
       body: JSON.stringify(payload),
     },
@@ -2150,15 +2324,16 @@ async function postSuggestionsMutation({ apiBase, key, timeoutMs, endpoint, payl
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Mission suggestions request failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Mission suggestions request failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
+    err.data = body?.data;
     throw err;
   }
   return body.data || {};
 }
 
-async function createRunQaqcBatch({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId }) {
+async function createRunQaqcBatch({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}/project-assistant/run-qaqc`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -2171,6 +2346,7 @@ async function createRunQaqcBatch({ apiBase, key, timeoutMs, payload, idempotenc
         Authorization: `Bearer ${key}`,
         "X-Idempotency-Key": String(idempotencyKey || "").trim(),
         ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
       },
       body: JSON.stringify(payload),
     },
@@ -2178,15 +2354,16 @@ async function createRunQaqcBatch({ apiBase, key, timeoutMs, payload, idempotenc
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Run QAQC request failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Run QAQC request failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
+    err.data = body?.data;
     throw err;
   }
   return body.data || {};
 }
 
-async function createMissionStatusUpdate({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId }) {
+async function createMissionStatusUpdate({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}/project-assistant/mission-status-update`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -2199,6 +2376,7 @@ async function createMissionStatusUpdate({ apiBase, key, timeoutMs, payload, ide
         Authorization: `Bearer ${key}`,
         "X-Idempotency-Key": String(idempotencyKey || "").trim(),
         ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
       },
       body: JSON.stringify(payload),
     },
@@ -2206,15 +2384,16 @@ async function createMissionStatusUpdate({ apiBase, key, timeoutMs, payload, ide
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Mission status update request failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Mission status update request failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
+    err.data = body?.data;
     throw err;
   }
   return body.data || {};
 }
 
-async function createMissionArchiveUpdate({ apiBase, key, timeoutMs, endpoint, payload, idempotencyKey, clientSessionId }) {
+async function createMissionArchiveUpdate({ apiBase, key, timeoutMs, endpoint, payload, idempotencyKey, clientSessionId, writeApproval }) {
   const fetchFn = await getFetch();
   const url = `${apiBase}${endpoint}`;
   const { resp, body } = await fetchJsonWithTimeout(
@@ -2227,6 +2406,7 @@ async function createMissionArchiveUpdate({ apiBase, key, timeoutMs, endpoint, p
         Authorization: `Bearer ${key}`,
         "X-Idempotency-Key": String(idempotencyKey || "").trim(),
         ...(String(clientSessionId || "").trim() ? { "X-Client-Session-Id": String(clientSessionId).trim() } : {}),
+        ...writeApprovalHeaders(writeApproval),
       },
       body: JSON.stringify(payload),
     },
@@ -2234,9 +2414,10 @@ async function createMissionArchiveUpdate({ apiBase, key, timeoutMs, endpoint, p
   );
 
   if (!resp.ok || body.status !== "success") {
-    const msg = body?.message || `Mission archive request failed (${resp.status})`;
+    const msg = formatProjectApiErrorMessage(body, `Mission archive request failed (${resp.status})`);
     const err = new Error(msg);
     err.status = resp.status;
+    err.data = body?.data;
     throw err;
   }
   return body.data || {};
@@ -2370,6 +2551,12 @@ async function runRunQaqc(args) {
   const apiBase = resolveApiBase(args);
   const shouldWait = Boolean(args.wait || args.sync);
   const shouldSync = Boolean(args.sync);
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: "run-qaqc",
+    artifactKind: "yaml",
+    targets: missionIds,
+    batchCount: missionIds.length,
+  });
 
   let data;
   try {
@@ -2380,6 +2567,7 @@ async function runRunQaqc(args) {
       payload,
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (err?.name === "AbortError") {
@@ -2523,6 +2711,12 @@ async function runMissionStatus(args) {
 
   const timeoutMs = resolveTimeoutMs(args);
   const apiBase = resolveApiBase(args);
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: "mission-status-update",
+    artifactKind: "yaml",
+    targets: missionIds,
+    batchCount: missionIds.length,
+  });
 
   let data;
   try {
@@ -2533,6 +2727,7 @@ async function runMissionStatus(args) {
       payload,
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (err?.name === "AbortError") {
@@ -2630,6 +2825,12 @@ async function runMissionArchiveCommand(args) {
   const timeoutMs = resolveTimeoutMs(args);
   const apiBase = resolveApiBase(args);
   const endpoint = "/project-assistant/mission-archive";
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: "mission-archive",
+    artifactKind: "yaml",
+    targets: missionIds,
+    batchCount: missionIds.length,
+  });
 
   let data;
   try {
@@ -2641,6 +2842,7 @@ async function runMissionArchiveCommand(args) {
       payload,
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (err?.name === "AbortError") {
@@ -2740,6 +2942,10 @@ async function runUpdateTeam(args) {
     operation: "update_team",
     payload,
   });
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: "update_team",
+    artifactKind: "markdown",
+  });
 
   let data;
   try {
@@ -2751,6 +2957,7 @@ async function runUpdateTeam(args) {
       endpoint: "/project-assistant/project-comment",
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (err?.name === "AbortError") {
@@ -2809,6 +3016,10 @@ async function runUpdateOwner(args) {
     operation: "update_owner",
     payload,
   });
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: "update_owner",
+    artifactKind: "markdown",
+  });
 
   let data;
   try {
@@ -2820,6 +3031,7 @@ async function runUpdateOwner(args) {
       endpoint: "/project-assistant/update-owner",
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (err?.name === "AbortError") {
@@ -2887,6 +3099,10 @@ async function runUpdateClient(args) {
     operation: "update_client",
     payload,
   });
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: "update_client",
+    artifactKind: "markdown",
+  });
 
   let data;
   try {
@@ -2898,6 +3114,7 @@ async function runUpdateClient(args) {
       endpoint: "/project-assistant/client-update-drafts",
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (err?.name === "AbortError") {
@@ -2990,20 +3207,48 @@ function stringifyYaml(value) {
   return `${JSON.stringify(value, null, 2)}\n`;
 }
 
-function stableJsonStringify(value) {
-  if (Array.isArray(value)) {
-    return `[${value.map((item) => stableJsonStringify(item)).join(",")}]`;
+function stableJsonStringify(value) {
+  if (Array.isArray(value)) {
+    return `[${value.map((item) => stableJsonStringify(item)).join(",")}]`;
   }
   if (value && typeof value === "object") {
     const entries = Object.keys(value)
       .sort()
       .map((key) => `${JSON.stringify(key)}:${stableJsonStringify(value[key])}`);
     return `{${entries.join(",")}}`;
-  }
-  return JSON.stringify(value);
-}
-
-function parseYaml(text) {
+  }
+  return JSON.stringify(value);
+}
+
+function jsonStringifyEnsureAscii(value) {
+  return JSON.stringify(String(value)).replace(/[^\x00-\x7F]/g, (char) => {
+    const code = char.charCodeAt(0);
+    return `\\u${code.toString(16).padStart(4, "0")}`;
+  });
+}
+
+function pythonJsonDumpsCompatible(value) {
+  if (Array.isArray(value)) {
+    return `[${value.map((item) => pythonJsonDumpsCompatible(item)).join(", ")}]`;
+  }
+  if (value && typeof value === "object") {
+    const entries = Object.keys(value)
+      .sort()
+      .map((key) => `${jsonStringifyEnsureAscii(key)}: ${pythonJsonDumpsCompatible(value[key])}`);
+    return `{${entries.join(", ")}}`;
+  }
+  if (value === null) return "null";
+  if (typeof value === "boolean") return value ? "true" : "false";
+  if (typeof value === "number") {
+    if (Number.isNaN(value)) return "NaN";
+    if (value === Infinity) return "Infinity";
+    if (value === -Infinity) return "-Infinity";
+    return JSON.stringify(value);
+  }
+  return jsonStringifyEnsureAscii(value);
+}
+
+function parseYaml(text) {
   const raw = String(text || "").trim();
   if (!raw) return null;
   if (raw.startsWith("{") || raw.startsWith("[")) {
@@ -4269,7 +4514,7 @@ async function fetchSuggestionsSyncSnapshot({ apiBase, key, timeoutMs, actorScop
   return body.data || {};
 }
 
-async function createProjectSuggestions({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId }) {
+async function createProjectSuggestions({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId, writeApproval }) {
   return postSuggestionsMutation({
     apiBase,
     key,
@@ -4278,10 +4523,11 @@ async function createProjectSuggestions({ apiBase, key, timeoutMs, payload, idem
     payload,
     idempotencyKey,
     clientSessionId,
+    writeApproval,
   });
 }
 
-async function reviseProjectSuggestions({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId }) {
+async function reviseProjectSuggestions({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId, writeApproval }) {
   return postSuggestionsMutation({
     apiBase,
     key,
@@ -4290,10 +4536,11 @@ async function reviseProjectSuggestions({ apiBase, key, timeoutMs, payload, idem
     payload,
     idempotencyKey,
     clientSessionId,
+    writeApproval,
   });
 }
 
-async function reviewProjectSuggestions({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId }) {
+async function reviewProjectSuggestions({ apiBase, key, timeoutMs, payload, idempotencyKey, clientSessionId, writeApproval }) {
   return postSuggestionsMutation({
     apiBase,
     key,
@@ -4302,6 +4549,7 @@ async function reviewProjectSuggestions({ apiBase, key, timeoutMs, payload, idem
     payload,
     idempotencyKey,
     clientSessionId,
+    writeApproval,
   });
 }
 
@@ -4419,6 +4667,11 @@ async function runCreatePrd(args) {
       operation: "create_prd",
       payload,
     });
+    const writeApproval = requireWriteApproval(args, payload, {
+      operation: "create_prd",
+      artifactKind: "markdown",
+      targets: prdApprovalTargets(payload),
+    });
     try {
       data = await postSuggestionsMutation({
         apiBase,
@@ -4428,6 +4681,7 @@ async function runCreatePrd(args) {
         endpoint: "/project-assistant/create-prd",
         idempotencyKey,
         clientSessionId,
+        writeApproval,
       });
     } catch (err) {
       if (err?.name === "AbortError") {
@@ -4478,6 +4732,13 @@ async function runCreatePrd(args) {
       operation: "create_prds",
       payload,
     });
+    const prdTargets = prdApprovalTargets(payload);
+    const writeApproval = requireWriteApproval(args, payload, {
+      operation: "create_prds",
+      artifactKind: "markdown",
+      targets: prdTargets,
+      batchCount: prdTargets.length,
+    });
     const data = await postSuggestionsMutation({
       apiBase,
       key,
@@ -4486,6 +4747,7 @@ async function runCreatePrd(args) {
       endpoint: "/project-assistant/create-prds",
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
     aggregated.project_id = data.project_id || null;
     aggregated.created_count = Number(data.created_count || 0);
@@ -5065,6 +5327,13 @@ async function runFeedbackValidateOrApply(args, mode) {
         payload,
       })
     : null;
+  const writeApproval = mode === "apply"
+    ? requireWriteApproval(args, payload, {
+        operation: `feedback_refinement_apply:${feedbackId}`,
+        artifactKind: "yaml",
+        targets: [feedbackId],
+      })
+    : null;
 
   let data;
   try {
@@ -5077,6 +5346,7 @@ async function runFeedbackValidateOrApply(args, mode) {
       payload,
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (args.json) {
@@ -5122,33 +5392,33 @@ function resolveFeedbackRequestIdArg(args) {
   return firstNonEmptyString(args["request-id"], args.requestId, args.request_id, args._?.[1]);
 }
 
-function resolveFeedbackEventIdArg(args) {
-  return firstNonEmptyString(args["event-id"], args.eventId, args.event_id, args._?.[1]);
-}
+function resolveFeedbackEventIdArg(args) {
+  return firstNonEmptyString(args["event-id"], args.eventId, args.event_id, args._?.[1]);
+}
 
-function resolveFeedbackVersionIdArg(args) {
-  return firstNonEmptyString(args["version-id"], args.versionId, args.version_id, args._?.[1]);
-}
-
-async function resolveFeedbackCommentContent(args, feedbackId) {
-  const explicit = firstNonEmptyString(args.body, args.content, args["body-markdown"], args.bodyMarkdown, args.body_markdown);
-  if (explicit) return explicit;
-
-  const bodyFile = firstNonEmptyString(args["body-file"], args.bodyFile, args.body_file);
-  if (bodyFile) {
-    const bodyPath = resolveInputFile(bodyFile, "Feedback comment body");
-    return fs.readFileSync(bodyPath, "utf8");
-  }
-
-  if (args.stdin) {
-    return readStdinText();
-  }
-
-  const positionals = Array.isArray(args._) ? args._.slice(1) : [];
-  const feedbackIdFromPositional = positionals[0] && String(positionals[0]) === String(feedbackId);
-  const textParts = feedbackIdFromPositional ? positionals.slice(1) : positionals;
-  return textParts.join(" ");
-}
+function resolveFeedbackVersionIdArg(args) {
+  return firstNonEmptyString(args["version-id"], args.versionId, args.version_id, args._?.[1]);
+}
+
+async function resolveFeedbackCommentContent(args, feedbackId) {
+  const explicit = firstNonEmptyString(args.body, args.content, args["body-markdown"], args.bodyMarkdown, args.body_markdown);
+  if (explicit) return explicit;
+
+  const bodyFile = firstNonEmptyString(args["body-file"], args.bodyFile, args.body_file);
+  if (bodyFile) {
+    const bodyPath = resolveInputFile(bodyFile, "Feedback comment body");
+    return fs.readFileSync(bodyPath, "utf8");
+  }
+
+  if (args.stdin) {
+    return readStdinText();
+  }
+
+  const positionals = Array.isArray(args._) ? args._.slice(1) : [];
+  const feedbackIdFromPositional = positionals[0] && String(positionals[0]) === String(feedbackId);
+  const textParts = feedbackIdFromPositional ? positionals.slice(1) : positionals;
+  return textParts.join(" ");
+}
 
 function summarizeFeedbackReviewRequestPayload(data) {
   const request = data?.request || data || {};
@@ -5207,6 +5477,11 @@ async function runFeedbackSubmit(args) {
     operation: `feedback_refinement_request_submit:${feedbackId}`,
     payload,
   });
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: `feedback_refinement_request_submit:${feedbackId}`,
+    artifactKind: "yaml",
+    targets: [feedbackId],
+  });
 
   let data;
   try {
@@ -5218,6 +5493,7 @@ async function runFeedbackSubmit(args) {
       payload,
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (args.json) {
@@ -5273,6 +5549,11 @@ async function runFeedbackRevise(args) {
     operation: `feedback_refinement_request_revise:${requestId}`,
     payload,
   });
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: `feedback_refinement_request_revise:${requestId}`,
+    artifactKind: "yaml",
+    targets: [requestId],
+  });
 
   let data;
   try {
@@ -5285,6 +5566,7 @@ async function runFeedbackRevise(args) {
       payload,
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (args.json) {
@@ -5416,6 +5698,15 @@ async function runFeedbackReviewDecision(args) {
       operation: `feedback_review_batch:${action}:${items.map((item) => item.request_id || item.id || "").join(",")}`,
       payload,
     });
+    const requestTargets = items
+      .map((item, index) => firstNonEmptyString(item.request_id, item.id, item._id) || `index:${index}`)
+      .filter(Boolean);
+    const writeApproval = requireWriteApproval(args, payload, {
+      operation: `feedback_review_batch:${action}`,
+      artifactKind: "yaml",
+      targets: requestTargets,
+      batchCount: items.length,
+    });
 
     let data;
     try {
@@ -5426,6 +5717,7 @@ async function runFeedbackReviewDecision(args) {
         payload,
         idempotencyKey,
         clientSessionId,
+        writeApproval,
       });
     } catch (err) {
       if (args.json) {
@@ -5475,6 +5767,11 @@ async function runFeedbackReviewDecision(args) {
     operation: `feedback_review_request_${operationAction}:${requestId}`,
     payload,
   });
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: `feedback_review_request_${operationAction}:${requestId}`,
+    artifactKind: "yaml",
+    targets: [requestId],
+  });
 
   let data;
   try {
@@ -5487,6 +5784,7 @@ async function runFeedbackReviewDecision(args) {
       payload,
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (args.json) {
@@ -5507,7 +5805,7 @@ async function runFeedbackReviewDecision(args) {
   if (postSync?.error) console.error(`Feedback sync warning: ${postSync.error}`);
 }
 
-async function runFeedbackMove(args) {
+async function runFeedbackMove(args) {
   const key = getProjectApiKey();
   if (!key) {
     console.error("Missing MYTE_API_KEY (project key) in environment/.env");
@@ -5545,6 +5843,12 @@ async function runFeedbackMove(args) {
       operation: `feedback_board_batch_move:${toState}:${feedbackIds.join(",")}`,
       payload,
     });
+    const writeApproval = requireWriteApproval(args, payload, {
+      operation: `feedback_board_batch_move:${toState}`,
+      artifactKind: "yaml",
+      targets: feedbackIds,
+      batchCount: feedbackIds.length,
+    });
     let data;
     try {
       data = await postFeedbackBatchBoardMutation({
@@ -5554,6 +5858,7 @@ async function runFeedbackMove(args) {
         payload,
         idempotencyKey,
         clientSessionId,
+        writeApproval,
       });
     } catch (err) {
       if (args.json) {
@@ -5581,6 +5886,11 @@ async function runFeedbackMove(args) {
     operation: `feedback_board_move:${feedbackId}`,
     payload,
   });
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: `feedback_board_move:${feedbackId}`,
+    artifactKind: "yaml",
+    targets: [feedbackId],
+  });
 
   let data;
   try {
@@ -5593,6 +5903,7 @@ async function runFeedbackMove(args) {
       payload,
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (args.json) {
@@ -5613,85 +5924,92 @@ async function runFeedbackMove(args) {
   console.log(`State: ${data.feedback_state || toState}`);
   if (data.event?.event_id) console.log(`Event: ${data.event.event_id}`);
   if (postSync?.error) console.error(`Feedback sync warning: ${postSync.error}`);
-}
-
-async function runFeedbackComment(args) {
-  const key = getProjectApiKey();
-  if (!key) {
-    console.error("Missing MYTE_API_KEY (project key) in environment/.env");
-    process.exit(1);
-  }
-  const feedbackId = resolveFeedbackIdArg(args);
-  if (!feedbackId) {
-    console.error("Missing --feedback-id.");
-    process.exit(1);
-  }
-
-  const content = String(await resolveFeedbackCommentContent(args, feedbackId) || "").trim();
-  if (!content) {
-    console.error("Missing feedback comment body. Use --body, --body-file, --stdin, or positional text.");
-    process.exit(1);
-  }
-
-  const payload = { content };
-  const timeoutMs = resolveTimeoutMs(args);
-  const apiBase = resolveApiBase(args);
-  const clientSessionId = firstNonEmptyString(args["client-session-id"], args.clientSessionId, args.client_session_id);
-  if (clientSessionId) payload.client_session_id = clientSessionId;
-  const idempotencyKey = resolveProjectMutationIdempotencyKey({
-    args,
-    operation: `feedback_comment_create:${feedbackId}`,
-    payload,
-  });
-
-  if (args["print-context"] || args.printContext || args["dry-run"] || args.dryRun) {
-    console.log(JSON.stringify(payload, null, 2));
-    return;
-  }
-
-  let data;
-  try {
-    data = await postFeedbackComment({
-      apiBase,
-      key,
-      timeoutMs,
-      feedbackId,
-      payload,
-      idempotencyKey,
-      clientSessionId,
-    });
-  } catch (err) {
-    if (args.json) {
-      console.log(JSON.stringify({ ok: false, status: err?.status || null, message: err?.message || String(err), data: err?.data || null }, null, 2));
-    } else {
-      console.error("Feedback comment failed:", err?.message || err);
-      if (err?.data) console.error(JSON.stringify(err.data, null, 2));
-    }
-    process.exit(1);
-  }
-
-  const postSync = await refreshFeedbackSnapshotAfterMutation(args, { apiBase, key, timeoutMs });
-  if (args.json) {
-    console.log(JSON.stringify({ ok: true, ...data, post_sync: postSync }, null, 2));
-    return;
-  }
-
-  if (data.comment_id) console.log(`Comment ID: ${data.comment_id}`);
-  console.log(`Feedback: ${data.feedback_id || feedbackId}`);
-  if (data.project_id) console.log(`Project ID: ${data.project_id}`);
-  if (data.user_name) console.log(`Author: ${data.user_name}`);
-  if (data.created_at) console.log(`Created At: ${data.created_at}`);
-  if (data.content) {
-    const body = String(data.content);
-    const truncated = body.length > 2000 ? `${body.slice(0, 2000)}\n...[truncated]` : body;
-    console.log("Content:");
-    console.log(truncated);
-  }
-  if (postSync?.data_root) console.log(`Synced feedback: ${postSync.data_root}`);
-  if (postSync?.error) console.error(`Feedback sync warning: ${postSync.error}`);
-}
-
-async function runFeedbackUndo(args) {
+}
+
+async function runFeedbackComment(args) {
+  const key = getProjectApiKey();
+  if (!key) {
+    console.error("Missing MYTE_API_KEY (project key) in environment/.env");
+    process.exit(1);
+  }
+  const feedbackId = resolveFeedbackIdArg(args);
+  if (!feedbackId) {
+    console.error("Missing --feedback-id.");
+    process.exit(1);
+  }
+
+  const content = String(await resolveFeedbackCommentContent(args, feedbackId) || "").trim();
+  if (!content) {
+    console.error("Missing feedback comment body. Use --body, --body-file, --stdin, or positional text.");
+    process.exit(1);
+  }
+
+  const payload = { content };
+  const timeoutMs = resolveTimeoutMs(args);
+  const apiBase = resolveApiBase(args);
+  const clientSessionId = firstNonEmptyString(args["client-session-id"], args.clientSessionId, args.client_session_id);
+  if (clientSessionId) payload.client_session_id = clientSessionId;
+  const idempotencyKey = resolveProjectMutationIdempotencyKey({
+    args,
+    operation: `feedback_comment_create:${feedbackId}`,
+    payload,
+  });
+
+  if (args["print-context"] || args.printContext || args["dry-run"] || args.dryRun) {
+    console.log(JSON.stringify(payload, null, 2));
+    return;
+  }
+
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: `feedback_comment_create:${feedbackId}`,
+    artifactKind: "markdown",
+    targets: [feedbackId],
+  });
+
+  let data;
+  try {
+    data = await postFeedbackComment({
+      apiBase,
+      key,
+      timeoutMs,
+      feedbackId,
+      payload,
+      idempotencyKey,
+      clientSessionId,
+      writeApproval,
+    });
+  } catch (err) {
+    if (args.json) {
+      console.log(JSON.stringify({ ok: false, status: err?.status || null, message: err?.message || String(err), data: err?.data || null }, null, 2));
+    } else {
+      console.error("Feedback comment failed:", err?.message || err);
+      if (err?.data) console.error(JSON.stringify(err.data, null, 2));
+    }
+    process.exit(1);
+  }
+
+  const postSync = await refreshFeedbackSnapshotAfterMutation(args, { apiBase, key, timeoutMs });
+  if (args.json) {
+    console.log(JSON.stringify({ ok: true, ...data, post_sync: postSync }, null, 2));
+    return;
+  }
+
+  if (data.comment_id) console.log(`Comment ID: ${data.comment_id}`);
+  console.log(`Feedback: ${data.feedback_id || feedbackId}`);
+  if (data.project_id) console.log(`Project ID: ${data.project_id}`);
+  if (data.user_name) console.log(`Author: ${data.user_name}`);
+  if (data.created_at) console.log(`Created At: ${data.created_at}`);
+  if (data.content) {
+    const body = String(data.content);
+    const truncated = body.length > 2000 ? `${body.slice(0, 2000)}\n...[truncated]` : body;
+    console.log("Content:");
+    console.log(truncated);
+  }
+  if (postSync?.data_root) console.log(`Synced feedback: ${postSync.data_root}`);
+  if (postSync?.error) console.error(`Feedback sync warning: ${postSync.error}`);
+}
+
+async function runFeedbackUndo(args) {
   const key = getProjectApiKey();
   if (!key) {
     console.error("Missing MYTE_API_KEY (project key) in environment/.env");
@@ -5717,6 +6035,17 @@ async function runFeedbackUndo(args) {
     payload,
   });
 
+  if (args["print-context"] || args.printContext || args["dry-run"] || args.dryRun) {
+    console.log(JSON.stringify(payload, null, 2));
+    return;
+  }
+
+  const writeApproval = requireWriteApproval(args, payload, {
+    operation: `feedback_event_undo:${feedbackId}:${eventId}`,
+    artifactKind: "yaml",
+    targets: [feedbackId, eventId],
+  });
+
   let data;
   try {
     data = await postFeedbackBoardMutation({
@@ -5728,6 +6057,7 @@ async function runFeedbackUndo(args) {
       payload,
       idempotencyKey,
       clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     if (args.json) {
@@ -5886,15 +6216,15 @@ async function runFeedback(args) {
     await runFeedbackReviewDecision(args);
     return;
   }
-  if (subcommand === "move") {
-    await runFeedbackMove(args);
-    return;
-  }
-  if (subcommand === "comment" || subcommand === "comments") {
-    await runFeedbackComment(args);
-    return;
-  }
-  if (subcommand === "undo") {
+  if (subcommand === "move") {
+    await runFeedbackMove(args);
+    return;
+  }
+  if (subcommand === "comment" || subcommand === "comments") {
+    await runFeedbackComment(args);
+    return;
+  }
+  if (subcommand === "undo") {
     await runFeedbackUndo(args);
     return;
   }
@@ -5910,7 +6240,7 @@ async function runFeedback(args) {
     await runFeedbackGetOrHistory(args, subcommand);
     return;
   }
-  console.error("Unknown feedback command. Use status, edit, assign, archive, submit, revise, reviews, review, move, comment, undo, prd-versions, prd-diff, get, history, validate, or apply.");
+  console.error("Unknown feedback command. Use status, edit, assign, archive, submit, revise, reviews, review, move, comment, undo, prd-versions, prd-diff, get, history, validate, or apply.");
   process.exit(1);
 }
 
@@ -6106,6 +6436,14 @@ async function runSuggestionsCreate(args) {
     return;
   }
 
+  const createTargets = suggestionApprovalTargets(context.payload);
+  const writeApproval = requireWriteApproval(args, context.payload, {
+    operation: "suggestions.create",
+    artifactKind: "yaml",
+    targets: createTargets,
+    batchCount: createTargets.length > 1 ? createTargets.length : null,
+  });
+
   let data;
   try {
     data = await createProjectSuggestions({
@@ -6115,6 +6453,7 @@ async function runSuggestionsCreate(args) {
       payload: context.payload,
       idempotencyKey: context.idempotencyKey,
       clientSessionId: context.clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     console.error("Suggestions create failed:", err?.message || err);
@@ -6191,6 +6530,14 @@ async function runSuggestionsRevise(args) {
     return;
   }
 
+  const reviseTargets = suggestionApprovalTargets(context.payload);
+  const writeApproval = requireWriteApproval(args, context.payload, {
+    operation: "suggestions.revise",
+    artifactKind: "yaml",
+    targets: reviseTargets,
+    batchCount: reviseTargets.length > 1 ? reviseTargets.length : null,
+  });
+
   let data;
   try {
     data = await reviseProjectSuggestions({
@@ -6200,6 +6547,7 @@ async function runSuggestionsRevise(args) {
       payload: context.payload,
       idempotencyKey: context.idempotencyKey,
       clientSessionId: context.clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     console.error("Suggestions revise failed:", err?.message || err);
@@ -6274,6 +6622,14 @@ async function runSuggestionsReview(args) {
     return;
   }
 
+  const reviewTargets = suggestionApprovalTargets(context.payload);
+  const writeApproval = requireWriteApproval(args, context.payload, {
+    operation: "suggestions.review",
+    artifactKind: "yaml",
+    targets: reviewTargets,
+    batchCount: reviewTargets.length > 1 ? reviewTargets.length : null,
+  });
+
   let data;
   try {
     data = await reviewProjectSuggestions({
@@ -6283,6 +6639,7 @@ async function runSuggestionsReview(args) {
       payload: context.payload,
       idempotencyKey: context.idempotencyKey,
       clientSessionId: context.clientSessionId,
+      writeApproval,
     });
   } catch (err) {
     console.error("Suggestions review failed:", err?.message || err);