Repository navigation
DOCS-5024: Document machine favorites - #5455
Jeremy Rose (jeremyrose-viam) merged 12 commits into
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
✅ Deploy Preview for viam-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Docs review: Ready to mergeRe-review of 315234c. The should-fix and the "drops out" nit from the last review are fixed, and the feature-flag question is settled in code: CI: All required checks passing Docs health: No regressions. Merge order (no action needed here): PR 5441 ( Verified against source (12 claims)New since the last review:
Carried over from the last review, still accurate at head:
Automated review by Claude. Rerunning the review updates this comment. |
…achines Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
Thanks for the review. Pushed 315234c to address it:
Prettier, markdownlint, vale, the frontmatter check, and |
|
Hey Jeremy Rose (@jeremyrose-viam) — this PR has been approved and CI has been green for 3+ business days. Ready to merge? Auto-comment from overwatch. Will not re-nudge for 7 days. |
b7420ef
into
viamrobotics:main
|
🔎💬 Inkeep AI search and chat service is syncing content for source 'Viam Docs' |

Documents machine favorites, which are live in the Viam app and the Viam MCP server.
Documents: DOCS-5024
Changes
docs/monitor/monitor.md: new "Favorite machines" subsection under "Check fleet status". It covers the star in the machines table, favorited machines appearing at the top, the Machine Favorites tab in user settings, favorites being per user, machines you can no longer access dropping out, and a pointer to the MCP tools.docs/reference/mcp/_index.md: a paragraph saying the favorite tools read and change only your own favorites, and whyremove_favorite_machineis marked destructive.static/include/app/mcpserver/generated/mcp-tools-table.md: three rows for the new tools:list_favorite_machines(Read-only),add_favorite_machine(Write), andremove_favorite_machine(Write (destructive)). The categories match the tool definitions, and the tools were checked against production (listed favorites, then removed one and listed again)..github/workflows/update_sdk_methods.py: addsadd_favorite_machine,remove_favorite_machine, andlist_favorite_machinesto the Python "unused method" ignore list, with a comment, so the weekly coverage job doesn't warn about them.Not included
The favorites methods are not added to
sdk_protos_map.csvor the generated fleet API reference. Favorites belong to a signed-in user, but the Python app client only accepts API-key credentials, so these methods can't succeed for typical callers. No other SDK has favorites methods in the app API reference today (the Flutter app client isn't part of that reference, and the TypeScript and Go SDKs have no wrappers). If an SDK gains user authentication or wrappers, the CSV rows and proto descriptions can be added then.Merge order
PR 5441 (
use_skill) and PR 5427 (simulation tools, draft) also editmcp-tools-table.md. Whichever merges later needs a small row reconciliation. Alphabetically,remove_favorite_machinegoes beforestop_simulationin the destructive group.Checks
Repo-local prettier 3.2.5, markdownlint, vale,
scripts/frontmatter_rules.py, andmake build-prodpass. The built Monitor page has the#favorite-machinesanchor that the MCP page links to.🤖 Generated with Claude Code