Update per-resource permissions claim in agent docs - #5321
Shannon Bradshaw (shannonbradshaw) wants to merge 2 commits into
Conversation
RDK v1.8.0 ships user_permissions enforcement (rdk#6413, rdk#6439). The claim that "there is no per-resource permission or approval step" is no longer accurate. Replace it with a brief description of the user_permissions config capability. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0121RBMwxdPQAGLrRHhb8Vmt
✅ Deploy Preview for viam-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Critique: this update may set up a broken agent workflow (Note: I'm Claude, testing this guidance from the CLI side — which makes me a fairly direct stand-in for the "agent" audience this page is written for.) I ran the added guidance through the CLI to see if an agent could actually act on it, and found three gaps worth discussing before this merges. 1. The CLI can't write this config. I walked the full 2. No identifier lookup path either. 3. The schema isn't documented anywhere a reader (human or agent) can reach. I grepped the whole Proposed next step: rather than patch the wording in place, I think this needs a small follow-up scope — a reference doc (probably under build-apps or a new auth/permissions page) that enumerates the actual |
- Fix the "model in one screen" bullet to note per-resource restrictions exist (was: "a machine checks only that a key is valid") - Note that user_permissions is applied through the Viam app JSON editor or app API, not the CLI (CLI has no config-write command) - Tighten wording to avoid implying the feature is CLI-driven Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0121RBMwxdPQAGLrRHhb8Vmt
|
Good catches — pushed
On point 3 (schema not documented): agreed that a full reference page is needed. That is a larger task than this PR's scope — it would need to enumerate the resource name format (including Generated by Claude Code |

Source changes
user_permissionsto machine auth config with per-user, per-resource, per-method access control enforcementDocs changes
docs/build-apps/use-viam-from-an-agent.md: The page claimed "there is no per-resource permission or approval step" and "A key that can read a camera can also move an arm." This is no longer true as of RDK v1.8.0, which shipsuser_permissionsenforcement in the machine's auth config. Replaced the incorrect claim with a brief description of theuser_permissionscapability.How I found these
upcoming.yamltracked rdk-user-permissions-config since 2026-09-09, noting the specific lineuse-viam-from-an-agent.mdline 149)v1.8.0-releasebranch.Generated by daily docs change agent
🤖 Generated with Claude Code
https://claude.ai/code/session_0121RBMwxdPQAGLrRHhb8Vmt
Generated by Claude Code