Merge pull request #5 from full-stack-plugins/feat/detect-lang-splits-v0.6.1

feat: 拆分 detect_lang.py 职责边界 + manifest bundle 测试动态化 (v0.6.1)

Author: loong10k

.agents/plugins/marketplace.json

@@ -9,20 +9,20 @@
       "source": {
         "source": "url",
         "url": "https://github.com/partme-ai/partme-codeguard-plugin.git",
-        "ref": "v0.6.0"
+        "ref": "v0.6.1"
       },
       "policy": {
         "installation": "AVAILABLE",
         "authentication": "ON_USE"
       },
       "category": "Developer Tools",
-      "version": "0.6.0",
+      "version": "0.6.1",
       "description": "Cross-language code lint enforcement for AI coding assistants (ZCode, Claude Code, Codex CLI, Kimi Code): Java, Rust, TypeScript, Python. PostToolUse hook auto-runs the native linter on every AI-written file and blocks on failure in strict mode.",
-      "icon": "https://cdn.jsdelivr.net/gh/full-stack-plugins/codeguard-plugin@v0.6.0/assets/official-logo.png",
+      "icon": "https://cdn.jsdelivr.net/gh/full-stack-plugins/codeguard-plugin@v0.6.1/assets/official-logo.png",
       "interface": {
         "displayName": "代码规范守卫",
         "shortDescription": "Make AI-written code pass lint on first try",
-        "logo": "https://cdn.jsdelivr.net/gh/full-stack-plugins/codeguard-plugin@v0.6.0/assets/official-logo.png"
+        "logo": "https://cdn.jsdelivr.net/gh/full-stack-plugins/codeguard-plugin@v0.6.1/assets/official-logo.png"
       }
     }
   ]

.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "codeguard",
-  "version": "0.6.0+codex.20260921",
+  "version": "0.6.1+codex.20260922",
   "description": "Cross-language code lint enforcement for AI coding assistants (ZCode, Claude Code, Codex CLI, Kimi Code): Java, Rust, TypeScript, Python. PostToolUse hook auto-runs the native linter on every AI-written file and blocks on failure in strict mode.",
   "author": {
     "name": "Full Stack Skills / PartMe.AI",

.zcode-plugin/plugin.json

@@ -5,7 +5,7 @@
     "en": "CodeGuard",
     "zh-CN": "代码规范检查"
   },
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "Cross-language code lint enforcement for AI coding assistants (ZCode, Claude Code, Codex CLI, Kimi Code): Java, Rust, TypeScript, Python. PostToolUse hook auto-runs the native linter on every AI-written file and blocks on failure in strict mode.",
   "description_i18n": {
     "en": "Cross-language code lint enforcement. PostToolUse hook auto-runs the language-specific linter on every AI-written file; failed lint blocks further writes when strict_mode is on. Ships ready-to-go .pre-commit-config.yaml templates for Java/Rust/TypeScript/Python.",

kimi.plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "codeguard",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "Cross-language code lint enforcement for AI coding assistants (ZCode, Claude Code, Codex CLI, Kimi Code): Java, Rust, TypeScript, Python. PostToolUse hook auto-runs the native linter on every AI-written file and blocks on failure in strict mode.",
   "author": {
     "name": "Full Stack Skills / PartMe.AI"

openspec/changes/archive/2026-09-22-add-detect-lang-splits/.openspec.yaml

@@ -0,0 +1,2 @@
+schema: spec-driven
+created: 2026-09-22

openspec/changes/archive/2026-09-22-add-detect-lang-splits/design.md

@@ -0,0 +1,43 @@
+## Context
+
+实测证据（code-review-graph 2026-09-22 build，HEAD `5591125`）：
+- 社区数 3：scripts-scan(42 节点) / hooks-gate(39 节点) / tests-skill(36 节点)。
+- 跨社区边 25，其中 19 条 CALLS 全部指向 `scripts/detect_lang.py` 的同一组函数：
+  - `ensure_user_path`（4 处，env_check / post_tool_lint / pre_tool_git_guard / user_prompt_validator）
+  - `detect_languages` / `detect_language`（5 处）
+  - `find_project_root`（3 处）
+  - `probe_toolchain`（5 处）
+  - `project_uses_linter`（5 处）
+  - `load_user_config`（4 处，含 user_prompt_validator / pre_tool_git_guard / post_tool_lint / env_check）
+
+耦合来源是三职责合一：所有 hook 既要 PATH 补齐（属于进程环境），又要项目配置（属于用户态），又要语言识别（属于注册表）。后续若新增 hook / 修改 PATH 策略 / 改配置 schema 都会让 hooks 全军覆没。
+
+## Decisions
+
+### 三个文件按职责一刀切
+
+- `scripts/paths.py`：仅 `ensure_user_path` + 内部静态目录与 fallback。依赖：std + `Path`。无项目根、无 JSON 解析。
+- `scripts/user_config.py`：`load_user_config` / `load_project_overrides` / `get_overrides`。依赖：std + JSON + `Path`。无 subprocess。
+- `scripts/detect_lang.py`：保留 `detect_languages` / `detect_language` / `find_project_root` / `probe_toolchain` / `project_uses_linter` / `extract_tool_binaries` / `LANG_COMMANDS` / `_load_registry`。依赖注册表 + subprocess。
+
+### 外部 API 不变
+
+`detect_lang.py` 顶部 `from paths import ensure_user_path` 与 `from user_config import load_user_config, load_project_overrides, get_overrides`，并 `__all__` 暴露全部历史符号。已有 6 处 hooks + 1 处 `fix.py` + 1 处 `run_check.py` + `tests/run_all.py` 全部无需改动。
+
+### 调用方注释化
+
+每个 hook 的 `from detect_lang import ...` 行末尾加注释 `# 实际定义见 scripts/paths.py / user_config.py`，便于未来定位到正确文件。
+
+### 不动锁与注册表
+
+`scripts/languages.json` 与 `skills.lock.json` 不变；拆分是纯结构调整，不影响任何语言识别结果。
+
+### 不下沉到插件本地技能
+
+PATH / 用户配置是 Hook 必需的运行时环境，与「面向用户的技能」无关——下沉到 `skills/` 会让 import 路径变得奇怪且增加发现成本。
+
+## Risks / Trade-offs
+
+- **首次 import 多一层**：`ensure_user_path` 调用点不变，但 Python 解释器多解析一个小文件（<10ms），可忽略。
+- **`detect_lang.py` 顶端 re-export 列表易过期**：新增函数时若忘记在 `detect_lang.py` 加 re-export，hook 会立刻 ImportError——这是设计意图（fail-fast），但需在 PR review 时人工检查。
+- **测试夹具路径**：`tests/run_all.py` 用 `sys.path.insert(0, str(PLUGIN / "scripts"))`，新文件位置在同一目录，零改动。

openspec/changes/archive/2026-09-22-add-detect-lang-splits/proposal.md

@@ -0,0 +1,24 @@
+## Why
+
+`scripts/detect_lang.py`（401 行）当前承担三职责合一：① 进程 PATH 补齐（`ensure_user_path` + 内部 helper）、② 用户/项目级配置加载（`load_user_config` / `load_project_overrides` / `get_overrides`）、③ 语言注册表与命令表（其余）。code-review-graph 已记录 hooks↔scripts 高耦合告警（19 条 CALLS 边集中指向 `detect_lang.py`），三职责合一让新增/修改任意一条职责都会触发全部 hooks 的回归面。
+
+## What Changes
+
+- 新增 `scripts/paths.py`：仅含 `ensure_user_path(from_login_shell=False) -> None` 及其内部 helper；零业务依赖。
+- 新增 `scripts/user_config.py`：仅含 `load_user_config` / `load_project_overrides` / `get_overrides`；与 hooks/commands 共享。
+- `scripts/detect_lang.py` 改为 re-export 上述符号以保持外部 API 不变（`from detect_lang import ensure_user_path, load_user_config` 仍工作），同时删除实现正文。
+- 所有 hooks（`env_check.py` / `pre_tool_git_guard.py` / `post_tool_lint.py` / `user_prompt_validator.py` / `stop_summary.py`）继续走 `detect_lang` re-export；显式注释每个 import 来自哪个新文件，便于后续单独演进。
+
+## Capabilities
+
+### New Capabilities
+
+- `language-gate-commands`: Defines the responsibility split between PATH handling, user configuration loading, and language detection command tables.
+
+### Modified Capabilities
+
+None.
+
+## Impact
+
+新增 2 个脚本文件（`scripts/paths.py` / `scripts/user_config.py`，预计合计 ~150 行）；修改 `scripts/detect_lang.py`（约 400 行缩到 ~250 行，但保持 API）；hooks 无外部行为变化，仅加注释。`scripts/languages.json` 与 OpenSpec 现有 `language-gate-commands` / `language-registry-doc-sync` 两个 spec 不变。code-review-graph 的 hooks↔scripts 跨社区边应从 19 降至 ≤4。

openspec/changes/archive/2026-09-22-add-detect-lang-splits/specs/language-gate-commands/spec.md

@@ -0,0 +1,33 @@
+## ADDED Requirements
+
+### Requirement: Process PATH enrichment SHALL live in scripts/paths.py
+
+The function `ensure_user_path(from_login_shell: bool = False) -> None` SHALL be defined in `scripts/paths.py` and re-exported from `scripts/detect_lang.py` so existing call sites continue to work without modification.
+
+#### Scenario: Hooks import ensure_user_path from detect_lang
+
+- **WHEN** any hook script imports `ensure_user_path` from `scripts/detect_lang`
+- **THEN** the import resolves to the symbol defined in `scripts/paths.py` without behavioral change
+
+#### Scenario: Direct import from scripts/paths
+
+- **WHEN** a new script needs to enrich PATH without depending on language detection
+- **THEN** it can `from paths import ensure_user_path` with no other dependencies required
+
+### Requirement: User configuration loading SHALL live in scripts/user_config.py
+
+The functions `load_user_config`, `load_project_overrides`, and `get_overrides` SHALL be defined in `scripts/user_config.py` and re-exported from `scripts/detect_lang.py`.
+
+#### Scenario: Hooks import load_user_config from detect_lang
+
+- **WHEN** any hook script imports configuration helpers from `scripts/detect_lang`
+- **THEN** the import resolves to the symbol defined in `scripts/user_config.py`
+
+### Requirement: scripts/detect_lang.py SHALL retain language-detection surface only
+
+`scripts/detect_lang.py` SHALL expose only the language-detection and command-table API: `LANG_COMMANDS`, `detect_languages`, `detect_language`, `find_project_root`, `probe_toolchain`, `project_uses_linter`, `extract_tool_binaries`, and the internal `_load_registry`. It SHALL NOT define PATH enrichment or configuration helpers in its own body.
+
+#### Scenario: Refactor introduces a third responsibility
+
+- **WHEN** a new function in `scripts/detect_lang.py` mixes PATH handling, config loading, or other unrelated concerns with language detection
+- **THEN** the change SHALL be rejected because it violates the single-responsibility boundary

openspec/changes/archive/2026-09-22-add-detect-lang-splits/tasks.md

@@ -0,0 +1,32 @@
+## 1. New files
+
+- [ ] 1.1 Create `scripts/paths.py` with `ensure_user_path(from_login_shell=False)` plus its internal static-dir and node-availability helpers, copied verbatim from `scripts/detect_lang.py`.
+- [ ] 1.2 Create `scripts/user_config.py` with `load_user_config`, `load_project_overrides`, `get_overrides`, copied verbatim from `scripts/detect_lang.py`.
+
+## 2. detect_lang.py refactor
+
+- [ ] 2.1 Delete the implementation bodies of `ensure_user_path` and configuration helpers from `scripts/detect_lang.py`.
+- [ ] 2.2 Replace them with re-exports (`from paths import ensure_user_path`, `from user_config import load_user_config, load_project_overrides, get_overrides`) and add `__all__` enumerating all externally imported symbols (existing + new re-exports).
+
+## 3. Hooks annotation
+
+- [ ] 3.1 In each of `hooks/env_check.py`, `hooks/pre_tool_git_guard.py`, `hooks/post_tool_lint.py`, `hooks/user_prompt_validator.py`, append an inline comment to the existing `from detect_lang import ...` line noting the source file (`paths.py` or `user_config.py`).
+
+## 4. Validation
+
+- [ ] 4.1 Run `python3 tests/run_all.py` — all hook simulation, language-registry, unit, CVE, edges, and field-regression subsets must pass.
+- [ ] 4.2 Run `python3 -m unittest discover -s tests -p 'test_*.py' -v` — `test_plugin_manifests.py` and `test_skill_vendor.py` must pass (pre-existing baseline).
+- [ ] 4.3 Run `git diff --check` — must be clean.
+- [ ] 4.4 Run `openspec validate --all --strict` — must pass (3 pre-existing specs + new `language-gate-commands`).
+- [ ] 4.5 Run `ruff check scripts/paths.py scripts/user_config.py scripts/detect_lang.py hooks/` — zero lint issues.
+
+## 5. Graph evidence
+
+- [ ] 5.1 Rebuild `code-review-graph` and verify the cross-community edge count between `hooks-gate` and `scripts-scan` drops to ≤4 (was 19).
+
+## 6. Release
+
+- [ ] 6.1 `node scripts/bump-plugin.mjs codeguard patch` — bump 0.6.0 → 0.6.1.
+- [ ] 6.2 Commit plugin repo change, push `main` + tag `v0.6.1`.
+- [ ] 6.3 In `full-stack-plugins` market repo, sync catalog `codeguard` to `0.6.1` and regenerate three marketplace manifests; commit and push.
+- [ ] 6.4 `openspec archive add-detect-lang-splits` after all tasks above are checked.

openspec/changes/archive/2026-09-22-add-manifest-bundle-dynamic/.openspec.yaml

@@ -0,0 +1,2 @@
+schema: spec-driven
+created: 2026-09-22