@opengsd/gsd-path 1.3.1 → 1.4.0

package/skills/path/scripts/worktree_paths.py

@@ -13,10 +13,13 @@ import os
 import tempfile
 from pathlib import Path
 
-try:
-    import _common
-except ImportError:
-    from scripts import _common
+if __package__:
+    from . import _common
+else:
+    try:
+        import _common
+    except ImportError:
+        from scripts import _common
 
 
 def _git(primary: Path, *args: str) -> str:
@@ -27,7 +30,10 @@ def _git(primary: Path, *args: str) -> str:
 
 
 def _identity(path: Path) -> str:
-    return hashlib.sha256(os.fsencode(path)).hexdigest()
+    digest = hashlib.sha256(os.fsencode(path)).hexdigest()
+    # Two identities nest in every managed path; full digests would use about
+    # 130 of Windows' 260-character MAX_PATH before any repository content.
+    return digest[:16] if os.name == "nt" else digest
 
 
 def _workspace(primary: Path, pin: bool) -> Path:
@@ -65,7 +71,7 @@ def _workspace(primary: Path, pin: bool) -> Path:
         receipt.parent.mkdir(parents=True, exist_ok=True)
         fd, temporary = tempfile.mkstemp(dir=receipt.parent)
         try:
-            with os.fdopen(fd, "w", encoding="utf-8") as handle:
+            with os.fdopen(fd, "w", encoding="utf-8", newline="\n") as handle:
                 json.dump({"primary": str(primary), "root": str(workspace)}, handle)
             try:
                 os.link(temporary, receipt)
@@ -78,15 +84,19 @@ def _workspace(primary: Path, pin: bool) -> Path:
 
 def worktree_path(primary: Path, kind: str, name: str, *, pin: bool = False) -> Path:
     primary = primary.resolve()
-    if kind not in {"task", "verify", "integrate"} or not name or Path(name).name != name or name in {".", ".."}:
+    if kind not in {"task", "verify", "integrate", "bound"} or not name or Path(name).name != name or name in {".", ".."}:
         raise ValueError("invalid managed worktree kind or name")
+    if kind == "bound":
+        return _workspace(primary, pin) / kind / name
     legacy = (primary.parent / f".{primary.name}-gsd-path-integrate-{name}"
               if kind == "integrate" else primary.parent / f"{primary.name}.gsd-path" / kind / name)
     branch = f"refs/heads/gsd-path-{kind}/{name}"
     records = _git(primary, "worktree", "list", "--porcelain", "-z").split("\0\0")
     for record in records:
         fields = record.split("\0")
-        if f"worktree {legacy}" in fields and f"branch {branch}" in fields:
+        # Compare as paths: git prints forward slashes on Windows.
+        worktrees = [Path(field[len("worktree "):]) for field in fields if field.startswith("worktree ")]
+        if legacy in worktrees and f"branch {branch}" in fields:
             return legacy
     # Keep collision checks and missing-directory recovery at the old location.
     if os.path.lexists(legacy):

package/skills/path/templates/codebase.md

@@ -4,7 +4,7 @@
      (brownfield mode), researchers (fifth standard dimension), the planner
      (conventions are binding), and reviewers. -->
 
-Repo root: <absolute path>
+Repo root: <absolute primary repo root, never a sidecar>
 Scanned: <date>
 Checks run: <exact commands run and their results, e.g. `npm test` → 37/40 pass>
 

package/skills/path/templates/docs-audit.md

@@ -3,7 +3,7 @@
 <!-- Written by the docs auditor ($gsd-path-docs-audit or $gsd-path-inspect).
      Every verdict carries evidence; a verdict without evidence is a defect. -->
 
-Repo root: <absolute path>
+Repo root: <absolute primary repo root, never a sidecar>
 Audited: <date>
 Audited HEAD: <40-hex commit the claims were checked at, or none>
 Alignment mode: <yes | no — yes when .project/ artifacts were also audited>

package/skills/path/templates/final-review.md

@@ -4,7 +4,10 @@
      criterion appears exactly once and carries checked evidence. A criterion
      listed in PLAN.md's Surface contract also names its Surface, and its
      Check is the walkthrough performed — never internal test output. Embedded
-     angle brackets are valid in concrete Check and Observed values. -->
+     angle brackets are valid in concrete Check and Observed values.
+     When .project/build/members.json locks members, add one line per locked
+     member after Reviewed HEAD: `Member reviewed HEAD: <member> <full SHA>`,
+     naming that member's bound branch tip. -->
 
 Reviewed HEAD: <full SHA>
 Overall verdict: <pass | blocked>
@@ -17,7 +20,7 @@ Overall verdict: <pass | blocked>
 ### SC1 — <criterion copied verbatim from INTENT.md>
 
 - **Verdict**: <met | not-met | unverifiable>
-- **Surface**: <the surface PLAN.md's Surface contract lists this criterion under; omit the whole field for any other criterion>
+- **Surface**: <exactly the surface name PLAN.md's Surface contract lists this criterion under, with no detail after it; omit the whole field for any other criterion>
 - **Check**: `<command actually run, or "none">`
 - **Observed**: <relevant output or observed behavior>
 - **Reference**: <file:line, artifact path, or "none">

package/skills/path/templates/gap-review.md

@@ -1,7 +1,8 @@
 # Gap Review — <N>: <risk identity>
 
 <!-- Written by one final gap reviewer. This file covers exactly one
-     cross-wave integration risk named in the review brief. -->
+     cross-wave integration risk named in the review brief. Repeat FINAL.md's
+     `Member reviewed HEAD: <member> <full SHA>` lines after Reviewed HEAD. -->
 
 Reviewed HEAD: <full SHA>
 Gap verdict: <pass | blocked>

package/skills/path/templates/task.md

@@ -10,6 +10,7 @@ worktree: null      # orchestrator-owned: isolated task worktree while active
 task_branch: null   # orchestrator-owned: gsd-path-task/<id> while parallel; null when serial
 # Optional explicit dispatch overrides: model: <advertised slug|inherit>
 # effort: <advertised effort|inherit>
+# repo: <member>    # multi-repo only: the MEMBERS.md member this task changes; omit for the coordinator
 files:              # every file this task may touch — dispatch checks overlap
   - <path>
 ---

package/skills/path/templates/wave-review.md

@@ -11,7 +11,9 @@
 Reviewed HEAD: <full review commit SHA>
 Review scope: final
      In each surface SC block also include Surface, Check, and Observed fields
-     using the final-review format. Record the actual walkthrough evidence once. -->
+     using the final-review format. Surface holds exactly the surface name from
+     PLAN.md's Surface contract; put walkthrough detail in Check and Observed.
+     Record the actual walkthrough evidence once. -->
 
 Wave verdict: <pass | blocked — blocked if any task fails>
 Cycle: <C>