@poetora/cli 0.1.9 → 0.1.11

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

@@ -1,234 +0,0 @@
-import AdmZip from 'adm-zip';
-import * as fs from 'fs';
-import * as fse from 'fs-extra';
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import { ExternalServiceError, FileSystemError } from '../../errors/index.js';
-import { TemplateService } from '../template.service.js';
-
-vi.mock('node:fs', () => ({
-  default: {
-    promises: {
-      readdir: vi.fn(),
-      writeFile: vi.fn(),
-      readFile: vi.fn(),
-    },
-  },
-  promises: {
-    readdir: vi.fn(),
-    writeFile: vi.fn(),
-    readFile: vi.fn(),
-  },
-}));
-
-vi.mock('fs-extra');
-vi.mock('adm-zip', () => ({
-  default: vi.fn(),
-}));
-
-describe('TemplateService', () => {
-  let service: TemplateService;
-
-  beforeEach(() => {
-    service = new TemplateService();
-    vi.clearAllMocks();
-  });
-
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  describe('checkDirectory', () => {
-    it('should return exists=true and hasContents=true for non-empty directory', async () => {
-      vi.mocked(fse.ensureDir).mockResolvedValue(undefined);
-      vi.mocked(fs.promises.readdir).mockResolvedValue([
-        'file1.txt',
-        'file2.txt',
-      ] as unknown as fs.Dirent[]);
-
-      const result = await service.checkDirectory('/test/dir');
-
-      expect(result).toEqual({
-        exists: true,
-        hasContents: true,
-      });
-      expect(fse.ensureDir).toHaveBeenCalledWith('/test/dir');
-    });
-
-    it('should return exists=true and hasContents=false for empty directory', async () => {
-      vi.mocked(fse.ensureDir).mockResolvedValue(undefined);
-      vi.mocked(fs.promises.readdir).mockResolvedValue([] as unknown as fs.Dirent[]);
-
-      const result = await service.checkDirectory('/test/empty');
-
-      expect(result).toEqual({
-        exists: true,
-        hasContents: false,
-      });
-    });
-
-    it('should return exists=false for non-existent directory on ENOENT', async () => {
-      const error = new Error('ENOENT') as NodeJS.ErrnoException;
-      error.code = 'ENOENT';
-      vi.mocked(fse.ensureDir).mockRejectedValue(error);
-
-      const result = await service.checkDirectory('/nonexistent');
-
-      expect(result).toEqual({
-        exists: false,
-        hasContents: false,
-      });
-    });
-
-    it('should throw FileSystemError for other errors', async () => {
-      vi.mocked(fse.ensureDir).mockRejectedValue(new Error('Permission denied'));
-
-      await expect(service.checkDirectory('/test/dir')).rejects.toThrow(FileSystemError);
-    });
-  });
-
-  describe('getAvailableThemes', () => {
-    it('should return available themes from docsConfigSchema', () => {
-      const themes = service.getAvailableThemes();
-
-      expect(themes).toBeInstanceOf(Array);
-      expect(themes.length).toBeGreaterThan(0);
-      expect(themes).toContain('ora');
-    });
-  });
-
-  describe('installTemplate', () => {
-    const mockFetch = vi.fn();
-    const originalFetch = global.fetch;
-
-    beforeEach(() => {
-      global.fetch = mockFetch;
-
-      // Mock successful download
-      mockFetch.mockResolvedValue({
-        ok: true,
-        arrayBuffer: vi.fn().mockResolvedValue(new ArrayBuffer(8)),
-      });
-
-      // Mock file operations
-      vi.mocked(fs.promises.writeFile).mockResolvedValue(undefined);
-      vi.mocked(fs.promises.readFile).mockResolvedValue('{}');
-      vi.mocked(fse.ensureDir).mockResolvedValue(undefined);
-      vi.mocked(fse.copy).mockResolvedValue(undefined);
-      vi.mocked(fse.remove).mockResolvedValue(undefined);
-
-      // Mock AdmZip constructor - must use function for constructor
-      vi.mocked(AdmZip).mockImplementation(function (this: AdmZip) {
-        return {
-          extractAllTo: vi.fn(),
-        } as unknown as AdmZip;
-      } as unknown as typeof AdmZip);
-    });
-
-    afterEach(() => {
-      global.fetch = originalFetch;
-    });
-
-    it('should successfully install template', async () => {
-      await service.installTemplate({
-        directory: '/test/project',
-        projectName: 'My Project',
-        theme: 'quartz',
-      });
-
-      expect(mockFetch).toHaveBeenCalledWith(
-        'https://github.com/poetora/starter/archive/refs/heads/main.zip'
-      );
-      expect(fse.copy).toHaveBeenCalled();
-      expect(fs.promises.writeFile).toHaveBeenCalledWith(
-        '/test/project/docs.json',
-        expect.stringContaining('My Project'),
-        'utf-8'
-      );
-    });
-
-    it('should throw ExternalServiceError on download failure', async () => {
-      mockFetch.mockResolvedValue({
-        ok: false,
-        status: 404,
-        statusText: 'Not Found',
-      });
-
-      await expect(
-        service.installTemplate({
-          directory: '/test/project',
-          projectName: 'My Project',
-          theme: 'quartz',
-        })
-      ).rejects.toThrow(ExternalServiceError);
-    });
-
-    it('should cleanup temporary files after successful installation', async () => {
-      await service.installTemplate({
-        directory: '/test/project',
-        projectName: 'My Project',
-        theme: 'quartz',
-      });
-
-      expect(fse.remove).toHaveBeenCalledWith('poetora-starter.zip');
-      expect(fse.remove).toHaveBeenCalledWith('poetora-starter-temp');
-    });
-
-    it('should cleanup temporary files even on error', async () => {
-      vi.mocked(fse.copy).mockRejectedValue(new Error('Copy failed'));
-
-      await expect(
-        service.installTemplate({
-          directory: '/test/project',
-          projectName: 'My Project',
-          theme: 'quartz',
-        })
-      ).rejects.toThrow();
-
-      expect(fse.remove).toHaveBeenCalled();
-    });
-
-    it('should handle missing docs.json by creating new config', async () => {
-      vi.mocked(fs.promises.readFile).mockRejectedValue(new Error('ENOENT'));
-
-      await service.installTemplate({
-        directory: '/test/project',
-        projectName: 'My Project',
-        theme: 'quartz',
-      });
-
-      expect(fs.promises.writeFile).toHaveBeenCalledWith(
-        '/test/project/docs.json',
-        expect.stringContaining('"name": "My Project"'),
-        'utf-8'
-      );
-    });
-
-    it('should preserve existing config fields when updating', async () => {
-      vi.mocked(fs.promises.readFile).mockResolvedValue(
-        JSON.stringify({
-          name: 'Old Name',
-          theme: 'old-theme',
-          customField: 'preserved',
-        })
-      );
-
-      await service.installTemplate({
-        directory: '/test/project',
-        projectName: 'New Project',
-        theme: 'quartz',
-      });
-
-      const writeCall = vi
-        .mocked(fs.promises.writeFile)
-        .mock.calls.find((call) => call[0] === '/test/project/docs.json');
-
-      expect(writeCall).toBeDefined();
-      const writtenConfig = JSON.parse(writeCall?.[1] as string);
-      expect(writtenConfig).toMatchObject({
-        name: 'New Project',
-        theme: 'quartz',
-        customField: 'preserved',
-      });
-    });
-  });
-});

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

@@ -1,165 +0,0 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
-import { VersionService } from '../version.service.js';
-
-describe('VersionService', () => {
-  let service: VersionService;
-  let _originalVersion: string;
-
-  beforeEach(() => {
-    service = new VersionService();
-    _originalVersion = process.version;
-  });
-
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  describe('parseNodeVersion', () => {
-    it('should parse version with v prefix', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('v20.17.0');
-
-      const version = service.parseNodeVersion();
-
-      expect(version.major).toBe(20);
-      expect(version.minor).toBe(17);
-      expect(version.patch).toBe(0);
-      expect(version.raw).toBe('20.17.0');
-    });
-
-    it('should parse version without v prefix', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('18.0.0');
-
-      const version = service.parseNodeVersion();
-
-      expect(version.major).toBe(18);
-      expect(version.minor).toBe(0);
-      expect(version.patch).toBe(0);
-    });
-
-    it('should parse version with only major.minor', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('v20.17');
-
-      const version = service.parseNodeVersion();
-
-      expect(version.major).toBe(20);
-      expect(version.minor).toBe(17);
-      expect(version.patch).toBe(0);
-    });
-  });
-
-  describe('checkNodeVersion', () => {
-    it('should pass for Node 20.17.0', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('v20.17.0');
-
-      const result = service.checkNodeVersion();
-
-      expect(result.isValid).toBe(true);
-      expect(result.hasWarning).toBe(false);
-    });
-
-    it('should pass for Node 18.0.0 (minimum)', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('v18.0.0');
-
-      const result = service.checkNodeVersion();
-
-      expect(result.isValid).toBe(true);
-      expect(result.hasWarning).toBe(true);
-      expect(result.message).toContain('20.17');
-    });
-
-    it('should pass for Node 24.x', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('v24.0.0');
-
-      const result = service.checkNodeVersion();
-
-      expect(result.isValid).toBe(true);
-    });
-
-    it('should fail for Node 17.x', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('v17.9.0');
-
-      const result = service.checkNodeVersion();
-
-      expect(result.isValid).toBe(false);
-      expect(result.message).toContain('18.0');
-    });
-
-    it('should fail for Node 25.x', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('v25.0.0');
-
-      const result = service.checkNodeVersion();
-
-      expect(result.isValid).toBe(false);
-      expect(result.message).toContain('not supported');
-    });
-
-    it('should warn for Node 19.x (below recommended)', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('v19.5.0');
-
-      const result = service.checkNodeVersion();
-
-      expect(result.isValid).toBe(true);
-      expect(result.hasWarning).toBe(true);
-      expect(result.message).toContain('recommended');
-    });
-
-    it('should warn for Node 20.16 (below recommended minor)', () => {
-      vi.spyOn(process, 'version', 'get').mockReturnValue('v20.16.0');
-
-      const result = service.checkNodeVersion();
-
-      expect(result.isValid).toBe(true);
-      expect(result.hasWarning).toBe(true);
-    });
-  });
-
-  describe('getCliVersion', () => {
-    it('should return test-cli in test mode', () => {
-      const originalEnv = process.env.CLI_TEST_MODE;
-      process.env.CLI_TEST_MODE = 'true';
-
-      const version = service.getCliVersion();
-
-      expect(version).toBe('test-cli');
-
-      process.env.CLI_TEST_MODE = originalEnv;
-    });
-
-    it('should return version from package.json', () => {
-      const version = service.getCliVersion();
-
-      expect(version).toBeDefined();
-      expect(typeof version).toBe('string');
-    });
-  });
-
-  describe('getLatestCliVersion', () => {
-    it('should return latest version from npm', () => {
-      const version = service.getLatestCliVersion('@poetora/cli');
-
-      expect(version).toBeDefined();
-      expect(typeof version).toBe('string');
-      expect(version).toMatch(/^\d+\.\d+\.\d+/);
-    });
-  });
-
-  describe('isVersionUpToDate', () => {
-    it('should return true when versions match', () => {
-      const result = service.isVersionUpToDate('1.0.0', '1.0.0');
-
-      expect(result).toBe(true);
-    });
-
-    it('should return false when versions differ', () => {
-      const result = service.isVersionUpToDate('1.0.0', '1.0.1');
-
-      expect(result).toBe(false);
-    });
-
-    it('should handle whitespace', () => {
-      const result = service.isVersionUpToDate('1.0.0  ', '  1.0.0');
-
-      expect(result).toBe(true);
-    });
-  });
-});

package/src/services/accessibility-check.service.ts

@@ -1,226 +0,0 @@
-import { getConfigObj, getConfigPath } from '@poetora/prebuild';
-import { getBackgroundColors } from '@poetora/shared';
-import path from 'path';
-import {
-  type AccessibilityCheckResult,
-  type ContrastResult,
-  checkDocsColors,
-} from '../accessibility.js';
-import { CMD_EXEC_PATH } from '../constants.js';
-import { type AccessibilityFixAttribute, checkMdxAccessibility } from '../mdxAccessibility.js';
-import { ConsoleLogger } from '../utils/console-logger.js';
-import type { ILogger } from '../utils/logger.interface.js';
-
-/**
- * Service for accessibility checking
- */
-export class AccessibilityCheckService {
-  constructor(private readonly logger: ILogger = new ConsoleLogger()) {}
-
-  /**
-   * Display contrast result with color-coded output
-   */
-  private displayContrastResult(
-    result: ContrastResult | null,
-    label: string,
-    prefix: string = ''
-  ): void {
-    if (!result) return;
-
-    const { recommendation } = result;
-
-    // Custom messages with our own tone
-    let statusText: string;
-    let detailMessage: string;
-
-    if (recommendation === 'pass') {
-      if (result.meetsAAA) {
-        statusText = '✓ Excellent';
-        detailMessage = `ratio ${result.ratio.toFixed(2)}:1 (AAA standard)`;
-      } else {
-        statusText = '✓ Good';
-        detailMessage = `ratio ${result.ratio.toFixed(2)}:1 (AA standard)`;
-      }
-    } else if (recommendation === 'warning') {
-      statusText = '⚠ Acceptable';
-      detailMessage = `ratio ${result.ratio.toFixed(2)}:1 (consider improving)`;
-    } else {
-      statusText = '✗ Poor';
-      detailMessage = `ratio ${result.ratio.toFixed(2)}:1 (below AA standard)`;
-    }
-
-    // Display with label in cyan and status+message in appropriate color
-    const labelText = `${prefix}${label}:`;
-    const statusColor =
-      recommendation === 'pass' ? 'green' : recommendation === 'warning' ? 'yellow' : 'red';
-
-    this.logger.log(`${labelText} ${this.colorize(statusText, statusColor)} ${detailMessage}`);
-  }
-
-  private colorize(text: string, color: 'green' | 'yellow' | 'red'): string {
-    // Use chalk directly for inline coloring
-    const chalk = (text: string, color: 'green' | 'yellow' | 'red') => {
-      const colors: Record<string, string> = {
-        green: '\x1b[32m',
-        yellow: '\x1b[33m',
-        red: '\x1b[31m',
-        reset: '\x1b[0m',
-      };
-      return `${colors[color]}${text}${colors.reset}`;
-    };
-    return chalk(text, color);
-  }
-
-  /**
-   * Check color contrast accessibility
-   */
-  private async checkColorAccessibility(): /* prettier-ignore */ Promise<number> {
-    try {
-      const docsConfigPath = await getConfigPath(CMD_EXEC_PATH);
-
-      if (!docsConfigPath) {
-        this.logger.error(
-          'No configuration file found. Please run this command from a directory with a docs.json file.'
-        );
-        return 1;
-      }
-
-      const config = await getConfigObj(CMD_EXEC_PATH);
-
-      if (!config?.colors) {
-        this.logger.warn('No colors section found in configuration file');
-        return 0;
-      }
-
-      const { colors, navigation } = config;
-      const { lightHex, darkHex } = getBackgroundColors(config);
-
-      const results: AccessibilityCheckResult = checkDocsColors(
-        colors,
-        { lightHex, darkHex },
-        navigation
-      );
-
-      this.logger.log('Checking color accessibility...');
-
-      this.displayContrastResult(
-        results.primaryContrast,
-        `Primary Color (${colors.primary}) vs Light Background`
-      );
-      this.displayContrastResult(
-        results.lightContrast,
-        `Light Color (${colors.light}) vs Dark Background`
-      );
-      this.displayContrastResult(
-        results.darkContrast,
-        `Dark Color (${colors.dark}) vs Dark Background`
-      );
-      this.displayContrastResult(
-        results.darkOnLightContrast,
-        `Dark Color (${colors.dark}) vs Light Background`
-      );
-
-      const anchorsWithResults = results.anchorResults.filter(
-        (anchor) => anchor.lightContrast || anchor.darkContrast
-      );
-
-      if (anchorsWithResults.length > 0) {
-        for (const anchor of anchorsWithResults) {
-          this.displayContrastResult(anchor.lightContrast, `${anchor.name} vs Light Background`);
-          this.displayContrastResult(anchor.darkContrast, `${anchor.name} vs Dark Background`);
-        }
-      }
-
-      // Overall assessment with custom messages
-      let overallDisplay: string;
-      if (results.overallScore === 'fail') {
-        overallDisplay =
-          this.colorize('✗ Action needed:', 'red') +
-          ' Update colors to meet accessibility standards';
-      } else if (results.overallScore === 'warning') {
-        overallDisplay = `${this.colorize('⚠ Room for improvement:', 'yellow')} Consider enhancing color contrast`;
-      } else {
-        overallDisplay = `${this.colorize('✓ All good:', 'green')} Colors meet accessibility guidelines`;
-      }
-
-      this.logger.logNewLine();
-      this.logger.log(overallDisplay);
-
-      return results.overallScore === 'fail' ? 1 : 0;
-    } catch (error) {
-      this.logger.error(`Failed to check color accessibility: ${error}`);
-      return 1;
-    }
-  }
-
-  /**
-   * Check MDX files for accessibility issues
-   */
-  private async checkMdxAccessibility(): /* prettier-ignore */ Promise<number> {
-    try {
-      this.logger.log('Checking mdx files for accessibility issues...');
-
-      const results = await checkMdxAccessibility();
-
-      if (results.missingAltAttributes.length === 0) {
-        this.logger.success('no accessibility issues found');
-        this.logger.info(
-          `Checked ${results.totalFiles} MDX files - all images and videos have alt attributes.`
-        );
-        return 0;
-      }
-
-      const issuesByFile: Record<string, AccessibilityFixAttribute[]> = {};
-      results.missingAltAttributes.forEach((issue) => {
-        if (!issuesByFile[issue.filePath]) {
-          issuesByFile[issue.filePath] = [];
-        }
-        issuesByFile[issue.filePath]?.push(issue);
-      });
-
-      this.logger.log(
-        `Found ${this.logger.highlight?.(results.missingAltAttributes.length.toString()) ?? results.missingAltAttributes.length} accessibility issue${results.missingAltAttributes.length === 1 ? '' : 's'} in ${this.logger.highlight?.(results.filesWithIssues.toString()) ?? results.filesWithIssues} file${results.filesWithIssues === 1 ? '' : 's'}:`
-      );
-
-      for (const [filePath, issues] of Object.entries(issuesByFile)) {
-        const relativePath = path.relative(process.cwd(), filePath);
-        this.logger.log(`${relativePath}:`);
-
-        for (const issue of issues) {
-          const location =
-            issue.line && issue.column ? ` (line ${issue.line}, col ${issue.column})` : '';
-          if (issue.element === 'a') {
-            this.logger.logColor(
-              ` ✗ Missing text attribute ${issue.tagName} element${location}`,
-              'red'
-            );
-          } else {
-            this.logger.logColor(
-              ` ✗ Missing alt attribute on ${issue.tagName} element${location}`,
-              'red'
-            );
-          }
-        }
-      }
-
-      this.logger.warn(
-        'Recommendation: Add alt attributes to all images and videos for better accessibility.'
-      );
-
-      return 1;
-    } catch (error) {
-      this.logger.error(`Failed to check MDX accessibility: ${error}`);
-      return 1;
-    }
-  }
-
-  /**
-   * Run accessibility checks on documentation
-   * @returns Exit code (0 for success, 1 for failure)
-   */
-  async checkAccessibility(): /* prettier-ignore */ Promise<number> {
-    const colorCheckCode = await this.checkColorAccessibility();
-    const mdxCheckCode = await this.checkMdxAccessibility();
-    return colorCheckCode || mdxCheckCode;
-  }
-}

package/src/services/index.ts

@@ -1,7 +0,0 @@
-export * from './accessibility-check.service.js';
-export * from './link.service.js';
-export * from './openapi-check.service.js';
-export * from './port.service.js';
-export * from './template.service.js';
-export * from './update.service.js';
-export * from './version.service.js';

package/src/services/link.service.ts

@@ -1,65 +0,0 @@
-import {
-  getBrokenInternalLinks,
-  type MdxPath,
-  renameFilesAndUpdateLinksInContent,
-} from '@poetora/link-rot';
-import * as path from 'path';
-
-import type { ILogger } from '../utils/index.js';
-
-/**
- * Service for link management
- */
-export class LinkService {
-  constructor(private readonly logger: ILogger) {}
-
-  /**
-   * Check for broken internal links
-   * @returns Broken links grouped by file
-   */
-  async checkBrokenLinks(): Promise<Record<string, string[]>> {
-    const brokenLinks = await getBrokenInternalLinks();
-
-    if (brokenLinks.length === 0) {
-      this.logger.success('no broken links found');
-      return {};
-    }
-
-    // Group broken links by file
-    const brokenLinksByFile: Record<string, string[]> = {};
-    brokenLinks.forEach((mdxPath: MdxPath) => {
-      const filename = path.join(mdxPath.relativeDir, mdxPath.filename);
-      const brokenLinksForFile = brokenLinksByFile[filename];
-      if (brokenLinksForFile) {
-        brokenLinksForFile.push(mdxPath.originalPath);
-      } else {
-        brokenLinksByFile[filename] = [mdxPath.originalPath];
-      }
-    });
-
-    // Display broken links with highlighted numbers
-    const fileCount = Object.keys(brokenLinksByFile).length;
-    this.logger.log(
-      `found ${this.logger.highlight?.(brokenLinks.length.toString()) ?? brokenLinks.length} broken link${brokenLinks.length === 1 ? '' : 's'} in ${this.logger.highlight?.(fileCount.toString()) ?? fileCount} file${fileCount === 1 ? '' : 's'}`
-    );
-    this.logger.logNewLine();
-
-    for (const [filename, links] of Object.entries(brokenLinksByFile)) {
-      // Underline filename for better visibility
-      this.logger.logColor(filename, 'cyan');
-      links.forEach((link) => {
-        this.logger.logColor(` ⎿  ${link}`, 'gray');
-      });
-      this.logger.logNewLine();
-    }
-
-    return brokenLinksByFile;
-  }
-
-  /**
-   * Rename a file and update all internal link references
-   */
-  async renameFile(from: string, to: string, force: boolean = false): Promise<void> {
-    await renameFilesAndUpdateLinksInContent(from, to, force);
-  }
-}