codna 0.2.71__tar.gz → 0.2.72__tar.gz

{codna-0.2.71 → codna-0.2.72}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codna
-Version: 0.2.71
+Version: 0.2.72
 Summary: Local-first repo intelligence and agent fixing for Codna.
 Author: Thyn
 Maintainer: Thyn

{codna-0.2.71 → codna-0.2.72}/codna/__init__.py

@@ -5,4 +5,4 @@ This package is the `codna` CLI. Users never touch "algenta" directly — the en
 is an internal dependency reached over HTTP.
 """
 
-__version__ = "0.2.71"
+__version__ = "0.2.72"

{codna-0.2.71 → codna-0.2.72}/codna.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codna
-Version: 0.2.71
+Version: 0.2.72
 Summary: Local-first repo intelligence and agent fixing for Codna.
 Author: Thyn
 Maintainer: Thyn

{codna-0.2.71 → codna-0.2.72}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "codna"
-version = "0.2.71"
+version = "0.2.72"
 description = "Local-first repo intelligence and agent fixing for Codna."
 readme = "README.md"
 requires-python = ">=3.10"

{codna-0.2.71 → codna-0.2.72}/tests/test_workflows.py

@@ -456,3 +456,78 @@ def test_webhook_ops_flags_on_the_worker_pool_are_staged_and_on_the_ingress_depl
     assert run("codna-webhook-worker", "unset", "CODNA_WEBHOOK_DRAIN_S") == ["secrets", "unset", "-a", "codna-webhook-worker", "--stage", "CODNA_WEBHOOK_DRAIN_S"]
     assert run("codna-webhook", "set", "CODNA_WEBHOOK_QUEUE_BACKEND=postgres CODNA_WEBHOOK_ROLE=ingress") == \
         ["secrets", "set", "-a", "codna-webhook", "CODNA_WEBHOOK_QUEUE_BACKEND=postgres", "CODNA_WEBHOOK_ROLE=ingress"]
+
+
+def _modern_bash() -> str:
+    """The step uses `mapfile` (bash 4+, the runner's bash 5). macOS ships bash 3.2, so prefer a newer
+    bash when one is installed and skip otherwise -- CI is the meaningful run."""
+    import shutil
+    import subprocess
+
+    for candidate in ("/opt/homebrew/bin/bash", "/usr/local/bin/bash", shutil.which("bash") or "bash"):
+        try:
+            major = subprocess.run([candidate, "-c", "echo ${BASH_VERSINFO[0]}"], capture_output=True, text=True, check=False).stdout.strip()
+        except OSError:
+            continue
+        if major.isdigit() and int(major) >= 4:
+            return candidate
+    pytest.skip("no bash >= 4 available for mapfile (the ubuntu runner has bash 5)")
+
+
+def _run_worker_start_stop_step(tmp_path, mode: str, count: str, machines: list[dict]):
+    """Run the worker-start/worker-stop step against a stub flyctl: `machine list --json` returns the
+    fixture, every other `machine ...` call is recorded. Returns (rc, recorded argv lines, output)."""
+    import json
+    import os
+    import stat
+    import subprocess
+
+    stub_dir = tmp_path / "bin"
+    stub_dir.mkdir(exist_ok=True)
+    record = tmp_path / "machine.argv"
+    record.unlink(missing_ok=True)
+    (tmp_path / "machines.json").write_text(json.dumps(machines))
+    stub = stub_dir / "flyctl"
+    stub.write_text(
+        "#!/usr/bin/env bash\n"
+        "if [ \"$1\" = machine ] && [ \"$2\" = list ]; then case \" $* \" in *\" --json \"*) cat \"$FLY_STUB_MACHINES\" ;; *) echo table ;; esac; exit 0; fi\n"
+        "printf '%s\\n' \"$*\" >> \"$FLY_STUB_RECORD\"\n"
+    )
+    stub.chmod(stub.stat().st_mode | stat.S_IEXEC)
+    proc = subprocess.run([_modern_bash(), "-c", _ops_step("worker-start")["run"]],
+                          env={**os.environ, "APP": "codna-webhook", "WORKER_APP": "codna-webhook-worker", "COUNT": count,
+                               "MODE": mode, "PATH": f"{stub_dir}:{os.environ['PATH']}",
+                               "FLY_STUB_MACHINES": str(tmp_path / "machines.json"), "FLY_STUB_RECORD": str(record)},
+                          capture_output=True, text=True, check=False)
+    return proc.returncode, (record.read_text().splitlines() if record.exists() else []), proc.stdout + proc.stderr
+
+
+_POOL = [
+    {"id": "m2", "name": "worker-2", "state": "stopped", "region": "iad"},
+    {"id": "m0", "name": "worker-0", "state": "stopped", "region": "iad"},
+    {"id": "m3", "name": "worker-3", "state": "created", "region": "ord"},
+    {"id": "m1", "name": "worker-1", "state": "stopped", "region": "ord"},
+]
+
+
+def test_webhook_ops_worker_start_updates_machines_so_staged_secrets_apply(tmp_path):
+    # `machine update` pins the machine to the app's current secrets version and starts it; a plain
+    # `machine start` reused the old (empty) secrets and both floor machines exited (run 35505335304)
+    rc, argv, out = _run_worker_start_stop_step(tmp_path, "start", "2", _POOL)
+    assert rc == 0, out
+    assert argv == ["machine update m0 -a codna-webhook-worker --yes", "machine update m1 -a codna-webhook-worker --yes"]
+    assert "flyctl machine start" not in _ops_step("worker-start")["run"]
+
+
+def test_webhook_ops_worker_stop_takes_the_highest_names_first_and_keeps_the_floor(tmp_path):
+    running = [dict(m, state="started") for m in _POOL]
+    rc, argv, out = _run_worker_start_stop_step(tmp_path, "stop", "2", running)
+    assert rc == 0, out
+    assert argv == ["machine stop m3 -a codna-webhook-worker", "machine stop m2 -a codna-webhook-worker"]
+    rc, argv, out = _run_worker_start_stop_step(tmp_path, "stop", "4", running)
+    assert rc == 0 and argv == [] and "nothing to stop" in out
+
+
+def test_webhook_ops_logs_action_reads_either_app():
+    run = _ops_step("logs")["run"]
+    assert 'flyctl logs -a "$TARGET"' in run and "codna-webhook|codna-webhook-worker" in run