Skip to content

Add-preview-version-output#13197

Merged
shahsimpson merged 1 commit intocloudflare:mainfrom
shahsimpson:add-preview-version-output
Apr 10, 2026
Merged

Add-preview-version-output#13197
shahsimpson merged 1 commit intocloudflare:mainfrom
shahsimpson:add-preview-version-output

Conversation

@shahsimpson
Copy link
Copy Markdown
Contributor

@shahsimpson shahsimpson commented Apr 1, 2026

Fixes BANDA-1738

Building atop #12983

This change adds a preview output entry to Wrangler’s output file stream, so wrangler preview writes structured preview metadata (preview id/name/slug/urls) alongside its linked deployment metadata (deployment id/urls).
The goal is to make output-file consumers (Builds) able to reliably read both preview and deployment URLs from JSONL output, without scraping CLI text output.


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: Documentation not necessary because: this updates Wrangler output-file payload behavior and test coverage, without introducing new user-facing commands or configuration.

IMG_8496


Open with Devin

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 1, 2026

🦋 Changeset detected

Latest commit: 874e0e5

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

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

@github-project-automation github-project-automation bot moved this to Untriaged in workers-sdk Apr 1, 2026
@workers-devprod workers-devprod requested review from a team and petebacondarwin and removed request for a team April 1, 2026 16:02
@workers-devprod
Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • @cloudflare/wrangler
Show detailed file reviewers
  • packages/workers-utils/src/config/config.ts: [@cloudflare/wrangler]
  • packages/workers-utils/src/config/environment.ts: [@cloudflare/wrangler]
  • packages/workers-utils/src/config/index.ts: [@cloudflare/wrangler]
  • packages/workers-utils/src/config/validation.ts: [@cloudflare/wrangler]
  • packages/workers-utils/tests/config/validation/normalize-and-validate-config.test.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/tests/index.test.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/tests/output.test.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/tests/preview.secret.test.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/tests/preview.settings.test.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/tests/preview.test.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/index.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/output.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/preview/api.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/preview/index.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/preview/preview.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/preview/secret.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/preview/settings.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/preview/shared.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/utils/box.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/utils/merge-deep.ts: [@cloudflare/wrangler]
  • packages/wrangler/turbo.json: [@cloudflare/wrangler]

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 1, 2026

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@13197

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@13197

miniflare

npm i https://pkg.pr.new/miniflare@13197

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@13197

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@13197

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@13197

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@13197

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@13197

wrangler

npm i https://pkg.pr.new/wrangler@13197

commit: 874e0e5

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 7 additional findings.

Open in Devin Review

@shahsimpson shahsimpson force-pushed the add-preview-version-output branch from 201a31e to 2ff0e41 Compare April 1, 2026 16:12
@shahsimpson shahsimpson marked this pull request as draft April 1, 2026 16:16
devin-ai-integration[bot]

This comment was marked as resolved.

@petebacondarwin petebacondarwin removed their request for review April 1, 2026 17:28
@shahsimpson shahsimpson force-pushed the add-preview-version-output branch 2 times, most recently from acda453 to 995b162 Compare April 1, 2026 18:52
@shahsimpson shahsimpson force-pushed the add-preview-version-output branch 3 times, most recently from f0af01a to dae721f Compare April 9, 2026 18:32
@shahsimpson shahsimpson marked this pull request as ready for review April 9, 2026 19:06
@workers-devprod workers-devprod requested review from a team and NuroDev and removed request for a team April 9, 2026 19:07
@workers-devprod
Copy link
Copy Markdown
Contributor

workers-devprod commented Apr 9, 2026

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@shahsimpson shahsimpson force-pushed the add-preview-version-output branch from dae721f to 2da6a00 Compare April 9, 2026 20:14
@shahsimpson shahsimpson force-pushed the add-preview-version-output branch from 2da6a00 to 874e0e5 Compare April 9, 2026 21:59
Copy link
Copy Markdown
Member

@dario-piotrowicz dario-piotrowicz left a comment

Choose a reason for hiding this comment

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

LGTM 🙂

Copy link
Copy Markdown
Contributor

@workers-devprod workers-devprod left a comment

Choose a reason for hiding this comment

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

Codeowners reviews satisfied

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Apr 10, 2026
@shahsimpson shahsimpson merged commit 4fd138b into cloudflare:main Apr 10, 2026
50 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Apr 10, 2026
petebacondarwin pushed a commit that referenced this pull request Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants