From 97e47f7054a35806639e877d18dba9f2958c9747 Mon Sep 17 00:00:00 2001 From: clh02467605 Date: Mon, 13 Jul 2026 14:38:35 +0800 Subject: [PATCH 1/3] refactor: update sync:skill-assets script to read from source directly - Modified the `sync:skill-assets` script in `package.json` to remove the dependency on building `bailian-cli-core` before generating references and syncing skill versions. --- .vite-hooks/pre-commit | 2 +- docs/agents/lint-toolchain.md | 2 +- package.json | 2 +- packages/cli/package.json | 2 +- tools/generate-reference.ts | 9 +++++---- 5 files changed, 9 insertions(+), 8 deletions(-) diff --git a/.vite-hooks/pre-commit b/.vite-hooks/pre-commit index a8e7da10..f9cb3615 100755 --- a/.vite-hooks/pre-commit +++ b/.vite-hooks/pre-commit @@ -1,7 +1,7 @@ #!/usr/bin/env sh set -eu -# Regenerate skill reference + SKILL metadata (needs bailian-cli-core dist). +# Regenerate skill reference + SKILL metadata from source (no package build). pnpm run sync:skill-assets # Stage generator output so it is included in this commit. diff --git a/docs/agents/lint-toolchain.md b/docs/agents/lint-toolchain.md index 9af9df5f..f2fbaadb 100644 --- a/docs/agents/lint-toolchain.md +++ b/docs/agents/lint-toolchain.md @@ -42,7 +42,7 @@ - [ ] `.vite-hooks/pre-commit` 改动后,`pnpm install` 重新软链(走 `prepare: vp config`) - [ ] 增加 hook 时,确认在干净 clone 后能自动激活 -- [ ] pre-commit 会跑 `pnpm run sync:skill-assets`(先 build core,再 `generate:reference` + `sync:skill-version`)并 `git add` skill 资产,最后 `vp staged` +- [ ] pre-commit 会跑 `pnpm run sync:skill-assets`(`generate:reference` + `sync:skill-version`,直接读源码、无需先 build)并 `git add` skill 资产,最后 `vp staged` ### F. CI / 发版工具 diff --git a/package.json b/package.json index c13e6db3..dc59209d 100644 --- a/package.json +++ b/package.json @@ -16,7 +16,7 @@ "ready": "vp check && vp run -r test && vp run -r build", "prepare": "vp config", "check": "vp check", - "sync:skill-assets": "pnpm --filter \"bailian-cli^...\" run build && pnpm --filter bailian-cli run generate:reference && pnpm --filter bailian-cli run sync:skill-version", + "sync:skill-assets": "pnpm --filter bailian-cli run generate:reference && pnpm --filter bailian-cli run sync:skill-version", "dev": "pnpm -F bailian-cli-core dev", "bl": "pnpm -F bailian-cli dev", "kscli": "pnpm -F knowledge-studio-cli dev", diff --git a/packages/cli/package.json b/packages/cli/package.json index a7b49323..17247363 100644 --- a/packages/cli/package.json +++ b/packages/cli/package.json @@ -40,7 +40,7 @@ "registry": "https://registry.npmjs.org/" }, "scripts": { - "generate:reference": "tsx ../../tools/generate-reference.ts && sh -c 'cd ../.. && vp check --fix skills/bailian-cli/reference'", + "generate:reference": "tsx ../../tools/generate-reference.ts", "sync:skill-version": "tsx ../../tools/sync-skill-metadata.ts", "build": "vp pack", "dev": "tsx src/main.ts", diff --git a/tools/generate-reference.ts b/tools/generate-reference.ts index 0753e966..e1d4ae85 100644 --- a/tools/generate-reference.ts +++ b/tools/generate-reference.ts @@ -7,8 +7,7 @@ * * Run: pnpm --filter bailian-cli run generate:reference * Also run via `pnpm run sync:skill-assets` or the repo pre-commit hook. - * Uses tsx because workspace packages resolve to source locally. - * Requires built `bailian-cli-core` for shared flag constants. + * Uses tsx and reads workspace packages from source */ import { mkdirSync, readdirSync, rmSync, writeFileSync } from "node:fs"; import { dirname, join } from "node:path"; @@ -19,8 +18,10 @@ import { MODEL_AUTH_FLAGS, OPENAPI_AUTH_FLAGS, credentialFlagDefs, -} from "../packages/core/dist/index.mjs"; -import type { AnyCommand, FlagDef, FlagsDef } from "../packages/core/src/index.ts"; + type AnyCommand, + type FlagDef, + type FlagsDef, +} from "../packages/core/src/index.ts"; import { commands } from "../packages/cli/src/commands.ts"; const __dirname = dirname(fileURLToPath(import.meta.url)); From 7004e588662a740f9aef4dfc0f3bece214dcfdc2 Mon Sep 17 00:00:00 2001 From: clh02467605 Date: Mon, 13 Jul 2026 14:45:25 +0800 Subject: [PATCH 2/3] fix: format skill reference during sync:skill-assets for CI CI runs sync:skill-assets before vp check without pre-commit's vp staged, so reference markdown must be formatted in the shared sync script. --- docs/agents/lint-toolchain.md | 2 +- package.json | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/agents/lint-toolchain.md b/docs/agents/lint-toolchain.md index f2fbaadb..550093be 100644 --- a/docs/agents/lint-toolchain.md +++ b/docs/agents/lint-toolchain.md @@ -42,7 +42,7 @@ - [ ] `.vite-hooks/pre-commit` 改动后,`pnpm install` 重新软链(走 `prepare: vp config`) - [ ] 增加 hook 时,确认在干净 clone 后能自动激活 -- [ ] pre-commit 会跑 `pnpm run sync:skill-assets`(`generate:reference` + `sync:skill-version`,直接读源码、无需先 build)并 `git add` skill 资产,最后 `vp staged` +- [ ] pre-commit 会跑 `pnpm run sync:skill-assets`(`generate:reference` + `sync:skill-version` + `vp check --fix` reference,直接读源码、无需先 build)并 `git add` skill 资产,最后 `vp staged` ### F. CI / 发版工具 diff --git a/package.json b/package.json index dc59209d..96437d3e 100644 --- a/package.json +++ b/package.json @@ -16,7 +16,7 @@ "ready": "vp check && vp run -r test && vp run -r build", "prepare": "vp config", "check": "vp check", - "sync:skill-assets": "pnpm --filter bailian-cli run generate:reference && pnpm --filter bailian-cli run sync:skill-version", + "sync:skill-assets": "pnpm --filter bailian-cli run generate:reference && pnpm --filter bailian-cli run sync:skill-version && vp check --fix skills/bailian-cli/reference", "dev": "pnpm -F bailian-cli-core dev", "bl": "pnpm -F bailian-cli dev", "kscli": "pnpm -F knowledge-studio-cli dev", From 2ae76e2013b0e5ec85a562276cafdbe175d3a97c Mon Sep 17 00:00:00 2001 From: clh02467605 Date: Mon, 13 Jul 2026 15:34:47 +0800 Subject: [PATCH 3/3] fix: keep reference formatting in generate:reference for release check release/check.mjs calls generate:reference directly then git diff --exit-code on skills/bailian-cli/reference/. Formatting must stay on that script; root sync:skill-assets should not duplicate it. --- docs/agents/lint-toolchain.md | 2 +- package.json | 2 +- packages/cli/package.json | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/agents/lint-toolchain.md b/docs/agents/lint-toolchain.md index 550093be..4d1eb8f7 100644 --- a/docs/agents/lint-toolchain.md +++ b/docs/agents/lint-toolchain.md @@ -42,7 +42,7 @@ - [ ] `.vite-hooks/pre-commit` 改动后,`pnpm install` 重新软链(走 `prepare: vp config`) - [ ] 增加 hook 时,确认在干净 clone 后能自动激活 -- [ ] pre-commit 会跑 `pnpm run sync:skill-assets`(`generate:reference` + `sync:skill-version` + `vp check --fix` reference,直接读源码、无需先 build)并 `git add` skill 资产,最后 `vp staged` +- [ ] pre-commit 会跑 `pnpm run sync:skill-assets`(`generate:reference` 含格式化 + `sync:skill-version`,直接读源码、无需先 build)并 `git add` skill 资产,最后 `vp staged` ### F. CI / 发版工具 diff --git a/package.json b/package.json index 96437d3e..dc59209d 100644 --- a/package.json +++ b/package.json @@ -16,7 +16,7 @@ "ready": "vp check && vp run -r test && vp run -r build", "prepare": "vp config", "check": "vp check", - "sync:skill-assets": "pnpm --filter bailian-cli run generate:reference && pnpm --filter bailian-cli run sync:skill-version && vp check --fix skills/bailian-cli/reference", + "sync:skill-assets": "pnpm --filter bailian-cli run generate:reference && pnpm --filter bailian-cli run sync:skill-version", "dev": "pnpm -F bailian-cli-core dev", "bl": "pnpm -F bailian-cli dev", "kscli": "pnpm -F knowledge-studio-cli dev", diff --git a/packages/cli/package.json b/packages/cli/package.json index 17247363..a7b49323 100644 --- a/packages/cli/package.json +++ b/packages/cli/package.json @@ -40,7 +40,7 @@ "registry": "https://registry.npmjs.org/" }, "scripts": { - "generate:reference": "tsx ../../tools/generate-reference.ts", + "generate:reference": "tsx ../../tools/generate-reference.ts && sh -c 'cd ../.. && vp check --fix skills/bailian-cli/reference'", "sync:skill-version": "tsx ../../tools/sync-skill-metadata.ts", "build": "vp pack", "dev": "tsx src/main.ts",