chore: release 1.2.2 (durable JSONL-watcher reliability)#527
Conversation
Bumps the package to 1.2.2 so the durable-watcher fix (#525) is publishable to PyPI and installable via the homebrew-layers tap. PyPI 1.2.0 predates the fix; the existing v1.2.1 git tag was never published and predates it too. Version synced across pyproject.toml, src/brainlayer/__init__.py and server.json (test_release_version_sync green). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FqgSTrcnkHocz2rfLrveXK
Follow-on to the queue_dir isolation: tests that build HealthCheckConfig without overriding pause_sentinel_path inherited the developer's real ~/.local/share/brainlayer/pause.sentinel. A live `brainlayer pause` sentinel (e.g. created during an operational pause) then suppressed `watcher_stalled` and broke test_health_check_reports_watcher_stalled_drain_no_progress_and_queue_backed_up. Extend the autouse fixture to redirect the live default sentinel path to an absent isolated file via _pause_sentinel_state, alongside the existing queue-stats redirect. Tests passing explicit paths are unaffected. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FqgSTrcnkHocz2rfLrveXK
There was a problem hiding this comment.
Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.
📝 WalkthroughWalkthroughVersion bumped from ChangesRelease 1.2.2
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
|
Closing as obsolete. The release/version-bump purpose (1.2.2) is long superseded — main is at 1.4.2 and the #525 watcher fix already shipped in v1.3.0+. The one still-useful piece — the pause-sentinel test isolation from commit 3791e81 — was salvaged and merged in #557 (ported into the autouse fixture in tests/test_stability_health_check.py). The version bumps (pyproject.toml/init.py/server.json) were intentionally discarded. Superseded by #557. |
Bumps the package to 1.2.2 so the durable JSONL-watcher fix (#525) is publishable to PyPI and installable via the
homebrew-layerstap.Why a new version
1.2.0predates the watcher fix.v1.2.1git tag was created on code wherepyproject.tomlstill read1.2.0, so its publish workflow builtbrainlayer-1.2.0(already on PyPI) and failed —v1.2.1was never published.pyproject.toml,src/brainlayer/__init__.py, andserver.json(bothversionandpackages[0].version) to1.2.2sotest_release_version_syncpasses and thePublish to PyPIworkflow can build+upload cleanly on thev1.2.2tag.Also included
A test-isolation fix in
test_stability_health_check.py: the autouse fixture now redirects the defaultpause_sentinel_path(and, as before,queue_dir) away from live developer state. Without it, a realbrainlayer pausesentinel on the dev machine suppresseswatcher_stalledand breakstest_health_check_reports_watcher_stalled_drain_no_progress_and_queue_backed_up. Full unit gate green (3042 passed).🤖 Generated with Claude Code
Note
Low Risk
Version-metadata and test-only changes; no production runtime behavior in this diff.
Overview
Release 1.2.2 — bumps
versionfrom1.2.0to1.2.2inpyproject.toml,src/brainlayer/__init__.py, andserver.json(top-level andpackages[0].version) so release/version-sync checks and the PyPI publish workflow can ship the durable JSONL-watcher fix that was not cleanly publishable on prior tags.Test isolation — renames and broadens the autouse fixture in
test_stability_health_check.py: it still redirects the default~/.brainlayer/queuefor_queue_stats, and now also swaps the default~/.local/share/brainlayer/pause.sentinelto an absent temp path via_pause_sentinel_state, preventing a realbrainlayer pauseon a dev machine from suppressingwatcher_stalledin tests that rely on defaultHealthCheckConfigpaths.Reviewed by Cursor Bugbot for commit 3791e81. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Release version 1.2.2 with improved health check test isolation
__version__pause_sentinel_pathby monkeypatching_pause_sentinel_state, preventing tests from touching~/.local/share/brainlayer/pause.sentinelMacroscope summarized 3791e81.
Summary by CodeRabbit