claudekit-cli 1.4.0 → 1.5.0

package/src/utils/file-scanner.ts

@@ -1,134 +0,0 @@
-import { join, relative, resolve } from "node:path";
-import { lstat, pathExists, readdir } from "fs-extra";
-import { logger } from "./logger.js";
-
-/**
- * Utility class for scanning directories and comparing file structures
- */
-export class FileScanner {
-	/**
-	 * Get all files in a directory recursively
-	 *
-	 * @param dirPath - Directory path to scan
-	 * @param relativeTo - Base path for calculating relative paths (defaults to dirPath)
-	 * @returns Array of relative file paths
-	 *
-	 * @example
-	 * ```typescript
-	 * const files = await FileScanner.getFiles('/path/to/dir');
-	 * // Returns: ['file1.txt', 'subdir/file2.txt', ...]
-	 * ```
-	 */
-	static async getFiles(dirPath: string, relativeTo?: string): Promise<string[]> {
-		const basePath = relativeTo || dirPath;
-		const files: string[] = [];
-
-		// Check if directory exists
-		if (!(await pathExists(dirPath))) {
-			return files;
-		}
-
-		try {
-			const entries = await readdir(dirPath);
-
-			for (const entry of entries) {
-				const fullPath = join(dirPath, entry);
-
-				// Security: Validate path to prevent traversal
-				if (!FileScanner.isSafePath(basePath, fullPath)) {
-					logger.warning(`Skipping potentially unsafe path: ${entry}`);
-					continue;
-				}
-
-				const stats = await lstat(fullPath);
-
-				// Skip symlinks for security
-				if (stats.isSymbolicLink()) {
-					logger.debug(`Skipping symlink: ${entry}`);
-					continue;
-				}
-
-				if (stats.isDirectory()) {
-					// Recursively scan subdirectories
-					const subFiles = await FileScanner.getFiles(fullPath, basePath);
-					files.push(...subFiles);
-				} else if (stats.isFile()) {
-					// Add relative path
-					const relativePath = relative(basePath, fullPath);
-					files.push(relativePath);
-				}
-			}
-		} catch (error) {
-			const errorMessage =
-				error instanceof Error
-					? `Failed to scan directory: ${dirPath} - ${error.message}`
-					: `Failed to scan directory: ${dirPath}`;
-			logger.error(errorMessage);
-			throw error;
-		}
-
-		return files;
-	}
-
-	/**
-	 * Find files in destination that don't exist in source
-	 *
-	 * @param destDir - Destination directory path
-	 * @param sourceDir - Source directory path
-	 * @param subPath - Subdirectory to compare (e.g., '.claude')
-	 * @returns Array of relative file paths that are custom (exist in dest but not in source)
-	 *
-	 * @example
-	 * ```typescript
-	 * const customFiles = await FileScanner.findCustomFiles(
-	 *   '/path/to/project',
-	 *   '/path/to/release',
-	 *   '.claude'
-	 * );
-	 * // Returns: ['.claude/custom-command.md', '.claude/workflows/my-workflow.md']
-	 * ```
-	 */
-	static async findCustomFiles(
-		destDir: string,
-		sourceDir: string,
-		subPath: string,
-	): Promise<string[]> {
-		const destSubDir = join(destDir, subPath);
-		const sourceSubDir = join(sourceDir, subPath);
-
-		// Get files from both directories
-		const destFiles = await FileScanner.getFiles(destSubDir, destDir);
-		const sourceFiles = await FileScanner.getFiles(sourceSubDir, sourceDir);
-
-		// Create a Set of source files for O(1) lookup
-		const sourceFileSet = new Set(sourceFiles);
-
-		// Find files in destination that don't exist in source
-		const customFiles = destFiles.filter((file) => !sourceFileSet.has(file));
-
-		if (customFiles.length > 0) {
-			logger.info(`Found ${customFiles.length} custom file(s) in ${subPath}/`);
-			customFiles.slice(0, 5).forEach((file) => logger.debug(`  - ${file}`));
-			if (customFiles.length > 5) {
-				logger.debug(`  ... and ${customFiles.length - 5} more`);
-			}
-		}
-
-		return customFiles;
-	}
-
-	/**
-	 * Validate path to prevent path traversal attacks
-	 *
-	 * @param basePath - Base directory path
-	 * @param targetPath - Target path to validate
-	 * @returns true if path is safe, false otherwise
-	 */
-	private static isSafePath(basePath: string, targetPath: string): boolean {
-		const resolvedBase = resolve(basePath);
-		const resolvedTarget = resolve(targetPath);
-
-		// Ensure target is within base
-		return resolvedTarget.startsWith(resolvedBase);
-	}
-}

package/src/utils/logger.ts

@@ -1,124 +0,0 @@
-import { type WriteStream, createWriteStream } from "node:fs";
-import pc from "picocolors";
-
-// Use ASCII-safe symbols to avoid unicode rendering issues in certain terminals
-const symbols = {
-	info: "[i]",
-	success: "[+]",
-	warning: "[!]",
-	error: "[x]",
-};
-
-interface LogContext {
-	[key: string]: any;
-}
-
-class Logger {
-	private verboseEnabled = false;
-	private logFileStream?: WriteStream;
-
-	info(message: string): void {
-		console.log(pc.blue(symbols.info), message);
-	}
-
-	success(message: string): void {
-		console.log(pc.green(symbols.success), message);
-	}
-
-	warning(message: string): void {
-		console.log(pc.yellow(symbols.warning), message);
-	}
-
-	error(message: string): void {
-		console.error(pc.red(symbols.error), message);
-	}
-
-	debug(message: string): void {
-		if (process.env.DEBUG) {
-			console.log(pc.gray("[DEBUG]"), message);
-		}
-	}
-
-	verbose(message: string, context?: LogContext): void {
-		if (!this.verboseEnabled) return;
-
-		const timestamp = this.getTimestamp();
-		const sanitizedMessage = this.sanitize(message);
-		const formattedContext = context ? this.formatContext(context) : "";
-
-		const logLine = `${timestamp} ${pc.gray("[VERBOSE]")} ${sanitizedMessage}${formattedContext}`;
-
-		console.error(logLine);
-
-		if (this.logFileStream) {
-			const plainLogLine = `${timestamp} [VERBOSE] ${sanitizedMessage}${formattedContext}`;
-			this.logFileStream.write(`${plainLogLine}\n`);
-		}
-	}
-
-	setVerbose(enabled: boolean): void {
-		this.verboseEnabled = enabled;
-		if (enabled) {
-			this.verbose("Verbose logging enabled");
-		}
-	}
-
-	isVerbose(): boolean {
-		return this.verboseEnabled;
-	}
-
-	setLogFile(path?: string): void {
-		if (this.logFileStream) {
-			this.logFileStream.end();
-			this.logFileStream = undefined;
-		}
-
-		if (path) {
-			this.logFileStream = createWriteStream(path, {
-				flags: "a",
-				mode: 0o600,
-			});
-			this.verbose(`Logging to file: ${path}`);
-		}
-	}
-
-	sanitize(text: string): string {
-		return text
-			.replace(/ghp_[a-zA-Z0-9]{36}/g, "ghp_***")
-			.replace(/github_pat_[a-zA-Z0-9_]{82}/g, "github_pat_***")
-			.replace(/gho_[a-zA-Z0-9]{36}/g, "gho_***")
-			.replace(/ghu_[a-zA-Z0-9]{36}/g, "ghu_***")
-			.replace(/ghs_[a-zA-Z0-9]{36}/g, "ghs_***")
-			.replace(/ghr_[a-zA-Z0-9]{36}/g, "ghr_***")
-			.replace(/Bearer [a-zA-Z0-9_-]+/g, "Bearer ***")
-			.replace(/token=[a-zA-Z0-9_-]+/g, "token=***");
-	}
-
-	private getTimestamp(): string {
-		return new Date().toISOString();
-	}
-
-	private formatContext(context: LogContext): string {
-		const sanitized = Object.entries(context).reduce((acc, [key, value]) => {
-			if (typeof value === "string") {
-				acc[key] = this.sanitize(value);
-			} else if (value && typeof value === "object") {
-				// Recursively sanitize nested objects
-				try {
-					const stringified = JSON.stringify(value);
-					const sanitizedStr = this.sanitize(stringified);
-					acc[key] = JSON.parse(sanitizedStr);
-				} catch {
-					acc[key] = "[Object]";
-				}
-			} else {
-				acc[key] = value;
-			}
-			return acc;
-		}, {} as LogContext);
-
-		return `\n  ${JSON.stringify(sanitized, null, 2).split("\n").join("\n  ")}`;
-	}
-}
-
-export const logger = new Logger();

package/src/utils/safe-prompts.ts

@@ -1,44 +0,0 @@
-import picocolors from "picocolors";
-
-/**
- * Safe wrapper around clack prompts that uses simple ASCII characters
- * instead of unicode box drawing to avoid rendering issues.
- */
-
-/**
- * Simple intro with ASCII characters
- */
-export function intro(message: string): void {
-	console.log();
-	console.log(picocolors.cyan(`> ${message}`));
-	console.log();
-}
-
-/**
- * Simple outro with ASCII characters
- */
-export function outro(message: string): void {
-	console.log();
-	console.log(picocolors.green(`[OK] ${message}`));
-	console.log();
-}
-
-/**
- * Simple note with ASCII box drawing
- */
-export function note(message: string, title?: string): void {
-	console.log();
-	if (title) {
-		console.log(picocolors.cyan(`  ${title}:`));
-		console.log();
-	}
-	// Split message into lines and indent each
-	const lines = message.split("\n");
-	for (const line of lines) {
-		console.log(`  ${line}`);
-	}
-	console.log();
-}
-
-// Re-export other clack functions unchanged
-export { select, confirm, text, isCancel } from "@clack/prompts";

package/src/utils/safe-spinner.ts

@@ -1,38 +0,0 @@
-import ora, { type Ora, type Options } from "ora";
-
-/**
- * Create a spinner with simple ASCII characters to avoid unicode rendering issues
- */
-export function createSpinner(options: string | Options): Ora {
-	const spinnerOptions: Options = typeof options === "string" ? { text: options } : options;
-
-	const spinner = ora({
-		...spinnerOptions,
-		// Use simple ASCII spinner instead of unicode
-		spinner: "dots",
-		// Override symbols to use ASCII
-		prefixText: "",
-	});
-
-	// Override succeed and fail methods to use ASCII symbols
-	spinner.succeed = (text?: string) => {
-		spinner.stopAndPersist({
-			symbol: "[+]",
-			text: text || spinner.text,
-		});
-		return spinner;
-	};
-
-	spinner.fail = (text?: string) => {
-		spinner.stopAndPersist({
-			symbol: "[x]",
-			text: text || spinner.text,
-		});
-		return spinner;
-	};
-
-	return spinner;
-}
-
-// Re-export Ora type for convenience
-export type { Ora } from "ora";

package/src/version.json

@@ -1,3 +0,0 @@
-{
-	"version": "1.2.1"
-}

package/test-integration/demo/.mcp.json

@@ -1,13 +0,0 @@
-{
-	"mcpServers": {
-		"human": {
-			"command": "npx",
-			"args": ["-y", "@goonnguyen/human-mcp@latest"],
-			"env": {
-				"GOOGLE_GEMINI_API_KEY": "",
-				"TRANSPORT_TYPE": "stdio",
-				"LOG_LEVEL": "info"
-			}
-		}
-	}
-}

package/test-integration/demo/.repomixignore

@@ -1,15 +0,0 @@
-docs/*
-plans/*
-assets/*
-dist/*
-coverage/*
-build/*
-ios/*
-android/*
-
-.claude/*
-.serena/*
-.pnpm-store/*
-.github/*
-.dart_tool/*
-.idea/*

package/test-integration/demo/CLAUDE.md

@@ -1,34 +0,0 @@
-# CLAUDE.md
-
-This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.
-
-## Role & Responsibilities
-
-Your role is to analyze user requirements, delegate tasks to appropriate sub-agents, and ensure cohesive delivery of features that meet specifications and architectural standards.
-
-## Workflows
-
-- Primary workflow: `./.claude/workflows/primary-workflow.md`
-- Development rules: `./.claude/workflows/development-rules.md`
-- Orchestration protocols: `./.claude/workflows/orchestration-protocol.md`
-- Documentation management: `./.claude/workflows/documentation-management.md`
-
-**IMPORTANT:** You must follow strictly the development rules in `./.claude/workflows/development-rules.md` file.
-**IMPORTANT:** Before you plan or proceed any implementation, always read the `./README.md` file first to get context.
-**IMPORTANT:** Sacrifice grammar for the sake of concision when writing reports.
-**IMPORTANT:** In reports, list any unresolved questions at the end, if any.
-
-## Documentation Management
-
-We keep all important docs in `./docs` folder and keep updating them, structure like below:
-
-```
-./docs
-├── project-overview-pdr.md
-├── code-standards.md
-├── codebase-summary.md
-├── design-guidelines.md
-├── deployment-guide.md
-├── system-architecture.md
-└── project-roadmap.md
-```

package/tests/commands/version.test.ts

@@ -1,297 +0,0 @@
-import { beforeEach, describe, expect, mock, test } from "bun:test";
-import type { GitHubRelease } from "../../src/types.js";
-import { AVAILABLE_KITS, VersionCommandOptionsSchema } from "../../src/types.js";
-
-describe("Version Command", () => {
-	beforeEach(() => {
-		// Set environment variable to avoid auth prompts during tests
-		process.env.GITHUB_TOKEN = "ghp_test_token_for_testing";
-	});
-
-	describe("VersionCommandOptionsSchema", () => {
-		test("should accept valid options with kit filter", () => {
-			const options = { kit: "engineer" as const };
-			const result = VersionCommandOptionsSchema.parse(options);
-			expect(result.kit).toBe("engineer");
-		});
-
-		test("should accept valid options with limit", () => {
-			const options = { limit: 10 };
-			const result = VersionCommandOptionsSchema.parse(options);
-			expect(result.limit).toBe(10);
-		});
-
-		test("should accept valid options with all flag", () => {
-			const options = { all: true };
-			const result = VersionCommandOptionsSchema.parse(options);
-			expect(result.all).toBe(true);
-		});
-
-		test("should accept all options combined", () => {
-			const options = { kit: "marketing" as const, limit: 20, all: true };
-			const result = VersionCommandOptionsSchema.parse(options);
-			expect(result.kit).toBe("marketing");
-			expect(result.limit).toBe(20);
-			expect(result.all).toBe(true);
-		});
-
-		test("should accept empty options", () => {
-			const options = {};
-			const result = VersionCommandOptionsSchema.parse(options);
-			expect(result.kit).toBeUndefined();
-			expect(result.limit).toBeUndefined();
-			expect(result.all).toBeUndefined();
-		});
-
-		test("should reject invalid kit type", () => {
-			const options = { kit: "invalid" };
-			expect(() => VersionCommandOptionsSchema.parse(options)).toThrow();
-		});
-	});
-
-	describe("Kit Configuration", () => {
-		test("should have engineer kit configured", () => {
-			const engineerKit = AVAILABLE_KITS.engineer;
-			expect(engineerKit.name).toBe("ClaudeKit Engineer");
-			expect(engineerKit.repo).toBe("claudekit-engineer");
-			expect(engineerKit.owner).toBe("claudekit");
-		});
-
-		test("should have marketing kit configured", () => {
-			const marketingKit = AVAILABLE_KITS.marketing;
-			expect(marketingKit.name).toBe("ClaudeKit Marketing");
-			expect(marketingKit.repo).toBe("claudekit-marketing");
-			expect(marketingKit.owner).toBe("claudekit");
-		});
-	});
-
-	describe("Release Data Handling", () => {
-		test("should handle release with all fields", () => {
-			const release: GitHubRelease = {
-				id: 1,
-				tag_name: "v1.0.0",
-				name: "Release 1.0.0",
-				draft: false,
-				prerelease: false,
-				assets: [],
-				published_at: "2024-01-01T00:00:00Z",
-			};
-
-			expect(release.tag_name).toBe("v1.0.0");
-			expect(release.name).toBe("Release 1.0.0");
-			expect(release.draft).toBe(false);
-			expect(release.prerelease).toBe(false);
-		});
-
-		test("should handle release without published_at", () => {
-			const release: GitHubRelease = {
-				id: 1,
-				tag_name: "v1.0.0",
-				name: "Release 1.0.0",
-				draft: false,
-				prerelease: false,
-				assets: [],
-			};
-
-			expect(release.published_at).toBeUndefined();
-		});
-
-		test("should handle draft release", () => {
-			const release: GitHubRelease = {
-				id: 1,
-				tag_name: "v1.0.0-draft",
-				name: "Draft Release",
-				draft: true,
-				prerelease: false,
-				assets: [],
-			};
-
-			expect(release.draft).toBe(true);
-		});
-
-		test("should handle prerelease", () => {
-			const release: GitHubRelease = {
-				id: 1,
-				tag_name: "v1.0.0-beta.1",
-				name: "Beta Release",
-				draft: false,
-				prerelease: true,
-				assets: [],
-			};
-
-			expect(release.prerelease).toBe(true);
-		});
-	});
-
-	describe("Date Formatting", () => {
-		test("should format recent dates correctly", () => {
-			const now = new Date();
-			const yesterday = new Date(now.getTime() - 24 * 60 * 60 * 1000);
-			const dateString = yesterday.toISOString();
-
-			// The actual formatting logic is in the command file
-			// We just verify the date string is valid
-			expect(dateString).toMatch(/^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}/);
-		});
-
-		test("should handle undefined date", () => {
-			const dateString = undefined;
-			expect(dateString).toBeUndefined();
-		});
-	});
-
-	describe("Release Filtering", () => {
-		const releases: GitHubRelease[] = [
-			{
-				id: 1,
-				tag_name: "v1.0.0",
-				name: "Stable Release",
-				draft: false,
-				prerelease: false,
-				assets: [],
-				published_at: "2024-01-01T00:00:00Z",
-			},
-			{
-				id: 2,
-				tag_name: "v1.1.0-beta.1",
-				name: "Beta Release",
-				draft: false,
-				prerelease: true,
-				assets: [],
-				published_at: "2024-01-02T00:00:00Z",
-			},
-			{
-				id: 3,
-				tag_name: "v1.2.0-draft",
-				name: "Draft Release",
-				draft: true,
-				prerelease: false,
-				assets: [],
-				published_at: "2024-01-03T00:00:00Z",
-			},
-		];
-
-		test("should filter out drafts by default", () => {
-			const stable = releases.filter((r) => !r.draft && !r.prerelease);
-			expect(stable).toHaveLength(1);
-			expect(stable[0].tag_name).toBe("v1.0.0");
-		});
-
-		test("should filter out prereleases by default", () => {
-			const stable = releases.filter((r) => !r.draft && !r.prerelease);
-			expect(stable.every((r) => !r.prerelease)).toBe(true);
-		});
-
-		test("should include all when --all flag is used", () => {
-			const all = releases; // No filtering when --all is true
-			expect(all).toHaveLength(3);
-		});
-
-		test("should handle empty release list", () => {
-			const empty: GitHubRelease[] = [];
-			expect(empty).toHaveLength(0);
-		});
-	});
-
-	describe("Command Options Validation", () => {
-		test("should validate limit as number", () => {
-			const validLimit = { limit: 50 };
-			const result = VersionCommandOptionsSchema.parse(validLimit);
-			expect(result.limit).toBe(50);
-		});
-
-		test("should validate all as boolean", () => {
-			const validAll = { all: false };
-			const result = VersionCommandOptionsSchema.parse(validAll);
-			expect(result.all).toBe(false);
-		});
-
-		test("should handle optional fields", () => {
-			const minimal = {};
-			const result = VersionCommandOptionsSchema.parse(minimal);
-			expect(result).toBeDefined();
-		});
-	});
-
-	describe("Error Scenarios", () => {
-		test("should handle invalid option types", () => {
-			const invalidLimit = { limit: "not-a-number" };
-			expect(() => VersionCommandOptionsSchema.parse(invalidLimit)).toThrow();
-		});
-
-		test("should handle invalid all flag type", () => {
-			const invalidAll = { all: "not-a-boolean" };
-			expect(() => VersionCommandOptionsSchema.parse(invalidAll)).toThrow();
-		});
-	});
-
-	describe("Assets Handling", () => {
-		test("should handle release with multiple assets", () => {
-			const release: GitHubRelease = {
-				id: 1,
-				tag_name: "v1.0.0",
-				name: "Release",
-				draft: false,
-				prerelease: false,
-				assets: [
-					{
-						id: 1,
-						name: "package.tar.gz",
-						browser_download_url: "https://example.com/package.tar.gz",
-						size: 1024,
-						content_type: "application/gzip",
-					},
-					{
-						id: 2,
-						name: "package.zip",
-						browser_download_url: "https://example.com/package.zip",
-						size: 2048,
-						content_type: "application/zip",
-					},
-				],
-			};
-
-			expect(release.assets).toHaveLength(2);
-		});
-
-		test("should handle release with no assets", () => {
-			const release: GitHubRelease = {
-				id: 1,
-				tag_name: "v1.0.0",
-				name: "Release",
-				draft: false,
-				prerelease: false,
-				assets: [],
-			};
-
-			expect(release.assets).toHaveLength(0);
-		});
-	});
-
-	describe("Integration Scenarios", () => {
-		test("should handle both kits in parallel", () => {
-			const kits = Object.keys(AVAILABLE_KITS);
-			expect(kits).toContain("engineer");
-			expect(kits).toContain("marketing");
-			expect(kits).toHaveLength(2);
-		});
-
-		test("should support filtering by engineer kit", () => {
-			const options = { kit: "engineer" as const };
-			const result = VersionCommandOptionsSchema.parse(options);
-			expect(result.kit).toBe("engineer");
-
-			const kitConfig = AVAILABLE_KITS[result.kit];
-			expect(kitConfig.repo).toBe("claudekit-engineer");
-		});
-
-		test("should support filtering by marketing kit", () => {
-			const options = { kit: "marketing" as const };
-			const result = VersionCommandOptionsSchema.parse(options);
-			expect(result.kit).toBe("marketing");
-
-			const kitConfig = AVAILABLE_KITS[result.kit];
-			expect(kitConfig.repo).toBe("claudekit-marketing");
-		});
-	});
-});