sync spec: composite-topic imported_model_ids on v2 document queries - #81
Open
dspangen wants to merge 1 commit into
Open
sync spec: composite-topic imported_model_ids on v2 document queries#81dspangen wants to merge 1 commit into
dspangen wants to merge 1 commit into
Conversation
Syncs api/openapi.json from exploreomni/omni@main (f75cae25d89). - `queryPresentations.data.*.query.imported_model_ids` (string[]) added to the v2 document create/patch-draft bodies and the v2 read response. Model IDs the query reaches through its composite topic's imports; the API authorizes the run gate on each. - `models update` description now documents the Connection Admin / Modeler requirements for renaming schema, shared, shared-extension, and branch models. - `dashboards download` gains a description covering how cross-model tiles are resolved (target user, else the API key's user) and omitted when unqueryable. - AI routine failure `code` enum: `no_gate` replaced by `not_expressible`. No new operations, and no new query or path params, so the command surface is unchanged. The new body field is reachable via `omni documents v2-create --schema --field queryPresentations.data.query`. Confidence: high Scope-risk: narrow Not-tested: live API call exercising imported_model_ids Co-Authored-By: Claude <noreply@anthropic.com>
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
1 participant
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.
Summary
Syncs
api/openapi.json(and the embeddedcmd/omni/openapi.json) fromexploreomni/omni@main(f75cae25d89).Headline change
queryPresentations.data.*.query.imported_model_ids(string[]) added to the v2 document bodies —documentsV2Create,documentsV2PatchDraftBody— and to the v2 read response. These are the model IDs a query reaches through its composite topic's imports; the API authorizes the run gate on each.Side effects pulled in by the full sync
models update(PATCH /api/v1/models/{modelId}): description now spells out that renaming a schema, shared, or shared-extension model requires Connection Admin, and renaming a branch requires Modeler.dashboards download(POST /api/v1/dashboards/{identifier}/download): new description covering how cross-model tiles resolve (the target user when named, else the API key's user) and that tiles on unqueryable models are omitted.codeenum:no_gateremoved,not_expressibleadded.Command surface
No new operations, and no new query or path params — the CLI's command and flag surface is unchanged.
imported_model_idsis a request-body field, so it arrives via--bodyrather than as a flag.Test plan
make buildmake test— all packages pass./bin/omni models update --helpand./bin/omni dashboards download --helpreflect the new descriptions./bin/omni documents v2-create --schema --field queryPresentations.data.queryshowsimported_model_idsimported_model_ids(not run)🤖 Generated with Claude Code
https://claude.ai/code/session_01UP53YBDjhQkkn3WAqMR3sf