Skip to content

Revert "feat(next)!: artifact support for remove" - #1474

Merged
eoghanriley merged 1 commit into
mainfrom
revert-1469-eoghanriley/cli-299
Aug 25, 2026
Merged

eoghanriley merged 1 commit into
mainfrom
revert-1469-eoghanriley/cli-299

Conversation

@eoghanriley

@eoghanriley eoghanriley commented Aug 25, 2026 •

Copy link
Copy Markdown
Contributor

Reverts #1469

reverting to keep it out of todays release

@eoghanriley
eoghanriley requested a review from a team as a code owner August 25, 2026 15:25
@linear-code

linear-code Bot commented Aug 25, 2026

Copy link
Copy Markdown

@greptile-apps

greptile-apps Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR reverts artifact-backed bundle removal and restores removal from bundle definitions.

  • Removes local .tar.zst and OCI-reference handling, verification flags, and artifact identity checks from the remove command and library.
  • Restores source-based Zarf package metadata lookup during removal.
  • Removes artifact-removal tests and supporting test utilities while retaining artifact inspection behavior needed by other workflows.

Confidence Score: 5/5

The PR appears safe to merge as the reviewed changes consistently restore definition-only bundle removal without an identified changed-code defect.

Artifact and OCI removal paths are deliberately removed across the CLI, library, Zarf adapter, and tests, while package-name validation remains enforced before cluster access.

Important Files Changed

Filename Overview
internal/cli/bundle/remove.go Restores definition-only CLI removal and removes artifact verification and preparation.
pkg/bundle/remove.go Removes artifact and OCI source preparation from the public removal operation while retaining downstream package-selection validation.
internal/zarf/remove.go Restores package-source metadata loading before deployed-state lookup and removal.
internal/artifact/inspect.go Restores direct package-signing metadata inspection independently of the removed shared metadata helper.
pkg/bundle/deploy.go Removes artifact-removal-specific metadata from prepared deployment sources without changing deployment resource cleanup.
tests/testutil/cluster.go Restores the definition-path-only removal helper and removes artifact-specific arguments.

Reviews (1): Last reviewed commit: "Revert "feat(next)!: artifact support fo..." | Re-trigger Greptile

@eoghanriley
eoghanriley merged commit d4689f8 into main Aug 25, 2026
26 checks passed
@eoghanriley
eoghanriley deleted the revert-1469-eoghanriley/cli-299 branch August 25, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants