Skip to content

chore: 8 个文件 13 处注释里的 .alpha/ 改为 .code-puppy(零行为变化) - #1205

Merged
jinjunnn merged 1 commit into
alphafrom
t1202-comments
Aug 31, 2026
Merged

jinjunnn merged 1 commit into
alphafrom
t1202-comments

Conversation

@jinjunnn

Copy link
Copy Markdown
Owner

Fixes #1202

.code-puppy 改名(#1192)只落到代码字面量与被它直接覆盖的注释。本 PR 涉及的 8 个文件里,
.alpha代码字面量为 0(路径全部经 alpha-workdir / project-config 取得),
而 13 处注释仍写旧址 —— 行为对,注释在骗人。零行为变化。

改动清单(文件:行,全部为注释行)

文件:行 改动
packages/ext/src/plugin-fanout.ts:1 <dir>/.alpha/plugins/*.js.code-puppy
packages/ext/src/plugin-fanout.ts:53 .alpha/plugins/*.js.code-puppy
packages/ui-mac/src/main/alpha-ext-trust.ts:3 .alpha/alpha.jsonc + .alpha/plugins/*.js.code-puppy(同一行两处)
packages/ui-mac/src/main/alpha-ext-trust.ts:4 .alpha/prefs.json.code-puppy
packages/ui-mac/src/main/cloud-ipc.ts:113 <directory>/.alpha/runs/….code-puppy
packages/ui-mac/src/main/cloud-ipc.ts:188 <directory>/.alpha/runs/….code-puppy
packages/ui-mac/src/main/claude-plugin-install.ts:95 <project>/.alpha/skills.code-puppy
packages/ui-mac/src/main/alpha-cloud-schedules.ts:5 .alpha/runs/.code-puppy
packages/ui-mac/src/main/alpha-cloud-schedules.ts:135 .alpha/runs/.code-puppy
packages/ui-mac/src/main/alpha-bridge.ts:1 `.alpha``.code-puppy`
packages/ui-mac/src/main/alpha-bridge.ts:2 <project>/.alpha.code-puppy
packages/ui-mac/src/main/alpha-installs.ts:7 <projectDir>/.alpha.code-puppy
packages/ui-mac/src/main/alpha-builtin-policy.ts:3 语义修正:~/.alpha/governance.json`alphaGlobalRoot()/governance.json`

最后一条不是路径过期而是语义错误::69 实际是 path.join(alphaGlobalRoot(), "governance.json"),
~/.alpha 按 ADR-019 修订已退役。注释改为指向那个函数,不写死任何绝对路径 ——
写死只是把一个会过期的事实换成另一个。

明确未动(改了会造成缺陷,且没有任何测试会红)

join(home, ".alpha") 的 8 处退役守卫 —— 职责是认出旧址并拒绝/清理,改名会让守卫指向
不存在的路径。逐行与 origin/alpha 做字节比对,9/9 SAME:

  • packages/ext/src/project-config.ts:30,77,90
  • packages/ui-mac/src/main/alpha-environment.ts:288,299
  • packages/ui-mac/src/main/alpha-workdir.ts:128
  • packages/ui-mac/src/main/engine-config-dangling.ts:200
  • packages/ui-mac/src/main/engine-config-truth-boot.ts:243
  • 同理不动:packages/ui-mac/src/main/cloud-dispatch-gate.cases.ts:28HISTORICAL_INJECTED_DENIED_PATHS

这 6 个文件一个都不在本 PR 的 --name-only 里。

退出条件 2 的自证:diff 只含注释行

$ git diff -U0 origin/alpha...HEAD -- 'packages/**' | grep '^[+-]' | grep -v '^[+-][+-]' \
    | grep -v '^[+-]\s*\(//\|\*\|/\*\)'
(空,grep rc=1)
$ git diff -U0 origin/alpha...HEAD -- 'packages/**' | grep -c '^[+-][^+-]'
26          # = 13 删 + 13 增,证明 diff 本身不空

空输出正是这里的期望结论,所以先证明这条命令测得出已知的坏 —— 临时把
project-config.ts:30 的退役守卫 join(home, ".alpha") 改成 ".code-puppy" 并提交,
同一条命令给出:

grep_pipeline_rc=0   行数=2
-  const retiredLexical = normalize(join(home, ".alpha"))
+  const retiredLexical = normalize(join(home, ".code-puppy"))

随后 git reset --hard 回到实验前的 361d49f64(SHA 逐字相同、git status 空、
被变异那行已还原),重跑同一条命令再次为空。

本机门(逐道单独取退出码,set -o pipefail)

命令 结果
north-star bash scripts/north-star-guard.sh rc=0✓ zero upstream package edits
typecheck ×4 bun run --cwd packages/{alpha-contracts-consumer,ext,ui-mac,opencode} typecheck rc=0,error TS 计数各 0
unit contracts-consumer bash scripts/bun-test-floor.sh 15 packages/alpha-contracts-consumer rc=0 — 60 pass / 0 fail / 6 files
unit ext bash scripts/bun-test-floor.sh 100 packages/ext rc=0 — 161 pass / 0 fail / 15 files
unit ui-mac ALPHA_KNOWN_FAILS_FILE=scripts/known-fails.tsv bash scripts/bun-test-floor.sh 3000 packages/ui-mac src rc=04654 pass / 0 fail / 327 files / 219.8s
unit opencode(alpha 自有 13 个闸门文件) bash scripts/bun-test-floor.sh 496 packages/opencode <13 files> rc=0 — 496 pass / 0 fail / 13 files

与 base fail-set 的差:。ui-mac 全量 0 fail,连 known-fails.tsv 登记的已知红本轮也没出现;
一个空的红集不可能包含 base 之外的成员。opencode 那 13 个文件是 gate-files.tsvpackages/opencode
的全部登记项,条数下界 496 = 登记簿逐行条数之和,实测 Ran 496 tests across 13 files(文件数与枚举数相等)。

未跑 scripts/alpha-check.sh:它第 [8/9] 步会真建探针 worktree 并写共享 .git/config
core.hooksPath,并行 lane 期间会互相制造假红。push 用 --no-verify 同因(pre-push 整跑 alpha-check)。

主动没做的

  • 超出票面 13 处的其它 .alpha 陈旧注释没有顺手改。 实测枚举:
    grep -ranE '\.alpha([^-a-zA-Z0-9_]|$)' packages/*/src(排除 *.test.ts / *.cases.ts)
    72 处,票面只覆盖其中 13 处。剩余里至少这些同样是「代码字面量为 0、注释指旧址」:
    alpha-cloud-consent.ts:6automation-types.ts:3automation-scheduler.ts:8
    ext-inventory.ts:237ext-trust-watcher.tsx:4,5artifact-workbench.tsx:4,84,95
    ext-project-adopt.ts:5,9,43,44ext-project-residuals.ts:2,14,45ext-config.ts:428,797
    engine-plugin-seal.ts:39platform/index.ts:7preload/types.ts:306,331,440,969,999,1061,1072,1167,1178
    另有一类不该改的:alpha-migrate.ts:3factory-skills.ts:4cloud-envelope-guard.ts:11
    记的是历史/已删规则,index.ts:772 是日志字符串字面量不是注释。需要逐条分类,
    是另一张票,不是本票的顺手活。
  • 没碰 ~/.opencode/opencode.jsonc 这类与 .alpha 无关的旧址表述。
  • 没做任何存量迁移、没动 alpha.jsonc 文件名(票面 out-of-scope)。
  • 没合并本 PR。

#1192 的改名只落到代码字面量与被它直接覆盖的注释。这 8 个文件的 `.alpha`
代码字面量为 0(路径全部经 alpha-workdir / project-config 取得),注释却仍
写旧址 —— 行为对,注释在骗人。

alpha-builtin-policy.ts:3 是语义错误而非路径过期:它写「真源 =
`~/.alpha/governance.json`」,而 :69 实际是 path.join(alphaGlobalRoot(),
"governance.json")。改为指 `alphaGlobalRoot()` 这个函数,不写死绝对路径 ——
写死只是把一个会过期的事实换成另一个。

零行为变化:diff 只含注释行。

明确未动(改了会造成缺陷,且没有任何测试会红):
- join(home, ".alpha") 的 8 处退役守卫 —— project-config.ts:30,77,90 /
  alpha-environment.ts:288,299 / alpha-workdir.ts:128 /
  engine-config-dangling.ts:200 / engine-config-truth-boot.ts:243。
  它们的职责是认出旧址并拒绝/清理。
- cloud-dispatch-gate.cases.ts:28 的 HISTORICAL_INJECTED_DENIED_PATHS。

Refs #1202
@jinjunnn
jinjunnn merged commit e35410e into alpha Aug 31, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[chore] 8 个文件 13 处注释仍写 .alpha/ 而代码已走 .code-puppy;其中 alpha-builtin-policy 把「真源」指向已退役根

1 participant