[jyaunches]

Summary

Removes two invalid environment-variable documentation allowlist entries after #9642 documented the corresponding runtime-adapter ports. This restores the documentation gate and changes no product behavior or product scope.

Changes

• Remove NEMOCLAW_BEDROCK_RUNTIME_ADAPTER_PORT from the internal-only documentation allowlist.
• Remove NEMOCLAW_HTTPS_PIN_RUNTIME_ADAPTER_PORT from the internal-only documentation allowlist.

Type of Change

• Code change (feature, bug fix, or refactor)
• Code change with doc updates
• Doc only (prose changes, no code sample modifications)
• Doc only (includes code sample changes)

Quality Gates

• Tests added or updated for changed behavior
• Existing tests cover changed behavior — justification: the repository environment-variable documentation gate rejects documented variables that remain in this allowlist.
• Tests not applicable — justification:
• Sensitive paths changed (security, policy, credentials, preflight, onboarding, inference, runner, sandbox, or messaging)
• Sensitive-path review completed or maintainer-approved waiver recorded — reviewer/approval link/justification:
• Non-success, skipped, or missing CI check accepted by maintainer — check name, approval link, and follow-up issue:

DGX Station Hardware Evidence

• Tested on DGX Station
• Tested commit:
• Station profile/scenario:
• Result:
• Supporting evidence:

Verification

• PR description includes a Signed-off-by: line and every commit appears as Verified in GitHub
• Normal pre-commit, commit-msg, and pre-push hooks passed, or npm run validate:pr passed after refreshing origin/main when hooks were skipped or unavailable
• Targeted behavior tests pass for the current change set, or tests are marked not applicable above — command/result or justification: npx tsx scripts/check-env-var-docs.mts passed.
• Applicable broad gate passed — npm test for broad runtime/test-harness changes; npm run check for repo-wide validation/coverage changes — command/result: not applicable to a two-entry allowlist cleanup.
• Quality Gates section completed with required justifications or waivers
• No secrets, API keys, or credentials committed
• npm run docs builds without warnings (doc changes only)
• Doc pages follow the style guide (doc changes only)
• New doc pages include SPDX header and frontmatter (new pages only)

---

Signed-off-by: Julie Yaunches jyaunches@nvidia.com

Summary by CodeRabbit

• Documentation
  • Removed obsolete Bedrock Runtime and HTTPS Pin Runtime adapter port environment variables from the CI documentation allowlist.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: e70f8a45-d567-483d-bbd9-68e1519697b7

📥 Commits

Reviewing files that changed from the base of the PR and between 01e0b92 and a6c8e08.

📒 Files selected for processing (1)

• ci/env-var-doc-allowlist.json

💤 Files with no reviewable changes (1)

• ci/env-var-doc-allowlist.json

Included review availability: Your plan provides up to 12 included reviews per hour; 6 remain after this review.

---

📝 Walkthrough

Walkthrough

The CI environment-variable documentation allowlist removes entries for the Bedrock Runtime adapter port and HTTPS Pin Runtime adapter port variables.

Changes

Environment variable documentation allowlist

Layer / File(s)	Summary
Remove obsolete runtime adapter entries ci/env-var-doc-allowlist.json	Removes the NEMOCLAW_BEDROCK_RUNTIME_ADAPTER_PORT and NEMOCLAW_HTTPS_PIN_RUNTIME_ADAPTER_PORT entries and their internal-use rationales.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to a6c8e

This localized cleanup only removes two stale documentation allowlist entries and does not change product behavior; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: removing stale documented environment variables from the allowlist.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}