Store app: disclose llm listings before install - #233
Conversation
Parses the llm block from /api/packages, labels catalogue entries by tier, and shows the spec/llm.md pre-install record on the detail screen. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QwYWVPse8spRrfMMcuTJPv
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Reviewer's GuideThe Compose store now parses LLM metadata from package listings, presents tier-specific catalogue labels and a spec-aligned “Before you install” disclosure on detail pages, and verifies the formatting and unknown prompt-handling behavior with focused tests. Sequence diagram for LLM listing disclosure before installsequenceDiagram
participant API as Package API
participant Store as Compose Store
participant Detail as Detail Screen
participant User
Store->>API: GET /api/packages
API-->>Store: PackageSummary with LlmDto
User->>Store: Open LLM listing
Store->>Detail: Render tierLabel and LlmDisclosure
Detail->>Detail: llmDisclosureLines(llm)
Detail-->>User: Before you install record
User->>Detail: Select Operator terms
Detail->>Detail: openExternal(terms)
Flow diagram for LLM disclosure renderingflowchart TD
A[Package listing received] --> B{llm block present?}
B -- No --> C[Render standard listing]
B -- Yes --> D[Use tierLabel: hosted llm or sandbox llm]
D --> E[Render Before you install]
E --> F["llmDisclosureLines(llm)"]
F --> G[Show runs, prompts, model, licence, size, runner needs, and setup token]
G --> H{Operator terms present?}
H -- Yes --> I[Show Operator terms link]
H -- No --> J[Show disclosure without terms link]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
🚀 Deploying Preview to Cloudflare 🚀Preview Deployments by commit
|
OpenCode security review |
|
No The other red checks aren't from this PR either: Vercel (a leftover project), Workers Builds (the Cloudflare preview), and Locally, Generated by Claude Code |
There was a problem hiding this comment.
Sorry @HereLiesAz, you've used your own review budget of 250,000 diff characters for the last 7 days.
You can request another review in 23 hours and 7 minutes by commenting @sourcery-ai review. Upgrade to get a review now.
This brings the web store's llm disclosure (#232) to the Compose store app,
storefront-cmp.models/PackageSummary.kt:PackageSummarygainsllm: LlmDto?. It's parsed from the block/api/packagesalready serves.models/LlmDisclosure.kt:llmDisclosureLinesbuilds the key/value lines thatspec/llm.md§ Discovery requires before install:A
promptsvalue it doesn't recognise reads as "unknown", never as safe.components/LlmDisclosure.kt: shows those lines as a "Before you install" record on the detail screen, with an "Operator terms" link. The header also gets a tier pill.components/Cascade.kt: catalogue entries end in "hosted llm" or "sandbox llm" instead of just "llm".Verification
LlmDisclosureTestpasses 4/4, covering endpoint, sandbox-weights, unknown handling and non-llm listings../gradlew desktopTest compileKotlinDesktoppasses.🤖 Generated with Claude Code
https://claude.ai/code/session_01QwYWVPse8spRrfMMcuTJPv
Generated by Claude Code
Summary by Sourcery
Expose the required LLM discovery information before installation in the Compose store.
New Features:
Enhancements:
Tests: