feat: tag keyless signup links from the CLI - #288
Merged
Merged
Conversation
The API links keyless prompts to signup with utm_source=keyless&utm_medium=api. The CLI now rewrites utm_medium to cli in the errors it prints for keyless scrape, search, parse, interact, research and developer requests, so accounts created from the CLI are attributed to the CLI.
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
No issues found across 4 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would auto-approve. Retags the keyless signup link in CLI error messages from utm_medium=api to utm_medium=cli; a focused, test-covered fix limited to error text with no unrelated behavior changes.
Re-trigger cubic
micahstairs
approved these changes
Sep 29, 2026
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
2 participants
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
The API links keyless prompts to signup tagged
utm_source=keyless&utm_medium=api, and the CLI prints the API's message as is. Accounts created from a CLI prompt would therefore be counted as API signups.Summary
withCliSignupTag(src/utils/client.ts) rewritesutm_source=keyless&utm_medium=apitoutm_medium=cliin the errors the CLI prints for keyless requests:keylessRequestandkeylessGet(scrape, search, research, developer)interactandinteract stopparseTest Plan
pnpm type-checkpnpm build && pnpm test: 644 passing, including the newsrc/__tests__/utils/client.test.ts, which fails when the rewrite is disabled.Summary by cubic
Retags the signup link in keyless error messages from
utm_medium=apitoutm_medium=cli, so accounts created from CLI prompts are attributed to the CLI rather than the API.interact,interact stop, andparse.Written for commit 2f47787. Summary will update on new commits.