Skip to content

update docs catalog#31675

Draft
ethulia wants to merge 1 commit into
productionfrom
ml/update-models
Draft

update docs catalog#31675
ethulia wants to merge 1 commit into
productionfrom
ml/update-models

Conversation

@ethulia

@ethulia ethulia commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

Screenshots (optional)

Documentation checklist

  • Is there a changelog entry (guidelines)? If you don't add one for something awesome and new (however small) — how will our customers find out? Changelogs are automatically posted to RSS feeds, the Discord, and X.
  • The change adheres to the documentation style guide.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Review

⚠️ Part of the review could not complete in commit 28fa2c4 — it will retry on the next push.

Code Review

This code review is in beta and may not always be helpful — use your judgment.

❌ This review could not complete this run; results may be incomplete. It will retry on the next push.

Style Guide Review

✅ No style-guide issues found.

Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.
/ignore-review-limit Permanently lifts the 2-review automatic limit for this PR. Future pushes will trigger reviews as normal.

@github-actions

Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/catalog-models/ @abhishekkankani, @palashgo, @thebongy, @roerohan, @kathayl, @mchenco, @zeke, @superhighfives, @mattrothenberg, @ethulia, @cloudflare/content-engineering, @cloudflare/product-owners

@github-actions

Copy link
Copy Markdown
Contributor

CI run failed: build logs

@ethulia ethulia marked this pull request as draft June 24, 2026 16:33
@ask-bonk ask-bonk Bot added content:edit Request for content edits content:new Request for new/missing content triage Waiting for a docs team review. labels Jun 24, 2026

@ask-bonk ask-bonk Bot 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.

Review summary

This PR updates the AI model catalog data in src/content/catalog-models/ with regenerated examples, code snippets, and metadata. It also adds 4 new model files.

What changed

  • 4 new models: alibaba/hh1.1-i2v, alibaba/hh1.1-r2v, alibaba/hh1.1-t2v, google/gemini-3.5-flash
  • ~115 existing models updated with refreshed examples, raw_response data, code snippets, and schema metadata
  • Several models switched from gatewayMetadata.keySource: "BYOK" to "Unified"

Issues found

Build-breaking: openai-gpt-5.1-chat.json has "schema": null

pnpm run check fails with:

catalog-models → openai-gpt-51-chat data does not match collection schema.
schema: Expected type "object", received "object"

The Zod schema for catalog-models allows schema to be the object shape or undefined (omitted), but not null:

schema: z.object({ ... }).optional()

In src/content/catalog-models/openai-gpt-5.1-chat.json, the schema field was removed from the diff but left as "schema": null at the end of the file. This must be fixed before the PR can merge.

Suggested fix: Either restore the JSON Schema object or remove the "schema": null key entirely.

Minor notes

  • PR title/description: The title "update docs catalog" is vague, and the description body is the empty template. Consider updating the title to something like [AI] Update Workers AI catalog models and adding a brief description of what changed and why.
  • Schema version: openai-gpt-5.1-chat.json changed schema_version from "1.0.0" to null. This is schema-valid (z.string().nullable()) but confirm this is intentional.
  • No redirects needed: Since no files were renamed or moved, redirects are not required.
  • Lint/format: Pass cleanly.

Labels applied

content:edit, content:new, triage

@ask-bonk

ask-bonk Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

I've completed the review of PR #31675. Here's what I did:

Review posted

Summary of findings

What the PR changes:

  • Updates ~115 existing AI model catalog JSON files in src/content/catalog-models/ with regenerated examples, raw_response data, code snippets, and metadata
  • Adds 4 new models: alibaba/hh1.1-i2v, alibaba/hh1.1-r2v, alibaba/hh1.1-t2v, google/gemini-3.5-flash
  • Several models updated from gatewayMetadata.keySource: "BYOK" to "Unified"

Build-blocking issue flagged:

  • openai-gpt-5.1-chat.json contains "schema": null, which fails pnpm run check because the Zod schema expects schema to be either an object or omitted — null is not allowed

Minor issues noted:

  • PR title ("update docs catalog") is vague and the description is empty
  • schema_version changed to null in one file — valid but worth confirming as intentional
  • Lint and format checks pass cleanly

Labels applied

  • content:edit
  • content:new
  • triage

github run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content:edit Request for content edits content:new Request for new/missing content size/xl triage Waiting for a docs team review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants