Skip to content

spec-sync: track V2 spec drift - #140

Open
yzld2002 wants to merge 1 commit into
mainfrom
spec-sync/v2
Open

spec-sync: track V2 spec drift#140
yzld2002 wants to merge 1 commit into
mainfrom
spec-sync/v2

Conversation

@yzld2002

@yzld2002 yzld2002 commented Aug 4, 2026

Copy link
Copy Markdown
Member

Automated V2 spec-sync PR (client.v2).

  • Commit 1 (mechanical): normalized V2 spec snapshot + regenerated reference models.
  • Commit 2 (AI, only if the spec diff needs SDK changes): client.v2 resources/methods/tests/docs wired from the diff, added after this PR opened. Workflow-only drift is excluded and an AI no-op is skipped, so some drifts produce a mechanical-only PR with no second commit.

Gates (surface-lock, V2 contract tests, lint/test/typecheck) must pass. When present, the AI commit is a draft a human finishes (the V2 ergonomic layer — unified Job, dual-host, schema coercion — is not in the spec). Human review required before merge.

What changed

AI-generated from the PR diff — verify against the actual changes.

  • No client/SDK surface changes in this diff — only spec/documentation text was updated (no parameter, field, or route changes).
  • parse/parse async job endpoints: output_save_url docs clarified — presigned URL must remain valid until job completion (not just at submit), with a minimum validity window (default 15 min, plus ~3s/page for analyze) enforced via 422 at submit time; no schema change.
  • model parameter (parse/analyze endpoints): doc-only update — now describes two model families, dpt-3-pro (highest quality) and dpt-3-fast (lower latency, no vision captioning), both supporting dated snapshots, -latest aliases, and bare family names.
  • /v2/workflow (spec-only, not part of SDK surface): description updated to note support for application/x-www-form-urlencoded in addition to JSON/multipart — not exposed via this SDK.

Copilot AI balanced review requested due to automatic review settings August 4, 2026 21:25
@yzld2002
yzld2002 deployed to spec-sync-contract August 4, 2026 21:25 — with GitHub Actions Active

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Tracks V2 API specification drift and refreshes generated reference models.

Changes:

  • Clarifies presigned output URL validity requirements.
  • Documents DPT-3 Fast model support.
  • Advertises form-urlencoded workflow requests, though the formal OpenAPI content schema is missing.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
specs/v2-aide.json Updates V2 API descriptions.
specs/_generated/v2_models.py Regenerates model field descriptions.

Comment thread specs/v2-aide.json
"/v2/workflow": {
"post": {
"description": "Run synchronously and return the result inline.\n\nAccepts ``application/json`` (the request fields as the body) or\n``multipart/form-data`` (file fields are staged automatically).\n\nReturns **504** if the work does not finish within the wait\nwindow — long-running work belongs on the async ``POST\n{path}/jobs`` route, which returns 202 immediately and is polled\nvia ``GET {path}/jobs/{job_id}``. On a 504 the workflow is\nCANCELLED (a sync caller can never collect the result); a retry\nstarts the work over as a fresh job.",
"description": "Run synchronously and return the result inline.\n\nAccepts ``application/json`` (the request fields as the body),\n``multipart/form-data`` (file fields are staged automatically),\nor ``application/x-www-form-urlencoded`` (text fields only).\n\nReturns **504** if the work does not finish within the wait\nwindow — long-running work belongs on the async ``POST\n{path}/jobs`` route, which returns 202 immediately and is polled\nvia ``GET {path}/jobs/{job_id}``. On a 504 the workflow is\nCANCELLED (a sync caller can never collect the result); a retry\nstarts the work over as a fresh job.",
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants