Inspect completed forecast runs

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Author: Copilot

pkg/cli/forecast.go

@@ -546,7 +546,7 @@ func forecastWorkflow(ctx context.Context, workflowName, startDate string, confi
 	// Fetch completed runs from the history window.
 	opts := ListWorkflowRunsOptions{
 		WorkflowName: apiName,
-		Status:       "success",
+		Status:       "completed",
 		StartDate:    startDate,
 		Limit:        config.SampleSize,
 		TargetCount:  config.SampleSize,
@@ -1115,7 +1115,7 @@ func evaluateForecast(ctx context.Context, workflowName string, forecast Forecas
 	// Fetch completed runs in the validation window.
 	opts := ListWorkflowRunsOptions{
 		WorkflowName: apiName,
-		Status:       "success",
+		Status:       "completed",
 		StartDate:    validationStartDate,
 		Limit:        config.SampleSize,
 		TargetCount:  config.SampleSize,

pkg/cli/forecast_test.go

@@ -350,7 +350,7 @@ func TestForecastWorkflow_IgnoresSkippedRuns(t *testing.T) {
 	assert.InEpsilon(t, 0.5, result.SuccessRate, 1e-9)
 }
 
-func TestForecastWorkflow_RequestsSuccessfulRuns(t *testing.T) {
+func TestForecastWorkflow_RequestsCompletedRuns(t *testing.T) {
 	originalList := forecastListWorkflowRunsPaginated
 	originalLoadAIC := forecastLoadCachedRunAIC
 	t.Cleanup(func() {
@@ -380,7 +380,7 @@ func TestForecastWorkflow_RequestsSuccessfulRuns(t *testing.T) {
 		SampleSize: 100,
 	}, 30)
 	require.NoError(t, err)
-	assert.Equal(t, "success", capturedOpts.Status)
+	assert.Equal(t, "completed", capturedOpts.Status)
 }
 
 func TestForecastWorkflow_ExcludesZeroAICRunsFromComputation(t *testing.T) {