From 9044decedf132d4fa28368ba773d859c3957a2da Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 30 Jun 2026 01:17:55 +0000
Subject: [PATCH 1/3] Initial plan


From 40872143cf4b1c47415f59104f6ae3c958fbc7b3 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 30 Jun 2026 01:25:34 +0000
Subject: [PATCH 2/3] chore: initial plan for pkg/cli refactor

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
---
 .../extensions/agentic-workflows-dashboard/web/index.html  | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/.github/extensions/agentic-workflows-dashboard/web/index.html b/.github/extensions/agentic-workflows-dashboard/web/index.html
index 423abe16532..9cd753eb93e 100644
--- a/.github/extensions/agentic-workflows-dashboard/web/index.html
+++ b/.github/extensions/agentic-workflows-dashboard/web/index.html
@@ -115,12 +115,7 @@ <h2 class="Box-title">Workflow definitions <span class="Counter ml-1" x-text="de
             <div class="Box-header d-flex flex-items-center flex-justify-between flex-wrap gap-2">
               <h2 class="Box-title">Workflow runs <span class="Counter ml-1" x-text="runsPaged.totalItems"></span></h2>
               <div class="d-flex flex-items-center gap-2">
-                <select
-                  class="form-select form-select-sm"
-                  @change="selectWorkflowFilter($event.target.value)"
-                  :disabled="loadingDefinitions || loadingRuns"
-                  aria-label="Select workflow"
-                >
+                <select class="form-select form-select-sm" @change="selectWorkflowFilter($event.target.value)" :disabled="loadingDefinitions || loadingRuns" aria-label="Select workflow">
                   <option value="">— select a workflow —</option>
                   <template x-for="def in definitions" :key="def.workflow">
                     <option :value="def.workflow" :selected="selectedWorkflowFilter === def.workflow" x-text="def.workflow"></option>

From 3d18f25790c6052d5a695b3ebf651763546309dc Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 30 Jun 2026 01:28:30 +0000
Subject: [PATCH 3/3] refactor: move isGHCLIAvailable() to git.go and upgrade
 getParentDir() to use path.Dir

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
---
 pkg/cli/agent_download.go | 16 ----------------
 pkg/cli/git.go            |  7 +++++++
 pkg/cli/helpers.go        | 15 ++++++++++-----
 3 files changed, 17 insertions(+), 21 deletions(-)
 delete mode 100644 pkg/cli/agent_download.go

diff --git a/pkg/cli/agent_download.go b/pkg/cli/agent_download.go
deleted file mode 100644
index 8df375bab36..00000000000
--- a/pkg/cli/agent_download.go
+++ /dev/null
@@ -1,16 +0,0 @@
-package cli
-
-import (
-	"os/exec"
-
-	"github.com/github/gh-aw/pkg/logger"
-)
-
-var agentDownloadLog = logger.New("cli:agent_download")
-
-func isGHCLIAvailable() bool {
-	cmd := exec.Command("gh", "--version")
-	available := cmd.Run() == nil
-	agentDownloadLog.Printf("Checked gh CLI availability: available=%v", available)
-	return available
-}
diff --git a/pkg/cli/git.go b/pkg/cli/git.go
index d8c1d9134b3..d0b860c9ad6 100644
--- a/pkg/cli/git.go
+++ b/pkg/cli/git.go
@@ -653,3 +653,10 @@ func stageAllChanges(verbose bool) error {
 	gitLog.Print("Successfully staged all changes")
 	return nil
 }
+
+func isGHCLIAvailable() bool {
+	cmd := exec.Command("gh", "--version")
+	available := cmd.Run() == nil
+	gitLog.Printf("Checked gh CLI availability: available=%v", available)
+	return available
+}
diff --git a/pkg/cli/helpers.go b/pkg/cli/helpers.go
index 1b3de94294c..8fa7a8df223 100644
--- a/pkg/cli/helpers.go
+++ b/pkg/cli/helpers.go
@@ -3,6 +3,7 @@ package cli
 import (
 	"fmt"
 	"os"
+	"path"
 	"path/filepath"
 	"strings"
 
@@ -12,13 +13,17 @@ import (
 
 var helpersLog = logger.New("cli:helpers")
 
-// getParentDir returns the directory part of a path
-func getParentDir(path string) string {
-	idx := strings.LastIndex(path, "/")
-	if idx == -1 {
+// getParentDir returns the directory part of a forward-slash workflow path.
+// Workflow paths always use forward slashes (e.g. ".github/workflows/foo.md"),
+// so path.Dir is correct here; filepath.Dir is not used to avoid OS-specific
+// separator handling. Returns "" when the path has no directory component
+// (i.e. path.Dir would return ".").
+func getParentDir(p string) string {
+	dir := path.Dir(p)
+	if dir == "." {
 		return ""
 	}
-	return path[:idx]
+	return dir
 }
 
 // getRepositoryRelativePath converts an absolute file path to a repository-relative path