Skip to content

chore: regenerate types from latest AdCP schema tarball#767

Closed
bokelley wants to merge 3 commits into
mainfrom
chore-schema-regen
Closed

chore: regenerate types from latest AdCP schema tarball#767
bokelley wants to merge 3 commits into
mainfrom
chore-schema-regen

Conversation

@bokelley

Copy link
Copy Markdown
Contributor

Routine type regen after upstream schema sync. npm run typecheck clean; npm test expected clean (runs in CI).

Unblocks matrix v12 — CI's "Validate generated files" fails on any branch with stale generated types, so landing this before the matrix keeps the failing-pair baseline clean.

bokelley and others added 3 commits April 22, 2026 00:07
…criminator

Schemas regenerated from upstream now require an `asset_type` discriminator
on every creative asset and `action` on each per-creative sync_creatives
response item. Updates:

- `src/lib/conformance/seeder.ts` ASSET_PLACEHOLDER — emit `asset_type` on
  every placeholder so seeded creatives pass the strict response-validation
  surface default in dev/test.
- `test/lib/conformance-seeder.test.js` — mock handler returns `action`,
  assertion expects `asset_type` on the synthesized text asset.
- `test/lib/request-validation.test.js` — video asset fixtures include
  `asset_type: 'video'`.
- `test/lib/zod-schemas.test.js` — GetCreativeFeatures fixture uses the new
  image-asset shape (`asset_type`, `width`, `height`).

Plus a changeset covering the regen itself.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Previous push didn't fire pull_request-event workflows.
Empty commit to nudge CI.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@bokelley

Copy link
Copy Markdown
Contributor Author

Closing as superseded.

PR #762 (merged 2026-04-22T02:55Z) already included the upstream schema regen (core.generated.ts / schemas.generated.ts / tools.generated.ts) along with the conformance-seeder fixes and test-fixture updates needed to keep the suite green under strict response validation. Current main matches what this PR was trying to produce — re-running npm run sync-schemas && npm run generate-types on main now produces zero diff.

Matrix v12 unblock was handled via #762.

@bokelley bokelley closed this Apr 22, 2026
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.

1 participant