fix: missing json:path from health cmd (#20221)

Author: Copilot

pkg/cli/logs_github_api.go

@@ -154,7 +154,7 @@ type ListWorkflowRunsOptions struct {
 // The processedCount and targetCount parameters are used to display progress in the spinner message.
 func listWorkflowRunsWithPagination(opts ListWorkflowRunsOptions) ([]WorkflowRun, int, error) {
 	logsGitHubAPILog.Printf("Listing workflow runs: workflow=%s, limit=%d, startDate=%s, endDate=%s, ref=%s", opts.WorkflowName, opts.Limit, opts.StartDate, opts.EndDate, opts.Ref)
-	args := []string{"run", "list", "--json", "databaseId,number,url,status,conclusion,workflowName,createdAt,startedAt,updatedAt,event,headBranch,headSha,displayTitle"}
+	args := []string{"run", "list", "--json", "databaseId,number,url,status,conclusion,workflowName,path,createdAt,startedAt,updatedAt,event,headBranch,headSha,displayTitle"}
 
 	// Add filters
 	if opts.WorkflowName != "" {
@@ -246,15 +246,27 @@ func listWorkflowRunsWithPagination(opts ListWorkflowRunsOptions) ([]WorkflowRun
 		return nil, 0, fmt.Errorf("failed to list workflow runs (exit code %d): %w", exitCode, err)
 	}
 
-	var runs []WorkflowRun
-	if err := json.Unmarshal(output, &runs); err != nil {
+	// gh run list outputs "path" for the workflow file path, but WorkflowRun uses "workflowPath".
+	// Unmarshal via a helper struct so both fields are captured correctly.
+	var rawRuns []struct {
+		WorkflowRun
+		Path string `json:"path"`
+	}
+	if err := json.Unmarshal(output, &rawRuns); err != nil {
 		// Stop spinner on parse error
 		if !opts.Verbose {
 			spinner.Stop()
 		}
 		return nil, 0, fmt.Errorf("failed to parse workflow runs: %w", err)
 	}
 
+	runs := make([]WorkflowRun, len(rawRuns))
+	for i, raw := range rawRuns {
+		run := raw.WorkflowRun
+		run.WorkflowPath = raw.Path
+		runs[i] = run
+	}
+
 	// Stop spinner silently - don't show per-iteration messages
 	if !opts.Verbose {
 		spinner.Stop()

pkg/cli/logs_github_api_test.go

@@ -0,0 +1,106 @@
+//go:build !integration
+
+package cli
+
+import (
+	"encoding/json"
+	"strings"
+	"testing"
+	"time"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+// TestWorkflowRunPathFieldUnmarshal verifies that the "path" key returned by
+// "gh run list --json" is correctly bridged to WorkflowRun.WorkflowPath during
+// unmarshaling.  The gh CLI uses "path" but WorkflowRun serialises the field as
+// "workflowPath" for backward compatibility, so a helper struct is used at the
+// unmarshal site.
+//
+// Regression: commit 61cc2d7ac (Jan 4 2026) silently dropped "path" from the
+// gh run list --json field list, causing WorkflowPath to always be "" and the
+// .lock.yml filter in fetchWorkflowRuns to discard every run.
+func TestWorkflowRunPathFieldUnmarshal(t *testing.T) {
+	// Simulate a single row of "gh run list --json path,workflowName,..."
+	rawJSON := `[
+		{
+			"databaseId": 42,
+			"workflowName": "My Workflow",
+			"path": ".github/workflows/my-workflow.lock.yml",
+			"status": "completed",
+			"conclusion": "success",
+			"createdAt": "2026-01-01T00:00:00Z",
+			"startedAt": "2026-01-01T00:00:01Z",
+			"updatedAt": "2026-01-01T00:01:00Z"
+		}
+	]`
+
+	var rawRuns []struct {
+		WorkflowRun
+		Path string `json:"path"`
+	}
+	require.NoError(t, json.Unmarshal([]byte(rawJSON), &rawRuns), "unmarshal should succeed")
+	require.Len(t, rawRuns, 1)
+
+	run := rawRuns[0].WorkflowRun
+	run.WorkflowPath = rawRuns[0].Path
+
+	assert.Equal(t, ".github/workflows/my-workflow.lock.yml", run.WorkflowPath,
+		"WorkflowPath should be populated from the 'path' JSON key")
+	assert.Equal(t, int64(42), run.DatabaseID)
+	assert.Equal(t, "My Workflow", run.WorkflowName)
+}
+
+// TestFetchWorkflowRunsLockYMLFilter verifies the .lock.yml suffix filter used
+// in fetchWorkflowRuns.  Only runs whose WorkflowPath ends in ".lock.yml" must
+// be retained; runs with a plain ".yml" path (regular Actions workflows) must
+// be excluded.
+func TestFetchWorkflowRunsLockYMLFilter(t *testing.T) {
+	runs := []WorkflowRun{
+		{
+			DatabaseID:   1,
+			WorkflowName: "Agentic Workflow",
+			WorkflowPath: ".github/workflows/agentic-workflow.lock.yml",
+			StartedAt:    time.Date(2026, 1, 1, 0, 0, 0, 0, time.UTC),
+			UpdatedAt:    time.Date(2026, 1, 1, 0, 5, 0, 0, time.UTC),
+		},
+		{
+			DatabaseID:   2,
+			WorkflowName: "Regular CI",
+			WorkflowPath: ".github/workflows/ci.yml",
+		},
+		{
+			DatabaseID:   3,
+			WorkflowName: "Another Agentic",
+			WorkflowPath: ".github/workflows/another.lock.yml",
+			StartedAt:    time.Date(2026, 1, 2, 0, 0, 0, 0, time.UTC),
+			UpdatedAt:    time.Date(2026, 1, 2, 0, 3, 0, 0, time.UTC),
+		},
+		{
+			// Run with empty WorkflowPath — must be excluded (mimics the pre-fix state
+			// where "path" was absent from the JSON query).
+			DatabaseID:   4,
+			WorkflowName: "Agentic But No Path",
+			WorkflowPath: "",
+		},
+	}
+
+	var filtered []WorkflowRun
+	for _, run := range runs {
+		if strings.HasSuffix(run.WorkflowPath, ".lock.yml") {
+			if run.Duration == 0 && !run.StartedAt.IsZero() && !run.UpdatedAt.IsZero() {
+				run.Duration = run.UpdatedAt.Sub(run.StartedAt)
+			}
+			filtered = append(filtered, run)
+		}
+	}
+
+	require.Len(t, filtered, 2, "only .lock.yml runs should pass the filter")
+
+	assert.Equal(t, int64(1), filtered[0].DatabaseID)
+	assert.Equal(t, 5*time.Minute, filtered[0].Duration, "duration should be calculated from StartedAt/UpdatedAt")
+
+	assert.Equal(t, int64(3), filtered[1].DatabaseID)
+	assert.Equal(t, 3*time.Minute, filtered[1].Duration)
+}