Skip to content

Conversation

@tiwilliam
Copy link
Owner

Upgrade PyO3 to version 0.26 and adapt the API to resolve a build error and support newer Python versions.

The _PyInterpreterFrame error indicated an incompatibility with the older PyO3 version and the current Python environment. Upgrading PyO3 resolved this by providing a compatible pyo3-ffi and required corresponding API adjustments in src/lib.rs.


Open in Cursor Open in Web

@cursor
Copy link

cursor bot commented Oct 19, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@tiwilliam tiwilliam marked this pull request as ready for review October 19, 2025 10:33
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on November 26

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

m.add_wrapped(exceptions)?;
let exc_py = exceptions(py);
let exc_bound = exc_py.bind(py);
m.add_submodule(&exc_bound)?;
Copy link

Choose a reason for hiding this comment

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

Bug: Duplicate Module Instantiation Causes Identity Issues

The exceptions module is instantiated twice: once for sys.modules registration and again when added as a submodule. This creates two separate module instances, which can lead to inconsistencies or identity issues if Python code expects a single module object.

Fix in Cursor Fix in Web

@tiwilliam tiwilliam merged commit 595080d into main Oct 19, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants