sync spec: document the cache values query run actually accepts - #100
Open
ernestoongaro wants to merge 1 commit into
Open
ernestoongaro wants to merge 1 commit into
ernestoongaro wants to merge 1 commit into
Conversation
query run's `cache` enum now lists what the API validates (Standard | SkipRequery | SkipCache | SkipCacheAndRebuildExtracts) instead of disabled | normal | refresh | refresh_all, which always returned 400. `timezone` is now documented on the request body. Upstream: exploreomni/omni#64011. Also pulled in by the full sync: - New `skills` group: list, get, create, update, delete - query wait: jobIds no longer required; deprecated job_ids documented (exposed as --job-ids-2) - models content-validator: new force_full_validation query param - Schematized document/dashboard payloads, composite topic schemas, AI routines and v2 draft app route doc updates Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NcBoDwoiTDYfkGVJgy2W3B
Collaborator
Author
|
Tested this build against a live instance.
Other changes in this sync
|
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.
Summary
Headline:
omni query run'scachefield now documents the values the API really accepts:Standard,SkipRequery,SkipCache,SkipCacheAndRebuildExtracts. The old values (disabled | normal | refresh | refresh_all) came from the spec and always returned a 400, so anyone followingomni query run --schemahit that error.timezoneis also documented now. Upstream fix: exploreomni/omni#64011.Also pulled in by the full sync:
skillsgroup:list,get,create,update,delete(from exploreomni/omni#62425). Create and update take--body.query wait:jobIdsis no longer required. The deprecatedjob_idsis documented and shows up as--job-ids-2, because the generator renames flags that collide.models content-validator: new--force-full-validationquery param.Test plan
make buildmake testomni query run --schemashows the newcacheenum andtimezoneomni skills --helplists the 5 new commandsomni query wait --help/omni models content-validator --helpshow the new flagsLive, read-only on a playground instance:
query runwithcacheset toStandard,SkipRequery,SkipCache,SkipCacheAndRebuildExtracts, and with no cache: all return resultsquery runwith the oldrefresh/normal: 400 from the API, as expected (the API never accepted them)query runwithtimezone: America/Los_Angeles: worksquery wait --job-ids <id>and--job-ids-2 <id>: both stream the finished jobskills list: returns a paginated responsemodels content-validator-get <model> --force-full-validation true: works🤖 Generated with Claude Code
https://claude.ai/code/session_01NcBoDwoiTDYfkGVJgy2W3B