@geoly-ai/skills-hub 0.3.0 → 0.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geoly-ai/skills-hub",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "geoly-ai 的 skill 分发中心 —— 安装、校验、审计",
   "type": "module",
   "bin": {

package/src/commands/context.mjs

@@ -176,6 +176,32 @@ export function uintArg(name, val) {
  * @param {object} globals  parseGlobals 的产物
  * @param {object} deps     🔴 只从 `main(argv, deps)` 的第二个形参来，argv/env 到不了这里
  */
+/**
+ * 本 CLI 自己的版本 —— 🔴 **从 package.json 读，不要硬编码。**
+ *
+ * ⚠️ 这里原本是字面量 `'0.0.0-m1'`。后果不是「显示得不好看」：
+ *    `timestamp.min_cli_version` 是按真实版本号比对的策略门，
+ *    而 `bin/skills-hub.mjs` 一个 dep 都不传 —— 于是**发布出去的 CLI
+ *    自报 0.0.0-m1，会被自己的 min_cli_version 当场挡死**。
+ *    2026-09-03 首次端到端安装时撞到：timestamp 写着 0.3.0，
+ *    而刚从 npm 装下来的 0.3.0 报的是 0.0.0-m1 → E_MIN_CLI_VERSION。
+ *
+ * 🔴 读不到就**抛**，不要退回一个假版本号：一个编出来的版本号会让
+ *    版本门做出错误判定，而那正是这道门要防的事。
+ */
+let cachedVersion;
+export function ownVersion() {
+  if (cachedVersion === undefined) {
+    const p = new URL('../../package.json', import.meta.url);
+    const v = JSON.parse(readFileSync(p, 'utf8')).version;
+    if (typeof v !== 'string' || v === '') {
+      throw new Error('读不出本 CLI 的版本号（package.json 的 version 不是非空字符串）');
+    }
+    cachedVersion = v;
+  }
+  return cachedVersion;
+}
+
 export function makeContext(globals, deps = {}) {
   const env = deps.env ?? process.env;
   const home = deps.home ?? homedir();
@@ -213,7 +239,7 @@ export function makeContext(globals, deps = {}) {
     cacheDir,
     /** 🔴 时间只从这里取：canonical JSON 要求严格 `YYYY-MM-DDTHH:MM:SSZ`，测试要能定死它 */
     now: deps.now ?? (() => new Date()),
-    cliVersion: deps.cliVersion ?? env.GEOLY_CLI_VERSION ?? '0.0.0-m1',
+    cliVersion: deps.cliVersion ?? env.GEOLY_CLI_VERSION ?? ownVersion(),
     /**
      * 🔴 验签器**没有逃生口**：`deps.verifier` 只有 `main(argv, deps)` 的调用方能给，
      *    而生产入口 `bin/skills-hub.mjs` 一个 dep 都不传。

package/src/commands/preheat-run.mjs

@@ -16,8 +16,10 @@
 import { existsSync } from 'node:fs';
 import { resolveCurrent } from '../snapshot.mjs';
 import { readTrustFloor, resolveStateDir } from '../trust.mjs';
+import { mkdirChainFsync } from '../atomic-fs.mjs';
 import { preheatAssets, preheatMetadata, promoteMetadata, discard, newBudget } from '../preheat.mjs';
 import { createCacheRegistry } from './registry.mjs';
+import { getVerifier } from './snapshot-access.mjs';
 
 /**
  * 联网刷新一次 metadata（timestamp + 当前快照）到本地缓存。
@@ -28,6 +30,16 @@ export async function preheatOnce({
   cacheDir, stateDir, verifier, cliVersion, now, fetchImpl, timeoutMs,
   budget = newBudget(),
 }) {
+  // 🔴 preheat 现在是**第一个**碰这两个目录的人。
+  //    在它之前，建目录那一步藏在 `resolveSnapshotForCommand()` 里面 ——
+  //    而 preheat 排在它前面，于是干净 home 上第一次安装直接 ENOENT。
+  //    2026-09-03 端到端撞到：`lstat '<home>/.local'`。
+  //    ⚠️ 顺序要紧：`resolveStateDir()` 内部是 `realpathSync`，**要求目录已存在**。
+  //    我第一版写成 `mkdirChainFsync(resolveStateDir(stateDir))` —— 先解析后创建，
+  //    在干净 home 上必然 ENOENT，而且报的是 `lstat '<home>/.local'`，
+  //    看起来完全不像「目录还没建」。
+  mkdirChainFsync(cacheDir);
+  mkdirChainFsync(stateDir);
   const { stagingDir, n } = await preheatMetadata({ cacheDir, fetchImpl, timeoutMs, budget });
   try {
     // 🔴 registry 指向 **staging**，不是 cache —— 验的必须是刚下回来的那份。
@@ -70,10 +82,23 @@ export async function preheatForInstall(ctx, out) {
   if (ctx.registryFactory) return { refreshed: false, reason: 'custom-registry' };
   const haveCache = existsSync(`${ctx.cacheDir}/timestamp.json`);
   try {
+    // 🔴 `ctx.verifier` 的缺省值是 **null**（不是 undefined）——
+    //    直接传会得到 `E_VERIFIER_MISSING`。命令面统一走 `getVerifier()`，
+    //    它在没注入时落到**真验签器**（内置信任根）。
+    //    ⚠️ 与 fetchImpl 是**同一个形状**：注入点缺省为 null，而下游按
+    //    「没给就用默认」写。一处栽了就该全仓找同形状的——这是第二处。
     const r = await preheatOnce({
       cacheDir: ctx.cacheDir, stateDir: ctx.stateDir,
-      verifier: ctx.verifier, cliVersion: ctx.cliVersion, now: ctx.now,
-      fetchImpl: ctx.fetchImpl,
+      verifier: await getVerifier(ctx),
+      cliVersion: ctx.cliVersion,
+      // 🔴 `ctx.now` 是**函数**（`() => new Date()`），而 `resolveCurrent` 要的是
+      //    **毫秒数**。直接传函数会一路走到 `makeFloor` 里 `new Date(fn).toISOString()`
+      //    → `RangeError: Invalid time value`，报出来的话完全看不出是这儿。
+      //    命令面原本就写着 `now: ctx.now().getTime()`（snapshot-access.mjs:54）——
+      //    ⚠️ 这是我今天**第三次**把 ctx 字段原样传下去而没看形状
+      //    （前两次：fetchImpl 与 verifier 的缺省是 null）。
+      now: ctx.now().getTime(),
+      fetchImpl: ctx.fetchImpl ?? undefined,
     });
     if (r.refreshed) out?.note?.(`已刷新到快照 ${r.n}`);
     return r;

package/src/download.mjs

@@ -135,11 +135,18 @@ export async function download(url, {
   redirectHosts = REDIRECT_HOSTS,
   maxRedirects = MAX_REDIRECTS,
   cap = MAX_DOWNLOAD_BYTES,
-  fetchImpl = globalThis.fetch,
+  fetchImpl,
   timeoutMs = DEFAULT_TIMEOUT_MS,
   what = url,
 } = {}) {
   assertDownloadUrl(url, host);
+  // 🔴 用 `??` 而不是默认参数：**默认参数只对 `undefined` 生效**。
+  //    `commands/context.mjs` 里 `fetchImpl` 的缺省值是 **`null`**（与 verifier
+  //    等注入点一致），null 会原样穿过默认参数，于是生产路径上拿到的是 null。
+  //    ⚠️ 单元测试**永远抓不到这一条** —— 它们每次都注入替身，走不到缺省分支。
+  //    2026-09-03 干净 home 端到端首次安装时才炸出来：
+  //    「当前 Node 没有内建 fetch」 —— 而 Node 25 明明有。
+  fetchImpl = fetchImpl ?? globalThis.fetch;
   if (typeof fetchImpl !== 'function') {
     throw new NetworkError('当前 Node 没有内建 fetch，且调用方没有注入 fetchImpl');
   }