researchloop 0.3.3__tar.gz → 0.3.4__tar.gz

{researchloop-0.3.3 → researchloop-0.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: researchloop
-Version: 0.3.3
+Version: 0.3.4
 Summary: Automated research sprint platform for HPC clusters
 License: MIT
 License-File: LICENSE

{researchloop-0.3.3 → researchloop-0.3.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "researchloop"
-version = "0.3.3"
+version = "0.3.4"
 description = "Automated research sprint platform for HPC clusters"
 readme = "README.md"
 license = {text = "MIT"}

researchloop-0.3.4/researchloop/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.3.4"

{researchloop-0.3.3 → researchloop-0.3.4}/researchloop/runner/job_templates/sge.sh.j2

@@ -241,6 +241,17 @@ print(json.dumps({
 }
 
 _heartbeat_loop() {
+    # The parent script runs with `set -euo pipefail`, which is inherited
+    # by this backgrounded subshell. An unprotected command-substitution
+    # failure here (e.g. `ls -t pattern | head -1` racing into SIGPIPE on
+    # ls under pipefail, or a glob that matches nothing) is enough to kill
+    # the entire watchdog and leave the sprint without heartbeats or
+    # STUCK_PIPE detection for the rest of its wall-clock. Disable strict
+    # mode for the watchdog — a noisy heartbeat is far better than a
+    # silently-dead one.
+    set +e
+    set +o pipefail
+
     # Watchdog: detect and recover from hung pipelines.
     #
     # STUCK_PIPE warn: the active step's stream-json file goes silent for

{researchloop-0.3.3 → researchloop-0.3.4}/researchloop/runner/job_templates/slurm.sh.j2

@@ -243,6 +243,17 @@ print(json.dumps({
 }
 
 _heartbeat_loop() {
+    # The parent script runs with `set -euo pipefail`, which is inherited
+    # by this backgrounded subshell. An unprotected command-substitution
+    # failure here (e.g. `ls -t pattern | head -1` racing into SIGPIPE on
+    # ls under pipefail, or a glob that matches nothing) is enough to kill
+    # the entire watchdog and leave the sprint without heartbeats or
+    # STUCK_PIPE detection for the rest of its wall-clock. Disable strict
+    # mode for the watchdog — a noisy heartbeat is far better than a
+    # silently-dead one.
+    set +e
+    set +o pipefail
+
     # Watchdog: detect and recover from hung pipelines.
     #
     # STUCK_PIPE warn: the active step's stream-json file goes silent for

{researchloop-0.3.3 → researchloop-0.3.4}/tests/test_runner.py

@@ -176,6 +176,21 @@ class TestJobScriptWatchdog:
         # Once-per-stuck-episode flag, not log-every-heartbeat.
         assert "stuck_warned" in script
 
+    def _assert_heartbeat_loop_disables_strict_mode(self, script: str) -> None:
+        # The parent script has set -euo pipefail. The backgrounded
+        # _heartbeat_loop inherits it — without explicitly disabling errexit
+        # and pipefail, a single command-substitution failure (e.g. an
+        # ls|head SIGPIPE race) can silently kill the watchdog and the
+        # sprint loses all heartbeat + STUCK_PIPE detection for the rest of
+        # its wall-clock. The first two `set` lines inside the function
+        # must disable both modes.
+        loop_idx = script.index("_heartbeat_loop() {")
+        body_idx = script.index("\n", loop_idx) + 1
+        # Check the disabling lines are at the top of the function body.
+        head = script[body_idx : body_idx + 800]
+        assert "set +e" in head
+        assert "set +o pipefail" in head
+
     def _assert_hung_pipeline_recovery_present(self, script: str) -> None:
         # claude must run in its own session so the watchdog can SIGTERM the
         # whole group (claude + any leaked Bash-tool subprocesses) by pgid.
@@ -209,3 +224,13 @@ class TestJobScriptWatchdog:
 
     def test_sge_template_includes_hung_pipeline_recovery(self):
         self._assert_hung_pipeline_recovery_present(_render_job_template("sge.sh.j2"))
+
+    def test_slurm_heartbeat_loop_disables_strict_mode(self):
+        self._assert_heartbeat_loop_disables_strict_mode(
+            _render_job_template("slurm.sh.j2")
+        )
+
+    def test_sge_heartbeat_loop_disables_strict_mode(self):
+        self._assert_heartbeat_loop_disables_strict_mode(
+            _render_job_template("sge.sh.j2")
+        )

{researchloop-0.3.3 → researchloop-0.3.4}/uv.lock

@@ -1152,7 +1152,7 @@ wheels = [
 
 [[package]]
 name = "researchloop"
-version = "0.3.3"
+version = "0.3.4"
 source = { editable = "." }
 dependencies = [
     { name = "aiosqlite" },

researchloop-0.3.3/researchloop/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.3.3"