Skip to content

feat(item): expose direct image links - #46

Merged
Microck merged 1 commit into
mainfrom
feat/item-image-links
Oct 5, 2026
Merged

Microck merged 1 commit into
mainfrom
feat/item-image-links

Conversation

@Microck

@Microck Microck commented Oct 5, 2026

Copy link
Copy Markdown
Owner

getting the direct photo links should be a command, without having to pick them out of the full item response.

adds wallapop item images ITEM: json returns a url array, jsonl returns one string per line, and --format pretty prints plain links in gallery order. item show --format pretty now includes every image link too. both accept an item hash or full url and use the existing item reader.

wallapop item images https://es.wallapop.com/item/thinkpad-x1-carbon-1092837465 --format pretty

validated with the full go test suite, vet, staticcheck, build, formatting, and a generated-reference drift check. tests cover output formats, hash/url inputs, stale page galleries, empty galleries, and invalid/missing items. local tests ran with CODEX_HOME unset and GOFLAGS=-buildvcs=false for the isolated jujutsu workspace.

@vercel

vercel Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
wallapop-cli Ready Ready Preview Oct 5, 2026 4:33pm UTC

@coderabbitai

coderabbitai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 36f24277-e205-4a6f-a40a-270b6becfd84
📥 Commits

Reviewing files that changed from the base of the PR and between 8f25194 and 651924c.

📒 Files selected for processing (6)
  • README.md
  • docs-site/content/docs/reference/item.mdx
  • docs/cli-spec.md
  • internal/cli/cli_test.go
  • internal/cli/item.go
  • internal/fakewallapop/fake.go

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


Adds wallapop item images ITEM to output gallery URLs in JSON, JSONL, or pretty format, and includes image URLs in pretty item show output.

  • Both commands accept an item hash or full URL and use item metadata; they do not download image files.
  • The author reports passing the Go test suite, vet, staticcheck, build, formatting, and generated-reference drift checks.
  • CodeRabbit’s review was not completed after the pull request base or head changed; rerun the review against the current changes.

Walkthrough

Adds item images ITEM to print an item’s image URLs in gallery order. The command accepts an item hash or URL and reads item metadata without downloading image bytes. JSON returns an array; JSONL and pretty output print one URL per line. Pretty item show output now includes full image URLs. Tests cover output formats, lookup errors, gallery order, and empty galleries.

Sequence Diagram(s)

sequenceDiagram
  participant Caller
  participant ItemImages as item images command
  participant ItemMetadata as item metadata
  participant OutputFormatter as output formatter
  Caller->>ItemImages: Provide item hash or URL
  ItemImages->>ItemMetadata: Read item metadata
  ItemMetadata-->>ItemImages: Return gallery image URLs
  ItemImages->>OutputFormatter: Print URLs in gallery order
Loading

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 65192

No actionable merge-blocking issue is established for the image-link command; it is ready for normal merge checks.

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@Microck

Microck commented Oct 5, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Pull request base or head changed.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@Microck
Microck merged commit 5f5d2c4 into main Oct 5, 2026
8 checks passed

This branch was successfully deployed

1 active deployment
Preview — 651924c2 Deployed Oct 5, 2026 by vercel[bot]
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