@1agh/maude 1.0.9 → 1.2.0

package/apps/studio/figma/codegen-fonts.test.ts

@@ -1,103 +0,0 @@
-// figma/codegen-fonts.ts — plan T18.
-//
-// The thing under test is not really the mapping, it is the REPORT. A font that
-// silently falls back looks fine and is not the design, and this import has
-// already shipped three "success" reports over lost content. So: every
-// substitution produces an entry, the entry is BOUNDED (DDR-219 D9), and a
-// family the project genuinely has produces none.
-
-import { describe, expect, test } from 'bun:test';
-
-import {
-  FontSubstitutions,
-  MAX_FAMILY_DETAIL,
-  resolveFontFamily,
-  SYSTEM_STACK,
-  splitFamilyAndStyle,
-  styleToWeight,
-} from './codegen-fonts.ts';
-import { ImportReport } from './sanitize.ts';
-
-const DS = [{ name: '--font-body', value: "'hanken grotesk','inter',sans-serif" }];
-
-describe('splitting Figma’s Family:Style', () => {
-  test.each([
-    ['SF Pro:Bold', 'SF Pro', 'Bold'],
-    ['SF Pro Display:Semibold', 'SF Pro Display', 'Semibold'],
-    ['Inter:Regular', 'Inter', 'Regular'],
-    ['Inter', 'Inter', null],
-  ])('%s', (raw, family, style) => {
-    expect(splitFamilyAndStyle(raw)).toEqual({ family, style });
-  });
-
-  test('the style half is a WEIGHT and must not become part of the family', () => {
-    expect(styleToWeight('Bold')).toBe(700);
-    expect(styleToWeight('Semibold')).toBe(600);
-    expect(styleToWeight('Regular')).toBe(400);
-    expect(styleToWeight('NotAWeight')).toBeNull();
-  });
-});
-
-describe('resolveFontFamily', () => {
-  test('a family the DS already declares resolves to the token, and is NOT a substitution', () => {
-    const r = resolveFontFamily('Inter:Regular', DS);
-    expect(r.css).toBe('var(--font-body)');
-    expect(r.substituted).toBe(false);
-  });
-
-  test('an absent family lands on the DS body token AND reports', () => {
-    // Measured on the dogfood machine: SF Pro is not installed, and the DS loads
-    // no webfont at all — so copying the name through lands on a serif fallback
-    // that looks fine and is not the design.
-    const r = resolveFontFamily('SF Pro:Bold', DS);
-    expect(r.css).toBe('var(--font-body)');
-    expect(r.substituted).toBe(true);
-    expect(r.requested).toBe('SF Pro');
-  });
-
-  test('with no DS at all it lands on a SANS stack, never a serif default', () => {
-    const r = resolveFontFamily('Nunito:Bold', []);
-    expect(r.css).toBe(SYSTEM_STACK);
-    expect(r.substituted).toBe(true);
-  });
-
-  test('the requested family NEVER reaches the artifact verbatim', () => {
-    const hostile = resolveFontFamily("Evil';}\n.x{color:red};:Bold", DS);
-    expect(hostile.css).not.toContain('color:red');
-    expect(hostile.requested).not.toContain(';');
-    expect(hostile.requested.length).toBeLessThanOrEqual(MAX_FAMILY_DETAIL);
-  });
-});
-
-describe('the report', () => {
-  test('one entry per FAMILY with a count — not one per element', () => {
-    const subs = new FontSubstitutions();
-    for (let i = 0; i < 40; i += 1) subs.note(resolveFontFamily('SF Pro:Bold', DS));
-    subs.note(resolveFontFamily('Nunito:Regular', DS));
-    const report = new ImportReport();
-    subs.flush(report, '425:2939');
-
-    // Forty identical entries would bury every other disposition and blow the
-    // summary's 200-line cap for no information.
-    expect(report.entries).toEqual([
-      { nodeId: '425:2939', type: 'FONT', disposition: 'font-substituted', detail: 'Nunito x1' },
-      { nodeId: '425:2939', type: 'FONT', disposition: 'font-substituted', detail: 'SF Pro x40' },
-    ]);
-  });
-
-  test('a family that survived produces NO entry', () => {
-    const subs = new FontSubstitutions();
-    subs.note(resolveFontFamily('Inter:Regular', DS));
-    const report = new ImportReport();
-    subs.flush(report, '1:1');
-    expect(report.entries).toEqual([]);
-  });
-
-  test('a hostile family cannot blow the detail bound — ImportReport would throw', () => {
-    const subs = new FontSubstitutions();
-    subs.note(resolveFontFamily(`${'A'.repeat(400)}:Bold`, DS));
-    const report = new ImportReport();
-    expect(() => subs.flush(report, '1:1')).not.toThrow();
-    expect(report.entries[0].detail?.length).toBeLessThanOrEqual(64);
-  });
-});

package/apps/studio/figma/codegen-values.test.ts

@@ -1,179 +0,0 @@
-// figma/codegen-values.ts — the codegen lane's value grammar.
-//
-// Two things are being defended here, and they pull in opposite directions:
-// a `var()` WITH a fallback must be ADMITTED (it is the fidelity this whole
-// route exists for), and nothing SHARED with the tree translator or DDR-172's
-// token importer may be widened to get it. So the tests assert both the new
-// admissions and that the old refusals still refuse.
-
-import { describe, expect, test } from 'bun:test';
-
-import {
-  cssPropToCamel,
-  isAllowedArbitraryProperty,
-  isCodegenColor,
-  isCodegenKeyword,
-  isCodegenLength,
-  isCodegenLengthList,
-  isCodegenNumber,
-  isCodegenShortValueList,
-  MAX_VALUE_LEN,
-  normalizeCalc,
-  splitVar,
-  unescapeArbitrary,
-} from './codegen-values.ts';
-import { isValidColorValue } from './style-map.ts';
-
-describe('the shared grammar is not widened', () => {
-  test('style-map still refuses a fallback-bearing var() — that is the point', () => {
-    // If this ever passes, someone widened `VAR_RE` instead of composing, which
-    // is verbatim the root pattern the DDR-216 review named twice.
-    expect(isValidColorValue('var(--black,#0f161e)')).toBe(false);
-    expect(isValidColorValue('var(--black)')).toBe(true);
-  });
-
-  test('the lane admits it locally', () => {
-    expect(isCodegenColor('var(--black,#0f161e)')).toBe(true);
-    expect(isCodegenColor('var(--black-10,rgba(15,22,30,0.1))')).toBe(true);
-  });
-});
-
-describe('isCodegenColor', () => {
-  test.each([
-    ['#0f161e', true],
-    ['rgba(15,22,30,0.1)', true],
-    ['var(--accent)', true],
-    ['var(--accent, #fff)', true],
-    ['white', true],
-    ['transparent', true],
-    // The fallback is validated as a colour IN ITS OWN RIGHT, so a URL smuggled
-    // into it is refused by the grammar and not merely by the canvas CSP.
-    ['var(--x,url(https://attacker.example/p))', false],
-    ['url(https://attacker.example/p)', false],
-    ['var(--x);color:red', false],
-    ['expression(alert(1))', false],
-    ['var(--ABC)', false],
-    ['#0f161e /* */', false],
-  ])('%s -> %s', (value, expected) => {
-    expect(isCodegenColor(value as string)).toBe(expected);
-  });
-
-  test('nested var() fallbacks terminate', () => {
-    expect(isCodegenColor('var(--a,var(--b,var(--c,#fff)))')).toBe(true);
-    expect(isCodegenColor('var(--a,var(--b,var(--c,var(--d,var(--e,#fff)))))')).toBe(false);
-  });
-
-  test('a value over the length cap is refused before any grammar runs', () => {
-    expect(isCodegenColor(`#${'0'.repeat(MAX_VALUE_LEN)}`)).toBe(false);
-  });
-});
-
-describe('lengths', () => {
-  test.each([
-    ['375px', true],
-    ['0', true],
-    ['4.17%', true],
-    ['1.5rem', true],
-    ['100vh', true],
-    ['auto', true],
-    ['calc(50% - 32.5px)', true],
-    ['99999999px', false],
-    ['red', false],
-    ['12px;color:red', false],
-    ['calc(50% - 32.5px) !important', false],
-  ])('%s -> %s', (v, expected) => {
-    expect(isCodegenLength(v as string)).toBe(expected);
-  });
-
-  test('a shorthand of up to four lengths', () => {
-    expect(isCodegenLengthList('37.5% 18.75% 26.56% 18.75%')).toBe(true);
-    expect(isCodegenLengthList('0 4.17%')).toBe(true);
-    expect(isCodegenLengthList('1px 2px 3px 4px 5px')).toBe(false);
-  });
-});
-
-describe('normalizeCalc', () => {
-  test('re-spaces the operator Figma omits', () => {
-    // `calc(50%-32.5px)` is INVALID CSS — a browser drops the declaration — so a
-    // pass-through would silently lose the position.
-    expect(normalizeCalc('calc(50%-32.5px)')).toBe('calc(50% - 32.5px)');
-    expect(normalizeCalc('calc(50%-0.02px)')).toBe('calc(50% - 0.02px)');
-  });
-
-  test('REGRESSION: a custom-property name is not an arithmetic expression', () => {
-    // The first version used a lookbehind regex and rewrote this to
-    // `var(--black - 10,…)`, which is a broken declaration on the single most
-    // common value shape this whole route exists to preserve.
-    expect(normalizeCalc('var(--black-10,rgba(15,22,30,0.1))')).toBe(
-      'var(--black-10,rgba(15,22,30,0.1))'
-    );
-    expect(normalizeCalc('calc(var(--gap-2) + 4px)')).toBe('calc(var(--gap-2) + 4px)');
-  });
-
-  test('a leading sign is not an operator', () => {
-    expect(normalizeCalc('calc(-4px)')).toBe('calc(-4px)');
-  });
-
-  test('a value with no calc is returned untouched', () => {
-    expect(normalizeCalc('1 0 0')).toBe('1 0 0');
-  });
-});
-
-describe('unescapeArbitrary', () => {
-  test('an underscore is a space', () => {
-    expect(unescapeArbitrary('37.5%_18.75%')).toBe('37.5% 18.75%');
-    expect(unescapeArbitrary("'SF_Pro:Bold'")).toBe("'SF Pro:Bold'");
-  });
-
-  test('an escaped underscore stays an underscore', () => {
-    expect(unescapeArbitrary('a\\_b_c')).toBe('a_b c');
-  });
-});
-
-describe('splitVar', () => {
-  test('splits at depth 1 only', () => {
-    expect(splitVar('var(--a,rgba(1,2,3,0.5))')).toEqual({
-      name: '--a',
-      fallback: 'rgba(1,2,3,0.5)',
-    });
-    expect(splitVar('var(--a)')).toEqual({ name: '--a', fallback: null });
-    expect(splitVar('#fff')).toBeNull();
-  });
-});
-
-describe('arbitrary properties are an allowlist', () => {
-  test('admits what Figma emits', () => {
-    expect(isAllowedArbitraryProperty('word-break')).toBe(true);
-    expect(isAllowedArbitraryProperty('font-variation-settings')).toBe(true);
-  });
-
-  test('refuses everything else, including things a denylist would forget', () => {
-    for (const prop of ['behavior', '-moz-binding', 'content', 'background-image', 'src']) {
-      expect(isAllowedArbitraryProperty(prop)).toBe(false);
-    }
-  });
-});
-
-describe('misc predicates', () => {
-  test('numbers are magnitude-bounded, not only shape-checked', () => {
-    expect(isCodegenNumber('0.5')).toBe(true);
-    expect(isCodegenNumber('999999999')).toBe(false);
-  });
-
-  test('keywords are short and lowercase', () => {
-    expect(isCodegenKeyword('max-content')).toBe(true);
-    expect(isCodegenKeyword('break-word')).toBe(true);
-    expect(isCodegenKeyword('URL(x)')).toBe(false);
-  });
-
-  test('short value lists', () => {
-    expect(isCodegenShortValueList('1 0 0')).toBe(true);
-    expect(isCodegenShortValueList('max-content')).toBe(true);
-    expect(isCodegenShortValueList('1 2 3 4 5 6 7')).toBe(false);
-  });
-
-  test('camelCase, because a hyphenated key in a JSX style OBJECT is a syntax error', () => {
-    expect(cssPropToCamel('word-break')).toBe('wordBreak');
-    expect(cssPropToCamel('font-variation-settings')).toBe('fontVariationSettings');
-  });
-});

package/apps/studio/figma/comments-to-strokes.test.ts

@@ -1,194 +0,0 @@
-// figma/comments-to-strokes.ts — the review record.
-//
-// Comments live on their own endpoint, nowhere in the document tree, so every
-// tree-walking version of this importer brought across exactly zero of them.
-
-import { describe, expect, test } from 'bun:test';
-
-import { STICKY_PALETTE } from '../annotations-model.ts';
-import type { FigmaComment } from './client.ts';
-import { commentsToStrokes, indexNodes, MAX_COMMENT_STROKES } from './comments-to-strokes.ts';
-import { ImportReport } from './sanitize.ts';
-import { normalizeDocument } from './types.ts';
-
-const KEY = 'dGNzRC2kmrmGnOxaBa0RI7';
-
-function page() {
-  const doc = normalizeDocument(
-    {
-      id: '0:0',
-      name: 'Page',
-      type: 'CANVAS',
-      children: [
-        {
-          id: '1:2',
-          name: 'Screen',
-          type: 'FRAME',
-          visible: true,
-          absoluteBoundingBox: { x: 1000, y: 500, width: 375, height: 812 },
-        },
-      ],
-    },
-    { fileKey: KEY, surface: 'design' }
-  );
-  return doc.root;
-}
-
-const comment = (over: Partial<FigmaComment> = {}): FigmaComment => ({
-  id: 'c1',
-  message: 'Chybí',
-  nodeId: '1:2',
-  x: 20,
-  y: 30,
-  resolved: false,
-  ...over,
-});
-
-const ORIGIN = { x: 1000, y: 500 };
-
-describe('positioning', () => {
-  test('a pin lands at its node position plus its offset, page-normalized', () => {
-    const { strokes } = commentsToStrokes(
-      [comment()],
-      indexNodes(page()),
-      ORIGIN,
-      new ImportReport()
-    );
-    expect(strokes).toHaveLength(1);
-    expect(strokes[0]).toMatchObject({ tool: 'sticky', x: 20, y: 30 });
-  });
-
-  test('a comment pinned to a node on another page is counted, not silently lost', () => {
-    const report = new ImportReport();
-    const { strokes, unplacedIds } = commentsToStrokes(
-      [comment({ nodeId: '9:9' })],
-      indexNodes(page()),
-      ORIGIN,
-      report
-    );
-    expect(strokes).toHaveLength(0);
-    expect(unplacedIds).toHaveLength(1);
-  });
-
-  test('a canvas-level pin uses its page coordinates directly', () => {
-    const { strokes } = commentsToStrokes(
-      [comment({ nodeId: '0:0', x: 1100, y: 600 })],
-      indexNodes(page()),
-      ORIGIN,
-      new ImportReport(),
-      '0:0'
-    );
-    expect(strokes[0]).toMatchObject({ x: 100, y: 100 });
-  });
-});
-
-describe('threads', () => {
-  test('replies fold into their root card in time order', () => {
-    const { strokes } = commentsToStrokes(
-      [
-        comment({ id: 'root', message: 'A/b test', author: 'maj' }),
-        comment({ id: 'r2', parentId: 'root', message: 'second', createdAt: '2026-02-01' }),
-        comment({ id: 'r1', parentId: 'root', message: 'first', createdAt: '2026-01-01' }),
-      ],
-      indexNodes(page()),
-      ORIGIN,
-      new ImportReport()
-    );
-    expect(strokes).toHaveLength(1);
-    expect((strokes[0] as { text: string }).text).toBe('maj: A/b test\n↳ first\n↳ second');
-  });
-});
-
-describe('resolved threads', () => {
-  test('are carried on grey paper rather than dropped', () => {
-    // A resolved comment is the record of a decision already made; dropping it
-    // is the same silent content loss the rest of this importer keeps relearning.
-    const { strokes } = commentsToStrokes(
-      [comment({ resolved: true })],
-      indexNodes(page()),
-      ORIGIN,
-      new ImportReport()
-    );
-    expect(strokes).toHaveLength(1);
-    expect((strokes[0] as { color: string }).color).toBe(STICKY_PALETTE[2]);
-    expect((strokes[0] as { text: string }).text).toStartWith('✓ ');
-  });
-
-  test('an open thread stays on the default yellow', () => {
-    const { strokes } = commentsToStrokes(
-      [comment()],
-      indexNodes(page()),
-      ORIGIN,
-      new ImportReport()
-    );
-    expect((strokes[0] as { color: string }).color).toBe(STICKY_PALETTE[0]);
-  });
-});
-
-describe('bounds', () => {
-  test('message text goes through the zero-glyph strip like every other import', () => {
-    const report = new ImportReport();
-    // U+E0041 — Unicode Tags block, zero glyphs, a payload channel.
-    const { strokes } = commentsToStrokes(
-      [comment({ message: `ok\u{E0041}` })],
-      indexNodes(page()),
-      ORIGIN,
-      report
-    );
-    expect((strokes[0] as { text: string }).text).toBe('ok');
-    expect(report.entries.some((e) => e.disposition === 'hidden-chars-dropped')).toBe(true);
-  });
-
-  test('an empty message produces no card', () => {
-    const { strokes } = commentsToStrokes(
-      [comment({ message: '   ' })],
-      indexNodes(page()),
-      ORIGIN,
-      new ImportReport()
-    );
-    expect(strokes).toHaveLength(0);
-  });
-
-  test('a pathological comment count is capped and reported', () => {
-    const many = Array.from({ length: MAX_COMMENT_STROKES + 25 }, (_, i) =>
-      comment({ id: `c${i}` })
-    );
-    const report = new ImportReport();
-    const { strokes } = commentsToStrokes(many, indexNodes(page()), ORIGIN, report);
-    expect(strokes).toHaveLength(MAX_COMMENT_STROKES);
-    expect(report.entries.some((e) => e.disposition === 'asset-cap-reached')).toBe(true);
-  });
-});
-
-describe('orphaned threads (the caller decides, not this module)', () => {
-  test('an unplaceable thread is returned by id, NOT reported here', () => {
-    // Per page, "not on this page" is the normal case — a comment lives on
-    // exactly one page. Reporting it here would fire once per page and read as
-    // content loss that never happened. Only the caller, which sees every page,
-    // can tell an unplaced thread from a deleted-target orphan.
-    const report = new ImportReport();
-    const { strokes, placedIds, unplacedIds } = commentsToStrokes(
-      [comment({ id: 'here' }), comment({ id: 'elsewhere', nodeId: '9:9' })],
-      indexNodes(page()),
-      ORIGIN,
-      report
-    );
-    expect(strokes).toHaveLength(1);
-    expect(placedIds).toEqual(['here']);
-    expect(unplacedIds).toEqual(['elsewhere']);
-    expect(report.entries.some((e) => e.disposition === 'comment-target-deleted')).toBe(false);
-  });
-
-  test('placedIds are raw Figma ids so the caller can intersect them', () => {
-    // The stroke id is sanitized (`figc_…`); the caller reconciles across pages
-    // on the RAW id, and a mismatch there would silently orphan every thread.
-    const { strokes, placedIds } = commentsToStrokes(
-      [comment({ id: '166-92:86' })],
-      indexNodes(page()),
-      ORIGIN,
-      new ImportReport()
-    );
-    expect(placedIds).toEqual(['166-92:86']);
-    expect((strokes[0] as { id: string }).id).not.toBe('166-92:86');
-  });
-});