From 5281be467d5ab084e716b3c96fedc6a698013438 Mon Sep 17 00:00:00 2001
From: Kynan Ware <47394200+BagToad@users.noreply.github.com>
Date: Fri, 29 Aug 2025 14:20:12 -0600
Subject: [PATCH] Refactor error handling in generatePullRequestNodeID

Simplified error handling by inlining the encoder.Encode call and removing redundant comments for improved code clarity.
---
 pkg/cmd/agent-task/capi/sessions.go | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/pkg/cmd/agent-task/capi/sessions.go b/pkg/cmd/agent-task/capi/sessions.go
index 3f42cadf4..6d1c96e4b 100644
--- a/pkg/cmd/agent-task/capi/sessions.go
+++ b/pkg/cmd/agent-task/capi/sessions.go
@@ -191,15 +191,11 @@ func generatePullRequestNodeID(repoID, pullRequestID int64) string {
 	encoder := msgpack.NewEncoder(&buf)
 	encoder.UseCompactInts(true)
 
-	// Encode the parts
-	err := encoder.Encode(parts)
-	if err != nil {
+	if err := encoder.Encode(parts); err != nil {
 		panic(err)
 	}
 
-	// Use URL-safe Base64 encoding without padding
 	encoded := base64.RawURLEncoding.EncodeToString(buf.Bytes())
 
-	// Return with the PR_ prefix
 	return "PR_" + encoded
 }