sase-github 0.2.7__tar.gz → 0.2.9__tar.gz

sase_github-0.2.9/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "0.2.9"
+}

{sase_github-0.2.7 → sase_github-0.2.9}/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [0.2.9](https://github.com/sase-org/sase-github/compare/v0.2.8...v0.2.9) (2026-08-29)
+
+
+### Features
+
+* **workspace:** identity-check #gh release and skip on handoff ([2571c9d](https://github.com/sase-org/sase-github/commit/2571c9d7466d4c6020a87b6ee86068c12170cdab))
+
+## [0.2.8](https://github.com/sase-org/sase-github/compare/v0.2.7...v0.2.8) (2026-08-29)
+
+
+### Features
+
+* **gh:** mark runner-bound workspace allocations ([f4663ae](https://github.com/sase-org/sase-github/commit/f4663ae526929b23378372005662c20b495bd0f7))
+
 ## [0.2.7](https://github.com/sase-org/sase-github/compare/v0.2.6...v0.2.7) (2026-08-28)
 
 

{sase_github-0.2.7 → sase_github-0.2.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: sase-github
-Version: 0.2.7
+Version: 0.2.9
 Summary: GitHub VCS plugin for sase
 License-Expression: MIT
 License-File: LICENSE

{sase_github-0.2.7 → sase_github-0.2.9}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "sase-github"
-version = "0.2.7"
+version = "0.2.9"
 description = "GitHub VCS plugin for sase"
 requires-python = ">=3.12"
 license = "MIT"

sase_github-0.2.9/src/sase_github/scripts/gh_release.py

@@ -0,0 +1,27 @@
+"""Release step for the #gh xprompt workflow."""
+
+from sase.workspace_provider.vcs_release import VcsReleaseResult, release_vcs_workspace
+
+_CALLER_TAG = "gh-release"
+
+
+def main(
+    *,
+    project_file: str,
+    workspace_num: int,
+    workspace_dir: str,
+    workflow_name: str,
+    cl_name: str | None,
+) -> None:
+    """Identity-checked, handoff-aware release of the #gh workspace claim."""
+    result: VcsReleaseResult = release_vcs_workspace(
+        project_file=project_file,
+        workspace_num=workspace_num,
+        workspace_dir=workspace_dir,
+        workflow_name=workflow_name,
+        cl_name=cl_name,
+        caller_tag=_CALLER_TAG,
+    )
+    print(f"released={'true' if result.released else 'false'}")
+    if result.skip_reason:
+        print(f"skip_reason={result.skip_reason}")

{sase_github-0.2.7 → sase_github-0.2.9}/src/sase_github/scripts/gh_setup.py

@@ -11,6 +11,7 @@ from sase.running_field import (
     find_runner_numbered_workspace,
     get_claimed_workspaces,
     release_workspace,
+    runner_has_placeholder_workspace,
 )
 from sase.sdd.store import materialize_sdd_store
 from sase.workspace_provider import resolve_ref
@@ -44,6 +45,7 @@ def main(
     # then incorrectly remove the RUNNING entry while the real workflow
     # (the parent process) is still alive.
     pid = os.getppid()
+    runner_bound_workspace = False
 
     # Skip claiming when pre-allocated — the launcher already claimed the workspace
     pre_allocated = os.environ.get("SASE_GH_PRE_ALLOCATED") == "1"
@@ -70,6 +72,9 @@ def main(
             workspace_num, workspace_dir = adopted
             pre_allocated = True
         else:
+            runner_bound_workspace = runner_has_placeholder_workspace(
+                project_file, pid=pid
+            )
             workspace_num, workspace_dir = _claim_next_workspace(
                 project_file=project_file,
                 primary_workspace_dir=resolved.primary_workspace_dir,
@@ -131,8 +136,9 @@ def main(
     print(f"checkout_target={resolved.checkout_target}")
     print(f"primary_workspace_dir={resolved.primary_workspace_dir}")
     # Don't release pre-allocated workspaces — the launcher handles that
-    should_release = release and not pre_allocated
+    should_release = release and not pre_allocated and not runner_bound_workspace
     print(f"should_release={'true' if should_release else 'false'}")
+    print(f"runner_bound_workspace={'true' if runner_bound_workspace else 'false'}")
     print(f"_chdir={workspace_dir}")
     print(f"meta_workspace={workspace_num}")
     print(f"workflow_name={workflow_name}")

{sase_github-0.2.7 → sase_github-0.2.9}/src/sase_github/xprompts/gh.yml

@@ -37,6 +37,7 @@ steps:
       checkout_target: word
       primary_workspace_dir: path
       should_release: bool
+      runner_bound_workspace: bool
       workflow_name: word
 
   - name: prepare
@@ -119,18 +120,15 @@ steps:
     finally: true
     if: "{{ setup.should_release }}"
     python: |
-      from sase.running_field import release_workspace
-      from sase.workspace_provider.occupant import clear_occupant_record
+      from sase_github.scripts.gh_release import main
 
-      release_workspace(
-          {{ setup.project_file | tojson }},
-          {{ setup.workspace_num }},
-          {{ setup.workflow_name | tojson }},
-          {{ gh_ref | tojson }} if "/" not in {{ gh_ref | tojson }} else None,
-          caller_tag="gh-release",
+      main(
+          project_file={{ setup.project_file | tojson }},
+          workspace_num={{ setup.workspace_num }},
+          workspace_dir={{ setup.workspace_dir | tojson }},
+          workflow_name={{ setup.workflow_name | tojson }},
+          cl_name={{ gh_ref | tojson }} if "/" not in {{ gh_ref | tojson }} else None,
       )
-      clear_occupant_record({{ setup.workspace_dir | tojson }})
-      print("released=true")
     output: { released: bool }
 
   - name: diff

{sase_github-0.2.7 → sase_github-0.2.9}/tests/test_gh_workspace_claims.py

@@ -23,7 +23,7 @@ from sase.workspace_provider.occupant import (
     write_occupant_record,
 )
 
-from sase_github.scripts import gh_setup
+from sase_github.scripts import gh_release, gh_setup
 from sase_github.workspace_plugin import GitHubWorkspacePlugin
 
 ROOT = Path(__file__).resolve().parents[1]
@@ -608,7 +608,7 @@ class TestGhSetupAdoptsRunnerWorkspace:
         assert claims[0].pid == os.getppid()
         assert read_occupant_record(str(workspace_dir)) is None
 
-    def test_does_not_adopt_when_parent_only_holds_placeholder(
+    def test_parent_placeholder_allocation_is_runner_bound(
         self, tmp_path: Path, capsys: pytest.CaptureFixture[str]
     ) -> None:
         placeholder = WorkspaceClaim(
@@ -635,9 +635,13 @@ class TestGhSetupAdoptsRunnerWorkspace:
             claim.workspace_num for claim in get_claimed_workspaces(project_file)
         }
         assert claimed_nums == {0, 10}
+        claims = get_claimed_workspaces(project_file)
+        allocated = next(claim for claim in claims if claim.workspace_num == 10)
+        assert allocated.pid == os.getppid()
         out = capsys.readouterr().out
         assert "workspace_num=10" in out
-        assert "should_release=true" in out
+        assert "should_release=false" in out
+        assert "runner_bound_workspace=true" in out
 
     def test_pinned_n_does_not_adopt_parent_claim(
         self, tmp_path: Path, capsys: pytest.CaptureFixture[str]
@@ -935,16 +939,99 @@ class TestWsSubmitAtomicClaim:
         )
 
 
-def test_gh_release_step_tags_caller() -> None:
+def test_gh_release_step_delegates_to_gh_release() -> None:
     text = (ROOT / "src" / "sase_github" / "xprompts" / "gh.yml").read_text(
         encoding="utf-8"
     )
-    assert 'caller_tag="gh-release"' in text
+    assert "from sase_github.scripts.gh_release import main" in text
+    assert "workspace_dir={{ setup.workspace_dir | tojson }}" in text
+    assert "release_workspace(" not in text
+    assert "clear_occupant_record(" not in text
 
 
-def test_gh_release_step_clears_occupant_record() -> None:
-    text = (ROOT / "src" / "sase_github" / "xprompts" / "gh.yml").read_text(
+def test_gh_release_caller_tag() -> None:
+    source = (ROOT / "src" / "sase_github" / "scripts" / "gh_release.py").read_text(
         encoding="utf-8"
     )
-    assert "clear_occupant_record(" in text
-    assert "setup.workspace_dir" in text.split("clear_occupant_record(", 1)[1]
+    assert '_CALLER_TAG = "gh-release"' in source
+
+
+def test_gh_release_step_releases_owned_claim_and_occupant(
+    tmp_path: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    runner_pid = os.getppid()
+    project_file = _write_project_file(
+        tmp_path,
+        running_claims=[WorkspaceClaim(23, "gh-acme/widget", None, pid=runner_pid)],
+    )
+    workspace_dir = tmp_path / "widget_23"
+    workspace_dir.mkdir()
+    write_occupant_record(
+        str(workspace_dir),
+        new_occupant_record(
+            pid=runner_pid,
+            workflow="gh-acme/widget",
+            project="gh_acme__widget",
+            workspace_num=23,
+        ),
+    )
+
+    with _isolate_ledger(tmp_path):
+        gh_release.main(
+            project_file=project_file,
+            workspace_num=23,
+            workspace_dir=str(workspace_dir),
+            workflow_name="gh-acme/widget",
+            cl_name=None,
+        )
+
+    out = capsys.readouterr().out
+    assert "released=true" in out
+    assert get_claimed_workspaces(project_file) == []
+    assert read_occupant_record(str(workspace_dir)) is None
+
+
+def test_gh_release_step_skips_on_handoff(
+    tmp_path: Path, capsys: pytest.CaptureFixture[str]
+) -> None:
+    runner_pid = os.getppid()
+    project_file = _write_project_file(
+        tmp_path,
+        running_claims=[WorkspaceClaim(23, "gh-acme/widget", None, pid=runner_pid)],
+    )
+    workspace_dir = tmp_path / "widget_23"
+    workspace_dir.mkdir()
+    write_occupant_record(
+        str(workspace_dir),
+        new_occupant_record(
+            pid=runner_pid,
+            workflow="gh-acme/widget",
+            project="gh_acme__widget",
+            workspace_num=23,
+        ),
+    )
+    artifacts = tmp_path / "artifacts"
+    artifacts.mkdir()
+    (artifacts / ".sase_monitor_pending").write_text("{}", encoding="utf-8")
+
+    with (
+        _isolate_ledger(tmp_path),
+        patch.dict(os.environ, {"SASE_ARTIFACTS_DIR": str(artifacts)}),
+    ):
+        gh_release.main(
+            project_file=project_file,
+            workspace_num=23,
+            workspace_dir=str(workspace_dir),
+            workflow_name="gh-acme/widget",
+            cl_name=None,
+        )
+
+    out = capsys.readouterr().out
+    assert "released=false" in out
+    assert "skip_reason=handoff" in out
+    assert [claim.workspace_num for claim in get_claimed_workspaces(project_file)] == [
+        23
+    ]
+    occupant = read_occupant_record(str(workspace_dir))
+    assert occupant is not None
+    assert occupant.pid == runner_pid

sase_github-0.2.7/.release-please-manifest.json

@@ -1,3 +0,0 @@
-{
-  ".": "0.2.7"
-}