feat(husky): add Flatten JSON tool and use npx zx

- Add support for 'includePatterns' in tool configuration
- Add 'Flatten JSON' tool configuration
- Update workflow to respect 'includePatterns'
- Switch git hooks to use 'npx zx' for better portability
- Add tests for new functionality
- Update documentation

Author: TerrorSquad

booster/.husky/commit-msg

@@ -12,4 +12,4 @@ cd "$ROOT_DIR"
 HOOK="$CWD/commit-msg.ts"
 
 # Use the generic runner script to execute the TypeScript hook
-exec "$CWD/shared/runner.sh" pnpm zx $HOOK "$@"
+exec "$CWD/shared/runner.sh" npx zx "$HOOK" "$@"

booster/.husky/pre-commit

@@ -12,4 +12,4 @@ cd "$ROOT_DIR"
 HOOK="$CWD/pre-commit.ts"
 
 # Use the generic runner script to execute the TypeScript hook
-exec "$CWD/shared/runner.sh" pnpm zx $HOOK "$@"
+exec "$CWD/shared/runner.sh" npx zx "$HOOK" "$@"

booster/.husky/pre-push

@@ -12,4 +12,4 @@ cd "$ROOT_DIR"
 HOOK="$CWD/pre-push.ts"
 
 # Use the generic runner script to execute the TypeScript hook
-exec "$CWD/shared/runner.sh" pnpm zx $HOOK "$@"
+exec "$CWD/shared/runner.sh" npx zx "$HOOK" "$@"

booster/.husky/shared/tools.ts

@@ -23,7 +23,7 @@ export const TOOLS: ToolConfig[] = [
   {
     name: 'ESLint',
     command: 'eslint',
-    args: ['--fix', '--cache'],
+    args: ['--fix', '--cache', '--no-warn-ignored'],
     type: 'node',
     stagesFilesAfter: true,
     extensions: ['.js', '.jsx', '.ts', '.tsx', '.vue', '.mjs', '.cjs'],
@@ -52,6 +52,16 @@ export const TOOLS: ToolConfig[] = [
     ],
     group: 'format',
   },
+  {
+    name: 'Flatten JSON',
+    command: 'node',
+    args: ['./tools/flatten-json.ts'],
+    type: 'node',
+    extensions: ['.json'],
+    includePatterns: ['apps/*/src/locales/*.json'],
+    stagesFilesAfter: true,
+    group: 'format',
+  },
   {
     name: 'Stylelint',
     command: 'stylelint',

booster/.husky/shared/types.ts

@@ -40,6 +40,8 @@ export interface ToolConfig {
   extensions?: string[]
   /** If true, re-stages files after execution (useful for fixers) */
   stagesFilesAfter?: boolean
+  /** Only run on files matching any of these glob patterns (or regexes) */
+  includePatterns?: (string | RegExp)[]
   /** If false, does not pass the list of staged files to the command. Default is true. */
   passFiles?: boolean
   /** If true, runs the command for each file individually. Default is false. */

booster/.husky/shared/workflow.ts

@@ -1,4 +1,4 @@
-import { which, fs } from 'zx'
+import { which, fs, path } from 'zx'
 import {
   ensureMutagenSync,
   exec,
@@ -222,7 +222,21 @@ async function prepareTool(tool: ToolConfig, files: string[]): Promise<PreparedT
     ? files.filter((file) => tool.extensions!.some((ext) => file.endsWith(ext)))
     : files
 
-  if (filesToRun.length === 0) return null
+  const filteredFiles = tool.includePatterns
+    ? filesToRun.filter((file) => {
+        return tool.includePatterns!.some((pattern) => {
+            if (pattern instanceof RegExp) {
+              return pattern.test(file)
+            }
+
+            // Use Node.js built-in globbing matching (requires Node.js 22+)
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
+            return (path as any).match(pattern, file);
+          });
+        })
+    : filesToRun;
+
+  if (filteredFiles.length === 0) return null
 
   // Check binary existence
   if (!(await isToolAvailable(tool))) {
@@ -232,7 +246,7 @@ async function prepareTool(tool: ToolConfig, files: string[]): Promise<PreparedT
 
   return {
     tool,
-    files: filesToRun,
+    files: filteredFiles,
     description: tool.description || `Running ${tool.name}...`,
   }
 }

booster/.husky/tests/shared/workflow.test.ts

@@ -11,7 +11,7 @@ import {
   initEnvironment
 } from '../../shared/core'
 import { shouldSkipDuringMerge, stageFiles } from '../../shared/git'
-import { fs, which, $ } from 'zx'
+import { fs, which, $, path } from 'zx'
 
 // Mock dependencies
 vi.mock('../../shared/core', () => ({
@@ -52,6 +52,9 @@ vi.mock('zx', () => ({
   which: vi.fn(),
   $: {
     verbose: false
+  },
+  path: {
+    match: vi.fn()
   }
 }))
 
@@ -382,6 +385,58 @@ describe('workflow.ts', () => {
 
       expect(exec).toHaveBeenCalledTimes(2)
     })
+
+    it('should filter files based on includePatterns (regex)', async () => {
+      vi.mocked(isSkipped).mockReturnValue(false)
+      vi.mocked(fs.pathExists).mockResolvedValue(true)
+      vi.mocked(exec).mockResolvedValue({} as any)
+
+      const regexTool: ToolConfig = {
+        ...mockTool,
+        type: 'node',
+        command: 'check',
+        extensions: ['.json'], // Add extension to match file extension check first
+        includePatterns: [/src\/locales\/.*\.json$/]
+      }
+
+      const files = ['apps/app1/src/locales/en.json', 'other/file.json']
+      await runQualityChecks(files, [regexTool])
+
+      // Should run only for matched file
+      expect(exec).toHaveBeenCalledTimes(1)
+      const callArgs = vi.mocked(exec).mock.calls[0][0]
+      expect(callArgs).toContain('apps/app1/src/locales/en.json')
+      expect(callArgs).not.toContain('other/file.json')
+    })
+
+    it('should filter files based on includePatterns (glob)', async () => {
+      vi.mocked(isSkipped).mockReturnValue(false)
+      vi.mocked(fs.pathExists).mockResolvedValue(true)
+      vi.mocked(exec).mockResolvedValue({} as any)
+
+      // Mock path.match behavior
+      // @ts-ignore
+      vi.mocked(path.match).mockImplementation((pattern, file) => {
+        // Simple mock: check if file ends with .json and path contains /locales/
+        return file.endsWith('.json') && file.includes('/locales/')
+      })
+
+      const globTool: ToolConfig = {
+        ...mockTool,
+        type: 'node',
+        command: 'check',
+        extensions: ['.json'], // Add extension to match file extension check first
+        includePatterns: ['apps/*/src/locales/*.json']
+      }
+
+      const files = ['apps/app1/src/locales/en.json', 'other/file.json']
+      await runQualityChecks(files, [globTool])
+
+      expect(exec).toHaveBeenCalledTimes(1)
+      const callArgs = vi.mocked(exec).mock.calls[0][0]
+      expect(callArgs).toContain('apps/app1/src/locales/en.json')
+      expect(callArgs).not.toContain('other/file.json')
+    })
   })
 
   describe('runHook', () => {

docs/content/3.tools/4.git_hooks.md

@@ -182,6 +182,16 @@ Disable, override, or add custom tools:
 }
 ```
 
+```json [Filter by Glob Pattern]
+{
+  "tools": {
+    "Flatten JSON": {
+      "includePatterns": ["apps/*/src/locales/*.json"]
+    }
+  }
+}
+```
+
 ::tip
 Alternative config file names: `.githooks.json` or set `GIT_HOOKS_CONFIG` environment variable to a custom path.
 ::
@@ -269,7 +279,7 @@ HOOKS_ONLY=lint,format git commit -m "..."
 | Group | Tools |
 |-------|-------|
 | `lint` | ESLint, Stylelint, PHP Syntax Check, TypeScript |
-| `format` | Prettier, ECS |
+| `format` | Prettier, ECS, Flatten JSON |
 | `analysis` | PHPStan, Psalm, Deptrac |
 | `refactor` | Rector |
 
@@ -278,6 +288,7 @@ HOOKS_ONLY=lint,format git commit -m "..."
 |-----------|---------------------|
 | ESLint | `SKIP_ESLINT` |
 | Prettier | `SKIP_PRETTIER` |
+| Flatten JSON | `SKIP_FLATTEN_JSON` |
 | Stylelint | `SKIP_STYLELINT` |
 | TypeScript | `SKIP_TYPESCRIPT` |
 | PHP Syntax Check | `SKIP_PHP_SYNTAX_CHECK` |