| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info ⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Plus Run ID: 547176e7-0939-4c94-a62a-5a060c18f029 📥 CommitsReviewing files that changed from the base of the PR and between 48a3a1f and fd9cf58. ⛔ Files ignored due to path filters (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 Walkthrough WalkthroughThe change stores PyType.bases as a typed Python tuple. It adds typed tuple construction and updates type validation, traversal, MRO handling, and bootstrap initialization. The type zoo and VM context now share the bootstrapped tuple type and empty tuple. ChangesTyped tuple bases and bootstrap hierarchy
Estimated code review effort: 4 (Complex) | ~45 minutes Merge Risk: ⚪ Minimal · up to fd9cf This change preserves tuple subclasses passed as type bases, and no actionable merge-blocking risk remains after normal checks and review. Sequence Diagram(s)sequenceDiagram
participant ContextInit
participant TypeZooInit
participant InitTypeHierarchy
participant CoreTypes
ContextInit->>TypeZooInit: initialize types and empty_tuple
TypeZooInit->>InitTypeHierarchy: create BootstrapTypeHierarchy
InitTypeHierarchy->>CoreTypes: allocate and initialize core types and tuples
InitTypeHierarchy-->>TypeZooInit: return hierarchy and empty_tuple
TypeZooInit-->>ContextInit: return TypeZoo and empty_tuple
Suggested reviewers: youknowone 🚥 Pre-merge checks | ✅ 5 ✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. ❤️ ShareComment @coderabbitai help to get the list of available commands. |
Sorry, something went wrong.
📦 Library DependenciesThe following Lib/ modules were modified. Here are their dependencies: [x] lib: cpython/Lib/types.py dependencies:
dependent tests: (57 tests)
Legend:
|
Sorry, something went wrong.
Store validated base classes in the original typed tuple, while retaining a bootstrap vector only until a Python tuple can be materialized. Assisted-by: gpt-5.6-sol
Initialize tuple and the canonical empty tuple alongside type and object so every type stores an actual typed bases tuple from construction. Remove the bootstrap bases representation and keep raw initialization Miri-safe. Assisted-by: Codex:gpt-5.6-sol
| Back | FazBrowse Home | New Git URL |
Summary
type.__bases__ always synthesized a fresh plain tuple from an internal Vec<PyTypeRef>, so a tuple subclass passed as a base class lost its identity (CPython gh-132176). test_tuple_subclass_as_bases no longer needs @unittest.expectedFailure.
Written with Claude Code
Summary by CodeRabbit