GeezerVison

vm/src/stdlib/sysconfig.rs (2)

1-4: Avoid a redundant sysconfig::sysconfig path; use a conventional inner module name.

Align with existing pattern (module/decl) to improve clarity.

-pub(crate) use sysconfig::make_module;
-
-#[pymodule(name = "_sysconfig")]
-pub(crate) mod sysconfig {
+pub(crate) use module::make_module;
+
+#[pymodule(name = "_sysconfig")]
+pub(crate) mod module {

---

7-13: Return PyResult and drop unwrap for better error hygiene.

Matches common style in stdlib shims and keeps exceptions propagatable.

-    #[pyfunction]
-    fn config_vars(vm: &VirtualMachine) -> PyDictRef {
-        let vars = vm.ctx.new_dict();
-        vars.set_item("Py_GIL_DISABLED", false.to_pyobject(vm), vm)
-            .unwrap();
-        vars
-    }
+    use crate::PyResult;
+
+    #[pyfunction]
+    fn config_vars(vm: &VirtualMachine) -> PyResult<PyDictRef> {
+        let vars = vm.ctx.new_dict();
+        vars.set_item("Py_GIL_DISABLED", vm.ctx.new_bool(false), vm)?;
+        Ok(vars)
+    }

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.