oh-my-harness 0.6.1 → 0.7.0

package/README.md

@@ -122,7 +122,7 @@ oh-my-harness automatically detects your project type and injects accurate facts
 | Language | Detection | Commands |
 |----------|-----------|----------|
 | 🟦 TypeScript/JS | package.json, tsconfig | pnpm/npm/yarn test, eslint, tsc |
-| 🐍 Python | pyproject.toml, requirements.txt | pytest, ruff, mypy |
+| 🐍 Python | pyproject.toml, requirements.txt, Pipfile, manage.py, .python-version | pytest, ruff, black, isort, mypy |
 | 🍎 Swift | Package.swift, .xcodeproj | swift test, xcodebuild |
 | 🦀 Rust | Cargo.toml | cargo test, cargo clippy |
 | 🐹 Go | go.mod | go test, golangci-lint |
@@ -154,7 +154,7 @@ All enforcement is powered by **catalog blocks** — reusable, parameterized hoo
 | ✏️ `lint-on-save` | auto-fix | Auto-lint on file save |
 | 🎨 `format-on-save` | auto-fix | Auto-format on file save |
 | 🔀 `auto-pr` | automation | Auto-create PR after push |
-| 🧪 `tdd-guard` | quality | Blocks source edits unless test modified first (JS/TS) |
+| 🧪 `tdd-guard` | quality | Blocks source edits unless test modified first (JS/TS/Python) |
 
 ### Usage in `harness.yaml`
 
@@ -331,7 +331,7 @@ oh-my-harness/
 │       ├── parse-intent.ts     # claude -p integration
 │       └── prompt-templates.ts # LLM prompt construction
 ├── presets/                # Built-in preset definitions
-└── tests/                  # 712+ tests (unit + integration)
+└── tests/                  # 745+ tests (unit + integration)
 ```
 
 ---

package/dist/catalog/blocks/command-guard.js

@@ -21,9 +21,9 @@ set -euo pipefail
 INPUT=$(cat)
 COMMAND=$(echo "$INPUT" | jq -r '.tool_input.command // empty' 2>/dev/null)
 [[ -z "$COMMAND" ]] && exit 0
-PATTERNS=({{#each patterns}}"{{this}}" {{/each}})
+PATTERNS=({{#each patterns}}"{{{this}}}" {{/each}})
 for PATTERN in "\${PATTERNS[@]}"; do
-  if echo "$COMMAND" | grep -qF "$PATTERN"; then
+  if echo "$COMMAND" | grep -qF -- "$PATTERN"; then
     _log_event "block" "oh-my-harness: command matches blocked pattern: $PATTERN"
     echo "{\\"decision\\": \\"block\\", \\"reason\\": \\"oh-my-harness: command matches blocked pattern: $PATTERN\\"}"
     exit 0

package/dist/catalog/blocks/commit-test-gate.js

@@ -15,8 +15,8 @@ set -euo pipefail
 INPUT=$(cat)
 COMMAND=$(echo "$INPUT" | jq -r '.tool_input.command // empty' 2>/dev/null)
 if echo "$COMMAND" | grep -qE "git commit"; then
-  echo "oh-my-harness: Running {{testCommand}} before commit..." >&2
-  if ! {{testCommand}} >&2 2>&1; then
+  echo "oh-my-harness: Running {{{testCommand}}} before commit..." >&2
+  if ! {{{testCommand}}} >&2 2>&1; then
     _log_event "block" "oh-my-harness: pre-commit check failed"
     echo "{\\"decision\\": \\"block\\", \\"reason\\": \\"oh-my-harness: pre-commit check failed\\"}"
     exit 0

package/dist/catalog/blocks/commit-typecheck-gate.js

@@ -15,8 +15,8 @@ set -euo pipefail
 INPUT=$(cat)
 COMMAND=$(echo "$INPUT" | jq -r '.tool_input.command // empty' 2>/dev/null)
 if echo "$COMMAND" | grep -qE "git commit"; then
-  echo "oh-my-harness: Running {{typecheckCommand}} before commit..." >&2
-  if ! {{typecheckCommand}} >&2 2>&1; then
+  echo "oh-my-harness: Running {{{typecheckCommand}}} before commit..." >&2
+  if ! {{{typecheckCommand}}} >&2 2>&1; then
     _log_event "block" "oh-my-harness: pre-commit check failed"
     echo "{\\"decision\\": \\"block\\", \\"reason\\": \\"oh-my-harness: pre-commit check failed\\"}"
     exit 0

package/dist/catalog/blocks/format-on-save.js

@@ -26,11 +26,11 @@ set -euo pipefail
 INPUT=$(cat)
 FILE_PATH=$(echo "$INPUT" | jq -r '.tool_input.file_path // .tool_input.path // empty' 2>/dev/null)
 [[ -z "$FILE_PATH" ]] && exit 0
-PATTERN='{{filePattern}}'
+PATTERN='{{{filePattern}}}'
 BASENAME=$(basename "$FILE_PATH")
 if [[ "$BASENAME" == $PATTERN ]]; then
-  echo "oh-my-harness: Running {{command}} on $FILE_PATH..." >&2
-  {{command}} "$FILE_PATH" >&2 2>&1 || true
+  echo "oh-my-harness: Running {{{command}}} on $FILE_PATH..." >&2
+  {{{command}}} "$FILE_PATH" >&2 2>&1 || true
 fi
 exit 0`,
 };

package/dist/catalog/blocks/lint-on-save.js

@@ -26,11 +26,11 @@ set -euo pipefail
 INPUT=$(cat)
 FILE_PATH=$(echo "$INPUT" | jq -r '.tool_input.file_path // .tool_input.path // empty' 2>/dev/null)
 [[ -z "$FILE_PATH" ]] && exit 0
-PATTERN='{{filePattern}}'
+PATTERN='{{{filePattern}}}'
 BASENAME=$(basename "$FILE_PATH")
 if [[ "$BASENAME" == $PATTERN ]]; then
-  echo "oh-my-harness: Running {{command}} on $FILE_PATH..." >&2
-  {{command}} "$FILE_PATH" >&2 2>&1 || true
+  echo "oh-my-harness: Running {{{command}}} on $FILE_PATH..." >&2
+  {{{command}}} "$FILE_PATH" >&2 2>&1 || true
 fi
 exit 0`,
 };

package/dist/catalog/blocks/lockfile-guard.js

@@ -22,7 +22,7 @@ INPUT=$(cat)
 FILE_PATH=$(echo "$INPUT" | jq -r '.tool_input.file_path // .tool_input.path // empty' 2>/dev/null)
 [[ -z "$FILE_PATH" ]] && exit 0
 BASENAME=$(basename "$FILE_PATH")
-LOCKFILES=({{#each lockfiles}}"{{this}}" {{/each}})
+LOCKFILES=({{#each lockfiles}}"{{{this}}}" {{/each}})
 for LOCKFILE in "\${LOCKFILES[@]}"; do
   if [[ "$BASENAME" == "$LOCKFILE" ]]; then
     _log_event "block" "oh-my-harness: direct edits to lockfile $BASENAME are blocked. Use the package manager instead."

package/dist/catalog/blocks/path-guard.js

@@ -20,7 +20,7 @@ set -euo pipefail
 INPUT=$(cat)
 FILE_PATH=$(echo "$INPUT" | jq -r '.tool_input.file_path // .tool_input.path // empty' 2>/dev/null)
 [[ -z "$FILE_PATH" ]] && exit 0
-BLOCKED_PATHS=({{#each blockedPaths}}"{{this}}" {{/each}})
+BLOCKED_PATHS=({{#each blockedPaths}}"{{{this}}}" {{/each}})
 for BLOCKED in "\${BLOCKED_PATHS[@]}"; do
   if [[ "$BLOCKED" == */ ]]; then
     if [[ "$FILE_PATH" == "$BLOCKED"* || "$FILE_PATH" == *"/$BLOCKED"* ]]; then

package/dist/catalog/blocks/secret-file-guard.js

@@ -22,7 +22,7 @@ INPUT=$(cat)
 FILE_PATH=$(echo "$INPUT" | jq -r '.tool_input.file_path // .tool_input.path // empty' 2>/dev/null)
 [[ -z "$FILE_PATH" ]] && exit 0
 BASENAME=$(basename "$FILE_PATH")
-PATTERNS=({{#each patterns}}"{{this}}" {{/each}})
+PATTERNS=({{#each patterns}}"{{{this}}}" {{/each}})
 for PATTERN in "\${PATTERNS[@]}"; do
   if [[ "$BASENAME" == $PATTERN ]]; then
     _log_event "block" "oh-my-harness: file $BASENAME matches secret file pattern: $PATTERN"

package/dist/catalog/blocks/tdd-guard.js

@@ -38,7 +38,12 @@ STATE_DIR=".claude/hooks/.state"
 HISTORY_FILE="\$STATE_DIR/edit-history.json"
 mkdir -p "\$STATE_DIR" 2>/dev/null || true
 
-if echo "\$FILE_PATH" | grep -qE '{{testPattern}}'; then
+TEST_RE='{{{testPattern}}}'
+SRC_RE='{{{srcPattern}}}'
+# Normalize double backslashes to single for [[ =~ ]] compatibility
+TEST_RE="\${TEST_RE//\\\\\\\\/\\\\}"
+SRC_RE="\${SRC_RE//\\\\\\\\/\\\\}"
+if [[ "\$FILE_PATH" =~ \$TEST_RE ]]; then
   # 테스트 파일 수정 → 기록 + 통과
   if [[ ! -f "\$HISTORY_FILE" ]]; then
     echo '{"edits":[]}' > "\$HISTORY_FILE"
@@ -50,13 +55,13 @@ if echo "\$FILE_PATH" | grep -qE '{{testPattern}}'; then
   exit 0
 fi
 
-# 소스 파일 (.ts/.tsx/.js/.jsx) 이 아니면 통과
-if ! echo "\$FILE_PATH" | grep -qE '{{srcPattern}}'; then
+# 소스 파일이 아니면 통과
+if [[ ! "\$FILE_PATH" =~ \$SRC_RE ]]; then
   exit 0
 fi
 
-# 대응 테스트 파일 확인
-BASENAME=$(basename "\$FILE_PATH" | sed -E 's/\\.(ts|tsx|js|jsx)$//')
+# 대응 테스트 파일 확인 — 확장자 제거
+BASENAME=$(basename "\$FILE_PATH" | sed -E 's/\\.[^.]+$//')
 TEST_SUFFIX=".test."
 
 if [[ ! -f "\$HISTORY_FILE" ]]; then
@@ -66,7 +71,7 @@ if [[ ! -f "\$HISTORY_FILE" ]]; then
 fi
 
 # edit-history에서 테스트 파일 검색
-if jq -e --arg b "\$BASENAME" '.edits[] | select(contains($b + ".test.") or contains($b + ".spec."))' "\$HISTORY_FILE" >/dev/null 2>&1; then
+if jq -e --arg b "\$BASENAME" '.edits[] | select(contains($b) and (contains(".test.") or contains(".spec.") or contains("test_")))' "\$HISTORY_FILE" >/dev/null 2>&1; then
   # 테스트 먼저 수정됨 → 소스 기록 + 통과
   UPDATED=$(jq --arg f "\$FILE_PATH" '.edits += [$f] | .edits |= unique' "\$HISTORY_FILE" 2>/dev/null) || true
   if [[ -n "\$UPDATED" ]]; then

package/dist/catalog/template-engine.js

@@ -9,6 +9,11 @@ export function applyDefaults(block, params) {
         if (result[param.name] === undefined && param.default !== undefined) {
             result[param.name] = param.default;
         }
+        // Auto-wrap string into array for string[] params
+        if (param.type === "string[]" && typeof result[param.name] === "string") {
+            const str = result[param.name];
+            result[param.name] = str.includes(",") ? str.split(",").map((s) => s.trim()).filter((s) => s.length > 0) : [str];
+        }
     }
     return result;
 }

package/dist/cli/command-checker.js

@@ -3,11 +3,15 @@ import { promisify } from "node:util";
 const execFileAsync = promisify(execFile);
 // 명령어에서 바이너리 추출 (첫 토큰, 래퍼 처리)
 export function extractExecutable(command) {
-    const parts = command.trim().split(/\s+/);
+    let parts = command.trim().split(/\s+/);
+    // Skip environment variable prefixes (KEY=VALUE)
+    while (parts.length > 0 && /^[A-Za-z_][A-Za-z0-9_]*=/.test(parts[0])) {
+        parts = parts.slice(1);
+    }
     // npm run X → npm
     // npx X → npx
     // bash script.sh → bash
-    return parts[0];
+    return parts[0] ?? "";
 }
 // 명령어 실행 가능 여부 확인
 export async function checkCommandExecutable(binary) {

package/dist/cli/harness-tester.js

@@ -189,13 +189,16 @@ export function generateBlockTestCases(hookEntries, blocks, currentBranch, regis
                 break;
             }
             case "lockfile-guard": {
-                cases.push({
-                    name: "package-lock.json → BLOCKED",
-                    category: "lockfile-guard",
-                    hookScript,
-                    input: { tool_name: "Edit", tool_input: { file_path: "package-lock.json" } },
-                    expectation: "block",
-                });
+                const lockfiles = params.lockfiles ?? ["package-lock.json"];
+                for (const lf of lockfiles) {
+                    cases.push({
+                        name: `${lf} → BLOCKED`,
+                        category: "lockfile-guard",
+                        hookScript,
+                        input: { tool_name: "Edit", tool_input: { file_path: lf } },
+                        expectation: "block",
+                    });
+                }
                 cases.push({
                     name: "package.json → ALLOWED",
                     category: "lockfile-guard",
@@ -224,12 +227,18 @@ export function generateBlockTestCases(hookEntries, blocks, currentBranch, regis
             }
             case "tdd-guard": {
                 const stateFile = ".claude/hooks/.state/edit-history.json";
+                const srcPat = params.srcPattern ?? "\\.(ts|tsx|js|jsx)$";
+                const testPat = params.testPattern ?? "\\.(test|spec)\\.(ts|tsx|js|jsx)$";
+                // Derive sample file extensions from patterns
+                const srcExt = srcPat.includes(".py") ? ".py" : ".ts";
+                const testFile = testPat.includes("test_") ? `test_example${srcExt}` : `example.test${srcExt}`;
+                const srcFile = `src/example${srcExt}`;
                 // block case: source file without prior test edit
                 cases.push({
-                    name: "src/event-logger.ts without test → BLOCKED",
+                    name: `${srcFile} without test → BLOCKED`,
                     category: "tdd-guard",
                     hookScript,
-                    input: { tool_name: "Edit", tool_input: { file_path: "src/event-logger.ts" } },
+                    input: { tool_name: "Edit", tool_input: { file_path: srcFile } },
                     expectation: "block",
                     setup: async (projectDir) => {
                         const historyPath = path.join(projectDir, stateFile);
@@ -252,10 +261,10 @@ export function generateBlockTestCases(hookEntries, blocks, currentBranch, regis
                 });
                 // allow case: test file edit
                 cases.push({
-                    name: "tests/unit/event-logger.test.ts → ALLOWED",
+                    name: `${testFile} → ALLOWED`,
                     category: "tdd-guard",
                     hookScript,
-                    input: { tool_name: "Edit", tool_input: { file_path: "tests/unit/event-logger.test.ts" } },
+                    input: { tool_name: "Edit", tool_input: { file_path: testFile } },
                     expectation: "allow",
                     setup: async (projectDir) => {
                         const historyPath = path.join(projectDir, stateFile);

package/dist/cli/stats/components/Blocks.js

@@ -10,5 +10,5 @@ export function Blocks({ data, selectedIndex }) {
     }
     const clampedIndex = Math.max(0, Math.min(selectedIndex, allBlocks.length - 1));
     const selected = allBlocks[clampedIndex];
-    return (_jsxs(Box, { paddingX: 1, children: [_jsx(Box, { flexDirection: "column", width: 28, children: allBlocks.map((block, i) => (_jsxs(Text, { inverse: i === clampedIndex, color: block.hits === 0 ? "gray" : undefined, children: [i === clampedIndex ? "▶" : " ", " ", block.id, block.hits > 0 ? ` (${block.hits})` : ""] }, block.id))) }), _jsx(Box, { flexDirection: "column", borderStyle: "single", paddingX: 1, flexGrow: 1, children: selected && _jsx(BlockDetail, { block: selected }) })] }));
+    return (_jsxs(Box, { paddingX: 1, children: [_jsx(Box, { flexDirection: "column", width: 28, children: allBlocks.map((block, i) => (_jsxs(Text, { inverse: i === clampedIndex, color: block.hits === 0 ? "gray" : undefined, children: [i === clampedIndex ? "▶" : " ", " ", block.id, block.hits > 0 ? ` (${block.hits})` : ""] }, i))) }), _jsx(Box, { flexDirection: "column", borderStyle: "single", paddingX: 1, flexGrow: 1, children: selected && _jsx(BlockDetail, { block: selected }) })] }));
 }

package/dist/cli/stats/components/Overview.js

@@ -2,5 +2,5 @@ import { jsxs as _jsxs, jsx as _jsx } from "react/jsx-runtime";
 import { Text, Box } from "ink";
 import { HitBar } from "./HitBar.js";
 export function Overview({ data }) {
-    return (_jsxs(Box, { flexDirection: "column", paddingX: 1, children: [_jsxs(Box, { marginBottom: 1, children: [_jsxs(Text, { bold: true, children: ["Active: ", data.activeBlocks.length] }), _jsx(Text, { children: "  " }), _jsxs(Text, { children: ["Events: ", data.totalEvents] }), _jsx(Text, { children: "  " }), _jsxs(Text, { children: ["Block rate: ", data.blockRate, "%"] })] }), data.activeBlocks.map(block => (_jsxs(Box, { children: [_jsx(Box, { width: 22, children: _jsx(Text, { color: block.canBlock ? "cyan" : "gray", children: block.id }) }), _jsx(HitBar, { blockCount: block.blockCount, allowCount: block.allowCount }), _jsxs(Text, { children: [" ", block.hits, " hits"] }), block.blockCount > 0 && _jsxs(Text, { color: "red", children: [" (", block.blockCount, " block)"] })] }, block.id))), data.dormantBlocks.length > 0 && (_jsxs(Box, { flexDirection: "column", marginTop: 1, children: [_jsx(Text, { bold: true, dimColor: true, children: "Dormant (0 hits):" }), data.dormantBlocks.map(block => (_jsxs(Text, { dimColor: true, children: ["  \u2591 ", block.id] }, block.id)))] }))] }));
+    return (_jsxs(Box, { flexDirection: "column", paddingX: 1, children: [_jsxs(Box, { marginBottom: 1, children: [_jsxs(Text, { bold: true, children: ["Active: ", data.activeBlocks.length] }), _jsx(Text, { children: "  " }), _jsxs(Text, { children: ["Events: ", data.totalEvents] }), _jsx(Text, { children: "  " }), _jsxs(Text, { children: ["Block rate: ", data.blockRate, "%"] })] }), data.activeBlocks.map((block, i) => (_jsxs(Box, { children: [_jsx(Box, { width: 22, children: _jsx(Text, { color: block.canBlock ? "cyan" : "gray", children: block.id }) }), _jsx(HitBar, { blockCount: block.blockCount, allowCount: block.allowCount }), _jsxs(Text, { children: [" ", block.hits, " hits"] }), block.blockCount > 0 && _jsxs(Text, { color: "red", children: [" (", block.blockCount, " block)"] })] }, i))), data.dormantBlocks.length > 0 && (_jsxs(Box, { flexDirection: "column", marginTop: 1, children: [_jsx(Text, { bold: true, dimColor: true, children: "Dormant (0 hits):" }), data.dormantBlocks.map((block, i) => (_jsxs(Text, { dimColor: true, children: ["  \u2591 ", block.id] }, i)))] }))] }));
 }

package/dist/cli/stats/data.d.ts

@@ -33,6 +33,7 @@ export interface StatsData {
     hourlyDistribution: HourlyBucket[];
     dateRange: DateRange;
 }
+export declare function deduplicateBlocks(blocks: BlockStats[]): BlockStats[];
 export declare function getActiveBlocks(hookEntries: HookEntry[], allBlocks: BuildingBlock[]): {
     block: BuildingBlock;
     params: Record<string, unknown>;

package/dist/cli/stats/data.js

@@ -4,6 +4,27 @@ import { builtinBlocks } from "../../catalog/blocks/index.js";
 import fs from "node:fs/promises";
 import path from "node:path";
 import yaml from "js-yaml";
+export function deduplicateBlocks(blocks) {
+    const map = new Map();
+    for (const b of blocks) {
+        const existing = map.get(b.id);
+        if (existing) {
+            existing.hits += b.hits;
+            existing.blockCount += b.blockCount;
+            existing.allowCount += b.allowCount;
+            if (b.lastHit && (!existing.lastHit || b.lastHit > existing.lastHit)) {
+                existing.lastHit = b.lastHit;
+            }
+            if (b.lastBlockReason) {
+                existing.lastBlockReason = b.lastBlockReason;
+            }
+        }
+        else {
+            map.set(b.id, { ...b });
+        }
+    }
+    return [...map.values()];
+}
 export function getActiveBlocks(hookEntries, allBlocks) {
     const result = [];
     for (const entry of hookEntries) {
@@ -94,9 +115,9 @@ export async function loadStatsData(projectDir, dateRange = "all") {
     }
     const activeBlocksWithParams = getActiveBlocks(hookEntries, builtinBlocks);
     const dormantIds = getDormantBlocks(activeBlocksWithParams, events);
-    const blocks = activeBlocksWithParams.map(ab => getBlockDetail(ab.block.id, events, builtinBlocks, ab.params));
-    const activeBlocks = blocks.filter(b => !dormantIds.includes(b.id));
-    const dormantBlocks = blocks.filter(b => dormantIds.includes(b.id));
+    const blocks = deduplicateBlocks(activeBlocksWithParams.map(ab => getBlockDetail(ab.block.id, events, builtinBlocks, ab.params)));
+    const activeBlocks = deduplicateBlocks(blocks.filter(b => !dormantIds.includes(b.id)));
+    const dormantBlocks = deduplicateBlocks(blocks.filter(b => dormantIds.includes(b.id)));
     const hourlyDistribution = getHourlyDistribution(events);
     const stats = aggregateStats(events);
     const peakHour = hourlyDistribution.reduce((max, b) => b.total > max.total ? b : max, hourlyDistribution[0]).hour;

package/dist/cli/tool-checker.js

@@ -17,7 +17,13 @@ function extractBinary(command) {
     const trimmed = command.trim();
     if (!trimmed)
         return undefined;
-    const parts = trimmed.split(/\s+/);
+    let parts = trimmed.split(/\s+/);
+    // Skip environment variable prefixes (KEY=VALUE)
+    while (parts.length > 0 && /^[A-Za-z_][A-Za-z0-9_]*=/.test(parts[0])) {
+        parts = parts.slice(1);
+    }
+    if (parts.length === 0)
+        return undefined;
     // Skip gradle wrapper commands (managed by project)
     if (parts[0] === "./gradlew" || parts[0] === "gradlew")
         return undefined;

package/dist/cli/tui/init-flow.d.ts

@@ -4,6 +4,7 @@ import type { ProjectFacts } from "../../detector/types.js";
 export declare function formatDepResults(deps: DepCheck[]): string;
 export declare function formatConfigSummary(config: HarnessConfig): string;
 export declare function formatProjectFacts(facts: ProjectFacts): string;
+export declare function buildPresetExtends(language: string, framework: string | undefined, packageManager: string | undefined): string[];
 export declare function runInitTUI(options?: {
     projectDir?: string;
     presetsDir?: string;

package/dist/cli/tui/init-flow.js

@@ -88,6 +88,16 @@ export function formatProjectFacts(facts) {
     }
     return lines.length > 0 ? lines.join("\n") : `  ${chalk.dim("No project signals detected")}`;
 }
+export function buildPresetExtends(language, framework, packageManager) {
+    const result = ["_base", language];
+    if (framework && framework !== "none") {
+        result.push(framework);
+    }
+    if (packageManager) {
+        result.push(packageManager);
+    }
+    return result;
+}
 function handleCancel(value) {
     if (p.isCancel(value)) {
         p.cancel("Operation cancelled.");
@@ -190,7 +200,15 @@ export async function runInitTUI(options) {
         const genSpinner = p.spinner();
         genSpinner.start("Generating harness configuration...");
         try {
-            harnessConfig = await generateHarnessConfig(description, undefined, undefined, projectFacts);
+            // Load catalog blocks so LLM knows available building blocks + validation
+            const { createDefaultRegistry } = await import("../../catalog/registry.js");
+            const catalogRegistry = await createDefaultRegistry();
+            const catalogBlocks = catalogRegistry.list().map((b) => ({
+                id: b.id,
+                description: b.description,
+                params: b.params.map((pp) => ({ name: pp.name, required: pp.required, default: pp.default, description: pp.description })),
+            }));
+            harnessConfig = await generateHarnessConfig(description, undefined, catalogBlocks, projectFacts);
             genSpinner.stop("Configuration generated");
         }
         catch (err) {
@@ -213,28 +231,124 @@ export async function runInitTUI(options) {
         }
     }
     else if (mode === "preset") {
-        // Step 4b: Preset Mode
-        const registry = new PresetRegistry();
-        await registry.discover(presetsDir);
-        const allPresets = registry.list().filter((e) => e.name !== "_base");
-        if (allPresets.length === 0) {
-            p.log.warn("No presets found (besides _base).");
-            presetNames = ["_base"];
+        // Step 4b: Preset Mode — 3-step interactive language/framework/PM selection
+        p.log.info(`${chalk.dim("_base preset is always included automatically.")}`);
+        // Step 4b-1: Language
+        const language = await p.select({
+            message: "Choose your language",
+            options: [
+                { value: "python", label: "Python" },
+                { value: "typescript", label: "TypeScript" },
+                { value: "javascript", label: "JavaScript" },
+                { value: "rust", label: "Rust" },
+                { value: "go", label: "Go" },
+                { value: "java", label: "Java" },
+                { value: "ruby", label: "Ruby" },
+                { value: "php", label: "PHP" },
+                { value: "swift", label: "Swift" },
+                { value: "dart", label: "Dart" },
+                { value: "csharp", label: "C#" },
+                { value: "elixir", label: "Elixir" },
+                { value: "scala", label: "Scala" },
+                { value: "zig", label: "Zig" },
+                { value: "cpp", label: "C/C++" },
+            ],
+        });
+        handleCancel(language);
+        const frameworkOptions = {
+            python: [
+                { value: "django", label: "Django" },
+                { value: "fastapi", label: "FastAPI" },
+                { value: "flask", label: "Flask" },
+                { value: "none", label: "None" },
+            ],
+            typescript: [
+                { value: "nextjs", label: "Next.js" },
+                { value: "react", label: "React" },
+                { value: "vue", label: "Vue" },
+                { value: "express", label: "Express" },
+                { value: "none", label: "None" },
+            ],
+            javascript: [
+                { value: "nextjs", label: "Next.js" },
+                { value: "react", label: "React" },
+                { value: "vue", label: "Vue" },
+                { value: "express", label: "Express" },
+                { value: "none", label: "None" },
+            ],
+            java: [
+                { value: "springboot", label: "Spring Boot" },
+                { value: "none", label: "None" },
+            ],
+            ruby: [
+                { value: "rails", label: "Rails" },
+                { value: "none", label: "None" },
+            ],
+            php: [
+                { value: "laravel", label: "Laravel" },
+                { value: "none", label: "None" },
+            ],
+            go: [
+                { value: "gin", label: "Gin" },
+                { value: "none", label: "None" },
+            ],
+            rust: [
+                { value: "actix", label: "Actix" },
+                { value: "none", label: "None" },
+            ],
+            dart: [
+                { value: "flutter", label: "Flutter" },
+                { value: "none", label: "None" },
+            ],
+            elixir: [
+                { value: "phoenix", label: "Phoenix" },
+                { value: "none", label: "None" },
+            ],
+        };
+        // Step 4b-2: Framework (only for languages with options)
+        let framework;
+        const fwOptions = frameworkOptions[language];
+        if (fwOptions && fwOptions.length > 1) {
+            const fw = await p.select({
+                message: "Choose your framework",
+                options: fwOptions,
+            });
+            handleCancel(fw);
+            framework = fw;
         }
-        else {
-            p.log.info(`${chalk.dim("_base preset is always included automatically.")}`);
-            const selected = await p.multiselect({
-                message: "Select presets to apply:",
-                options: allPresets.map((entry) => ({
-                    value: entry.name,
-                    label: entry.config.displayName,
-                    hint: entry.config.description,
-                })),
-                required: true,
+        const pmOptions = {
+            python: [
+                { value: "uv", label: "uv" },
+                { value: "pipenv", label: "pipenv" },
+                { value: "pip", label: "pip" },
+            ],
+            typescript: [
+                { value: "pnpm", label: "pnpm" },
+                { value: "npm", label: "npm" },
+                { value: "yarn", label: "yarn" },
+            ],
+            javascript: [
+                { value: "pnpm", label: "pnpm" },
+                { value: "npm", label: "npm" },
+                { value: "yarn", label: "yarn" },
+            ],
+            java: [
+                { value: "gradle", label: "gradle" },
+                { value: "maven", label: "maven" },
+            ],
+        };
+        // Step 4b-3: Package Manager (only for languages with multiple options)
+        let packageManager;
+        const pmOpts = pmOptions[language];
+        if (pmOpts && pmOpts.length > 1) {
+            const pm = await p.select({
+                message: "Choose your package manager",
+                options: pmOpts,
             });
-            handleCancel(selected);
-            presetNames = ["_base", ...selected];
+            handleCancel(pm);
+            packageManager = pm;
         }
+        presetNames = buildPresetExtends(language, framework, packageManager);
     }
     else if (mode === "import") {
         // Step 4c: Import Mode

package/dist/core/harness-converter-v2.js

@@ -13,11 +13,8 @@ export async function harnessToMergedConfigV2(harness, registry, projectDir) {
     // Resolve registry — use provided one or create the default
     const resolvedRegistry = registry ?? (await createDefaultRegistry());
     const catalogResult = await convertHookEntries(allHookEntries, resolvedRegistry, projectDir ?? ".");
-    // If there are errors, return base config with catalogErrors attached
-    if (catalogResult.errors.length > 0) {
-        return { ...base, catalogErrors: catalogResult.errors };
-    }
     // Convert hooksConfig entries from catalog into HookDefinition format.
+    // Errors are reported as warnings but don't block valid hooks.
     const additionalPreToolUse = [];
     const additionalPostToolUse = [];
     for (const [event, entries] of Object.entries(catalogResult.hooksConfig)) {
@@ -45,5 +42,6 @@ export async function harnessToMergedConfigV2(harness, registry, projectDir) {
             preToolUse: [...base.hooks.preToolUse, ...additionalPreToolUse],
             postToolUse: [...base.hooks.postToolUse, ...additionalPostToolUse],
         },
+        ...(catalogResult.errors.length > 0 ? { catalogErrors: catalogResult.errors } : {}),
     };
 }

package/dist/detector/detectors/node.js

@@ -43,6 +43,44 @@ export const nodeDetector = {
         if (!(await fileExists(packageJsonPath))) {
             return {};
         }
+        // Check if this is a real JS/TS project or just tooling (husky, linter wrappers)
+        const pkgCheck = await readJsonFile(packageJsonPath);
+        const deps = pkgCheck && typeof pkgCheck.dependencies === "object" && pkgCheck.dependencies !== null
+            ? Object.keys(pkgCheck.dependencies)
+            : [];
+        const devDeps = pkgCheck && typeof pkgCheck.devDependencies === "object" && pkgCheck.devDependencies !== null
+            ? Object.keys(pkgCheck.devDependencies)
+            : [];
+        const pkgScriptsRaw = pkgCheck && typeof pkgCheck.scripts === "object" && pkgCheck.scripts !== null
+            ? Object.values(pkgCheck.scripts).join(" ")
+            : "";
+        const jsDevIndicators = /typescript|react|vue|angular|next|vite|webpack|eslint|jest|vitest|mocha|babel|svelte|nuxt/;
+        const jsScriptIndicators = /vitest|jest|mocha|tsc|eslint|webpack|vite|next|react-scripts/;
+        // Also check for JS ecosystem config files
+        const jsConfigFiles = [
+            "tsconfig.json", ".eslintrc", ".eslintrc.js", ".eslintrc.json",
+            "eslint.config.js", "eslint.config.ts", "eslint.config.mjs",
+            "biome.json", "next.config.js", "next.config.ts", "next.config.mjs",
+            "vite.config.ts", "vite.config.js", "webpack.config.js",
+        ];
+        let hasJsConfigFile = false;
+        for (const cf of jsConfigFiles) {
+            if (await fileExists(path.join(projectDir, cf))) {
+                hasJsConfigFile = true;
+                break;
+            }
+        }
+        const hasWorkspaces = pkgCheck && (Array.isArray(pkgCheck.workspaces) || typeof pkgCheck.workspaces === "object");
+        const hasPackageManager = pkgCheck && typeof pkgCheck.packageManager === "string";
+        const hasJsIndicator = deps.length > 0
+            || devDeps.some((d) => jsDevIndicators.test(d))
+            || jsScriptIndicators.test(pkgScriptsRaw)
+            || hasJsConfigFile
+            || hasWorkspaces
+            || hasPackageManager;
+        if (!hasJsIndicator) {
+            return {};
+        }
         const detectedFiles = ["package.json"];
         const languages = [];
         const frameworks = [];