@ripla/godd-mcp 1.0.7-canary.8 → 1.0.7

package/notes/app/src/test/mockPeriodicSync.ts

@@ -0,0 +1,68 @@
+import { useEffect, useRef } from "react";
+
+/**
+ * `usePeriodicSync` のテスト用 faithful mock を生成する。
+ *
+ * 実フックと同じく `onSync` を ref で捕捉し、effect の依存を
+ * `[enabled, intervalMs, skipInitialSync]` に限定する。初回 mount の即時 sync は
+ * `skipInitialSync` が有効な場合に抑制し、テストから明示的に poll を発火できる
+ * `triggerPeriodicSync` を提供する。
+ *
+ * `isSyncing` / `visibilitychange` / `setInterval` はモデル化しない。対象テストは
+ * `triggerPeriodicSync` で poll を明示発火するため、これらの挙動に依存しない。
+ */
+
+type TriggerPeriodicSync = {
+  mockImplementation: (implementation: () => void) => unknown;
+};
+
+type UsePeriodicSyncOptions = {
+  onSync: () => void;
+  enabled: boolean;
+  intervalMs: number;
+  skipInitialSync?: boolean;
+};
+
+type PeriodicSyncMock = {
+  usePeriodicSync: (options: UsePeriodicSyncOptions) => void;
+};
+
+/**
+ * @public Imported via `require()` (not static `import`) from inside `vi.mock(...)`
+ * factories in {@link ../components/FileInfoPanel.branch-tab.test.tsx} and
+ * {@link ../components/IssueDetailView.test.tsx} to satisfy vi.mock hoisting —
+ * static analysis (knip) does not follow that dynamic require, so it flags this
+ * export as unused even though it is genuinely consumed.
+ */
+export function faithfulPeriodicSyncMock(
+  triggerPeriodicSync: TriggerPeriodicSync,
+): PeriodicSyncMock {
+  return {
+    usePeriodicSync: ({
+      onSync,
+      enabled,
+      intervalMs,
+      skipInitialSync,
+    }: UsePeriodicSyncOptions) => {
+      const onSyncRef = useRef(onSync);
+      const isInitialMountRef = useRef(true);
+
+      useEffect(() => {
+        onSyncRef.current = onSync;
+      }, [onSync]);
+
+      useEffect(() => {
+        // trigger は常に最新 onSync を呼ぶ（mock 差し替え後の closed 応答を確実に拾う）。
+        triggerPeriodicSync.mockImplementation(() => {
+          if (enabled) onSyncRef.current();
+        });
+
+        const isInitialMount = isInitialMountRef.current;
+        isInitialMountRef.current = false;
+        if (!enabled || intervalMs <= 0) return;
+        if (skipInitialSync && isInitialMount) return;
+        onSyncRef.current();
+      }, [enabled, intervalMs, skipInitialSync]);
+    },
+  };
+}

package/notes/app/src/test/mocks.ts

@@ -25,7 +25,7 @@ type CollaborationValue = ReturnType<typeof useCollaboration>;
 type CommentsValue = ReturnType<typeof useComments>;
 
 /** 既定の認証ユーザー（editor）。role/user はテスト側で上書きできる */
-export const DEFAULT_AUTH_USER: UserInfo = {
+const DEFAULT_AUTH_USER: UserInfo = {
   id: "u1",
   username: "alice",
   displayName: "Alice",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ripla/godd-mcp",
-  "version": "1.0.7-canary.8",
+  "version": "1.0.7",
   "type": "module",
   "description": "GoDD MCP Server - AI-powered development workflow tools via Model Context Protocol (slash commands support)",
   "main": "dist/index.js",
@@ -18,6 +18,7 @@
     "templates/terraform/",
     "templates/scripts/",
     "templates/notes-docs/",
+    "templates/feedback/",
     "templates/notes-compose.yml",
     "README.md"
   ],
@@ -28,6 +29,8 @@
     "zod": "^3.24.2"
   },
   "devDependencies": {
+    "@anthropic-ai/bedrock-sdk": "^0.32.1",
+    "@anthropic-ai/sdk": "^0.116.0",
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^22.13.4",
     "@yao-pkg/pkg": "^6.13.1",
@@ -49,12 +52,13 @@
     "test": "vitest run",
     "test:watch": "vitest",
     "encrypt": "tsx src/admin/encrypt.ts",
-    "keygen": "tsx src/admin/keygen.ts",
+    "keygen": "tsx src/license/keygen.ts",
     "build:user": "tsx src/admin/build-customer.ts",
     "godd": "tsx src/cli/godd.ts",
     "godd:init": "tsx src/cli/godd.ts init",
     "godd:install": "tsx src/cli/godd.ts install",
     "build:dist": "node esbuild.config.js && node scripts/build-dist.js",
-    "build:dist:current": "node esbuild.config.js && node scripts/build-dist.js --current"
+    "build:dist:current": "node esbuild.config.js && node scripts/build-dist.js --current",
+    "eval": "tsx tests/eval/run-eval.ts"
   }
 }

package/templates/feedback/feedback-lambda/index.mjs

@@ -0,0 +1,342 @@
+// Lambda Function URL handler — フィードバック受付 → GitHub Issue 起票
+// 認証: GoDD Notes Bot (GitHub App) のインストールトークンを都度生成
+
+import crypto from "node:crypto";
+import {
+  SecretsManagerClient,
+  GetSecretValueCommand,
+} from "@aws-sdk/client-secrets-manager";
+
+const GITHUB_API = "https://api.github.com";
+const TOKEN_BUFFER_SECONDS = 300;
+const JWT_EXPIRY_SECONDS = 600;
+// Function URL のリクエスト上限は 6MB。原画は送らない設計のため 3MB で十分
+const MAX_IMAGE_BYTES = 3 * 1024 * 1024;
+const PNG_SIGNATURE = Buffer.from([0x89, 0x50, 0x4e, 0x47, 0x0d, 0x0a, 0x1a, 0x0a]);
+
+const sm = new SecretsManagerClient();
+
+// シークレットのキャッシュ（コールドスタートごとにリセット）
+const secretCache = {};
+
+async function getSecret(secretName) {
+  if (secretCache[secretName]) return secretCache[secretName];
+  const res = await sm.send(
+    new GetSecretValueCommand({ SecretId: secretName }),
+  );
+  const value = res.SecretString;
+  secretCache[secretName] = value;
+  return value;
+}
+
+// GitHub App インストールトークンのキャッシュ
+let cachedToken = null;
+let cachedTokenExpiresAt = 0;
+
+function generateJWT(appId, privateKey) {
+  const now = Math.floor(Date.now() / 1000);
+  const header = { alg: "RS256", typ: "JWT" };
+  const payload = { iat: now - 60, exp: now + JWT_EXPIRY_SECONDS, iss: appId };
+
+  const b64url = (obj) =>
+    Buffer.from(JSON.stringify(obj)).toString("base64url");
+  const unsigned = `${b64url(header)}.${b64url(payload)}`;
+
+  const sign = crypto.createSign("RSA-SHA256");
+  sign.update(unsigned);
+  const signature = sign.sign(privateKey.replace(/\\n/g, "\n"), "base64url");
+
+  return `${unsigned}.${signature}`;
+}
+
+async function getInstallationToken() {
+  const now = Date.now() / 1000;
+  if (cachedToken && now < cachedTokenExpiresAt - TOKEN_BUFFER_SECONDS) {
+    return cachedToken;
+  }
+
+  const appId = await getSecret(env("GITHUB_APP_ID_SECRET"));
+  const privateKey = await getSecret(env("GITHUB_APP_PRIVATE_KEY_SECRET"));
+  const installationId = await getSecret(
+    env("GITHUB_APP_INSTALLATION_ID_SECRET"),
+  );
+
+  const jwt = generateJWT(appId, privateKey);
+  // トークンは対象リポジトリ + 必要権限に絞って発行する（漏洩時の影響を
+  // インストール配下の他リポジトリへ波及させない）
+  const repoName = env("GITHUB_REPO").split("/")[1];
+  const res = await fetch(
+    `${GITHUB_API}/app/installations/${installationId}/access_tokens`,
+    {
+      method: "POST",
+      headers: {
+        authorization: `Bearer ${jwt}`,
+        accept: "application/vnd.github+json",
+      },
+      body: JSON.stringify({
+        repositories: [repoName],
+        permissions: { contents: "write", issues: "write" },
+      }),
+    },
+  );
+
+  if (!res.ok) {
+    const detail = await res.text().catch(() => "");
+    throw new Error(`GitHub App token error (${res.status}): ${detail}`);
+  }
+
+  const data = await res.json();
+  cachedToken = data.token;
+  cachedTokenExpiresAt = data.expires_at
+    ? new Date(data.expires_at).getTime() / 1000
+    : now + 3600;
+
+  return cachedToken;
+}
+
+function env(key) {
+  const v = process.env[key];
+  if (!v) throw new Error(`環境変数 ${key} が未設定です`);
+  return v;
+}
+
+function respond(statusCode, body) {
+  return {
+    statusCode,
+    headers: { "content-type": "application/json" },
+    body: JSON.stringify(body),
+  };
+}
+
+function decodeDataUrl(dataUrl) {
+  const match = dataUrl.match(/^data:image\/\w+;base64,(.+)$/);
+  if (!match) throw new Error("無効な data URL です");
+  return Buffer.from(match[1], "base64");
+}
+
+async function ghFetch(path, token, opts = {}) {
+  const res = await fetch(`${GITHUB_API}${path}`, {
+    ...opts,
+    headers: {
+      accept: "application/vnd.github+json",
+      authorization: `Bearer ${token}`,
+      "x-github-api-version": "2022-11-28",
+      ...opts.headers,
+    },
+  });
+  const body = await res.json().catch(() => null);
+  if (!res.ok) {
+    const msg = body?.message ?? res.statusText;
+    throw new Error(`GitHub API ${res.status}: ${msg}`);
+  }
+  return body;
+}
+
+async function ensureBranch(repo, branch, token) {
+  const ref = await ghFetch(
+    `/repos/${repo}/git/ref/heads/${branch}`,
+    token,
+  ).catch(() => null);
+  if (ref) return;
+
+  // デフォルトブランチは main とは限らない（master 等）ためリポジトリ情報から取得する
+  const repoInfo = await ghFetch(`/repos/${repo}`, token);
+  const defaultRef = await ghFetch(
+    `/repos/${repo}/git/ref/heads/${repoInfo.default_branch}`,
+    token,
+  );
+  const sha = defaultRef.object.sha;
+
+  await ghFetch(`/repos/${repo}/git/refs`, token, {
+    method: "POST",
+    body: JSON.stringify({ ref: `refs/heads/${branch}`, sha }),
+  });
+}
+
+async function ensureLabel(repo, token) {
+  const label = await ghFetch(`/repos/${repo}/labels/feedback`, token).catch(
+    () => null,
+  );
+  if (label) return;
+
+  // 既存ラベルは触らず、無い場合のみ作成する（同時作成の競合は無視して続行）
+  await ghFetch(`/repos/${repo}/labels`, token, {
+    method: "POST",
+    body: JSON.stringify({
+      name: "feedback",
+      color: "D93F0B",
+      description: "フィードバックウィジェットからの自動起票",
+    }),
+  }).catch(() => {});
+}
+
+async function commitImage(repo, branch, filePath, imageBuffer, token) {
+  await ghFetch(`/repos/${repo}/contents/${filePath}`, token, {
+    method: "PUT",
+    body: JSON.stringify({
+      message: `feedback: add screenshot ${filePath}`,
+      content: imageBuffer.toString("base64"),
+      branch,
+    }),
+  });
+  return `https://github.com/${repo}/blob/${branch}/${filePath}`;
+}
+
+// Markdown テーブルのセルに利用者入力を埋め込むためのエスケープ。
+// パイプ・改行を無害化しないと表の構造ごと壊せてしまう（偽の指示文の埋め込みも含む）
+function cell(value) {
+  return String(value).replace(/\|/g, "\\|").replace(/\r?\n/g, " ");
+}
+
+function buildIssueBody(comment, imageUrl, page, meta, pin, extra = {}) {
+  const lines = [
+    "## フィードバック",
+    "",
+    // 起票者・画面名は任意入力のため、未指定なら見出しごと省略する
+    ...(extra.reporter ? ["### 起票者", "", cell(extra.reporter), ""] : []),
+    ...(extra.screenName ? ["### 画面名", "", cell(extra.screenName), ""] : []),
+    "### 指摘・要望コメント",
+    "",
+    `> ${comment.replace(/\n/g, "\n> ")}`,
+    "",
+    "### スクリーンショット（指摘箇所）",
+    `[スクリーンショットを開く](${imageUrl})`,
+    "",
+    "### メタ情報",
+    `| 項目 | 値 |`,
+    `|---|---|`,
+    // 起票者・画面名は任意入力のため、未指定なら行ごと省略する
+    ...(extra.reporter ? [`| 起票者 | ${cell(extra.reporter)} |`] : []),
+    ...(extra.screenName ? [`| 画面名 | ${cell(extra.screenName)} |`] : []),
+    `| ページ | ${cell(page.path)} |`,
+    `| タイトル | ${cell(page.title)} |`,
+    `| Viewport | ${meta.viewport.width}×${meta.viewport.height} |`,
+    `| UA | ${cell(meta.userAgent)} |`,
+    `| 撮影日時 | ${cell(meta.capturedAt)} |`,
+    `| ピン位置 | (${pin.x.toFixed(1)}%, ${pin.y.toFixed(1)}%) |`,
+  ];
+  return lines.join("\n");
+}
+
+// ペイロードの形状検証。未検証のまま参照すると TypeError → 500 になり、
+// 内部エラーメッセージが送信者へ漏れるため、先に固定文言の 400 で弾く
+function validatePayload(payload) {
+  if (typeof payload !== "object" || payload === null) return "不正なペイロードです";
+  const { comment, annotatedScreenshot, pin, page, meta, reporter, screenName } =
+    payload;
+  if (typeof comment !== "string" || !comment.trim()) return "comment は必須です";
+  if (typeof annotatedScreenshot !== "string")
+    return "annotatedScreenshot は必須です";
+  if (
+    typeof pin !== "object" || pin === null ||
+    !Number.isFinite(pin.x) || !Number.isFinite(pin.y)
+  )
+    return "pin は必須です";
+  if (
+    typeof page !== "object" || page === null ||
+    typeof page.path !== "string" || typeof page.title !== "string"
+  )
+    return "page が不正です";
+  if (
+    typeof meta !== "object" || meta === null ||
+    typeof meta.viewport !== "object" || meta.viewport === null ||
+    !Number.isFinite(meta.viewport.width) || !Number.isFinite(meta.viewport.height) ||
+    typeof meta.userAgent !== "string" || typeof meta.capturedAt !== "string"
+  )
+    return "meta が不正です";
+  if (reporter !== undefined && typeof reporter !== "string")
+    return "reporter が不正です";
+  if (screenName !== undefined && typeof screenName !== "string")
+    return "screenName が不正です";
+  return null;
+}
+
+function isValidToken(provided, expected) {
+  if (typeof provided !== "string") return false;
+  const a = Buffer.from(provided);
+  const b = Buffer.from(expected);
+  // timingSafeEqual は同長でしか比較できないため、長さ不一致は即不正とする
+  if (a.length !== b.length) return false;
+  return crypto.timingSafeEqual(a, b);
+}
+
+export async function handler(event) {
+  // トークン未設定はフェイルクローズ（設定ミスのまま無認証公開しない）
+  const expectedToken = process.env.FEEDBACK_TOKEN;
+  if (!expectedToken) {
+    console.error("FEEDBACK_TOKEN が未設定のためリクエストを拒否しました");
+    return respond(500, { error: "サーバー設定エラー" });
+  }
+  if (!isValidToken(event.headers?.["x-feedback-token"], expectedToken)) {
+    return respond(403, { error: "Forbidden" });
+  }
+
+  try {
+    const token = await getInstallationToken();
+    const repo = env("GITHUB_REPO");
+    const branch = process.env.GITHUB_BRANCH ?? "feedback-assets";
+
+    let payload;
+    try {
+      payload = JSON.parse(event.body);
+    } catch {
+      return respond(400, { error: "JSON の解析に失敗しました" });
+    }
+    const validationError = validatePayload(payload);
+    if (validationError) return respond(400, { error: validationError });
+    const { comment, annotatedScreenshot, pin, page, meta, reporter, screenName } =
+      payload;
+
+    let imageBuffer;
+    try {
+      imageBuffer = decodeDataUrl(annotatedScreenshot);
+    } catch {
+      return respond(400, { error: "画像の形式が不正です" });
+    }
+    if (imageBuffer.length > MAX_IMAGE_BYTES) {
+      return respond(400, {
+        error: "画像が大きすぎます（3MB 以下）。ウィンドウを縮小して撮り直してください",
+      });
+    }
+    // PNG マジックバイト検証: 任意バイト列の置き場として使われるのを防ぐ
+    if (!imageBuffer.subarray(0, 8).equals(PNG_SIGNATURE)) {
+      return respond(400, { error: "PNG 画像のみ受け付けます" });
+    }
+
+    const ts = new Date().toISOString().replace(/[:.]/g, "-").slice(0, 19);
+    const rand = Math.random().toString(36).slice(2, 8);
+    const filePath = `feedback/${ts}_${rand}.png`;
+
+    await ensureBranch(repo, branch, token);
+    await ensureLabel(repo, token);
+    const imageUrl = await commitImage(
+      repo,
+      branch,
+      filePath,
+      imageBuffer,
+      token,
+    );
+
+    const issueBody = buildIssueBody(comment.trim(), imageUrl, page, meta, pin, {
+      reporter: reporter?.trim(),
+      screenName: screenName?.trim(),
+    });
+    const issue = await ghFetch(`/repos/${repo}/issues`, token, {
+      method: "POST",
+      body: JSON.stringify({
+        title: `[フィードバック] ${comment.trim().replace(/\r?\n/g, " ").slice(0, 60)}`,
+        body: issueBody,
+        labels: ["feedback"],
+      }),
+    });
+
+    return respond(
+      200,
+      { issueUrl: issue.html_url, issueNumber: issue.number },
+    );
+  } catch (e) {
+    // 詳細は CloudWatch のみに残し、送信者へは内部情報を返さない
+    console.error(e);
+    return respond(500, { error: "内部エラーが発生しました" });
+  }
+}