fix(cli): accept attribution overrides per command - #119
Merged
Conversation
JJasonSun
changed the title
Merged
N-45div
added a commit
to N-45div/call-e-integrations
that referenced
this pull request
Sep 9, 2026
Resolve packages/cli/package.json so the syntax check keeps both lib/sanitize.js and scripts/run-agent-command.mjs. The shell-free launcher contract and per-command attribution flags from CALLE-AI#115/CALLE-AI#119 are preserved unchanged.
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.
CLI callers can pass
--source,--integration, and--integration-versionas attribution overrides in each command's argument array. Each option overrides its matching environment value and reuses the existing broker/MCP header and telemetry context. Missing or invalid values returninvalid_argumentsbefore network requests.The CLI reference documents allowed values, partial-context defaults, and repeating the options on follow-up requests. Recovery coverage executes returned
next_argvarrays with these options and verifies that a lost response still results in one logical call.Fixes #111.
Validation:
944b9533a64501c9b537fc4cca9838598e5b52c2. Linux passed 170 JavaScript tests, with one Windows-only skip, plus 8 Python OAuth tests. Windows passed 101 CLI tests with no skips, including all 67 documented argument arrays in each of PowerShell 7, Windows PowerShell, and cmd.pnpm test: 170 passed, with one Windows-only skip on macOS.pnpm check,pnpm pack:dry-run, andgit diff --checkpassed.Includes a patch changeset for
@call-e/cliin.changeset/clear-shell-attribution.md.