Skip to content

spec-sync: track V1 spec drift - #102

Closed
yzld2002 wants to merge 1 commit into
mainfrom
spec-sync/v1
Closed

spec-sync: track V1 spec drift#102
yzld2002 wants to merge 1 commit into
mainfrom
spec-sync/v1

Conversation

@yzld2002

@yzld2002 yzld2002 commented Jul 8, 2026

Copy link
Copy Markdown
Member

Automated spec-sync PR.

  • Commit 1 (mechanical): normalized spec snapshot + regenerated reference models.
  • Commit 2 (AI): resources/methods/tests/docs wired from the spec diff (added after this PR opened).

Gates (surface-lock, contract tests, lint/test/typecheck) must pass. Human review required before merge.

Copilot AI review requested due to automatic review settings July 8, 2026 03:17

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

This spec-sync PR updates the V1 ADE OpenAPI snapshot to reflect upstream API drift, specifically adding async extract job endpoints so generated clients/docs stay aligned with the service contract.

Changes:

  • Added /v1/ade/extract/jobs GET (list jobs) and POST (create async extract job) path definitions.
  • Added /v1/ade/extract/jobs/{job_id} GET (job status) path definition with additional response codes (including 206 for schema-validation warnings).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yzld2002 yzld2002 closed this in #103 Jul 8, 2026
tian-lan-landing added a commit that referenced this pull request Aug 3, 2026
…#139)

claude-sonnet-5 runs adaptive thinking when `thinking` is unset. The
thinking tokens count against max_tokens and the first content block is
a thinking block, so `.content[0].text` came back empty and the
"What changed" section was silently dropped. Same failure as the
TypeScript side (PR #102, #103); this one just hadn't tripped yet.

Turn thinking off, join all text blocks instead of indexing [0], bump
the cap 700 -> 1000, and log the response shape on empty.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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