From 182fa214784a38bca02237f050de243f76667511 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 3 Mar 2026 04:32:19 +0000 Subject: [PATCH 1/2] Initial plan From c865c54c7af13919465d39bbe7c50fa2d11c5183 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 3 Mar 2026 04:44:00 +0000 Subject: [PATCH 2/2] chore: delete zombie test files from pkg/workflow/ (batch 2 of 2) Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- pkg/workflow/log_parser_new_format_test.go | 19 -------- pkg/workflow/log_parser_snapshot_test.go | 19 -------- pkg/workflow/log_parser_test.go | 43 ------------------- pkg/workflow/main_export_validation_test.go | 19 -------- pkg/workflow/noop_bundling_test.go | 13 ------ pkg/workflow/patch_generation_test.go | 13 ------ .../safe_inputs_http_integration_test.go | 31 ------------- .../safe_outputs_cjs_extension_test.go | 31 ------------- .../safe_outputs_github_token_test.go | 31 ------------- ...fe_outputs_mcp_bundler_integration_test.go | 14 ------ pkg/workflow/safe_outputs_mcp_server_test.go | 43 ------------------- pkg/workflow/staged_preview_bundling_test.go | 13 ------ pkg/workflow/token_counting_test.go | 19 -------- 13 files changed, 308 deletions(-) delete mode 100644 pkg/workflow/log_parser_new_format_test.go delete mode 100644 pkg/workflow/log_parser_snapshot_test.go delete mode 100644 pkg/workflow/log_parser_test.go delete mode 100644 pkg/workflow/main_export_validation_test.go delete mode 100644 pkg/workflow/noop_bundling_test.go delete mode 100644 pkg/workflow/patch_generation_test.go delete mode 100644 pkg/workflow/safe_inputs_http_integration_test.go delete mode 100644 pkg/workflow/safe_outputs_cjs_extension_test.go delete mode 100644 pkg/workflow/safe_outputs_github_token_test.go delete mode 100644 pkg/workflow/safe_outputs_mcp_bundler_integration_test.go delete mode 100644 pkg/workflow/safe_outputs_mcp_server_test.go delete mode 100644 pkg/workflow/staged_preview_bundling_test.go delete mode 100644 pkg/workflow/token_counting_test.go diff --git a/pkg/workflow/log_parser_new_format_test.go b/pkg/workflow/log_parser_new_format_test.go deleted file mode 100644 index bb4e0ee2dda..00000000000 --- a/pkg/workflow/log_parser_new_format_test.go +++ /dev/null @@ -1,19 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestParseClaudeLogNewFormat tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestParseClaudeLogNewFormat(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestParseClaudeLogNewFormatJSScript tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestParseClaudeLogNewFormatJSScript(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/log_parser_snapshot_test.go b/pkg/workflow/log_parser_snapshot_test.go deleted file mode 100644 index fd79223b643..00000000000 --- a/pkg/workflow/log_parser_snapshot_test.go +++ /dev/null @@ -1,19 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestLogParserSnapshots tests script functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestLogParserSnapshots(t *testing.T) { - t.Skip("Script tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestLogParserScriptRetrieval tests script functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestLogParserScriptRetrieval(t *testing.T) { - t.Skip("Script tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/log_parser_test.go b/pkg/workflow/log_parser_test.go deleted file mode 100644 index 95b25aa3645..00000000000 --- a/pkg/workflow/log_parser_test.go +++ /dev/null @@ -1,43 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestLogParserScriptMethods tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestLogParserScriptMethods(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestGetLogParserScript tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestGetLogParserScript(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestParseClaudeLogSmoke tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestParseClaudeLogSmoke(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestParseClaudeLogInitialization tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestParseClaudeLogInitialization(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestParseClaudeMixedFormatLog tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestParseClaudeMixedFormatLog(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestClaudeEngineMixedFormatParsing tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestClaudeEngineMixedFormatParsing(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/main_export_validation_test.go b/pkg/workflow/main_export_validation_test.go deleted file mode 100644 index 3df2a6bc272..00000000000 --- a/pkg/workflow/main_export_validation_test.go +++ /dev/null @@ -1,19 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestScriptsExportMain tests script functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestScriptsExportMain(t *testing.T) { - t.Skip("Script tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestScriptsWithMainExportPattern tests script functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestScriptsWithMainExportPattern(t *testing.T) { - t.Skip("Script tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/noop_bundling_test.go b/pkg/workflow/noop_bundling_test.go deleted file mode 100644 index d9a2c2c5a1f..00000000000 --- a/pkg/workflow/noop_bundling_test.go +++ /dev/null @@ -1,13 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestNoOpScriptBundling tests script functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestNoOpScriptBundling(t *testing.T) { - t.Skip("Script tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/patch_generation_test.go b/pkg/workflow/patch_generation_test.go deleted file mode 100644 index c0d30923361..00000000000 --- a/pkg/workflow/patch_generation_test.go +++ /dev/null @@ -1,13 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestPullRequestPatchGeneration tests workflow functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestPullRequestPatchGeneration(t *testing.T) { - t.Skip("Workflow tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/safe_inputs_http_integration_test.go b/pkg/workflow/safe_inputs_http_integration_test.go deleted file mode 100644 index 2b1ff783daa..00000000000 --- a/pkg/workflow/safe_inputs_http_integration_test.go +++ /dev/null @@ -1,31 +0,0 @@ -//go:build integration - -package workflow - -import ( - "testing" -) - -// TestSafeInputsHTTPServer_Integration tests integration functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestSafeInputsHTTPServer_Integration(t *testing.T) { - t.Skip("Integration tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeInputsHTTPWithSecrets_Integration tests integration functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestSafeInputsHTTPWithSecrets_Integration(t *testing.T) { - t.Skip("Integration tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeInputsHTTPEntryPointScript_Integration tests integration functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestSafeInputsHTTPEntryPointScript_Integration(t *testing.T) { - t.Skip("Integration tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeInputsHTTPServerReadinessCheck_Integration tests integration functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestSafeInputsHTTPServerReadinessCheck_Integration(t *testing.T) { - t.Skip("Integration tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/safe_outputs_cjs_extension_test.go b/pkg/workflow/safe_outputs_cjs_extension_test.go deleted file mode 100644 index 6b0856619b1..00000000000 --- a/pkg/workflow/safe_outputs_cjs_extension_test.go +++ /dev/null @@ -1,31 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestSafeOutputsMCPServerUsesCjsExtension tests integration functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestSafeOutputsMCPServerUsesCjsExtension(t *testing.T) { - t.Skip("Integration tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeInputsMCPServerUsesCjsExtension tests integration functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestSafeInputsMCPServerUsesCjsExtension(t *testing.T) { - t.Skip("Integration tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeInputsToolsConfigUsesCjsExtension tests integration functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestSafeInputsToolsConfigUsesCjsExtension(t *testing.T) { - t.Skip("Integration tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestJavaScriptSourcesUseCjsExtension tests integration functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestJavaScriptSourcesUseCjsExtension(t *testing.T) { - t.Skip("Integration tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/safe_outputs_github_token_test.go b/pkg/workflow/safe_outputs_github_token_test.go deleted file mode 100644 index 8497d941c6b..00000000000 --- a/pkg/workflow/safe_outputs_github_token_test.go +++ /dev/null @@ -1,31 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestSafeOutputsGitHubTokenConfiguration tests workflow compilation -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestSafeOutputsGitHubTokenConfiguration(t *testing.T) { - t.Skip("Workflow compilation tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeOutputsGitHubTokenIntegration tests workflow compilation -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestSafeOutputsGitHubTokenIntegration(t *testing.T) { - t.Skip("Workflow compilation tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestAddSafeOutputGitHubTokenFunction tests workflow compilation -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestAddSafeOutputGitHubTokenFunction(t *testing.T) { - t.Skip("Workflow compilation tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestIndividualConfigGitHubTokenConfiguration tests workflow compilation -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestIndividualConfigGitHubTokenConfiguration(t *testing.T) { - t.Skip("Workflow compilation tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/safe_outputs_mcp_bundler_integration_test.go b/pkg/workflow/safe_outputs_mcp_bundler_integration_test.go deleted file mode 100644 index dcb0b30d47a..00000000000 --- a/pkg/workflow/safe_outputs_mcp_bundler_integration_test.go +++ /dev/null @@ -1,14 +0,0 @@ -//go:build integration - -package workflow - -import ( - "testing" -) - -// SKIPPED: Scripts now use require() pattern and are loaded at runtime from external files -// TestSafeOutputsMCPBundlerIntegration tests that the safe-outputs workflow -// correctly includes child_process imports in the generated .cjs files -func TestSafeOutputsMCPBundlerIntegration(t *testing.T) { - t.Skip("Test skipped - safe-outputs MCP scripts now use require() pattern and are loaded at runtime from external files") -} diff --git a/pkg/workflow/safe_outputs_mcp_server_test.go b/pkg/workflow/safe_outputs_mcp_server_test.go deleted file mode 100644 index a56a5fb9578..00000000000 --- a/pkg/workflow/safe_outputs_mcp_server_test.go +++ /dev/null @@ -1,43 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestSafeOutputsMCPServer_Initialize tests MCP server functionality -// SKIPPED: MCP server scripts are now loaded from external files at runtime -func TestSafeOutputsMCPServer_Initialize(t *testing.T) { - t.Skip("MCP server tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeOutputsMCPServer_ListTools tests MCP server functionality -// SKIPPED: MCP server scripts are now loaded from external files at runtime -func TestSafeOutputsMCPServer_ListTools(t *testing.T) { - t.Skip("MCP server tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeOutputsMCPServer_CreateIssue tests MCP server functionality -// SKIPPED: MCP server scripts are now loaded from external files at runtime -func TestSafeOutputsMCPServer_CreateIssue(t *testing.T) { - t.Skip("MCP server tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeOutputsMCPServer_MissingTool tests MCP server functionality -// SKIPPED: MCP server scripts are now loaded from external files at runtime -func TestSafeOutputsMCPServer_MissingTool(t *testing.T) { - t.Skip("MCP server tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeOutputsMCPServer_UnknownTool tests MCP server functionality -// SKIPPED: MCP server scripts are now loaded from external files at runtime -func TestSafeOutputsMCPServer_UnknownTool(t *testing.T) { - t.Skip("MCP server tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestSafeOutputsMCPServer_MultipleTools tests MCP server functionality -// SKIPPED: MCP server scripts are now loaded from external files at runtime -func TestSafeOutputsMCPServer_MultipleTools(t *testing.T) { - t.Skip("MCP server tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/staged_preview_bundling_test.go b/pkg/workflow/staged_preview_bundling_test.go deleted file mode 100644 index 6cf2634941a..00000000000 --- a/pkg/workflow/staged_preview_bundling_test.go +++ /dev/null @@ -1,13 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestStagedPreviewInlined tests that staged preview functionality is bundled -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestStagedPreviewInlined(t *testing.T) { - t.Skip("Staged preview bundling tests skipped - scripts now use require() pattern to load external files at runtime") -} diff --git a/pkg/workflow/token_counting_test.go b/pkg/workflow/token_counting_test.go deleted file mode 100644 index ce030de9ce4..00000000000 --- a/pkg/workflow/token_counting_test.go +++ /dev/null @@ -1,19 +0,0 @@ -//go:build !integration - -package workflow - -import ( - "testing" -) - -// TestTokenCountingConsistency tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestTokenCountingConsistency(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -} - -// TestTokenCountingWithoutCacheTokens tests log parser functionality -// SKIPPED: Scripts are now loaded from external files at runtime using require() pattern -func TestTokenCountingWithoutCacheTokens(t *testing.T) { - t.Skip("Log parser tests skipped - scripts now use require() pattern to load external files at runtime") -}