@gotza02/sequential-thinking 10000.0.2 → 10000.0.4

package/dist/tools/sports/tools/match.d.ts

@@ -3,4 +3,8 @@
  * Tools for match analysis and live scores
  */
 import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+/**
+ * Perform comprehensive match analysis using web search
+ */
+export declare function performMatchAnalysis(homeTeam: string, awayTeam: string, league?: string, context?: string): Promise<string>;
 export declare function registerMatchTools(server: McpServer): void;

package/dist/tools/sports/tools/match.js

@@ -65,7 +65,7 @@ async function findMatchId(homeTeam, awayTeam, league) {
 /**
  * Perform comprehensive match analysis using web search
  */
-async function performMatchAnalysis(homeTeam, awayTeam, league, context) {
+export async function performMatchAnalysis(homeTeam, awayTeam, league, context) {
     const leagueStr = league ? `${league} ` : '';
     const baseQuery = `${leagueStr}${homeTeam} vs ${awayTeam}`;
     const dateQuery = getDateContext();

package/dist/tools/sports/tools/match.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/tools/sports/tools/match.test.js

@@ -0,0 +1,86 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { performMatchAnalysis } from './match.js';
+// Mock dependencies
+const mockSearchProvider = {
+    search: vi.fn(),
+};
+vi.mock('../providers/search.js', () => ({
+    getSearchProvider: () => mockSearchProvider,
+    SearchProvider: class {
+    },
+}));
+vi.mock('../providers/scraper.js', () => ({
+    scrapeMatchContent: vi.fn(),
+    findBestMatchUrl: vi.fn(),
+    findBestNewsUrl: vi.fn(),
+}));
+vi.mock('../providers/api.js', () => ({
+    createAPIProvider: () => ({
+        searchTeams: vi.fn().mockResolvedValue({ success: false }),
+        getLiveMatches: vi.fn().mockResolvedValue({ success: false }),
+        getStatus: () => ({ available: false })
+    })
+}));
+// Import mocked modules to setup return values
+import { scrapeMatchContent, findBestMatchUrl, findBestNewsUrl } from '../providers/scraper.js';
+describe('Match Analysis Tool', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+        // Default mock behaviors
+        mockSearchProvider.search.mockResolvedValue([
+            { title: 'Result 1', url: 'https://example.com/1', snippet: 'Snippet 1' }
+        ]);
+        scrapeMatchContent.mockResolvedValue({ success: true, data: 'Scraped Content' });
+        findBestMatchUrl.mockReturnValue(null);
+        findBestNewsUrl.mockReturnValue(null);
+    });
+    it('should perform basic analysis with search results', async () => {
+        const result = await performMatchAnalysis('Arsenal', 'Chelsea');
+        expect(mockSearchProvider.search).toHaveBeenCalled();
+        expect(result).toContain('Arsenal vs Chelsea');
+        expect(result).toContain('Result 1');
+        expect(result).toContain('INSTRUCTIONS: Act as a World-Class Football Analysis Panel');
+    });
+    it('should scrape stats URL when found', async () => {
+        findBestMatchUrl.mockReturnValue('https://understat.com/match/123');
+        const result = await performMatchAnalysis('Arsenal', 'Chelsea');
+        expect(findBestMatchUrl).toHaveBeenCalled();
+        expect(scrapeMatchContent).toHaveBeenCalledWith('https://understat.com/match/123');
+        expect(result).toContain('DEEP DIVE ANALYSIS: STATS & DATA');
+        expect(result).toContain('Scraped Content');
+    });
+    it('should perform dual-source scraping (Stats + News)', async () => {
+        // Setup distinct URLs for stats and news
+        findBestMatchUrl.mockReturnValue('https://understat.com/match/123');
+        findBestNewsUrl.mockReturnValue('https://bbc.com/sport/football/456');
+        // Mock scraping to return different content based on URL
+        scrapeMatchContent.mockImplementation(async (url) => {
+            if (url.includes('understat'))
+                return { success: true, data: 'STATS_DATA_HERE' };
+            if (url.includes('bbc'))
+                return { success: true, data: 'NEWS_DATA_HERE' };
+            return { success: false };
+        });
+        const result = await performMatchAnalysis('Arsenal', 'Chelsea');
+        expect(scrapeMatchContent).toHaveBeenCalledTimes(2);
+        expect(result).toContain('DEEP DIVE ANALYSIS: STATS & DATA');
+        expect(result).toContain('STATS_DATA_HERE');
+        expect(result).toContain('DEEP DIVE ANALYSIS: NEWS & LINEUPS');
+        expect(result).toContain('NEWS_DATA_HERE');
+    });
+    it('should not scrape news if it is the same as stats URL', async () => {
+        const url = 'https://example.com/match';
+        findBestMatchUrl.mockReturnValue(url);
+        findBestNewsUrl.mockReturnValue(url); // Same URL
+        await performMatchAnalysis('Arsenal', 'Chelsea');
+        expect(scrapeMatchContent).toHaveBeenCalledTimes(1); // Should only scrape once
+    });
+    it('should handle scraper failures gracefully', async () => {
+        findBestMatchUrl.mockReturnValue('https://bad-url.com');
+        scrapeMatchContent.mockResolvedValue({ success: false, error: 'Timeout' });
+        const result = await performMatchAnalysis('Arsenal', 'Chelsea');
+        expect(result).toContain('Stats scrape failed: Timeout');
+        // Should still contain the main analysis structure
+        expect(result).toContain('INSTRUCTIONS: Act as a World-Class Football Analysis Panel');
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotza02/sequential-thinking",
-  "version": "10000.0.2",
+  "version": "10000.0.4",
   "publishConfig": {
     "access": "public"
   },
@@ -44,7 +44,8 @@
     "jsdom": "^27.4.0",
     "turndown": "^7.2.2",
     "typescript": "^5.9.3",
-    "yargs": "^18.0.0"
+    "yargs": "^18.0.0",
+    "zod": "^3.22.4"
   },
   "devDependencies": {
     "@types/cors": "^2.8.19",