fix: order the 429 check before the 4xx check in the workflow skill - #3915

Open
Nixxx19 wants to merge 1 commit into
vercel:mainfrom
Nixxx19:fix/skill-429-unreachable
Open

fix: order the 429 check before the 4xx check in the workflow skill#3915
Nixxx19 wants to merge 1 commit into
vercel:mainfrom
Nixxx19:fix/skill-429-unreachable

Conversation

@Nixxx19

@Nixxx19Nixxx19 commented Sep 1, 2026•
edited
Loading

Copy link
Copy Markdown

Description

fixes#3652

the error handling snippet in skills/workflow/SKILL.md checks 4xx before 429, so the RetryableError branch under it can never run. 429 >= 400 && 429 < 500 is true, the FatalError throws first, and rate limits end up treated as permanent failures with no retry.

swapping the two blocks is the whole fix.

this matters more than a normal docs typo because the file is the agent facing skill and gets autoloaded on workflow tasks, so the snippet is copied close to verbatim into generated code. it inverts the behaviour for the most common transient http failure there is.

also bumped the skill version to 1.12, matching how content changes to this file have been handled before (1.6 through 1.10 each bumped, the technical writing pass in #3704 did not since it changed no behaviour).

How did you test your changes?

no runtime code changed, so there is nothing to unit test. what i verified instead:

  • confirmed the branch really is unreachable, 429 satisfies >= 400 && < 500 so the first throw always wins
  • grepped for the same pattern everywhere else it could appear: v4 and v5 docs, all six skills/*, docs/app, and the workbench. every other occurrence already orders correctly, so this is the only wrong one. docs/content/docs/v5/api-reference/workflow/fetch.mdx handles 429 inside the 4xx block, and errors-and-retries.mdx and rate-limiting.mdx both check 429 first
  • kept the diff to the reordering only. reading Retry-After off the response like fetch.mdx does would be a nicer example but it widens the change beyond the bug

PR Checklist - Required to merge

  • 📦 pnpm changeset was run to create a changelog for this PR
    • used --empty, per the template note for documentation changes. skills/ is not in the pnpm workspace so there is no package to version, and the bot confirms "changesets to release 0 packages"
  • 🔒 DCO sign-off passes (run git commit --signoff on your commits)
  • 📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete

@Nixxx19
Nixxx19 requested a review from a team as a code ownerSeptember 1, 2026 06:14
@changeset-bot

changeset-botBot commented Sep 1, 2026•
edited
Loading

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: b4eb17a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercelBot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

@Nixxx19 is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@Nixxx19
Nixxx19force-pushed the fix/skill-429-unreachable branch from 1b53693 to 388234eCompareSeptember 1, 2026 06:24
@Nixxx19
fix: order the 429 check before the 4xx check in the workflow skill
b4eb17a
Signed-off-by: nityam <185968020+Nixxx19@users.noreply.github.com>
@Nixxx19
Nixxx19force-pushed the fix/skill-429-unreachable branch from 388234e to b4eb17aCompareSeptember 1, 2026 06:57
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

skills/workflow/SKILL.md: the 429 RetryableError branch is unreachable (429 is caught by the preceding 400-499 FatalError)

1 participant

@Nixxx19
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
fix: order the 429 check before the 4xx check in the workflow skill - #3915
Open
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Open
fix: order the 429 check before the 4xx check in the workflow skill#3915
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Conversation
@Nixxx19
@Nixxx19Nixxx19
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

Description
fixes#3652
the error handling snippet in skills/workflow/SKILL.md checks 4xx before 429, so the RetryableError branch under it can never run. 429 >= 400 && 429 < 500 is true, the FatalError throws first, and rate limits end up treated as permanent failures with no retry.
swapping the two blocks is the whole fix.
this matters more than a normal docs typo because the file is the agent facing skill and gets autoloaded on workflow tasks, so the snippet is copied close to verbatim into generated code. it inverts the behaviour for the most common transient http failure there is.
also bumped the skill version to 1.12, matching how content changes to this file have been handled before (1.6 through 1.10 each bumped, the technical writing pass in #3704 did not since it changed no behaviour).
How did you test your changes?
no runtime code changed, so there is nothing to unit test. what i verified instead:
  • confirmed the branch really is unreachable, 429 satisfies >= 400 && < 500 so the first throw always wins
  • grepped for the same pattern everywhere else it could appear: v4 and v5 docs, all six skills/*, docs/app, and the workbench. every other occurrence already orders correctly, so this is the only wrong one. docs/content/docs/v5/api-reference/workflow/fetch.mdx handles 429 inside the 4xx block, and errors-and-retries.mdx and rate-limiting.mdx both check 429 first
  • kept the diff to the reordering only. reading Retry-After off the response like fetch.mdx does would be a nicer example but it widens the change beyond the bug
PR Checklist - Required to merge
  •  📦 pnpm changeset was run to create a changelog for this PR

    • used --empty, per the template note for documentation changes. skills/ is not in the pnpm workspace so there is no package to version, and the bot confirms "changesets to release 0 packages"
  •  🔒 DCO sign-off passes (run git commit --signoff on your commits)
  •  📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
@Nixxx19
Nixxx19
requested a review
from a team
as a code ownerSeptember 1, 2026 06:14
@changeset-bot
changeset-botBot
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: b4eb17a
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 0 packages
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
@vercel
vercelBot
commented
Sep 1, 2026

Copy link


Copy Markdown

Contributor
@Nixxx19 is attempting to deploy a commit to the Vercel Labs Team on Vercel.
A member of the Team first needs to authorize it.
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
1b53693 to
388234eCompareSeptember 1, 2026 06:24
@Nixxx19
fix: order the 429 check before the 4xx check in the workflow skill
b4eb17a
Signed-off-by: nityam <185968020+Nixxx19@users.noreply.github.com>
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
388234e to
b4eb17aCompareSeptember 1, 2026 06:57
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

skills/workflow/SKILL.md: the 429 RetryableError branch is unreachable (429 is caught by the preceding 400-499 FatalError)


1 participant

@Nixxx19

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix: order the 429 check before the 4xx check in the workflow skill - #3915
Open
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Open
fix: order the 429 check before the 4xx check in the workflow skill#3915
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Conversation
@Nixxx19
@Nixxx19Nixxx19
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

Description
fixes#3652
the error handling snippet in skills/workflow/SKILL.md checks 4xx before 429, so the RetryableError branch under it can never run. 429 >= 400 && 429 < 500 is true, the FatalError throws first, and rate limits end up treated as permanent failures with no retry.
swapping the two blocks is the whole fix.
this matters more than a normal docs typo because the file is the agent facing skill and gets autoloaded on workflow tasks, so the snippet is copied close to verbatim into generated code. it inverts the behaviour for the most common transient http failure there is.
also bumped the skill version to 1.12, matching how content changes to this file have been handled before (1.6 through 1.10 each bumped, the technical writing pass in #3704 did not since it changed no behaviour).
How did you test your changes?
no runtime code changed, so there is nothing to unit test. what i verified instead:
  • confirmed the branch really is unreachable, 429 satisfies >= 400 && < 500 so the first throw always wins
  • grepped for the same pattern everywhere else it could appear: v4 and v5 docs, all six skills/*, docs/app, and the workbench. every other occurrence already orders correctly, so this is the only wrong one. docs/content/docs/v5/api-reference/workflow/fetch.mdx handles 429 inside the 4xx block, and errors-and-retries.mdx and rate-limiting.mdx both check 429 first
  • kept the diff to the reordering only. reading Retry-After off the response like fetch.mdx does would be a nicer example but it widens the change beyond the bug
PR Checklist - Required to merge
  •  📦 pnpm changeset was run to create a changelog for this PR

    • used --empty, per the template note for documentation changes. skills/ is not in the pnpm workspace so there is no package to version, and the bot confirms "changesets to release 0 packages"
  •  🔒 DCO sign-off passes (run git commit --signoff on your commits)
  •  📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
@Nixxx19
Nixxx19
requested a review
from a team
as a code ownerSeptember 1, 2026 06:14
@changeset-bot
changeset-botBot
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: b4eb17a
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 0 packages
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
@vercel
vercelBot
commented
Sep 1, 2026

Copy link


Copy Markdown

Contributor
@Nixxx19 is attempting to deploy a commit to the Vercel Labs Team on Vercel.
A member of the Team first needs to authorize it.
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
1b53693 to
388234eCompareSeptember 1, 2026 06:24
@Nixxx19
fix: order the 429 check before the 4xx check in the workflow skill
b4eb17a
Signed-off-by: nityam <185968020+Nixxx19@users.noreply.github.com>
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
388234e to
b4eb17aCompareSeptember 1, 2026 06:57
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

skills/workflow/SKILL.md: the 429 RetryableError branch is unreachable (429 is caught by the preceding 400-499 FatalError)


1 participant

@Nixxx19

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix: order the 429 check before the 4xx check in the workflow skill - #3915
Open
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Open
fix: order the 429 check before the 4xx check in the workflow skill#3915
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Conversation
@Nixxx19
@Nixxx19Nixxx19
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

Description
fixes#3652
the error handling snippet in skills/workflow/SKILL.md checks 4xx before 429, so the RetryableError branch under it can never run. 429 >= 400 && 429 < 500 is true, the FatalError throws first, and rate limits end up treated as permanent failures with no retry.
swapping the two blocks is the whole fix.
this matters more than a normal docs typo because the file is the agent facing skill and gets autoloaded on workflow tasks, so the snippet is copied close to verbatim into generated code. it inverts the behaviour for the most common transient http failure there is.
also bumped the skill version to 1.12, matching how content changes to this file have been handled before (1.6 through 1.10 each bumped, the technical writing pass in #3704 did not since it changed no behaviour).
How did you test your changes?
no runtime code changed, so there is nothing to unit test. what i verified instead:
  • confirmed the branch really is unreachable, 429 satisfies >= 400 && < 500 so the first throw always wins
  • grepped for the same pattern everywhere else it could appear: v4 and v5 docs, all six skills/*, docs/app, and the workbench. every other occurrence already orders correctly, so this is the only wrong one. docs/content/docs/v5/api-reference/workflow/fetch.mdx handles 429 inside the 4xx block, and errors-and-retries.mdx and rate-limiting.mdx both check 429 first
  • kept the diff to the reordering only. reading Retry-After off the response like fetch.mdx does would be a nicer example but it widens the change beyond the bug
PR Checklist - Required to merge
  •  📦 pnpm changeset was run to create a changelog for this PR

    • used --empty, per the template note for documentation changes. skills/ is not in the pnpm workspace so there is no package to version, and the bot confirms "changesets to release 0 packages"
  •  🔒 DCO sign-off passes (run git commit --signoff on your commits)
  •  📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
@Nixxx19
Nixxx19
requested a review
from a team
as a code ownerSeptember 1, 2026 06:14
@changeset-bot
changeset-botBot
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: b4eb17a
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 0 packages
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
@vercel
vercelBot
commented
Sep 1, 2026

Copy link


Copy Markdown

Contributor
@Nixxx19 is attempting to deploy a commit to the Vercel Labs Team on Vercel.
A member of the Team first needs to authorize it.
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
1b53693 to
388234eCompareSeptember 1, 2026 06:24
@Nixxx19
fix: order the 429 check before the 4xx check in the workflow skill
b4eb17a
Signed-off-by: nityam <185968020+Nixxx19@users.noreply.github.com>
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
388234e to
b4eb17aCompareSeptember 1, 2026 06:57
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

skills/workflow/SKILL.md: the 429 RetryableError branch is unreachable (429 is caught by the preceding 400-499 FatalError)


1 participant

@Nixxx19

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
fix: order the 429 check before the 4xx check in the workflow skill - #3915
Open
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Open
fix: order the 429 check before the 4xx check in the workflow skill#3915
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Conversation
@Nixxx19
@Nixxx19Nixxx19
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

Description
fixes#3652
the error handling snippet in skills/workflow/SKILL.md checks 4xx before 429, so the RetryableError branch under it can never run. 429 >= 400 && 429 < 500 is true, the FatalError throws first, and rate limits end up treated as permanent failures with no retry.
swapping the two blocks is the whole fix.
this matters more than a normal docs typo because the file is the agent facing skill and gets autoloaded on workflow tasks, so the snippet is copied close to verbatim into generated code. it inverts the behaviour for the most common transient http failure there is.
also bumped the skill version to 1.12, matching how content changes to this file have been handled before (1.6 through 1.10 each bumped, the technical writing pass in #3704 did not since it changed no behaviour).
How did you test your changes?
no runtime code changed, so there is nothing to unit test. what i verified instead:
  • confirmed the branch really is unreachable, 429 satisfies >= 400 && < 500 so the first throw always wins
  • grepped for the same pattern everywhere else it could appear: v4 and v5 docs, all six skills/*, docs/app, and the workbench. every other occurrence already orders correctly, so this is the only wrong one. docs/content/docs/v5/api-reference/workflow/fetch.mdx handles 429 inside the 4xx block, and errors-and-retries.mdx and rate-limiting.mdx both check 429 first
  • kept the diff to the reordering only. reading Retry-After off the response like fetch.mdx does would be a nicer example but it widens the change beyond the bug
PR Checklist - Required to merge
  •  📦 pnpm changeset was run to create a changelog for this PR

    • used --empty, per the template note for documentation changes. skills/ is not in the pnpm workspace so there is no package to version, and the bot confirms "changesets to release 0 packages"
  •  🔒 DCO sign-off passes (run git commit --signoff on your commits)
  •  📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
@Nixxx19
Nixxx19
requested a review
from a team
as a code ownerSeptember 1, 2026 06:14
@changeset-bot
changeset-botBot
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: b4eb17a
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 0 packages
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
@vercel
vercelBot
commented
Sep 1, 2026

Copy link


Copy Markdown

Contributor
@Nixxx19 is attempting to deploy a commit to the Vercel Labs Team on Vercel.
A member of the Team first needs to authorize it.
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
1b53693 to
388234eCompareSeptember 1, 2026 06:24
@Nixxx19
fix: order the 429 check before the 4xx check in the workflow skill
b4eb17a
Signed-off-by: nityam <185968020+Nixxx19@users.noreply.github.com>
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
388234e to
b4eb17aCompareSeptember 1, 2026 06:57
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

skills/workflow/SKILL.md: the 429 RetryableError branch is unreachable (429 is caught by the preceding 400-499 FatalError)


1 participant

@Nixxx19

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix: order the 429 check before the 4xx check in the workflow skill - #3915
Open
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Open
fix: order the 429 check before the 4xx check in the workflow skill#3915
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Conversation
@Nixxx19
@Nixxx19Nixxx19
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

Description
fixes#3652
the error handling snippet in skills/workflow/SKILL.md checks 4xx before 429, so the RetryableError branch under it can never run. 429 >= 400 && 429 < 500 is true, the FatalError throws first, and rate limits end up treated as permanent failures with no retry.
swapping the two blocks is the whole fix.
this matters more than a normal docs typo because the file is the agent facing skill and gets autoloaded on workflow tasks, so the snippet is copied close to verbatim into generated code. it inverts the behaviour for the most common transient http failure there is.
also bumped the skill version to 1.12, matching how content changes to this file have been handled before (1.6 through 1.10 each bumped, the technical writing pass in #3704 did not since it changed no behaviour).
How did you test your changes?
no runtime code changed, so there is nothing to unit test. what i verified instead:
  • confirmed the branch really is unreachable, 429 satisfies >= 400 && < 500 so the first throw always wins
  • grepped for the same pattern everywhere else it could appear: v4 and v5 docs, all six skills/*, docs/app, and the workbench. every other occurrence already orders correctly, so this is the only wrong one. docs/content/docs/v5/api-reference/workflow/fetch.mdx handles 429 inside the 4xx block, and errors-and-retries.mdx and rate-limiting.mdx both check 429 first
  • kept the diff to the reordering only. reading Retry-After off the response like fetch.mdx does would be a nicer example but it widens the change beyond the bug
PR Checklist - Required to merge
  •  📦 pnpm changeset was run to create a changelog for this PR

    • used --empty, per the template note for documentation changes. skills/ is not in the pnpm workspace so there is no package to version, and the bot confirms "changesets to release 0 packages"
  •  🔒 DCO sign-off passes (run git commit --signoff on your commits)
  •  📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
@Nixxx19
Nixxx19
requested a review
from a team
as a code ownerSeptember 1, 2026 06:14
@changeset-bot
changeset-botBot
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: b4eb17a
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 0 packages
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
@vercel
vercelBot
commented
Sep 1, 2026

Copy link


Copy Markdown

Contributor
@Nixxx19 is attempting to deploy a commit to the Vercel Labs Team on Vercel.
A member of the Team first needs to authorize it.
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
1b53693 to
388234eCompareSeptember 1, 2026 06:24
@Nixxx19
fix: order the 429 check before the 4xx check in the workflow skill
b4eb17a
Signed-off-by: nityam <185968020+Nixxx19@users.noreply.github.com>
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
388234e to
b4eb17aCompareSeptember 1, 2026 06:57
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

skills/workflow/SKILL.md: the 429 RetryableError branch is unreachable (429 is caught by the preceding 400-499 FatalError)


1 participant

@Nixxx19

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
fix: order the 429 check before the 4xx check in the workflow skill - #3915
Open
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Open
fix: order the 429 check before the 4xx check in the workflow skill#3915
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Conversation
@Nixxx19
@Nixxx19Nixxx19
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

Description
fixes#3652
the error handling snippet in skills/workflow/SKILL.md checks 4xx before 429, so the RetryableError branch under it can never run. 429 >= 400 && 429 < 500 is true, the FatalError throws first, and rate limits end up treated as permanent failures with no retry.
swapping the two blocks is the whole fix.
this matters more than a normal docs typo because the file is the agent facing skill and gets autoloaded on workflow tasks, so the snippet is copied close to verbatim into generated code. it inverts the behaviour for the most common transient http failure there is.
also bumped the skill version to 1.12, matching how content changes to this file have been handled before (1.6 through 1.10 each bumped, the technical writing pass in #3704 did not since it changed no behaviour).
How did you test your changes?
no runtime code changed, so there is nothing to unit test. what i verified instead:
  • confirmed the branch really is unreachable, 429 satisfies >= 400 && < 500 so the first throw always wins
  • grepped for the same pattern everywhere else it could appear: v4 and v5 docs, all six skills/*, docs/app, and the workbench. every other occurrence already orders correctly, so this is the only wrong one. docs/content/docs/v5/api-reference/workflow/fetch.mdx handles 429 inside the 4xx block, and errors-and-retries.mdx and rate-limiting.mdx both check 429 first
  • kept the diff to the reordering only. reading Retry-After off the response like fetch.mdx does would be a nicer example but it widens the change beyond the bug
PR Checklist - Required to merge
  •  📦 pnpm changeset was run to create a changelog for this PR

    • used --empty, per the template note for documentation changes. skills/ is not in the pnpm workspace so there is no package to version, and the bot confirms "changesets to release 0 packages"
  •  🔒 DCO sign-off passes (run git commit --signoff on your commits)
  •  📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
@Nixxx19
Nixxx19
requested a review
from a team
as a code ownerSeptember 1, 2026 06:14
@changeset-bot
changeset-botBot
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: b4eb17a
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 0 packages
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
@vercel
vercelBot
commented
Sep 1, 2026

Copy link


Copy Markdown

Contributor
@Nixxx19 is attempting to deploy a commit to the Vercel Labs Team on Vercel.
A member of the Team first needs to authorize it.
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
1b53693 to
388234eCompareSeptember 1, 2026 06:24
@Nixxx19
fix: order the 429 check before the 4xx check in the workflow skill
b4eb17a
Signed-off-by: nityam <185968020+Nixxx19@users.noreply.github.com>
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
388234e to
b4eb17aCompareSeptember 1, 2026 06:57
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

skills/workflow/SKILL.md: the 429 RetryableError branch is unreachable (429 is caught by the preceding 400-499 FatalError)


1 participant

@Nixxx19

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
fix: order the 429 check before the 4xx check in the workflow skill - #3915
Open
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Open
fix: order the 429 check before the 4xx check in the workflow skill#3915
Nixxx19 wants to merge 1 commit into
vercel:mainfrom 
Nixxx19:fix/skill-429-unreachable
Conversation
@Nixxx19
@Nixxx19Nixxx19
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

Description
fixes#3652
the error handling snippet in skills/workflow/SKILL.md checks 4xx before 429, so the RetryableError branch under it can never run. 429 >= 400 && 429 < 500 is true, the FatalError throws first, and rate limits end up treated as permanent failures with no retry.
swapping the two blocks is the whole fix.
this matters more than a normal docs typo because the file is the agent facing skill and gets autoloaded on workflow tasks, so the snippet is copied close to verbatim into generated code. it inverts the behaviour for the most common transient http failure there is.
also bumped the skill version to 1.12, matching how content changes to this file have been handled before (1.6 through 1.10 each bumped, the technical writing pass in #3704 did not since it changed no behaviour).
How did you test your changes?
no runtime code changed, so there is nothing to unit test. what i verified instead:
  • confirmed the branch really is unreachable, 429 satisfies >= 400 && < 500 so the first throw always wins
  • grepped for the same pattern everywhere else it could appear: v4 and v5 docs, all six skills/*, docs/app, and the workbench. every other occurrence already orders correctly, so this is the only wrong one. docs/content/docs/v5/api-reference/workflow/fetch.mdx handles 429 inside the 4xx block, and errors-and-retries.mdx and rate-limiting.mdx both check 429 first
  • kept the diff to the reordering only. reading Retry-After off the response like fetch.mdx does would be a nicer example but it widens the change beyond the bug
PR Checklist - Required to merge
  •  📦 pnpm changeset was run to create a changelog for this PR

    • used --empty, per the template note for documentation changes. skills/ is not in the pnpm workspace so there is no package to version, and the bot confirms "changesets to release 0 packages"
  •  🔒 DCO sign-off passes (run git commit --signoff on your commits)
  •  📝 Ping @vercel/workflow in a comment once the PR is ready, and the above checklist is complete
@Nixxx19
Nixxx19
requested a review
from a team
as a code ownerSeptember 1, 2026 06:14
@changeset-bot
changeset-botBot
commented
Sep 1, 2026•

edited

Loading

Copy link


Copy Markdown

🦋 Changeset detected
Latest commit: b4eb17a
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 0 packages
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
@vercel
vercelBot
commented
Sep 1, 2026

Copy link


Copy Markdown

Contributor
@Nixxx19 is attempting to deploy a commit to the Vercel Labs Team on Vercel.
A member of the Team first needs to authorize it.
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
1b53693 to
388234eCompareSeptember 1, 2026 06:24
@Nixxx19
fix: order the 429 check before the 4xx check in the workflow skill
b4eb17a
Signed-off-by: nityam <185968020+Nixxx19@users.noreply.github.com>
@Nixxx19
Nixxx19force-pushed
the
fix/skill-429-unreachable
branch
from
388234e to
b4eb17aCompareSeptember 1, 2026 06:57
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

skills/workflow/SKILL.md: the 429 RetryableError branch is unreachable (429 is caught by the preceding 400-499 FatalError)


1 participant

@Nixxx19