gc is vm/stdlib#6929

youknowone

Summary by CodeRabbit

New Features
- Added bz2 and compression modules to the standard library, expanding data compression capabilities.
Chores
- Refactored internal module organization for improved code structure and maintainability. Updated visibility policies to align with architectural best practices.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Adds internal bz2 and compression modules to stdlib, removes gc from that stdlib declaration, registers a private gc module in crates/vm/src/stdlib, and narrows PyObject::gc_finalized visibility to crate-only; also updates import paths in the gc stdlib file.

Changes

Cohort / File(s)	Summary
Stdlib core modules `crates/stdlib/src/lib.rs`	Adds `mod bz2;` and `mod compression;` (compression marked internal); removes previous `gc` declaration from this file and adds `bz2::module_def(ctx)` and `compression::module_def(ctx)` to `stdlib_module_defs`.
VM stdlib registration `crates/vm/src/stdlib/mod.rs`	Adds private `mod gc;` and registers `gc::module_def(ctx)` in `builtin_module_defs`.
GC implementation changes `crates/vm/src/stdlib/gc.rs`	Updates imports to use direct crate-root types (`PyObjectRef`, `PyResult`, `VirtualMachine`, `builtins::PyListRef`, `function::{FuncArgs, OptionalArg}`) and removes a comment in `is_finalized` (function now directly returns `obj.gc_finalized()`).
API visibility `crates/vm/src/object/core.rs`	Changes `PyObject::gc_finalized` visibility from `pub fn` to `pub(crate) fn`, restricting access to the crate.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

gc module internal structure and API #6910: Modifies VM gc integration and gc stdlib registration—closely related to the gc module registration changes here.
impl gc finialized #6689: Adds/adjusts gc_finalized API and GC bit mechanics that intersect with the visibility change to gc_finalized.

Suggested reviewers

ShaharNaveh

Poem

🐰 I hopped through modules, tidy and spry,

Added bz2, compression — gave gc a new sky.
A quieter finale, visibility small,
The rabbit applauds changes, big and small. 🥕✨

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

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.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

github-actions

Code has been automatically formatted

The code in this PR has been formatted using:

cargo fmt --all
Please pull the latest changes before pushing again:

git pull origin gc-location

gc is vm/stdlib

6c4beb0

Auto-format: cargo fmt --all

1b6548d

youknowone marked this pull request as ready for review February 1, 2026 04:21

Merge branch 'main' into gc-location

5c35499

youknowone deleted the gc-location branch February 1, 2026 04:21

This was referenced Feb 26, 2026

Update tarfile from v3.14.3 and fix bugs #7223

Merged

Replace GC tracking HashSet with intrusive linked list #7328

Merged

coderabbitai Bot mentioned this pull request Mar 12, 2026

If the host_env feature is enabled, then enable the _ctypes module on the Android platform. #7402

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

gc is vm/stdlib#6929