Skip to content

Scan step stubs should return ErrNotImplemented (pipeline_step_scan_*.go) #67

Closed

Description

@intel352

Problem

module/pipeline_step_scan_sast.go, pipeline_step_scan_container.go, and pipeline_step_scan_deps.go all contain:

// TODO: Execute via sandbox.DockerSandbox once the sandbox package is available.

The steps currently appear to succeed silently without doing anything, giving false confidence in CI/CD pipelines.

Fix Direction

Either integrate with the sandbox package if now available, or have each step return a clear sentinel error (e.g. ErrNotImplemented) so pipeline authors know the step is non-functional. Add documentation noting the limitation.

Context

See docs/DEFERRED_ISSUES.md — Incomplete Implementations section.

Activity

  1. added
    tech-debtCode quality, smells, and incomplete implementations
    on Feb 22, 2026
  2. added a commit that references this issue on Feb 23, 2026
    06fc44e
  3. added a commit that references this issue on Feb 23, 2026
    f1fc56c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    tech-debtCode quality, smells, and incomplete implementations

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions