Fix JSR bug + Major simplification (v0.3.0)

[drernie]

Summary

This PR includes:

1. Bug fix for JSR installation paths (issue Hook installation generates invalid file paths from JSR #9)
2. Version management tooling with dev pre-release support
3. Major architectural simplification (v0.3.0) - self-contained hooks with simple command arrays

Part 1: Bug Fix (Issue #9)

Changes

• Fixed JSR installation path generation (src/install.ts)
  • Detects JSR installations with import.meta.url.startsWith("jsr:")
  • For JSR: Uses .href to preserve full jsr:@scope/package@version/path format
  • For local: Uses .pathname for absolute filesystem path
  • Previously generated invalid paths like /@theswanfactory/deno-hooks/0.2.0/src/run.ts
  • Now generates valid paths like jsr:@theswanfactory/deno-hooks@0.2.0/src/run.ts

Test Improvements

• Strengthened integration test validation to catch path format bugs

Part 2: Version Management

New Features

• Added comprehensive version management script (scripts/version.ts)

  • deno task version - Display current version
  • deno task version patch/minor/major - Bump version automatically
  • deno task version tag - Create and push release tag
  • deno task version dev - Create dev pre-release tag

• Dev pre-release workflow for testing JSR installations

  • deno task tag-dev - Runs tests and creates timestamped dev tag
  • deno task tag - Runs tests and creates stable release tag
  • Dev versions use format: 0.3.0-dev.1734197345

Part 3: Major Architectural Simplification (v0.3.0)

Breaking Changes

Simplified Configuration Format

Before (v0.2.x):

hooks:
  pre-commit:
    - id: deno-fmt
      run: deno-fmt
      glob: "*.{ts,js}"
      pass_filenames: true

After (v0.3.0):

hooks:
  pre-commit:
    - deno task fmt
    - deno task lint

Self-Contained Hook Scripts

Generated hooks are now standalone bash scripts with no dependencies on deno-hooks:

#!/bin/sh
# Generated by deno-hooks - DO NOT EDIT
# To update, run: deno task hooks

set -e

deno task fmt
deno task lint

echo "✓ All hooks passed"

What Was Removed

• ❌ Built-in hooks (deno-fmt, deno-lint, deno-test) - use deno task instead
• ❌ File glob pattern matching and pass_filenames logic
• ❌ Runtime execution system (src/run.ts, src/executor.ts, src/files.ts)
• ❌ Complex hook type definitions (src/hook.ts)
• ❌ 200+ lines of complex code

What Was Added

• ✅ Self-contained hook scripts (work without deno-hooks installed)
• ✅ Support for ANY shell command (not just deno commands)
• ✅ Success message "✓ All hooks passed"
• ✅ New examples: advanced.yml and deno-json-config.json
• ✅ Fail-fast behavior with set -e

Benefits

1. Simpler: No complex file filtering or built-in hooks
2. Transparent: Generated hooks are readable shell scripts
3. Portable: Hooks work without deno-hooks installed
4. Flexible: Users can run ANY command (deno, npm, custom scripts)
5. Debuggable: Easy to see what's happening
6. No Magic: Just runs commands in order

Files Modified

Core implementation:

• src/config.ts - Simplified to parse command arrays
• src/install.ts - Rewritten to generate self-contained scripts
• src/mod.ts - Simplified exports
• src/test-hook.test.ts - Updated tests

Files removed:

• src/run.ts (deleted)
• src/executor.ts (deleted)
• src/files.ts (deleted)
• src/hook.ts (deleted)

Documentation:

• README.md - Complete rewrite for new architecture
• CLAUDE.md - Updated developer guide
• CHANGELOG.md - Added v0.3.0 section

Examples:

• examples/basic.yml - Updated to new format
• examples/advanced.yml - New advanced example
• examples/deno-json-config.json - New deno.json example

Tests:

• scripts/test-fake-repo.ts - Updated integration tests

Migration Guide

For Existing Users (v0.2.x → v0.3.0)

1. Update your configuration file (deno-hooks.yml or deno.json):

   • Change from hook objects to simple command strings
   • Replace built-in hooks (deno-fmt) with deno task fmt
   • Remove glob, pass_filenames, etc.

2. Define tasks in deno.json:

   {
     "tasks": {
       "fmt": "deno fmt --check",
       "lint": "deno lint",
       "test": "deno test -A"
     }
   }

3. Run deno task hooks to reinstall with new format

Test Results

All tests passing:

• ✅ Unit tests: 2/2 passed
• ✅ Integration tests: 4/4 passed
  • Hook script is self-contained
  • Hooks catch bad formatting
  • Hooks catch lint errors
  • Hooks allow good commits

Closes

Fixes #9

---

🤖 Generated with Claude Code