@a9n-shoji/rvw 0.10.0 → 0.12.0

package/CHANGELOG.md

@@ -5,6 +5,25 @@
 
 ## [Unreleased]
 
+## [0.12.0] - 2026-10-05
+
+### Added
+
+- 複数のPull Requestを横断するコメント一覧を追加。コメントの本文、対象PR、解決状態をまとめて確認できます。
+- コメント一覧から返信し、対象のコード位置へ移動できるようにしました。
+
+## [0.11.0] - 2026-10-02
+
+### Added
+
+- PR一覧にアーカイブ・解除と、既定ONの「アーカイブ済みを非表示」を追加。Closed / Mergedの絞り込みと独立して利用できます。
+- 独自Skillから操作できる`rvw pr archive`、`rvw pr unarchive`、`rvw pr list`と対応APIを追加。Machine protocolは6を維持し、`pullRequest.archive`と`pullRequest.list`を追加。
+
+### Changed
+
+- アーカイブ状態はローカルDBへ保存し、同期・reset後も維持。アーカイブ済みでも直接閲覧・コメントを継続できます。同梱Skillによる自動アーカイブは行いません。
+- 最終ページが空になった場合の自動補正はブラウザ履歴を置き換え、「戻る」がループしないようにしました。
+
 ## [0.10.0] - 2026-09-23
 
 ### Added

package/README.md

@@ -71,6 +71,14 @@ PR全体の流れを把握するために、Agentが生成した解説（Walkthr
 
 ---
 
+## PRを横断してコメントを読む
+
+入口の`Pull Requests / Comments`でPR単位とコメント単位を切り替えられます。CommentsではPRごとに
+全スレッドの本文・返信を展開表示し、その場で返信、解決／再開できます。PR／スレッド／全体の折りたたみ、
+状態・リポジトリ・PRの絞り込み、返信本文も含む検索に対応します。「PRで開く」でコメントの対象箇所へ移動し、
+戻ると一覧の位置と下書きを保ちます。外部からの返信は自動反映され、並びや表示対象の変更は
+「新しい更新を反映」で適用します。
+
 ## Agent連携（Codex / Claude Code）
 
 ローカルの Agent に Skill を登録することで、PRの構成図生成や、行コメントに対する調査・返信を自動化できます。
@@ -154,6 +162,11 @@ default / namespace / package import、re-export、tsconfig paths、完全なmod
 
 ---
 
+## 見終わったPRを整理する
+
+PR一覧の各行からアーカイブ・解除できます。アーカイブ済みは既定で非表示になり、チェックを外すと再表示できます。
+PRの閲覧やコメントは引き続き利用できます。[操作と独自Skillでの自動化](https://github.com/a9n-shoji/rvw/blob/main/docs/usage.md#pr一覧のアーカイブ)を参照してください。
+
 ## ドキュメント
 
 ソースから `pnpm demo` を実行すると、デモ環境（認証・Agent不要）をローカルで試せます。

package/dist/cli.mjs

@@ -18753,7 +18753,7 @@ function parseWalkthroughUri(uri) {
 }
 
 // src/shared/constants.ts
-var APP_VERSION = "0.10.0";
+var APP_VERSION = "0.12.0";
 var PROTOCOL_VERSION = 6;
 var VIEWER_ID_HEADER = "x-rvw-viewer-id";
 var VIEWER_OPEN_LEASE_HEADER = "x-rvw-viewer-open-lease";
@@ -19014,6 +19014,7 @@ function mapPullRequest(row) {
     githubState: nullableGitHubPullRequestState(row, "github_state"),
     githubIsDraft: nullableBoolean(row, "github_is_draft"),
     githubApprovalCount: nullableNumber(row, "github_approval_count"),
+    archivedAt: nullableString(row, "archived_at"),
     fetchedAt: stringValue(row, "fetched_at"),
     createdAt: stringValue(row, "created_at"),
     updatedAt: stringValue(row, "updated_at")
@@ -19566,7 +19567,7 @@ var RvwDatabase = class {
       "SELECT * FROM pull_requests WHERE github_state = 'OPEN' OR github_state IS NULL ORDER BY updated_at DESC"
     ).all().map(mapPullRequest);
   }
-  listPullRequestSummaries(offset, limit, hideClosedOrMerged = true) {
+  listPullRequestSummaries(offset, limit, hideClosedOrMerged = true, hideArchived = true) {
     const hideClosedOrMergedValue = hideClosedOrMerged ? 1 : 0;
     const rows = this.database.prepare(
       `WITH page AS (
@@ -19580,9 +19581,10 @@ var RvwDatabase = class {
              github_updated_at,
              github_state,
              github_is_draft,
-             github_approval_count
+             github_approval_count,
+             archived_at
            FROM pull_requests
-           WHERE ? = 0 OR github_state IS NULL OR github_state = 'OPEN'
+           WHERE (? = 0 OR github_state IS NULL OR github_state = 'OPEN') AND (? = 0 OR archived_at IS NULL)
            ORDER BY github_updated_at DESC, id DESC
            LIMIT ? OFFSET ?
          ), comment_counts AS (
@@ -19615,6 +19617,7 @@ var RvwDatabase = class {
            pr.github_state,
            pr.github_is_draft,
            pr.github_approval_count,
+           pr.archived_at,
            COALESCE(comment_counts.unresolved_count, 0) AS unresolved_comment_count,
            COALESCE(comment_counts.resolved_count, 0) AS resolved_comment_count,
            COALESCE(walkthrough_counts.walkthrough_count, 0) AS walkthrough_count,
@@ -19624,10 +19627,10 @@ var RvwDatabase = class {
          LEFT JOIN walkthrough_counts ON walkthrough_counts.pull_request_id = pr.id
          LEFT JOIN structure_counts ON structure_counts.pull_request_id = pr.id
          ORDER BY pr.github_updated_at DESC, pr.id DESC`
-    ).all(hideClosedOrMergedValue, limit, offset);
+    ).all(hideClosedOrMergedValue, hideArchived ? 1 : 0, limit, offset);
     const totalRow = this.database.prepare(
-      "SELECT COUNT(*) AS total FROM pull_requests WHERE ? = 0 OR github_state IS NULL OR github_state = 'OPEN'"
-    ).get(hideClosedOrMergedValue);
+      "SELECT COUNT(*) AS total FROM pull_requests WHERE (? = 0 OR github_state IS NULL OR github_state = 'OPEN') AND (? = 0 OR archived_at IS NULL)"
+    ).get(hideClosedOrMergedValue, hideArchived ? 1 : 0);
     if (!totalRow) throw new RvwError("DATABASE_ERROR", "Pull Request\u4EF6\u6570\u3092\u53D6\u5F97\u3067\u304D\u307E\u305B\u3093\u3002");
     return {
       items: rows.map((row) => ({
@@ -19641,6 +19644,7 @@ var RvwDatabase = class {
         githubState: nullableGitHubPullRequestState(row, "github_state"),
         githubIsDraft: nullableBoolean(row, "github_is_draft"),
         githubApprovalCount: nullableNumber(row, "github_approval_count"),
+        archivedAt: nullableString(row, "archived_at"),
         unresolvedCommentCount: numberValue(row, "unresolved_comment_count"),
         resolvedCommentCount: numberValue(row, "resolved_comment_count"),
         walkthroughCount: numberValue(row, "walkthrough_count"),
@@ -19649,6 +19653,18 @@ var RvwDatabase = class {
       total: numberValue(totalRow, "total")
     };
   }
+  setPullRequestArchived(id, archived) {
+    return this.immediateTransaction(() => {
+      const current = this.getPullRequest(id);
+      if (!current)
+        throw new RvwError("PR_NOT_FOUND", "Pull Request\u304C\u898B\u3064\u304B\u308A\u307E\u305B\u3093\u3002", { status: 404 });
+      if (current.archivedAt !== null === archived) return current;
+      const archivedAt = archived ? (/* @__PURE__ */ new Date()).toISOString() : null;
+      this.database.prepare("UPDATE pull_requests SET archived_at = ? WHERE id = ?").run(archivedAt, id);
+      this.incrementDomainRevisions(["pullRequests"]);
+      return { ...current, archivedAt };
+    });
+  }
   updatePullRequestGitHubStatuses(updates) {
     if (updates.length === 0) return;
     this.immediateTransaction(() => {
@@ -20412,6 +20428,70 @@ var RvwDatabase = class {
     ).get(id);
     return row ? this.mapComment(row) : null;
   }
+  listCommentFeed(input) {
+    const parameters = [
+      input.status,
+      input.status,
+      input.hideClosedOrMerged ? 1 : 0,
+      input.repository,
+      input.repository,
+      input.pullRequestId,
+      input.pullRequestId,
+      input.search,
+      input.search
+    ];
+    const matching = `WITH matching AS (
+      SELECT c.id, c.pull_request_id,
+        (SELECT MAX(p.created_at) FROM comment_posts p WHERE p.comment_id = c.id) AS posted_at
+      FROM comments c JOIN pull_requests pr ON pr.id = c.pull_request_id
+      WHERE (? = 'all' OR CASE WHEN ? = 'resolved' THEN c.resolved_at IS NOT NULL ELSE c.resolved_at IS NULL END)
+        AND (? = 0 OR pr.github_state IS NULL OR pr.github_state = 'OPEN')
+        AND (? = '' OR pr.owner || '/' || pr.repository = ?)
+        AND (? = '' OR pr.id = ?)
+        AND (? = '' OR EXISTS (SELECT 1 FROM comment_posts p WHERE p.comment_id = c.id
+          AND instr(lower(p.body), lower(?)) > 0))
+    )`;
+    const totals = this.database.prepare(
+      `${matching}
+      SELECT COUNT(*) AS comments, COUNT(DISTINCT pull_request_id) AS groups FROM matching`
+    ).get(...parameters);
+    const rows = this.database.prepare(
+      `${matching}, page AS (
+      SELECT pull_request_id, MAX(posted_at) AS posted_at FROM matching
+      GROUP BY pull_request_id ORDER BY posted_at DESC, pull_request_id DESC LIMIT ?
+    ) SELECT m.id, m.pull_request_id FROM matching m JOIN page p ON p.pull_request_id = m.pull_request_id
+      ORDER BY p.posted_at DESC, p.pull_request_id DESC, m.posted_at DESC, m.id DESC`
+    ).all(...parameters, input.limit);
+    const pullRequests = this.database.prepare(
+      `SELECT id, owner, repository, number, latest_title, github_state, github_is_draft
+       FROM pull_requests ORDER BY owner, repository, number DESC`
+    ).all().map((row) => ({
+      id: stringValue(row, "id"),
+      owner: stringValue(row, "owner"),
+      repository: stringValue(row, "repository"),
+      number: numberValue(row, "number"),
+      title: stringValue(row, "latest_title"),
+      githubState: nullableString(row, "github_state"),
+      githubIsDraft: row.github_is_draft === null ? null : row.github_is_draft === 1
+    }));
+    const byId = new Map(pullRequests.map((pr) => [pr.id, pr]));
+    const groups = [];
+    for (const row of rows) {
+      const id = stringValue(row, "pull_request_id");
+      let group = groups.at(-1);
+      if (group?.pullRequest.id !== id) {
+        group = { pullRequest: byId.get(id), commentIds: [] };
+        groups.push(group);
+      }
+      group.commentIds.push(stringValue(row, "id"));
+    }
+    return {
+      groups,
+      pullRequests,
+      totalGroups: numberValue(totals, "groups"),
+      totalComments: numberValue(totals, "comments")
+    };
+  }
   listComments(pullRequestId, resolved) {
     const where = resolved === void 0 ? "" : resolved ? " AND c.resolved_at IS NOT NULL" : " AND c.resolved_at IS NULL";
     return this.database.prepare(
@@ -43057,6 +43137,9 @@ var RvwService = class {
       throw new RvwError("PR_NOT_FOUND", "Pull Request\u304C\u898B\u3064\u304B\u308A\u307E\u305B\u3093\u3002", { status: 404 });
     return pullRequest;
   }
+  setPullRequestArchived(id, archived) {
+    return this.database.setPullRequestArchived(id, archived);
+  }
   listPullRequests(input) {
     const offset = input.offset ?? 0;
     const limit = input.limit ?? DEFAULT_PULL_REQUEST_LIST_LIMIT;
@@ -43070,7 +43153,12 @@ var RvwService = class {
         `limit\u306F1\u4EE5\u4E0A${MAX_PULL_REQUEST_LIST_LIMIT}\u4EE5\u4E0B\u306E\u6574\u6570\u306B\u3057\u3066\u304F\u3060\u3055\u3044\u3002`
       );
     }
-    const page = this.database.listPullRequestSummaries(offset, limit, hideClosedOrMerged);
+    const page = this.database.listPullRequestSummaries(
+      offset,
+      limit,
+      hideClosedOrMerged,
+      input.hideArchived ?? true
+    );
     const returned = page.items.length;
     const hasMore = offset + returned < page.total;
     return {
@@ -49938,6 +50026,13 @@ var structurePublishInputSchema = external_exports.object({
 }).strict().superRefine(refineStructureContent);
 var agentCommandInputSchemas = {
   doctor: external_exports.object({ cwd: nonEmptyString }).strict(),
+  "pr.list": external_exports.object({
+    offset: external_exports.number().int().min(0).default(0),
+    limit: external_exports.number().int().min(1).max(MAX_PULL_REQUEST_LIST_LIMIT).default(DEFAULT_PULL_REQUEST_LIST_LIMIT),
+    hideClosedOrMerged: external_exports.boolean().default(true),
+    hideArchived: external_exports.boolean().default(true)
+  }).strict(),
+  "pr.archive": external_exports.object({ reference: nonEmptyString, archived: external_exports.boolean() }).strict(),
   "pr.refresh": external_exports.object({ reference: nonEmptyString }).strict(),
   "pr.sync": pullRequestSyncInputSchema.extend({
     repositoryPath: nonEmptyString.optional(),
@@ -50013,6 +50108,7 @@ var openPullRequestSchema = external_exports.object({
   reference: external_exports.string().min(1).optional(),
   cwd: external_exports.string().min(1)
 });
+var pullRequestArchiveSchema = external_exports.object({ archived: external_exports.boolean() }).strict();
 var resetSchema = external_exports.object({ yes: external_exports.boolean() });
 var structureDeleteSchema = external_exports.object({ expectedUpdatedAt: external_exports.string().min(1).max(100) });
 var viewerIdSchema = external_exports.uuid();
@@ -50021,6 +50117,7 @@ var themePreferenceSchema = external_exports.object({ themePreference: external_
 var pullRequestListQuerySchema = external_exports.object({
   offset: external_exports.coerce.number().int().min(0).default(0),
   limit: external_exports.coerce.number().int().min(1).max(MAX_PULL_REQUEST_LIST_LIMIT).default(DEFAULT_PULL_REQUEST_LIST_LIMIT),
+  hideArchived: external_exports.enum(["true", "false"]).default("true").transform((value) => value === "true"),
   hideClosedOrMerged: external_exports.enum(["true", "false"]).default("true").transform((value) => value === "true")
 });
 var createCommentSchema = external_exports.object({
@@ -50229,10 +50326,29 @@ function createApp(service, options) {
     const query = pullRequestListQuerySchema.parse({
       offset: context.req.query("offset"),
       limit: context.req.query("limit"),
+      hideArchived: context.req.query("hideArchived"),
       hideClosedOrMerged: context.req.query("hideClosedOrMerged")
     });
     return context.json({ ok: true, ...service.listPullRequests(query) });
   });
+  app.patch("/api/pull-requests/:id/archive", async (context) => {
+    const input = pullRequestArchiveSchema.parse(await context.req.json());
+    return context.json({
+      ok: true,
+      pullRequest: service.setPullRequestArchived(context.req.param("id"), input.archived)
+    });
+  });
+  app.get("/api/comment-feed", (context) => {
+    const input = external_exports.object({
+      status: external_exports.enum(["unresolved", "resolved", "all"]).default("unresolved"),
+      hideClosedOrMerged: external_exports.enum(["true", "false"]).default("true").transform((value) => value === "true"),
+      repository: external_exports.string().max(300).default(""),
+      pullRequestId: external_exports.union([external_exports.uuid(), external_exports.literal("")]).default(""),
+      search: external_exports.string().max(500).default(""),
+      limit: external_exports.coerce.number().int().min(1).max(1e3).default(20)
+    }).parse(context.req.query());
+    return context.json({ ok: true, ...service.database.listCommentFeed(input) });
+  });
   app.post(
     "/api/pull-requests/refresh-statuses",
     async (context) => context.json({ ok: true, ...await service.refreshPullRequestStatuses() })
@@ -51019,6 +51135,16 @@ async function dispatchAgentSocketRequest(service, rawRequest) {
       const input = parseOperationInput("doctor", request.input);
       return await service.doctor(input.cwd);
     }
+    case "pr.list": {
+      return service.listPullRequests(parseOperationInput("pr.list", request.input));
+    }
+    case "pr.archive": {
+      const input = parseOperationInput("pr.archive", request.input);
+      return service.setPullRequestArchived(
+        service.resolveStoredPullRequest(input.reference).id,
+        input.archived
+      );
+    }
     case "pr.refresh": {
       const input = parseOperationInput("pr.refresh", request.input);
       return await service.refreshByReference(input.reference);
@@ -52840,6 +52966,8 @@ function createProgram(runtimeFactory = defaultRuntimeFactory) {
         "comment.resolve",
         "comment.reopen",
         "pullRequest.sync",
+        "pullRequest.list",
+        "pullRequest.archive",
         "structure.list",
         "structure.read",
         "structure.preview",
@@ -52906,6 +53034,47 @@ function createProgram(runtimeFactory = defaultRuntimeFactory) {
     }
   });
   const pr = program2.command("pr").description("Pull Request\u72B6\u614B\u3092\u7BA1\u7406");
+  pr.command("list").option("--json", "JSON\u3067\u51FA\u529B").option("--offset <offset>", "\u958B\u59CB\u4F4D\u7F6E", "0").option("--limit <limit>", "\u53D6\u5F97\u4EF6\u6570\uFF08\u6700\u5927100\uFF09", "50").option("--include-archived", "\u30A2\u30FC\u30AB\u30A4\u30D6\u6E08\u307F\u3082\u542B\u3081\u308B").option("--include-closed", "Closed / Merged\u3082\u542B\u3081\u308B").action(
+    async (options) => {
+      const input = {
+        offset: Number(options.offset),
+        limit: Number(options.limit),
+        hideArchived: !options.includeArchived,
+        hideClosedOrMerged: !options.includeClosed
+      };
+      const result = await callService(
+        "pr.list",
+        input,
+        () => getRuntime().service.listPullRequests(input)
+      );
+      writeOutput(
+        options,
+        { ok: true, ...result },
+        result.items.map(
+          (item) => `${item.owner}/${item.repository}#${item.number} ${item.title}${item.archivedAt ? " [archived]" : ""}`
+        ).join("\n") || "\u8A72\u5F53\u3059\u308BPull Request\u306F\u3042\u308A\u307E\u305B\u3093\u3002"
+      );
+    }
+  );
+  for (const [command, archived] of [
+    ["archive", true],
+    ["unarchive", false]
+  ]) {
+    pr.command(command).argument("<pull-request>", "\u767B\u9332\u6E08\u307FPR URL\u307E\u305F\u306F\u756A\u53F7").option("--json", "JSON\u3067\u51FA\u529B").action(async (reference, options) => {
+      const pullRequest = await callService("pr.archive", { reference, archived }, () => {
+        const service = getRuntime().service;
+        return service.setPullRequestArchived(
+          service.resolveStoredPullRequest(reference).id,
+          archived
+        );
+      });
+      writeOutput(
+        options,
+        { ok: true, pullRequest },
+        archived ? "\u30A2\u30FC\u30AB\u30A4\u30D6\u3057\u307E\u3057\u305F\u3002" : "\u30A2\u30FC\u30AB\u30A4\u30D6\u3092\u89E3\u9664\u3057\u307E\u3057\u305F\u3002"
+      );
+    });
+  }
   pr.command("refresh").argument("<pull-request>", "\u767B\u9332\u6E08\u307FPR URL\u307E\u305F\u306F\u756A\u53F7").option("--json", "JSON\u3067\u51FA\u529B").action(async (reference, options) => {
     const result = await callService(
       "pr.refresh",