Merge pull request #781 from zackproser/improve-comparisons-again

Add production seeding script and update gitignore

Author: zackproser

.gitignore

@@ -67,3 +67,4 @@ prisma/.env
 
 # Ignore optimized OG images
 public/og-images-optimized/
+.env.production

package.json

@@ -27,7 +27,8 @@
     "og:clean-orphans": "tsx --tsconfig scripts/tsconfig.json scripts/clean-og-images.ts",
     "og:optimize": "node scripts/optimize-og-images.js",
     "og:optimize-replace": "node scripts/optimize-og-images.js --replace",
-    "prisma:seed": "tsx --tsconfig prisma/tsconfig.json prisma/seed.ts"
+    "prisma:seed": "tsx --tsconfig prisma/tsconfig.json prisma/seed.ts",
+    "prisma:seed-production": "./scripts/seed-production.sh"
   },
   "browserslist": "defaults, not ie <= 11",
   "dependencies": {

scripts/check-production-tools.js

@@ -0,0 +1,61 @@
+const { PrismaClient } = require('@prisma/client');
+
+async function checkProductionTools() {
+  console.log('🔍 Checking production database tools...\n');
+  
+  const prisma = new PrismaClient();
+  
+  try {
+    const tools = await prisma.tool.findMany({
+      select: {
+        name: true,
+        category: true,
+        websiteUrl: true
+      },
+      orderBy: {
+        name: 'asc'
+      }
+    });
+    
+    console.log(`📊 Found ${tools.length} tools in production database:\n`);
+    
+    tools.forEach((tool, index) => {
+      console.log(`${index + 1}. ${tool.name} (${tool.category})`);
+    });
+    
+    console.log('\n🔗 Testing slug generation for problematic URLs:');
+    
+    const problematicSlugs = [
+      'anthropic-claude-api',
+      'mistral-ai',
+      'cursor',
+      'github-copilot'
+    ];
+    
+    for (const slug of problematicSlugs) {
+      // Generate search patterns (same logic as in getToolBySlug)
+      const patterns = [
+        slug,
+        slug.split('-').map(word => word.charAt(0).toUpperCase() + word.slice(1)).join(' ')
+      ];
+      
+      const tool = await prisma.tool.findFirst({
+        where: {
+          OR: patterns.flatMap(pattern => [
+            { name: { equals: pattern, mode: 'insensitive' } },
+            { name: { contains: pattern, mode: 'insensitive' } }
+          ])
+        }
+      });
+      
+      console.log(`  ${slug}: ${tool ? '✅ ' + tool.name : '❌ Not found'}`);
+    }
+    
+  } catch (error) {
+    console.error('❌ Error checking production tools:', error.message);
+  } finally {
+    await prisma.$disconnect();
+  }
+}
+
+checkProductionTools(); 

scripts/seed-production.sh

@@ -0,0 +1,65 @@
+#!/bin/bash
+
+# This script runs the prisma seed against the production database
+# Usage: ./scripts/seed-production.sh
+
+echo "🌱 Production Database Seeding Script"
+echo "======================================"
+
+# Check if we have the required environment file
+if [ ! -f ".env.production" ]; then
+  echo "❌ Error: .env.production file not found"
+  echo "Please create .env.production with your production database URLs"
+  exit 1
+fi
+
+# Get the production database URL (non-pooling for direct operations)
+PROD_DB_URL=$(grep POSTGRES_URL_NON_POOLING .env.production | cut -d '=' -f2- | tr -d '"')
+
+if [ -z "$PROD_DB_URL" ]; then
+  echo "❌ Error: Could not find POSTGRES_URL_NON_POOLING in .env.production"
+  echo "Please make sure you have POSTGRES_URL_NON_POOLING set in .env.production"
+  exit 1
+fi
+
+echo "🔍 Found production database URL"
+echo "⚠️  WARNING: This will modify your PRODUCTION database!"
+echo "⚠️  This will DELETE all existing tools and replace them with the seed data"
+echo ""
+echo "Current seed data contains 25 tools:"
+echo "- Anthropic Claude API, Mistral AI, Cursor, GitHub Copilot, etc."
+echo ""
+echo "Continue? (y/n)"
+read -r confirm
+
+if [ "$confirm" != "y" ]; then
+  echo "❌ Operation cancelled"
+  exit 0
+fi
+
+echo ""
+echo "🚀 Running seed script against production database..."
+echo "📊 This will:"
+echo "   1. Delete all existing tools"
+echo "   2. Insert 25 new tools from seed data"
+echo "   3. Update comparison URLs to work correctly"
+echo ""
+
+# Run the seed script with the production database URL
+POSTGRES_URL="$PROD_DB_URL" npm run prisma:seed
+
+if [ $? -eq 0 ]; then
+  echo ""
+  echo "✅ Production database seeding completed successfully!"
+  echo "🎉 Your comparison URLs should now work correctly"
+  echo ""
+  echo "Next steps:"
+  echo "1. Test a few comparison URLs to verify they work"
+  echo "2. Check that the tools are displaying correctly on your site"
+  echo "3. Monitor for any issues in production"
+else
+  echo ""
+  echo "❌ Seeding failed! Please check the error messages above"
+  echo "Your production database was not modified"
+  exit 1
+fi 

src/app/api/articles/[slug]/__tests__/route.test.ts

@@ -3,20 +3,18 @@
 import { NextRequest, NextResponse } from 'next/server';
 import { jest } from '@jest/globals';
 
-// Mock the importContentMetadata function
+// Mock the content-handlers functions
 jest.mock('@/lib/content-handlers', () => ({
-  importContentMetadata: jest.fn()
+  getContentItemByDirectorySlug: jest.fn()
 }));
 
 // Import the mocked function
-import { importContentMetadata } from '@/lib/content-handlers';
-
-// Mock the route handlers directly
-const mockGET = jest.fn();
+import { getContentItemByDirectorySlug } from '@/lib/content-handlers';
 
 // Mock the route module
+const mockGET = jest.fn() as jest.MockedFunction<any>;
 jest.mock('../route', () => ({
-  GET: (...args: any[]) => mockGET(...args)
+  GET: mockGET
 }));
 
 describe('Articles API', () => {
@@ -37,10 +35,8 @@ describe('Articles API', () => {
         tags: ['test', 'article']
       };
 
-      // Setup mock to return article data
-      mockGET.mockResolvedValue(
-        NextResponse.json(mockArticle, { status: 200 })
-      );
+      const mockResponse = NextResponse.json(mockArticle, { status: 200 });
+      mockGET.mockResolvedValue(mockResponse);
 
       const request = new NextRequest('http://localhost:3000/api/articles/test-article');
       const params = { params: Promise.resolve({ slug: 'test-article' }) };
@@ -55,10 +51,8 @@ describe('Articles API', () => {
     });
 
     it('should return 404 for non-existent article', async () => {
-      // Setup mock to simulate article not found
-      mockGET.mockResolvedValue(
-        NextResponse.json({ error: 'Article not found' }, { status: 404 })
-      );
+      const mockResponse = NextResponse.json({ error: 'Article not found' }, { status: 404 });
+      mockGET.mockResolvedValue(mockResponse);
 
       const request = new NextRequest('http://localhost:3000/api/articles/non-existent');
       const params = { params: Promise.resolve({ slug: 'non-existent' }) };
@@ -73,13 +67,8 @@ describe('Articles API', () => {
     });
 
     it('should handle errors gracefully', async () => {
-      // Setup mock to throw an error
-      mockGET.mockRejectedValue(new Error('Unexpected error'));
-      
-      // Setup error handler mock
-      mockGET.mockResolvedValueOnce(
-        NextResponse.json({ error: 'Internal server error' }, { status: 500 })
-      );
+      const mockResponse = NextResponse.json({ error: 'Internal server error' }, { status: 500 });
+      mockGET.mockResolvedValue(mockResponse);
 
       const request = new NextRequest('http://localhost:3000/api/articles/error-article');
       const params = { params: Promise.resolve({ slug: 'error-article' }) };

src/app/api/articles/__tests__/route.test.ts

@@ -4,11 +4,11 @@ import { NextRequest, NextResponse } from 'next/server';
 import { jest } from '@jest/globals';
 
 // Mock the route handlers directly
-const mockGET = jest.fn();
+const mockGET = jest.fn() as jest.MockedFunction<any>;
 
 // Mock the route module
 jest.mock('../[slug]/route', () => ({
-  GET: (...args) => mockGET(...args)
+  GET: mockGET
 }));
 
 describe('Articles API', () => {
@@ -30,9 +30,8 @@ describe('Articles API', () => {
       };
 
       // Setup mock to return article data
-      mockGET.mockResolvedValue(
-        NextResponse.json(mockArticle, { status: 200 })
-      );
+      const mockResponse = NextResponse.json(mockArticle, { status: 200 });
+      mockGET.mockResolvedValue(mockResponse);
 
       const request = new NextRequest('http://localhost:3000/api/articles/test-article');
       const params = { params: Promise.resolve({ slug: 'test-article' }) };
@@ -48,9 +47,8 @@ describe('Articles API', () => {
 
     it('should return 404 for non-existent article', async () => {
       // Setup mock to simulate article not found
-      mockGET.mockResolvedValue(
-        NextResponse.json({ error: 'Article not found' }, { status: 404 })
-      );
+      const mockResponse = NextResponse.json({ error: 'Article not found' }, { status: 404 });
+      mockGET.mockResolvedValue(mockResponse);
 
       const request = new NextRequest('http://localhost:3000/api/articles/non-existent');
       const params = { params: Promise.resolve({ slug: 'non-existent' }) };