@pie-element/cli 0.1.1 → 0.1.2-next.0

package/src/lib/mathquill/pie-extractor.ts

@@ -1,194 +0,0 @@
-import { writeFileSync } from 'node:fs';
-import { join } from 'node:path';
-import type { Logger } from '../../utils/logger.js';
-
-/**
- * PIE Feature Extractor
- *
- * Extracts PIE-specific customizations from the legacy bundle:
- * - Matrix commands (pmatrix, bmatrix, vmatrix)
- * - LRN exponent notation
- */
-
-export interface PIEExtractionResult {
-  features: string[];
-  files: string[];
-}
-
-export async function extractPIEFeatures(
-  _mathquillPath: string,
-  outputPath: string,
-  logger: Logger
-): Promise<PIEExtractionResult> {
-  const features: string[] = [];
-  const files: string[] = [];
-
-  // Extract matrices
-  logger.progress('   Extracting matrix commands...');
-  const matricesFile = join(outputPath, 'matrices.ts');
-  writeFileSync(matricesFile, generateMatricesCode());
-  features.push('matrices');
-  files.push(matricesFile);
-
-  // Extract LRN exponent
-  logger.progress('   Extracting LRN exponent...');
-  const lrnExponentFile = join(outputPath, 'lrn-exponent.ts');
-  writeFileSync(lrnExponentFile, generateLRNExponentCode());
-  features.push('lrn-exponent');
-  files.push(lrnExponentFile);
-
-  // Copy matrix styles
-  logger.progress('   Creating matrix styles...');
-  const stylesFile = join(outputPath, 'styles.css');
-  writeFileSync(stylesFile, generateMatrixStyles());
-  files.push(stylesFile);
-
-  // Create index file
-  const indexFile = join(outputPath, 'index.ts');
-  writeFileSync(indexFile, generatePIEIndexCode());
-  files.push(indexFile);
-
-  return { features, files };
-}
-
-function generateMatricesCode(): string {
-  return `/**
- * Matrix Commands (PIE)
- *
- * Source: PIE fork (2019)
- * File: src/legacy/mathquill-bundle.js lines 6600-6980
- *
- * Adds support for LaTeX matrix commands:
- * - \\pmatrix: Matrix with parentheses ()
- * - \\bmatrix: Matrix with square brackets []
- * - \\vmatrix: Matrix with vertical bars ||
- *
- * These are standard LaTeX matrix types used in mathematics education.
- */
-
-import type { MathQuillInterface } from 'mathquill';
-import './styles.css';
-
-export function addMatrixCommands(MQ: MathQuillInterface): void {
-  // Access MathQuill internals
-  const mqInternal = MQ as any;
-
-  if (!mqInternal.L?.LatexCmds) {
-    console.warn('MathQuill internals not accessible for matrix commands');
-    return;
-  }
-
-  // NOTE: This is a placeholder implementation
-  // The full PIE matrix implementation from the legacy bundle is complex
-  // and needs to be carefully ported to work with the Desmos fork.
-  //
-  // The legacy implementation (lines 6600-6980 in mathquill-bundle.js) includes:
-  // - Matrix base class with row/column management
-  // - Cell navigation and editing
-  // - LaTeX parsing and generation
-  // - HTML template generation
-  // - Reflow and sizing logic
-  //
-  // For the initial migration, we document that this needs implementation.
-  // The matrices from the legacy bundle should be extracted and adapted
-  // to work with the modern TypeScript Desmos fork.
-
-  // TODO: Extract and port full matrix implementation from legacy bundle
-  // Lines 6600-6980 in packages/shared/mathquill/src/legacy/mathquill-bundle.js
-
-  console.info('Matrix commands (pmatrix, bmatrix, vmatrix) need implementation');
-  console.info('See: packages/shared/mathquill/src/legacy/mathquill-bundle.js lines 6600-6980');
-}
-`;
-}
-
-function generateLRNExponentCode(): string {
-  return `/**
- * LRN Exponent Command (PIE)
- *
- * Source: PIE fork (2019)
- * File: src/legacy/mathquill-bundle.js
- *
- * Custom exponent notation for learning platforms.
- * Provides a specialized exponent input format.
- *
- * Usage: \\lrnexponent{base}{exponent}
- */
-
-import type { MathQuillInterface } from 'mathquill';
-
-export function addLRNExponent(MQ: MathQuillInterface): void {
-  // Access MathQuill internals
-  const mqInternal = MQ as any;
-
-  if (!mqInternal.L?.LatexCmds) {
-    console.warn('MathQuill internals not accessible for LRN exponent');
-    return;
-  }
-
-  // NOTE: This is a placeholder implementation
-  // The full PIE lrnexponent implementation needs to be extracted
-  // from the legacy bundle and ported to TypeScript.
-  //
-  // The legacy implementation includes:
-  // - Custom HTML template with specific styling
-  // - Special handling for base and exponent parts
-  // - Text template for plain text representation
-  //
-  // TODO: Extract and port lrnexponent implementation from legacy bundle
-  // Search for 'lrnexponent' in packages/shared/mathquill/src/legacy/mathquill-bundle.js
-
-  console.info('LRN exponent command needs implementation');
-  console.info('See: packages/shared/mathquill/src/legacy/mathquill-bundle.js (search: lrnexponent)');
-}
-`;
-}
-
-function generateMatrixStyles(): string {
-  return `/**
- * Matrix Styles (PIE)
- *
- * CSS styling for matrix rendering
- * Ported from packages/shared/mathquill/src/css/matrixed.less
- */
-
-.mq-math-mode .mq-matrix {
-  display: inline-block;
-  vertical-align: middle;
-}
-
-.mq-math-mode .mq-matrix table {
-  border-collapse: collapse;
-  border-spacing: 0;
-  margin: 0.1em 0;
-}
-
-.mq-math-mode .mq-matrix td {
-  padding: 0.2em 0.4em;
-  text-align: center;
-  vertical-align: middle;
-}
-
-.mq-math-mode .mq-matrix .mq-paren {
-  font-size: 1.2em;
-  vertical-align: middle;
-}
-
-.mq-math-mode .mq-matrix .mq-scaled {
-  display: inline-block;
-  transform-origin: center;
-}
-`;
-}
-
-function generatePIEIndexCode(): string {
-  return `/**
- * PIE Extensions
- *
- * PIE-specific customizations for educational assessment
- */
-
-export { addMatrixCommands } from './matrices.js';
-export { addLRNExponent } from './lrn-exponent.js';
-`;
-}

package/src/lib/upstream/repo-utils.ts

@@ -1,18 +0,0 @@
-import { existsSync } from 'node:fs';
-
-type RepoSpec = {
-  label: string;
-  path: string;
-  extraMessage?: string;
-};
-
-export const formatRepoNotFound = (label: string, path: string, extraMessage?: string): string =>
-  `${label} not found at ${path}${extraMessage ? `\n${extraMessage}` : ''}`;
-
-export const assertReposExist = (repos: RepoSpec[]): void => {
-  for (const repo of repos) {
-    if (!existsSync(repo.path)) {
-      throw new Error(formatRepoNotFound(repo.label, repo.path, repo.extraMessage));
-    }
-  }
-};

package/src/lib/upstream/sync-compatibility.ts

@@ -1,89 +0,0 @@
-/**
- * Compatibility checking utilities for upstream sync operations
- *
- * Consolidates subdirectory-level ESM compatibility checks that were
- * duplicated across controllers-strategy and react-strategy.
- */
-
-import type { SyncContext } from './sync-strategy.js';
-
-export type SubdirectoryType = 'src' | 'configure' | 'controller';
-
-/**
- * Check if a subdirectory is ESM-compatible based on the compatibility report
- *
- * This function consolidates the duplicated compatibility checking logic from
- * ControllersStrategy and ReactComponentsStrategy.
- *
- * @param elementName - Name of the element (e.g., "multiple-choice")
- * @param subdir - Subdirectory to check ("src", "configure", or "controller")
- * @param context - Sync context containing the compatibility report
- * @returns true if compatible or no report available, false if incompatible
- */
-export function isSubdirectoryCompatible(
-  elementName: string,
-  subdir: SubdirectoryType,
-  context: SyncContext
-): boolean {
-  // If no compatibility report, assume compatible (backward compatibility)
-  if (!context.compatibilityReport) {
-    return true;
-  }
-
-  const elementDetail = context.compatibilityReport.elementDetails[elementName];
-  if (!elementDetail) {
-    return true; // Element not in report, assume compatible
-  }
-
-  // Check subdirectory-specific compatibility
-  switch (subdir) {
-    case 'configure':
-      return elementDetail.configure?.compatible ?? elementDetail.compatible;
-    case 'controller':
-      return elementDetail.controller?.compatible ?? elementDetail.compatible;
-    case 'src':
-      return elementDetail.studentUI?.compatible ?? elementDetail.compatible;
-    default:
-      return elementDetail.compatible;
-  }
-}
-
-/**
- * Check if an element is ESM-compatible overall
- */
-export function isElementCompatible(elementName: string, context: SyncContext): boolean {
-  if (!context.compatibilityReport) {
-    return true;
-  }
-
-  return context.compatibilityReport.elements.includes(elementName);
-}
-
-/**
- * Get compatibility blockers for an element subdirectory
- */
-export function getCompatibilityBlockers(
-  elementName: string,
-  subdir: SubdirectoryType,
-  context: SyncContext
-): string[] {
-  if (!context.compatibilityReport) {
-    return [];
-  }
-
-  const elementDetail = context.compatibilityReport.elementDetails[elementName];
-  if (!elementDetail) {
-    return [];
-  }
-
-  switch (subdir) {
-    case 'configure':
-      return elementDetail.configure?.blockers ?? [];
-    case 'controller':
-      return elementDetail.controller?.blockers ?? [];
-    case 'src':
-      return elementDetail.studentUI?.blockers ?? [];
-    default:
-      return [];
-  }
-}

package/src/lib/upstream/sync-constants.ts

@@ -1,77 +0,0 @@
-/**
- * Shared constants for upstream sync operations
- */
-
-// Default repository paths
-export const DEFAULT_PATHS = {
-  PIE_ELEMENTS: '../pie-elements',
-  PIE_LIB: '../pie-lib',
-  PIE_ELEMENTS_NG: '.',
-} as const;
-
-// Workspace dependency patterns
-export const WORKSPACE = {
-  VERSION: 'workspace:*',
-  PIE_LIB_PREFIX: '@pie-lib/',
-  PIE_ELEMENT_PREFIX: '@pie-element/',
-  PIE_FRAMEWORK_PREFIX: '@pie-framework/',
-} as const;
-
-// Upstream element packages intentionally excluded from sync.
-// These templates are not used in pie-elements-ng runtime or are temporarily disabled.
-export const EXCLUDED_UPSTREAM_ELEMENTS = [
-  'boilerplate-item-type',
-  'math-inline',
-  'math-templated',
-] as const;
-
-// Build tool versions
-export const BUILD_TOOLS = {
-  VITE: '^6.0.0',
-  TYPESCRIPT: '^5.9.3',
-  VITE_REACT_PLUGIN: '^4.2.0',
-  LODASH_ES: '^4.17.22',
-} as const;
-
-// React versions
-export const REACT = {
-  VERSION: '^18.0.0',
-  TYPES_VERSION: '^18.2.0',
-} as const;
-
-// Directory patterns to skip
-export const SKIP_PATTERNS = {
-  TEST_DIRS: ['__tests__', '__mocks__'],
-  FILE_EXTENSIONS: ['.js', '.jsx', '.ts', '.tsx'],
-} as const;
-
-// File types
-export const FILE_TYPES = {
-  JS: '.js',
-  JSX: '.jsx',
-  TS: '.ts',
-  TSX: '.tsx',
-} as const;
-
-// Package.json defaults
-export const PACKAGE_DEFAULTS = {
-  TYPE: 'module',
-  SIDE_EFFECTS: false,
-  FILES: ['dist', 'src'],
-} as const;
-
-// Build scripts
-export const SCRIPTS = {
-  BUILD: 'bun x vite build && bun x tsc --emitDeclarationOnly',
-  BUILD_WITH_IIFE:
-    'bun x vite build && bun x vite build --config vite.config.iife.ts && bun x tsc --emitDeclarationOnly',
-  DEV: 'bun x vite',
-  DEMO: 'bun x vite --mode demo',
-  TEST: 'bun x vitest run',
-} as const;
-
-// Compatibility report file
-export const COMPATIBILITY_FILE = './.compatibility/report.json';
-
-// Upstream history file
-export const HISTORY_FILE = '.upstream-sync-history.json';

package/src/lib/upstream/sync-controllers-strategy.ts

@@ -1,291 +0,0 @@
-/**
- * Controller sync strategy - syncs controller logic from upstream pie-elements
- *
- * Synced from: pie-elements/packages/{element}/controller/src/
- * Target: packages/elements-react/{element}/src/controller/
- */
-
-import { existsSync } from 'node:fs';
-import { mkdir, readFile, writeFile } from 'node:fs/promises';
-import { dirname, join } from 'node:path';
-import { convertJsToTs } from '../../utils/conversion.js';
-import { getCurrentCommit } from '../../utils/git.js';
-import type { SyncStrategy, SyncContext, SyncConfig, SyncResult } from './sync-strategy.js';
-import { cleanDirectory, existsAny, readdir } from './sync-filesystem.js';
-import { createExternalFunction } from './sync-externals.js';
-import { createControllerTransformPipeline } from './sync-transforms.js';
-import { ensureElementPackageJson } from './sync-package-manager.js';
-import { isSubdirectoryCompatible } from './sync-compatibility.js';
-import { EXCLUDED_UPSTREAM_ELEMENTS } from './sync-constants.js';
-
-interface InternalSyncResult {
-  filesChecked: number;
-  filesCopied: number;
-  filesSkipped: number;
-  filesUpdated: number;
-}
-
-export class ControllersStrategy implements SyncStrategy {
-  private touchedElementPackages = new Set<string>();
-  private result: InternalSyncResult = {
-    filesChecked: 0,
-    filesCopied: 0,
-    filesSkipped: 0,
-    filesUpdated: 0,
-  };
-
-  getName(): string {
-    return 'controllers';
-  }
-
-  getDescription(): string {
-    return 'controllers';
-  }
-
-  shouldRun(config: SyncConfig): boolean {
-    return config.syncControllers;
-  }
-
-  async execute(context: SyncContext): Promise<SyncResult> {
-    const { config, logger } = context;
-    this.touchedElementPackages.clear();
-    this.result = {
-      filesChecked: 0,
-      filesCopied: 0,
-      filesSkipped: 0,
-      filesUpdated: 0,
-    };
-
-    if (!logger.isVerbose()) {
-      logger.progressStart('Syncing controllers...');
-    } else {
-      logger.section('📦 Syncing controllers');
-    }
-
-    const upstreamElementsDir = join(config.pieElements, 'packages');
-    const targetBaseDir = join(config.pieElementsNg, 'packages/elements-react');
-
-    const upstreamCommit = getCurrentCommit(config.pieElements);
-
-    const packages = await readdir(upstreamElementsDir);
-
-    for (const pkg of packages) {
-      if (EXCLUDED_UPSTREAM_ELEMENTS.includes(pkg as (typeof EXCLUDED_UPSTREAM_ELEMENTS)[number])) {
-        continue;
-      }
-
-      if (context.packageFilter && pkg !== context.packageFilter) {
-        continue;
-      }
-
-      // Check if controller exists upstream
-      const controllerPath = join(upstreamElementsDir, pkg, 'controller/src/index.js');
-      if (!existsSync(controllerPath)) {
-        continue;
-      }
-
-      // Check if controller subdirectory is ESM-compatible
-      const controllerCompatible = isSubdirectoryCompatible(pkg, 'controller', context);
-      if (!controllerCompatible) {
-        if (logger.isVerbose()) {
-          logger.info(`  ⏭️  ${pkg}: skipping controller/ (not ESM-compatible)`);
-        }
-        continue;
-      }
-
-      this.result.filesChecked++;
-
-      // Target: packages/elements-react/{element}/src/controller/index.ts
-      const targetDir = join(targetBaseDir, pkg, 'src/controller');
-      const targetPath = join(targetDir, 'index.ts');
-
-      // Clean target controller subtree first so removed upstream files don't linger
-      await this.cleanTargetDir(targetDir, `elements-react/${pkg}/src/controller`, logger);
-
-      // Read source
-      let sourceContent = await readFile(controllerPath, 'utf-8');
-
-      // Apply all standard transformations
-      const transformPipeline = createControllerTransformPipeline();
-      sourceContent = transformPipeline(sourceContent);
-
-      // Convert JS to TS
-      const { code: converted } = convertJsToTs(sourceContent, {
-        sourcePath: `pie-elements/packages/${pkg}/controller/src/index.js`,
-        commit: upstreamCommit,
-      });
-
-      let elementChanged = false;
-
-      // Sync index.ts
-      // NOTE: This comparison is exact (===) to ensure any changes in:
-      // - Upstream source code
-      // - Transformation logic (lodash-es, event packages, etc.)
-      // - Sync version (tracked in @sync-version header)
-      // will trigger a file update. This prioritizes correctness over speed.
-      const isNew = !existsSync(targetPath);
-      if (!isNew) {
-        const currentContent = await readFile(targetPath, 'utf-8');
-        if (currentContent === converted) {
-          this.result.filesSkipped++;
-        } else {
-          this.result.filesUpdated++;
-          elementChanged = true;
-          await mkdir(dirname(targetPath), { recursive: true });
-          await writeFile(targetPath, converted, 'utf-8');
-        }
-      } else {
-        this.result.filesCopied++;
-        elementChanged = true;
-        await mkdir(dirname(targetPath), { recursive: true });
-        await writeFile(targetPath, converted, 'utf-8');
-      }
-
-      // Also sync related files (defaults.js, utils.js, scoring.js, tickUtils.js)
-      const relatedFiles = ['defaults.js', 'utils.js', 'scoring.js', 'tickUtils.js'];
-      for (const file of relatedFiles) {
-        const relatedPath = join(upstreamElementsDir, pkg, 'controller/src', file);
-        if (existsSync(relatedPath)) {
-          const relatedTarget = join(targetDir, file.replace('.js', '.ts'));
-          let relatedContent = await readFile(relatedPath, 'utf-8');
-
-          // Apply all standard transformations
-          relatedContent = transformPipeline(relatedContent);
-
-          const { code: relatedConverted } = convertJsToTs(relatedContent, {
-            sourcePath: `pie-elements/packages/${pkg}/controller/src/${file}`,
-            commit: upstreamCommit,
-          });
-
-          const relatedIsNew = !existsSync(relatedTarget);
-          if (!relatedIsNew) {
-            const currentRelated = await readFile(relatedTarget, 'utf-8');
-            if (currentRelated === relatedConverted) {
-              this.result.filesSkipped++;
-              continue;
-            }
-            this.result.filesUpdated++;
-            elementChanged = true;
-          } else {
-            this.result.filesCopied++;
-            elementChanged = true;
-          }
-
-          await mkdir(dirname(relatedTarget), { recursive: true });
-          await writeFile(relatedTarget, relatedConverted, 'utf-8');
-        }
-      }
-
-      // Ensure package.json has ESM module support (even in controller-only sync)
-      let wrotePkgJson = false;
-      const elementDir = join(targetBaseDir, pkg);
-      wrotePkgJson = await ensureElementPackageJson(pkg, elementDir, config);
-      await this.ensureElementViteConfig(pkg, elementDir, logger);
-
-      if (elementChanged || wrotePkgJson) {
-        if (logger.isVerbose()) {
-          logger.success(`  🔄 ${pkg}: controller package updated`);
-        }
-        this.touchedElementPackages.add(pkg);
-      }
-    }
-
-    if (!logger.isVerbose()) {
-      logger.progressCompleteWithCount(this.touchedElementPackages.size, 'packages');
-    } else {
-      logger.info(`\nSynced ${this.touchedElementPackages.size} controller package(s)`);
-    }
-    return {
-      count: this.touchedElementPackages.size,
-      packageNames: Array.from(this.touchedElementPackages).sort(),
-    };
-  }
-
-  private async cleanTargetDir(targetDir: string, label: string, logger: any): Promise<void> {
-    await cleanDirectory(targetDir, label, { dryRun: false, verbose: false }, logger);
-  }
-
-  private async ensureElementViteConfig(
-    _elementName: string,
-    elementDir: string,
-    _logger: any
-  ): Promise<void> {
-    if (!existsSync(elementDir)) {
-      return;
-    }
-
-    const viteConfigPath = join(elementDir, 'vite.config.ts');
-
-    // Always regenerate to reflect current directory structure
-    // (e.g., if configure/ was skipped due to ESM incompatibility)
-
-    // Detect entry points
-    const entryPoints: Record<string, string> = {};
-    const indexEntry = existsAny([
-      join(elementDir, 'src/index.ts'),
-      join(elementDir, 'src/index.tsx'),
-    ]);
-    if (indexEntry) {
-      entryPoints.index = 'src/index.ts';
-    }
-
-    const hasController = existsAny([
-      join(elementDir, 'src/controller/index.ts'),
-      join(elementDir, 'src/controller/index.tsx'),
-    ]);
-    if (hasController) {
-      entryPoints['controller/index'] = 'src/controller/index.ts';
-    }
-
-    const hasConfigure = existsAny([
-      join(elementDir, 'src/configure/index.ts'),
-      join(elementDir, 'src/configure/index.tsx'),
-    ]);
-    if (hasConfigure) {
-      entryPoints['configure/index'] = 'src/configure/index.ts';
-    }
-
-    const hasDelivery = existsAny([
-      join(elementDir, 'src/delivery/index.ts'),
-      join(elementDir, 'src/delivery/index.tsx'),
-    ]);
-    if (hasDelivery) {
-      entryPoints['delivery/index'] = 'src/delivery/index.ts';
-    }
-
-    if (Object.keys(entryPoints).length === 0) {
-      return; // No entry points, skip vite config
-    }
-
-    // Format entry points for vite config
-    const entryLines = Object.entries(entryPoints)
-      .map(([key, value]) => `        '${key}': resolve(__dirname, '${value}'),`)
-      .join('\n');
-
-    const viteConfig = `import { resolve } from 'node:path';
-import react from '@vitejs/plugin-react';
-import { defineConfig } from 'vite';
-
-export default defineConfig({
-  plugins: [react()],
-  build: {
-    lib: {
-      entry: {
-${entryLines}
-      },
-      formats: ['es'],
-    },
-    rollupOptions: {
-      external: ${createExternalFunction('pielib')},
-      output: {
-        preserveModules: true,
-        preserveModulesRoot: 'src',
-      },
-    },
-  },
-});
-`;
-
-    await writeFile(viteConfigPath, viteConfig, 'utf-8');
-  }
-}