Skip to content

fix: remove google namespace from packaged modules#327

Merged
tstirrat15 merged 1 commit intoauthzed:mainfrom
ivanauth:fix/issue-322-remove-google-namespace-from-package
Mar 30, 2026
Merged

fix: remove google namespace from packaged modules#327
tstirrat15 merged 1 commit intoauthzed:mainfrom
ivanauth:fix/issue-322-remove-google-namespace-from-package

Conversation

@ivanauth
Copy link
Copy Markdown
Contributor

@ivanauth ivanauth commented Mar 30, 2026

Summary

  • Removes "google" from the module-name list in pyproject.toml so google/api/ and google/rpc/ protobuf files are no longer bundled in the wheel
  • These files conflict with googleapis-common-protos (already a declared dependency), which provides the same modules at runtime

Commit 957ddf6 ("Use uv for dependency management and packaging") added "google" to the module-name list, causing the wheel to ship google/api/annotations_pb2.py, google/api/http_pb2.py, and google/rpc/status_pb2.py. These overlap with googleapis-common-protos, and concurrent installation can corrupt shared file paths.

Closes #322

@ivanauth ivanauth requested a review from a team as a code owner March 30, 2026 19:21
Copy link
Copy Markdown
Contributor

@tstirrat15 tstirrat15 left a comment

Choose a reason for hiding this comment

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

Ah, yes, this make sense.

@tstirrat15 tstirrat15 added this pull request to the merge queue Mar 30, 2026
Merged via the queue into authzed:main with commit a194c12 Mar 30, 2026
13 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

authzed-py installs extra files in "global" packages namespace

2 participants