You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Updated docs/content/docs/proxy-execute.mdx Parameters table: changed connected_account_id from "No" (optional) to "Yes" (required) and removed the incorrect "Defaults to the most recent active account for the project" description
Reasoning
Users calling the proxy execute endpoint without connected_account_id receive a 400 error (code 2811, ExternalProxy_MissingAuthContext) but the docs say the parameter is optional with a default fallback
The parameter requirement changed as part of intentional security hardening; docs need to reflect the current behavior
Claude finished @decimal-pr-bot[bot]'s task in 50s —— View job
Review Progress
Read docs-reviewer guidelines
Run git diff to see changes
Apply review checklist to changed files
Post review feedback
Review
Looks good. This is a one-line factual correction in docs/content/docs/proxy-execute.mdx:168 — flipping connected_account_id / connectedAccountId from optional to required and dropping the inaccurate "defaults to the most recent active account" sentence.
The change matches the actual API behavior described in the PR (missing param returns 400 / ExternalProxy_MissingAuthContext) and is consistent with the rest of the page — every code example on this page already passes the parameter, so marking it required aligns the table with the examples developers will copy. No missing-step, outdated-pattern, or CI-gap issues. Nothing else needs updating.
| Branch
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
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 of Changes
docs/content/docs/proxy-execute.mdxParameters table: changedconnected_account_idfrom "No" (optional) to "Yes" (required) and removed the incorrect "Defaults to the most recent active account for the project" descriptionReasoning
connected_account_idreceive a 400 error (code 2811, ExternalProxy_MissingAuthContext) but the docs say the parameter is optional with a default fallback