[CLI Flag Review] Weekly CLI Flag Consistency Report — 2026-03-04 #1144
Closed
Replies: 1 comment
-
|
This discussion was automatically closed because it expired on 2026-03-11T14:15:12.481Z.
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
📊 Summary
src/cli.ts)cli-reference.mddocs/usage.md(detailed section)✅ Flags Status
Main Command Options
cli.tsusage.mdcli-reference.mdREADME.md--allow-domains--allow-domains-file--block-domains--block-domains-file--log-level--keep-containers--tty--work-dir--build-local--agent-image--image-registry--image-tag--skip-pull-e, --env--env-all-v, --mount--container-workdir--dns-servers--proxy-logs-dir--enable-host-access--allow-host-ports--ssl-bump--allow-urls--enable-api-proxy--copilot-api-target--rate-limit-rpm--rate-limit-rph--rate-limit-bytes-pm--no-rate-limitawf logsSubcommand Optionscli.tsusage.mdcli-reference.md-f, --follow--format--source--list--with-pidawf logs statsSubcommand Optionscli.tsusage.mdcli-reference.md--format--sourceawf logs summarySubcommand Optionscli.tsusage.mdcli-reference.md--format--source🔴 Issue 1:
--enable-api-proxymissing fromcli-reference.mddocs-site/src/content/docs/reference/cli-reference.md--enable-api-proxyflag (and all related API proxy flags) are completely absent from the CLI reference document.### --enable-api-proxysection with description, examples, and links todocs/api-proxy-sidecar.md.cli-reference.md(confirmed withgrepreturning 0 matches).--enable-api-proxy,--copilot-api-target, and all--rate-limit-*flags to the Options Summary table, and add detailed sections in the Options Details area.🔴 Issue 2:
--copilot-api-targetundocumented everywheredocs/usage.md,docs-site/src/content/docs/reference/cli-reference.md,README.md,docs/api-proxy-sidecar.md,AGENTS.md,CLAUDE.md--copilot-api-target (host)is a user-facing CLI flag that overrides the Copilot API target hostname. It's implemented insrc/cli.ts(line ~800) with the description "Defaults to api.githubcopilot.com. Useful for GHES deployments." — but appears in no documentation file.docs/usage.mdanddocs-site/src/content/docs/reference/cli-reference.md, with a note indocs/api-proxy-sidecar.mdsince it only applies when--enable-api-proxyis used.AGENTS.mddoes mentionCOPILOT_API_TARGETenv var but not the corresponding CLI flag.docs/api-proxy-sidecar.mdas it is scoped to that feature, and list in the main CLI reference.🔴 Issue 3: Rate-limiting flags (
--rate-limit-rpm,--rate-limit-rph,--rate-limit-bytes-pm,--no-rate-limit) undocumented everywheredocs/usage.md,docs-site/src/content/docs/reference/cli-reference.md,README.md,docs/api-proxy-sidecar.mdsrc/cli.ts(lines ~800–820) but appear in no documentation file whatsoever.docs/api-proxy-sidecar.md(as they require--enable-api-proxy) and in the CLI reference. The implementation note says the defaults when rate limiting is enabled are: rpm=600, rph=10000, bytesPm=52428800 (50MB) — none of this is documented.docs/api-proxy-sidecar.mdcovering all four flags and their defaults.cli-reference.md.docs/usage.md.--allow-host-portsincorrect default incli-reference.mddocs-site/src/content/docs/reference/cli-reference.md, line 43--allow-host-portsas80,443. However, insrc/cli.ts, this option has no default value (thecommanderoption is not given a default argument).docs/usage.mdcorrectly documents: "When--enable-host-accessis enabled without--allow-host-ports, all ports onhost.docker.internalare currently allowed."cli-reference.mdshould read—(no default) with a note that all ports are allowed when omitted (current behavior, planned to change to 80,443 in a future version).| --allow-host-ports (ports) | string | 80,443 | ...— this is incorrect.—and add a note: "When omitted, all ports are currently allowed (will default to 80,443 in a future version)."--enable-api-proxyhas no dedicated section indocs/usage.mddocs/usage.md--enable-api-proxyappears only in the help synopsis block (line 63) but has no dedicated### --enable-api-proxysection with description, examples, or cross-references — unlike every other flag.docs/api-proxy-sidecar.mdfor full details.### --enable-api-proxysection pointing todocs/api-proxy-sidecar.mdfor complete documentation.📋 Recommendations
🔴 High Priority
Document the 5 undocumented flags (
--copilot-api-target,--rate-limit-rpm,--rate-limit-rph,--rate-limit-bytes-pm,--no-rate-limit):docs/api-proxy-sidecar.mdcovering all rate-limit flags including their defaults (rpm=600, rph=10000, bytesPm=50MB) and the--no-rate-limitoverride.--copilot-api-targettodocs/api-proxy-sidecar.mdwith a GHES usage example.cli-reference.md.Fix
--enable-api-proxyincli-reference.md— add entry to summary table and a detailed section.Fix
--allow-host-portsdefault value incli-reference.md(line 43) — change80,443to—with a note about current behavior.Add
--enable-api-proxysection todocs/usage.md(even a brief one with a pointer todocs/api-proxy-sidecar.md).📝 Low Priority
📁 Files Analyzed
src/cli.tsdocs/usage.mddocs-site/src/content/docs/reference/cli-reference.mdREADME.mdAGENTS.mdCLAUDE.mddocs/api-proxy-sidecar.mdBeta Was this translation helpful? Give feedback.
All reactions