@tdk-landscape/tdk-cli-core 1.3.53 → 1.3.54

package/cli/src/commands/__tests__/upgrade.test.ts

@@ -1,198 +0,0 @@
-import { createHash } from "node:crypto";
-import { chmodSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-
-const execSyncMock = vi.fn();
-vi.mock("node:child_process", () => ({
-  execSync: (...args: unknown[]) => execSyncMock(...args),
-}));
-
-const { upgradeViaBinary, isWritable, parseChecksums } = await import("../upgrade.js");
-
-import type { BinaryRelease } from "../upgrade.js";
-
-// Regression coverage for: `tdk upgrade` used to only swap the `tdk` binary
-// and leave the bundled tdk-cli/ engine+templates folder next to it stale or
-// missing, so an upgraded binary still crashed with the loadTemplate ENOENT
-// (the exact error this was meant to fix). It must refresh both, every time.
-const BINARY_BYTES = "fake tdk binary";
-const ENGINE_BYTES = "fake engine tarball";
-const sha256 = (text: string) => createHash("sha256").update(text).digest("hex");
-
-describe("upgradeViaBinary", () => {
-  let installDir: string;
-  let tdkPath: string;
-  let release: BinaryRelease;
-  let checksums: string;
-  const downloaded: string[] = [];
-
-  // Stands in for curl: `curl -fsSL -o "<path>" "<url>"` writes the asset's
-  // bytes to <path>; `curl -fsSL "<checksums url>"` returns checksums.txt.
-  function fakeExec(command: string): string {
-    const download = command.match(/^curl -fsSL -o (".*?") (".*?")$/);
-    if (download) {
-      const path = JSON.parse(download[1]) as string;
-      const url = JSON.parse(download[2]) as string;
-      writeFileSync(path, url === release.engineDownloadUrl ? ENGINE_BYTES : BINARY_BYTES);
-      downloaded.push(path);
-      return "";
-    }
-    if (command.includes(release.checksumsUrl)) return checksums;
-    return "";
-  }
-
-  beforeEach(() => {
-    installDir = mkdtempSync(join(tmpdir(), "tdk-upgrade-test-"));
-    tdkPath = join(installDir, "tdk");
-    release = {
-      tag: "v1.3.21",
-      assetName: "tdk-darwin-arm64",
-      downloadUrl: "https://example.test/tdk-darwin-arm64",
-      engineDownloadUrl: "https://example.test/tdk-cli-engine.tar.gz",
-      checksumsUrl: "https://example.test/checksums.txt",
-    };
-    checksums = [
-      `${sha256(BINARY_BYTES)}  tdk-darwin-arm64`,
-      `${sha256("other platform")}  tdk-linux-amd64`,
-      `${sha256(ENGINE_BYTES)}  tdk-cli-engine.tar.gz`,
-    ].join("\n");
-    execSyncMock.mockReset();
-    execSyncMock.mockImplementation(fakeExec);
-  });
-
-  afterEach(() => {
-    rmSync(installDir, { recursive: true, force: true });
-    for (const path of downloaded.splice(0)) rmSync(path, { force: true });
-  });
-
-  function commandsRun(): string[] {
-    return execSyncMock.mock.calls.map((call) => call[0] as string);
-  }
-
-  it("downloads and extracts the bundled engine tarball next to the binary, not just the binary itself", async () => {
-    const ok = await upgradeViaBinary(tdkPath, release);
-    expect(ok).toBe(true);
-
-    const commands = commandsRun();
-    const engineDir = join(installDir, "tdk-cli");
-
-    expect(commands.some((c) => c.includes("curl") && c.includes(release.downloadUrl))).toBe(true);
-    expect(commands.some((c) => c.includes("curl") && c.includes(release.engineDownloadUrl))).toBe(
-      true,
-    );
-    expect(commands.some((c) => c.startsWith("rm -rf") && c.includes(engineDir))).toBe(true);
-    expect(commands.some((c) => c.startsWith("mkdir -p") && c.includes(engineDir))).toBe(true);
-    expect(
-      commands.some(
-        (c) =>
-          c.includes("tar -xzf") && c.includes(engineDir) && c.includes("--strip-components=1"),
-      ),
-    ).toBe(true);
-  });
-
-  it("downloads both the binary and the engine tarball before installing either", async () => {
-    await upgradeViaBinary(tdkPath, release);
-
-    const commands = commandsRun();
-    const binaryCurlIndex = commands.findIndex((c) => c.includes(release.downloadUrl));
-    const engineCurlIndex = commands.findIndex((c) => c.includes(release.engineDownloadUrl));
-    const mvIndex = commands.findIndex((c) => c.startsWith("mv "));
-    const tarIndex = commands.findIndex((c) => c.includes("tar -xzf"));
-
-    expect(binaryCurlIndex).toBeGreaterThanOrEqual(0);
-    expect(engineCurlIndex).toBeGreaterThanOrEqual(0);
-    expect(mvIndex).toBeGreaterThan(binaryCurlIndex);
-    expect(mvIndex).toBeGreaterThan(engineCurlIndex);
-    expect(tarIndex).toBeGreaterThan(engineCurlIndex);
-  });
-
-  it("verifies checksums after both downloads and before installing anything", async () => {
-    await upgradeViaBinary(tdkPath, release);
-
-    const commands = commandsRun();
-    const checksumsIndex = commands.findIndex((c) => c.includes(release.checksumsUrl));
-    const engineCurlIndex = commands.findIndex((c) => c.includes(release.engineDownloadUrl));
-    const mvIndex = commands.findIndex((c) => c.startsWith("mv "));
-
-    expect(checksumsIndex).toBeGreaterThan(engineCurlIndex);
-    expect(mvIndex).toBeGreaterThan(checksumsIndex);
-  });
-
-  it.each([
-    [
-      "the binary does not match",
-      () => checksums.replace(sha256(BINARY_BYTES), sha256("tampered")),
-    ],
-    [
-      "the engine tarball does not match",
-      () => checksums.replace(sha256(ENGINE_BYTES), sha256("x")),
-    ],
-    ["checksums.txt has no entry for the binary", () => checksums.replace("tdk-darwin-arm64", "x")],
-  ])("refuses to install when %s", async (_case, makeChecksums) => {
-    checksums = makeChecksums();
-    const ok = await upgradeViaBinary(tdkPath, release);
-
-    expect(ok).toBe(false);
-    const commands = commandsRun();
-    expect(commands.some((c) => c.startsWith("mv "))).toBe(false);
-    expect(commands.some((c) => c.includes("tar -xzf"))).toBe(false);
-    expect(commands.some((c) => c.startsWith("rm -rf"))).toBe(false);
-  });
-
-  it("still installs releases whose checksums.txt predates the engine entry", async () => {
-    checksums = checksums
-      .split("\n")
-      .filter((line) => !line.endsWith("tdk-cli-engine.tar.gz"))
-      .join("\n");
-    const ok = await upgradeViaBinary(tdkPath, release);
-
-    expect(ok).toBe(true);
-    expect(commandsRun().some((c) => c.startsWith("mv "))).toBe(true);
-  });
-
-  // root ignores the write-permission bit (accessSync(W_OK) legitimately
-  // returns true even on a 0o555 dir), so these checks only hold for
-  // non-root users.
-  const isRoot = process.getuid?.() === 0;
-
-  it.skipIf(isRoot)(
-    "never shells out to sudo, and fails fast without touching the network when the install dir isn't writable",
-    async () => {
-      chmodSync(installDir, 0o555);
-      try {
-        const ok = await upgradeViaBinary(tdkPath, release);
-        expect(ok).toBe(false);
-        expect(execSyncMock).not.toHaveBeenCalled();
-        expect(commandsRun().some((c) => c.includes("sudo"))).toBe(false);
-      } finally {
-        chmodSync(installDir, 0o755);
-      }
-    },
-  );
-
-  it.skipIf(isRoot)("isWritable reflects actual filesystem permissions", () => {
-    expect(isWritable(installDir)).toBe(true);
-    chmodSync(installDir, 0o555);
-    try {
-      expect(isWritable(installDir)).toBe(false);
-    } finally {
-      chmodSync(installDir, 0o755);
-    }
-  });
-});
-
-describe("parseChecksums", () => {
-  it("reads both text-mode and binary-mode (*name) shasum lines and ignores junk", () => {
-    const a = "a".repeat(64);
-    const b = "B".repeat(64);
-    const sums = parseChecksums(
-      `${a}  tdk-linux-amd64\n${b} *tdk-cli-engine.tar.gz\n\nnot a checksum line\n`,
-    );
-
-    expect(sums.get("tdk-linux-amd64")).toBe(a);
-    expect(sums.get("tdk-cli-engine.tar.gz")).toBe("b".repeat(64));
-    expect(sums.size).toBe(2);
-  });
-});

package/cli/src/commands/completion.ts

@@ -1,300 +0,0 @@
-import { existsSync, mkdirSync, writeFileSync } from "node:fs";
-import { homedir } from "node:os";
-import { join } from "node:path";
-import chalk from "chalk";
-import { Command } from "commander";
-
-const BASH_COMPLETION = `# TDK CLI Bash Completion
-_tdk_completions() {
-    local cur prev opts
-    COMPREPLY=()
-    cur="\${COMP_WORDS[COMP_CWORD]}"
-    prev="\${COMP_WORDS[COMP_CWORD-1]}"
-    
-    # Main commands
-    local commands="project projects stack stacks resource resources up down status ui doctor version completion help upgrade"
-    
-    # Options for specific commands
-    case "\${prev}" in
-        tdk)
-            COMPREPLY=( $(compgen -W "\${commands}" -- \${cur}) )
-            return 0
-            ;;
-        up)
-            # Complete with stack names and resource names
-            local stacks=$(tdk stacks 2>/dev/null | grep '^  -' | sed 's/^  - //' | awk '{print $1}')
-            local resources=$(tdk resources 2>/dev/null | grep -E '^[a-z0-9-]+' | awk '{print $1}')
-            COMPREPLY=( $(compgen -W "\${stacks} \${resources}" -- \${cur}) )
-            return 0
-            ;;
-        stack)
-            # Complete with existing stack names
-            local stacks=$(tdk stacks 2>/dev/null | grep '^  -' | sed 's/^  - //' | awk '{print $1}')
-            COMPREPLY=( $(compgen -W "\${stacks}" -- \${cur}) )
-            return 0
-            ;;
-        resource)
-            # Suggest common resource names or types
-            COMPREPLY=( $(compgen -W "api frontend backend worker --type --stack" -- \${cur}) )
-            return 0
-            ;;
-        --type)
-            COMPREPLY=( $(compgen -W "backend frontend worker" -- \${cur}) )
-            return 0
-            ;;
-        --stack)
-            local stacks=$(tdk stacks 2>/dev/null | grep '^  -' | sed 's/^  - //' | awk '{print $1}')
-            COMPREPLY=( $(compgen -W "\${stacks}" -- \${cur}) )
-            return 0
-            ;;
-    esac
-    
-    # Global options
-    if [[ \${cur} == -* ]]; then
-        local options="--verbose --version --help --force --dry-run"
-        COMPREPLY=( $(compgen -W "\${options}" -- \${cur}) )
-        return 0
-    fi
-}
-
-complete -F _tdk_completions tdk
-`;
-
-const ZSH_COMPLETION = `#compdef tdk
-
-# TDK CLI Zsh Completion
-
-_tdk() {
-    local curcontext="$curcontext" state line
-    typeset -A opt_args
-    
-    _arguments -C 
-        '(-h --help)'{-h,--help}'[Show help]' 
-        '(-v --version)'{-v,--version}'[Show version]' 
-        '--verbose[Enable verbose output]' 
-        '1: :_tdk_commands' 
-        '*:: :->args'
-    
-    case "$line[1]" in
-        up)
-            _arguments 
-                '--verbose[Enable verbose output]'
-                '1: :_tdk_stacks_and_resources'
-            ;;
-        stack)
-            _arguments 
-                '--list[List resources without a stack]'
-                '1: :_tdk_stacks'
-            ;;
-        stacks)
-            _arguments 
-                '--services[Include resources in each stack]'
-            ;;
-        resource)
-            _arguments 
-                '--type[Resource type]:type:(backend frontend worker)' 
-                '--stack[Assign to stack]:stack:_tdk_stacks' 
-                '1:resource name:'
-            ;;
-        resources)
-            _arguments 
-                '--stack[Filter by stack]:stack:_tdk_stacks' 
-                '--no-stack[Show unassigned only]' 
-                '--ports[Show port assignments]'
-            ;;
-        projects)
-            _arguments 
-                '--check[Validate master configs]'
-            ;;
-        doctor)
-            _arguments 
-                '--fix[Attempt to fix issues]'
-            ;;
-        upgrade)
-            _arguments
-                '--force[Force upgrade even if on latest]'
-                '--dry-run[Show what would be upgraded]'
-            ;;
-    esac
-}
-
-_tdk_commands() {
-    local commands=(
-        'project:Initialize project with master configs'
-        'projects:Show project information'
-        'stack:Organize resources into stacks'
-        'stacks:List all stacks'
-        'resource:Create new resource'
-        'resources:List all resources'
-        'up:Start services'
-        'down:Stop all services'
-        'status:Show resource status'
-        'ui:Open interactive UI'
-        'doctor:Check environment'
-        'version:Show version'
-        'upgrade:Upgrade TDK CLI'
-        'completion:Generate shell completions'
-        'help:Show help'
-    )
-    _describe -t commands 'tdk command' commands
-}
-
-_tdk_stacks() {
-    local stacks
-    stacks=($(tdk stacks 2>/dev/null | grep '^  -' | sed 's/^  - //' | awk '{print $1}'))
-    _describe -t stacks 'stack' stacks
-}
-
-_tdk_resources() {
-    local resources
-    resources=($(tdk resources 2>/dev/null | grep -E '^[a-z0-9-]+' | awk '{print $1}'))
-    _describe -t resources 'resource' resources
-}
-
-_tdk_stacks_and_resources() {
-    _tdk_stacks
-    _tdk_resources
-}
-
-compdef _tdk tdk
-`;
-
-const FISH_COMPLETION = `# TDK CLI Fish Completion
-
-# Disable file completions for most commands
-complete -c tdk -f
-
-# Main commands
-complete -c tdk -n '__fish_use_subcommand' -a 'project' -d 'Initialize project with master configs'
-complete -c tdk -n '__fish_use_subcommand' -a 'projects' -d 'Show project information'
-complete -c tdk -n '__fish_use_subcommand' -a 'stack' -d 'Organize resources into stacks'
-complete -c tdk -n '__fish_use_subcommand' -a 'stacks' -d 'List all stacks'
-complete -c tdk -n '__fish_use_subcommand' -a 'resource' -d 'Create new resource'
-complete -c tdk -n '__fish_use_subcommand' -a 'resources' -d 'List all resources'
-complete -c tdk -n '__fish_use_subcommand' -a 'up' -d 'Start services'
-complete -c tdk -n '__fish_use_subcommand' -a 'down' -d 'Stop all services'
-complete -c tdk -n '__fish_use_subcommand' -a 'status' -d 'Show resource status'
-complete -c tdk -n '__fish_use_subcommand' -a 'ui' -d 'Open interactive UI'
-complete -c tdk -n '__fish_use_subcommand' -a 'doctor' -d 'Check environment'
-complete -c tdk -n '__fish_use_subcommand' -a 'version' -d 'Show version'
-complete -c tdk -n '__fish_use_subcommand' -a 'upgrade' -d 'Upgrade TDK CLI'
-complete -c tdk -n '__fish_use_subcommand' -a 'completion' -d 'Generate shell completions'
-complete -c tdk -n '__fish_use_subcommand' -a 'help' -d 'Show help'
-
-# Global options
-complete -c tdk -s h -l help -d 'Show help'
-complete -c tdk -s v -l version -d 'Show version'
-complete -c tdk -l verbose -d 'Enable verbose output'
-
-# Command-specific completions
-# resource command options
-complete -c tdk -n '__fish_seen_subcommand_from resource' -l type -d 'Resource type' -a 'backend frontend worker'
-complete -c tdk -n '__fish_seen_subcommand_from resource' -l stack -d 'Assign to stack' -a '(tdk stacks 2>/dev/null | string match -r "^  - " | string replace "  - " "")'
-
-# resources command options
-complete -c tdk -n '__fish_seen_subcommand_from resources' -l stack -d 'Filter by stack' -a '(tdk stacks 2>/dev/null | string match -r "^  - " | string replace "  - " "")'
-complete -c tdk -n '__fish_seen_subcommand_from resources' -l no-stack -d 'Show unassigned only'
-complete -c tdk -n '__fish_seen_subcommand_from resources' -l ports -d 'Show port assignments'
-
-# stack command options
-complete -c tdk -n '__fish_seen_subcommand_from stack' -l list -d 'List resources without a stack'
-
-# stacks command options
-complete -c tdk -n '__fish_seen_subcommand_from stacks' -l services -d 'Include resources in each stack'
-
-# projects command options
-complete -c tdk -n '__fish_seen_subcommand_from projects' -l check -d 'Validate master configs'
-
-# up command completions
-complete -c tdk -n '__fish_seen_subcommand_from up' -a '(tdk stacks 2>/dev/null | string match -r "^  - " | string replace "  - " "")'
-complete -c tdk -n '__fish_seen_subcommand_from up' -a '(tdk resources 2>/dev/null | string match -r "^[a-z0-9-]+")'
-
-# doctor command options
-complete -c tdk -n '__fish_seen_subcommand_from doctor' -l fix -d 'Attempt to fix issues'
-
-# upgrade command options
-complete -c tdk -n '__fish_seen_subcommand_from upgrade' -l force -d 'Force upgrade even if on latest'
-complete -c tdk -n '__fish_seen_subcommand_from upgrade' -l dry-run -d 'Show what would be upgraded'
-`;
-
-export const completionCommand = new Command("completion")
-  .description("Generate shell completion scripts")
-  .option("-s, --shell <shell>", "Target shell (bash, zsh, fish)", "bash")
-  .option("-o, --output <path>", "Output file path (default: stdout)")
-  .option("--install", "Install to shell config automatically")
-  .action((options) => {
-    const shell = options.shell.toLowerCase();
-
-    let completionScript: string;
-    let filename: string;
-
-    switch (shell) {
-      case "bash":
-        completionScript = BASH_COMPLETION;
-        filename = "tdk-completion.bash";
-        break;
-      case "zsh":
-        completionScript = ZSH_COMPLETION;
-        filename = "_tdk";
-        break;
-      case "fish":
-        completionScript = FISH_COMPLETION;
-        filename = "tdk.fish";
-        break;
-      default:
-        console.error(chalk.red(`❌ Unsupported shell: ${shell}`));
-        console.log(chalk.gray("Supported shells: bash, zsh, fish"));
-        process.exit(1);
-    }
-
-    if (options.install) {
-      const home = homedir();
-      let installPath: string;
-      let installInstructions: string;
-
-      switch (shell) {
-        case "bash": {
-          installPath = join(home, ".bash_completion.d", filename);
-          const bashDir = join(home, ".bash_completion.d");
-          if (!existsSync(bashDir)) {
-            mkdirSync(bashDir, { recursive: true });
-          }
-          installInstructions = `\n# Add to ~/.bashrc:\nsource ~/.bash_completion.d/${filename}`;
-          break;
-        }
-        case "zsh": {
-          installPath = join(home, ".zsh", "completions", filename);
-          const zshDir = join(home, ".zsh", "completions");
-          if (!existsSync(zshDir)) {
-            mkdirSync(zshDir, { recursive: true });
-          }
-          installInstructions =
-            "\n# Add to ~/.zshrc:\nfpath+=(~/.zsh/completions)\nautoload -U compinit && compinit";
-          break;
-        }
-        case "fish": {
-          installPath = join(home, ".config", "fish", "completions", filename);
-          const fishDir = join(home, ".config", "fish", "completions");
-          if (!existsSync(fishDir)) {
-            mkdirSync(fishDir, { recursive: true });
-          }
-          installInstructions = "\n# Fish completions loaded automatically";
-          break;
-        }
-        default:
-          installPath = "";
-          installInstructions = "";
-      }
-
-      writeFileSync(installPath, completionScript, "utf-8");
-      console.log(chalk.green(`✅ Installed ${shell} completion to:`));
-      console.log(chalk.cyan(`   ${installPath}`));
-      console.log(chalk.yellow(installInstructions));
-    } else if (options.output) {
-      writeFileSync(options.output, completionScript, "utf-8");
-      console.log(chalk.green(`✅ Written ${shell} completion to:`));
-      console.log(chalk.cyan(`   ${options.output}`));
-    } else {
-      console.log(completionScript);
-    }
-  });