fix(hooks): respect Deptrac config in pre-push artifact generation

The pre-push hook now checks the .git-hooks.config.json tools section
when deciding whether to generate the Deptrac image. Setting
`"Deptrac": { "enabled": false }` will now skip both the static
analysis (pre-commit) AND the image generation (pre-push).

Skipping can also be done via:
- SKIP_DEPTRAC=1 env var
- SKIP_DEPTRAC_IMAGE=1 env var
- skip.artifacts: true in config

Also updated schema to include explicit tool name properties for
better IDE autocomplete support.

Author: TerrorSquad

booster/.husky/pre-push.ts

@@ -37,15 +37,34 @@ export async function runTests(): Promise<boolean> {
   return true
 }
 
-export async function handleArtifacts(): Promise<void> {
+/**
+ * Check if a tool is disabled in the config
+ */
+function isToolDisabled(toolName: string, config: Awaited<ReturnType<typeof loadConfig>>): boolean {
+  if (!config.tools) return false
+  
+  // Case-insensitive lookup
+  const configKey = Object.keys(config.tools).find(
+    key => key.toLowerCase() === toolName.toLowerCase()
+  )
+  
+  if (!configKey) return false
+  return config.tools[configKey]?.enabled === false
+}
+
+export async function handleArtifacts(config: Awaited<ReturnType<typeof loadConfig>>): Promise<void> {
   // Generate artifacts - these are informational and don't block the push
   // Developers should commit these manually if needed
 
-  if (!isSkipped('deptrac_image')) {
+  // Check both env var and config file for Deptrac image
+  if (!isSkipped('deptrac_image') && !isSkipped('deptrac') && !isToolDisabled('Deptrac', config)) {
     await generateDeptracImage()
+  } else {
+    log.skip('Deptrac image generation skipped')
   }
 
-  if (!isSkipped('api_docs')) {
+  // Check both env var and config file for API docs
+  if (!isSkipped('api_docs') && !isToolDisabled('API Docs', config)) {
     try {
       const result = await generateApiDocs()
       if (result.changed) {
@@ -55,6 +74,8 @@ export async function handleArtifacts(): Promise<void> {
       // API docs generation is optional, don't fail the push
       log.warn('API docs generation failed, but continuing with push')
     }
+  } else {
+    log.skip('API docs generation skipped')
   }
 }
 
@@ -70,7 +91,7 @@ await runHook(GitHook.PrePush, async () => {
 
   // Generate artifacts - informational only, don't block push
   if (!isHookSkippedByConfig('artifacts', config)) {
-    await handleArtifacts()
+    await handleArtifacts(config)
   } else {
     log.info('Skipping artifact generation (disabled in config)')
   }

booster/.husky/tests/pre-push.test.mjs

@@ -16,6 +16,7 @@ vi.mock('../shared/index.ts', () => ({
     success: vi.fn(),
     error: vi.fn(),
     warn: vi.fn(),
+    skip: vi.fn(),
   },
   runHook: vi.fn(), // Prevent automatic execution
   exec: vi.fn(),
@@ -66,22 +67,24 @@ describe('Pre-push Hook', () => {
   })
 
   describe('handleArtifacts', () => {
+    const emptyConfig = {}
+
     it('should generate both deptrac image and api docs', async () => {
       isSkipped.mockReturnValue(false)
       generateDeptracImage.mockResolvedValue({ generated: true, changed: true })
       generateApiDocs.mockResolvedValue({ generated: true, changed: false })
 
-      await handleArtifacts()
+      await handleArtifacts(emptyConfig)
 
       expect(generateDeptracImage).toHaveBeenCalled()
       expect(generateApiDocs).toHaveBeenCalled()
     })
 
     it('should skip deptrac if env var is set', async () => {
-      isSkipped.mockImplementation((name) => name === 'deptrac_image')
+      isSkipped.mockImplementation((name) => name === 'deptrac_image' || name === 'deptrac')
       generateApiDocs.mockResolvedValue({ generated: true, changed: false })
 
-      await handleArtifacts()
+      await handleArtifacts(emptyConfig)
 
       expect(generateDeptracImage).not.toHaveBeenCalled()
       expect(generateApiDocs).toHaveBeenCalled()
@@ -91,7 +94,7 @@ describe('Pre-push Hook', () => {
       isSkipped.mockImplementation((name) => name === 'api_docs')
       generateDeptracImage.mockResolvedValue({ generated: true, changed: false })
 
-      await handleArtifacts()
+      await handleArtifacts(emptyConfig)
 
       expect(generateDeptracImage).toHaveBeenCalled()
       expect(generateApiDocs).not.toHaveBeenCalled()
@@ -103,7 +106,39 @@ describe('Pre-push Hook', () => {
       generateApiDocs.mockRejectedValue(new Error('Failed'))
 
       // Should not throw
-      await expect(handleArtifacts()).resolves.not.toThrow()
+      await expect(handleArtifacts(emptyConfig)).resolves.not.toThrow()
+    })
+
+    it('should skip deptrac if disabled in config', async () => {
+      isSkipped.mockReturnValue(false)
+      generateApiDocs.mockResolvedValue({ generated: true, changed: false })
+
+      const configWithDeptracDisabled = {
+        tools: {
+          Deptrac: { enabled: false }
+        }
+      }
+
+      await handleArtifacts(configWithDeptracDisabled)
+
+      expect(generateDeptracImage).not.toHaveBeenCalled()
+      expect(generateApiDocs).toHaveBeenCalled()
+    })
+
+    it('should skip deptrac if disabled with lowercase name in config', async () => {
+      isSkipped.mockReturnValue(false)
+      generateApiDocs.mockResolvedValue({ generated: true, changed: false })
+
+      const configWithDeptracDisabled = {
+        tools: {
+          deptrac: { enabled: false }
+        }
+      }
+
+      await handleArtifacts(configWithDeptracDisabled)
+
+      expect(generateDeptracImage).not.toHaveBeenCalled()
+      expect(generateApiDocs).toHaveBeenCalled()
     })
   })
 })