task-ledger 1.2.5 → 1.2.7

package/README.md

@@ -118,7 +118,8 @@
 - 临时文件名唯一（`mkstemp`），写完 `fsync` 再 `os.replace`，无论成败都清理临时文件
 - key 按字母序落盘，减少两人同时新增时的文本冲突
 - 冲突时**报错要求重跑**，不静默覆盖
-- 写入成功后脚本会**自动同步**：把本次改动的台账文件 `git add <精确文件>` + `commit`（消息 `ledger(<命令>): <提交者>`）；当前分支有上游跟踪分支时再 `push`。全程确定性脚本，不依赖大模型
+- 写入成功后脚本会**自动同步**：把本次改动的台账文件 `git add <精确文件>` + `commit`（消息 `chore(ledger): <命令> <提交者>`，即 conventional commits 格式，多数仓库装了 commitlint 也认）；当前分支有上游跟踪分支时再 `push`。全程确定性脚本，不依赖大模型
+- 仓库若装了自定义提交检查并拒了这条消息，脚本会**再用 `--no-verify` 提交一次**（台账是数据文件，不过 lint）；仍失败则把 git 的报错打到 stderr 并要求手动提交 —— **同步失败绝不静默**，但退出码保持 0（台账本身已写成功）
 - 无 git 仓库 / 无上游跟踪 / 被 `.gitignore` 排除 / 无实际变更 都会静默跳过；`--no-sync` 或环境变量 `TASK_LEDGER_NO_SYNC=1` 可关闭
 - 冲突**仍不靠 `git pull` / `rebase` 解决**：并发靠指纹复查 + 原子写防覆盖，冲突时**报错要求重跑**，不静默合并
 

package/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: task-ledger
 description: 本地任务台账，跨会话记录在办任务的归属、进度与起止。当用户要登记一件新的事（新建任务 / 建个坑 / 这活我领了）、记录已有任务的进展（接着做 / 记一条进展 / 这轮先记下）、查看任务状态或历史（现在该做什么 / 这活谁在做 / 上周做过什么）时加载；对某个具体任务要求结案或退回时，只认用户的明确指令；只是改代码、讨论方案时不要用。
-version: 1.2.2
+version: 1.2.6
 allowed-tools: Read,Write,Bash,Grep
 ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "task-ledger",
-  "version": "1.2.5",
+  "version": "1.2.7",
   "description": "Task-ledger skill for coding agents: per-workspace progress tracking (ledger/progress.json, history, worklog) driven by a CLI script. Standalone install for CodeBuddy and other agents; also bundled inside the dsh-task-ledger DSH plugin.",
   "keywords": [
     "skill",

package/scripts/progress.py

@@ -354,9 +354,17 @@ def commit_guarded(
     write_json(path, data)
 
 
-def _git(*args: str, timeout: int | None = None) -> subprocess.CompletedProcess:
-    """跑一条 git 命令，吞掉输出（同步用，不污染 stdout）。"""
-    return subprocess.run(list(args), capture_output=True, text=True, timeout=timeout)
+def _git(
+    *args: str, timeout: int | None = None, check: bool = False
+) -> subprocess.CompletedProcess:
+    """跑一条 git 命令，吞掉输出（同步用，不污染 stdout）。
+
+    check=True 让失败抛异常（推送要能被下面的兜底捕获）；提交则靠返回码分辨，
+    因为不同仓库的提交检查（commitlint 等）会拒掉消息，需要看 stderr 原文。
+    """
+    return subprocess.run(
+        list(args), capture_output=True, text=True, timeout=timeout, check=check
+    )
 
 
 def _git_actor(repo_root: str) -> str:
@@ -369,7 +377,8 @@ def git_sync(action: str) -> None:
     """写成功后调用一次：把本次改动的台账文件提交；有上游跟踪分支则推送。
 
     纯脚本、确定性，不依赖大模型。无 git / 无上游 / 全被 ignore / 无实际变更 都静默跳过。
-    同步是尽力而为：任何异常都不影响已完成的台账写入（只打印提示）。
+    同步失败（提交被仓库钩子拒后连 --no-verify 也失败、推送被拒、意外异常）一律打到
+    stderr 提示，绝不静默；但退出码不变 —— 台账本身已经写成功，别让调用方误判去重做。
     """
     if os.environ.get("TASK_LEDGER_NO_SYNC"):
         return
@@ -382,29 +391,49 @@ def git_sync(action: str) -> None:
         if top.returncode != 0:
             return  # 不是 git 项目 → 跳过（规则1：有才提交）
         root = top.stdout.strip()
+        g = ("git", "-C", root)
         # 只 add 本次真实改动、且未被 ignore 的文件
         to_add = [
             str(f)
             for f in files
-            if _git("git", "-C", root, "check-ignore", "-q", str(f)).returncode != 0
+            if _git(*g, "check-ignore", "-q", str(f)).returncode != 0
         ]
         if not to_add:
             return  # 全被 ignore → 尊重用户选择，不强行提交
-        _git("git", "-C", root, "add", *to_add)
-        if _git("git", "-C", root, "diff", "--cached", "--quiet").returncode == 0:
+        _git(*g, "add", *to_add)
+        if _git(*g, "diff", "--cached", "--quiet").returncode == 0:
             return  # 无实际变更
         actor = _git_actor(root)
-        _git("git", "-C", root, "commit", "-m", f"ledger({action}): {actor}")
+        # 消息按 conventional commits 写：多数仓库装了 commitlint，只认白名单里的 type
+        msg = f"chore(ledger): {action} {actor}"
+        done = _git(*g, "commit", "-m", msg)
+        if done.returncode != 0:
+            # 各仓库提交规则各不相同（type 白名单、大小写、scope 枚举……），台账是数据文件，
+            # 过不了就让这一次跳过钩子提交，别把台账永久锁在本地暂存区。
+            done = _git(*g, "commit", "--no-verify", "-m", msg)
+            if done.returncode == 0:
+                print(
+                    f"提示：仓库提交检查拒绝了消息「{msg}」，已用 --no-verify 提交本次台账改动。",
+                    file=sys.stderr,
+                )
+        if done.returncode != 0:
+            print(
+                "提示：台账已写入本地，但自动提交失败，请手动提交 ledger/："
+                f"{(done.stderr or done.stdout or '').strip()}",
+                file=sys.stderr,
+            )
+            return
         # 规则2：有上游跟踪分支才推
-        if _git("git", "-C", root, "rev-parse", "--abbrev-ref", "@{upstream}").returncode != 0:
+        upstream = _git(*g, "rev-parse", "--abbrev-ref", "@{upstream}")
+        if upstream.returncode != 0:
             return
         try:
-            _git("git", "-C", root, "push", timeout=30)
+            _git(*g, "push", timeout=30, check=True)
         except (subprocess.CalledProcessError, subprocess.TimeoutExpired):
             # 非快进：自动 rebase 一次再推；冲突 / 超时则放弃，绝不 --force
             try:
-                _git("git", "-C", root, "pull", "--rebase", "--autostash", timeout=30)
-                _git("git", "-C", root, "push", timeout=30)
+                _git(*g, "pull", "--rebase", "--autostash", timeout=30, check=True)
+                _git(*g, "push", timeout=30, check=True)
             except (subprocess.CalledProcessError, subprocess.TimeoutExpired):
                 print(
                     "提示：本地已提交，但推送被拒（他人已推 / 需认证 / 超时）。"
@@ -412,8 +441,81 @@ def git_sync(action: str) -> None:
                     file=sys.stderr,
                 )
     except Exception as exc:  # noqa: BLE001 —— 同步失败不应污染台账主流程
-        print(f"提示：写后自动同步未执行（{type(exc).__name__}: {exc}）。数据已写入本地。",
-              file=sys.stderr)
+        print(
+            f"提示：写后自动同步未执行（{type(exc).__name__}: {exc}）。数据已写入本地。",
+            file=sys.stderr,
+        )
+
+
+# 只读命令不碰 git；其余（含写类与 tags）写前刷新 ledger/。
+READ_ONLY_COMMANDS = {"list", "show", "history", "validate"}
+
+
+def git_presync(action: str, args: argparse.Namespace) -> None:
+    """写命令执行前把 ledger/ 刷成远端最新，避免基于过期读取整文件回写。
+
+    只动 ledger/ 目录（不碰项目其余文件），与 git_sync 共用 --no-sync / TASK_LEDGER_NO_SYNC。
+    无 git / 无上游 / 远端尚无该路径 静默跳过；拉取失败只提示不中止（推送端 rebase 兜底）。
+    本地未提交的 ledger 改动会被远端覆盖——正是「取最新再改」语义，仅打提示。
+    """
+    if os.environ.get("TASK_LEDGER_NO_SYNC"):
+        return
+    if action in READ_ONLY_COMMANDS:
+        return
+    # tags --all 是纯读，不拉取
+    if action == "tags" and not getattr(args, "rename", None):
+        return
+    try:
+        repo = find_repo(getattr(args, "repo", None))
+    except Fail:
+        return
+    ledger_dir = repo / "ledger"
+    top = _git("git", "-C", str(repo), "rev-parse", "--show-toplevel")
+    if top.returncode != 0:
+        return
+    git_root = Path(top.stdout.strip())
+    try:
+        ledger_rel = str(ledger_dir.resolve().relative_to(git_root.resolve()))
+    except ValueError:
+        return  # ledger 不在 git 仓内
+    ups = _git("git", "-C", str(git_root), "rev-parse", "--abbrev-ref", "@{upstream}")
+    if ups.returncode != 0:
+        return  # 无上游跟踪分支
+    upstream = ups.stdout.strip()
+    remote = upstream.split("/", 1)[0]
+    try:
+        _git("git", "-C", str(git_root), "fetch", remote, timeout=30, check=True)
+    except (subprocess.CalledProcessError, subprocess.TimeoutExpired):
+        print(
+            "提示：写前拉取失败（离线 / 无权限 / 超时），将基于本地当前内容写入；"
+            "若他人已改动，推送时可能冲突。",
+            file=sys.stderr,
+        )
+        return
+    if _git(
+        "git", "-C", str(git_root), "status", "--porcelain", ledger_rel
+    ).stdout.strip():
+        print(
+            "提示：本地 ledger/ 有未提交改动，写前拉取将用远端最新覆盖。",
+            file=sys.stderr,
+        )
+    # 只取远端 ledger/ 到工作树；该路径在远端不存在（全新台账）时忽略
+    try:
+        _git(
+            "git",
+            "-C",
+            str(git_root),
+            "checkout",
+            upstream,
+            "--",
+            ledger_rel,
+            timeout=30,
+            check=True,
+        )
+    except (subprocess.CalledProcessError, subprocess.TimeoutExpired):
+        pass
+    # 取消预暂存：只更新工作树，避免把他人改动混进本次提交（git_sync 只 add 本次改动的文件）
+    _git("git", "-C", str(git_root), "reset", "--", ledger_rel)
 
 
 def require_task_id(tid: str) -> str:
@@ -2277,6 +2379,7 @@ def main(argv: list[str] | None = None) -> int:
     if getattr(args, "no_sync", False):
         os.environ["TASK_LEDGER_NO_SYNC"] = "1"
     try:
+        git_presync(args.command, args)  # 写前刷新 ledger/ 到远端最新
         args.func(args)
     except Fail as exc:
         print(f"错误：{exc}", file=sys.stderr)