[eoghanriley]

Description

This PR improves performance when inspecting local artifacts. Previously the inspect call path would walk artifacts like

package 1 manifest → archive walk
package 1 zarf.yaml → archive walk
package 2 manifest → archive walk
package 2 zarf.yaml → archive walk

with this PR that gets flattened to

all package manifests → one archive walk
all zarf.yaml blobs   → one archive walk

Related Issue

Fixes CLI-306

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Other (security config, docs update, etc)

Checklist before merging

• Test, docs, adr added or updated as needed
• Contributor Guide Steps followed

[greptile-apps]

Confidence Score: 5/5

[Medium risk] Refactors artifact metadata reading for performance.

The PR appears safe to merge; no outstanding finding or new actionable regression was established.

Summary

The PR batches local archive reads for package manifests and zarf.yaml layers, while retaining package-ordered errors and adding coverage for missing and malformed metadata.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Bundle packages] --> B[Resolve manifest descriptors]
  B --> C[One archive walk for manifests]
  C --> D[Locate zarf.yaml descriptors]
  D --> E[One archive walk for zarf.yaml]
  E --> F[Return metadata or first package-ordered error]

Loading

_{Reviews (2) · Last reviewed commit: "cleanup error handling"}

[chance-coleman]

nice! LGTM

[bradclawsie]

lgtm, and I explored it a bit with codex, which also found no issues

[bradclawsie]

still lgtm!