Commit 64c17d1d76d6

Vincent Demeester <vincent@sbr.pm>
2026-02-11 16:23:02
fix(github): added missing ctx parameter to getPRHeadSha
The function was calling execGh with ctx but ctx was not in scope. Added ctx parameter to function signature and updated all calls.
main
1 parent 35b3a32
Changed files (1)
dots
pi
agent
extensions
github
actions
pr.ts
dots/pi/agent/extensions/github/actions/pr.ts
@@ -712,6 +712,7 @@ async function getPRTitle(
  */
 async function getPRHeadSha(
 	pi: ExtensionAPI,
+	ctx: ExtensionContext,
 	prNumber: number,
 	signal?: AbortSignal,
 ): Promise<string | null> {
@@ -779,7 +780,7 @@ export async function handlePRLineComment(
 
 	onUpdate?.({ content: [{ type: "text", text: `Fetching PR #${params.number} HEAD SHA...` }] });
 
-	const commitId = await getPRHeadSha(pi, params.number, signal);
+	const commitId = await getPRHeadSha(pi, ctx, params.number, signal);
 	if (!commitId) {
 		return {
 			content: [{ type: "text", text: `Error: Could not get HEAD commit SHA for PR #${params.number}` }],
@@ -895,7 +896,7 @@ export async function handlePRReviewWithComments(
 
 	onUpdate?.({ content: [{ type: "text", text: `Fetching PR #${params.number} HEAD SHA...` }] });
 
-	const commitId = await getPRHeadSha(pi, params.number, signal);
+	const commitId = await getPRHeadSha(pi, ctx, params.number, signal);
 	if (!commitId) {
 		return {
 			content: [{ type: "text", text: `Error: Could not get HEAD commit SHA for PR #${params.number}` }],