feat: offer the save-this-scan link on anonymous scans (0.6.0) - #5
Merged
Conversation
POST /api/scan returns a claim-bearing `saveUrl` to keyless callers, but the CLI dropped it — so `agent-ready scan` produced a real result and then a dead end, with no way to keep it. Render it as the footer CTA. One CTA, one link: the save link replaces the pricing nudge on anonymous scans, and the quota note below it loses its competing /pricing URL while keeping the Pro facts. The bare report URL is suppressed too — it is the same page minus the claim, so following it silently loses ownership. Keyed (Pro) scans are unchanged: they already belong to an account, so monitoring stays the right next step. Copy promises only what the free tier grants. Score history and monitoring are both Pro-gated server-side, so naming them as reasons to create a free account would break at signup. Also fixes a drift that shipped in 0.5.2: `--version` printed 0.5.1, because nothing tied src/cli.ts VERSION to package.json. A wrong version misdirects every bug report, so both are now 0.6.0 with a sync guard mirroring agent-ready-mcp's. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
POST /api/scanreturns a claim-bearingsaveUrlto keyless callers (main repo PR #206), but the CLI dropped it. Soagent-ready scandelivered a real result and then a dead end — no way for the user to keep it.What changed
formatScantakes an optionalsaveUrl. When present the scan is anonymous and claimable, so saving it becomes the footer CTA:One CTA, one link:
/pricingURL but keeps the Pro factsKeyed (Pro) scans are untouched: they already belong to an account, so the weekly-monitoring nudge stays.
Copy
Promises only what
freeactually grants. Score history and monitoring are both Pro-gated server-side (the API 403s a free user; the dashboard hides both panels), so offering them as reasons to sign up would break the promise at signup. A test asserts this.Version drift fix
--versionprinted0.5.1while npm had0.5.2— nothing tiedsrc/cli.tsVERSION topackage.json. Both are now0.6.0, with a sync guard mirroring the oneagent-ready-mcpalready has.Verification
77 tests pass, typecheck clean, builds. Covers: link rendered, pricing nudge replaced, exactly one of our URLs, free-tier copy honest, and graceful fallback when the server issues no link (no
CLAIM_SECRET).🤖 Generated with Claude Code