Skip to content

[RSDK-14767] Add apis field to Model for composite models - #919

Closed
Daniel Botros (danielbotros) wants to merge 5 commits into
viamrobotics:mainfrom
danielbotros:composite-model-apis
Closed

Daniel Botros (danielbotros) wants to merge 5 commits into
viamrobotics:mainfrom
danielbotros:composite-model-apis

Conversation

@danielbotros

@danielbotros Daniel Botros (danielbotros) commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

RSDK-14767

Adds an apis field to app.Model so the registry records the full co-equal API set a model serves. apis is the authoritative set going forward; api is retained as a back-compat pointer (always one of apis) so tooling that reads only api keeps working.

Foundation for the Composable APIs CLI + app-backend work. Dependents (bump go.viam.com/api after this merges):

🤖 Generated with Claude Code

A composite model serves more than one co-equal API. Add a repeated apis
field (number 6) to app.Model so the registry can record a model's full API
set. The existing api field stays and is always one of apis, so tooling that
reads only api keeps working; single-API models may omit apis.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@danielbotros Daniel Botros (danielbotros) added the ready-for-protos add this when you want protos to compile on every commit label Oct 13, 2026
@danielbotros
Daniel Botros (danielbotros) marked this pull request as ready for review October 13, 2026 18:09
@danielbotros Daniel Botros (danielbotros) added ready-for-protos add this when you want protos to compile on every commit and removed ready-for-protos add this when you want protos to compile on every commit labels Oct 13, 2026
@danielbotros Daniel Botros (danielbotros) removed the ready-for-protos add this when you want protos to compile on every commit label Oct 13, 2026
@danielbotros Daniel Botros (danielbotros) changed the title app: add apis field to Model for composite (multi-API) models RSDK-14767 : app: add apis field to Model for composite (multi-API) models Oct 13, 2026
@danielbotros Daniel Botros (danielbotros) changed the title RSDK-14767 : app: add apis field to Model for composite (multi-API) models RSDK-14767: Add apis field to Model for composite models Oct 13, 2026
@danielbotros Daniel Botros (danielbotros) changed the title RSDK-14767: Add apis field to Model for composite models [RSDK-14767] Add apis field to Model for composite models Oct 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

protos-compiled safe to test committer is a member of this org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants