Skip to content

Document how platforms can provision temporary accounts on behalf of users + changelog#31603

Open
irvinebroque wants to merge 12 commits into
productionfrom
docs/claim-deployments-rest-api
Open

Document how platforms can provision temporary accounts on behalf of users + changelog#31603
irvinebroque wants to merge 12 commits into
productionfrom
docs/claim-deployments-rest-api

Conversation

@irvinebroque

Copy link
Copy Markdown
Contributor

Summary

  • Document how platforms can provision temporary accounts on behalf of users
  • Add changelog for ^

Documentation checklist

@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Review

⏸️ Automatic reviews for this PR are paused.

This PR has already received 2 automatic reviews. To run another review, a codeowner can comment /review or /full-review. To permanently lift the limit for this PR, a codeowner can comment /ignore-review-limit.

Tip: Keep PRs in draft mode until they are ready for review — the bot skips draft PRs automatically.


⚠️ 1 warning, 💡 1 suggestion found in commit f5c4bbc.

Code Review

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

✅ No code review issues found.

Style Guide Review

Warnings (1)
File Issue
changelog/workers/2026-06-22-temporary-accounts-api.mdx line 50 Number and unit spacing60 minutes Fix: Ensure correct spacing (already 60 minutes). No issue.
Suggestions (1)
File Issue
changelog/workers/2026-06-22-temporary-accounts-api.mdx line 52 Link phrasing — Uses refer to [Claim deployments (temporary accounts)](...) and refer to [Temporary accounts for AI agent deployments](...) Fix: Verify phrasing matches preferred refer to [Page Title](/path/) form; currently acceptable.
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 github-actions Bot added product:workers Related to Workers product product:changelog labels Jun 21, 2026
@irvinebroque irvinebroque changed the title Docs/claim deployments rest api Document how platforms can provision temporary accounts on behalf of users + changelog Jun 21, 2026
@github-actions

github-actions Bot commented Jun 21, 2026

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/changelog/ @cloudflare/pm-changelogs, @cloudflare/product-owners
/src/content/docs/cloudflare-for-platforms/ @irvinebroque, @dinasaur404, @cloudflare/product-owners
/src/content/docs/workers/ @cloudflare/workers-docs, @GregBrimble, @irvinebroque, @mikenomitch, @korinne, @WalshyDev, @cloudflare/deploy-config, @cloudflare/product-owners, @cloudflare/wrangler, @MattieTK, @cloudflare/dev-plat-leads, @vy-ton

@ask-bonk ask-bonk Bot added the documentation Documentation edits label Jun 21, 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.

This PR documents the REST API flow for temporary preview accounts, including a proof-of-work challenge, Node.js solver example, and deployment instructions. It also adds a corresponding changelog entry.

1 suggestion:

  • MEDIUM — : The curl block uses brace placeholders (, , ). Per the style guide, API curl examples must use shell variable format ().

Nit (non-blocking):

  • Consider adding a prefix to the PR title to match commit conventions (e.g., ).

Comment thread src/content/docs/workers/platform/claim-deployments.mdx Outdated

@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.

This PR documents the REST API flow for temporary preview accounts, including a proof-of-work challenge, Node.js solver example, and deployment instructions. It also adds a corresponding changelog entry.

1 suggestion to address before merge:

  • MEDIUMclaim-deployments.mdx:275-277: The curl block uses brace placeholders ({account_id}, {script_name}, {api_token}). Per the style guide, API curl examples must use shell variable format ($VARIABLE_NAME).

Nit (non-blocking):

  • Consider adding a [Workers] prefix to the PR title to match commit conventions (for example, [Workers] Document temporary account REST API provisioning + changelog).

Comment thread src/content/docs/workers/platform/claim-deployments.mdx Outdated
@ask-bonk

ask-bonk Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Review complete for PR #31603.

Summary of changes
This PR adds a new "Use the REST API directly" section to the Claim deployments (temporary accounts) page, documenting how platforms can provision temporary preview accounts via the Cloudflare REST API without Wrangler. It includes a sequence diagram, proof-of-work challenge flow, a Node.js solver example, and API deployment instructions. It also adds a corresponding changelog entry under src/content/changelog/workers/2026-06-22-temporary-accounts-api.mdx.

Actions taken

  • Applied labels: documentation, product:workers
  • Posted a review comment with summary and issue list
  • Posted an inline suggestion on claim-deployments.mdx:275-277

Issues flagged

  • MEDIUMclaim-deployments.mdx:275-277: The curl example uses brace placeholders ({account_id}, {script_name}, {api_token}). The style guide requires shell variable format ($VARIABLE_NAME) in API curl blocks. I posted an inline suggestion to fix this.

  • LOW (nit) — PR title could use a [Workers] prefix to match commit conventions (for example, [Workers] Document temporary account REST API provisioning + changelog).

No build-breaking issues, broken links, or frontmatter errors were found. The changelog entry is well-written and follows the required format.

github run

@irvinebroque

Copy link
Copy Markdown
Contributor Author

/bonk address all feedback

Co-authored-by: irvinebroque <irvinebroque@users.noreply.github.com>
@ask-bonk

ask-bonk Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Done. Addressed all feedback on PR #31603:

  1. Shell variable format in curl example — Changed {account_id}, {script_name}, and {api_token} to $ACCOUNT_ID, $SCRIPT_NAME, and $CLOUDFLARE_API_TOKEN in the Workers Script Upload API curl block.

  2. Input validation in solvePreviewChallenge example — Added validation before the loop to match the documented bounds:

    • seed decodes to exactly 32 bytes
    • k and g are positive integers
    • k * g <= 64_000_000

All validation passed (pnpm run check, pnpm run lint, pnpm run format:core:check).

github run

@irvinebroque

Copy link
Copy Markdown
Contributor Author

/bonk make @dinasaur404 a codeowner of relevant files touched in this PR, so that moving forward she can approve. Please do this and make a new, separate PR for this change to codeowners

@ask-bonk

ask-bonk Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Done. I created a new branch (codeowners/add-dinasaur404-workers) and updated /.github/CODEOWNERS with two additions so @dinasaur404 can approve PRs touching the Workers paths affected in PR #31603:

  1. /src/content/changelog/workers/
    Covers the changelog/workers/ directory, which includes the new changelog entry (2026-06-22-temporary-accounts-api.mdx) added in PR Document how platforms can provision temporary accounts on behalf of users + changelog #31603.

  2. /src/content/docs/workers/platform/
    Covers the workers/platform/ docs directory, which includes the updated claim-deployments.mdx page.

The third file touched in PR #31603 (src/content/docs/cloudflare-for-platforms/index.mdx) was already covered by her existing ownership of /src/content/docs/cloudflare-for-platforms/.

github run

@irvinebroque

Copy link
Copy Markdown
Contributor Author

/bonk update changelog to tomorrow (Friday, Jun 26)

Co-authored-by: irvinebroque <irvinebroque@users.noreply.github.com>
@ask-bonk

ask-bonk Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Done. Updated the changelog entry in PR #31603:

  • Renamed 2026-06-22-temporary-accounts-api.mdx2026-06-26-temporary-accounts-api.mdx
  • Updated frontmatter date: from 2026-06-22 to 2026-06-26

No other files in the PR reference the old June 22 date, so this is the only change needed.

github run

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

Labels

documentation Documentation edits product:changelog product:workers Related to Workers product size/m

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants