@elvis1513/auto-coding-skill 1.0.1 → 1.0.3

package/README.md

@@ -35,7 +35,7 @@ npm install -g git+https://github.com/elvis1513/auto-coding-skill.git
 - Synced reusable workflow improvements from a production project back into this skill.
 - Moved repo-side helper entrypoint to `docs/tools/autopipeline`.
 - Tightened regression matrix rules: rows start as `TODO`, and `PASS` requires real execution evidence.
-- Added Jenkins API verification flow with credentials sourced from `docs/ENGINEERING.md` or environment variables.
+- Added Jenkins API verification flow with credentials sourced from `docs/ENGINEERING.md`.
 
 ## Optimized Standard Flow
 
@@ -53,12 +53,11 @@ npm install -g git+https://github.com/elvis1513/auto-coding-skill.git
 3. 开发实现
    - 只修改本次任务必要文件，不做无关重构。
 4. 本地轻量校验
-   - build
-   - 单元测试或关键快速测试
-   - lint / typecheck
-   - API 文档检查
-   - Jenkinsfile / 脚本语法检查
+   - 优先执行一个项目自定义快速门禁命令
+   - 若未配置，再执行 quick test / test / build 中最先配置的一项
    - `git diff --check`
+   - API 文档检查
+   - Jenkins 配置检查
 5. 立即提交推送
    - 本地轻量校验通过后，commit + push，触发 Jenkins。
 6. Jenkins 验证
@@ -75,7 +74,7 @@ npm install -g git+https://github.com/elvis1513/auto-coding-skill.git
 默认不做：
 - 本地 Docker Compose 启动
 - 本地 Docker build
-- 本地完整 smoke / regression
+- 本地完整 regression
 - 每个小改动强制 `check-matrix`
 - 每个小改动强制生成 summary
 - 未真实执行就要求 regression matrix 全 `PASS`
@@ -83,7 +82,7 @@ npm install -g git+https://github.com/elvis1513/auto-coding-skill.git
 
 按需保留：
 - `runtime-up` / `runtime-down`
-- 本地 health / smoke / regression
+- 本地 health
 - `check-matrix`
 - `gen-summary`
 - deployment runbook / deployment record
@@ -120,9 +119,7 @@ It must be committed to Git. Do not add it to `.gitignore`.
 
 默认必填：
 - `project.name`
-- `commands.build`
-- `commands.quick_test` 或 `commands.test`
-- `commands.lint` 或 `commands.typecheck`
+- `commands.light_gate` 或 `commands.quick_test` 或 `commands.test` 或 `commands.build`
 - `target_env.name`
 - `target_env.frontend_base_url`
 - `target_env.frontend_username`
@@ -130,6 +127,8 @@ It must be committed to Git. Do not add it to `.gitignore`.
 - `target_env.backend_base_url`
 - `target_env.backend_username`
 - `target_env.backend_password`
+- `target_env.backend_root_username`
+- `target_env.backend_root_password`
 - `target_env.health_base_url`
 - `target_env.health_path`
 - `jenkins.base_url`
@@ -200,6 +199,7 @@ It must be committed to Git. Do not add it to `.gitignore`.
 - How to record:
   - Fill YAML frontmatter once.
   - Keep target env front/backend usernames and passwords, Jenkins UI/API usernames and passwords, commands, docs paths here only.
+  - Target environment also includes backend server root username/password.
   - This file is expected to be committed to Git and maintained in plaintext for this workflow.
   - Remaining environment keys are all mandatory; blank values, TODO-like placeholders, and incorrect URL/path formats are treated as blocking errors by `doctor`.
   - Do not duplicate config elsewhere.
@@ -291,12 +291,6 @@ python3 docs/tools/autopipeline/ap.py commit-push <TASK_ID> \
 Available on-demand commands:
 
 ```bash
-python3 docs/tools/autopipeline/ap.py run build
-python3 docs/tools/autopipeline/ap.py run test
-python3 docs/tools/autopipeline/ap.py run quick_test
-python3 docs/tools/autopipeline/ap.py run lint
-python3 docs/tools/autopipeline/ap.py run typecheck
-python3 docs/tools/autopipeline/ap.py run script_syntax
 python3 docs/tools/autopipeline/ap.py doctor
 python3 docs/tools/autopipeline/ap.py verify-api-docs
 python3 docs/tools/autopipeline/ap.py verify-jenkins
@@ -306,8 +300,6 @@ python3 docs/tools/autopipeline/ap.py verify-jenkins-build --job-url <job-url> -
 python3 docs/tools/autopipeline/ap.py verify-jenkins-build --multibranch-root-job <root-job> --branch-name <branch> --build-number <number>
 python3 docs/tools/autopipeline/ap.py runtime-up
 python3 docs/tools/autopipeline/ap.py wait-health --scope runtime
-python3 docs/tools/autopipeline/ap.py run smoke
-python3 docs/tools/autopipeline/ap.py run regression
 python3 docs/tools/autopipeline/ap.py runtime-down
 python3 docs/tools/autopipeline/ap.py check-matrix
 python3 docs/tools/autopipeline/ap.py gen-summary <TASK_ID>
@@ -332,7 +324,7 @@ python3 docs/tools/autopipeline/ap.py gen-summary <TASK_ID>
 - `doctor`
   - Checks whether the default lightweight workflow is actually configured instead of silently skipping gates.
 - `light-gate`
-  - Now fails if required commands are missing instead of returning `OK` after skipping everything.
+  - Now prefers one curated fast gate command instead of serially running every expensive check.
 - `verify-target`
   - Performs real target-environment verification beyond health checks when you provide key backend/frontend paths.
 - `commit-push --record-closure`

package/cli/assets/skill/SKILL.md

@@ -77,9 +77,7 @@ Do not hide `docs/ENGINEERING.md` in `.gitignore`.
 
 Minimum required config for the default flow:
 - `project.name`
-- `commands.build`
-- `commands.quick_test` or `commands.test`
-- `commands.lint` or `commands.typecheck`
+- `commands.light_gate` or `commands.quick_test` or `commands.test` or `commands.build`
 - `target_env.name`
 - `target_env.frontend_base_url`
 - `target_env.frontend_username`
@@ -87,6 +85,8 @@ Minimum required config for the default flow:
 - `target_env.backend_base_url`
 - `target_env.backend_username`
 - `target_env.backend_password`
+- `target_env.backend_root_username`
+- `target_env.backend_root_password`
 - `target_env.health_base_url`
 - `target_env.health_path`
 - `jenkins.base_url`
@@ -138,8 +138,6 @@ On-demand commands:
 ```bash
 python3 docs/tools/autopipeline/ap.py runtime-up
 python3 docs/tools/autopipeline/ap.py wait-health --scope runtime
-python3 docs/tools/autopipeline/ap.py run smoke
-python3 docs/tools/autopipeline/ap.py run regression
 python3 docs/tools/autopipeline/ap.py runtime-down
 python3 docs/tools/autopipeline/ap.py check-matrix
 python3 docs/tools/autopipeline/ap.py gen-summary <TASK_ID>
@@ -147,9 +145,9 @@ python3 docs/tools/autopipeline/ap.py gen-summary <TASK_ID>
 
 ## Quality policy
 
-- Default local gate is lightweight only: build, unit/quick test, lint, typecheck, API docs, Jenkinsfile / script syntax, `git diff --check`.
+- Default local gate is lightweight and time-bounded: prefer one curated project command via `commands.light_gate`, then run only diff/API/Jenkins checks.
 - `doctor` should be used early to catch missing or invalid config before the first implementation loop.
-- `light-gate` now fails if the required default commands are not configured.
+- `light-gate` now fails if no usable fast gate command is configured.
 - `doctor`, `light-gate`, and `commit-push` all fail when required environment fields are missing, placeholder-like, or syntactically invalid.
 - Do not require local Docker Compose or full local regression for every small change.
 - Jenkins and target environment verification are more valuable than repeated local simulation of deploy-only problems.

package/cli/assets/skill/data/templates/ENGINEERING.md

@@ -10,26 +10,17 @@ project:
   jenkinsfile: "Jenkinsfile"
 
 commands:
+  light_gate: ""
   build: ""
   test: ""
   quick_test: ""
   lint: ""
   typecheck: ""
   format: ""
-  script_syntax: ""
-  diff_check: ""
-  docker_build: ""
-  compose_up: ""
-  compose_down: ""
-  smoke: ""
-  regression: ""
 
 runtime:
   docker_compose_file: ""
   docker_service: ""
-  container_name: ""
-  image: ""
-  app_port: ""
   health_base_url: ""
   health_path: ""
   env_file: ""
@@ -43,6 +34,8 @@ target_env:
   backend_base_url: ""
   backend_username: ""
   backend_password: ""
+  backend_root_username: ""
+  backend_root_password: ""
   health_base_url: ""
   health_path: ""
 
@@ -79,7 +72,7 @@ docs:
 默认原则：
 - 默认不要求本地 Docker Compose 启动。
 - 默认不要求本地 Docker build。
-- 默认不要求本地完整 smoke / regression。
+- 默认不要求本地完整 regression。
 - 默认不要求每个小改动生成长 summary。
 - 默认不要求 regression matrix 全 PASS。
 - 默认不要求 deployment record。
@@ -97,21 +90,20 @@ docs:
 ## 0. 配置填写（必须）
 
 先填写 `docs/ENGINEERING.md` frontmatter 中的所有空值。重点包括：
-- `commands.*`：本地轻量校验命令
+- `commands.light_gate`：推荐配置一个项目级快速门禁命令，作为默认本地校验入口
 - `target_env.*`：目标环境前端 / 后端地址、用户名、密码，必须全部填写且真实可用
 - `jenkins.*`：Jenkins UI/API 用户名、密码、Job、分支、镜像、部署环境，必须全部填写且真实可用
 
 字段说明：
 - `target_env.backend_username` / `target_env.backend_password`：目标环境后台账号
+- `target_env.backend_root_username` / `target_env.backend_root_password`：目标环境后台服务器 root 账号
 - `target_env.frontend_username` / `target_env.frontend_password`：目标环境前端登录账号
 - `jenkins.ui_username` / `jenkins.ui_password`：Jenkins 页面登录账号
 - `jenkins.api_user` / `jenkins.api_password`：Jenkins API 用户名 / 密码
 
 默认必填：
 - `project.name`
-- `commands.build`
-- `commands.quick_test` 或 `commands.test`
-- `commands.lint` 或 `commands.typecheck`
+- `commands.light_gate` 或 `commands.quick_test` 或 `commands.test` 或 `commands.build`
 - `target_env.name`
 - `target_env.frontend_base_url`
 - `target_env.frontend_username`
@@ -119,6 +111,8 @@ docs:
 - `target_env.backend_base_url`
 - `target_env.backend_username`
 - `target_env.backend_password`
+- `target_env.backend_root_username`
+- `target_env.backend_root_password`
 - `target_env.health_base_url`
 - `target_env.health_path`
 - `jenkins.ui_username`
@@ -133,7 +127,7 @@ docs:
 
 按需填写：
 - `runtime.*`：仅在本地运行诊断时使用
-- `commands.compose_up` / `commands.compose_down` / `commands.smoke` / `commands.regression`
+- `commands.build` / `commands.test` / `commands.quick_test` / `commands.lint` / `commands.typecheck` / `commands.format`：按项目实际情况保留
 
 ---
 
@@ -197,13 +191,12 @@ docs:
    只修改本次任务必要文件，不做无关重构。
 
 4. 本地轻量校验  
-   默认只跑：
-   - 编译 / build
-   - 单元测试或关键快速测试
-   - lint / typecheck
-   - API 文档检查
-   - Jenkinsfile / 脚本语法检查
+   默认只跑最少必要检查：
+   - 优先执行 `commands.light_gate`
+   - 若未配置，则执行 `quick_test` / `test` / `build` 中最先配置的一项
    - `git diff --check`
+   - API 文档检查
+   - Jenkins 配置检查
 
 5. 立即提交推送  
    轻量校验通过后，commit + push，触发 Jenkins。
@@ -250,8 +243,6 @@ docs:
 - `runtime-up`
 - `runtime-down`
 - 本地 health check
-- 本地 `smoke`
-- 本地 `regression`
 - `check-matrix`
 - `gen-summary`
 
@@ -271,7 +262,7 @@ docs:
 
 说明：
 - `doctor`：检查默认流程必填项和常见配置错误。
-- `light-gate`：默认轻量门禁。
+- `light-gate`：默认轻量门禁，优先执行项目自定义快速门禁命令。
 - `verify-target`：目标环境健康检查 + 按需关键 API / 页面验证。
 - `record-closure`：默认轻量闭环记录。
 - `check-matrix`、`gen-summary`、`runtime-up/down`：保留为按需工具。

package/cli/assets/skill/data/templates/docs/reviews/_TEMPLATE-REVIEW.md

@@ -3,18 +3,17 @@
 > 仅在任务需要独立 review 记录时使用；不是每个小改动的默认强制文档。
 
 ## 1. 静态分析结果
-- Command：lint / typecheck / script_syntax / diff_check
+- Command：light_gate / diff_check / verify_api_docs / verify_jenkins
 - Summary：
 - Issues：
 
 ## 2. 本地轻量校验
-- build：
-- test or quick_test：
+- light_gate or quick_test/test/build：
 - api docs：
-- jenkinsfile / scripts：
+- jenkins：
 
 ## 3. 按需本地运行验证（如果有）
-- runtime-up / health / smoke / regression：
+- runtime-up / health：
 
 ## 4. Jenkins 与目标环境
 - Jenkins readiness：

package/cli/assets/skill/data/templates/docs/tasks/summaries/_TEMPLATE-TASK-SUMMARY.md

@@ -19,7 +19,7 @@
 - 兼容性影响：
 
 ## 3. 质量证据
-- 本地轻量校验：build / test or quick_test / lint / typecheck / api docs / jenkinsfile / diff-check
+- 本地轻量校验：light_gate or quick_test/test/build / api docs / jenkins / diff-check
 - Jenkins Build：
 - 目标环境验证：
 - 闭环记录：`docs/tasks/closure-log.md`

package/cli/assets/skill/scripts/ap.py

@@ -317,12 +317,12 @@ def _resolve_jenkins_job_candidates(
 ) -> List[str]:
     jenkins_cfg = (cfg.get("jenkins") or {})
     effective_branch = str(branch_name or "").strip()
-    if not effective_branch:
+    effective_root = str(multibranch_root_job or "").strip()
+    if not effective_branch and effective_root:
         inferred_branch = _resolve_git_branch_name(repo, git_ref or "HEAD")
         if inferred_branch:
             effective_branch = inferred_branch
 
-    effective_root = str(multibranch_root_job or "").strip()
     explicit_url = str(job_url or "").strip()
     explicit_name = str(job_name or "").strip()
     configured_url = str(jenkins_cfg.get("job_url") or "").strip()
@@ -423,6 +423,8 @@ def cmd_gen_summary(args: argparse.Namespace) -> None:
     staged = run(["git", "diff", "--cached", "--name-only"], cwd=repo, check=False).stdout.strip()
     unstaged = run(["git", "diff", "--name-only"], cwd=repo, check=False).stdout.strip()
     status = run(["git", "status", "--porcelain=v1"], cwd=repo, check=False).stdout.strip()
+    staged_block = "- " + staged.replace("\n", "\n- ") if staged else "- (none)"
+    unstaged_block = "- " + unstaged.replace("\n", "\n- ") if unstaged else "- (none)"
 
     content = f"""# Task Summary — {task_id} — {title}
 
@@ -442,9 +444,9 @@ def cmd_gen_summary(args: argparse.Namespace) -> None:
 ## 2. 变更概览
 ### Git change snapshot
 - Staged files:
-{('- ' + staged.replace('\n','\n- ')) if staged else '- (none)'}
+{staged_block}
 - Unstaged files:
-{('- ' + unstaged.replace('\n','\n- ')) if unstaged else '- (none)'}
+{unstaged_block}
 - Status:
 ```text
 {status}
@@ -454,7 +456,7 @@ def cmd_gen_summary(args: argparse.Namespace) -> None:
 - 变更记录位置：`{api_change_log}`
 
 ## 4. 质量证据
-- 本地轻量校验：build / test or quick_test / lint / typecheck / api docs / jenkinsfile / diff-check — TODO
+- 本地轻量校验：light_gate or quick_test/test/build / api docs / jenkins / diff-check — TODO
 - Jenkins Build：TODO
 - 目标环境验证：TODO
 - 闭环记录：TODO
@@ -572,14 +574,8 @@ def _require_explicit_field(missing: List[str], field: str, value: object) -> No
 
 
 def _run_git_diff_check(repo: Path, cfg: dict) -> None:
-    commands = (cfg.get("commands") or {})
-    configured = str(commands.get("diff_check") or "").strip()
-    if configured:
-        print(f"[diff-check] {configured}")
-        run_shell(configured, cwd=repo)
-    else:
-        print("[diff-check] git diff --check")
-        run(["git", "diff", "--check"], cwd=repo)
+    print("[diff-check] git diff --check")
+    run(["git", "diff", "--check"], cwd=repo)
     print("[diff-check] OK")
 
 
@@ -613,38 +609,20 @@ def cmd_light_gate(args: argparse.Namespace) -> None:
     executed: List[str] = []
     missing: List[str] = []
 
-    if not str(commands.get("build") or "").strip():
-        missing.append("commands.build")
-    else:
-        _run_configured_command(repo, cfg, "build")
-        executed.append("build")
-
-    if str(commands.get("quick_test") or "").strip():
+    if str(commands.get("light_gate") or "").strip():
+        _run_configured_command(repo, cfg, "light_gate")
+        executed.append("light_gate")
+    elif str(commands.get("quick_test") or "").strip():
         _run_configured_command(repo, cfg, "quick_test")
         executed.append("quick_test")
     elif str(commands.get("test") or "").strip():
         _run_configured_command(repo, cfg, "test")
         executed.append("test")
+    elif str(commands.get("build") or "").strip():
+        _run_configured_command(repo, cfg, "build")
+        executed.append("build")
     else:
-        missing.append("commands.quick_test or commands.test")
-
-    static_executed = False
-    if str(commands.get("lint") or "").strip():
-        _run_configured_command(repo, cfg, "lint")
-        executed.append("lint")
-        static_executed = True
-
-    if str(commands.get("typecheck") or "").strip():
-        _run_configured_command(repo, cfg, "typecheck")
-        executed.append("typecheck")
-        static_executed = True
-
-    if not static_executed:
-        missing.append("commands.lint or commands.typecheck")
-
-    if str(commands.get("script_syntax") or "").strip():
-        _run_configured_command(repo, cfg, "script_syntax")
-        executed.append("script_syntax")
+        missing.append("commands.light_gate or commands.quick_test or commands.test or commands.build")
 
     if missing:
         raise APError(
@@ -664,8 +642,6 @@ def cmd_runtime_up(args: argparse.Namespace) -> None:
     repo = Path(args.repo).resolve()
     cfg = _load_cfg(repo)
     runtime_cfg = (cfg.get("runtime") or {})
-    if _run_configured_command(repo, cfg, "compose_up"):
-        return
     compose_args = _compose_base_args(runtime_cfg) + ["up", "-d"]
     docker_service = str(runtime_cfg.get("docker_service") or "").strip()
     if docker_service:
@@ -679,8 +655,6 @@ def cmd_runtime_down(args: argparse.Namespace) -> None:
     repo = Path(args.repo).resolve()
     cfg = _load_cfg(repo)
     runtime_cfg = (cfg.get("runtime") or {})
-    if _run_configured_command(repo, cfg, "compose_down"):
-        return
     compose_args = _compose_base_args(runtime_cfg) + ["down", "--remove-orphans"]
     print(f"[runtime-down] {' '.join(compose_args)}")
     run(compose_args, cwd=repo)
@@ -811,12 +785,13 @@ def cmd_doctor(args: argparse.Namespace) -> None:
 
     if not str(project_cfg.get("name") or "").strip():
         missing.append("project.name")
-    if not str(commands.get("build") or "").strip():
-        missing.append("commands.build")
-    if not (str(commands.get("quick_test") or "").strip() or str(commands.get("test") or "").strip()):
-        missing.append("commands.quick_test or commands.test")
-    if not (str(commands.get("lint") or "").strip() or str(commands.get("typecheck") or "").strip()):
-        missing.append("commands.lint or commands.typecheck")
+    if not (
+        str(commands.get("light_gate") or "").strip()
+        or str(commands.get("quick_test") or "").strip()
+        or str(commands.get("test") or "").strip()
+        or str(commands.get("build") or "").strip()
+    ):
+        missing.append("commands.light_gate or commands.quick_test or commands.test or commands.build")
     _require_explicit_field(missing, "target_env.name", target_cfg.get("name"))
     _require_explicit_field(missing, "target_env.frontend_base_url", target_cfg.get("frontend_base_url"))
     _require_explicit_field(missing, "target_env.frontend_username", target_cfg.get("frontend_username"))
@@ -824,6 +799,8 @@ def cmd_doctor(args: argparse.Namespace) -> None:
     _require_explicit_field(missing, "target_env.backend_base_url", target_cfg.get("backend_base_url"))
     _require_explicit_field(missing, "target_env.backend_username", target_cfg.get("backend_username"))
     _require_explicit_field(missing, "target_env.backend_password", target_cfg.get("backend_password"))
+    _require_explicit_field(missing, "target_env.backend_root_username", target_cfg.get("backend_root_username"))
+    _require_explicit_field(missing, "target_env.backend_root_password", target_cfg.get("backend_root_password"))
     _require_explicit_field(missing, "target_env.health_base_url", target_cfg.get("health_base_url"))
     _require_explicit_field(missing, "target_env.health_path", target_cfg.get("health_path"))
 
@@ -856,8 +833,8 @@ def cmd_doctor(args: argparse.Namespace) -> None:
     _validate_url_field(warnings, "jenkins.job_url", jenkins_cfg.get("job_url"))
 
     runtime_enabled = any(str(runtime_cfg.get(key) or "").strip() for key in ["docker_compose_file", "docker_service", "health_base_url", "health_path"])
-    if runtime_enabled and not (str(commands.get("compose_up") or "").strip() or str(runtime_cfg.get("docker_compose_file") or "").strip()):
-        warnings.append("runtime config is partially enabled but compose_up or docker_compose_file is missing")
+    if runtime_enabled and not str(runtime_cfg.get("docker_compose_file") or "").strip():
+        warnings.append("runtime config is partially enabled but runtime.docker_compose_file is missing")
 
     try:
         timeout_s = int(jenkins_cfg.get("deploy_timeout_sec") or 0)
@@ -893,7 +870,9 @@ def cmd_verify_jenkins_build(args: argparse.Namespace) -> None:
     max_builds = int(args.max_builds or 20)
     timeout_s = int(args.timeout_sec or 300)
     poll_s = int(args.poll_sec or 5)
-    inferred_branch = _resolve_git_branch_name(repo, git_ref)
+    inferred_branch = ""
+    if args.multibranch_root_job and not args.branch_name:
+        inferred_branch = _resolve_git_branch_name(repo, git_ref)
     branch_hint = str(args.branch_name or inferred_branch or "").strip()
     root_hint = str(
         args.multibranch_root_job

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elvis1513/auto-coding-skill",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "CLI installer for auto-coding-skill (Claude Code + Codex CLI) with Go fullstack + Jenkins workflow support.",
   "type": "module",
   "bin": {
@@ -13,7 +13,7 @@
     "LICENSE"
   ],
   "scripts": {
-    "test": "node -c cli/src/index.js",
+    "test": "node -c cli/src/index.js && python3 -c \"import ast, pathlib; files = ['src/auto-coding-skill/scripts/ap.py', 'src/auto-coding-skill/scripts/core.py', 'src/auto-coding-skill/scripts/http_checks.py', 'cli/assets/skill/scripts/ap.py', 'cli/assets/skill/scripts/core.py', 'cli/assets/skill/scripts/http_checks.py']; [ast.parse(pathlib.Path(rel).read_text(encoding='utf-8'), filename=rel, feature_version=(3, 11)) for rel in files]; print('python311-grammar-ok')\" && python3 -m py_compile src/auto-coding-skill/scripts/ap.py src/auto-coding-skill/scripts/core.py src/auto-coding-skill/scripts/http_checks.py cli/assets/skill/scripts/ap.py cli/assets/skill/scripts/core.py cli/assets/skill/scripts/http_checks.py && find src cli -type d -name '__pycache__' -prune -exec rm -rf {} +",
     "sync-assets": "node cli/src/sync-assets.js",
     "release:check": "npm run sync-assets && npm run test && npm pack --dry-run"
   },