Fix CGO fuzz job compile failure by exposing shared test helpers to integration-tagged builds

[Copilot]

The CGO workflow’s fuzz matrix failed on main during FuzzSanitizeOutput, not due to fuzz findings but because pkg/workflow tests failed to compile under -tags integration. Shared helper symbols used by integration tests were excluded from that build context.

• Root cause

  • Helper functions used across workflow tests (containsInNonCommentLines, extractJobSection, indexInNonCommentLines, boolPtr, strPtr) lived in _test.go files constrained by //go:build !integration, so they were unavailable when fuzz jobs compiled with integration tags.

• Code changes

  • Removed restrictive !integration build constraints from:
    • pkg/workflow/compiler_test_helpers_test.go
    • pkg/workflow/test_helpers_shared_test.go
  • This keeps helpers test-only, but makes them visible to both default and integration-tagged test compilation paths.

• Effect on workflow behavior

  • pkg/workflow test compilation no longer fails in integration-tagged fuzz job setup, allowing fuzz targets to execute instead of failing at compile time.

// before
//go:build !integration

package workflow

// after
package workflow

[Copilot]

Pull request overview

This PR fixes a CGO fuzz workflow compilation failure by making shared pkg/workflow test helper functions available when tests are compiled with -tags integration. It does this by removing //go:build !integration constraints from helper-only _test.go files so integration-tagged fuzz targets can compile and run.

Changes:

• Removed the !integration build constraint from shared workflow test helper files.
• Ensured helper symbols (e.g., containsInNonCommentLines, extractJobSection, boolPtr, strPtr) are available to both default and integration-tagged test builds.

Show a summary per file

File	Description
pkg/workflow/test_helpers_shared_test.go	Removes !integration build tag so pointer/test helper utilities are available in integration-tagged test builds.
pkg/workflow/compiler_test_helpers_test.go	Removes !integration build tag so compiler-related test helpers are available in integration-tagged test builds (needed by integration-tagged tests/fuzz targets).

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 2/2 changed files
• Comments generated: 0