Skip to content

refactor(cli): fold selectors into inspect - #414

Open
0x76agabond wants to merge 1 commit into
Perfect-Abstractions:mainfrom
0x76agabond:feature/inspect-without-selectors
Open

0x76agabond wants to merge 1 commit into
Perfect-Abstractions:mainfrom
0x76agabond:feature/inspect-without-selectors

Conversation

@0x76agabond

@0x76agabond 0x76agabond commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Remove the redundant compose selectors command. compose inspect remains the single view for diamond facets and selectors.

Changes Made

  • Remove the selectors command, pipeline, and related tests.
  • Show Unknown signature beside selectors that cannot be resolved in inspect output.
  • Add inspect output tests and a patch changeset.

Verification

  • CLI tests: 175 passed on the original branch.
  • CLI lint and build passed on the original branch.
  • Manual Anvil fixture: inspect shows 10 facets, 23 selectors, and the unknown-signature warning.
  • The PR branch is based directly on main and contains only the inspect/selectors change.

@netlify

netlify Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

👷 Deploy request for compose-diamonds pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 9ce8929

@changeset-bot

changeset-bot Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9ce8929

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@perfect-abstractions/compose-cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

CLI Coverage

Commit: 9ce8929

Metric Coverage Covered/Total
Lines 70% 2087/2974
Functions 73% 437/595
Branches 60% 1188/1965

@0x76agabond
0x76agabond force-pushed the feature/inspect-without-selectors branch from 116864d to 9ce8929 Compare October 10, 2026 19:22

This branch has not been deployed

No deployments
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.

1 participant