Merge pull request #12 from OGMatrix/dev

Applying fixes to prod

Author: OGMatrix

src/cli/manage.ts

@@ -221,6 +221,16 @@ async function fetchJson(url: string): Promise<unknown> {
     };
     https
       .get(url, options, (res) => {
+        // Handle redirects (GitHub release assets redirect to the actual download URL)
+        if (res.statusCode === 301 || res.statusCode === 302) {
+          if (!res.headers.location) {
+            return reject(new Error('Redirect location missing'));
+          }
+          res.resume();
+          fetchJson(res.headers.location).then(resolve).catch(reject);
+          return;
+        }
+
         if (res.statusCode !== 200) {
           res.resume();
           return reject(new Error(`Request failed with status code ${res.statusCode}`));

src/index.test.ts

@@ -3,7 +3,7 @@
  * Tests cover: DbVersioning, Tool Handlers, Utility Functions
  */
 
-import { describe, it, expect, vi, beforeEach, afterEach, type Mock } from 'vitest';
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import fs from 'fs';
 
 // ============================================================================
@@ -809,90 +809,3 @@ describe('Version Comparison Edge Cases', () => {
     expect(versioning.compareVersions('1.0.0', '2.0.0')).toBe(-1);
   });
 });
-
-// ============================================================================
-// ModExamplesService Tests
-// ============================================================================
-
-describe('ModExamplesService', () => {
-  let ModExamplesService: typeof import('./services/mod-examples-service.js').ModExamplesService;
-  let mockDb: { prepare: Mock; close: Mock };
-  let mockPrepare: Mock;
-  let mockGet: Mock;
-  let mockAll: Mock;
-
-  beforeEach(async () => {
-    vi.resetModules();
-
-    mockGet = vi.fn();
-    mockAll = vi.fn();
-    mockPrepare = vi.fn().mockReturnValue({
-      get: mockGet,
-      all: mockAll,
-    });
-    mockDb = {
-      prepare: mockPrepare,
-      close: vi.fn(),
-    };
-
-    vi.doMock('better-sqlite3', () => {
-      return {
-        default: vi.fn(function () {
-          return mockDb;
-        }),
-      };
-    });
-
-    const module = await import('./services/mod-examples-service.js');
-    ModExamplesService = module.ModExamplesService;
-  });
-
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  it('searchExamples should return empty array when no results', () => {
-    mockAll.mockReturnValue([]);
-    const service = new ModExamplesService();
-    const results = service.searchExamples({ query: 'test' });
-    expect(results).toEqual([]);
-  });
-
-  it('searchExamples should parse JSON fields correctly', () => {
-    const mockRow = {
-      id: 1,
-      mod_name: 'TestMod',
-      mod_repo: 'test/repo',
-      file_path: 'src/Test.java',
-      file_url: 'http://example.com',
-      start_line: 1,
-      end_line: 10,
-      title: 'Test Example',
-      code: 'code',
-      language: 'java',
-      caption: 'caption',
-      explanation: 'explanation',
-      pattern_type: 'mixin',
-      complexity: 'beginner',
-      category: 'items',
-      categoryName: 'Items',
-      bestPractices: '["practice"]',
-      potentialPitfalls: '["pitfall"]',
-      useCases: '["usecase"]',
-      keywords: '["keyword"]',
-      minecraftConcepts: '["concept"]',
-      qualityScore: 100,
-      isFeatured: 1,
-      tags: '["tag"]',
-      imports: '[{"path":"p","type":"t","isCritical":true}]',
-      apiReferences: '[{"className":"c","apiType":"a"}]',
-    };
-    mockAll.mockReturnValue([mockRow]);
-
-    const service = new ModExamplesService();
-    const results = service.searchExamples({ query: 'test' });
-
-    expect(results[0].bestPractices).toEqual(['practice']);
-    expect(results[0].imports).toHaveLength(1);
-  });
-});