Commit 96e05c7
authored
docs(policy): list the teams preset in the Open tier table (#9504)
## Summary
The Policy Tiers table in `docs/reference/network-policies.mdx` did not
list the `teams` preset in its Open
row, while the `open` tier in `nemoclaw-blueprint/policies/tiers.yaml`
has applied that preset since commit
`17d03317b` (#5585). An operator who read this table before selecting
Open was not told that the tier opens
Microsoft Teams egress by default. The Open row now names `teams` in the
position it occupies in
`tiers.yaml`, with the experimental tag that
`docs/reference/platform-support.mdx` records for the channel.
## Related Issue
Fixes #9503
This repeats a correction the project already accepted. Issue #3688
reported the identical drift for
`wechat`, and PR #4276 (commit `a5768a244`) corrected the same row the
same way. PR #5585 added `teams` to
the `open` tier and stated that it would "Leave user-facing
documentation to a follow-up docs-owner change";
that follow-up did not reach this page.
## Changes
- `docs/reference/network-policies.mdx`: add `` `teams` (experimental)
`` to the Open row of the Policy
Tiers table, between `whatsapp` and `jira`, matching the preset order in
`nemoclaw-blueprint/policies/tiers.yaml`.
One existing table row changed. Net line delta is ±0 (1 insertion, 1
deletion, one file). No new
abstraction, configuration, fallback, migration, or compatibility path.
Checked and unchanged: the Restricted, Balanced, and Personal rows
already match `tiers.yaml`, so `teams`
is the table's only omission.
Not changed, and offered here instead: the `<AgentOnly
variant="openclaw">` note earlier on the same page
says "The baseline policy does not include messaging endpoints for
Telegram, Discord, Slack, WeChat, or
WhatsApp." That sentence omits Microsoft Teams and also Google Chat.
Google Chat is not a tier preset, so
that sentence belongs to a different premise than this tier-table drift,
and adding only Teams would leave
a new partial list. Say the word and I will extend that note in this PR
or a separate one.
## Type of Change
- [ ] Code change (feature, bug fix, or refactor)
- [ ] Code change with doc updates
- [x] 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:
- [x] Tests not applicable — justification: the change is one
documentation table cell and alters no
executable behavior. `npm run docs` is the validation for this page. If
you want a guard against the next
recurrence, the natural home is the existing tier-loader test surface
for `src/lib/policy/tiers.ts`: one
case that reads `resolveTierPresets("open")` and asserts each preset
name appears in the Open row of
`docs/reference/network-policies.mdx`. That adds lines, so I left it out
of this ±0 change and will add it
in this PR if you prefer it here.
- [ ] 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:
## Documentation Writer Review
- [x] Documentation writer subagent reviewed the completed changes
- Result: `docs-updated`
- Evidence: Updated docs/reference/network-policies.mdx. The writing
rules and documentation style were reviewed.
- Agent: Pi CLI
<!-- docs-review-head-sha: 0473cf7 -->
<!-- docs-review-agents-blob-sha:
993bdd8 -->
## Verification
- [x] PR description includes a `Signed-off-by:` line and every commit
appears as `Verified` in GitHub
- [x] 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 — `npm run validate:pr` passed after
refreshing `origin/main` and rebasing onto the current `main`
- [x] Targeted behavior tests pass for the current change set, or tests
are marked not applicable above — marked not applicable above; `npm run
docs` exits 0
- [ ] Applicable broad gate passed — `npm test` for broad
runtime/test-harness changes; `npm run check` for repo-wide
validation/coverage changes — command/result:
- [x] Quality Gates section completed with required justifications or
waivers
- [x] No secrets, API keys, or credentials committed
- [x] `npm run docs` builds without warnings (doc changes only) — 0
errors. The two reported warnings are
the unauthenticated Fern redirects check and the site accent-color
contrast ratio; both appear identically
on an unmodified checkout of this page.
- [x] Doc pages follow the [style
guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md)
(doc changes only)
- [ ] New doc pages include SPDX header and frontmatter (new pages only)
---
Signed-off-by: Udaya Tejas <udayatejas2004@gmail.com>
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit
* **Documentation**
* Updated network policy reference documentation to include the
experimental `teams` preset in the Open policy tier.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
Signed-off-by: Udaya Tejas <udayatejas2004@gmail.com>1 parent 918cdee commit 96e05c7
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
138 | 138 | | |
139 | 139 | | |
140 | 140 | | |
141 | | - | |
| 141 | + | |
142 | 142 | | |
143 | 143 | | |
144 | 144 | | |
| |||
0 commit comments