Skip to content

Document platform 429 response replacement in Functions - #71

Open
jordesign wants to merge 1 commit into
mainfrom
docs/functions-429-response-warning
Open

Document platform 429 response replacement in Functions#71
jordesign wants to merge 1 commit into
mainfrom
docs/functions-429-response-warning

Conversation

@jordesign

@jordesign jordesign commented Oct 12, 2026 •
edited by blacksmith-sh Bot
Loading

Copy link
Copy Markdown

Summary

Add a warning beside the Functions Request/Response contract: WP Cloud can replace a handler-generated HTTP 429 JSON response with platform HTML, so applications must not rely on their response body or headers, including Retry-After, surviving.

Link the public WP Cloud HTTP status-code reference. The warning is limited to 429 and does not claim that every 4xx/5xx is replaced or prescribe 409 as a universal rate-limit status.

Verification

Using Bun 1.3.11 and Node 26:

  • check, validate, and build passed.
  • audit, verify:routes, and verify:public-safety passed.
  • verify:prose passed with Vale 3.17.1.
  • Inspected the built Functions page for the warning, Retry-After caveat, and provider reference.
  • git diff --check passed.

Documentation only. No runtime or provider settings changed; authored 5xx behavior remains unverified.

View in Indent View in Slack
Tag @indent to continue the conversation here.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@indent

indent Bot commented Oct 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Issues

No issues found.

CI Checks

All required checks passed. The non-required Spacefast Builds check failed, and it fails identically on main, so this PR did not cause it.

Failing Spacefast Builds
  • Deploy preview fails with 'Static deploys cannot upload runtime control paths: "storage/index.html"'. The same failure appears on main, and this PR only edits one paragraph in content/(dynamic)/functions.mdx, so it is unrelated.

Review agents

Select any unchecked box below to run or rerun that agent.

Passed (1)
  • Indent Review Agent · Docs-only 429 note is accurate, correctly scoped, and passes public-safety; no issues found.
Full results

Indent Review Agent

  • Summary: Docs-only 429 note is accurate, correctly scoped, and passes public-safety; no issues found.
  • Last ran on commit: d85ab8aa
  • Latest result 
    {
  "summary": "Docs-only 429 note is accurate, correctly scoped, and passes public-safety; no issues found.",
  "findings": []
}

@coderabbitai

coderabbitai Bot commented Oct 12, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 64895b4c-a2d8-4b47-b50d-d10541cfa5fe
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@indent

indent Bot commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

Required CI passed. The separate Spacefast preview build failed because the publisher rejects storage/index.html, unrelated to this documentation change. That restriction already has a fix in spacefast/monorepo#4033, so no duplicate fix is needed here.

View session

This branch had an error being deployed

1 failed deployment
preview/docs/functions-429-response-warning — d85ab8aa Deployed Oct 12, 2026 by spacefast[bot]
Sign up for free to 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.

1 participant

@jordesign