Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughThis PR introduces lazy module registration for D3 sessions, implements AST-based automatic import detection for d3functions, renames API execution functions for clarity, and expands CI testing across Python 3.11–3.13. GitHub Actions workflows for release and test publishing are also added. Changes
Sequence Diagram(s)sequenceDiagram
actor User
participant Session as D3Session<br/>.execute()
participant Registry as D3Function<br/>._available_packages
participant API as d3_api_execute<br/>(payload)
participant Designer as Designer<br/>Instance
User->>Session: execute(payload with moduleName)
activate Session
alt moduleName not in registered_modules
Session->>Registry: Check if module exists
Registry-->>Session: Module found
Session->>API: d3_api_register_module(moduleName)
activate API
API->>Designer: Send registration
Designer-->>API: Acknowledge
deactivate API
Session->>Session: Add to registered_modules
else moduleName already registered
Session->>Session: Skip registration
end
Session->>API: d3_api_execute(payload)
activate API
API->>Designer: Execute payload
Designer-->>API: Result
deactivate API
API-->>Session: Response
deactivate Session
Session-->>User: Return result
Estimated code review effort🎯 4 (Complex) | ⏱️ ~50 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
e489922 to
f5e8a07
Compare
add tag on release restrict action to main
f5e8a07 to
53ffa96
Compare
1e5f6bd to
d7ed34e
Compare
Add release action
Summary
Adds a manual GitHub Actions workflow to publish the designer-plugin package to PyPI.
Summary by CodeRabbit
New Features
execute()orrpc()call (lazy registration).API Changes
d3_api_plugin→d3_api_execute,d3_api_aplugin→d3_api_aexecute.context_modulesparameter type changed to set.add_packages_in_current_file()helper.Documentation