docs: link the hosted service from the README - #5
Open
rchildress87 wants to merge 1 commit into
Open
Conversation
The CLI never mentioned Disco Cloud, so the only path between the open source scanner and the hosted product ran one way. Adds a Hosted section between "Why disco" and "Install": what the CLI is on its own first, the six verbs that have no hosted equivalent, then what the service adds. Every claim checked against source — MIT in LICENSE, the free cap and null trial period in the SaaS plan and Paddle catalog, and each CLI-only verb confirmed absent from the hosted app rather than assumed. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TyMpXR3ctnUmvRgGyVXwTD
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
1 participant
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.
The CLI never mentioned Disco Cloud anywhere — grep for
saas,hosted,commercial, ordiscocloudacrossREADME.md,FEATURES.md,CONTRIBUTING.md,SECURITY.md, andCODE_STRUCTURE.mdreturned nothing. The link between the open source scanner and the hosted product ran one way, SaaS to CLI only.What this adds
One
## Hostedsection, placed after "Why disco" and before "Install". Placement is deliberate: the first three sections stay entirely about the tool, so a reader learns whatdiscodoes and why before encountering any commercial mention.The section leads with what the CLI is on its own — MIT, local, no telemetry, no paid edition — and names the six verbs that have no hosted equivalent, before describing what the service adds. The wording for the split is taken from the pricing page so the two do not drift.
Claims and where each was checked
Every factual claim was verified against source rather than assumed:
LICENSE:1internal/providers/aws/aws_servicereference.go:19,internal/providers/gcp/gcp_coverage.go:69); no analytics, update check, or crash reportingcmd/graph.go:63(path),cmd/diff.go:49,cmd/graph.go:71(--orphans-only),cmd/tag_coverage.go:45,cmd/check.go:102(SARIF), README:19 (byte-identical JSON)internal/return no feature — only unrelated prose and one doc comment for a compliance-pack entitlement that nothing reads yetinternal/plan/plan.go:41in the hosted apptrial_period: nullon both prices in the hosted app's Paddle cataloginternal/plan/plan.go:63— "There is deliberately no user or seat cap"The absence half of the fourth row is the one that mattered most: claiming the hosted product lacks something it actually ships would be the worst possible defect in this file, so each of the six was checked in both directions.
One softening applied after review: the CLI-only verbs are described as having no hosted equivalent "today" rather than "by design", since a curated OPA compliance pack is a planned hosted entitlement (
internal/plan/plan.go:77) even though nothing reads that field yet.🤖 Generated with Claude Code
https://claude.ai/code/session_01TyMpXR3ctnUmvRgGyVXwTD