Skip to content

fix: tags no longer required#854

Merged
deloreyj merged 1 commit intomainfrom
fix-llm-template
Dec 8, 2025
Merged

fix: tags no longer required#854
deloreyj merged 1 commit intomainfrom
fix-llm-template

Conversation

@deloreyj
Copy link
Contributor

@deloreyj deloreyj commented Dec 8, 2025

Description

Fixes the LLM chat template, which was broken because the Workers AI binding no longer accepts an empty array of tags. Tags are also no longer required, but the types have not been updated. This PR removes the tags property from the request payload and adds a //@ts-expect-error for the missing tags property.

Checklist

  • Template Metadata
    • template directory ends with -template
    • "cloudflare" section of package.json is populated
    • template preview image uploaded to Images
    • README is populated and uses <!-- dash-content-start --> and <!-- dash-content-end --> to designate the Dash readme preview
    • .gitignore file exists
    • package.json contains a deploy command
    • package.json contains private: true and no version field

Example package.json

"private": true,
"cloudflare": {
  "label": "Worker + D1 Database",
  "products": [
    "Workers",
    "D1"
  ],
  "categories": [
    "storage"
  ],
  "docs_url": "https://developers.cloudflare.com/d1/",
  "preview_image_url": "https://imagedelivery.net/wSMYJvS3Xw-n339CbDyDIA/cb7cb0a9-6102-4822-633c-b76b7bb25900/public"
}

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

Dashboard preview link

messages,
max_tokens: 1024,
},
// @ts-expect-error tags is no longer required
Copy link

Choose a reason for hiding this comment

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

Where are the types defined? Can they be updated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Types have been updated, but not released yet: cloudflare/workerd#5590

@deloreyj deloreyj merged commit ffa1c2d into main Dec 8, 2025
2 checks passed
@deloreyj deloreyj deleted the fix-llm-template branch December 8, 2025 20:59
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