Repository navigation
Add arm manual mode/properties and ListRobotsForLocations to SDK method docs - #5360
Merged
Jeremy Rose (jeremyrose-viam) merged 1 commit intoSep 28, 2026
Conversation
…od docs Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for viam-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Jeremy Rose (jeremyrose-viam)
requested a review
from Eliza Farley (elizafarley)
September 25, 2026 17:58
Eliza Farley (elizafarley)
approved these changes
Sep 25, 2026
Jeremy Rose (jeremyrose-viam)
merged commit Sep 28, 2026
3e863d3
into
viamrobotics:main
13 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Refs #5243
Adds SDK method mappings for four methods that the weekly SDK method coverage job reports as unused, then regenerates the method docs. The last two comments on #5243 flag these as new SDK surface.
New methods
SetManualModeGetManualModeGetPropertiesListRobotsForLocationsapp.protoEach gets a row in
sdk_protos_map.csvand a one-sentence description inoverrides/protos/, written from the proto's own comments.ListRobotsForLocationsisn't in the TypeScript or Flutter SDKs yet; its TypeScript and Flutter columns are blank and will need filling when those SDKs add it.Regeneration
The diff is additions only: the four new method sections in
arm.mdandapp.md, plus their rows inarm-table.mdandapp-table.md. No other generated content changed, so there's no SDK drift since #5351.Effect on #5243
With these mappings, the coverage run reports 32 unused methods, down from 48 on the 2026-09-23 run. 14 of the 16 come from this PR (3 arm methods × 4 SDKs, plus
ListRobotsForLocationsin 2 SDKs). The other 2 are the Pythonmove_through_joint_positionsandget_3d_modelsmappings from #5284, regenerated in #5351. The remaining warnings (app*Sequence*methods, the OAuth pair,getGeometriesacross components, and others) are outside this PR's scope, so this doesn't close the issue.Checks
make build-prodpasses.vale3.12.0 (CI's pinned version) reports 0 errors on all changed and added files.🤖 Generated with Claude Code