Skip to content

fix: show asset pattern and output directory in the dependencies listing - #73

Merged
PaulTaykalo merged 1 commit into
mainfrom
fix/list-installed-assets
Aug 11, 2026
Merged

PaulTaykalo merged 1 commit into
mainfrom
fix/list-installed-assets

Conversation

@PaulTaykalo

@PaulTaykalo PaulTaykalo commented Aug 11, 2026 •

Copy link
Copy Markdown
Collaborator

Description

The [Read] dependencies listing printed one repo(tag) line per manifest entry. When a repo has multiple entries with different asset patterns (or different output directories), the output was a wall of identical lines that said nothing about what is actually being installed.

Each listed line now includes the asset pattern (falling back to contents when there is no pattern) and the output directory when the entry has one.

Before:

[Read] Found 4 dependencies:
   owner/multi-asset-repo(1.2.3)
   owner/multi-asset-repo(1.2.3)
   owner/framework(2.0.0)
   owner/framework(2.0.0)

After:

[Read] Found 4 dependencies:
   owner/multi-asset-repo(1.2.3) FirstAsset.xcframework.zip
   owner/multi-asset-repo(1.2.3) SecondAsset.xcframework.zip
   owner/framework(2.0.0) Framework.zip -> sandbox
   owner/framework(2.0.0) Framework.zip -> nonsandbox

The listing also emits one line per asset for dependencies that use the nested assets: form, instead of one line per dependency.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Additional Context

Verified with swift build && swift test (all tests pass) and a real resolve run against a consumer manifest in a scratch directory.

🤖 Generated with Claude Code

@PaulTaykalo
PaulTaykalo requested a review from nekrich as a code owner August 11, 2026 06:21
Previously the [Read] listing printed one 'repo(tag)' line per manifest
entry, so a repo with multiple assets (different patterns/outputs)
produced many identical lines that didn't say what is actually
installed. Each listed line now includes the asset pattern (falling
back to contents) and the output directory when present, e.g.:

   owner/repo(1.0.0) Asset.xcframework.zip -> sandbox

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@PaulTaykalo
PaulTaykalo force-pushed the fix/list-installed-assets branch from 41cab04 to 75c4913 Compare August 11, 2026 06:27
@PaulTaykalo
PaulTaykalo enabled auto-merge (squash) August 11, 2026 06:27
@PaulTaykalo
PaulTaykalo merged commit a8e380a into main Aug 11, 2026
7 checks passed
@PaulTaykalo
PaulTaykalo deleted the fix/list-installed-assets branch August 11, 2026 06:47
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.

2 participants