takt-marp 0.1.0 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "takt-marp",
-  "version": "0.1.0",
+  "version": "0.2.1",
   "description": "takt-marp",
   "repository": {
     "type": "git",

package/scripts/lib/takt-marp-cli.mjs

@@ -6,11 +6,16 @@ import path from "node:path";
 import { parseArgs } from "node:util";
 import { initializeProject } from "./takt-marp-project-init.mjs";
 import { packageScriptPath } from "./takt-marp-runtime-context.mjs";
-import { formatError, SlideWorkflowError } from "./takt-marp-slide-workflow.mjs";
+import {
+  APPROVAL_COMMANDS,
+  formatError,
+  SlideWorkflowError,
+} from "./takt-marp-slide-workflow.mjs";
 
 const WORKFLOW_COMMANDS = ["plan", "compose", "polish", "deliver"];
-const VALID_COMMANDS = ["init", ...WORKFLOW_COMMANDS, "smoke"];
+const VALID_COMMANDS = ["init", ...WORKFLOW_COMMANDS, "approve", "smoke"];
 const RUNNER_SCRIPT = "scripts/takt-marp-run-slide-workflow.mjs";
+const APPROVE_SCRIPT = "scripts/takt-marp-approve-slide-workflow-state.mjs";
 const SMOKE_SCRIPT = "scripts/takt-marp-validate-slide-workflow-smoke.mjs";
 const REQUIRED_PROJECT_DIRS = [".takt/workflows", ".takt/facets"];
 
@@ -24,6 +29,8 @@ function usage() {
     "  compose <slides/deck> [options]   Run the compose workflow for a deck in the current project",
     "  polish <slides/deck> [options]    Run the polish workflow for a deck in the current project",
     "  deliver <slides/deck> [options]   Run the deliver workflow for a deck in the current project",
+    "  approve <slides/deck> <command> --by <name> [--force]",
+    "                                    Approve a workflow state (command: plan or compose)",
     "  smoke [--provider <name>]         Run smoke validation in a temporary project (default provider: mock)",
     "",
     "Workflow options (passed through to the workflow runner unchanged):",
@@ -171,6 +178,62 @@ async function runSmoke(args) {
   return exitCode;
 }
 
+function approveUsage() {
+  return [
+    "Usage: takt-marp approve <slides/deck> <command> --by <name> [--force]",
+    "",
+    `Commands: ${APPROVAL_COMMANDS.join(", ")}`,
+    "",
+    "Options:",
+    "  --by <name>   Approver identifier (required)",
+    "  --force       Overwrite an existing approval",
+    "  --help, -h    Show this message",
+  ].join("\n");
+}
+
+async function runApprove(args) {
+  let parsed;
+  try {
+    parsed = parseArgs({
+      args,
+      allowPositionals: true,
+      options: {
+        by: { type: "string" },
+        force: { type: "boolean", default: false },
+        help: { type: "boolean", short: "h", default: false },
+      },
+    });
+  } catch (error) {
+    throw new SlideWorkflowError(
+      `Invalid approve arguments: ${error.message}. Run 'takt-marp approve --help' for usage.`,
+      "INVALID_ARGS",
+    );
+  }
+  if (parsed.values.help) {
+    console.log(approveUsage());
+    return 0;
+  }
+  assertProjectInitialized();
+  if (parsed.positionals.length !== 2) {
+    throw new SlideWorkflowError(
+      `Expected <slides/deck> and <command>. Run 'takt-marp approve --help' for usage.`,
+      "INVALID_ARGS",
+    );
+  }
+  const [target, command] = parsed.positionals;
+  if (!parsed.values.by) {
+    throw new SlideWorkflowError(
+      `Missing required --by <name>. Run 'takt-marp approve --help' for usage.`,
+      "INVALID_ARGS",
+    );
+  }
+  const approveArgs = [target, command, "--by", parsed.values.by];
+  if (parsed.values.force) {
+    approveArgs.push("--force");
+  }
+  return runPackageScript(APPROVE_SCRIPT, approveArgs);
+}
+
 export async function runCli(argv) {
   const [command, ...rest] = argv;
   if (command === undefined || command === "--help" || command === "-h" || command === "help") {
@@ -188,6 +251,9 @@ export async function runCli(argv) {
     if (command === "init") {
       return await runInit(rest);
     }
+    if (command === "approve") {
+      return await runApprove(rest);
+    }
     if (command === "smoke") {
       return await runSmoke(rest);
     }

package/scripts/takt-marp-validate-slide-workflow-foundation.mjs

@@ -20,6 +20,7 @@ import {
   supervisionPath,
   writeApproval,
 } from "./lib/takt-marp-slide-workflow.mjs";
+import { runCli } from "./lib/takt-marp-cli.mjs";
 
 const checks = [];
 const SCRIPT_DIR = path.dirname(fileURLToPath(import.meta.url));
@@ -306,6 +307,47 @@ async function main() {
     assert(verifyResult.stdout.includes("skipped for non-successful verify report"), `verify verifier did not report skip: ${verifyResult.stdout}`);
   });
 
+  await check("approve command shows help without initialized project", async () => {
+    const root = await fixtureRoot();
+    const originalCwd = process.cwd();
+    let output;
+    try {
+      process.chdir(root);
+      output = await captureStdout(() => runCli(["approve", "--help"]));
+    } finally {
+      process.chdir(originalCwd);
+    }
+    assert(output.includes("Usage: takt-marp approve"), `approve help missing usage: ${output}`);
+  });
+
+  await check("approve command requires initialized project", async () => {
+    const root = await fixtureRoot();
+    const originalCwd = process.cwd();
+    try {
+      process.chdir(root);
+      const code = await runCli(["approve", "slides/demo", "plan", "--by", "foundation-test"]);
+      assert(code !== 0, "approve unexpectedly succeeded in uninitialized project");
+    } finally {
+      process.chdir(originalCwd);
+    }
+  });
+
+  await check("approve command writes approval file", async () => {
+    const root = await initializedFixtureRoot();
+    const targetInfo = await makeDeck(root, "demo");
+    await writeSupervision(targetInfo, "plan", "planned", "passed", "run-plan-1");
+    const originalCwd = process.cwd();
+    try {
+      process.chdir(root);
+      const code = await runCli(["approve", targetInfo.target, "plan", "--by", "foundation-test"]);
+      assert(code === 0, `approve command failed: check stderr`);
+    } finally {
+      process.chdir(originalCwd);
+    }
+    const approval = await readFile(path.join(targetInfo.reviewPath, "plan-approval.md"), "utf8");
+    assert(approval.includes("approved_by: foundation-test"), `approval file missing approver: ${approval}`);
+  });
+
   await check("package scripts expose canonical entrypoints only", async () => {
     const pkg = JSON.parse(await readFile(path.join(process.cwd(), "package.json"), "utf8"));
     const scripts = pkg.scripts ?? {};
@@ -348,6 +390,32 @@ async function fixtureRoot() {
   return root;
 }
 
+async function initializedFixtureRoot() {
+  const root = await fixtureRoot();
+  await mkdir(path.join(root, ".takt", "facets"), { recursive: true });
+  return root;
+}
+
+async function captureStdout(fn) {
+  const originalWrite = process.stdout.write.bind(process.stdout);
+  const chunks = [];
+  process.stdout.write = (chunk, encoding, callback) => {
+    chunks.push(typeof chunk === "string" ? chunk : chunk.toString());
+    if (typeof encoding === "function") {
+      encoding();
+    } else if (typeof callback === "function") {
+      callback();
+    }
+    return true;
+  };
+  try {
+    await fn();
+  } finally {
+    process.stdout.write = originalWrite;
+  }
+  return chunks.join("");
+}
+
 async function makeDeck(root, deckName) {
   const deckPath = path.join(root, "slides", deckName);
   await mkdir(path.join(deckPath, "review"), { recursive: true });

package/templates/project/facets/instructions/takt-marp-visual-generate.md

@@ -9,17 +9,19 @@ planとSLIDES.mdに必要なSVG sourceを生成してください。
    - inline SVG: `SLIDES.md` の該当スライドの図版領域(placeholder 部分)だけを編集し、SVG markup を直接記述します。front matter に `html: true` が未設定の場合は追加してください。
    - `Visual: existing: ...`: 明示された既存画像への参照を `SLIDES.md` の該当スライドの placeholder 部分に挿入します。
 3. SVGはSVGファーストVisualポリシーに従ってください(外部ファイル・inline 両形式に適用)。
-4. 図形内テキストが図形からはみ出していないか確認してください。長いファイル名や英字ラベルは `tspan` で分割し、左右20px以上の余白を残してください。
-5. Marp配置時にスライド枠からSVGがはみ出しにくいよう、`SLIDES.md` の画像指定は個別サイズではなく `visual`、`visual-dense`、`visual-full` classのCSSで制御してください。inline SVGの場合も同様に、親classのcontainment(`--visual-max-height`系token)でサイズを制御してください。
-6. SVGをもう少し大きくしたい場合は、個別スライドの `![h:...]` を増やすのではなく、class側の `--visual-max-height` を調整してください。render evidence による確認は polish command に委譲してください。
-7. 外部ファイル形式の場合は `SLIDES.md`の参照パスが実在するSVGを指すようにしてください。
-8. 既存画像は明示されたものだけ使ってください。
-9. render output の生成や表示品質の最終判定は行わず、source artifact の作成に集中してください。
+4. SVGキャンバスの約90%を図形・矢印・ラベルなどの意味のある要素で使い、余白は約10%以下に抑えてください。`viewBox 0 0 1100 540` では外周の安全域を左右20-30px、上下16-24px程度にし、6:4程度に余白が目立つ構図を作らないでください。
+5. 図形内テキストが図形からはみ出していないか確認してください。長いファイル名や英字ラベルは `tspan` で分割し、左右20px以上の余白を残してください。
+6. Marp配置時にスライド枠からSVGがはみ出しにくいよう、`SLIDES.md` の画像指定は個別サイズではなく `visual`、`visual-dense`、`visual-full` classのCSSで制御してください。inline SVGの場合も同様に、親classのcontainment(`--visual-max-height`系token)でサイズを制御してください。
+7. SVGをもう少し大きくしたい場合は、個別スライドの `![h:...]` を増やすのではなく、class側の `--visual-max-height` を調整してください。render evidence による確認は polish command に委譲してください。
+8. 外部ファイル形式の場合は `SLIDES.md`の参照パスが実在するSVGを指すようにしてください。
+9. 既存画像は明示されたものだけ使ってください。
+10. render output の生成や表示品質の最終判定は行わず、source artifact の作成に集中してください。
 
 **必須出力**
 ## Visual Result
 - Status: generated / needs_input
 - SVG files: (外部ファイル形式・inline形式の両方を記載)
+- Canvas usage checks:
 - Text fit checks:
 - Source reference checks:
 - Files changed:

package/templates/project/facets/policies/takt-marp-svg-first-visual.md

@@ -11,6 +11,7 @@ Marpスライド内の図解は、レビュー可能で修正しやすいSVGを
 | SVG-first | 図解、比較、フロー、概念絵は原則SVGで作る |
 | 別ファイル管理(既定) | SVGは原則`images/*.svg`として保存しMarpから参照する。スライド固有の図版は使い分け基準に従いinlineも可 |
 | 1 SVG 1 message | 1枚のSVGに複数の主張を詰め込まない |
+| 9:1 の面積配分 | SVGキャンバスは図版が約9割、余白が約1割になるように使う |
 | 可読性優先 | 900-1080px幅で読める文字量とサイズにする |
 | テキスト収まり優先 | 文字列は必ず図形内の余白に収め、長いラベルは折り返す |
 | 差分レビュー可能 | 意味のある要素名、整った構造、過度なminifyを避ける |
@@ -20,7 +21,8 @@ Marpスライド内の図解は、レビュー可能で修正しやすいSVGを
 - `viewBox` は原則 `0 0 1100 540`。
 - font-family は `'Noto Sans JP','Hiragino Sans','Yu Gothic',sans-serif`(knowledge の日本語優先フォールバックスタックと同順)。
 - 背景は白または透明。
-- 外周余白は40-60px。
+- SVG全体の面積配分は「図版9:余白1」を目安にし、図形・矢印・ラベルなどの意味のある要素がキャンバスの約90%を使うように配置する。
+- 外周余白は安全域として最小限にし、`viewBox 0 0 1100 540` では左右20-30px、上下16-24pxを目安にする。余白を広く取って図版が6:4程度に小さく見える構図は禁止する。
 - 図形内テキストは左右20px以上、上下12px以上の内側余白を残す。
 - ファイル名、URL、長い英字ラベルは1行で入れず、`tspan`で2行以上に分割する。
 - 箱の幅が160px未満の場合、中央ラベルは短語だけにする。`plan.md` のような短い名前でも幅200px以上を優先する。
@@ -37,7 +39,7 @@ Marpスライド内の図解は、レビュー可能で修正しやすいSVGを
 - 標準classは `visual`、本文が多い場合は `visual-dense`、図だけを大きく見せる場合は `visual-full` を使う。
 - `visual` の目安は `--visual-max-height: 280px` から `300px`。`visual-dense` は `240px` から `260px`、`visual-full` は `360px` から `410px` を目安にする。
 - `![w:900]` 以上の幅指定は、本文がほぼないスライドに限定する。
-- SVG内の重要要素は上下左右40px以上の内側余白を残し、縮小表示時に端の矢印やラベルが切れないようにする。
+- SVG内の重要要素は、端で切れない範囲で外周近くまで広げる。`viewBox 0 0 1100 540` では左右20-30px、上下16-24px程度の安全域を残し、意味のある図版領域を約90%まで拡大する。
 - SVG単体の `viewBox` 内に収まっていても、Marpに配置した結果スライド枠からはみ出す場合は失敗として扱う。
 
 ## 外部SVGとinline SVGの使い分け基準