Skip to content

Support DO_NOT_TRACK=1 - #14924

Merged
edmundhung merged 8 commits into
cloudflare:mainfrom
ariesclark:patch-1
Aug 11, 2026
Merged

Support DO_NOT_TRACK=1#14924
edmundhung merged 8 commits into
cloudflare:mainfrom
ariesclark:patch-1

Conversation

@ariesclark

@ariesclark ariesclark commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Adds support for the DO_NOT_TRACK environment variable in both Wrangler & create-cloudflare alongside their unique opt out environment variables.


  • 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: not documented previously

A picture of a cute animal (not mandatory, but encouraged)
PXL_20260728_123949722
A photo of a pretty bird I saw a couple days ago (does anyone know what bird it is?)


Open in Devin Review

@changeset-bot

changeset-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 3a0fce3

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

This PR includes changesets to release 10 packages
Name Type
create-cloudflare Minor
@cloudflare/workers-utils Minor
wrangler Minor
@cloudflare/autoconfig Patch
@cloudflare/cli-shared-helpers Patch
@cloudflare/deploy-helpers Patch
@cloudflare/remote-bindings Patch
@cloudflare/vite-plugin Patch
@cloudflare/vitest-pool-workers Patch
@cloudflare/workers-auth Patch

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

@workers-devprod
workers-devprod requested review from a team and edmundhung and removed request for a team July 29, 2026 22:16
@workers-devprod

workers-devprod commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@pkg-pr-new

pkg-pr-new Bot commented Jul 30, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

npm i https://pkg.pr.new/@cloudflare/autoconfig@14924

@cloudflare/build-output-utils

npm i https://pkg.pr.new/@cloudflare/build-output-utils@14924

@cloudflare/config

npm i https://pkg.pr.new/@cloudflare/config@14924

create-cloudflare

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

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@14924

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-functions

npm i https://pkg.pr.new/@cloudflare/pages-functions@14924

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@14924

@cloudflare/workers-editor-shared

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

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@14924

wrangler

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

commit: 3a0fce3

@edmundhung
edmundhung force-pushed the patch-1 branch 3 times, most recently from 1c98d0a to 569fefc Compare July 30, 2026 13:21
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@edmundhung edmundhung left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We like the idea. Thanks for the PR!

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

Codeowners reviews satisfied

@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Jul 30, 2026
@edmundhung
edmundhung force-pushed the patch-1 branch 2 times, most recently from 09c6c76 to b3a2221 Compare July 31, 2026 10:28
devin-ai-integration[bot]

This comment was marked as resolved.

@workers-devprod

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

devin-ai-integration[bot]

This comment was marked as resolved.

Copilot AI lite review requested due to automatic review settings August 11, 2026 09:51

Copilot AI 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.

Pull request overview

Adds cross-tool support for the DO_NOT_TRACK environment variable as a universal telemetry opt-out, aligning Wrangler and Create Cloudflare behavior while centralizing parsing/precedence logic in @cloudflare/workers-utils.

Changes:

  • Document DO_NOT_TRACK=1 telemetry opt-out (and precedence) in Wrangler and Create Cloudflare telemetry docs.
  • Add shared DO_NOT_TRACK parsing (isDoNotTrackEnabled()) and integrate it into Wrangler’s telemetry resolution path.
  • Update Create Cloudflare telemetry enablement/status resolution and add/extend tests in both packages.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/wrangler/telemetry.md Documents DO_NOT_TRACK=1 as a telemetry opt-out with highest precedence.
packages/wrangler/src/tests/metrics.test.ts Adds coverage ensuring DO_NOT_TRACK=1 disables telemetry and overrides other settings.
packages/workers-utils/src/environment-variables/misc-variables.ts Introduces isDoNotTrackEnabled() and applies it to Wrangler telemetry env resolution.
packages/workers-utils/src/environment-variables/factory.ts Adds DO_NOT_TRACK to the supported environment variable name union.
packages/create-cloudflare/telemetry.md Documents DO_NOT_TRACK=1 as a telemetry opt-out with highest precedence.
packages/create-cloudflare/src/metrics.ts Implements DO_NOT_TRACK-aware telemetry resolution and status reporting.
packages/create-cloudflare/src/tests/metrics.test.ts Adds tests for DO_NOT_TRACK behavior and status output; improves test cleanup.
.changeset/wrangler-do-not-track.md Releases Wrangler change as a minor feature.
.changeset/workers-utils-do-not-track.md Releases shared DO_NOT_TRACK utilities as a minor feature.
.changeset/nine-taxis-shave.md Releases Create Cloudflare change as a minor feature.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@workers-devprod

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@edmundhung
edmundhung merged commit 0aa8fa5 into cloudflare:main Aug 11, 2026
75 of 81 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Aug 11, 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.

4 participants