Description
pkg/workflow/compiler_custom_jobs.go contains 8+ calls to fmt.Errorf without the %w verb (lines 280, 444, 594, 669+). These break error chain inspection and are inconsistent with compiler_jobs.go which wraps all errors correctly.
Suggested Changes
Audit all fmt.Errorf calls in compiler_custom_jobs.go and replace non-wrapping forms with fmt.Errorf("...: %w", err) where there is an underlying error to wrap.
Files Affected
pkg/workflow/compiler_custom_jobs.go (lines ~280, 444, 594, 669+)
Success Criteria
- All
fmt.Errorf calls that have a cause error use %w for wrapping
make golint-custom passes for changed files- All existing tests continue to pass
- Zero non-wrapping fmt.Errorf with a cause argument in
compiler_custom_jobs.go
Source
Extracted from Daily Compiler Code Quality Report - 2026-07-19
Priority
Medium — improves error chain transparency for debugging; consistent with the codebase standard established in compiler_jobs.go
🔍 Task mining by Discussion Task Miner - Code Quality Improvement Agent · 48.6 AIC · ⌖ 5.07 AIC · ⊞ 7K · ◷
Description
pkg/workflow/compiler_custom_jobs.gocontains 8+ calls tofmt.Errorfwithout the%wverb (lines 280, 444, 594, 669+). These break error chain inspection and are inconsistent withcompiler_jobs.gowhich wraps all errors correctly.Suggested Changes
Audit all
fmt.Errorfcalls incompiler_custom_jobs.goand replace non-wrapping forms withfmt.Errorf("...: %w", err)where there is an underlying error to wrap.Files Affected
pkg/workflow/compiler_custom_jobs.go(lines ~280, 444, 594, 669+)Success Criteria
fmt.Errorfcalls that have a cause error use%wfor wrappingmake golint-custompasses for changed filescompiler_custom_jobs.goSource
Extracted from Daily Compiler Code Quality Report - 2026-07-19
Priority
Medium — improves error chain transparency for debugging; consistent with the codebase standard established in
compiler_jobs.go