From 7929502df19bee6e7726e344241b4e8750bb0467 Mon Sep 17 00:00:00 2001
From: Sam Morrow <info@sam-morrow.com>
Date: Wed, 7 Jan 2026 16:11:03 +0100
Subject: [PATCH] Fix nil pointer dereference in completion handler

The CompleteParams.Context field is optional (marked omitempty) and can be
nil when clients don't send it. The code was accessing Context.Arguments
directly without checking if Context was nil first, causing a panic.

This fix adds a nil check for Context before accessing Arguments.
---
 pkg/github/repository_resource_completions.go | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/pkg/github/repository_resource_completions.go b/pkg/github/repository_resource_completions.go
index aeb2d88a6..c70cfe948 100644
--- a/pkg/github/repository_resource_completions.go
+++ b/pkg/github/repository_resource_completions.go
@@ -33,8 +33,10 @@ func RepositoryResourceCompletionHandler(getClient GetClientFn) func(ctx context
 
 		argName := req.Params.Argument.Name
 		argValue := req.Params.Argument.Value
-		resolved := req.Params.Context.Arguments
-		if resolved == nil {
+		var resolved map[string]string
+		if req.Params.Context != nil && req.Params.Context.Arguments != nil {
+			resolved = req.Params.Context.Arguments
+		} else {
 			resolved = map[string]string{}
 		}