Sync spec: v2 app sub-resource routes; drop removed v1 document writes - #93
Merged
Merged
Conversation
Spec sync from omni@main picks up the new alpha app sub-resource surface (GET published app, GET/PUT draft app on the main and named drafts) and the removal of the deprecated v1 document write endpoints (documentsUpdate / documentsPut). The removed operations had CLI-side references: the documentsUpdate flags-only shorthand mapping is deleted (its tier keeps coverage via modelsGitSync), and the deprecated-command --schema test now uses documents transfer-ownership, the one deprecated operation left in the spec. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Nen49iqqgJs1XaS27YLZgb
Contributor
Author
|
Full disclosure, I asked Claude to update the CLI based on the API merged to main and this is what came out of it. There is a lot of stuff that is not my own work, but it should be derived from the openapi spec, so figured it all should be included to match the actual API. |
Dylan-C-Israel
approved these changes
Sep 8, 2026
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.
What
Spec sync from
omni@main. A sync PR carries everything since the last sync (v1.2.1, 09-02) — the sections below separate the feature this sync exists for from what rides along. The line count is doubled by the spec being embedded twice (api/openapi.json+cmd/omni/openapi.json, same file).The feature: v2 app sub-resource routes (alpha)
Five operations added —
documents v2-get-app,v2-get-draft-app,v2-get-main-draft-app,v2-put-app,v2-put-app-auto-draft— plus their 8DocumentsV2App*schemas. The commands generate themselves from the spec (no CLI code), and their--helpcarries the Alpha note verbatim.Rides along: endpoints removed from prod (forced the Go changes)
PATCH/PUT /api/v1/documents/{identifier}(documentsUpdate/documentsPut) were removed in exploreomni/omni#62046 — the endpoints are physically gone, so keeping them in the embedded spec would ship commands that 404 everywhere. CLI-side cleanup:documentsUpdateflags-only shorthand mapping is deleted (the flags-only tier keeps test coverage viamodelsGitSync);TestDeprecatedCommand_SchemaEmitsCleanJSONnow usesdocuments transfer-ownership— the one deprecated operation left in the spec (also aPUT, so the method assertion is unchanged).Rides along: new routes already live on prod
DELETE /api/v1/models/{modelId}(modelsDelete, exploreomni/omni#62377)POST /api/v1/ai/jobs/{jobId}/feedback(aiJobFeedbackSubmit, exploreomni/omni#61910)RoutineCondition*schemasRides along: wording/schema tweaks on existing operations
~15 operations picked up description or schema-detail changes (permits/boost provisioning exploreomni/omni#62043, routines, dashboards download, eval runs, dbt-sync, content search). No operation changed identity.
Testing
make testgreen,gofmtclean. Built and spot-checked: the five new app commands generate with the expected positionals/flags and the Alpha help text;documents update/putare gone.Remaining app routes (PATCH edits, DELETE, create-with-app) are still in review on the omni side and will ride a future sync.
🤖 Generated with Claude Code
https://claude.ai/code/session_01Nen49iqqgJs1XaS27YLZgb