starred/skills

Fork 0

mirror of https://github.com/cloudflare/skills.git synced 2026-04-25 22:55:51 +03:00

[PR #29] [MERGED] fix incorrect workflows reference docs #29

New issue

Closed

opened 2026-03-02 03:21:19 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 03:21:19 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cloudflare/skills/pull/29
Author: @elithrar
Created: 2/24/2026
Status: ✅ Merged
Merged: 2/24/2026
Merged by: @elithrar

Base: main ← Head: fix-workflows-reference-errors

📝 Commits (3)

30746e5 fix incorrect workflows reference docs (subrequest limits, event.payload)
5048162 fix stale 1,000 subrequest limit across all reference docs
3465192 add retrieval-over-pretraining guidance to cloudflare skill

📊 Changes

10 files changed (+31 additions, -16 deletions)

View changed files

📝 skills/cloudflare/SKILL.md (+15 -0)
📝 skills/cloudflare/references/pages/configuration.md (+1 -1)
📝 skills/cloudflare/references/pages/gotchas.md (+1 -1)
📝 skills/cloudflare/references/workers-playground/gotchas.md (+1 -1)
📝 skills/cloudflare/references/workers/gotchas.md (+1 -1)
📝 skills/cloudflare/references/workflows/README.md (+1 -1)
📝 skills/cloudflare/references/workflows/api.md (+2 -2)
📝 skills/cloudflare/references/workflows/gotchas.md (+1 -1)
📝 skills/cloudflare/references/workflows/patterns.md (+7 -7)
📝 skills/cloudflare/references/wrangler/gotchas.md (+1 -1)

📄 Description

Fixes incorrect subrequest limits and wrong property name across Workflows and other reference docs. Adds retrieval guidance to the cloudflare skill to prevent future staleness.

Subrequest limit was wrong in two ways (gotchas.md):

Listed as "per step" — it's per workflow instance
Paid tier showed 1,000 — actual default is 10,000, configurable up to 10M via limits.subrequests

event.params doesn't exist on WorkflowEvent<T> (api.md, patterns.md, README.md):

The runtime type has event.payload, not event.params
Validated against workerd generated types and the official hello-world-workflows template in workers-sdk
Replaced all 10 occurrences across 3 files

Stale 1,000 subrequest limit in other references:

Fixed in workers, wrangler, workers-playground, and pages gotchas/configuration
All updated from 1,000 to 10,000 to match current paid plan default

Retrieval guidance added to cloudflare SKILL.md:

Adds "prefer retrieval over pre-training" instruction with retrieval sources table
Directs agents to verify limits, pricing, types, and config against official docs before citing them

Refs #22, #23

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cloudflare/skills/pull/29 **Author:** [@elithrar](https://github.com/elithrar) **Created:** 2/24/2026 **Status:** ✅ Merged **Merged:** 2/24/2026 **Merged by:** [@elithrar](https://github.com/elithrar) **Base:** `main` ← **Head:** `fix-workflows-reference-errors` --- ### 📝 Commits (3) - [`30746e5`](https://github.com/cloudflare/skills/commit/30746e5a7c9a983202540bf8ac54222c467c676d) fix incorrect workflows reference docs (subrequest limits, event.payload) - [`5048162`](https://github.com/cloudflare/skills/commit/5048162d99c454691eabe6470eb6bfb4acd34547) fix stale 1,000 subrequest limit across all reference docs - [`3465192`](https://github.com/cloudflare/skills/commit/3465192adaef851ec4ac1cec8333d2267981d875) add retrieval-over-pretraining guidance to cloudflare skill ### 📊 Changes **10 files changed** (+31 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `skills/cloudflare/SKILL.md` (+15 -0) 📝 `skills/cloudflare/references/pages/configuration.md` (+1 -1) 📝 `skills/cloudflare/references/pages/gotchas.md` (+1 -1) 📝 `skills/cloudflare/references/workers-playground/gotchas.md` (+1 -1) 📝 `skills/cloudflare/references/workers/gotchas.md` (+1 -1) 📝 `skills/cloudflare/references/workflows/README.md` (+1 -1) 📝 `skills/cloudflare/references/workflows/api.md` (+2 -2) 📝 `skills/cloudflare/references/workflows/gotchas.md` (+1 -1) 📝 `skills/cloudflare/references/workflows/patterns.md` (+7 -7) 📝 `skills/cloudflare/references/wrangler/gotchas.md` (+1 -1) </details> ### 📄 Description Fixes incorrect subrequest limits and wrong property name across Workflows and other reference docs. Adds retrieval guidance to the cloudflare skill to prevent future staleness. **Subrequest limit was wrong in two ways** (gotchas.md): - Listed as "per step" — it's per workflow instance - Paid tier showed 1,000 — actual default is 10,000, configurable up to 10M via `limits.subrequests` **`event.params` doesn't exist on `WorkflowEvent<T>`** (api.md, patterns.md, README.md): - The runtime type has `event.payload`, not `event.params` - Validated against `workerd` generated types and the official `hello-world-workflows` template in `workers-sdk` - Replaced all 10 occurrences across 3 files **Stale 1,000 subrequest limit in other references:** - Fixed in workers, wrangler, workers-playground, and pages gotchas/configuration - All updated from 1,000 to 10,000 to match current paid plan default **Retrieval guidance added to cloudflare SKILL.md:** - Adds "prefer retrieval over pre-training" instruction with retrieval sources table - Directs agents to verify limits, pricing, types, and config against official docs before citing them Refs #22, #23 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-02 03:21:19 +03:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/skills#29

No description provided.

Rows
Columns