fix: treat missing Atmos config in BASE as empty baseline in atmos describe affected

[Copilot]

atmos describe affected fatally errors with failed to find import on greenfield branches (or when the base ref predates Atmos adoption) because ErrFailedToFindImport from BASE stack processing was propagated as a hard failure. The correct behavior is to treat an unconfigured BASE as an empty baseline — everything in HEAD is new, therefore everything is affected.

Changes

• internal/exec/describe_affected_utils.go — executeDescribeAffected now handles ErrFailedToFindImport alongside ErrNoStackManifestsFound in both BASE processing paths:

  • FindAllStackConfigsInPathsForStack returning ErrFailedToFindImport (stacks directory absent in BASE) → sets remoteStackConfigFilesAbsolutePaths = []string{}
  • ExecuteDescribeStacks returning ErrFailedToFindImport (imports unresolvable in BASE) → sets remoteStacks = map[string]any{}

  Both cases emit a WARN log with actionable context:

  WARN No Atmos stack manifests found in BASE; treating BASE as empty (all HEAD components will be reported as affected)
       hint="This is expected for greenfield branches or when the base branch does not yet use Atmos"
  

• tests/describe_affected_greenfield_test.go — Integration test that initializes a bare-minimum git repo (single commit, no Atmos config) as the BASE and asserts all known HEAD components (component-1, component-2 in prod/nonprod) appear in the affected output without error.

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 52c225d.

Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

[codecov]

Codecov Report

❌ Patch coverage is 38.46154% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.25%. Comparing base (12c72e9) to head (52c225d).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/exec/describe_affected_utils.go	38.46%	5 Missing and 3 partials ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2296   +/-   ##
=======================================
  Coverage   77.24%   77.25%           
=======================================
  Files        1074     1074           
  Lines      101900   101910   +10     
=======================================
+ Hits        78716    78730   +14     
+ Misses      18846    18837    -9     
- Partials     4338     4343    +5     

Flag	Coverage Δ
unittests	77.25% <38.46%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
internal/exec/describe_affected_utils.go	77.57% <38.46%> (+2.09%)	⬆️

... and 6 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[atmos-pro]

Tip

Atmos Pro  

No affected stacks workflow was detected for this pull request.
If this is expected, no action is needed.
Learn More.

[atmos-pro]

Note

Atmos Pro  

Waiting for your GitHub Actions workflow to upload affected stacks.
Learn More.

[github-actions]

These changes were released in v1.216.0-rc.0.