Skip to content

docs(skills): cycle-d — date-time format + video asset width pitfalls#781

Merged
bokelley merged 2 commits into
mainfrom
cycle-d-datetime-and-video-width
Apr 22, 2026
Merged

docs(skills): cycle-d — date-time format + video asset width pitfalls#781
bokelley merged 2 commits into
mainfrom
cycle-d-datetime-and-video-width

Conversation

@bokelley

Copy link
Copy Markdown
Contributor

Summary

Matrix v15 surfaced 2 narrow drift classes after the 3.0 GA schema sync (#773) added stricter validation:

  1. reporting_period.start/end are ISO 8601 date-time strings per GA — date-only ('2026-04-21') now fails format validation. Mocks should use new Date().toISOString(). 4 hits on get_media_buy_delivery. Added to seller, retail-media, generative-seller, and creative-agent pitfalls.
  2. videoAsset requires width + height per GA (previously optional on VideoAsset). Mocks returning {url} alone fail at /creative_manifest/assets/<name>/width. 2 hits. Added to creative-agent + generative-seller pitfalls with concrete pixel-values example.

Both are GA-tightening classes. No SDK code change.

Trajectory (same filter, same 8 pairs)

v10 v11 v12 v13 v14 v15
Handler throws 14 6 0 0 0 0
-32602 10 5 0 0 0 0
Grader schema fails 11 11 0 0 0
VALIDATION_ERROR classes 6 3 2 2

Expected v16 after merge: VALIDATION_ERROR classes → 0 on the schema axis. Residual failures will be storyboard-specific step expectations (generative quality grading, governance denial shape specifics) — the tight-loop per-pair phase I flagged at the start of Cycle A.

Test plan

  • npm run typecheck clean
  • Matrix v16 post-merge for verification

Matrix v15 surfaced 2 narrow drift classes after the 3.0 GA schema
sync (#773) added stricter validation:

- reporting_period.start/end are ISO 8601 date-time strings per GA;
  date-only ('2026-04-21') now fails format validation. Added to
  seller + retail-media + generative-seller + creative-agent
  pitfall callouts.
- videoAsset({...}) requires width + height per GA (previously
  optional). Mocks returning {url} alone fail at /creative_manifest
  /assets/<name>/width. Added to creative-agent + generative-seller
  pitfalls with concrete pixel-values example.

Both are narrow GA-tightening classes. No SDK code change. Residual
after this lands is storyboard-specific step expectations — the
tight-loop per-pair phase.
@bokelley bokelley merged commit 369aea8 into main Apr 22, 2026
9 checks passed
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