@ultime/codecraft 1.2.0 → 1.2.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 1.2.1 - 2026-10-27
+
+- 修复 pack 自动判定：识别 Nuxt 2 的 `pages/`、Nuxt 3 的 `app/`、uni-app 的 `manifest.json + pages.json`、Taro / uni-app 依赖，以及 workspace 内的嵌套应用。
+- 新增 `trae-cn` 安装目标（`~/.trae-cn/skills`）；`--agent` 顺序改为 generic / claude / cursor / trae / trae-cn，不再默认优先 Trae。
+- 扫描 0 个源文件时输出警告并以 2 退出，避免把空扫描当成通过。
+- README 补充 agent 列表、`--dest`、`Files: 0` 与 Trae CN 的说明。
+
 ## 1.2.0 - 2026-10-27
 
 - 新增 `ignore` 例外配置（A12）：`rule` / `file` / `reason` 必填，`expires` 可选；支持 `*` / `**` / `?` 路径匹配；安全规则 `A21` 不可豁免。

package/README.md

@@ -58,15 +58,19 @@ npx skills-npm setup
 npx @ultime/codecraft install
 ```
 
-自动检测当前项目的 agent，把 `SKILL.md`、`docs/`、`scenarios/`、`prompts/` 复制到 `.agents/skills/codecraft` 和对应 agent 目录（`.claude/skills`、`.trae/skills`、`.cursor/skills`）。
+自动检测当前项目的 agent，把 `SKILL.md`、`docs/`、`scenarios/`、`prompts/` 复制到 `.agents/skills/codecraft` 和对应 agent 目录（`.claude/skills`、`.cursor/skills`、`.trae/skills`、`.trae-cn/skills`）。不传 `--agent` 时只按检测结果安装，不优先任何一家。
 
 ```bash
-npx @ultime/codecraft install --agent claude          # 指定 agent，可逗号分隔
-npx @ultime/codecraft install --global --agent claude # 装到用户目录
-npx @ultime/codecraft install --dry-run               # 只看计划
-npx @ultime/codecraft install --force                 # 覆盖已有目录
+npx @ultime/codecraft install --agent generic          # 指定 agent，可逗号分隔
+npx @ultime/codecraft install --global --agent claude  # 装到用户目录
+npx @ultime/codecraft install --global --agent trae-cn # Trae CN 的全局目录
+npx @ultime/codecraft install --global --dest .trae-cn/skills  # 自定义目录
+npx @ultime/codecraft install --dry-run                # 只看计划
+npx @ultime/codecraft install --force                  # 覆盖已有目录
 ```
 
+可选 agent：`generic` / `claude` / `cursor` / `trae` / `trae-cn`；未知目录用 `--dest`。
+
 ### 方式 C：手动指针
 
 不复制文件，只在项目规则里留一行：
@@ -264,9 +268,11 @@ ROADMAP.md    后续版本计划与交接说明
 ## FAQ
 
 - **静态检查误报怎么办？** 在 `codecraft.config.json` 的 `ignore` 里登记 `rule` + `file` + `reason`，必要时加 `expires`；不要直接删规则。
+- **`Files: 0` 是什么意思？** 没有扫到源文件，命令会输出警告并以 2 退出；用 `--pack app|library` 指定正确类型，或检查 `--root`。
+- **Trae CN 装到哪？** 用 `--agent trae-cn`（`~/.trae-cn/skills`）；未知目录用 `--dest`。
 - **`check` / `audit` / `plan` 有什么区别？** `check` 只跑内置规则；`audit` 额外跑 config 里的 gates；`plan` 只读，把 findings 转成计划，默认不跑 gates。
 - **为什么不自动跑 typecheck / test？** 门禁由项目提供，Codecraft 只编排，避免第二套实现。
-- **monorepo 为什么被判成 library？** 有 `packages/` 或根 `package.json` 即按 library；可用 `--pack` 显式覆盖。
+- **monorepo 为什么被判成 app / library？** 有 `pages/`、`app/`、`manifest.json + pages.json`、Taro / uni-app 依赖或嵌套应用时按 `app`；有 `packages/` 或 `src/` 时按 `library`；仍不确定就显式传 `--pack`。
 - **支持哪些文件？** 只扫 JS/TS 相关扩展名与 `package.json`；其它语言不覆盖。
 
 ## 边界

package/VERSION

@@ -1 +1 @@
-1.2.0
+1.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ultime/codecraft",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "面向 JS/TS 的代码规范与架构设计（前端默认，Node 后端可复用结构规则），作为可安装的 skill 包。",
   "keywords": [
     "ai",

package/src/cli.mjs

@@ -24,9 +24,9 @@ Options:
   --root <path>            Project root (default: cwd)
   --pack app|library|design-system|auto  Rule pack (default: auto)
   --config <path>          Config file (default: codecraft.config.json)
-  --agent auto|claude|trae|cursor|generic  Install target, comma-separated (default: auto)
+  --agent auto|generic|claude|cursor|trae|trae-cn  Install target, comma-separated (default: auto)
   --global                 Install into the home directory
-  --dest <path>            Explicit destination (relative to root / home)
+  --dest <path>            Explicit destination (relative to root / home), e.g. .trae-cn/skills
   --force                  Replace an existing codecraft skill directory
   --dry-run                Show the install plan without writing
   --no-gates               Skip gates in audit / plan
@@ -154,6 +154,11 @@ export function main(argv) {
   const findings = result.findings.map((finding) => ({ ...finding, severity }))
   const filtered = applyIgnores(findings, config.config.ignore ?? [])
   const activeFindings = filtered.findings
+  const warnings = []
+  if (result.files === 0 && result.findings.length === 0) {
+    warnings.push(`0 source files scanned under ${root}; pass --pack app|library or check --root`)
+    console.error(`codecraft: warning: ${warnings[0]}`)
+  }
   const gates = shouldRunGates(command, args)
     ? runGates(root, config.config.gates ?? [], { failFast: args.failFast })
     : []
@@ -170,6 +175,7 @@ export function main(argv) {
     findings: activeFindings.length,
     ignored: filtered.ignored.length,
     expired: filtered.expired.length,
+    warnings,
     plan: plan.length,
     gates: gates.length,
   }
@@ -207,6 +213,7 @@ export function main(argv) {
   const failed =
     activeFindings.some((finding) => finding.severity === 'error') ||
     gates.some((gate) => gate.exitCode !== 0)
+  if (warnings.length) return 2
   return failed ? 1 : 0
 }
 
@@ -281,13 +288,83 @@ function parseArgs(argv) {
   return args
 }
 
-function detectPack(root) {
-  if (isDir(path.join(root, 'apps')) || isDir(path.join(root, 'src', 'pages'))) return 'app'
+export function detectPack(root) {
+  if (isDir(path.join(root, 'apps'))) return 'app'
+  if (hasAppMarkers(root)) return 'app'
   if (isDir(path.join(root, 'packages'))) return 'library'
+  if (hasNestedApp(root)) return 'app'
+  if (isDir(path.join(root, 'src'))) return 'library'
   if (fs.existsSync(path.join(root, 'package.json'))) return 'library'
   return 'app'
 }
 
+const APP_DIR_MARKERS = ['src/pages', 'pages', 'app', 'src/app']
+const APP_FILE_MARKERS = [
+  'nuxt.config.js',
+  'nuxt.config.ts',
+  'nuxt.config.mjs',
+  'app.config.js',
+  'app.config.ts',
+  'src/app.config.js',
+  'src/app.config.ts',
+  'taro.config.js',
+  'taro.config.ts',
+]
+const APP_DEPS = ['nuxt', '@tarojs/taro', '@dcloudio/uni-app', 'uni-app', 'next']
+
+function hasAppMarkers(root) {
+  if (APP_DIR_MARKERS.some((dir) => isDir(path.join(root, dir)))) return true
+  if (APP_FILE_MARKERS.some((file) => isFile(path.join(root, file)))) return true
+  if (hasUniAppFiles(root)) return true
+  return hasAppDependency(root)
+}
+
+function hasNestedApp(root) {
+  let entries
+  try {
+    entries = fs.readdirSync(root, { withFileTypes: true })
+  } catch {
+    return false
+  }
+  for (const entry of entries) {
+    if (!entry.isDirectory()) continue
+    if (['node_modules', 'dist', 'build', 'target', '.git', '.nuxt', '.output', 'coverage'].includes(entry.name)) {
+      continue
+    }
+    const child = path.join(root, entry.name)
+    if (APP_DIR_MARKERS.some((dir) => isDir(path.join(child, dir)))) return true
+    if (APP_FILE_MARKERS.some((file) => isFile(path.join(child, file)))) return true
+    if (hasUniAppFiles(child)) return true
+    if (hasAppDependency(child)) return true
+  }
+  return false
+}
+
+function hasUniAppFiles(root) {
+  const atRoot = isFile(path.join(root, 'manifest.json')) && isFile(path.join(root, 'pages.json'))
+  const atSrc = isFile(path.join(root, 'src', 'manifest.json')) && isFile(path.join(root, 'src', 'pages.json'))
+  return atRoot || atSrc
+}
+
+function hasAppDependency(root) {
+  let manifest
+  try {
+    manifest = JSON.parse(fs.readFileSync(path.join(root, 'package.json'), 'utf8'))
+  } catch {
+    return false
+  }
+  const deps = { ...(manifest.dependencies ?? {}), ...(manifest.devDependencies ?? {}) }
+  return APP_DEPS.some((dep) => dep in deps)
+}
+
+function isFile(target) {
+  try {
+    return fs.statSync(target).isFile()
+  } catch {
+    return false
+  }
+}
+
 function fail(message) {
   console.error(`codecraft: ${message}\n\n${HELP}`)
   process.exit(2)
@@ -482,6 +559,7 @@ function header(meta) {
     `- Findings: ${meta.findings}`,
     `- Ignored: ${meta.ignored ?? 0}`,
     `- Expired exceptions: ${meta.expired ?? 0}`,
+    `- Warnings: ${meta.warnings?.length ?? 0}`,
     `- Gates: ${meta.gates}`,
     '',
   ]

package/src/install.mjs

@@ -8,17 +8,19 @@ const SKILL_FILES = ['SKILL.md']
 const SKILL_DIRS = ['docs', 'scenarios', 'prompts']
 
 const AGENT_DIRS = {
+  generic: '.agents/skills',
   claude: '.claude/skills',
-  trae: '.trae/skills',
   cursor: '.cursor/skills',
-  generic: '.agents/skills',
+  trae: '.trae/skills',
+  'trae-cn': '.trae-cn/skills',
 }
 
 const AGENT_MARKERS = {
+  generic: ['.agents'],
   claude: ['.claude', 'CLAUDE.md'],
-  trae: ['.trae'],
   cursor: ['.cursor'],
-  generic: ['.agents'],
+  trae: ['.trae'],
+  'trae-cn': ['.trae-cn'],
 }
 
 export const AGENTS = Object.keys(AGENT_DIRS)