Address final Coderabbit comments

Gavin Williams · Gavin Williams · commit 967380d5288c · 2026-04-15T13:36:08.000+01:00
diff --git a/packages/web/src/features/agents/review-agent/nodes/gitlabMrParser.ts b/packages/web/src/features/agents/review-agent/nodes/gitlabMrParser.ts
@@ -97,7 +97,7 @@ export const gitlabMrParser = async (
         repo: repoName,
         file_diffs: filteredSourcebotFileDiffs,
         number: mrIid,
-        head_sha: mr.sha ?? "",
+        head_sha: mr.sha ?? mrPayload.object_attributes.last_commit.id,
         diff_refs: mr.diff_refs != null
             ? mr.diff_refs as { base_sha: string; head_sha: string; start_sha: string }
             : undefined,
diff --git a/packages/web/src/features/agents/review-agent/types.ts b/packages/web/src/features/agents/review-agent/types.ts
@@ -71,7 +71,7 @@ const gitLabDiffRefsSchema = z.object({
 }).nullable().optional();
 
 export const gitLabMergeRequestPayloadSchema = z.object({
-    object_kind: z.string(),
+    object_kind: z.literal('merge_request'),
     object_attributes: z.object({
         iid: z.number(),
         title: z.string(),
@@ -85,10 +85,10 @@ export const gitLabMergeRequestPayloadSchema = z.object({
 export type GitLabMergeRequestPayload = z.infer<typeof gitLabMergeRequestPayloadSchema>;
 
 export const gitLabNotePayloadSchema = z.object({
-    object_kind: z.string(),
+    object_kind: z.literal('note'),
     object_attributes: z.object({
         note: z.string(),
-        noteable_type: z.string(),
+        noteable_type: z.literal('MergeRequest'),
     }),
     merge_request: z.object({
         iid: z.number(),
