@theholocron/github-client 1.32.1 → 1.34.0

package/dist/index.d.mts

@@ -221,12 +221,28 @@ interface GitHubPullRequest {
     ref: string;
   };
 }
+/** A commit's name/email as git itself recorded it — `commit.author`/`commit.committer`, distinct from the GitHub-account `author` below (a commit can carry any email, linked to a GitHub account or not). */
+interface GitHubCommitIdentity {
+  name: string;
+  email: string;
+}
 /** One commit as returned by `GET /pulls/{pull_number}/commits` — only the fields any known consumer needs. */
 interface GitHubPullRequestCommit {
   sha: string;
   commit: {
     message: string;
+    author: GitHubCommitIdentity;
+    committer: GitHubCommitIdentity;
   };
+  /** The commit's linked GitHub account, when GitHub can associate one — `null` for a commit email with no linked account. `type: "Bot"` identifies an app-authored commit (e.g. Dependabot, a prior Sentinel auto-fix-commit). */
+  author: {
+    login: string;
+    type: "User" | "Bot";
+  } | null;
+  /** More than one entry marks a merge commit — it has no authorship of its own beyond its parents', the reason DCO enforcement (and this org's own CI) never requires a signoff on one. */
+  parents: Array<{
+    sha: string;
+  }>;
 }
 /** One file as returned by `GET /pulls/{pull_number}/files` — only the fields any known consumer needs. */
 interface GitHubPullRequestFile {
@@ -276,6 +292,19 @@ interface ReviewThread {
   isResolved: boolean;
   /** The thread's first comment's author login (e.g. `"the-holocron-sentinel[bot]"`) — `undefined` if GitHub ever returns a thread with no comments, which shouldn't happen in practice. Callers filter on this before resolving anything, so a bot never touches a human reviewer's own thread. */
   authorLogin: string | undefined;
+  /** The file the thread is anchored to on the PR's current diff — `null` once GitHub can no longer place it there (e.g. the file was since deleted). */
+  path: string | null;
+  /** 1-indexed line the thread is anchored to — `null` in the same "can no longer place it" case as `path`. */
+  line: number | null;
+  /**
+   * The thread's first comment's own body — the exact text a caller like
+   * Sentinel posted for a specific finding. Lets a caller diff a prior
+   * thread's own content against the current push's findings (holocron#860)
+   * to decide whether that finding is still live, rather than only knowing
+   * a thread exists. `undefined` in the same no-comments case as
+   * `authorLogin`.
+   */
+  body: string | undefined;
 }
 //#endregion
 //#region ../../../../../src/repos/repos.d.ts

package/dist/index.mjs

@@ -276,9 +276,12 @@ const LIST_REVIEW_THREADS_QUERY = `
 					nodes {
 						id
 						isResolved
+						path
+						line
 						comments(first: 1) {
 							nodes {
 								author { login }
+								body
 							}
 						}
 					}
@@ -331,7 +334,10 @@ function pulls(rest, graphql) {
 			})).repository.pullRequest.reviewThreads.nodes.map((node) => ({
 				id: node.id,
 				isResolved: node.isResolved,
-				authorLogin: node.comments.nodes[0]?.author?.login
+				path: node.path,
+				line: node.line,
+				authorLogin: node.comments.nodes[0]?.author?.login,
+				body: node.comments.nodes[0]?.body
 			}));
 		},
 		/**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@theholocron/github-client",
-  "version": "1.32.1",
+  "version": "1.34.0",
   "description": "A TypeScript client for the GitHub REST API",
   "keywords": [
     "github",
@@ -33,7 +33,7 @@
   ],
   "dependencies": {
     "@theholocron/observability": "^0.3.0",
-    "@theholocron/http-client": "^1.32.1"
+    "@theholocron/http-client": "^1.34.0"
   },
   "devDependencies": {
     "@theholocron/cli": "5.0.0-alpha.79",