Document an agent-<name> convention for MAPBOX_CLI_INSTALL_SOURCE - #36
Merged
Merged
Conversation
Distinguishing an install a coding agent triggered on someone's behalf from a human or CI one is useful signal, and the installers already have a mechanism that carries it: MAPBOX_CLI_INSTALL_SOURCE, which rides along in the User-Agent as `src/<value>`. Comment-only in both installers — agent-<name> (agent-claude-code, agent-cursor), so an access-log query can tell these apart without a second reporting mechanism the way Atlassian's twg installer stands up for the same question. A dash rather than the slash an earlier proposal used: both installers' existing sanitizer strips `/`, so agent/claude-code would silently collapse into agentclaude-code and lose the separator that makes the convention legible. Freeform otherwise, same as every other value here — no fixed list of agent names to keep in sync in either installer. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
zmofei
previously approved these changes
Sep 23, 2026
…l-source-convention # Conflicts: # CHANGELOG.md
zmofei
approved these changes
Sep 23, 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.
Summary
install.sh/install.ps1: documentsagent-<name>(agent-claude-code,agent-cursor) as the convention forMAPBOX_CLI_INSTALL_SOURCEwhen a coding agent invokes the installer on someone's behalf, so an access-log query can tell these installs apart from a human or CI one — without standing up a second analytics endpoint the way Atlassian'stwginstaller does for the same question./(tr -cd/-replacekeep only letters, digits,.,-,_), soagent/claude-codewould silently collapse intoagentclaude-codeand lose the separator that makes the convention legible.MAPBOX_CLI_INSTALL_SOURCEvalue today — no fixed list of agent names added to either installer.Test plan
sh scripts/test-install.sh— all cases pass (comment-only change, confirmed nothing else moved)