@emasoft/svg-matrix 1.0.12 → 1.0.14

package/bin/svg-matrix.js

@@ -111,6 +111,11 @@ const DEFAULT_CONFIG = {
   resolveMarkers: true,       // Instantiate markers as path geometry
   resolvePatterns: true,      // Expand pattern fills to tiled geometry
   bakeGradients: true,        // Bake gradientTransform into gradient coords
+  // NOTE: Verification is ALWAYS enabled - precision is non-negotiable
+  // E2E verification precision controls
+  clipSegments: 64,           // Polygon samples for clip operations (higher = more precise)
+  bezierArcs: 8,              // Bezier arcs for circles/ellipses (multiple of 4; 8=π/4 optimal)
+  e2eTolerance: '1e-10',      // E2E verification tolerance (tighter with more segments)
 };
 
 /** @type {CLIConfig} */
@@ -538,6 +543,22 @@ ${colors.bright}FLATTEN OPTIONS:${colors.reset}
   --no-patterns           Skip pattern expansion
   --no-gradients          Skip gradient transform baking
 
+${colors.bright}E2E VERIFICATION OPTIONS:${colors.reset}
+  --clip-segments <n>     Polygon samples for clipping (default: 64)
+                          Higher = better curve approximation, tighter tolerance
+                          Recommended: 64 (balanced), 128 (high), 256 (very high)
+  --bezier-arcs <n>       Bezier arcs for circles/ellipses (default: 8)
+                          Must be multiple of 4. Multiples of 8 are optimal (π/4).
+                          8: ~0.0004% error (π/4 optimal base)
+                          16: ~0.000007% error (high precision)
+                          32: ~0.0000004% error, 64: ~0.00000001% error
+  --e2e-tolerance <exp>   E2E verification tolerance exponent (default: 1e-10)
+                          Examples: 1e-8, 1e-10, 1e-12, 1e-14
+                          Tighter tolerance requires more clip-segments
+
+  ${colors.dim}Note: Mathematical verification is ALWAYS enabled.${colors.reset}
+  ${colors.dim}Precision is non-negotiable in this library.${colors.reset}
+
 ${colors.bright}EXAMPLES:${colors.reset}
   svg-matrix flatten input.svg -o output.svg
   svg-matrix flatten ./svgs/ -o ./output/ --transform-only
@@ -626,6 +647,9 @@ function processFlatten(inputPath, outputPath) {
     const pipelineOptions = {
       precision: config.precision,
       curveSegments: 20,
+      clipSegments: config.clipSegments,     // Higher segments for clip accuracy (default 64)
+      bezierArcs: config.bezierArcs,         // Bezier arcs for circles/ellipses (default 16)
+      e2eTolerance: config.e2eTolerance,     // Configurable E2E tolerance (default 1e-10)
       resolveUse: config.resolveUse,
       resolveMarkers: config.resolveMarkers,
       resolvePatterns: config.resolvePatterns,
@@ -634,6 +658,7 @@ function processFlatten(inputPath, outputPath) {
       flattenTransforms: true, // Always flatten transforms
       bakeGradients: config.bakeGradients,
       removeUnusedDefs: true,
+      // NOTE: Verification is ALWAYS enabled - precision is non-negotiable
     };
 
     // Run the full flatten pipeline
@@ -655,6 +680,49 @@ function processFlatten(inputPath, outputPath) {
       logInfo('No transform dependencies found');
     }
 
+    // Report verification results (ALWAYS - precision is non-negotiable)
+    if (stats.verifications) {
+      const v = stats.verifications;
+      const total = v.passed + v.failed;
+      if (total > 0) {
+        const verifyStatus = v.allPassed
+          ? `${colors.green}VERIFIED${colors.reset}`
+          : `${colors.red}${v.failed} FAILED${colors.reset}`;
+        logInfo(`Verification: ${v.passed}/${total} - ${verifyStatus}`);
+
+        // Show detailed results in verbose mode
+        if (config.verbose) {
+          if (v.matrices.length > 0) {
+            logDebug(`  Matrix verifications: ${v.matrices.filter(m => m.valid).length}/${v.matrices.length} passed`);
+          }
+          if (v.transforms.length > 0) {
+            logDebug(`  Transform round-trips: ${v.transforms.filter(t => t.valid).length}/${v.transforms.length} passed`);
+          }
+          if (v.polygons.length > 0) {
+            logDebug(`  Polygon intersections: ${v.polygons.filter(p => p.valid).length}/${v.polygons.length} passed`);
+          }
+          if (v.gradients.length > 0) {
+            logDebug(`  Gradient transforms: ${v.gradients.filter(g => g.valid).length}/${v.gradients.length} passed`);
+          }
+          if (v.e2e && v.e2e.length > 0) {
+            logDebug(`  E2E area conservation: ${v.e2e.filter(e => e.valid).length}/${v.e2e.length} passed`);
+          }
+        }
+
+        // Always show failed verifications (not just in verbose mode)
+        const allVerifications = [...v.matrices, ...v.transforms, ...v.polygons, ...v.gradients, ...(v.e2e || [])];
+        const failed = allVerifications.filter(vr => !vr.valid);
+        if (failed.length > 0) {
+          for (const f of failed.slice(0, 3)) {
+            logError(`${colors.red}VERIFICATION FAILED:${colors.reset} ${f.message}`);
+          }
+          if (failed.length > 3) {
+            logError(`...and ${failed.length - 3} more failed verifications`);
+          }
+        }
+      }
+    }
+
     // Report any errors
     if (stats.errors.length > 0) {
       for (const err of stats.errors.slice(0, 5)) {
@@ -944,6 +1012,35 @@ function parseArgs(args) {
       case '--no-markers': cfg.resolveMarkers = false; break;
       case '--no-patterns': cfg.resolvePatterns = false; break;
       case '--no-gradients': cfg.bakeGradients = false; break;
+      // E2E verification precision options
+      case '--clip-segments': {
+        const segs = parseInt(args[++i], 10);
+        if (isNaN(segs) || segs < 8 || segs > 512) {
+          logError('clip-segments must be between 8 and 512');
+          process.exit(CONSTANTS.EXIT_ERROR);
+        }
+        cfg.clipSegments = segs;
+        break;
+      }
+      case '--bezier-arcs': {
+        const arcs = parseInt(args[++i], 10);
+        if (isNaN(arcs) || arcs < 4 || arcs > 128) {
+          logError('bezier-arcs must be between 4 and 128');
+          process.exit(CONSTANTS.EXIT_ERROR);
+        }
+        cfg.bezierArcs = arcs;
+        break;
+      }
+      case '--e2e-tolerance': {
+        const tol = args[++i];
+        if (!/^1e-\d+$/.test(tol)) {
+          logError('e2e-tolerance must be in format 1e-N (e.g., 1e-10, 1e-12)');
+          process.exit(CONSTANTS.EXIT_ERROR);
+        }
+        cfg.e2eTolerance = tol;
+        break;
+      }
+      // NOTE: --verify removed - verification is ALWAYS enabled
       default:
         if (arg.startsWith('-')) { logError(`Unknown option: ${arg}`); process.exit(CONSTANTS.EXIT_ERROR); }
         if (['flatten', 'convert', 'normalize', 'info', 'help', 'version'].includes(arg) && cfg.command === 'help') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@emasoft/svg-matrix",
-  "version": "1.0.12",
+  "version": "1.0.14",
   "description": "Arbitrary-precision matrix, vector and affine transformation library for JavaScript using decimal.js",
   "type": "module",
   "main": "src/index.js",

package/scripts/postinstall.js

@@ -1,15 +1,7 @@
 #!/usr/bin/env node
 /**
  * @fileoverview Post-install welcome message for @emasoft/svg-matrix
- * Displays a compact summary of available features after npm install.
- *
- * Design principles:
- * - Fail silently: npm install must never fail because of postinstall.
- *   Any error is caught and swallowed with exit(0).
- * - Be respectful of terminal real estate: Keep the message compact.
- *   Users want to continue working, not read documentation.
- * - Cross-platform: Handle Windows legacy terminals that don't support ANSI.
- * - Standards-compliant: Respect NO_COLOR environment variable.
+ * Displays a summary of CLI commands and API functions after npm install.
  *
  * @module scripts/postinstall
  * @license MIT
@@ -19,234 +11,124 @@ import { readFileSync } from 'fs';
 import { fileURLToPath } from 'url';
 import { dirname, join } from 'path';
 
-// ============================================================================
-// VERSION DETECTION
-// ============================================================================
-/**
- * Read version from package.json dynamically.
- *
- * Why dynamic reading instead of hardcoding:
- * - Avoids version mismatch bugs where postinstall shows wrong version.
- * - Single source of truth in package.json.
- *
- * Why wrapped in try-catch:
- * - postinstall scripts must NEVER fail - this would break npm install.
- * - Return 'unknown' gracefully if file read fails for any reason.
- *
- * @returns {string} Version string or 'unknown' on error
- */
 function getVersion() {
   try {
-    // Why: ESM doesn't have __dirname, must derive from import.meta.url
     const __dirname = dirname(fileURLToPath(import.meta.url));
-    // Why: Go up one level because this script is in scripts/ subfolder
     const pkg = JSON.parse(readFileSync(join(__dirname, '..', 'package.json'), 'utf8'));
     return pkg.version || 'unknown';
   } catch {
-    // Why: Return safe default - never throw from postinstall
     return 'unknown';
   }
 }
 
-// ============================================================================
-// COLOR SUPPORT DETECTION
-// ============================================================================
-/**
- * Check if colors should be disabled.
- *
- * Why this function:
- * - Not all terminals support ANSI escape codes.
- * - Showing raw escape codes is worse than no colors.
- * - NO_COLOR is an emerging standard: https://no-color.org
- *
- * @returns {boolean} True if colors should be disabled
- */
 function shouldDisableColors() {
-  // Why: NO_COLOR standard - presence (any value) means disable colors
-  // Check for presence, not truthiness, per spec
-  if (process.env.NO_COLOR !== undefined) {
-    return true;
-  }
-
-  // Why: Windows cmd.exe (not PowerShell/Terminal) doesn't support ANSI by default
-  // Check for known ANSI-capable Windows terminals
+  if (process.env.NO_COLOR !== undefined) return true;
   if (process.platform === 'win32') {
-    const supportsAnsi =
-      process.env.WT_SESSION ||           // Windows Terminal sets this
-      process.env.ConEmuANSI === 'ON' ||  // ConEmu explicitly signals support
-      process.env.TERM_PROGRAM ||         // VS Code, Hyper, etc. set this
-      process.env.ANSICON;                // ANSICON utility for legacy cmd
-    // Why: Disable colors unless we detect ANSI support
-    return !supportsAnsi;
+    return !(process.env.WT_SESSION || process.env.ConEmuANSI === 'ON' ||
+             process.env.TERM_PROGRAM || process.env.ANSICON);
   }
-
-  // Why: Unix terminals generally support ANSI, so enable by default
   return false;
 }
 
-/**
- * ANSI color codes for terminal output.
- *
- * Why function instead of constant:
- * - Need to check color support at runtime, not module load time.
- * - Environment variables may change between import and execution.
- *
- * @param {boolean} disabled - Whether colors are disabled
- * @returns {Object} Color code object
- */
 function getColors(disabled) {
-  // Why: Return empty strings instead of undefined to avoid string concat issues
   if (disabled) {
-    return {
-      reset: '', bright: '', dim: '',
-      cyan: '', green: '', yellow: '',
-      magenta: '', blue: '', white: '',
-    };
+    return { reset: '', bright: '', dim: '', cyan: '', green: '',
+             yellow: '', magenta: '', blue: '', white: '' };
   }
-  // Why: Use standard ANSI SGR (Select Graphic Rendition) codes
-  // Format: ESC[<code>m where ESC is \x1b
   return {
-    reset: '\x1b[0m',      // Reset all attributes
-    bright: '\x1b[1m',     // Bold/bright
-    dim: '\x1b[2m',        // Dim/faint
-    cyan: '\x1b[36m',      // Cyan foreground
-    green: '\x1b[32m',     // Green foreground
-    yellow: '\x1b[33m',    // Yellow foreground
-    magenta: '\x1b[35m',   // Magenta foreground
-    blue: '\x1b[34m',      // Blue foreground
-    white: '\x1b[37m',     // White foreground
+    reset: '\x1b[0m', bright: '\x1b[1m', dim: '\x1b[2m',
+    cyan: '\x1b[36m', green: '\x1b[32m', yellow: '\x1b[33m',
+    magenta: '\x1b[35m', blue: '\x1b[34m', white: '\x1b[37m',
   };
 }
 
-// ============================================================================
-// UNICODE SUPPORT DETECTION
-// ============================================================================
-/**
- * Check if terminal likely supports Unicode box drawing characters.
- *
- * Why this check:
- * - Some older terminals or SSH sessions may not render Unicode correctly.
- * - Windows cmd.exe before Windows 10 has limited Unicode support.
- * - Better to show ASCII fallback than broken box characters.
- *
- * @returns {boolean} True if Unicode is likely supported
- */
 function supportsUnicode() {
-  // Why: Check common environment indicators for Unicode support
-
-  // Windows legacy cmd.exe typically has codepage issues
   if (process.platform === 'win32') {
-    // Modern Windows Terminal supports Unicode
-    if (process.env.WT_SESSION) return true;
-    // Check for UTF-8 codepage
-    if (process.env.CHCP === '65001') return true;
-    // ConEmu and other modern terminals
-    if (process.env.ConEmuANSI === 'ON') return true;
-    if (process.env.TERM_PROGRAM) return true;
-    // Default to ASCII on Windows for safety
-    return false;
+    return !!(process.env.WT_SESSION || process.env.CHCP === '65001' ||
+              process.env.ConEmuANSI === 'ON' || process.env.TERM_PROGRAM);
   }
-
-  // Why: Unix terminals generally support Unicode if LANG/LC_CTYPE mentions UTF-8
-  const lang = process.env.LANG || process.env.LC_CTYPE || '';
-  if (lang.toLowerCase().includes('utf')) return true;
-
-  // Why: Modern terminal emulators typically support Unicode
-  if (process.env.TERM_PROGRAM) return true;
-
-  // Default to Unicode on Unix (most modern systems support it)
   return true;
 }
 
-// ============================================================================
-// CI DETECTION
-// ============================================================================
-/**
- * Check if running in CI environment.
- *
- * Why skip in CI:
- * - CI logs should be clean and focused on build/test output.
- * - Welcome messages just add noise to CI logs.
- * - CI systems can set CI=true to suppress this and similar messages.
- *
- * @returns {boolean} True if in CI
- */
 function isCI() {
-  // Why: Check multiple CI indicators since there's no universal standard
-  // Each CI system sets different environment variables
-  return !!(
-    process.env.CI ||                    // GitHub Actions, GitLab CI, CircleCI
-    process.env.CONTINUOUS_INTEGRATION || // Travis CI
-    process.env.GITHUB_ACTIONS ||        // GitHub Actions (redundant with CI but explicit)
-    process.env.GITLAB_CI ||             // GitLab CI
-    process.env.CIRCLECI ||              // CircleCI
-    process.env.TRAVIS ||                // Travis CI
-    process.env.JENKINS_URL ||           // Jenkins
-    process.env.BUILD_ID                 // Various CI systems
-  );
+  return !!(process.env.CI || process.env.CONTINUOUS_INTEGRATION ||
+            process.env.GITHUB_ACTIONS || process.env.GITLAB_CI ||
+            process.env.CIRCLECI || process.env.TRAVIS ||
+            process.env.JENKINS_URL || process.env.BUILD_ID);
 }
 
-/**
- * Display the welcome message.
- * Main entry point for the postinstall script.
- *
- * Why this design:
- * - Skip silently in CI: CI environments don't need welcome messages, and
- *   they clutter build logs. Most CI systems set CI=true or similar.
- * - Skip in non-TTY: If stdout isn't a terminal (e.g., piped to file),
- *   ANSI codes would be visible as garbage characters.
- * - Compact message: Users just installed the package and want to continue
- *   their work. A brief message with the essentials respects their time.
- *   Full documentation belongs in README, not terminal output.
- */
-function showWelcome() {
-  // Why: CI builds don't need welcome messages, they just add noise to logs
-  if (isCI()) {
-    process.exit(0);
-  }
-
-  // Why: Non-TTY means output is being piped/redirected - ANSI codes would be garbage
-  if (!process.stdout.isTTY) {
-    process.exit(0);
-  }
-
-  const version = getVersion();
-  const colorsDisabled = shouldDisableColors();
-  const c = getColors(colorsDisabled);
-  const unicode = supportsUnicode();
-
-  // Why: Use ASCII box characters as fallback for terminals without Unicode
-  const box = unicode ? {
-    tl: '╭', tr: '╮', bl: '╰', br: '╯', h: '─', v: '│'
-  } : {
-    tl: '+', tr: '+', bl: '+', br: '+', h: '-', v: '|'
-  };
+// Strip ANSI escape codes for accurate length calculation
+function stripAnsi(s) {
+  return s.replace(/\x1b\[[0-9;]*m/g, '');
+}
 
-  // Why: Keep message compact - users want to continue their work, not read a manual.
-  // Link to docs for those who want more. Use box drawing for visual structure.
-  const message = `
-${c.cyan}${box.tl}${box.h.repeat(45)}${box.tr}${c.reset}
-${c.cyan}${box.v}${c.reset} ${c.bright}@emasoft/svg-matrix${c.reset} v${version}              ${c.cyan}${box.v}${c.reset}
-${c.cyan}${box.v}${c.reset} High-precision SVG matrix transforms       ${c.cyan}${box.v}${c.reset}
-${c.cyan}${box.bl}${box.h.repeat(45)}${box.br}${c.reset}
+// Pad string to width W (accounting for ANSI codes)
+function pad(s, w) {
+  const visible = stripAnsi(s).length;
+  return s + ' '.repeat(Math.max(0, w - visible));
+}
 
-${c.green}CLI:${c.reset}  npx svg-matrix flatten input.svg -o out.svg
-${c.green}API:${c.reset}  import { Matrix, Transforms2D } from '@emasoft/svg-matrix'
-${c.green}Docs:${c.reset} ${c.blue}https://github.com/Emasoft/SVG-MATRIX#readme${c.reset}
-`;
+function showWelcome() {
+  if (isCI()) process.exit(0);
+  if (!process.stdout.isTTY) process.exit(0);
 
-  console.log(message);
+  const version = getVersion();
+  const c = getColors(shouldDisableColors());
+  const u = supportsUnicode();
+
+  const B = u
+    ? { tl: '╭', tr: '╮', bl: '╰', br: '╯', h: '─', v: '│', dot: '•' }
+    : { tl: '+', tr: '+', bl: '+', br: '+', h: '-', v: '|', dot: '*' };
+
+  const W = 66;
+  const hr = B.h.repeat(W);
+  const R = (s) => pad(s, W);
+
+  console.log(`
+${c.cyan}${B.tl}${hr}${B.tr}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.bright}@emasoft/svg-matrix${c.reset} v${version}`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.dim}Arbitrary-precision SVG transforms with decimal.js${c.reset}`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${hr}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.yellow}CLI Commands:${c.reset}`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.green}svg-matrix flatten${c.reset} input.svg -o output.svg`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`    ${c.dim}Bake all transforms into path coordinates${c.reset}`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.green}Options:${c.reset}`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`    ${c.dim}--precision N${c.reset}       Output decimal places (default: 6)`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`    ${c.dim}--clip-segments N${c.reset}   Polygon sampling for clips (default: 64)`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`    ${c.dim}--bezier-arcs N${c.reset}     Bezier arcs for curves (default: 8)`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`    ${c.dim}--e2e-tolerance N${c.reset}   Verification tolerance (default: 1e-10)`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`    ${c.dim}--verbose${c.reset}           Show processing details`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${hr}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.yellow}JavaScript API:${c.reset}`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.magenta}import${c.reset} { Matrix, Vector, Transforms2D } ${c.magenta}from${c.reset} '@emasoft/svg-matrix'`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.green}${B.dot} Matrix${c.reset}        Arbitrary-precision matrix operations`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.green}${B.dot} Vector${c.reset}        High-precision vector math`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.green}${B.dot} Transforms2D${c.reset}  rotate, scale, translate, skew, reflect`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.green}${B.dot} Transforms3D${c.reset}  3D affine transformations`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${hr}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.yellow}New in v1.0.13:${c.reset}`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.green}${B.dot}${c.reset} High-precision Bezier circle/ellipse approximation`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.green}${B.dot}${c.reset} E2E verification for clip-path operations`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R(`  ${c.green}${B.dot}${c.reset} Configurable: --clip-segments, --bezier-arcs, --e2e-tolerance`)}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.v}${c.reset}${R('')}${c.cyan}${B.v}${c.reset}
+${c.cyan}${B.bl}${hr}${B.br}${c.reset}
+
+  ${c.blue}Docs:${c.reset} https://github.com/Emasoft/SVG-MATRIX#readme
+  ${c.blue}Help:${c.reset} svg-matrix --help
+`);
 }
 
-// ============================================================================
-// MAIN ENTRY
-// ============================================================================
-// Why try-catch: postinstall scripts must NEVER fail or throw.
-// A failed postinstall would break `npm install` for users, which is
-// absolutely unacceptable. Catch everything and exit cleanly.
 try {
   showWelcome();
 } catch {
-  // Why: Silent exit - don't break npm install for a welcome message
   process.exit(0);
 }

package/src/clip-path-resolver.js

@@ -41,6 +41,10 @@ import {
   parseTransformAttribute,
   transformPathData
 } from './svg-flatten.js';
+import {
+  circleToPathDataHP,
+  ellipseToPathDataHP
+} from './geometry-to-path.js';
 import { Logger } from './logger.js';
 
 // Alias for cleaner code
@@ -414,14 +418,32 @@ function removeDuplicateConsecutive(points) {
  * const ctm = Transforms2D.translation(10, 20);
  * const polygon = shapeToPolygon(rect, ctm, 30);
  */
-export function shapeToPolygon(element, ctm = null, samples = DEFAULT_CURVE_SAMPLES) {
+/**
+ * Convert SVG shape to polygon.
+ *
+ * @param {Object} element - Shape element (circle, ellipse, rect, etc.)
+ * @param {Matrix} ctm - Current transform matrix (optional)
+ * @param {number} samples - Samples per curve for polygon conversion
+ * @param {number} bezierArcs - Number of Bezier arcs for circles/ellipses (4=standard, 16 or 64=HP)
+ */
+export function shapeToPolygon(element, ctm = null, samples = DEFAULT_CURVE_SAMPLES, bezierArcs = 4) {
   let pathData;
   switch (element.type) {
     case 'circle':
-      pathData = circleToPath(D(element.cx || 0), D(element.cy || 0), D(element.r || 0));
+      // Use high-precision Bezier arcs for better curve approximation
+      if (bezierArcs > 4) {
+        pathData = circleToPathDataHP(element.cx || 0, element.cy || 0, element.r || 0, bezierArcs, 10);
+      } else {
+        pathData = circleToPath(D(element.cx || 0), D(element.cy || 0), D(element.r || 0));
+      }
       break;
     case 'ellipse':
-      pathData = ellipseToPath(D(element.cx || 0), D(element.cy || 0), D(element.rx || 0), D(element.ry || 0));
+      // Use high-precision Bezier arcs for better curve approximation
+      if (bezierArcs > 4) {
+        pathData = ellipseToPathDataHP(element.cx || 0, element.cy || 0, element.rx || 0, element.ry || 0, bezierArcs, 10);
+      } else {
+        pathData = ellipseToPath(D(element.cx || 0), D(element.cy || 0), D(element.rx || 0), D(element.ry || 0));
+      }
       break;
     case 'rect':
       pathData = rectToPath(D(element.x || 0), D(element.y || 0), D(element.width || 0), D(element.height || 0),