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

package/notes-app/src/components/IssueDetailView.test.tsx

@@ -1,201 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import { render, screen, fireEvent, waitFor, act } from "@testing-library/react";
-
-vi.mock("@/lib/api", () => ({
-  createIssueApi: vi.fn(),
-  updateIssueApi: vi.fn(),
-  getIssueApi: vi.fn(),
-  getErrorMessage: vi.fn((e: unknown) => String(e)),
-  isAbortError: vi.fn(() => false),
-}));
-
-vi.mock("@/contexts/AuthContext", () => ({
-  useAuth: () => ({ canEdit: true }),
-}));
-
-const mockShowToast = vi.hoisted(() => vi.fn());
-vi.mock("@/contexts/ToastContext", () => ({
-  useToast: () => ({ showToast: mockShowToast }),
-}));
-
-vi.mock("@/components/FileContentView", () => ({
-  default: ({ content }: { content: string }) => (
-    <div data-testid="file-content-view">{content}</div>
-  ),
-}));
-
-import { createIssueApi, updateIssueApi, getIssueApi } from "@/lib/api";
-import IssueDetailView from "./IssueDetailView";
-
-const mockCreateIssue = vi.mocked(createIssueApi);
-const mockUpdateIssue = vi.mocked(updateIssueApi);
-const mockGetIssue = vi.mocked(getIssueApi);
-
-const MOCK_ISSUE = {
-  number: 123,
-  title: "テスト Issue",
-  body: "本文テスト",
-  state: "open" as const,
-  author: "testuser",
-  assignees: [],
-  comments: 0,
-  createdAt: "2026-05-19T00:00:00Z",
-  updatedAt: "2026-05-19T00:00:00Z",
-  url: "https://github.com/test/repo/issues/123",
-  labels: [],
-};
-
-describe("IssueDetailView — 新規 Issue 作成", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it("作成フォームが表示される（isNew=true）", () => {
-    render(
-      <IssueDetailView issueNumber="new" onCreated={vi.fn()} onClose={vi.fn()} />,
-    );
-    expect(screen.getByText("新規 Issue 作成")).toBeTruthy();
-    expect(screen.getByPlaceholderText("Issue のタイトル")).toBeTruthy();
-    expect(screen.getByText("作成")).toBeTruthy();
-  });
-
-  it("タイトル未入力時は作成ボタンが disabled", () => {
-    render(
-      <IssueDetailView issueNumber="new" onCreated={vi.fn()} onClose={vi.fn()} />,
-    );
-    const btn = screen.getByText("作成");
-    expect((btn as HTMLButtonElement).disabled).toBe(true);
-  });
-
-  it("作成成功後にプレビュー画面へ遷移し、onCreated が呼ばれる", async () => {
-    mockCreateIssue.mockResolvedValue(MOCK_ISSUE);
-
-    const onCreated = vi.fn();
-    render(
-      <IssueDetailView issueNumber="new" onCreated={onCreated} onClose={vi.fn()} />,
-    );
-
-    const titleInput = screen.getByPlaceholderText("Issue のタイトル");
-    fireEvent.change(titleInput, { target: { value: "テスト Issue" } });
-
-    await act(async () => {
-      fireEvent.click(screen.getByText("作成"));
-    });
-
-    // プレビュー画面に遷移（編集フォームが消える）
-    await waitFor(() => {
-      expect(screen.queryByText("新規 Issue 作成")).toBeNull();
-    });
-
-    // Issue タイトルがプレビュー表示される
-    expect(screen.getByText("テスト Issue")).toBeTruthy();
-
-    // onCreated コールバックが作成された Issue 番号で呼ばれる
-    expect(onCreated).toHaveBeenCalledWith(123);
-  });
-
-  it("キャンセルボタンで onClose が呼ばれる", () => {
-    const onClose = vi.fn();
-    render(
-      <IssueDetailView issueNumber="new" onCreated={vi.fn()} onClose={onClose} />,
-    );
-    fireEvent.click(screen.getByText("キャンセル"));
-    expect(onClose).toHaveBeenCalledOnce();
-  });
-});
-
-describe("IssueDetailView — 既存 Issue 更新", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockGetIssue.mockResolvedValue(MOCK_ISSUE);
-  });
-
-  it("更新成功後にプレビュー画面へ遷移する（既存の更新と同じ挙動）", async () => {
-    const updatedIssue = { ...MOCK_ISSUE, title: "更新後タイトル" };
-    mockUpdateIssue.mockResolvedValue(updatedIssue);
-
-    render(
-      <IssueDetailView issueNumber={123} onCreated={vi.fn()} onClose={vi.fn()} />,
-    );
-
-    await waitFor(() => {
-      expect(screen.getByText("テスト Issue")).toBeTruthy();
-    });
-
-    fireEvent.click(screen.getByText("編集"));
-
-    const titleInput = screen.getByDisplayValue("テスト Issue");
-    fireEvent.change(titleInput, { target: { value: "更新後タイトル" } });
-
-    await act(async () => {
-      fireEvent.click(screen.getByText("保存"));
-    });
-
-    await waitFor(() => {
-      expect(screen.queryByText("保存")).toBeNull();
-    });
-
-    expect(screen.getByText("更新後タイトル")).toBeTruthy();
-  });
-});
-
-// #708 固有シナリオ
-describe("IssueDetailView — #708 「新規 Issue」ボタンのリセット挙動", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mockGetIssue.mockResolvedValue(MOCK_ISSUE);
-  });
-
-  it("既存 Issue 編集・保存後に 'new' でリマウントすると空のフォームが表示される", async () => {
-    const updatedIssue = { ...MOCK_ISSUE, title: "更新後" };
-    mockUpdateIssue.mockResolvedValue(updatedIssue);
-
-    const { unmount } = render(
-      <IssueDetailView issueNumber={123} onCreated={vi.fn()} onClose={vi.fn()} />,
-    );
-
-    await waitFor(() => {
-      expect(screen.getByText("テスト Issue")).toBeTruthy();
-    });
-
-    fireEvent.click(screen.getByText("編集"));
-    const titleInput = screen.getByDisplayValue("テスト Issue");
-    fireEvent.change(titleInput, { target: { value: "更新後" } });
-    await act(async () => {
-      fireEvent.click(screen.getByText("保存"));
-    });
-    await waitFor(() => {
-      expect(screen.queryByText("保存")).toBeNull();
-    });
-
-    // 親コンポーネントが key を変えることでリマウント（「新規 Issue」ボタン相当）
-    unmount();
-    render(
-      <IssueDetailView issueNumber="new" onCreated={vi.fn()} onClose={vi.fn()} />,
-    );
-
-    expect(screen.getByText("新規 Issue 作成")).toBeTruthy();
-    const newTitleInput = screen.getByPlaceholderText("Issue のタイトル") as HTMLInputElement;
-    expect(newTitleInput.value).toBe("");
-  });
-
-  it("新規 Issue フォーム入力途中にリマウントすると空のフォームにリセットされる", () => {
-    const { unmount } = render(
-      <IssueDetailView issueNumber="new" onCreated={vi.fn()} onClose={vi.fn()} />,
-    );
-
-    const titleInput = screen.getByPlaceholderText("Issue のタイトル") as HTMLInputElement;
-    fireEvent.change(titleInput, { target: { value: "途中入力" } });
-    expect(titleInput.value).toBe("途中入力");
-
-    // 「新規 Issue」ボタン再クリック相当（newIssueCounter インクリメント → リマウント）
-    unmount();
-    render(
-      <IssueDetailView issueNumber="new" onCreated={vi.fn()} onClose={vi.fn()} />,
-    );
-
-    const freshInput = screen.getByPlaceholderText("Issue のタイトル") as HTMLInputElement;
-    expect(freshInput.value).toBe("");
-    expect(screen.getByText("新規 Issue 作成")).toBeTruthy();
-  });
-});

package/notes-app/src/components/IssueDetailView.tsx

@@ -1,318 +0,0 @@
-import { useState, useEffect, useCallback, lazy, Suspense } from "react";
-import {
-  getIssueApi,
-  createIssueApi,
-  updateIssueApi,
-  getErrorMessage,
-  isAbortError,
-} from "@/lib/api";
-import type { IssueDetail } from "@/lib/api";
-import { useAuth } from "@/contexts/AuthContext";
-import { useToast } from "@/contexts/ToastContext";
-import {
-  CircleDot,
-  CircleCheck,
-  ExternalLink,
-  Save,
-  X,
-  Pencil,
-  Plus,
-} from "lucide-react";
-import { ContentSkeleton, LoadingCircle } from "@/components/Skeleton";
-import LabelSelector from "@/components/LabelSelector";
-
-const FileContentView = lazy(() => import("@/components/FileContentView"));
-
-type IssueDetailViewProps = {
-  issueNumber: number | "new";
-  onCreated?: (number: number) => void;
-  onClose?: () => void;
-};
-
-export default function IssueDetailView({
-  issueNumber,
-  onCreated,
-  onClose,
-}: IssueDetailViewProps) {
-  const { canEdit } = useAuth();
-  const { showToast } = useToast();
-
-  const isNew = issueNumber === "new";
-  const [issue, setIssue] = useState<IssueDetail | null>(null);
-  const [loading, setLoading] = useState(!isNew);
-  const [editing, setEditing] = useState(isNew);
-  const [saving, setSaving] = useState(false);
-
-  const [title, setTitle] = useState("");
-  const [body, setBody] = useState("");
-  const [selectedLabels, setSelectedLabels] = useState<string[]>([]);
-
-  const loadIssue = useCallback(() => {
-    if (isNew) return;
-    setIssue(null);
-    setLoading(true);
-    getIssueApi(issueNumber as number)
-      .then((data) => {
-        setIssue(data);
-        setTitle(data.title);
-        setBody(data.body);
-        setSelectedLabels(data.labels.map((l) => l.name));
-      })
-      .catch((e) => {
-        if (isAbortError(e)) return;
-        setIssue(null);
-        showToast(`Issue の読み込みに失敗しました: ${getErrorMessage(e)}`, "error");
-      })
-      .finally(() => setLoading(false));
-  }, [issueNumber, isNew, showToast]);
-
-  useEffect(() => {
-    loadIssue();
-  }, [loadIssue]);
-
-  const handleSave = useCallback(async () => {
-    if (!title.trim()) {
-      showToast("タイトルを入力してください", "error");
-      return;
-    }
-    setSaving(true);
-    try {
-      if (isNew) {
-        const created = await createIssueApi({
-          title: title.trim(),
-          body,
-          labels: selectedLabels.length > 0 ? selectedLabels : undefined,
-        });
-        showToast(`Issue #${created.number} を作成しました`, "success");
-        setIssue(created);
-        setEditing(false);
-        onCreated?.(created.number);
-      } else {
-        const updated = await updateIssueApi(issueNumber as number, {
-          title: title.trim(),
-          body,
-          labels: selectedLabels,
-        });
-        setIssue(updated);
-        setEditing(false);
-        showToast("Issue を更新しました", "success");
-      }
-    } catch (e) {
-      showToast(`保存に失敗しました: ${getErrorMessage(e)}`, "error");
-    } finally {
-      setSaving(false);
-    }
-  }, [title, body, selectedLabels, isNew, issueNumber, showToast, onCreated]);
-
-  const handleToggleState = useCallback(async () => {
-    if (!issue) return;
-    const newState = issue.state === "open" ? "closed" : "open";
-    setSaving(true);
-    try {
-      const updated = await updateIssueApi(issue.number, { state: newState });
-      setIssue(updated);
-      showToast(
-        newState === "closed" ? "Issue をクローズしました" : "Issue を再オープンしました",
-        "success",
-      );
-    } catch (e) {
-      showToast(`状態変更に失敗しました: ${getErrorMessage(e)}`, "error");
-    } finally {
-      setSaving(false);
-    }
-  }, [issue, showToast]);
-
-  if (loading) {
-    return <LoadingCircle message="Issue を読み込み中…" />;
-  }
-
-  if (editing) {
-    return (
-      <div className="p-4 sm:p-6 max-w-3xl mx-auto">
-        <h2 className="text-lg font-bold text-gray-200 mb-4 flex items-center gap-2">
-          {isNew ? (
-            <>
-              <Plus className="w-5 h-5 text-green-400" />
-              新規 Issue 作成
-            </>
-          ) : (
-            <>
-              <Pencil className="w-5 h-5 text-blue-400" />
-              Issue #{issueNumber} 編集
-            </>
-          )}
-        </h2>
-
-        <div className="space-y-4">
-          <div>
-            <label className="block text-xs text-gray-400 mb-1">タイトル</label>
-            <input
-              type="text"
-              value={title}
-              onChange={(e) => setTitle(e.target.value)}
-              placeholder="Issue のタイトル"
-              className="w-full px-3 py-2 rounded bg-gray-900 border border-gray-600 text-gray-200 placeholder-gray-500 outline-none focus:border-blue-500 text-sm"
-              autoFocus
-            />
-          </div>
-
-          <LabelSelector
-            selected={selectedLabels}
-            onChange={setSelectedLabels}
-            disabled={saving}
-          />
-
-          <div>
-            <label className="block text-xs text-gray-400 mb-1">本文（Markdown）</label>
-            <textarea
-              value={body}
-              onChange={(e) => setBody(e.target.value)}
-              placeholder="Issue の詳細を Markdown で記述…"
-              rows={12}
-              className="w-full px-3 py-2 rounded bg-gray-900 border border-gray-600 text-gray-200 placeholder-gray-500 outline-none focus:border-blue-500 text-sm font-mono resize-y"
-            />
-          </div>
-
-          <div className="flex items-center gap-2">
-            <button
-              type="button"
-              onClick={handleSave}
-              disabled={saving || !title.trim()}
-              className="px-4 py-2 rounded text-sm bg-blue-600 hover:bg-blue-700 text-white transition-colors disabled:opacity-50 flex items-center gap-1.5"
-            >
-              <Save className="w-4 h-4" />
-              {saving ? "保存中…" : isNew ? "作成" : "保存"}
-            </button>
-            <button
-              type="button"
-              onClick={() => {
-                if (isNew) {
-                  onClose?.();
-                } else {
-                  setTitle(issue?.title ?? "");
-                  setBody(issue?.body ?? "");
-                  setSelectedLabels(issue?.labels.map((l) => l.name) ?? []);
-                  setEditing(false);
-                }
-              }}
-              className="px-4 py-2 rounded text-sm text-gray-400 hover:text-gray-200 hover:bg-gray-700 transition-colors flex items-center gap-1.5"
-            >
-              <X className="w-4 h-4" />
-              キャンセル
-            </button>
-          </div>
-        </div>
-      </div>
-    );
-  }
-
-  if (!issue) {
-    return (
-      <div className="flex items-center justify-center h-full p-6">
-        <p className="text-gray-500">Issue が見つかりませんでした</p>
-      </div>
-    );
-  }
-
-  return (
-    <div className="p-4 sm:p-6 max-w-3xl mx-auto">
-      {/* Header */}
-      <div className="mb-4">
-        <h1 className="text-xl font-bold text-gray-100 flex items-start gap-2">
-          {issue.state === "open" ? (
-            <CircleDot className="w-5 h-5 text-green-400 mt-1 shrink-0" />
-          ) : (
-            <CircleCheck className="w-5 h-5 text-purple-400 mt-1 shrink-0" />
-          )}
-          <span>{issue.title}</span>
-          <span className="text-gray-500 font-normal">#{issue.number}</span>
-        </h1>
-
-        <div className="flex items-center gap-3 mt-2 text-xs text-gray-400">
-          <span>{issue.author}</span>
-          <span>
-            {new Date(issue.createdAt).toLocaleDateString("ja-JP")}
-          </span>
-          {issue.url && (
-            <a
-              href={issue.url}
-              target="_blank"
-              rel="noopener noreferrer"
-              className="flex items-center gap-1 text-blue-400 hover:text-blue-300"
-            >
-              <ExternalLink className="w-3 h-3" />
-              GitHub
-            </a>
-          )}
-        </div>
-
-        {issue.labels.length > 0 && (
-          <div className="flex flex-wrap gap-1 mt-2">
-            {issue.labels.map((l) => (
-              <span
-                key={l.name}
-                className="px-2 py-0.5 rounded-full text-xs"
-                style={{
-                  backgroundColor: `#${l.color}22`,
-                  color: `#${l.color}`,
-                  border: `1px solid #${l.color}44`,
-                }}
-              >
-                {l.name}
-              </span>
-            ))}
-          </div>
-        )}
-
-        {/* Action buttons */}
-        {canEdit && (
-          <div className="flex items-center gap-2 mt-3">
-            <button
-              type="button"
-              onClick={() => setEditing(true)}
-              className="px-3 py-1 rounded text-sm bg-gray-700 hover:bg-gray-600 text-gray-200 transition-colors flex items-center gap-1.5"
-            >
-              <Pencil className="w-3.5 h-3.5" />
-              編集
-            </button>
-            <button
-              type="button"
-              onClick={handleToggleState}
-              disabled={saving}
-              className={`px-3 py-1 rounded text-sm transition-colors flex items-center gap-1.5 ${
-                issue.state === "open"
-                  ? "bg-purple-700/30 text-purple-300 hover:bg-purple-700/50"
-                  : "bg-green-700/30 text-green-300 hover:bg-green-700/50"
-              }`}
-            >
-              {issue.state === "open" ? (
-                <>
-                  <CircleCheck className="w-3.5 h-3.5" />
-                  クローズ
-                </>
-              ) : (
-                <>
-                  <CircleDot className="w-3.5 h-3.5" />
-                  再オープン
-                </>
-              )}
-            </button>
-          </div>
-        )}
-      </div>
-
-      {/* Body */}
-      {issue.body ? (
-        <div className="border-t border-gray-700 pt-4">
-          <Suspense fallback={<ContentSkeleton />}>
-            <FileContentView path="issue.md" content={issue.body} encoding="utf-8" />
-          </Suspense>
-        </div>
-      ) : (
-        <div className="border-t border-gray-700 pt-4">
-          <p className="text-gray-500 text-sm italic">本文がありません</p>
-        </div>
-      )}
-    </div>
-  );
-}

package/notes-app/src/components/IssueList.test.tsx

@@ -1,93 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import { render, screen, waitFor } from "@testing-library/react";
-
-vi.mock("@/lib/api", () => ({
-  listIssuesApi: vi.fn(),
-  getErrorMessage: vi.fn((e: unknown) => String(e)),
-  isAbortError: vi.fn(() => false),
-}));
-
-import { listIssuesApi } from "@/lib/api";
-import IssueList from "./IssueList";
-
-const mockListIssues = vi.mocked(listIssuesApi);
-
-const MOCK_ISSUES = [
-  {
-    number: 10,
-    title: "Issue 10",
-    state: "open" as const,
-    author: "user1",
-    assignees: [],
-    comments: 0,
-    labels: [],
-    createdAt: "2026-05-19T00:00:00Z",
-    updatedAt: "2026-05-19T00:00:00Z",
-  },
-  {
-    number: 9,
-    title: "Issue 9",
-    state: "open" as const,
-    author: "user1",
-    assignees: [],
-    comments: 0,
-    labels: [],
-    createdAt: "2026-05-18T00:00:00Z",
-    updatedAt: "2026-05-18T00:00:00Z",
-  },
-];
-
-describe("IssueList", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it("Issueリストを取得して表示する", async () => {
-    mockListIssues.mockResolvedValue({ issues: MOCK_ISSUES });
-
-    render(<IssueList onSelect={vi.fn()} selectedIssue={null} />);
-
-    await waitFor(() => {
-      expect(screen.getByText("Issue 10")).toBeTruthy();
-      expect(screen.getByText("Issue 9")).toBeTruthy();
-    });
-  });
-
-  it("reloadTrigger が変わると listIssuesApi を再呼び出しする", async () => {
-    mockListIssues.mockResolvedValue({ issues: MOCK_ISSUES });
-
-    const { rerender } = render(
-      <IssueList onSelect={vi.fn()} selectedIssue={null} reloadTrigger={0} />,
-    );
-
-    await waitFor(() => {
-      expect(mockListIssues).toHaveBeenCalledTimes(1);
-    });
-
-    rerender(
-      <IssueList onSelect={vi.fn()} selectedIssue={null} reloadTrigger={1} />,
-    );
-
-    await waitFor(() => {
-      expect(mockListIssues).toHaveBeenCalledTimes(2);
-    });
-  });
-
-  it("reloadTrigger が同じ値のままなら再呼び出しされない", async () => {
-    mockListIssues.mockResolvedValue({ issues: MOCK_ISSUES });
-
-    const { rerender } = render(
-      <IssueList onSelect={vi.fn()} selectedIssue={null} reloadTrigger={0} />,
-    );
-
-    await waitFor(() => {
-      expect(mockListIssues).toHaveBeenCalledTimes(1);
-    });
-
-    rerender(
-      <IssueList onSelect={vi.fn()} selectedIssue={null} reloadTrigger={0} />,
-    );
-
-    expect(mockListIssues).toHaveBeenCalledTimes(1);
-  });
-});