@jtalk22/slack-mcp 3.1.0 → 3.2.1

package/scripts/verify-core.js

@@ -1,159 +0,0 @@
-#!/usr/bin/env node
-/**
- * Core Stability Verification Script
- *
- * Tests:
- * 1. Atomic write - no .tmp artifacts remain after write
- * 2. Server exits cleanly (unref timer doesn't cause zombie)
- */
-
-import { writeFileSync, readFileSync, existsSync, renameSync, unlinkSync, readdirSync } from "fs";
-import { spawn } from "child_process";
-import { homedir } from "os";
-import { join, dirname } from "path";
-import { fileURLToPath } from "url";
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const TEST_DIR = join(homedir(), ".slack-mcp-test");
-const TEST_FILE = join(TEST_DIR, "test-atomic.json");
-
-// ============ Test 1: Atomic Write ============
-
-function atomicWriteSync(filePath, content) {
-  const tempPath = `${filePath}.${process.pid}.tmp`;
-  try {
-    writeFileSync(tempPath, content);
-    renameSync(tempPath, filePath);
-  } catch (e) {
-    try { unlinkSync(tempPath); } catch {}
-    throw e;
-  }
-}
-
-async function testAtomicWrite() {
-  console.log("\n[TEST 1] Atomic Write");
-  console.log("─".repeat(40));
-
-  // Setup - ensure test directory exists
-  const { execSync } = await import("child_process");
-  try {
-    execSync(`mkdir -p "${TEST_DIR}"`);
-  } catch {}
-
-  // Test successful write
-  const testData = { test: "data", timestamp: Date.now() };
-  atomicWriteSync(TEST_FILE, JSON.stringify(testData, null, 2));
-
-  // Verify file exists
-  if (!existsSync(TEST_FILE)) {
-    console.log("  FAIL: File was not created");
-    return false;
-  }
-
-  // Verify content
-  const readBack = JSON.parse(readFileSync(TEST_FILE, "utf-8"));
-  if (readBack.test !== "data") {
-    console.log("  FAIL: Content mismatch");
-    return false;
-  }
-
-  // Check for .tmp artifacts in test dir
-  const files = readdirSync(TEST_DIR);
-  const tmpFiles = files.filter(f => f.endsWith(".tmp"));
-  if (tmpFiles.length > 0) {
-    console.log(`  FAIL: Found .tmp artifacts: ${tmpFiles.join(", ")}`);
-    return false;
-  }
-
-  // Cleanup
-  try { unlinkSync(TEST_FILE); } catch {}
-
-  console.log("  PASS: Atomic write completed, no .tmp artifacts");
-  return true;
-}
-
-// ============ Test 2: Server Exit (No Zombie) ============
-
-async function testServerExit() {
-  console.log("\n[TEST 2] Server Clean Exit (No Zombie)");
-  console.log("─".repeat(40));
-
-  const serverPath = join(__dirname, "../src/server.js");
-
-  return new Promise((resolve) => {
-    const timeout = 5000; // 5 second timeout
-    let exitCode = null;
-    let timedOut = false;
-
-    // Spawn server process
-    const proc = spawn("node", [serverPath], {
-      stdio: ["pipe", "pipe", "pipe"],
-      env: { ...process.env, SLACK_TOKEN: "test", SLACK_COOKIE: "test" }
-    });
-
-    // Set timeout - if process doesn't exit after stdin closes, it's a zombie
-    const timer = setTimeout(() => {
-      timedOut = true;
-      console.log("  FAIL: Server did not exit within 5s (zombie process detected)");
-      proc.kill("SIGKILL");
-      resolve(false);
-    }, timeout);
-
-    proc.on("exit", (code) => {
-      exitCode = code;
-      clearTimeout(timer);
-      if (!timedOut) {
-        console.log(`  PASS: Server exited cleanly (code: ${code})`);
-        resolve(true);
-      }
-    });
-
-    proc.on("error", (err) => {
-      clearTimeout(timer);
-      console.log(`  INFO: Server spawn error (expected if no SDK): ${err.message}`);
-      // This is OK - we're testing exit behavior, not full functionality
-      resolve(true);
-    });
-
-    // Close stdin immediately to simulate MCP client disconnect
-    proc.stdin.end();
-
-    // Give it a moment then send SIGTERM
-    setTimeout(() => {
-      if (exitCode === null && !timedOut) {
-        proc.kill("SIGTERM");
-      }
-    }, 1000);
-  });
-}
-
-// ============ Main ============
-
-async function main() {
-  console.log("╔════════════════════════════════════════╗");
-  console.log("║  Core Stability Verification Tests     ║");
-  console.log("╚════════════════════════════════════════╝");
-
-  const results = [];
-
-  results.push(await testAtomicWrite());
-  results.push(await testServerExit());
-
-  console.log("\n" + "═".repeat(40));
-  const passed = results.filter(r => r).length;
-  const total = results.length;
-
-  if (passed === total) {
-    console.log(`\n✓ ALL TESTS PASSED (${passed}/${total})`);
-    console.log("\nCore stability features verified");
-    process.exit(0);
-  } else {
-    console.log(`\n✗ TESTS FAILED (${passed}/${total})`);
-    process.exit(1);
-  }
-}
-
-main().catch(e => {
-  console.error("Test error:", e);
-  process.exit(1);
-});

package/scripts/verify-install-flow.js

@@ -1,193 +0,0 @@
-#!/usr/bin/env node
-
-import { spawnSync } from "node:child_process";
-import { mkdtempSync, readFileSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
-
-const PKG = "@jtalk22/slack-mcp";
-const repoRoot = join(dirname(fileURLToPath(import.meta.url)), "..");
-const strictPublished = process.argv.includes("--strict-published");
-const PUBLISHED_SPEC = `${PKG}@latest`;
-const localVersion = JSON.parse(readFileSync(join(repoRoot, "package.json"), "utf8")).version;
-
-function runNpx(args, options = {}) {
-  const cmdArgs = ["-y", PUBLISHED_SPEC, ...args];
-  const result = spawnSync("npx", cmdArgs, {
-    cwd: options.cwd,
-    env: options.env,
-    encoding: "utf8",
-    timeout: 120000,
-  });
-
-  return {
-    args: cmdArgs.join(" "),
-    status: result.status,
-    stdout: (result.stdout || "").trim(),
-    stderr: (result.stderr || "").trim(),
-    error: result.error,
-  };
-}
-
-function assert(condition, message, details = "") {
-  if (!condition) {
-    const suffix = details ? `\n${details}` : "";
-    throw new Error(`${message}${suffix}`);
-  }
-}
-
-function printResult(label, result) {
-  console.log(`\n[${label}] npx ${result.args}`);
-  console.log(`exit=${result.status}`);
-  if (result.stdout) {
-    console.log("stdout:");
-    console.log(result.stdout);
-  }
-  if (result.stderr) {
-    console.log("stderr:");
-    console.log(result.stderr);
-  }
-}
-
-function runLocalSetupStatus(options = {}) {
-  const result = spawnSync("node", [join(repoRoot, "scripts/setup-wizard.js"), "--status"], {
-    cwd: repoRoot,
-    env: options.env,
-    encoding: "utf8",
-    timeout: 120000,
-  });
-
-  return {
-    args: "node scripts/setup-wizard.js --status",
-    status: result.status,
-    stdout: (result.stdout || "").trim(),
-    stderr: (result.stderr || "").trim(),
-    error: result.error,
-  };
-}
-
-function runLocalDoctor(options = {}) {
-  const result = spawnSync("node", [join(repoRoot, "scripts/setup-wizard.js"), "--doctor"], {
-    cwd: repoRoot,
-    env: options.env,
-    encoding: "utf8",
-    timeout: 120000,
-  });
-
-  return {
-    args: "node scripts/setup-wizard.js --doctor",
-    status: result.status,
-    stdout: (result.stdout || "").trim(),
-    stderr: (result.stderr || "").trim(),
-    error: result.error,
-  };
-}
-
-function main() {
-  const testHome = mkdtempSync(join(tmpdir(), "slack-mcp-install-check-"));
-
-  // Force a clean environment so --status reflects missing credentials.
-  const env = { ...process.env, HOME: testHome, USERPROFILE: testHome };
-  delete env.SLACK_TOKEN;
-  delete env.SLACK_COOKIE;
-
-  try {
-    const versionResult = runNpx(["--version"], { cwd: testHome, env });
-    printResult("version", versionResult);
-    assert(
-      versionResult.status === 0,
-      "Expected --version to exit 0",
-      versionResult.stderr || versionResult.stdout,
-    );
-    const publishedMatchesLocal = versionResult.stdout.includes(localVersion);
-    if (strictPublished) {
-      assert(
-        publishedMatchesLocal,
-        `Expected published npx version to match local ${localVersion}`,
-        versionResult.stdout,
-      );
-    } else if (!publishedMatchesLocal) {
-      console.log(
-        `warning: npx resolved ${versionResult.stdout || "unknown"} while local version is ${localVersion}; strict published checks are deferred until publish.`
-      );
-    }
-
-    const helpResult = runNpx(["--help"], { cwd: testHome, env });
-    printResult("help", helpResult);
-    assert(
-      helpResult.status === 0,
-      "Expected --help to exit 0",
-      helpResult.stderr || helpResult.stdout,
-    );
-
-    const statusResult = runNpx(["--status"], { cwd: testHome, env });
-    printResult("status", statusResult);
-    assert(
-      statusResult.status !== 0,
-      "Expected --status to exit non-zero when credentials are missing",
-      statusResult.stderr || statusResult.stdout,
-    );
-    if (strictPublished) {
-      assert(
-        !statusResult.stderr.includes("Attempting Chrome auto-extraction"),
-        "Expected npx --status to be read-only without auto-extraction side effects",
-        statusResult.stderr,
-      );
-    } else if (statusResult.stderr.includes("Attempting Chrome auto-extraction")) {
-      console.log("warning: published npx --status still has extraction side effects; re-run with --strict-published after publish.");
-    }
-
-    const localStatusResult = runLocalSetupStatus({ env });
-    printResult("local-status", localStatusResult);
-    assert(
-      localStatusResult.status !== 0,
-      "Expected local --status to exit non-zero when credentials are missing",
-      localStatusResult.stderr || localStatusResult.stdout,
-    );
-    assert(
-      !localStatusResult.stderr.includes("Attempting Chrome auto-extraction"),
-      "Expected local --status to be read-only without auto-extraction side effects",
-      localStatusResult.stderr,
-    );
-
-    const localDoctorMissingResult = runLocalDoctor({ env });
-    printResult("local-doctor-missing", localDoctorMissingResult);
-    assert(
-      localDoctorMissingResult.status === 1,
-      "Expected local --doctor to exit 1 when credentials are missing",
-      localDoctorMissingResult.stderr || localDoctorMissingResult.stdout,
-    );
-
-    const invalidEnv = {
-      ...env,
-      SLACK_TOKEN: "xoxc-invalid-token",
-      SLACK_COOKIE: "xoxd-invalid-cookie",
-    };
-    const localDoctorInvalidResult = runLocalDoctor({ env: invalidEnv });
-    printResult("local-doctor-invalid", localDoctorInvalidResult);
-    assert(
-      localDoctorInvalidResult.status === 2,
-      "Expected local --doctor to exit 2 when credentials are invalid",
-      localDoctorInvalidResult.stderr || localDoctorInvalidResult.stdout,
-    );
-
-    const runtimeEnv = {
-      ...invalidEnv,
-      SLACK_MCP_AUTH_TEST_URL: "http://127.0.0.1:9/auth.test"
-    };
-    const localDoctorRuntimeResult = runLocalDoctor({ env: runtimeEnv });
-    printResult("local-doctor-runtime", localDoctorRuntimeResult);
-    assert(
-      localDoctorRuntimeResult.status === 3,
-      "Expected local --doctor to exit 3 when runtime connectivity fails",
-      localDoctorRuntimeResult.stderr || localDoctorRuntimeResult.stdout,
-    );
-
-    console.log("\nInstall flow verification passed.");
-  } finally {
-    rmSync(testHome, { recursive: true, force: true });
-  }
-}
-
-main();

package/scripts/verify-web.js

@@ -1,273 +0,0 @@
-#!/usr/bin/env node
-/**
- * Web UI Verification Script
- *
- * Tests:
- * 1. Server starts and prints Magic Link
- * 2. /demo.html contains "STATIC PREVIEW" banner
- * 3. /?key=... serves the dashboard (index.html)
- * 4. /demo-video.html and /public/demo-video.html media assets are reachable
- * 5. Server shuts down cleanly
- */
-
-import { spawn } from "child_process";
-import { dirname, join } from "path";
-import { fileURLToPath } from "url";
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const SERVER_PATH = join(__dirname, "../src/web-server.js");
-const PORT = 3456; // Use non-standard port to avoid conflicts
-const TIMEOUT = 15000;
-
-let serverProc = null;
-
-function log(msg) {
-  console.log(`  ${msg}`);
-}
-
-function cleanup() {
-  if (serverProc) {
-    serverProc.kill("SIGTERM");
-    serverProc = null;
-  }
-}
-
-process.on("exit", cleanup);
-process.on("SIGINT", () => { cleanup(); process.exit(1); });
-process.on("SIGTERM", () => { cleanup(); process.exit(1); });
-
-async function startServer() {
-  return new Promise((resolve, reject) => {
-    let magicLink = null;
-    let apiKey = null;
-    let output = "";
-
-    serverProc = spawn("node", [SERVER_PATH], {
-      env: { ...process.env, PORT: String(PORT) },
-      stdio: ["pipe", "pipe", "pipe"]
-    });
-
-    const timeout = setTimeout(() => {
-      reject(new Error("Server startup timeout - no magic link detected"));
-    }, TIMEOUT);
-
-    serverProc.stderr.on("data", (data) => {
-      const text = data.toString();
-      output += text;
-
-      // Look for magic link pattern
-      const match = text.match(/Dashboard:\s*(http:\/\/[^\s]+)/);
-      if (match) {
-        magicLink = match[1];
-        // Extract key from URL
-        const keyMatch = magicLink.match(/[?&]key=([^&\s]+)/);
-        if (keyMatch) {
-          apiKey = keyMatch[1];
-        }
-      }
-
-      // Server is ready when we see the full banner
-      if (output.includes("Dashboard:") && output.includes("API Key:")) {
-        clearTimeout(timeout);
-        resolve({ magicLink, apiKey });
-      }
-    });
-
-    serverProc.on("error", (err) => {
-      clearTimeout(timeout);
-      reject(err);
-    });
-
-    serverProc.on("exit", (code) => {
-      if (code !== null && code !== 0) {
-        clearTimeout(timeout);
-        reject(new Error(`Server exited with code ${code}`));
-      }
-    });
-  });
-}
-
-async function testDemoPage() {
-  const url = `http://localhost:${PORT}/demo.html`;
-  const res = await fetch(url);
-
-  if (!res.ok) {
-    throw new Error(`Failed to fetch demo.html: ${res.status}`);
-  }
-
-  const html = await res.text();
-
-  if (!html.includes("STATIC PREVIEW")) {
-    throw new Error("demo.html missing 'STATIC PREVIEW' banner");
-  }
-
-  if (!html.includes("Who is Alex?")) {
-    throw new Error("demo.html missing anonymized 'Who is Alex?' scenario");
-  }
-
-  return true;
-}
-
-async function testDashboard(apiKey) {
-  const url = `http://localhost:${PORT}/?key=${apiKey}`;
-  const res = await fetch(url);
-
-  if (!res.ok) {
-    throw new Error(`Failed to fetch dashboard: ${res.status}`);
-  }
-
-  const html = await res.text();
-
-  // Should serve index.html (the dashboard)
-  if (!html.includes("Slack Web API")) {
-    throw new Error("Dashboard page missing 'Slack Web API' title");
-  }
-
-  if (!html.includes("authModal")) {
-    throw new Error("Dashboard missing auth modal");
-  }
-
-  return true;
-}
-
-async function testApiWithKey(apiKey) {
-  // Test that API rejects bad key
-  const badRes = await fetch(`http://localhost:${PORT}/health`, {
-    headers: { "Authorization": "Bearer bad-key" }
-  });
-
-  if (badRes.status !== 401) {
-    throw new Error(`Expected 401 for bad key, got ${badRes.status}`);
-  }
-
-  return true;
-}
-
-async function testDemoVideoAssets() {
-  const demoVideoPaths = ["/demo-video.html", "/public/demo-video.html"];
-  const requiredAssetCandidates = [
-    [
-      "/docs/images/demo-poster.png",
-      "https://jtalk22.github.io/slack-mcp-server/docs/images/demo-poster.png",
-    ],
-    [
-      "/docs/videos/demo-claude.webm",
-      "https://jtalk22.github.io/slack-mcp-server/docs/videos/demo-claude.webm",
-    ],
-  ];
-
-  for (const pagePath of demoVideoPaths) {
-    const demoVideoUrl = `http://localhost:${PORT}${pagePath}`;
-    const demoVideoRes = await fetch(demoVideoUrl);
-
-    if (!demoVideoRes.ok) {
-      throw new Error(`Failed to fetch ${pagePath}: ${demoVideoRes.status}`);
-    }
-
-    const demoVideoHtml = await demoVideoRes.text();
-
-    for (const candidates of requiredAssetCandidates) {
-      const matched = candidates.find((candidate) => demoVideoHtml.includes(candidate));
-      if (!matched) {
-        throw new Error(`${pagePath} missing expected media reference: ${candidates.join(" OR ")}`);
-      }
-
-      const assetUrl = matched.startsWith("http")
-        ? matched
-        : `http://localhost:${PORT}${matched}`;
-
-      const assetRes = await fetch(assetUrl);
-      if (!assetRes.ok) {
-        throw new Error(`Demo media not reachable from ${pagePath}: ${assetUrl} (status ${assetRes.status})`);
-      }
-    }
-  }
-
-  return true;
-}
-
-async function main() {
-  console.log("╔════════════════════════════════════════╗");
-  console.log("║  Web UI Verification Tests             ║");
-  console.log("╚════════════════════════════════════════╝");
-
-  const results = [];
-
-  try {
-    // Test 1: Server starts with magic link
-    console.log("\n[TEST 1] Server Startup & Magic Link");
-    console.log("─".repeat(40));
-
-    const { magicLink, apiKey } = await startServer();
-
-    if (!magicLink) {
-      throw new Error("No magic link found");
-    }
-    if (!apiKey) {
-      throw new Error("No API key found in magic link");
-    }
-
-    log(`Magic Link: ${magicLink}`);
-    log(`API Key: ${apiKey.substring(0, 20)}...`);
-    log("PASS: Server started with magic link");
-    results.push(true);
-
-    // Test 2: Demo page
-    console.log("\n[TEST 2] Demo Page (/demo.html)");
-    console.log("─".repeat(40));
-
-    await testDemoPage();
-    log("PASS: Demo page serves correctly with STATIC PREVIEW banner");
-    results.push(true);
-
-    // Test 3: Dashboard
-    console.log("\n[TEST 3] Dashboard (/?key=...)");
-    console.log("─".repeat(40));
-
-    await testDashboard(apiKey);
-    log("PASS: Dashboard serves with auth modal");
-    results.push(true);
-
-    // Test 4: API auth
-    console.log("\n[TEST 4] API Authentication");
-    console.log("─".repeat(40));
-
-    await testApiWithKey(apiKey);
-    log("PASS: API correctly rejects bad keys");
-    results.push(true);
-
-    // Test 5: Demo video/media paths
-    console.log("\n[TEST 5] Demo Video Media Reachability");
-    console.log("─".repeat(40));
-
-    await testDemoVideoAssets();
-    log("PASS: demo-video media assets are reachable");
-    results.push(true);
-
-  } catch (err) {
-    console.log(`  FAIL: ${err.message}`);
-    results.push(false);
-  } finally {
-    cleanup();
-  }
-
-  // Summary
-  console.log("\n" + "═".repeat(40));
-  const passed = results.filter(r => r).length;
-  const total = results.length;
-
-  if (passed === total) {
-    console.log(`\n✓ ALL TESTS PASSED (${passed}/${total})`);
-    console.log("\nWeb UI features verified");
-    process.exit(0);
-  } else {
-    console.log(`\n✗ TESTS FAILED (${passed}/${total})`);
-    process.exit(1);
-  }
-}
-
-main().catch(e => {
-  console.error("Test error:", e);
-  cleanup();
-  process.exit(1);
-});