From cec468c713b00c2a06c2a83aafcbc97c73fda554 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 24 Jan 2026 14:55:05 +0000 Subject: [PATCH 1/2] Initial plan From 08f354db916ad35e4b4cb984c60e6f3b0562e955 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 24 Jan 2026 15:10:48 +0000 Subject: [PATCH 2/2] Add error wrapping to compiler activation job builders - Wrap error in buildPreActivationJob when extracting custom fields - Wrap error in buildMainJob when generating main job steps - Improves debugging by providing context about where failures occur - Increases error wrapping from 1 to 3 instances in file - All activation tests pass Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- pkg/workflow/compiler_activation_jobs.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/workflow/compiler_activation_jobs.go b/pkg/workflow/compiler_activation_jobs.go index 0a67d87f53..028422fe25 100644 --- a/pkg/workflow/compiler_activation_jobs.go +++ b/pkg/workflow/compiler_activation_jobs.go @@ -22,7 +22,7 @@ func (c *Compiler) buildPreActivationJob(data *WorkflowData, needsPermissionChec // Extract custom steps and outputs from jobs.pre-activation if present customSteps, customOutputs, err := c.extractPreActivationCustomFields(data.Jobs) if err != nil { - return nil, err + return nil, fmt.Errorf("failed to extract pre-activation custom fields: %w", err) } // Add setup step to copy activation scripts (required - no inline fallback) @@ -642,7 +642,7 @@ func (c *Compiler) buildMainJob(data *WorkflowData, activationJobCreated bool) ( // but capture it into a string instead of writing directly var stepBuilder strings.Builder if err := c.generateMainJobSteps(&stepBuilder, data); err != nil { - return nil, err + return nil, fmt.Errorf("failed to generate main job steps: %w", err) } // Split the steps content into individual step entries