@poetora/cli 0.1.9 → 0.1.11

package/src/constants.ts

@@ -1,4 +0,0 @@
-import os from 'os';
-
-export const HOME_DIR = os.homedir();
-export const CMD_EXEC_PATH = process.cwd();

package/src/errors/cli-error.ts

@@ -1,83 +0,0 @@
-/**
- * Base CLI error class for all custom errors
- */
-export class CliError extends Error {
-  constructor(
-    message: string,
-    public readonly code: string,
-    public readonly exitCode: number = 1
-  ) {
-    super(message);
-    this.name = 'CliError';
-    Error.captureStackTrace(this, this.constructor);
-  }
-}
-
-/**
- * Error thrown when environment requirements are not met
- */
-export class InvalidEnvironmentError extends CliError {
-  constructor(message: string) {
-    super(message, 'INVALID_ENVIRONMENT', 1);
-    this.name = 'InvalidEnvironmentError';
-  }
-}
-
-/**
- * Error thrown when no available port is found
- */
-export class NoAvailablePortError extends CliError {
-  constructor(message: string) {
-    super(message, 'NO_AVAILABLE_PORT', 1);
-    this.name = 'NoAvailablePortError';
-  }
-}
-
-/**
- * Error thrown when configuration file is not found
- */
-export class ConfigNotFoundError extends CliError {
-  constructor(message: string) {
-    super(message, 'CONFIG_NOT_FOUND', 1);
-    this.name = 'ConfigNotFoundError';
-  }
-}
-
-/**
- * Error thrown when command validation fails
- */
-export class ValidationError extends CliError {
-  constructor(
-    message: string,
-    public readonly field?: string
-  ) {
-    super(message, 'VALIDATION_ERROR', 1);
-    this.name = 'ValidationError';
-  }
-}
-
-/**
- * Error thrown when file operations fail
- */
-export class FileSystemError extends CliError {
-  constructor(
-    message: string,
-    public readonly filePath?: string
-  ) {
-    super(message, 'FILE_SYSTEM_ERROR', 1);
-    this.name = 'FileSystemError';
-  }
-}
-
-/**
- * Error thrown when external service calls fail
- */
-export class ExternalServiceError extends CliError {
-  constructor(
-    message: string,
-    public readonly service?: string
-  ) {
-    super(message, 'EXTERNAL_SERVICE_ERROR', 1);
-    this.name = 'ExternalServiceError';
-  }
-}

package/src/errors/index.ts

@@ -1 +0,0 @@
-export * from './cli-error.js';

package/src/index.ts

@@ -1,110 +0,0 @@
-#!/usr/bin/env node
-import { type ChildProcess, spawn } from 'child_process';
-import path from 'path';
-import { fileURLToPath } from 'url';
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = path.dirname(__filename);
-
-const packageName =
-  path.basename(process.argv[1] ?? '') === 'index'
-    ? 'poet'
-    : path.basename(process.argv[1] ?? '') || 'poet';
-
-let cli: ChildProcess | null = null;
-let isShuttingDown = false;
-let hasExited = false;
-
-const cleanup = async (): Promise<void> => {
-  if (isShuttingDown) return;
-  isShuttingDown = true;
-
-  if (cli && !cli.killed) {
-    try {
-      cli.kill('SIGTERM');
-
-      await new Promise<void>((resolve) => {
-        const timeout = setTimeout(() => {
-          if (cli && !cli.killed) {
-            cli.kill('SIGKILL');
-          }
-          resolve();
-        }, 5000);
-
-        cli?.once('exit', () => {
-          clearTimeout(timeout);
-          resolve();
-        });
-      });
-    } catch (_error) {
-      // ignore
-    }
-  }
-};
-
-const exitProcess = (code: number) => {
-  if (hasExited) return;
-  hasExited = true;
-  process.exit(code);
-};
-
-const killSignals = ['SIGINT', 'SIGTERM', 'SIGQUIT', 'SIGHUP'];
-killSignals.forEach((signal) => {
-  process.on(signal, async () => {
-    await cleanup();
-    exitProcess(0);
-  });
-});
-
-process.on('uncaughtException', async () => {
-  await cleanup();
-  exitProcess(1);
-});
-
-process.on('unhandledRejection', async () => {
-  await cleanup();
-  exitProcess(1);
-});
-
-try {
-  cli = spawn(
-    'node',
-    ['--no-deprecation', path.join(__dirname, '../bin/start'), ...process.argv.slice(2)],
-    {
-      stdio: 'inherit',
-      env: {
-        ...process.env,
-        POETORA_PACKAGE_NAME: packageName,
-        CLI_TEST_MODE: process.env.CLI_TEST_MODE ?? 'false',
-      },
-      shell: process.platform === 'win32',
-      windowsHide: process.platform === 'win32',
-      detached: false,
-    }
-  );
-
-  cli.on('error', async (error) => {
-    console.error(`Failed to start ${packageName}: ${error.message}`);
-    await cleanup();
-    exitProcess(1);
-  });
-
-  cli.on('exit', (code) => {
-    exitProcess(code ?? 0);
-  });
-} catch (error) {
-  console.error(`Failed to start ${packageName}: ${error}`);
-  exitProcess(1);
-}
-
-process.on('exit', () => {
-  if (cli && !cli.killed) {
-    try {
-      cli.kill('SIGKILL');
-    } catch (_error) {
-      // ignore
-    }
-  }
-});
-
-export { cli };

package/src/mdxAccessibility.ts

@@ -1,132 +0,0 @@
-import { categorizeFilePaths, getPoetIgnore } from '@poetora/prebuild';
-import { coreRemark } from '@poetora/shared';
-import fs from 'fs';
-import type { Node, Root, Text } from 'mdast';
-import type { MdxJsxFlowElement } from 'mdast-util-mdx-jsx';
-import path from 'path';
-import { visit } from 'unist-util-visit';
-
-export interface AccessibilityFixAttribute {
-  filePath: string;
-  line?: number;
-  column?: number;
-  element: 'img' | 'video' | 'a';
-  tagName: string;
-}
-
-export interface MdxAccessibilityResult {
-  missingAltAttributes: AccessibilityFixAttribute[];
-  totalFiles: number;
-  filesWithIssues: number;
-}
-
-const checkAltAttributes = (filePath: string, content: string): AccessibilityFixAttribute[] => {
-  const issues: AccessibilityFixAttribute[] = [];
-
-  const visitElements = () => {
-    return (tree: Root) => {
-      visit(tree, (node) => {
-        if (node.type === 'image') {
-          if (!node.alt || node.alt.trim() === '') {
-            issues.push({
-              filePath,
-              line: node.position?.start.line,
-              column: node.position?.start.column,
-              element: 'img',
-              tagName: 'image (markdown)',
-            });
-          }
-          return;
-        }
-
-        const mdxJsxElement = node as MdxJsxFlowElement;
-        if (mdxJsxElement.name === 'img' || mdxJsxElement.name === 'video') {
-          const altAttrIndex = mdxJsxElement.attributes.findIndex(
-            (attr) => attr.type === 'mdxJsxAttribute' && attr.name === 'alt'
-          );
-
-          const altAttribute = mdxJsxElement.attributes[altAttrIndex];
-          const hasValidAlt =
-            altAttribute &&
-            typeof altAttribute.value === 'string' &&
-            altAttribute.value.trim() !== '';
-
-          if (!hasValidAlt) {
-            issues.push({
-              filePath,
-              line: node.position?.start.line,
-              column: node.position?.start.column,
-              element: mdxJsxElement.name,
-              tagName: mdxJsxElement.name,
-            });
-          }
-        } else if (mdxJsxElement.name === 'a') {
-          const hasTextContent = (children: Node[]): boolean => {
-            return children.some((child) => {
-              if (child.type === 'text') {
-                const textNode = child as Text;
-                return textNode.value.trim() !== '';
-              }
-              if ('children' in child && Array.isArray(child.children)) {
-                return hasTextContent(child.children as Node[]);
-              }
-              return false;
-            });
-          };
-
-          if (!hasTextContent(mdxJsxElement.children as Node[])) {
-            issues.push({
-              filePath,
-              line: node.position?.start.line,
-              column: node.position?.start.column,
-              element: 'a',
-              tagName: '<a>',
-            });
-          }
-        }
-      });
-      return tree;
-    };
-  };
-
-  try {
-    coreRemark().use(visitElements).processSync(content);
-  } catch (error) {
-    console.warn(`Warning: Could not parse ${filePath}: ${error}`);
-  }
-
-  return issues;
-};
-
-export const checkMdxAccessibility = async (
-  baseDir: string = process.cwd()
-): Promise<MdxAccessibilityResult> => {
-  const poetIgnore = await getPoetIgnore(baseDir);
-  const { contentFilenames } = await categorizeFilePaths(baseDir, poetIgnore);
-  const mdxFiles: string[] = [];
-  for (const file of contentFilenames) {
-    mdxFiles.push(path.join(baseDir, file));
-  }
-  const allIssues: AccessibilityFixAttribute[] = [];
-  const filesWithIssues = new Set<string>();
-
-  for (const filePath of mdxFiles) {
-    try {
-      const content = fs.readFileSync(filePath, 'utf-8');
-      const issues = checkAltAttributes(filePath, content);
-
-      if (issues.length > 0) {
-        allIssues.push(...issues);
-        filesWithIssues.add(filePath);
-      }
-    } catch (error) {
-      console.warn(`Warning: Could not read file ${filePath}: ${error}`);
-    }
-  }
-
-  return {
-    missingAltAttributes: allIssues,
-    totalFiles: mdxFiles.length,
-    filesWithIssues: filesWithIssues.size,
-  };
-};

package/src/middlewares.ts

@@ -1,73 +0,0 @@
-import { addLog, ErrorLog } from '@poetora/previewing';
-import React from 'react';
-
-/**
- * Yargs middleware functions
- * These run before command execution
- */
-
-/**
- * Check if Node.js version is supported
- * Middleware that validates Node version before any command runs
- *
- * Minimum requirement: Node.js >= 18.0.0
- * Note: Individual commands may have stricter requirements (e.g., dev command)
- */
-export const checkNodeVersion = async (): Promise<void> => {
-  let nodeVersionString = process.version;
-  if (nodeVersionString.charAt(0) === 'v') {
-    nodeVersionString = nodeVersionString.slice(1);
-  }
-  const versionArr = nodeVersionString.split('.');
-  const versionStr = versionArr[0];
-  if (!versionStr) {
-    addLog(
-      React.createElement(ErrorLog, {
-        message: `Unable to determine Node.js version (got "${process.version}"). Please ensure you are running Node.js >= 18.0.0.`,
-      })
-    );
-    process.exit(1);
-  }
-  const majorVersion = parseInt(versionStr, 10);
-
-  if (majorVersion < 18) {
-    addLog(
-      React.createElement(ErrorLog, {
-        message: `poetora requires Node.js >= 18.0.0 (current version ${nodeVersionString}). Please upgrade Node.js and try again.`,
-      })
-    );
-    process.exit(1);
-  }
-};
-
-/**
- * Suppress common console warnings that don't affect functionality
- * Filters out known noise from dependencies
- */
-export const suppressConsoleWarnings = (): void => {
-  // Ignore tailwind warnings and punycode deprecation warning
-  const ignoredMessages = [
-    'No utility classes were detected',
-    'https://tailwindcss.com/docs/content-configuration',
-    'DeprecationWarning',
-    'punycode',
-  ];
-
-  const originalConsoleError = console.error;
-  console.error = (...args) => {
-    const message = args.join(' ');
-    if (ignoredMessages.some((ignoredMessage) => message.includes(ignoredMessage))) {
-      return;
-    }
-    originalConsoleError.apply(console, args);
-  };
-
-  const originalConsoleWarn = console.warn;
-  console.warn = (...args) => {
-    const message = args.join(' ');
-    if (ignoredMessages.some((ignoredMessage) => message.includes(ignoredMessage))) {
-      return;
-    }
-    originalConsoleWarn.apply(console, args);
-  };
-};

package/src/services/__tests__/port.service.test.ts

@@ -1,83 +0,0 @@
-import detect from 'detect-port';
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-
-import { NoAvailablePortError } from '../../errors/index.js';
-import { CLI_CONSTANTS } from '../../types/index.js';
-import { PortService } from '../port.service.js';
-
-vi.mock('detect-port');
-
-describe('PortService', () => {
-  let service: PortService;
-  const mockDetect = vi.mocked(detect);
-
-  beforeEach(() => {
-    service = new PortService();
-    vi.clearAllMocks();
-  });
-
-  describe('findAvailablePort', () => {
-    it('should return preferred port when available', async () => {
-      mockDetect.mockResolvedValue(3000);
-
-      const port = await service.findAvailablePort(3000);
-
-      expect(port).toBe(3000);
-      expect(mockDetect).toHaveBeenCalledWith(3000);
-    });
-
-    it('should return default port when no preferred port specified', async () => {
-      mockDetect.mockResolvedValue(CLI_CONSTANTS.PORT.DEFAULT);
-
-      const port = await service.findAvailablePort();
-
-      expect(port).toBe(CLI_CONSTANTS.PORT.DEFAULT);
-    });
-
-    it('should try next port when first is occupied', async () => {
-      mockDetect.mockResolvedValueOnce(3001); // 3000 occupied
-      mockDetect.mockResolvedValueOnce(3001); // 3001 available
-
-      const port = await service.findAvailablePort(3000);
-
-      expect(port).toBe(3001);
-      expect(mockDetect).toHaveBeenCalledTimes(2);
-      expect(mockDetect).toHaveBeenNthCalledWith(1, 3000);
-      expect(mockDetect).toHaveBeenNthCalledWith(2, 3001);
-    });
-
-    it('should try multiple ports until finding available one', async () => {
-      // Ports 3000, 3001, 3002 are occupied
-      mockDetect.mockResolvedValueOnce(3001);
-      mockDetect.mockResolvedValueOnce(3002);
-      mockDetect.mockResolvedValueOnce(3003);
-      // Port 3003 is available
-      mockDetect.mockResolvedValueOnce(3003);
-
-      const port = await service.findAvailablePort(3000);
-
-      expect(port).toBe(3003);
-      expect(mockDetect).toHaveBeenCalledTimes(4);
-    });
-
-    it('should throw NoAvailablePortError after MAX_ATTEMPTS', async () => {
-      // All ports are occupied
-      mockDetect.mockImplementation((port) => Promise.resolve((port as number) + 1));
-
-      await expect(service.findAvailablePort(3000)).rejects.toThrow(NoAvailablePortError);
-      expect(mockDetect).toHaveBeenCalledTimes(CLI_CONSTANTS.PORT.MAX_ATTEMPTS);
-    });
-
-    it('should include start port in error message', async () => {
-      mockDetect.mockImplementation((port) => Promise.resolve((port as number) + 1));
-
-      try {
-        await service.findAvailablePort(5000);
-        expect.fail('Should have thrown error');
-      } catch (error) {
-        expect(error).toBeInstanceOf(NoAvailablePortError);
-        expect((error as NoAvailablePortError).message).toContain('5000');
-      }
-    });
-  });
-});