@tamyla/clodo-framework 4.0.13 → 4.0.14

package/scripts/test-downstream-install.js

@@ -0,0 +1,237 @@
+#!/usr/bin/env node
+
+/**
+ * DOWNSTREAM USER PACKAGE TEST
+ * 
+ * Simulates exactly what a downstream user does:
+ * 1. Creates tarball via npm pack (exactly what npm publish does)
+ * 2. Creates fresh directory
+ * 3. Runs npm install with the tarball
+ * 4. Tests importing all exports
+ * 5. Tests CLI commands
+ * 
+ * This catches issues that internal tests miss because it tests
+ * the actual packaged distribution, not source files.
+ */
+
+import fs from 'fs';
+import path from 'path';
+import { execSync } from 'child_process';
+import { tmpdir } from 'os';
+import { fileURLToPath } from 'url';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const projectRoot = path.join(__dirname, '..');
+
+const colors = {
+  reset: '\x1b[0m',
+  green: '\x1b[32m',
+  red: '\x1b[31m',
+  yellow: '\x1b[33m',
+  blue: '\x1b[34m',
+  cyan: '\x1b[36m',
+};
+
+function log(color, text) {
+  console.log(`${color}${text}${colors.reset}`);
+}
+
+async function testDownstreamInstall() {
+  log(colors.cyan, '\n╔═══════════════════════════════════════════════════════════════╗');
+  log(colors.cyan, '║  DOWNSTREAM USER PACKAGE TEST                                  ║');
+  log(colors.cyan, '║  Testing tarball installation and imports                       ║');
+  log(colors.cyan, '╚═══════════════════════════════════════════════════════════════╝\n');
+
+  const testDir = path.join(tmpdir(), `clodo-user-test-${Date.now()}`);
+  let tarballPath = null;
+
+  try {
+    // Step 1: Create tarball
+    log(colors.blue, '📦 Step 1: Creating npm tarball');
+    log(colors.blue, '─'.repeat(60));
+    
+    try {
+      const output = execSync('npm pack --silent', { 
+        cwd: projectRoot,
+        encoding: 'utf8'
+      }).trim();
+      
+      tarballPath = path.join(projectRoot, output);
+      if (!fs.existsSync(tarballPath)) {
+        throw new Error(`Tarball created but not found: ${tarballPath}`);
+      }
+      
+      const size = (fs.statSync(tarballPath).size / 1024).toFixed(2);
+      log(colors.green, `✅ Created tarball: ${path.basename(tarballPath)} (${size} KB)\n`);
+    } catch (e) {
+      throw new Error(`Failed to create tarball: ${e.message}`);
+    }
+
+    // Step 2: Create fresh test directory
+    log(colors.blue, '📁 Step 2: Creating fresh test directory');
+    log(colors.blue, '─'.repeat(60));
+    
+    if (fs.existsSync(testDir)) {
+      fs.rmSync(testDir, { recursive: true, force: true });
+    }
+    fs.mkdirSync(testDir, { recursive: true });
+    
+    const packageJson = {
+      name: 'clodo-user-test',
+      version: '1.0.0',
+      type: 'module',
+      private: true,
+      dependencies: {
+        '@tamyla/clodo-framework': `file:${tarballPath}`
+      }
+    };
+    
+    fs.writeFileSync(
+      path.join(testDir, 'package.json'),
+      JSON.stringify(packageJson, null, 2)
+    );
+    
+    log(colors.green, `✅ Created test directory: ${testDir}\n`);
+
+    // Step 3: Install from tarball
+    log(colors.blue, '⬇️  Step 3: Installing package from tarball');
+    log(colors.blue, '─'.repeat(60));
+    
+    try {
+      execSync('npm install --no-save', {
+        cwd: testDir,
+        stdio: 'pipe',
+        encoding: 'utf8'
+      });
+      log(colors.green, `✅ Installation successful\n`);
+    } catch (e) {
+      throw new Error(`Failed to install from tarball: ${e.message}`);
+    }
+
+    // Step 4: Test imports
+    log(colors.blue, '🧪 Step 4: Testing module imports');
+    log(colors.blue, '─'.repeat(60));
+
+    const testScript = `
+import('@tamyla/clodo-framework')
+  .then(mod => {
+    const exports = Object.keys(mod);
+    console.log('Loaded exports: ' + exports.length);
+    if (exports.length < 20) throw new Error('Not enough exports: ' + exports.length);
+    
+    // Test key exports
+    const required = [
+      'Clodo', 'SchemaManager', 'ModuleManager', 'initializeService',
+      'GenericDataService', 'EnhancedRouter', 'SecurityCLI',
+      'ServiceOrchestrator', 'DeploymentValidator'
+    ];
+    
+    const missing = required.filter(exp => !exports.includes(exp));
+    if (missing.length > 0) {
+      throw new Error('Missing key exports: ' + missing.join(', '));
+    }
+    
+    console.log('All key exports present');
+    process.exit(0);
+  })
+  .catch(err => {
+    console.error('Import failed: ' + err.message);
+    process.exit(1);
+  });
+`;
+
+    const testFile = path.join(testDir, 'test-imports.mjs');
+    fs.writeFileSync(testFile, testScript);
+    
+    try {
+      const output = execSync('node test-imports.mjs', {
+        cwd: testDir,
+        encoding: 'utf8'
+      });
+      
+      log(colors.green, `✅ ${output.trim()}\n`);
+    } catch (e) {
+      throw new Error(`Import test failed: ${e.message}`);
+    }
+
+    // Step 5: Test CLI commands with actual subcommands
+    log(colors.blue, '🛠️  Step 5: Testing CLI commands');
+    log(colors.blue, '─'.repeat(60));
+
+    // On Windows, npm bin shims are bash scripts. Use direct node execution instead
+    const cliTests = [
+      { 
+        path: 'dist/cli/clodo-service.js',
+        args: '--version',
+        name: 'clodo-service --version'
+      },
+      { 
+        path: 'dist/cli/clodo-simple.js',
+        args: '--help',
+        name: 'clodo-simple --help'
+      },
+      { 
+        path: 'dist/cli/security-cli.js',
+        args: '--help',
+        name: 'clodo-security --help'
+      }
+    ];
+
+    for (const test of cliTests) {
+      try {
+        const cliPath = path.join(testDir, 'node_modules/@tamyla/clodo-framework', test.path);
+        
+        if (!fs.existsSync(cliPath)) {
+          throw new Error(`CLI file not found: ${cliPath}`);
+        }
+        
+        const output = execSync(`node ${cliPath} ${test.args}`, {
+          cwd: testDir,
+          stdio: 'pipe',
+          encoding: 'utf8',
+          timeout: 5000
+        });
+        
+        // Check that we got actual output
+        if (output && output.trim().length > 0) {
+          log(colors.green, `✅ ${test.name} executed successfully`);
+        } else {
+          throw new Error('No output from command');
+        }
+      } catch (e) {
+        throw new Error(`CLI test failed for ${test.name}: ${e.message}`);
+      }
+    }
+
+    log(colors.green, '\n');
+
+    // Success
+    log(colors.cyan, '═'.repeat(60));
+    log(colors.green, '✅ ALL DOWNSTREAM USER TESTS PASSED!');
+    log(colors.cyan, '═'.repeat(60));
+    log(colors.green, `\n🎉 Package is safe for downstream users to install!\n`);
+    
+    process.exit(0);
+
+  } catch (error) {
+    log(colors.red, '\n' + '═'.repeat(60));
+    log(colors.red, '❌ DOWNSTREAM USER TEST FAILED');
+    log(colors.red, '═'.repeat(60));
+    log(colors.red, `\nError: ${error.message}\n`);
+    process.exit(1);
+
+  } finally {
+    // Cleanup
+    if (fs.existsSync(testDir)) {
+      fs.rmSync(testDir, { recursive: true, force: true });
+    }
+    if (tarballPath && fs.existsSync(tarballPath)) {
+      fs.unlinkSync(tarballPath);
+    }
+  }
+}
+
+testDownstreamInstall().catch(err => {
+  log(colors.red, `\n❌ Fatal error: ${err.message}\n`);
+  process.exit(1);
+});

package/scripts/test-local-package.ps1

@@ -0,0 +1,126 @@
+#!/usr/bin/env pwsh
+# Local package testing script for Windows
+
+$ErrorActionPreference = "Stop"
+
+Write-Host ""
+Write-Host "=========================================================" -ForegroundColor Cyan
+Write-Host "  LOCAL PACKAGE TESTING SCRIPT" -ForegroundColor Cyan
+Write-Host "=========================================================" -ForegroundColor Cyan
+Write-Host ""
+
+$testsPassed = 0
+$tempDir = Join-Path $env:TEMP "clodo-test-$(Get-Random)"
+New-Item -ItemType Directory -Path $tempDir -Force | Out-Null
+
+try {
+    # Test 1: Build
+    Write-Host "[1] Building package..."
+    npm run build > $null 2>&1
+    Write-Host "    OK - Build complete" -ForegroundColor Green
+    $testsPassed++
+    
+    # Test 2: Create tarball
+    Write-Host "[2] Creating tarball..."
+    $tarball = npm pack --silent 2>$null
+    if ($tarball) {
+        Write-Host "    OK - Created: $tarball" -ForegroundColor Green
+        $testsPassed++
+    } else {
+        throw "Failed to create package"
+    }
+    
+    # Test 3: Verify import paths in source
+    Write-Host "[3] Verifying import paths..."
+    $opsContent = Get-Content "dist/utils/cloudflare/ops.js" -Raw
+    if ($opsContent -match "from.*lib/shared/cloudflare/ops") {
+        Write-Host "    OK - ops.js paths correct" -ForegroundColor Green
+    } else {
+        throw "ops.js has wrong import path"
+    }
+    $testsPassed++
+    
+    # Test 4: Test CLI
+    Write-Host "[4] Testing CLI..."
+    $cliOutput = & node dist/cli/clodo-service.js --help 2>&1
+    if ($LASTEXITCODE -eq 0) {
+        Write-Host "    OK - CLI functional" -ForegroundColor Green
+        $testsPassed++
+    } else {
+        throw "CLI test failed"
+    }
+    
+    # Test 5: Test module import
+    Write-Host "[5] Testing module imports..."
+    $testCode = @'
+const mod = require('./dist/index.js');
+if (Object.keys(mod).length > 20) {
+    console.log('Module loaded with ' + Object.keys(mod).length + ' exports');
+} else {
+    throw 'Not enough exports';
+}
+'@
+    $testCode | & node 2>$null
+    if ($LASTEXITCODE -eq 0) {
+        Write-Host "    OK - Module imports work" -ForegroundColor Green
+        $testsPassed++
+    } else {
+        throw "Module import failed"
+    }
+    
+    # Test 6: Verify tarball contents
+    Write-Host "[6] Verifying tarball structure..."
+    $tarballPath = Get-Item $tarball -ErrorAction Stop
+    Push-Location $tempDir
+    tar -xzf $tarballPath.FullName | Out-Null
+    $extractedDir = Get-ChildItem -Directory | Select-Object -First 1
+    
+    if ((Test-Path "$($extractedDir.FullName)/package.json") -and 
+        (Test-Path "$($extractedDir.FullName)/dist/index.js")) {
+        Write-Host "    OK - Tarball structure valid" -ForegroundColor Green
+        $testsPassed++
+    } else {
+        throw "Tarball missing required files"
+    }
+    
+    # Success!
+    Write-Host ""
+    Write-Host "=========================================================" -ForegroundColor Green
+    Write-Host "SUCCESS - ALL TESTS PASSED" -ForegroundColor Green
+    Write-Host "=========================================================" -ForegroundColor Green
+    Write-Host ""
+    
+    $packageSize = "{0:N2} MB" -f ((Get-Item $tarball -ErrorAction SilentlyContinue).Length / 1MB)
+    Write-Host "SUMMARY:"
+    Write-Host "  Package: $tarball"
+    Write-Host "  Size: $packageSize"
+    Write-Host "  Tests: $testsPassed/6 passed"
+    Write-Host ""
+    Write-Host "NEXT STEPS:"
+    Write-Host "  1. git add ."
+    Write-Host "  2. git commit -m 'fix: correct import paths'"
+    Write-Host "  3. git push origin main"
+    Write-Host "  4. GitHub Actions will run semantic-release"
+    Write-Host ""
+
+} catch {
+    Write-Host ""
+    Write-Host "=========================================================" -ForegroundColor Red
+    Write-Host "FAILED - TESTS DID NOT PASS" -ForegroundColor Red
+    Write-Host "=========================================================" -ForegroundColor Red
+    Write-Host ""
+    Write-Host "Error: $_" -ForegroundColor Red
+    Write-Host ""
+    Write-Host "DO NOT PUSH - Fix issues first!" -ForegroundColor Yellow
+    Write-Host ""
+    exit 1
+
+} finally {
+    Pop-Location -ErrorAction SilentlyContinue
+    Pop-Location -ErrorAction SilentlyContinue
+    Pop-Location -ErrorAction SilentlyContinue
+    
+    if (Test-Path $tempDir) {
+        Remove-Item -Recurse -Force $tempDir -ErrorAction SilentlyContinue
+    }
+}

package/scripts/test-local-package.sh

@@ -0,0 +1,166 @@
+#!/bin/bash
+# scripts/test-local-package.sh
+# 
+# Local package testing before semantic release
+# Simulates what npm will receive when published
+# 
+# Usage: bash scripts/test-local-package.sh
+
+set -e
+
+echo "════════════════════════════════════════════════════════════"
+echo "  LOCAL PACKAGE TESTING SCRIPT"
+echo "════════════════════════════════════════════════════════════"
+echo ""
+
+# Colors for output
+GREEN='\033[0;32m'
+RED='\033[0;31m'
+YELLOW='\033[1;33m'
+NC='\033[0m' # No Color
+
+TEMP_DIR="/tmp/clodo-package-test-$$"
+PACKAGE_NAME="clodo-framework-local.tgz"
+
+cleanup() {
+  echo ""
+  echo "Cleaning up temporary directory: $TEMP_DIR"
+  rm -rf "$TEMP_DIR"
+}
+
+trap cleanup EXIT
+
+echo "1️⃣  BUILDING PACKAGE..."
+echo "   Running: npm run build"
+npm run build > /dev/null 2>&1
+echo -e "   ${GREEN}✅ Build successful${NC}"
+
+echo ""
+echo "2️⃣  CREATING TARBALL..."
+echo "   Running: npm pack --dry-run"
+if npm pack > /dev/null 2>&1; then
+  echo -e "   ${GREEN}✅ Package dry-run successful${NC}"
+else
+  echo -e "   ${RED}❌ Package creation failed${NC}"
+  exit 1
+fi
+
+echo ""
+echo "3️⃣  PACKAGING FOR REAL..."
+TARBALL=$(npm pack --silent)
+echo -e "   ${GREEN}✅ Created: $TARBALL${NC}"
+
+echo ""
+echo "4️⃣  EXTRACTING PACKAGE..."
+mkdir -p "$TEMP_DIR"
+cd "$TEMP_DIR"
+tar -xzf "../${TARBALL}"
+EXTRACTED_DIR=$(ls -d */ | head -1)
+cd "$EXTRACTED_DIR"
+echo -e "   ${GREEN}✅ Extracted to: $EXTRACTED_DIR${NC}"
+
+echo ""
+echo "5️⃣  CHECKING PACKAGE STRUCTURE..."
+echo "   Verifying critical files exist:"
+
+FILES_TO_CHECK=(
+  "dist/index.js"
+  "dist/cli/clodo-service.js"
+  "dist/lib/shared/cloudflare/ops.js"
+  "dist/utils/cloudflare/ops.js"
+  "dist/simple-api.js"
+  "dist/config/validation-config.json"
+  "package.json"
+  "README.md"
+  "LICENSE"
+)
+
+for file in "${FILES_TO_CHECK[@]}"; do
+  if [ -f "$file" ]; then
+    echo -e "   ${GREEN}✅${NC} $file"
+  else
+    echo -e "   ${RED}❌${NC} $file (MISSING!)"
+    exit 1
+  fi
+done
+
+echo ""
+echo "6️⃣  VERIFYING IMPORT PATHS..."
+echo "   Checking dist/utils/cloudflare/ops.js"
+if grep -q "from.*lib/shared/cloudflare/ops.js" "dist/utils/cloudflare/ops.js"; then
+  echo -e "   ${GREEN}✅${NC} ops.js has correct import path"
+else
+  echo -e "   ${RED}❌${NC} ops.js has WRONG import path"
+  exit 1
+fi
+
+echo "   Checking CLI command imports"
+if grep -q "from '../../lib/shared/utils/cli-options.js'" "dist/cli/commands/create.js"; then
+  echo -e "   ${GREEN}✅${NC} CLI commands have correct import paths"
+else
+  echo -e "   ${RED}❌${NC} CLI commands have WRONG import paths"
+  exit 1
+fi
+
+echo ""
+echo "7️⃣  TESTING CLI FROM PACKAGE..."
+echo "   Running: node dist/cli/clodo-service.js --help"
+if node dist/cli/clodo-service.js --help > /dev/null 2>&1; then
+  echo -e "   ${GREEN}✅ CLI works${NC}"
+else
+  echo -e "   ${RED}❌ CLI is broken${NC}"
+  exit 1
+fi
+
+echo ""
+echo "8️⃣  TESTING MODULE IMPORTS FROM PACKAGE..."
+echo "   Testing: require('./dist/index.js')"
+if node -e "
+  try {
+    const mod = require('./dist/index.js');
+    console.log('   ✅ Module imports successfully');
+    console.log('   ✅ Found', Object.keys(mod).length, 'exports');
+  } catch (e) {
+    console.log('   ❌ Module import failed:', e.message);
+    process.exit(1);
+  }
+" 2>/dev/null; then
+  echo -e "   ${GREEN}✅ Module imports work${NC}"
+else
+  echo -e "   ${RED}❌ Module import failed${NC}"
+  exit 1
+fi
+
+echo ""
+echo "9️⃣  TESTING SPECIFIC EXPORTS..."
+node -e "
+  const mod = require('./dist/index.js');
+  const testExports = ['Clodo', 'ConfigurationManager', 'EnhancedRouter', 'ServiceOrchestrator'];
+  
+  testExports.forEach(exp => {
+    if (mod[exp]) {
+      console.log('   ✅ ' + exp);
+    } else {
+      console.log('   ❌ ' + exp + ' NOT FOUND');
+      process.exit(1);
+    }
+  });
+" 2>/dev/null || exit 1
+
+echo ""
+echo "════════════════════════════════════════════════════════════"
+echo -e "${GREEN}✅ ALL LOCAL PACKAGE TESTS PASSED${NC}"
+echo "════════════════════════════════════════════════════════════"
+echo ""
+echo "📦 SUMMARY:"
+echo "   Package file: $TARBALL"
+echo "   Package size: $(du -h ../"$TARBALL" | cut -f1)"
+echo ""
+echo "🚀 NEXT STEPS:"
+echo "   1. Commit your changes: git add . && git commit -m 'fix: path issues'"
+echo "   2. Push to main: git push origin main"
+echo "   3. GitHub Actions will run semantic-release"
+echo "   4. Package will be published to npm automatically"
+echo ""
+
+# Cleanup is automatic via trap