oh-my-harness 0.4.1 → 0.4.2

package/dist/catalog/converter.js

@@ -1,5 +1,5 @@
 import path from "path";
-import { renderTemplate, validateParams } from "./template-engine.js";
+import { renderTemplate, validateParams, applyDefaults } from "./template-engine.js";
 export async function convertHookEntries(entries, registry, projectDir) {
     const hooksConfig = {};
     const scripts = new Map();
@@ -11,7 +11,8 @@ export async function convertHookEntries(entries, registry, projectDir) {
             errors.push(`Unknown block id: "${entry.block}"`);
             continue;
         }
-        const paramErrors = validateParams(block, entry.params);
+        const resolvedParams = applyDefaults(block, entry.params);
+        const paramErrors = validateParams(block, resolvedParams);
         if (paramErrors.length > 0) {
             errors.push(...paramErrors);
             continue;
@@ -23,7 +24,7 @@ export async function convertHookEntries(entries, registry, projectDir) {
         seenBlockIds.add(entry.block);
         let scriptContent;
         try {
-            scriptContent = renderTemplate(block.template, entry.params);
+            scriptContent = renderTemplate(block.template, resolvedParams);
         }
         catch (err) {
             errors.push(`Failed to render block "${entry.block}": ${err.message}`);

package/dist/catalog/template-engine.d.ts

@@ -1,3 +1,4 @@
 import type { BuildingBlock } from "./types.js";
 export declare function renderTemplate(template: string, params: Record<string, unknown>): string;
+export declare function applyDefaults(block: BuildingBlock, params: Record<string, unknown>): Record<string, unknown>;
 export declare function validateParams(block: BuildingBlock, params: Record<string, unknown>): string[];

package/dist/catalog/template-engine.js

@@ -3,6 +3,15 @@ export function renderTemplate(template, params) {
     const compiled = Handlebars.compile(template);
     return compiled(params);
 }
+export function applyDefaults(block, params) {
+    const result = { ...params };
+    for (const param of block.params) {
+        if (result[param.name] === undefined && param.default !== undefined) {
+            result[param.name] = param.default;
+        }
+    }
+    return result;
+}
 export function validateParams(block, params) {
     const errors = [];
     for (const param of block.params) {

package/dist/catalog/types.d.ts

@@ -109,11 +109,11 @@ export declare const BuildingBlockSchema: z.ZodObject<{
 }>;
 export declare const HookEntrySchema: z.ZodObject<{
     block: z.ZodString;
-    params: z.ZodRecord<z.ZodString, z.ZodUnknown>;
+    params: z.ZodDefault<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
 }, "strip", z.ZodTypeAny, {
     params: Record<string, unknown>;
     block: string;
 }, {
-    params: Record<string, unknown>;
     block: string;
+    params?: Record<string, unknown> | undefined;
 }>;

package/dist/catalog/types.js

@@ -20,5 +20,5 @@ export const BuildingBlockSchema = z.object({
 });
 export const HookEntrySchema = z.object({
     block: z.string(),
-    params: z.record(z.unknown()),
+    params: z.record(z.unknown()).default({}),
 });

package/dist/core/harness-schema.d.ts

@@ -65,7 +65,7 @@ export declare const HarnessConfigSchema: z.ZodObject<{
         content: string;
         priority?: number | undefined;
     }>, "many">;
-    enforcement: z.ZodObject<{
+    enforcement: z.ZodDefault<z.ZodObject<{
         preCommit: z.ZodDefault<z.ZodArray<z.ZodString, "many">>;
         blockedPaths: z.ZodDefault<z.ZodArray<z.ZodString, "many">>;
         blockedCommands: z.ZodDefault<z.ZodArray<z.ZodString, "many">>;
@@ -95,16 +95,16 @@ export declare const HarnessConfigSchema: z.ZodObject<{
             command: string;
             pattern: string;
         }[] | undefined;
-    }>;
+    }>>;
     hooks: z.ZodDefault<z.ZodArray<z.ZodObject<{
         block: z.ZodString;
-        params: z.ZodRecord<z.ZodString, z.ZodUnknown>;
+        params: z.ZodDefault<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
     }, "strip", z.ZodTypeAny, {
         params: Record<string, unknown>;
         block: string;
     }, {
-        params: Record<string, unknown>;
         block: string;
+        params?: Record<string, unknown> | undefined;
     }>, "many">>;
     permissions: z.ZodDefault<z.ZodObject<{
         allow: z.ZodDefault<z.ZodArray<z.ZodString, "many">>;
@@ -172,23 +172,23 @@ export declare const HarnessConfigSchema: z.ZodObject<{
         content: string;
         priority?: number | undefined;
     }[];
-    enforcement: {
-        blockedPaths?: string[] | undefined;
-        preCommit?: string[] | undefined;
-        blockedCommands?: string[] | undefined;
-        postSave?: {
-            command: string;
-            pattern: string;
-        }[] | undefined;
-    };
     permissions?: {
         allow?: string[] | undefined;
         deny?: string[] | undefined;
     } | undefined;
     version?: "1.0" | undefined;
     hooks?: {
-        params: Record<string, unknown>;
         block: string;
+        params?: Record<string, unknown> | undefined;
     }[] | undefined;
+    enforcement?: {
+        blockedPaths?: string[] | undefined;
+        preCommit?: string[] | undefined;
+        blockedCommands?: string[] | undefined;
+        postSave?: {
+            command: string;
+            pattern: string;
+        }[] | undefined;
+    } | undefined;
 }>;
 export type HarnessConfig = z.infer<typeof HarnessConfigSchema>;

package/dist/core/harness-schema.js

@@ -31,7 +31,7 @@ export const HarnessConfigSchema = z.object({
             pattern: z.string(),
             command: z.string(),
         })).default([]),
-    }),
+    }).default({}),
     // Catalog-based hooks (v2)
     hooks: z.array(HookEntrySchema).default([]),
     // Permissions

package/dist/nl/prompt-templates.js

@@ -101,14 +101,19 @@ hooks:
       typecheckCommand: "npx tsc --noEmit"
   - block: lint-on-save
     params:
-      lintCommand: "npx eslint --fix"
-      pattern: "*.{ts,tsx}"
+      filePattern: "*.ts"
+      command: "npx eslint --fix"
   - block: path-guard
     params:
-      blockedPaths: ".next/,node_modules/,*.min.js"
+      blockedPaths:
+        - ".next/"
+        - "node_modules/"
+        - "*.min.js"
   - block: command-guard
     params:
-      blockedCommands: "rm -rf /,sudo rm"
+      patterns:
+        - "rm -rf /"
+        - "sudo rm"
 permissions:
   allow:
     - "Bash(pnpm install*)"
@@ -145,14 +150,20 @@ hooks:
       testCommand: "uv run pytest"
   - block: lint-on-save
     params:
-      lintCommand: "ruff check --fix"
-      pattern: "*.py"
+      filePattern: "*.py"
+      command: "ruff check --fix"
   - block: path-guard
     params:
-      blockedPaths: "__pycache__/,.venv/,*.pyc"
+      blockedPaths:
+        - "__pycache__/"
+        - ".venv/"
+        - "*.pyc"
   - block: command-guard
     params:
-      blockedCommands: "rm -rf /,sudo rm,pip install"
+      patterns:
+        - "rm -rf /"
+        - "sudo rm"
+        - "pip install"
 permissions:
   allow:
     - "Bash(pytest*)"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-harness",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "Tame your AI coding agents with natural language",
   "type": "module",
   "bin": {