Bradley/r2 event notification get

[bthwaites]

What this PR solves / how to test

• Updates the expected R2 Event Notification GET payload from the Cloudflare API.
• Updates the R2 action types that are included in the object-delete event type.
• Updates the format of the table of rules that is displayed after running wrangler r2 bucket notification get

Note: The change to the GET payload is not backwards compatible. Users will need to update to the latest version of wrangler to continue using Event Notifications after the GA launch. Backwards compatibility is added here for the case that we need to revert the API changes.

Fixes #[insert GH or internal issue number(s)].
Internal issues: R2-2379 and R2-2380

Author has addressed the following

• Tests
  • TODO (before merge)
  • Included
  • Not necessary because:
• E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)
  • I don't know
  • Required / Maybe required
  • Not required because: there are no e2e tests for event notifications
• Changeset (Changeset guidelines)
  • TODO (before merge)
  • Included
  • Not necessary because:
• Public documentation
  • TODO (before merge)
  • Cloudflare docs PR(s):
  • Not necessary because: Covered by existing docs (https://developers.cloudflare.com/workers/wrangler/commands/#notification-get)

[changeset-bot]

🦋 Changeset detected

Latest commit: fc10687

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

This PR includes changesets to release 2 packages

Name	Type
wrangler	Minor
@cloudflare/vitest-pool-workers	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

[github-actions]

A wrangler prerelease is available for testing. You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/10817376489/npm-package-wrangler-6652

You can reference the automatically updated head of this PR with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/6652/npm-package-wrangler-6652

Or you can use npx with this latest build directly:

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/10817376489/npm-package-wrangler-6652 dev path/to/script.js

Additional artifacts:

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/10817376489/npm-package-create-cloudflare-6652 --no-auto-update

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/10817376489/npm-package-cloudflare-kv-asset-handler-6652

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/10817376489/npm-package-miniflare-6652

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/10817376489/npm-package-cloudflare-pages-shared-6652

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/10817376489/npm-package-cloudflare-vitest-pool-workers-6652

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/10817376489/npm-package-cloudflare-workers-editor-shared-6652

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/10817376489/npm-package-cloudflare-workers-shared-6652

Note that these links will no longer work once the GitHub Actions artifact expires.

---

wrangler@3.76.0 includes the following runtime dependencies:

Package	Constraint	Resolved
miniflare	workspace:*	3.20240821.2
workerd	1.20240909.0	1.20240909.0
workerd --version	1.20240909.0	2024-09-09

Please ensure constraints are pinned, and miniflare/workerd minor versions match.

[RamIdeas]

If the event notification feature is currently not GA, this can be a minor/patch bump

This PR cannot be merged while this is a major bump

[RamIdeas]

It seems this subcommand is not marked as experimental in the subhelp output:

$ npx wrangler r2 bucket --help             
wrangler r2 bucket

Manage R2 buckets

COMMANDS
  wrangler r2 bucket create <name>  Create a new R2 bucket
  wrangler r2 bucket update         Update bucket state
  wrangler r2 bucket list           List R2 buckets
  wrangler r2 bucket delete <name>  Delete an R2 bucket
  wrangler r2 bucket sippy          Manage Sippy incremental migration on an R2 bucket
  wrangler r2 bucket notification   Manage event notifications for an R2 bucket

GLOBAL FLAGS
  -j, --experimental-json-config  Experimental: support wrangler.json  [boolean]
  -c, --config                    Path to .toml configuration file  [string]
  -e, --env                       Environment to use for operations and .env files  [string]
  -h, --help                      Show help  [boolean]
  -v, --version                   Show version number  [boolean]

Can you add ${chalk.hex(betaCmdColor)("[open beta]")} to the end of the subcommand description please?

[RamIdeas]

This table is getting quite wide. Would it make sense to switch to a labelled list? (like deployments list)

Easily done with the formatLabelledValues util

[bthwaites]

Yes I like this suggestion, I'll check it out.

[RamIdeas]

To render as a labelled list instead:

Suggested change

	logger.table(tableOutput);
	logger.log(tableOutput.map((x) => formatLabelledValues(x)).join("\n\n"));