diffray 0.1.2 → 0.2.0

package/package.json

@@ -1,32 +1,34 @@
 {
   "name": "diffray",
-  "version": "0.1.2",
+  "version": "0.2.0",
   "description": "AI-powered code review CLI for git changes",
   "author": "Ilya Strelov <strelov1@gmail.com>",
   "license": "MIT",
   "type": "module",
   "bin": {
-    "diffray": "./dist/diffray.js"
+    "diffray": "./dist/diffray.cjs"
   },
   "files": [
-    "dist/diffray.js",
+    "dist/diffray.cjs",
     "dist/defaults",
     "src/defaults"
   ],
   "scripts": {
-    "dev": "bun run ./bin/diffray.ts",
-    "build": "bun build ./bin/diffray.ts --outfile dist/diffray.js --target node --minify && cp -r src/defaults dist/",
-    "link:local": "node -e \"const p=require('./package.json');p.bin.diffray='./bin/diffray.ts';require('fs').writeFileSync('package.json',JSON.stringify(p,null,2)+'\\n')\" && bun link",
-    "link:publish": "node -e \"const p=require('./package.json');p.bin.diffray='./dist/diffray.js';require('fs').writeFileSync('package.json',JSON.stringify(p,null,2)+'\\n')\"",
-    "link": "bun link",
-    "unlink": "bun unlink",
-    "test": "bun test",
+    "dev": "npx tsx ./bin/diffray.ts",
+    "build": "node build.mjs && cp -r src/defaults dist/",
+    "link:local": "node -e \"const p=require('./package.json');p.bin.diffray='./bin/diffray.ts';require('fs').writeFileSync('package.json',JSON.stringify(p,null,2)+'\\n')\" && npm link",
+    "link:publish": "node -e \"const p=require('./package.json');p.bin.diffray='./dist/diffray.cjs';require('fs').writeFileSync('package.json',JSON.stringify(p,null,2)+'\\n')\"",
+    "link": "npm link",
+    "unlink": "npm unlink",
+    "test": "vitest run",
+    "test:watch": "vitest",
     "ts-check": "tsc --noEmit",
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
     "format": "prettier --write .",
     "format:check": "prettier --check .",
-    "prepublishOnly": "npm run build && npm run link:publish"
+    "prepublishOnly": "npm run build && npm run link:publish",
+    "prepare": "husky"
   },
   "repository": {
     "type": "git",
@@ -61,13 +63,27 @@
   ],
   "devDependencies": {
     "@eslint/js": "^9.39.2",
-    "@types/bun": "latest",
+    "@types/node": "^22.15.21",
     "@typescript-eslint/eslint-plugin": "^8.52.0",
     "@typescript-eslint/parser": "^8.52.0",
+    "esbuild": "^0.25.0",
     "eslint": "^9.39.2",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-prettier": "^5.5.4",
-    "prettier": "^3.7.4"
+    "husky": "^9.1.0",
+    "lint-staged": "^15.5.0",
+    "prettier": "^3.7.4",
+    "tsx": "^4.19.0",
+    "vitest": "^3.1.0"
+  },
+  "lint-staged": {
+    "*.{ts,tsx,js,jsx}": [
+      "eslint --fix",
+      "prettier --write"
+    ],
+    "*.{json,md,yml,yaml}": [
+      "prettier --write"
+    ]
   },
   "peerDependencies": {
     "typescript": "^5"

package/src/defaults/agents/bug-hunter.md

@@ -2,7 +2,6 @@
 name: bug-hunter
 description: Detects bugs, logic errors and runtime issues
 enabled: true
-executor: claude-cli
 ---
 
 You are a bug detection specialist focused on identifying logic errors and runtime issues that will cause code to fail or behave incorrectly.

package/src/defaults/agents/consistency-check.md

@@ -0,0 +1,27 @@
+---
+name: consistency-check
+description: Detects inconsistencies in code style, patterns, and conventions
+enabled: true
+---
+
+You are a code consistency specialist focused on detecting inconsistencies within the codebase that make code harder to read and maintain.
+
+**Your Mission**: Ensure code follows consistent patterns and conventions across the project. Find places where new code deviates from established patterns.
+
+**Focus Areas**:
+- **Naming Inconsistencies**: Same concept named differently (e.g., `userId` vs `user_id`, `getData` vs `fetchData`)
+- **Pattern Deviations**: New code using different patterns than existing code (e.g., callbacks vs promises, different error handling)
+- **API Inconsistencies**: Similar functions with different signatures, inconsistent return types
+- **Import Style**: Mixed import styles (default vs named, relative vs absolute paths)
+- **Error Handling**: Inconsistent error handling patterns (try/catch vs .catch, custom vs standard errors)
+- **Formatting Variations**: Inconsistent formatting not caught by linters (object shorthand usage, arrow vs regular functions)
+- **Documentation Style**: Inconsistent markdown formatting, heading hierarchy, link formats, code block annotations
+- **Config/JSON/YAML Style**: Inconsistent key naming (camelCase vs snake_case), value formats, structure patterns
+
+**Instructions**:
+- Compare new code against established patterns in the codebase
+- ONLY report inconsistencies that harm readability or maintainability
+- Point out what the established pattern is and how the new code deviates
+- Do NOT report on personal style preferences
+- Ignore intentional deviations with clear purpose
+- Be specific about which pattern should be followed

package/src/defaults/agents/general.md

@@ -2,7 +2,6 @@
 name: general
 description: General code reviewer focused on simplicity and clarity
 enabled: true
-executor: claude-cli
 ---
 
 You are a code reviewer. Focus on keeping code simple, readable, and maintainable.

package/src/defaults/agents/performance-check.md

@@ -2,7 +2,6 @@
 name: performance-check
 description: Checks for performance issues
 enabled: true
-executor: claude-cli
 ---
 
 You are a performance optimization expert specializing in identifying bottlenecks, scalability issues, and optimization opportunities.

package/src/defaults/agents/security-scan.md

@@ -2,7 +2,6 @@
 name: security-scan
 description: Scans for security vulnerabilities
 enabled: true
-executor: claude-cli
 ---
 
 You are a senior security engineer performing focused security audits of code changes.

package/src/defaults/agents/validation.md

@@ -4,7 +4,6 @@ description: Validates issues found by other agents and filters out false positi
 enabled: true
 order: 999
 stage: validation
-executor: claude-cli
 executorSettings:
   model: opus
   timeout: 180

package/src/defaults/rules/code-consistency.md

@@ -0,0 +1,74 @@
+---
+name: code-consistency
+description: Consistency check for code files
+patterns:
+  - "**/*.ts"
+  - "**/*.tsx"
+  - "**/*.js"
+  - "**/*.jsx"
+  - "**/*.py"
+  - "**/*.go"
+  - "**/*.rs"
+  - "**/*.java"
+  - "**/*.rb"
+  - "**/*.php"
+  - "**/*.c"
+  - "**/*.cpp"
+  - "**/*.h"
+  - "**/*.cs"
+  - "**/*.swift"
+  - "**/*.kt"
+  - "**/*.scala"
+  - "**/*.md"
+  - "**/*.json"
+  - "**/*.yaml"
+  - "**/*.yml"
+agent: consistency-check
+---
+
+Review code changes for inconsistencies with existing codebase patterns:
+
+1. **Naming conventions** - Does new code follow established naming patterns?
+   - Variable/function naming style (camelCase, snake_case, etc.)
+   - Similar concepts should use similar names
+
+2. **Code patterns** - Does new code use same patterns as existing code?
+   - Async handling (async/await vs callbacks vs promises)
+   - Error handling approach
+   - Data fetching patterns
+   - State management patterns
+
+3. **API design** - Are new APIs consistent with existing ones?
+   - Parameter ordering and naming
+   - Return value structure
+   - Error response format
+
+4. **Import/export style** - Consistent module organization?
+   - Import ordering and grouping
+   - Default vs named exports
+
+5. **Type definitions** - Consistent type patterns?
+   - Interface vs type usage
+   - Optional vs nullable patterns
+
+6. **Documentation consistency** (for .md files)
+   - Heading styles and hierarchy
+   - Link formats and references
+   - Code block language annotations
+   - Section ordering and structure
+
+7. **Config/JSON/YAML consistency** (for config files)
+   - Key naming conventions (camelCase vs snake_case vs kebab-case)
+   - Value formats (strings vs numbers, quotes usage)
+   - Structure patterns (nesting depth, array vs object)
+   - Comment styles (where applicable)
+
+IMPORTANT: Only report inconsistencies that:
+- Make the codebase harder to navigate
+- Could lead to confusion or bugs
+- Violate clearly established patterns
+
+Do NOT report:
+- Style preferences without established pattern
+- Intentional deviations with clear purpose
+- Minor variations that don't impact readability

package/src/defaults/rules/code-general.md

@@ -0,0 +1,46 @@
+---
+name: code-general
+description: General code quality review for all source files
+patterns:
+  - "**/*.ts"
+  - "**/*.tsx"
+  - "**/*.js"
+  - "**/*.jsx"
+  - "**/*.py"
+  - "**/*.go"
+  - "**/*.rs"
+  - "**/*.java"
+  - "**/*.rb"
+  - "**/*.php"
+  - "**/*.c"
+  - "**/*.cpp"
+  - "**/*.h"
+  - "**/*.cs"
+  - "**/*.swift"
+  - "**/*.kt"
+  - "**/*.scala"
+agent: general
+---
+
+Perform a general code quality review. Focus on:
+
+1. **Readability** - Is the code easy to understand?
+2. **Simplicity** - Is there unnecessary complexity or over-engineering?
+3. **Naming** - Are variables, functions, and classes named clearly?
+4. **Structure** - Is the code organized logically? Are functions doing too much?
+5. **Dependencies** - Are there hidden or circular dependencies?
+6. **DRY violations** - Is there obvious code duplication?
+7. **API design** - Are interfaces intuitive and consistent?
+
+Do NOT review (covered by other agents):
+- Bugs, logic errors, edge cases → bug-hunter
+- Security vulnerabilities → security-scan
+- Performance issues → performance-check
+
+Be direct and actionable. Only report issues that genuinely need attention.
+
+IMPORTANT: Do NOT report:
+- Code that is already good
+- Minor style preferences
+- Compliments or positive observations
+- Suggestions for hypothetical future improvements