@hupan56/wlkj 2.5.0 → 2.7.0

package/templates/qoder/scripts/check_qoderwork_consistency.py

@@ -0,0 +1,166 @@
+#!/usr/bin/env python3
+# -*- coding: utf-8 -*-
+"""
+check_qoderwork_consistency.py — 防退化检查：命令/skills 文件不得用裸相对路径调脚本。
+
+背景：
+  QoderWork 桌面端工作目录是临时会话目录(workspace/xxx), 不是仓库根。
+  命令/技能文件里写 `python .qoder/scripts/xxx.py` → 在 QoderWork 下找不到脚本 →
+  Shell 直接报错, Python 根本没启动。这就是"ZCode 行、QoderWork 不行"的根因。
+
+  统一规约: 所有脚本调用必须用仓库根定位, 二选一:
+    ① python "$R/.qoder/scripts/xxx.py" ...    (R = repo_root.py 输出)
+    ② cd "$R" && python .qoder/scripts/xxx.py ...  (cd 到仓库根后相对路径)
+
+  本脚本扫 .qoder/commands/**/*.md 和 .qoder/skills/**/SKILL.md,
+  找出违反规约的裸相对路径调用, 退出码非 0 表示有退化。
+  可挂 pre-commit / CI。
+
+Usage:
+    python .qoder/scripts/check_qoderwork_consistency.py
+    python .qoder/scripts/check_qoderwork_consistency.py --strict   # 警告也算失败
+Exit:
+    0 = 通过(无裸相对路径调用, 或仅有 --strict 才算失败的警告)
+    1 = 发现裸相对路径调用(退化)
+"""
+import argparse
+import os
+import re
+import sys
+
+if sys.platform == 'win32':
+    try:
+        sys.stdout.reconfigure(encoding='utf-8')
+    except (AttributeError, IOError):
+        pass
+
+sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+from common.paths import get_repo_root  # noqa: E402
+
+# 扫描目标: commands 全部 .md + skills 的 SKILL.md
+SCAN_GLOBS = [
+    ('commands', '.qoder/commands', '*.md'),
+    ('skills', '.qoder/skills', 'SKILL.md'),
+]
+
+# 裸相对路径调用: `python .qoder/scripts/xxx` 但前面没有 cd "$R" 或 "$R/ 前缀
+# 匹配 `python .qoder/scripts/` 或 `python3 .qoder/scripts/`, 不带 $R 引号包裹
+BARE_PAT = re.compile(r'(?<![\"\$])\bpython3?\s+\.qoder/scripts/([A-Za-z0-9_./-]+)')
+
+# 已安全模式: cd "$R" && python .qoder/scripts/...
+SAFE_CD_PAT = re.compile(r'cd\s+["\']?\$R["\']?\s*&&\s*python3?\s+\.qoder/scripts/')
+
+# 仓库根定位块存在的标志
+ROOT_LOCATOR_PAT = re.compile(r'repo_root\.py|R=\$\(python.*repo_root')
+
+# 在 install_qoderwork.py 提示行里的相对路径(说明性文字, 非真实调用), 豁免
+HINT_CTX = 'install_qoderwork.py'
+
+
+def scan_file(path, strict):
+    """扫一个文件, 返回 (errors, warnings, root_block_present)。
+    errors = 裸相对路径真实调用(必修); warnings = 无 R 块等(建议修)。
+    """
+    errors = []
+    warnings = []
+    try:
+        with open(path, encoding='utf-8') as f:
+            lines = f.readlines()
+    except (OSError, IOError):
+        return errors, warnings, True
+
+    root_block = bool(ROOT_LOCATOR_PAT.search(''.join(lines)))
+
+    for i, line in enumerate(lines, 1):
+        # 豁免: 说明性文字里提到 install_qoderwork.py 的相对路径(这是"找不到时跑这个"的指引)
+        if HINT_CTX in line:
+            continue
+        # 豁免: 已安全的 cd "$R" && python 模式
+        if SAFE_CD_PAT.search(line):
+            continue
+        # 找裸相对路径
+        for m in BARE_PAT.finditer(line):
+            script = m.group(1)
+            # 过滤掉明显是文字描述(行里既无 bash 代码块上下文也无命令感)
+            # 简化: 只要匹配到就算(宁可误报让 reviewer 看一眼)
+            errors.append((i, line.rstrip(), script))
+
+    # 无 R 块但文件里有 $R 引用 → 警告
+    if not root_block and '$R' in ''.join(lines):
+        warnings.append((0, '(无 R 定位块, 但文件里用了 $R)', ''))
+
+    return errors, warnings, root_block
+
+
+def main():
+    ap = argparse.ArgumentParser(description='检查命令/skills 是否含裸相对路径(防 ZCode/QoderWork 割裂)')
+    ap.add_argument('--strict', action='store_true', help='警告也算失败')
+    args = ap.parse_args()
+
+    repo = get_repo_root()
+    total_errors = 0
+    total_warnings = 0
+    files_with_errors = []
+    files_with_warnings = []
+
+    for label, rel_dir, pattern in SCAN_GLOBS:
+        scan_dir = repo / rel_dir
+        if not scan_dir.is_dir():
+            continue
+        for dirpath, dirnames, filenames in os.walk(str(scan_dir)):
+            for fn in filenames:
+                import fnmatch
+                if not fnmatch.fnmatch(fn, pattern):
+                    continue
+                path = os.path.join(dirpath, fn)
+                errors, warnings, root_block = scan_file(path, args.strict)
+                rel = os.path.relpath(path, str(repo))
+                if errors:
+                    files_with_errors.append((rel, errors))
+                    total_errors += len(errors)
+                if warnings:
+                    files_with_warnings.append((rel, warnings))
+                    total_warnings += len(warnings)
+
+    # 输出
+    print('=' * 64)
+    print('QoderWork 一致性检查 (防 ZCode 行 / QoderWork 挂 的割裂)')
+    print('=' * 64)
+
+    if files_with_errors:
+        print()
+        print('🔴 裸相对路径调用 (QoderWork 必挂, 必修):')
+        for rel, errs in files_with_errors:
+            print('  ' + rel)
+            for lineno, line, script in errs:
+                snippet = line.strip()[:80]
+                print('    L%d: %s' % (lineno, snippet))
+
+    if files_with_warnings:
+        print()
+        print('🟡 警告 (建议修, --strict 才算失败):')
+        for rel, ws in files_with_warnings:
+            print('  ' + rel)
+            for lineno, line, script in ws:
+                print('    %s' % line)
+
+    if not files_with_errors and not files_with_warnings:
+        print('✅ 全部通过: 命令/skills 无裸相对路径调用, ZCode ≡ QoderWork 一致。')
+
+    print('-' * 64)
+    print('统计: 错误 %d (文件 %d) / 警告 %d (文件 %d)'
+          % (total_errors, len(files_with_errors), total_warnings, len(files_with_warnings)))
+    print()
+    print('规约: 脚本调用必须二选一:')
+    print('  ① python "$R/.qoder/scripts/xxx.py" ...    (R = repo_root.py 输出)')
+    print('  ② cd "$R" && python .qoder/scripts/xxx.py ...')
+
+    if total_errors > 0:
+        return 1
+    if args.strict and total_warnings > 0:
+        return 1
+    return 0
+
+
+if __name__ == '__main__':
+    sys.exit(main())

package/templates/qoder/scripts/common/developer.py

@@ -73,9 +73,11 @@ def _detect_git_author() -> str | None:
 
 
 def _ensure_git_config(name: str, repo_root: Path) -> None:
-    """确保 git user.name/email 已配置。未配则用 --local 设为 init 的名字。
+    """确认 git user.name/email 已配置。未配则提示 (绝不自动造假账号)。
 
-    检测顺序: 全局 > local。都没有就提示用户。
+    旧逻辑会塞 name@local 假账号, 与零信任 push 门禁冲突, 已废弃。
+    真正的 git 配置由 setup.py configure_git_identity() 在 init 前完成。
+    这里只做兜底检测: 如果走到这还没配, 提示用户去配。
     """
     import subprocess
     # 先看 git 是否可用
@@ -87,23 +89,20 @@ def _ensure_git_config(name: str, repo_root: Path) -> None:
 
     author = _detect_git_author()
     if author:
-        print(f"  [OK] git 作者: {author}")
+        # 检测到疑似假账号也要提示 (虽然配了, 但 push 会被门禁拒)
+        if "@local" in author:
+            print(f"  [WARN] git 作者疑似假账号: {author} (push 门禁会拒绝)")
+            print(f"       请重跑 'wlkj init' 配置真实团队 git 账号")
+        else:
+            print(f"  [OK] git 作者: {author}")
         return
 
-    # git 未配 user.name/email: 用 init 名字设 local config (不影响全局)
-    print(f"  [设置] git 未配置 user.name/email, 用你的名字设为 local:")
-    fake_email = f"{name}@local"
-    try:
-        subprocess.run(["git", "config", "--local", "user.name", name],
-                       cwd=str(repo_root), capture_output=True, timeout=5)
-        subprocess.run(["git", "config", "--local", "user.email", fake_email],
-                       cwd=str(repo_root), capture_output=True, timeout=5)
-        print(f"    user.name = {name}")
-        print(f"    user.email = {fake_email}")
-        print(f"    (仅本仓库生效。如有团队 git 账号, 自行改: git config --local user.email 你的邮箱)")
-    except (FileNotFoundError, OSError, subprocess.SubprocessError) as e:
-        print(f"  [WARN] 设置 git config 失败 (不阻塞): {e}")
-        print(f"    请手动跑: git config user.name {name} && git config user.email 你的邮箱")
+    # git 未配: 不造假, 只提示
+    print(f"  [WARN] git 未配置 user.name/email")
+    print(f"       push/commit 会失败。请配置团队 git 账号:")
+    print(f"       git config --global user.name '你的名字'")
+    print(f"       git config --global user.email '你的团队邮箱'")
+    print(f"       或重跑: wlkj init <你的名字>")
 
 
 def init_developer(name: str, role: str | None = None,
@@ -173,7 +172,15 @@ def init_developer(name: str, role: str | None = None,
     #    这是 push 门禁的前提: 没 member.json → syncgate 拒绝 push
     git_author = _detect_git_author()
     try:
-        from common.identity import register_member, IdentityError
+        from common.identity import register_member, IdentityError, get_member
+        # 修正: 旧 member.json 若是假账号(@local), 且现在 git 配了真账号 → 用真账号覆盖
+        # register_member 已有"已注册则只更新 git_author, 密钥不动"的逻辑, 这里只加提示
+        existing_member = get_member(name, repo_root)
+        if existing_member:
+            old_author = existing_member.get("git_author", "")
+            if (old_author and "@local" in old_author
+                    and git_author and "@local" not in git_author):
+                print(f"  [修正] member.json git_author 从假账号更新为真账号: {git_author}")
         try:
             register_member(name, role or "pm", git_author, repo_root)
             print(f"  [OK] 身份注册: member.json + 签名密钥已生成")
@@ -182,7 +189,7 @@ def init_developer(name: str, role: str | None = None,
     except ImportError:
         pass  # identity 模块不可用, 跳过 (向后兼容)
 
-    # 5. 确保 git 配置 (检测 + 自动设 local)
+    # 5. 确保 git 配置 (只检测+提示, 不造假; 真账号由 setup.py 在 init 前配好)
     _ensure_git_config(name, repo_root)
 
     print(f"Developer initialized: {name}" + (f" ({role})" if role else ""))

package/templates/qoder/scripts/common/events.py

@@ -0,0 +1,46 @@
+# -*- coding: utf-8 -*-
+"""events.py - 流水线事件字典 (Single Source of Truth)
+
+为什么需要它:
+  之前 learn.py 的 stats_map 和 learn_aggregate.py 的 SQL FILTER 各自硬编码
+  事件名, 两边漂移 → learning_stats 表的 reviews/commits 永远是 0 (没人埋点)。
+  统一到这里, 任何 skill 加埋点只需在此声明, 两端自动对齐。
+
+事件 → 统计列 → 归属 skill 的映射是 SSOT:
+  EVENT_TYPES[event] = {
+      'stat':   该事件累加到 learning.yaml / learning_stats 的哪一列 (None=不计数)
+      'skill':  归属哪个 skill (用于「埋点覆盖率」自检)
+      'label':  人类可读名 (/wl-status 展示用)
+  }
+"""
+
+# 事件类型注册表 —— 唯一事实源
+# 新增埋点: 在这里加一行 → learn.py 计数 + learn_aggregate.py SQL 自动生效
+EVENT_TYPES = {
+    # PRD 链路
+    'prd_accepted':   {'stat': 'prds_generated',  'skill': 'wl-prd-full',   'label': 'PRD 发布'},
+    'prd_rejected':   {'stat': 'rejections',      'skill': 'wl-prd-full',   'label': 'PRD 打回'},
+    'review_done':    {'stat': 'reviews_done',    'skill': 'prd-review',    'label': 'PRD 评审'},
+    # 设计链路
+    'design_import':  {'stat': 'designs_imported','skill': 'design-import', 'label': '设计录入'},
+    # 任务链路
+    'task_completed': {'stat': 'tasks_completed', 'skill': 'wl-task',       'label': '任务完成'},
+    # 提交链路
+    'commit_done':    {'stat': 'commits_done',    'skill': 'wl-commit',     'label': '代码提交'},
+    # 流水线自检事件 (不计入正向统计, 用于健康度看板)
+    'rule_violation': {'stat': None,             'skill': None,            'label': '规则违规'},
+}
+
+
+def stats_event_map():
+    """返回 {event: stat_column} (只含需要计数的, None 的过滤掉)。
+    learn.py 用它驱动 _update_stats。
+    """
+    return {ev: meta['stat'] for ev, meta in EVENT_TYPES.items() if meta['stat']}
+
+
+def event_to_sql_filter():
+    """返回 {stat_column: event} (反向映射)。
+    learn_aggregate.py 用它生成 SQL FILTER 子句, 不再硬编码。
+    """
+    return {meta['stat']: ev for ev, meta in EVENT_TYPES.items() if meta['stat']}