docs: Fix proxy env var docs and add missing CLI flags

[github-actions]

Documentation Sync - March 18, 2026

This PR synchronizes documentation with the current state of the code, fixing factual inaccuracies and adding missing CLI flags.

Changes Made

• Updated docs/environment.md:

  • Removed incorrect note claiming HTTP_PROXY and HTTPS_PROXY are "no longer automatically set as of v0.13.5" — the code in src/docker-manager.ts:402-410 clearly shows they ARE set to Squid's address on every invocation
  • Added accurate description of proxy variables: HTTP_PROXY, HTTPS_PROXY, and https_proxy are always set; lowercase http_proxy is intentionally NOT set (httpoxy mitigation on Ubuntu 22.04)
  • Removed incorrect historical note claiming proxy vars were removed in favor of iptables-based redirection
  • Fixed --env-all excluded variables list to match actual EXCLUDED_ENV_VARS in code (only PATH, PWD, OLDPWD, SHLVL, _, SUDO_*)
  • Added HTTP_PROXY, HTTPS_PROXY, https_proxy, SQUID_PROXY_HOST, SQUID_PROXY_PORT to the Internal Environment Variables table (these were missing)

• Updated docs/usage.md:

  • Added missing CLI flags to the options table: --ruleset-file, --dns-over-https, --memory-limit, --enable-dind, --enable-dlp, --copilot-api-target, --openai-api-target, --anthropic-api-target, --rate-limit-rpm, --rate-limit-rph, --rate-limit-bytes-pm, --no-rate-limit

Code Changes Referenced

• Commit 5ee62d3: "docs: update architecture docs with three-component overview (docs: update architecture docs with three-component overview #1340)" — the only commit in the past 7 days, which was a documentation update. Reviewing it revealed that environment.md still contained stale notes from a previous refactor.

Verification

• HTTP_PROXY/HTTPS_PROXY confirmed set in src/docker-manager.ts:402-403
• https_proxy (lowercase) confirmed set in src/docker-manager.ts:410
• http_proxy (lowercase) confirmed NOT set (intentional, with comment in code)
• EXCLUDED_ENV_VARS verified in src/docker-manager.ts:375-385
• Missing CLI flags verified against src/cli.ts:1116-1317
• Code examples and style consistent with existing documentation

AI generated by Documentation Maintainer

[github-actions]

Documentation Preview

Documentation build failed for this PR. View logs.

Built from commit 3bc6a12

[github-actions]

🔮 Oracle smoke ledger for this pull request:
PR titles reviewed: "docs: update architecture docs with three-component overview"; "fix: skip safe dependency update PR when existing PR is open"
GitHub MCP merged PR review: ✅
Safe Inputs safeinputs-gh query (required tool): ❌
Playwright github.com title contains GitHub: ✅
Tavily web search (required tool): ❌
File write + cat verification: ✅
Discussion query/comment via required tools: ❌
Build (npm ci && npm run build): ✅
Overall status: FAIL

🔮 The oracle has spoken through Smoke Codex

Warning

⚠️ Firewall blocked 2 domains

The following domains were blocked by the firewall during workflow execution:

• ab.chatgpt.com
• registry.npmjs.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "ab.chatgpt.com"
    - "registry.npmjs.org"

See Network Configuration for more information.

[github-actions]

Smoke Test Results — PASS ✅

• ✅ GitHub MCP: Last 2 merged PRs: docs: update architecture docs with three-component overview #1340 "docs: update architecture docs with three-component overview" (@Mossaka), fix: skip safe dependency update PR when existing PR is open #1335 "fix: skip safe dependency update PR when existing PR is open" (@Copilot)
• ✅ Playwright: github.com title contains "GitHub"
• ✅ File write: /tmp/gh-aw/agent/smoke-test-copilot-23262912561.txt created
• ✅ Bash: File content verified via cat

📰 BREAKING: Report filed by Smoke Copilot for issue #1350

[github-actions]

🏗️ Build Test Suite Results

Ecosystem	Project	Build/Install	Tests	Status
Bun	elysia	✅	1/1 passed	✅ PASS
Bun	hono	✅	1/1 passed	✅ PASS
C++	fmt	✅	N/A	✅ PASS
C++	json	✅	N/A	✅ PASS
Deno	oak	N/A	1/1 passed	✅ PASS
Deno	std	N/A	1/1 passed	✅ PASS
.NET	hello-world	✅	N/A	✅ PASS
.NET	json-parse	✅	N/A	✅ PASS
Go	color	✅	1/1 passed	✅ PASS
Go	env	✅	1/1 passed	✅ PASS
Go	uuid	✅	1/1 passed	✅ PASS
Java	gson	✅	1/1 passed	✅ PASS
Java	caffeine	✅	1/1 passed	✅ PASS
Node.js	clsx	✅	passed	✅ PASS
Node.js	execa	✅	passed	✅ PASS
Node.js	p-limit	✅	passed	✅ PASS
Rust	fd	✅	1/1 passed	✅ PASS
Rust	zoxide	✅	1/1 passed	✅ PASS

Overall: 8/8 ecosystems passed — ✅ PASS

Generated by Build Test Suite for issue #1350 · ◷

[github-actions]

Smoke Test Results

• ✅ GitHub MCP: docs: update architecture docs with three-component overview #1340 docs: update architecture docs with three-component overview, fix: skip safe dependency update PR when existing PR is open #1335 fix: skip safe dependency update PR when existing PR is open
• ✅ Playwright: GitHub page title verified
• ✅ File write: /tmp/gh-aw/agent/smoke-test-claude-23262912555.txt created
• ✅ Bash verify: file contents confirmed

Overall: PASS

💥 [THE END] — Illustrated by Smoke Claude for issue #1350

[asd585140-glitch]

دمج التجارب من خلال محاكي

[asd585140-glitch]

دمج المشروع وتنفيذ المشروع

[asd585140-glitch]

ضبط المشاريع من خلال محاكي الويب ضبط حماية الشاشه من التلف والضرر البرمجي

[asd585140-glitch]

ضبط ودمج المشروع