Configurable rules for exclusions in bundling of python_modules

[dom96]

Implements a new python_modules_excludes wrangler config field. The user can specify a list of globs in that field to exclude them from being vendored. By default every file in python_modules is included but for certain packages they contain so much stuff that sometimes it's good to be able to filter it out, especially since our bundle sizes are limited.

---

• Tests
  • Tests included
  • Tests not necessary because:
• Public documentation
  • Cloudflare docs PR(s): Adds docs for python_modules_excludes. cloudflare-docs#25725
  • Documentation not necessary because:
• Wrangler V3 Backport
  • Wrangler PR:
  • Not necessary because: new feature

---

Test Plan

pnpm run test --filter wrangler -- -t "should print vendor modules correctly in table" src/__tests__/deploy.test.ts

cd packages/wrangler/
pnpm run build
pnpm test:e2e --run --testNamePattern="can exclude vendored module during"

[changeset-bot]

🦋 Changeset detected

Latest commit: fbade0e

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

[pkg-pr-new]

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

wrangler

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

commit: fbade0e

[hoodmane]

Why do you mix console.log() and print()? Is there a reason not to choose one and stick with it?

[hoodmane]

Suggested change

	except:
	except ImportError:

[hoodmane]

Probably we will eventually want to exclude these by default. But that can be left to followup work.

[hoodmane]

Looks very reasonable to me. Thanks @dom96!