Replace the python package URL to R2

[ryanking13]

Replaces the Python fetch URL from GCS to R2. Note that our prod system uses a different path so this only affect workerd.

This is currently blocked by cloudflare/pyodide-build-scripts#14 (All 0.28.2 tests pass while 0.26.0a2 tests fail), hence opening as a draft. Done

I also removed WORKERD_INDEX_URL which I don't think we use anymore (no package loading in the runtime).

[ask-bonk]

UnknownError: "Internal server error"

github run

[ask-bonk]

@ryanking13 Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

[ryanking13]

/bonk try again

[ask-bonk]

Review Summary

The PR makes two changes: (1) switches the Python package fetch URL from GCS to R2, and (2) removes the now-unused WORKERD_INDEX_URL/indexURL parameter from the TS layer. Both are reasonable cleanup. A few items to address:

• [before-merge] py_wd_test.bzl has disk cache intentionally disabled with a TODO(before merge) comment — needs to be reverted before landing.
• [nit] The hardcoded indexURL = '' could use a clarifying comment explaining why it's empty now.
• The proxy.js simplification correctly removes hostnames that are no longer reachable after the URL migration.
• No remaining references to the old GCS URL or pyodide-packages.runtime-playground hostname exist in the codebase.

This review was generated by an AI assistant (ask-bonk) and may contain inaccuracies.

[ask-bonk]

This is marked TODO(before merge). Just flagging for visibility — make sure to revert the disk cache disablement before this lands. The commented-out lines (35-39) should be restored to:

Suggested change

	# args.append("../+pyodide+all_pyodide_wheels_%s" % pkg_tag)
	args = args + ["--pyodide-package-disk-cache-dir"]
	# +pyodide+ is a bzlmod canonical repository name
	args.append("../+pyodide+all_pyodide_wheels_%s" % pkg_tag)

[ryanking13]

Reverted after testing locally

[ask-bonk]

Review posted successfully on PR #6410. Here's a summary of what I flagged:

1. py_wd_test.bzl (before-merge TODO): The disk cache is intentionally disabled with a TODO(before merge) marker. Posted a suggested edit to restore the original lines when ready to land.

2. python.ts (nit): indexURL is now hardcoded to '' with no explanation. Suggested adding a comment clarifying that packages are loaded via ArtifactBundler in workerd mode, not Pyodide's URL-based loader.

3. proxy.js (clarification): The previous proxy forwarded package-related hostnames to env.INTERNET. After this change, only example.com is handled. Asked for confirmation this is intentional (it should be, since C++ handles package fetching, not the JS proxy).

The URL migration itself from GCS (storage.googleapis.com/cloudflare-edgeworker-python-packages/) to R2 (pyodide-capnp-bin.edgeworker.net/) looks correct — all references to the old URLs have been cleaned up, and the WORKERD_INDEX_URL removal is safe since packages are loaded entirely through the C++ ArtifactBundler layer, not Pyodide's built-in URL fetcher.

github run