Skip to content

feat(uploads): metadata discoverability polish for the CLI#160

Merged
Zach Dunn (zachdunn) merged 1 commit into
mainfrom
claude/cli-metadata-dx-ux-23ea31
Jul 14, 2026
Merged

feat(uploads): metadata discoverability polish for the CLI#160
Zach Dunn (zachdunn) merged 1 commit into
mainfrom
claude/cli-metadata-dx-ux-23ea31

Conversation

@zachdunn

Copy link
Copy Markdown
Member

Follow-up to #157/#158 and the #144 publish, from the DX/UX review in #159. These are the CLI-side discoverability gaps for per-file metadata — additive, no behavior changes to existing flows.

Changes

  • find / list --meta show matched metadata inline. Human output now prints key url k=v k=v (sorted). LIST_HELP already promised "returned items include their matched metadata" but only --json delivered it.
  • meta get on a bare object prints (no metadata) to stderr instead of nothing (an empty print read as failure). stdout stays clean for piping.
  • attach discoverability hint. After upload it prints >> find these later: uploads find gh.ref=<owner/repo#num>, so the gh.* metadata it auto-writes is discoverable.
  • README now lists the stdio MCP set_metadata/find_files tools (they shipped in 0.8.0 but were missing from the tool enumeration) and documents the metadata param on put/attach and the hosted put.

Deliberately not hinting metadata on plain uploads list: that needs a per-object HEAD hydration the design avoids (same reason visibility isn't annotated on listings, per #159).

Verification

  • Full package suite: 284 passed (3 new tests spying real stdout/stderr on runFind, runMeta, runAttach).
  • typecheck clean, oxlint/oxfmt clean (pre-commit hooks passed).
  • Changeset included (patch bump).

The remaining #159 items (web search UI, presigned-upload capture, engineering debts) stay tracked there.

- find / list --meta print matched metadata inline in human output
  (LIST_HELP already promised this; previously only --json)
- meta get on a bare object prints "(no metadata)" to stderr, not nothing
- attach hints "find these later: uploads find gh.ref=…" so its auto
  gh.* metadata is discoverable
- README lists the stdio set_metadata/find_files tools and the
  put/attach metadata param
@coderabbitai

coderabbitai Bot commented Jul 14, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (2)
  • coderabbit:review
  • review
🚫 Excluded labels (none allowed) (1)
  • wip

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4ca7184a-4e11-4528-81b9-dd6c5becb016

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/cli-metadata-dx-ux-23ea31

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
uploads-auth 1b9f970 Commit Preview URL

Branch Preview URL
Jul 14 2026, 01:35 AM

@zachdunn Zach Dunn (zachdunn) merged commit 528d895 into main Jul 14, 2026
4 checks passed
@zachdunn Zach Dunn (zachdunn) deleted the claude/cli-metadata-dx-ux-23ea31 branch July 14, 2026 01:37
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