Fix GitHub tests to mock httpRequest correctly

Update mock configuration to use vi.fn() directly in factory and import from source files. Resolves vitest hoisting issues with CommonJS compiled output.

Author: jdalton

test/registry/github.test.mts

@@ -5,20 +5,22 @@ import path from 'node:path'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
 // Mock the httpRequest module before importing github module.
-const mockHttpRequest = vi.fn()
-vi.mock('../../registry/dist/lib/http-request', () => ({
-  httpRequest: mockHttpRequest,
-}))
+vi.mock('../../registry/src/lib/http-request', () => {
+  return {
+    httpRequest: vi.fn(),
+  }
+})
 
 import {
   clearRefCache,
   fetchGitHub,
   getGitHubToken,
   getRefCacheSize,
   resolveRefToSha,
-} from '../../registry/dist/lib/github'
+} from '../../registry/src/lib/github'
+import { httpRequest as httpRequestActual } from '../../registry/src/lib/http-request'
 
-const httpRequest = mockHttpRequest
+const httpRequest = httpRequestActual as unknown as ReturnType<typeof vi.fn>
 
 describe('github module', () => {
   let originalEnv: NodeJS.ProcessEnv