nugit-cli 0.0.1 → 0.1.0

package/src/stack-view/fetch-pr-data.js

@@ -1,4 +1,11 @@
-import { githubGetPull, githubListPullFiles, githubRestJson } from "../github-rest.js";
+import {
+  githubGetPull,
+  githubListPullFiles,
+  githubRestJson,
+  githubCompareRefs
+} from "../github-rest.js";
+import { githubListPullReviewsAll } from "../github-pr-social.js";
+import { compareLooksLikeMainMergesOnly } from "../review-hub/review-autoapprove.js";
 
 /**
  * @param {unknown[]} items
@@ -80,35 +87,127 @@ function summarizeReviewState(reviews) {
   return "none";
 }
 
+/**
+ * @param {string} owner
+ * @param {string} repo
+ * @param {unknown[]} reviews
+ * @param {string} viewerLogin
+ * @param {string} headSha
+ * @param {string} defaultBranch
+ */
+async function buildViewerReviewMeta(owner, repo, reviews, viewerLogin, headSha, defaultBranch) {
+  /** @type {{ staleApproval: boolean, lastApprovalCommit: string | null, headSha: string, dismissedReview: boolean, riskyChangeAfterApproval: boolean | null, commitsSinceApproval: number }} */
+  const meta = {
+    staleApproval: false,
+    lastApprovalCommit: null,
+    headSha,
+    dismissedReview: false,
+    riskyChangeAfterApproval: null,
+    commitsSinceApproval: 0
+  };
+  if (!viewerLogin || !headSha) {
+    return meta;
+  }
+  for (const rv of reviews) {
+    if (!rv || typeof rv !== "object") continue;
+    const r = /** @type {Record<string, unknown>} */ (rv);
+    const user = r.user && typeof r.user === "object" ? /** @type {Record<string, unknown>} */ (r.user) : {};
+    const login = typeof user.login === "string" ? user.login : "";
+    if (login !== viewerLogin) continue;
+    const state = typeof r.state === "string" ? r.state.toUpperCase() : "";
+    if (state === "DISMISSED") {
+      meta.dismissedReview = true;
+    }
+  }
+  let lastApprovalCommit = null;
+  for (let i = reviews.length - 1; i >= 0; i--) {
+    const rv = reviews[i];
+    if (!rv || typeof rv !== "object") continue;
+    const r = /** @type {Record<string, unknown>} */ (rv);
+    const user = r.user && typeof r.user === "object" ? /** @type {Record<string, unknown>} */ (r.user) : {};
+    const login = typeof user.login === "string" ? user.login : "";
+    if (login !== viewerLogin) continue;
+    const state = typeof r.state === "string" ? r.state.toUpperCase() : "";
+    if (state === "APPROVED" && typeof r.commit_id === "string") {
+      lastApprovalCommit = r.commit_id;
+      break;
+    }
+  }
+  meta.lastApprovalCommit = lastApprovalCommit;
+  if (lastApprovalCommit && lastApprovalCommit !== headSha) {
+    meta.staleApproval = true;
+    try {
+      const cmp = await githubCompareRefs(owner, repo, lastApprovalCommit, headSha);
+      const total = typeof cmp.total_commits === "number" ? cmp.total_commits : 0;
+      meta.commitsSinceApproval = total;
+      const mergeOnly = compareLooksLikeMainMergesOnly(
+        /** @type {Record<string, unknown>} */ (cmp),
+        defaultBranch
+      );
+      meta.riskyChangeAfterApproval = !mergeOnly;
+    } catch {
+      meta.riskyChangeAfterApproval = true;
+    }
+  }
+  return meta;
+}
+
 /**
  * @param {string} owner
  * @param {string} repo
  * @param {Array<{ pr_number: number, position: number, head_branch?: string, base_branch?: string }>} prEntries
  * @param {number} [concurrency]
+ * @param {{ viewerLogin?: string, defaultBranch?: string, fullReviewFetch?: boolean }} [fetchOpts]
  */
-export async function fetchStackPrDetails(owner, repo, prEntries, concurrency = 6) {
+export async function fetchStackPrDetails(owner, repo, prEntries, concurrency = 6, fetchOpts = {}) {
+  const viewerLogin = fetchOpts.viewerLogin || "";
+  const defaultBranch = fetchOpts.defaultBranch || "main";
+  const fullFetch = !!fetchOpts.fullReviewFetch || !!viewerLogin;
+
   const sorted = [...prEntries].sort((a, b) => a.position - b.position);
   return mapInBatches(sorted, concurrency, async (entry) => {
     const n = entry.pr_number;
     try {
       const pull = await githubGetPull(owner, repo, n);
+      const headSha =
+        pull && typeof pull === "object" && pull.head && typeof pull.head === "object"
+          ? String(/** @type {Record<string, unknown>} */ (pull.head).sha || "")
+          : "";
+
       const [issueRes, reviewRes, filesRes, reviewsRes] = await Promise.allSettled([
         githubListIssueCommentsFirstPage(owner, repo, n),
         githubListPullReviewCommentsFirstPage(owner, repo, n),
         githubListPullFiles(owner, repo, n),
-        githubListPullReviewsFirstPage(owner, repo, n)
+        fullFetch
+          ? githubListPullReviewsAll(owner, repo, n)
+          : githubListPullReviewsFirstPage(owner, repo, n)
       ]);
       const issueComments = issueRes.status === "fulfilled" ? issueRes.value : [];
       const reviewComments = reviewRes.status === "fulfilled" ? reviewRes.value : [];
       const files = filesRes.status === "fulfilled" ? filesRes.value : [];
       const reviews = reviewsRes.status === "fulfilled" ? reviewsRes.value : [];
+      const reviewList = Array.isArray(reviews) ? reviews : [];
+
+      let viewerReviewMeta = null;
+      if (viewerLogin && headSha) {
+        viewerReviewMeta = await buildViewerReviewMeta(
+          owner,
+          repo,
+          reviewList,
+          viewerLogin,
+          headSha,
+          defaultBranch
+        );
+      }
+
       return {
         entry,
         pull,
         issueComments,
         reviewComments,
         files: Array.isArray(files) ? files : [],
-        reviewSummary: summarizeReviewState(reviews),
+        reviewSummary: summarizeReviewState(reviewList),
+        viewerReviewMeta,
         error: null
       };
     } catch (e) {
@@ -119,6 +218,7 @@ export async function fetchStackPrDetails(owner, repo, prEntries, concurrency =
         reviewComments: [],
         files: [],
         reviewSummary: "none",
+        viewerReviewMeta: null,
         error: String(/** @type {Error} */ (e).message || e)
       };
     }

package/src/stack-view/infer-chains-to-pick-stacks.js

@@ -0,0 +1,70 @@
+/**
+ * Build StackPickInk-shaped entries from inferred PR chains (bottom → tip) and GitHub pull payloads.
+ *
+ * @param {number[][]} chains
+ * @param {unknown[]} pulls
+ * @returns {object[]}
+ */
+export function inferChainsToPickStacks(chains, pulls) {
+  /** @type {Map<number, Record<string, unknown>>} */
+  const byNum = new Map();
+  for (const raw of pulls) {
+    if (!raw || typeof raw !== "object") continue;
+    const p = /** @type {Record<string, unknown>} */ (raw);
+    const num = p.number;
+    if (typeof num !== "number") continue;
+    byNum.set(num, p);
+  }
+
+  return chains.map((chain, inferChainIndex) => {
+    /** @type {{ pr_number: number, position: number, head_branch?: string, title?: string, additions?: number, deletions?: number }[]} */
+    const prRows = [];
+    let sumAdd = 0;
+    let sumDel = 0;
+    for (let i = 0; i < chain.length; i++) {
+      const prn = chain[i];
+      const pull = byNum.get(prn);
+      const head =
+        pull && pull.head && typeof pull.head === "object"
+          ? /** @type {Record<string, unknown>} */ (pull.head)
+          : {};
+      const ref = typeof head.ref === "string" ? head.ref : "";
+      const add = pull && typeof pull.additions === "number" ? pull.additions : 0;
+      const del = pull && typeof pull.deletions === "number" ? pull.deletions : 0;
+      sumAdd += add;
+      sumDel += del;
+      prRows.push({
+        pr_number: prn,
+        position: i,
+        head_branch: ref,
+        title: pull && typeof pull.title === "string" ? pull.title : undefined,
+        additions: add,
+        deletions: del
+      });
+    }
+
+    const tipPr = chain[chain.length - 1];
+    const tipPull = tipPr ? byNum.get(tipPr) : undefined;
+    const tipHead = prRows.length ? String(prRows[prRows.length - 1].head_branch || "") : "";
+    const tip_updated_at =
+      tipPull && typeof tipPull.updated_at === "string" ? tipPull.updated_at : "";
+    const user =
+      tipPull && tipPull.user && typeof tipPull.user === "object"
+        ? /** @type {Record<string, unknown>} */ (tipPull.user)
+        : {};
+    const login = typeof user.login === "string" ? user.login : "";
+
+    const hasLineStats = sumAdd > 0 || sumDel > 0;
+
+    return {
+      tip_pr_number: tipPr,
+      tip_head_branch: tipHead,
+      pr_count: chain.length,
+      created_by: login,
+      prs: prRows,
+      tip_updated_at,
+      inferChainIndex,
+      ...(hasLineStats ? { inferDiffAdd: sumAdd, inferDiffDel: sumDel } : {})
+    };
+  });
+}