harveyz-skill 0.2.1 → 0.3.0

package/skills/harness/git-workflow-init/references/workflow-config.yml

@@ -0,0 +1,99 @@
+# workflow-config.yml
+# Git 工作流配置文件 — 由 /git-workflow-init skill 读取、审核并部署
+#
+# 使用方式：将此文件复制到项目根目录，按需修改后运行 /git-workflow-init
+#
+# 每个节对应一个 git hook，注释掉整节 = 不安装该 hook
+
+# ── 元信息 ────────────────────────────────────────────────────────────────────
+meta:
+  name: "my-project-workflow"
+  version: "1.0.0"
+  # preset 决定各节的默认值，可在下方局部覆盖
+  # 可选值：gitflow | github-flow | trunk-based | custom
+  preset: gitflow
+
+# ── 分支拓扑与保护（生成 pre-commit hook）────────────────────────────────────
+#
+# allow_direct_commit: false → 直接提交时 exit 1（硬拦截）
+# allow_direct_commit: true  → 直接提交时仅打印警告，不阻断
+# merge_from 使用 shell glob 语法（* 匹配任意字符，不含 /）
+branches:
+  protected:
+    - name: main
+      allow_direct_commit: false
+      merge_from:
+        - staging
+        - "release/*"
+
+    - name: staging
+      allow_direct_commit: false
+      merge_from:
+        - "feature/*"
+        - "fix/*"
+        - "chore/*"
+        - "doc/*"
+
+# ── 分支命名规范（生成 post-checkout hook）───────────────────────────────────
+#
+# enforce: true  → 切换到不合规分支时打印警告（advisory，不阻断）
+# enforce: false → 跳过，不安装此 hook
+# allowed_patterns 使用 POSIX ERE 语法
+branch_naming:
+  enforce: true
+  allowed_patterns:
+    - "^feature/.+"
+    - "^fix/.+"
+    - "^chore/.+"
+    - "^doc/.+"
+    - "^release/[0-9]+\\.[0-9]+\\.[0-9]+(-.+)?$"
+  exempt:                      # 永远豁免命名检查的分支
+    - main
+    - staging
+
+# ── 提交信息格式（生成 commit-msg hook）──────────────────────────────────────
+#
+# format 可选值：
+#   conventional → 使用下方 conventional 节的设置
+#   regex        → 使用自定义 pattern 字段（需同时填写 pattern）
+#   none         → 不检查格式（但若 enforce: true 会安装空 hook）
+commit_message:
+  enforce: true
+  format: conventional
+  # format: regex 时启用此行：
+  # pattern: "^(JIRA-\\d+|NOJIRA): .{1,80}"
+  conventional:
+    types:
+      - feat
+      - fix
+      - chore
+      - docs
+      - refactor
+      - test
+      - style
+      - perf
+    require_scope: false         # true = feat(scope): msg 中 scope 必填
+    max_subject_length: 80
+
+# ── Tag 命名规范（生成 pre-push hook 的 tag 段）───────────────────────────────
+#
+# push_guard: true → 推送不合规 tag 时 exit 1
+# allowed_patterns 使用 POSIX ERE 语法，满足其中一个即通过
+# require_annotated: true → 强制使用 git tag -a，拒绝 lightweight tag
+tags:
+  enforce: true
+  push_guard: true
+  allowed_patterns:
+    - "^v[0-9]+\\.[0-9]+\\.[0-9]+$"           # 正式版：v1.2.3
+    - "^v[0-9]+\\.[0-9]+\\.[0-9]+-.+$"         # 预发布：v1.2.3-beta.1、v1.2.3-rc.2
+  require_annotated: true
+
+# ── 推送限制（生成 pre-push hook 的分支段）───────────────────────────────────
+#
+# block_force_push 列出的分支，force push 时 exit 1
+# enforce: false 时整节跳过，不安装 pre-push hook（除非 tags 节已启用）
+push_rules:
+  enforce: true
+  block_force_push:
+    - main
+    - staging

package/skills-index.json

@@ -8,7 +8,7 @@
   "bundleMeta": {
     "analysis": "分析工具（skill-analyzer）",
     "brainstorming": "设计与规划工具（brainstorming + writing-plans）",
-    "dev": "开发工作流（executing-plans + systematic-debugging + using-git-worktrees）",
+    "dev": "开发工作流（executing-plans + systematic-debugging + using-git-worktrees + git-workflow-init）",
     "document": "文档工具（diataxis-docs）",
     "harness": "测试工具（full-stack-debug-env）",
     "task": "任务管理（pm-task-dispatch + task-close）",
@@ -20,6 +20,7 @@
     { "path": "harness/diataxis-docs", "bundle": "document" },
     { "path": "harness/full-stack-debug-env", "bundle": "harness" },
     { "path": "superpowers-fork/brainstorming", "bundle": "brainstorming" },
+    { "path": "harness/git-workflow-init", "bundle": "dev" },
     { "path": "superpowers-fork/executing-plans", "bundle": "dev" },
     { "path": "superpowers-fork/systematic-debugging", "bundle": "dev" },
     { "path": "superpowers-fork/using-git-worktrees", "bundle": "dev" },

package/tools/p-launch/p-launch.sh

@@ -68,18 +68,43 @@ _launch() {
   local path="$1"
   local name="${path:t}"
   local display="${path/$HOME/~}"
-  local cursor_ok=false ghostty_ok=false
+  local cursor_ok=false ghostty_ok=false ghostty_err="not installed"
 
-  # Cursor IDE
+  # Cursor IDE — CLI first, fall back to .app
   if command -v cursor &>/dev/null; then
     cursor "$path" &>/dev/null &
     cursor_ok=true
+  elif [[ -d "/Applications/Cursor.app" ]]; then
+    /usr/bin/open -na "Cursor" --args "$path" && cursor_ok=true
   fi
 
-  # Ghostty — force a new window at the project directory
-  if [[ -d "/Applications/Ghostty.app" ]]; then
-    open -na "Ghostty" --args --working-directory="$path"
-    ghostty_ok=true
+  # Ghostty — find app via Spotlight (handles /Applications, ~/Applications,
+  # or any custom install path), then invoke "New Ghostty Window Here" service
+  # via NSPerformService — same code path as Finder right-click service.
+  local ghostty_app
+  ghostty_app=$(mdfind "kMDItemCFBundleIdentifier == 'com.mitchellh.ghostty'" 2>/dev/null | /usr/bin/head -1)
+  if [[ -z "$ghostty_app" ]]; then
+    # mdfind can miss apps before first Spotlight index; fall back to known paths
+    for _p in "/Applications/Ghostty.app" "$HOME/Applications/Ghostty.app"; do
+      [[ -d "$_p" ]] && { ghostty_app="$_p"; break }
+    done
+  fi
+
+  if [[ -n "$ghostty_app" ]]; then
+    ghostty_err="failed to open"
+    # Ghostty's service handler does dirname on the path, so pass a child of
+    # the target dir — dirname(target/child) == target.
+    local _child service_path
+    _child=$(/bin/ls -1A "$path" 2>/dev/null | /usr/bin/head -1)
+    service_path="${path}/${_child}"
+    ${_OSASCRIPT:-/usr/bin/osascript} 2>/dev/null <<OSASCRIPT && ghostty_ok=true
+use framework "AppKit"
+use scripting additions
+set thePboard to current application's NSPasteboard's generalPasteboard()
+thePboard's clearContents()
+thePboard's setPropertyList:{"${service_path}"} forType:"NSFilenamesPboardType"
+return current application's NSPerformService("New Ghostty Window Here", thePboard)
+OSASCRIPT
   fi
 
   # ── Launch Report ──────────────────────────────────────────────────────────
@@ -96,7 +121,7 @@ _launch() {
   if $ghostty_ok; then
     printf "  ${C[gr]}✓${C[rs]} Ghostty  new window opened\n"
   else
-    printf "  ${C[yl]}⚠${C[rs]} Ghostty  /Applications/Ghostty.app not found\n"
+    printf "  ${C[yl]}⚠${C[rs]} Ghostty  %s\n" "$ghostty_err"
   fi
 
   printf '\n'

package/tools/p-launch/tests/unit.bats

@@ -6,7 +6,8 @@ setup() {
   SCRIPT="$(cd "${BATS_TEST_DIRNAME}/.." && pwd)/p-launch.sh"
   TEST_DIR="$(mktemp -d)"
   MOCK_HOME="${TEST_DIR}/home"
-  mkdir -p "${MOCK_HOME}"
+  MOCK_BIN="${TEST_DIR}/bin"
+  mkdir -p "${MOCK_HOME}" "${MOCK_BIN}"
 }
 
 teardown() {
@@ -149,3 +150,79 @@ _src() {
   [ "$status" -eq 0 ]
   [ "${#lines[@]}" -eq 2 ]
 }
+
+# ── _launch: Cursor ────────────────────────────────────────────────────────────
+
+_launch_src() {
+  local code="$1"
+  # _OSASCRIPT lets tests inject a mock without relying on PATH,
+  # since p-launch.sh calls /usr/bin/osascript via full path.
+  local mock_osascript="${MOCK_BIN}/osascript"
+  local osascript_override=""
+  [[ -x "$mock_osascript" ]] && osascript_override="export _OSASCRIPT='${mock_osascript}'"
+  zsh -c "
+    export _P_LAUNCH_TEST=1
+    export HOME='${MOCK_HOME}'
+    export PATH='${MOCK_BIN}:${PATH}'
+    ${osascript_override}
+    source '${SCRIPT}'
+    ${code}
+  "
+}
+
+@test "_launch: reports ✓ Cursor when cursor CLI is in PATH" {
+  printf '#!/bin/sh\nexit 0\n' > "${MOCK_BIN}/cursor"
+  chmod +x "${MOCK_BIN}/cursor"
+
+  run _launch_src "_launch '${TEST_DIR}'"
+  [ "$status" -eq 0 ]
+  [[ "$output" == *"✓"*"Cursor"* ]]
+}
+
+@test "_launch: reports ⚠ Cursor when CLI absent and Cursor.app not found" {
+  [[ ! -d "/Applications/Cursor.app" ]] || skip "Cursor.app installed — cannot test absence"
+
+  run _launch_src "_launch '${TEST_DIR}'"
+  [ "$status" -eq 0 ]
+  [[ "$output" == *"⚠"*"Cursor"* ]]
+}
+
+# ── _launch: Ghostty ───────────────────────────────────────────────────────────
+
+@test "_launch: reports ✓ Ghostty when osascript succeeds" {
+  [[ -d "/Applications/Ghostty.app" ]] || skip "Ghostty.app not installed"
+  printf '#!/bin/sh\nexit 0\n' > "${MOCK_BIN}/osascript"
+  chmod +x "${MOCK_BIN}/osascript"
+
+  run _launch_src "_launch '${TEST_DIR}'"
+  [ "$status" -eq 0 ]
+  [[ "$output" == *"✓"*"Ghostty"* ]]
+}
+
+@test "_launch: reports ⚠ Ghostty when osascript fails" {
+  [[ -d "/Applications/Ghostty.app" ]] || skip "Ghostty.app not installed"
+  printf '#!/bin/sh\nexit 1\n' > "${MOCK_BIN}/osascript"
+  chmod +x "${MOCK_BIN}/osascript"
+
+  run _launch_src "_launch '${TEST_DIR}'"
+  [ "$status" -eq 0 ]
+  [[ "$output" == *"⚠"*"Ghostty"* ]]
+}
+
+@test "_launch: reports ⚠ Ghostty when Ghostty.app not found" {
+  [[ ! -d "/Applications/Ghostty.app" ]] || skip "Ghostty.app installed — cannot test absence"
+
+  run _launch_src "_launch '${TEST_DIR}'"
+  [ "$status" -eq 0 ]
+  [[ "$output" == *"⚠"*"Ghostty"* ]]
+}
+
+@test "_launch: osascript receives working directory in script" {
+  [[ -d "/Applications/Ghostty.app" ]] || skip "Ghostty.app not installed"
+  printf '#!/bin/sh\nprintf "ARGS:%%s\n" "$@"\nexit 0\n' \
+    > "${MOCK_BIN}/osascript"
+  chmod +x "${MOCK_BIN}/osascript"
+
+  run _launch_src "_launch '${TEST_DIR}'"
+  [[ "$output" == *"${TEST_DIR}"* ]]
+}