{{ message }}
Repository navigation
improvement(comparisons): noindex self-hosted marketing pages, link every comparison, and add Zapier, Retool, Flowise lead answers - #9227
Merged
Conversation
…very comparison, and add Zapier, Retool, Flowise lead answers
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
All reported issues were addressed across 9 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
Contributor
 [Medium impact] The PR appears safe to merge; no actionable new issues were established. SummaryThe PR keeps self-hosted marketing content out of search results and expands the comparison pages.
No actionable new issues were established. No previous review threads were supplied. Reviews (2) · Last reviewed commit: "fix(proxy): noindex llms.txt, llms-full...." · Reviewed by Greptile |
…mbeddings claim to its docs
…-hosted deployments
Collaborator
Author
|
@greptile |
Collaborator
Author
|
@cubic-dev-ai review this PR |
Contributor
This branch was previously deployed
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.
Summary
X-Robots-Tag: noindex, nofollowon the marketing pages, so a third-party copy of the site (with whatever facts its build shipped) never competes with sim.ai in search or answer engines. sim.ai is unchanged:isHostedis true there, the same flag the proxy already uses to redirect a self-hosted/to/loginisLandingPathfrom the theme provider intolib/landing/routes.tsso the proxy and the theme provider share one definitionType of Change
Testing
proxy.test.ts: 29/29. New cases cover self-hosted landing pages (noindex), hosted landing pages (indexable), and a self-hosted non-landing page (unchanged). Each was confirmed red by reverting or loosening the proxy clausecheck:source-conventions,check:unused-exports,check:comment-hygiene,check:client-boundary,check:test-patterns,check:file-namesChecklist
test-auditauthoring gate)