技能包：配置与代码分家 —— 每包自建 env/ 放本机专属配置，scripts/ 不再写死本机路径
用户 2026-10-11 原话： · 19:22「这个环境都是针对一台机器的，同步到别的机器把环境都搞坏了」 · 19:28「不用挪出 用env文件夹区分就行，重点是 脚本是脚本 配置是配置」 · 19:29「每个技能 需要环境配置的 都有自己的 env文件夹」 · 19:30「修改完成后要仔细检查 避免调整后 各技能脚本运行失败」 改动： 1、session-mechanism 新增 env/（roots.env＝本机真身不入库 ＋ roots.env.example＝样例入库）， install.py 的 ROOTS_ENV 改指 env/roots.env；全包脚本/hook 的配置定位统一为 「向上 4 层 × env/roots.env 优先 → 包根 roots.env 兼容 → 技能库那份兜底」。 2、去硬编码：collabctl / keepalive / supervise-launch / board-launch 里的本机路径与本机工作区 清单全清，改为 DSH_AIROOT（兜底 dirname(DSH_WS_ROOT)）＋ 扫 <AIROOT>/*/.workbuddy/collab/ 自动发现；保活任务名按 collabd-keepalive-<工作区名> 现推；启动器默认值改 ~/.workbuddy 形态。 3、browser-harness：bh_chrome_keepalive.py 默认值全部改 env 驱动，新增 env/bh.env.example。 4、.gitignore：env/ 下只放行 *.example（三行三种层级）＋ 兜住旧位置的包根 roots.env。 5、改包纪律.md 新增「四、包结构：脚本是脚本，配置是配置」，立为全仓通用规矩。 6、文档同步：SKILL.md / deploy.md（新增 §3b 配置层级、§6 换机器必查 6 件事）/ rules.md / pitfalls.md / supervise-persistence.md。 验收（本机）：py_compile 34 个 py 全过；selftest PASS 109 / FAIL 0； 逐文件实证 8 个 loader 命中 env/roots.env；preflight-lock.sh 同目录实测解出两个根； install.py --manifest 77 份、语法失败 0。 异机验证（包拷成「新克隆」（只有 *.example）＋ 另造假工作区 ws-alpha/ws-beta）： install.py --apply 自生成该机 env/roots.env；collabctl/keepalive 自动发现的确实是 ws-alpha/ws-beta、任务名现推（写死清单确已消除）；部署副本本地无 roots.env 时靠 CODEBUDDY_CONFIG_DIR 兜底读到技能库那份；整套 selftest 有宿主库时 PASS 108 / FAIL 1 （唯一 FAIL＝该区没有常驻心跳，环境所致；本机常驻在跑时同一套为 PASS 109 / FAIL 0）。
This commit is contained in:
admin committed
1 parent
90cc4315d5
commit
d81f75f173
37 files changed
+763
-291
No files matched your search
@@ -13,24 +13,27 @@
|
||||
python selftest.py -k 投递 # 只跑名字含"投递"的用例
|
||||
"""
|
||||
from __future__ import annotations
|
||||
# [session-mechanism] roots.env 外置（由 install.py 生成；缺失则回落到按位置推导）
|
||||
# [session-mechanism] 环境配置外置（由 install.py 生成）；2026-10-11 起落在本包 `env/roots.env`
|
||||
def _sm_load_roots():
|
||||
import os as _os
|
||||
_here = _os.path.dirname(_os.path.abspath(__file__))
|
||||
# 🔴 2026-10-11：「脚本是脚本 配置是配置」⇒ 配置落本包 `env/`。
|
||||
# 首选 `env/roots.env`；并兼容旧的包根 `roots.env`（拷贝未同步时照旧能跑）。
|
||||
for _up in range(4):
|
||||
_p = _os.path.join(_here, *([".."] * _up), "roots.env")
|
||||
_p = _os.path.normpath(_p)
|
||||
if _os.path.isfile(_p):
|
||||
try:
|
||||
with open(_p, encoding="utf-8") as _f:
|
||||
for _ln in _f:
|
||||
_ln = _ln.strip()
|
||||
if _ln and not _ln.startswith("#") and "=" in _ln:
|
||||
_k, _v = _ln.split("=", 1)
|
||||
_os.environ.setdefault(_k.strip(), _v.strip())
|
||||
except Exception:
|
||||
pass
|
||||
return
|
||||
_base = _os.path.normpath(_os.path.join(_here, *([".."] * _up)))
|
||||
for _rel in (("env", "roots.env"), ("roots.env",)):
|
||||
_p = _os.path.join(_base, *_rel)
|
||||
if _os.path.isfile(_p):
|
||||
try:
|
||||
with open(_p, encoding="utf-8") as _f:
|
||||
for _ln in _f:
|
||||
_ln = _ln.strip()
|
||||
if _ln and not _ln.startswith("#") and "=" in _ln:
|
||||
_k, _v = _ln.split("=", 1)
|
||||
_os.environ.setdefault(_k.strip(), _v.strip())
|
||||
except Exception:
|
||||
pass
|
||||
return
|
||||
_sm_load_roots()
|
||||
# 🔴 2026-10-01 加 · **输出编码兜底**：脚本一旦被重定向（钩子/常驻/后台任务都会这么干），
|
||||
# Windows 本地编码（GBK）编不出 ⛔/✅/🔴 这类字符 ⇒ `print` 抛 UnicodeEncodeError
|
||||
@@ -5833,7 +5836,7 @@ def t_hook_reads_ws_from_stdin_cwd():
|
||||
|
||||
⚖️ 同源判据的既有正确写法（⛔ 必须对齐，不能自创）：
|
||||
`wb-result-hook.py` / `lock-guard-hook.py` 都是**先读 stdin 的
|
||||
`payload.cwd`**，env / `roots.env` 只作兜底。
|
||||
`payload.cwd`**，env / `env/roots.env` 只作兜底。
|
||||
|
||||
🔴 判据要点：⛔ **不许只查"读没读 cwd"**——还要钉住
|
||||
① stdin **只读一次**（`sys.stdin.buffer.read()` 是一次性的）
|
||||
@@ -6454,7 +6457,9 @@ def t_mirror_drift():
|
||||
(fx_cp / "a.py").write_text("A=1\n", encoding="utf-8")
|
||||
(fx_cp / "sub" / "b.py").write_text("B=1\n# TAMPER\n", encoding="utf-8") # 待更新
|
||||
(fx_cp / "r_stale.py").write_text("STALE\n", encoding="utf-8") # 真残留
|
||||
(fx_cp / "roots.env").write_text("DSH_WS_ROOT=/fixture/ONLY\n", encoding="utf-8") # 白名单
|
||||
# 🔴 2026-10-11：白名单落点在 `env/roots.env`（配置与代码分家）⇒ 夹具照真实结构摆
|
||||
(fx_cp / "env").mkdir(parents=True, exist_ok=True)
|
||||
(fx_cp / "env" / "roots.env").write_text("DSH_WS_ROOT=/fixture/ONLY\n", encoding="utf-8") # 白名单
|
||||
|
||||
g, c, only_c, only_g, changed = wm.diff(fx_src, fx_cp)
|
||||
det = [
|
||||
@@ -6464,7 +6469,7 @@ def t_mirror_drift():
|
||||
"c.py" in only_g and len(only_g) == 1),
|
||||
("抓到**待更新**（两边都有但内容不同）",
|
||||
"sub/b.py" in changed and "a.py" not in changed),
|
||||
("**没把白名单当漂移**（roots.env 不该出现在任一差异里）",
|
||||
("**没把白名单当漂移**（env/roots.env 不该出现在任一差异里）",
|
||||
not any("roots.env" in x for x in (only_c + only_g + changed))),
|
||||
]
|
||||
|
||||
@@ -6480,10 +6485,10 @@ def t_mirror_drift():
|
||||
and (fx_cp / "c.py").read_bytes() == (fx_src / "c.py").read_bytes()),
|
||||
("同步后**篡改已被覆盖回全局版**",
|
||||
(fx_cp / "sub" / "b.py").read_bytes() == (fx_src / "sub" / "b.py").read_bytes()),
|
||||
("🔴 **白名单内容原样保住**（roots.env 仍写 /fixture/ONLY，"
|
||||
("🔴 **白名单内容原样保住**（env/roots.env 仍写 /fixture/ONLY，"
|
||||
"⛔ 被冲回全局值就会让副本去读写别人的仓库）",
|
||||
(fx_cp / "roots.env").exists()
|
||||
and "/fixture/ONLY" in (fx_cp / "roots.env").read_text(encoding="utf-8")),
|
||||
(fx_cp / "env" / "roots.env").exists()
|
||||
and "/fixture/ONLY" in (fx_cp / "env" / "roots.env").read_text(encoding="utf-8")),
|
||||
("**幂等**：再 sync 一次 rc=0", wm.do_sync(fx_src, fx_cp) == 0),
|
||||
]
|
||||
|
||||
@@ -6493,7 +6498,7 @@ def t_mirror_drift():
|
||||
wsrc = mp.read_text(encoding="utf-8")
|
||||
det += [
|
||||
("覆盖循环里有**白名单守卫** `if k in KEEP: continue`"
|
||||
"（⛔ 拿掉它 ⇒ roots.env 被全局值覆盖 ⇒ 副本去读写别人的仓库）",
|
||||
"（⛔ 拿掉它 ⇒ env/roots.env 被全局值覆盖 ⇒ 副本去读写别人的仓库）",
|
||||
re.search(r"for k in sorted\(g\):\s*\n\s*if k in KEEP:\s*\n\s*continue", wsrc) is not None),
|
||||
("`--sync` 先**备份再删**（⛔ 破坏性动作前必须留退路）",
|
||||
wsrc.index("copytree") < wsrc.index("unlink()")),
|
||||
@@ -6543,30 +6548,31 @@ def t_mirror_drift():
|
||||
]
|
||||
|
||||
# ③ 有备份时**能**捞回：造一次"先 sync（留备份）→ 再删 → 再 sync"
|
||||
(fx_cp2 / "roots.env").write_text("DSH_WS_ROOT=/fixture/BACKUPED\n", encoding="utf-8")
|
||||
(fx_cp2 / "env").mkdir(parents=True, exist_ok=True)
|
||||
(fx_cp2 / "env" / "roots.env").write_text("DSH_WS_ROOT=/fixture/BACKUPED\n", encoding="utf-8")
|
||||
(fx_cp2 / "sub" / "b.py").write_text("B=1\n# DIFF\n", encoding="utf-8") # 制造漂移 ⇒ 触发备份
|
||||
wm.do_sync(fx_src2, fx_cp2) # ← 这次备份里有 roots.env
|
||||
(fx_cp2 / "roots.env").unlink()
|
||||
wm.do_sync(fx_src2, fx_cp2) # ← 这次备份里有 env/roots.env
|
||||
(fx_cp2 / "env" / "roots.env").unlink()
|
||||
rc_bk = wm.do_sync(fx_src2, fx_cp2)
|
||||
det += [
|
||||
("🔴 **有备份时，白名单被删后 `--sync` 能捞回**"
|
||||
"（`roots.env` 丢失 ⇒ 副本按位置推导去读写**别的仓库**，静默且难查）",
|
||||
rc_bk == 0 and (fx_cp2 / "roots.env").exists()
|
||||
and "/fixture/BACKUPED" in (fx_cp2 / "roots.env").read_text(encoding="utf-8")),
|
||||
"（`env/roots.env` 丢失 ⇒ 副本按位置推导去读写**别的仓库**，静默且难查）",
|
||||
rc_bk == 0 and (fx_cp2 / "env" / "roots.env").exists()
|
||||
and "/fixture/BACKUPED" in (fx_cp2 / "env" / "roots.env").read_text(encoding="utf-8")),
|
||||
]
|
||||
shutil.rmtree(str(fx_src2.parent), ignore_errors=True)
|
||||
shutil.rmtree(str(fx_src.parent), ignore_errors=True)
|
||||
return det
|
||||
|
||||
|
||||
@case("副本同步：`roots.env` 白名单**只按文件名**判（⛔ 换副本不许靠「我记得跳过哪个」）")
|
||||
@case("副本同步：`env/roots.env` 白名单**只按路径**判（⛔ 换副本不许靠「我记得跳过哪个」）")
|
||||
def t_mirror_keep_by_name():
|
||||
import importlib.util
|
||||
spec = importlib.util.spec_from_file_location("wm_selftest2", HERE / "workspace_mirror.py")
|
||||
wm = importlib.util.module_from_spec(spec)
|
||||
spec.loader.exec_module(wm)
|
||||
return [
|
||||
("白名单含 `roots.env`（⛔ 覆盖它 ⇒ 副本读写到别的仓库）", "roots.env" in wm.KEEP),
|
||||
("白名单含 `env/roots.env`（⛔ 覆盖它 ⇒ 副本读写到别的仓库）", "env/roots.env" in wm.KEEP),
|
||||
("白名单含 `副本使用说明.md`（副本独有的口径文档）", "副本使用说明.md" in wm.KEEP),
|
||||
("白名单含 `install.log`（副本自己的安装历史）", "install.log" in wm.KEEP),
|
||||
("白名单是**元组**（⛔ 散着写容易漏）", isinstance(wm.KEEP, tuple)),
|
||||
|
||||
Reference in new issue
Block a user