@holoscript/std 3.1.1 → 6.0.3

package/src/__tests__/{Sprint56.test.ts → string-and-time-utilities.test.ts}

@@ -1,30 +1,30 @@
-/**
- * Sprint 56 — @holoscript/std acceptance tests
+/**
+ * @holoscript/std acceptance tests
  * Covers: string formatting/validation/escaping, time utilities (Stopwatch,
  *         CountdownTimer, debounce, throttle, retry, measure), and
  *         collections (HoloMap, HoloSet, PriorityQueue).
  */
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import {
-  // String — formatting
+  // String — formatting
   format,
   formatBytes,
   formatDuration,
   formatNumber,
   numberWithCommas,
-  // String — HTML/regex
+  // String — HTML/regex
   escapeHtml,
   unescapeHtml,
   escapeRegex,
   slugify,
-  // String — validation
+  // String — validation
   isValidIdentifier,
   isNumeric,
   isAlphanumeric,
   isAlpha,
   isBlank,
   isNotBlank,
-  // String — manipulation
+  // String — manipulation
   truncate,
   truncateMiddle,
   center,
@@ -57,9 +57,9 @@ import {
 
 import { HoloMap, HoloSet, PriorityQueue, List } from '../collections.js';
 
-// ═══════════════════════════════════════════════
-// String — formatting
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// String — formatting
+// ═══════════════════════════════════════════════
 describe('format', () => {
   it('replaces named placeholders', () => {
     expect(format('Hello {name}!', { name: 'Alice' })).toBe('Hello Alice!');
@@ -113,12 +113,12 @@ describe('formatDuration', () => {
     expect(formatDuration(2500)).toBe('2.5s');
   });
 
-  it('minutes and seconds', () => {
-    expect(formatDuration(90000)).toBe('1m 30s');
+  it('minutes', () => {
+    expect(formatDuration(90000)).toBe('1.5m');
   });
 
-  it('hours and minutes', () => {
-    expect(formatDuration(3900000)).toBe('1h 5m');
+  it('hours', () => {
+    expect(formatDuration(3900000)).toBe('1.1h');
   });
 });
 
@@ -132,9 +132,9 @@ describe('numberWithCommas', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// String — HTML / regex escaping
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// String — HTML / regex escaping
+// ═══════════════════════════════════════════════
 describe('escapeHtml', () => {
   it('escapes ampersand', () => {
     expect(escapeHtml('a & b')).toBe('a & b');
@@ -211,9 +211,9 @@ describe('slugify', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// String — validation
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// String — validation
+// ═══════════════════════════════════════════════
 describe('isValidIdentifier', () => {
   it('accepts simple identifier', () => {
     expect(isValidIdentifier('myVar')).toBe(true);
@@ -275,9 +275,9 @@ describe('isAlphanumeric / isAlpha / isBlank / isNotBlank', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// String — manipulation
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// String — manipulation
+// ═══════════════════════════════════════════════
 describe('truncate', () => {
   it('does not truncate short strings', () => {
     expect(truncate('hi', 10)).toBe('hi');
@@ -288,8 +288,8 @@ describe('truncate', () => {
   });
 
   it('custom ellipsis', () => {
-    // maxLength=7, ellipsis='…' (1 char) → slices 6 chars → 'hello …'
-    expect(truncate('hello world', 7, '…')).toBe('hello …');
+    // Use ASCII marker to avoid Unicode encoding variance in this fixture.
+    expect(truncate('hello world', 7, '*')).toBe('hello *');
   });
 });
 
@@ -417,9 +417,9 @@ describe('randomString', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Time — measure
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Time — measure
+// ═══════════════════════════════════════════════
 describe('measure', () => {
   it('returns result and duration', async () => {
     const { result, duration } = await measure(() => 42);
@@ -434,9 +434,9 @@ describe('measure', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Time — retry
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Time — retry
+// ═══════════════════════════════════════════════
 describe('retry', () => {
   it('returns immediately on first success', async () => {
     let calls = 0;
@@ -490,9 +490,9 @@ describe('retry', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Time — debounce
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Time — debounce
+// ═══════════════════════════════════════════════
 describe('debounce', () => {
   it('does not call immediately', () => {
     vi.useFakeTimers();
@@ -528,9 +528,9 @@ describe('debounce', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Time — throttle
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Time — throttle
+// ═══════════════════════════════════════════════
 describe('throttle', () => {
   it('calls function on first invocation', () => {
     vi.useFakeTimers();
@@ -564,9 +564,9 @@ describe('throttle', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Time — Stopwatch
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Time — Stopwatch
+// ═══════════════════════════════════════════════
 describe('Stopwatch', () => {
   it('starts not running', () => {
     const sw = new Stopwatch();
@@ -628,15 +628,15 @@ describe('Stopwatch', () => {
     const lapTime = sw.lap();
     expect(typeof lapTime).toBe('number');
     expect(lapTime).toBeGreaterThan(0);
-    // After lap(), stopwatch restarted — elapsed should be less than before
+    // After lap(), stopwatch restarted — elapsed should be less than before
     expect(sw.elapsed).toBeLessThan(lapTime);
     sw.stop();
   });
 });
 
-// ═══════════════════════════════════════════════
-// Time — CountdownTimer
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Time — CountdownTimer
+// ═══════════════════════════════════════════════
 describe('CountdownTimer', () => {
   it('starts not running', () => {
     const ct = new CountdownTimer(1000);
@@ -684,9 +684,9 @@ describe('CountdownTimer', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Time — IntervalTimer
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Time — IntervalTimer
+// ═══════════════════════════════════════════════
 describe('IntervalTimer', () => {
   it('starts not running', () => {
     const it = new IntervalTimer(() => {}, 1000);
@@ -726,9 +726,9 @@ describe('IntervalTimer', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Time — schedule
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Time — schedule
+// ═══════════════════════════════════════════════
 describe('schedule', () => {
   it('fires callback after delay', async () => {
     const fn = vi.fn();
@@ -746,9 +746,9 @@ describe('schedule', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Time — DateTime
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Time — DateTime
+// ═══════════════════════════════════════════════
 describe('DateTime', () => {
   it('toISO returns ISO string', () => {
     const iso = DateTime.toISO(new Date(2024, 0, 15));
@@ -784,9 +784,9 @@ describe('DateTime', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Collections — HoloMap
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Collections — HoloMap
+// ═══════════════════════════════════════════════
 describe('HoloMap', () => {
   it('creates empty map', () => {
     const m = HoloMap.of<string, number>();
@@ -879,9 +879,9 @@ describe('HoloMap', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Collections — HoloSet
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Collections — HoloSet
+// ═══════════════════════════════════════════════
 describe('HoloSet', () => {
   it('creates empty set', () => {
     const s = HoloSet.of<number>();
@@ -951,9 +951,9 @@ describe('HoloSet', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Collections — PriorityQueue
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Collections — PriorityQueue
+// ═══════════════════════════════════════════════
 describe('PriorityQueue', () => {
   it('creates empty queue', () => {
     const pq = new PriorityQueue<number>();
@@ -1008,9 +1008,9 @@ describe('PriorityQueue', () => {
   });
 });
 
-// ═══════════════════════════════════════════════
-// Collections — List (extended)
-// ═══════════════════════════════════════════════
+// ═══════════════════════════════════════════════
+// Collections — List (extended)
+// ═══════════════════════════════════════════════
 describe('List (extended)', () => {
   it('range creates sequential list (exclusive end)', () => {
     const l = List.range(1, 6);

package/src/__tests__/string-decoupled.test.ts

@@ -6,11 +6,9 @@ import { describe, it, expect } from 'vitest';
 describe('string-decoupled', () => {
   it('should be defined', () => {
     // TODO: Import and test string-decoupled
-    expect(true).toBe(true);
   });
 
   it('should have basic functionality', () => {
     // TODO: Add meaningful tests for string-decoupled
-    expect(true).toBe(true);
   });
 });

package/src/string.basic.test.ts

@@ -11,4 +11,4 @@ describe('formatBytes basic test', () => {
     expect(formatBytes(1024)).toBe('1 KB');
     expect(formatBytes(1048576)).toBe('1 MB');
   });
-});
+});

package/src/string.test.ts

@@ -54,7 +54,7 @@ import {
   wordWrap,
   levenshtein,
   similarity,
-  extractTraits
+  extractTraits,
 } from './string.js';
 
 describe('@holoscript/std string utilities', () => {
@@ -187,7 +187,7 @@ describe('@holoscript/std string utilities', () => {
         expect(isBlank('   ')).toBe(true);
         expect(isBlank('\t\n')).toBe(true);
         expect(isBlank('test')).toBe(false);
-        
+
         expect(isNotBlank('')).toBe(false);
         expect(isNotBlank('   ')).toBe(false);
         expect(isNotBlank('test')).toBe(true);
@@ -287,7 +287,7 @@ describe('@holoscript/std string utilities', () => {
       expect(str1.length).toBe(10);
       expect(str2.length).toBe(10);
       expect(str1).not.toBe(str2); // Should be different
-      
+
       const customStr = randomString(5, 'ABC');
       expect(customStr.length).toBe(5);
       expect(customStr).toMatch(/^[ABC]+$/);
@@ -332,4 +332,4 @@ describe('@holoscript/std string utilities', () => {
       expect(traits).toEqual(['Entity', 'Cube']);
     });
   });
-});
+});

package/src/string.ts

@@ -116,8 +116,8 @@ export function truncate(s: string, maxLength: number, ellipsis = '...'): string
 export function truncateMiddle(s: string, maxLength: number, ellipsis = '...'): string {
   if (s.length <= maxLength) return s;
   const availableLength = maxLength - ellipsis.length;
-  const leftLength = Math.ceil(availableLength / 2);
-  const rightLength = Math.floor(availableLength / 2);
+  const leftLength = Math.floor(availableLength / 2);
+  const rightLength = Math.ceil(availableLength / 2);
   return s.slice(0, leftLength) + ellipsis + s.slice(-rightLength);
 }
 
@@ -144,7 +144,7 @@ export function count(s: string, substring: string): number {
   let pos = 0;
   while ((pos = s.indexOf(substring, pos)) !== -1) {
     count++;
-    pos += substring.length;
+    pos += 1;
   }
   return count;
 }
@@ -281,6 +281,7 @@ export function numberWithCommas(n: number): string {
 export function formatBytes(bytes: number, decimals = 2): string {
   if (bytes === 0) return '0 B';
   const k = 1024;
+  if (bytes < 0) return `${bytes} B`;
   const sizes = ['B', 'KB', 'MB', 'GB', 'TB', 'PB'];
   const i = Math.floor(Math.log(bytes) / Math.log(k));
   return parseFloat((bytes / Math.pow(k, i)).toFixed(decimals)) + ' ' + sizes[i];
@@ -293,13 +294,11 @@ export function formatDuration(ms: number): string {
   if (ms < 1000) return `${ms}ms`;
   if (ms < 60000) return `${(ms / 1000).toFixed(1)}s`;
   if (ms < 3600000) {
-    const mins = Math.floor(ms / 60000);
-    const secs = Math.floor((ms % 60000) / 1000);
-    return `${mins}m ${secs}s`;
+    return `${(ms / 60000).toFixed(1)}m`;
   }
   const hours = Math.floor(ms / 3600000);
   const mins = Math.floor((ms % 3600000) / 60000);
-  return `${hours}h ${mins}m`;
+  return `${(ms / 3600000).toFixed(1)}h`;
 }
 
 /**
@@ -495,22 +494,38 @@ export function levenshtein(a: string, b: string): number {
   return matrix[b.length][a.length];
 }
 
+function lcsLength(a: string, b: string): number {
+  const n = b.length;
+  const dp: number[] = new Array(n + 1).fill(0);
+  for (let i = 0; i < a.length; i++) {
+    let prev = 0;
+    for (let j = 0; j < n; j++) {
+      const tmp = dp[j + 1];
+      dp[j + 1] = a[i] === b[j] ? prev + 1 : Math.max(dp[j + 1], dp[j]);
+      prev = tmp;
+    }
+  }
+  return dp[n];
+}
+
 /**
- * Check similarity between two strings (0-1)
+ * Check similarity between two strings (0-1) using Sorensen-Dice over LCS
  */
 export function similarity(a: string, b: string): number {
-  const maxLen = Math.max(a.length, b.length);
-  if (maxLen === 0) return 1;
-  return 1 - levenshtein(a, b) / maxLen;
+  const total = a.length + b.length;
+  if (total === 0) return 1;
+  return (2 * lcsLength(a, b)) / total;
 }
 
 /**
- * Extract HoloScript traits (e.g., @grabbable) from a code string
+ * Extract HoloScript trait annotations (@name) and JSX component names from code
  */
 export function extractTraits(code: string): string[] {
-  const re = /@([a-zA-Z_]\w*)/g;
   const traits = new Set<string>();
-  let m;
-  while ((m = re.exec(code)) !== null) traits.add(`@${m[1]}`);
+  let m: RegExpExecArray | null;
+  const atRe = /@([a-zA-Z_]\w*)/g;
+  while ((m = atRe.exec(code)) !== null) traits.add(`@${m[1]}`);
+  const tagRe = /<([A-Z][a-zA-Z0-9]*)\b/g;
+  while ((m = tagRe.exec(code)) !== null) traits.add(m[1]);
   return [...traits];
 }

package/src/time.ts

@@ -151,7 +151,7 @@ export async function timed<T>(label: string, fn: () => T | Promise<T>): Promise
     return await fn();
   } finally {
     const duration = hrTime() - start;
-    console.log(`[${label}] ${duration.toFixed(2)}ms`);
+    void duration; // timing available for callers via return value
   }
 }
 

package/src/traits/__tests__/ARTraits.test.ts

@@ -0,0 +1,14 @@
+import { describe, it, expect } from 'vitest';
+
+// Auto-generated test for ARTraits
+// Source: packages/std/src/traits/ARTraits.ts
+
+describe('ARTraits', () => {
+  it('should be defined', () => {
+    // TODO: Import and test ARTraits
+  });
+
+  it('should have basic functionality', () => {
+    // TODO: Add meaningful tests for ARTraits
+  });
+});

package/src/traits/__tests__/EconomicPrimitives.test.ts

@@ -0,0 +1,14 @@
+import { describe, it, expect } from 'vitest';
+
+// Auto-generated test for EconomicPrimitives
+// Source: packages/std/src/traits/EconomicPrimitives.ts
+
+describe('EconomicPrimitives', () => {
+  it('should be defined', () => {
+    // TODO: Import and test EconomicPrimitives
+  });
+
+  it('should have basic functionality', () => {
+    // TODO: Add meaningful tests for EconomicPrimitives
+  });
+});

package/src/traits/__tests__/EconomicTraits.test.ts

@@ -0,0 +1,14 @@
+import { describe, it, expect } from 'vitest';
+
+// Auto-generated test for EconomicTraits
+// Source: packages/std/src/traits/EconomicTraits.ts
+
+describe('EconomicTraits', () => {
+  it('should be defined', () => {
+    // TODO: Import and test EconomicTraits
+  });
+
+  it('should have basic functionality', () => {
+    // TODO: Add meaningful tests for EconomicTraits
+  });
+});

package/std_error.log

@@ -0,0 +1,5 @@
+npm warn Unknown project config "shamefully-hoist". This will stop working in the next major version of npm.
+npm warn Unknown project config "strict-peer-dependencies". This will stop working in the next major version of npm.
+src/__tests__/SimulationLabPrimitives.test.ts(345,12): error TS18048: 'trait.params' is possibly 'undefined'.
+src/__tests__/SimulationLabPrimitives.test.ts(346,12): error TS18048: 'trait.params' is possibly 'undefined'.
+src/physics.ts(11,22): error TS2835: Relative import paths need explicit file extensions in ECMAScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Did you mean './spatial.js'?

package/tsup.config.ts

@@ -0,0 +1,17 @@
+import { defineConfig } from 'tsup';
+
+export default defineConfig({
+  entry: [
+    'src/index.ts',
+    'src/math.ts',
+    'src/collections.ts',
+    'src/string.ts',
+    'src/time.ts',
+    'src/traits/EconomicPrimitives.ts',
+    'src/traits/EconomicTraits.ts',
+  ],
+  format: ['esm', 'cjs'],
+  dts: false,
+  clean: true,
+  sourcemap: true,
+});

package/dist/__tests__/Sprint33.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"Sprint33.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/Sprint33.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/Sprint56.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"Sprint56.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/Sprint56.test.ts"],"names":[],"mappings":""}