Merge pull request #149 from QuantGeekDev/fix/65-tsx-ts-support

fix: support .ts file extensions in loaders for tsx compatibility

Author: QuantGeekDev

src/loaders/promptLoader.ts

@@ -7,8 +7,8 @@ export class PromptLoader extends BaseLoader<PromptProtocol> {
     super(
       {
         subdirectory: 'prompts',
-        excludedFiles: ['BasePrompt.js', '*.test.js', '*.spec.js'],
-        extensions: ['.js'],
+        excludedFiles: ['BasePrompt.js', 'BasePrompt.ts', '*.test.js', '*.spec.js', '*.test.ts', '*.spec.ts', '*.d.ts'],
+        extensions: ['.js', '.ts'],
       },
       basePath
     );

src/loaders/resourceLoader.ts

@@ -7,8 +7,8 @@ export class ResourceLoader extends BaseLoader<ResourceProtocol> {
     super(
       {
         subdirectory: 'resources',
-        excludedFiles: ['BaseResource.js', '*.test.js', '*.spec.js'],
-        extensions: ['.js'],
+        excludedFiles: ['BaseResource.js', 'BaseResource.ts', '*.test.js', '*.spec.js', '*.test.ts', '*.spec.ts', '*.d.ts'],
+        extensions: ['.js', '.ts'],
       },
       basePath
     );

src/loaders/toolLoader.ts

@@ -7,8 +7,8 @@ export class ToolLoader extends BaseLoader<ToolProtocol> {
     super(
       {
         subdirectory: 'tools',
-        excludedFiles: ['BaseTool.js', '*.test.js', '*.spec.js'],
-        extensions: ['.js'],
+        excludedFiles: ['BaseTool.js', 'BaseTool.ts', '*.test.js', '*.spec.js', '*.test.ts', '*.spec.ts', '*.d.ts'],
+        extensions: ['.js', '.ts'],
       },
       basePath
     );

tests/loaders/tsx-support.test.ts

@@ -0,0 +1,69 @@
+import { describe, it, expect } from '@jest/globals';
+import { ToolLoader } from '../../src/loaders/toolLoader.js';
+import { PromptLoader } from '../../src/loaders/promptLoader.js';
+import { ResourceLoader } from '../../src/loaders/resourceLoader.js';
+
+describe('Loader TypeScript file support (tsx compatibility)', () => {
+  describe('ToolLoader', () => {
+    it('should accept .ts extension in config', () => {
+      const loader = new ToolLoader('/tmp/test');
+      // Access the protected config via any cast
+      const config = (loader as any).config;
+      expect(config.extensions).toContain('.js');
+      expect(config.extensions).toContain('.ts');
+    });
+
+    it('should exclude .d.ts files', () => {
+      const loader = new ToolLoader('/tmp/test');
+      const config = (loader as any).config;
+      expect(config.excludedFiles).toContain('*.d.ts');
+    });
+
+    it('should exclude TypeScript test files', () => {
+      const loader = new ToolLoader('/tmp/test');
+      const config = (loader as any).config;
+      expect(config.excludedFiles).toContain('*.test.ts');
+      expect(config.excludedFiles).toContain('*.spec.ts');
+    });
+
+    it('should exclude base class TypeScript files', () => {
+      const loader = new ToolLoader('/tmp/test');
+      const config = (loader as any).config;
+      expect(config.excludedFiles).toContain('BaseTool.ts');
+    });
+  });
+
+  describe('PromptLoader', () => {
+    it('should accept .ts extension in config', () => {
+      const loader = new PromptLoader('/tmp/test');
+      const config = (loader as any).config;
+      expect(config.extensions).toContain('.js');
+      expect(config.extensions).toContain('.ts');
+    });
+
+    it('should exclude .d.ts and test files', () => {
+      const loader = new PromptLoader('/tmp/test');
+      const config = (loader as any).config;
+      expect(config.excludedFiles).toContain('*.d.ts');
+      expect(config.excludedFiles).toContain('*.test.ts');
+      expect(config.excludedFiles).toContain('BasePrompt.ts');
+    });
+  });
+
+  describe('ResourceLoader', () => {
+    it('should accept .ts extension in config', () => {
+      const loader = new ResourceLoader('/tmp/test');
+      const config = (loader as any).config;
+      expect(config.extensions).toContain('.js');
+      expect(config.extensions).toContain('.ts');
+    });
+
+    it('should exclude .d.ts and test files', () => {
+      const loader = new ResourceLoader('/tmp/test');
+      const config = (loader as any).config;
+      expect(config.excludedFiles).toContain('*.d.ts');
+      expect(config.excludedFiles).toContain('*.test.ts');
+      expect(config.excludedFiles).toContain('BaseResource.ts');
+    });
+  });
+});