Conversation
|
Important Review skippedIgnore keyword(s) in the title. ⛔ Ignored keywords (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
There was a problem hiding this comment.
Pull request overview
Release PR for SynthOrg v0.6.5, updating project version metadata and generated changelog entries to reflect the new release (per Release Please).
Changes:
- Bump library/package version references from
0.6.4to0.6.5 - Add the
0.6.5entry to.github/CHANGELOG.md - Update LICENSE “Change Date” to align with the new release date
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
src/synthorg/__init__.py |
Updates __version__ to 0.6.5. |
pyproject.toml |
Updates Commitizen version to 0.6.5 for tagging/release automation. |
LICENSE |
Adjusts BSL “Change Date” from April 7, 2029 to April 8, 2029 (consistent with the 2026-04-08 release date). |
.github/CHANGELOG.md |
Adds the autogenerated 0.6.5 release notes section. |
.github/.release-please-manifest.json |
Updates Release Please manifest version to 0.6.5. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Preview DeploymentURL: https://pr-1149.synthorg-pr-preview.pages.dev Built from commit |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches. Re-running this action after a short time may resolve the issue. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
There was a problem hiding this comment.
Code Review
This pull request updates the project version to 0.6.5 across the manifest, changelog, and configuration files. A critical Python syntax error was identified in the exception handling logic of the newly added communication hardening features, specifically regarding the incorrect syntax for catching multiple exceptions which is invalid in Python 3.
|
|
||
| ### Bug Fixes | ||
|
|
||
| * communication hardening -- meeting cooldown, circuit breaker backoff, debate fallback ([#1140](https://github.com/Aureliolo/synthorg/issues/1140)) ([fe82894](https://github.com/Aureliolo/synthorg/commit/fe82894feaadd6fde8c17e1b7a7c50c64fecb10d)), closes [#1115](https://github.com/Aureliolo/synthorg/issues/1115) [#1116](https://github.com/Aureliolo/synthorg/issues/1116) [#1117](https://github.com/Aureliolo/synthorg/issues/1117) |
There was a problem hiding this comment.
The implementation of the communication hardening features mentioned here contains a critical Python syntax error that will prevent the code from running. The except MemoryError, RecursionError: syntax used in several files is invalid in Python 3 (it results in a SyntaxError). Even in Python 2, it would catch MemoryError and assign it to a variable named RecursionError instead of catching both types. This issue is present in src/synthorg/communication/loop_prevention/circuit_breaker.py (lines 256 and 298) and src/synthorg/communication/conflict_resolution/debate_strategy.py (line 104). These should be updated to use a tuple: except (MemoryError, RecursionError):.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1149 +/- ##
=======================================
Coverage 88.88% 88.88%
=======================================
Files 857 857
Lines 50440 50440
Branches 5072 5072
=======================================
Hits 44835 44835
Misses 4643 4643
Partials 962 962 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
8e23d1c to
da54132
Compare
2b6fcb0 to
d2dd707
Compare
f27c3eb to
4c47bfc
Compare
644750a to
244564e
Compare
c507d2b to
b070e6f
Compare
f11e945 to
c4115de
Compare
1f87cf1 to
654a452
Compare
f5eb365 to
d8a5477
Compare
🤖 I have created a release beep boop
0.6.5 (2026-04-08)
Features
Bug Fixes
CI/CD
Maintenance
6869258to5e59aaein /docker/backend in the all group (#1141) (01e99c2)6869258to5e59aaein /docker/sandbox in the all group (#1143) (ea755bd)6869258to5e59aaein /docker/web in the all group (#1142) (5416dd9)This PR was generated with Release Please. See documentation.