070959e133
landing-page-staging / Deploy landing page to staging (push) Has been skipped
landing-page-ci / Validate landing page (push) Failing after 4s
visual-baseline / Capture visual baselines (push) Has been cancelled
bake-plugin-previews / Bake plugin previews (push) Has been cancelled
206 lines
7.6 KiB
TypeScript
206 lines
7.6 KiB
TypeScript
import { describe, expect, it } from 'vitest';
|
|
|
|
import { parseProvenance } from '../../src/lib/parse-provenance';
|
|
|
|
const FRESH = `# DESIGN.md
|
|
|
|
## Summary
|
|
|
|
Some content.
|
|
|
|
## Provenance
|
|
|
|
- Project ID: 818cf7a8-8399-4220-a507-07802d8842a8
|
|
- Design system: alphatrace
|
|
- Current artifact: deck.html
|
|
- Transcript message count: 42
|
|
- Generated UTC timestamp: 2026-05-08T11:55:00Z
|
|
`;
|
|
|
|
describe('parseProvenance', () => {
|
|
it('returns all five fields populated for a happy-path input', () => {
|
|
const result = parseProvenance(FRESH);
|
|
expect(result).not.toBeNull();
|
|
expect(result!.projectId).toBe('818cf7a8-8399-4220-a507-07802d8842a8');
|
|
expect(result!.designSystemId).toBe('alphatrace');
|
|
expect(result!.currentArtifact).toBe('deck.html');
|
|
expect(result!.transcriptMessageCount).toBe(42);
|
|
expect(result!.generatedAt).not.toBeNull();
|
|
expect(result!.generatedAt!.toISOString()).toBe('2026-05-08T11:55:00.000Z');
|
|
});
|
|
|
|
it('returns null when the Provenance section is missing', () => {
|
|
const text = `# DESIGN.md\n\n## Summary\n\nThis spec has no provenance.\n`;
|
|
expect(parseProvenance(text)).toBeNull();
|
|
});
|
|
|
|
it('treats the "none" sentinel for design system as null', () => {
|
|
const text = `## Provenance
|
|
|
|
- Project ID: abc-123
|
|
- Design system: none
|
|
- Current artifact: none
|
|
- Transcript message count: 7
|
|
- Generated UTC timestamp: 2026-05-08T00:00:00Z
|
|
`;
|
|
const result = parseProvenance(text);
|
|
expect(result).not.toBeNull();
|
|
expect(result!.designSystemId).toBeNull();
|
|
expect(result!.currentArtifact).toBeNull();
|
|
// Other fields still populated.
|
|
expect(result!.projectId).toBe('abc-123');
|
|
expect(result!.transcriptMessageCount).toBe(7);
|
|
});
|
|
|
|
it('returns generatedAt: null when the timestamp is malformed (no throw)', () => {
|
|
const text = `## Provenance
|
|
|
|
- Project ID: abc-123
|
|
- Design system: alphatrace
|
|
- Current artifact: deck.html
|
|
- Transcript message count: 42
|
|
- Generated UTC timestamp: not-a-date
|
|
`;
|
|
const result = parseProvenance(text);
|
|
expect(result).not.toBeNull();
|
|
expect(result!.generatedAt).toBeNull();
|
|
// Surrounding fields still populated.
|
|
expect(result!.transcriptMessageCount).toBe(42);
|
|
});
|
|
|
|
// Issue #1580: the daemon's synthesis prompt does not pin field-label
|
|
// syntax (finalize-design.ts:560-565), so Claude renders Provenance
|
|
// fields with Markdown-bold labels per Markdown convention. The
|
|
// pre-fix regexes' `[:\s]+` separator stops at the trailing `**`
|
|
// after the colon, leaking `** ` into every captured value and
|
|
// making transcriptMessageCount + generatedAt parse as null.
|
|
it('parses bold-labelled fields with backticked values (live DESIGN.md shape)', () => {
|
|
// Verbatim shape from a finalized DESIGN.md emitted by Claude
|
|
// against the prod synthesis prompt. UUID + filename are
|
|
// illustrative placeholders, not user data.
|
|
const text = `## Provenance
|
|
|
|
- **Project ID:** \`00000000-0000-0000-0000-000000000000\`
|
|
- **Design system:** \`default\` (Neutral Modern — not applied; wireframe overrides all tokens)
|
|
- **Current artifact:** \`prototype.html\` (single-file, 1,922 lines, 57KB)
|
|
- **Transcript message count:** 4
|
|
- **Generated UTC timestamp:** 2026-05-13T12:27:21.499Z
|
|
`;
|
|
const result = parseProvenance(text);
|
|
expect(result).not.toBeNull();
|
|
// Backticks may remain in the captured value (out of scope to
|
|
// strip per #1580 spec); the `** ` Markdown-bold prefix must not.
|
|
expect(result!.projectId).toBe('`00000000-0000-0000-0000-000000000000`');
|
|
expect(result!.designSystemId).toBe('`default` (Neutral Modern — not applied; wireframe overrides all tokens)');
|
|
expect(result!.currentArtifact).toBe('`prototype.html` (single-file, 1,922 lines, 57KB)');
|
|
expect(result!.transcriptMessageCount).toBe(4);
|
|
expect(result!.generatedAt).not.toBeNull();
|
|
expect(result!.generatedAt!.toISOString()).toBe('2026-05-13T12:27:21.499Z');
|
|
});
|
|
|
|
it('parses bold-labelled fields with plain values and a short "Generated:" label', () => {
|
|
const text = `## Provenance
|
|
|
|
- **Project ID:** abc-123
|
|
- **Design system:** my-system
|
|
- **Current artifact:** deck.html
|
|
- **Transcript message count:** 12
|
|
- **Generated:** 2026-05-08T11:55:00Z
|
|
`;
|
|
const result = parseProvenance(text);
|
|
expect(result).not.toBeNull();
|
|
expect(result!.projectId).toBe('abc-123');
|
|
expect(result!.designSystemId).toBe('my-system');
|
|
expect(result!.currentArtifact).toBe('deck.html');
|
|
expect(result!.transcriptMessageCount).toBe(12);
|
|
expect(result!.generatedAt).not.toBeNull();
|
|
expect(result!.generatedAt!.toISOString()).toBe('2026-05-08T11:55:00.000Z');
|
|
});
|
|
|
|
// PR #1584 review (lefarcen): the round-1 strip used `^[\s*_]+` /
|
|
// `[\s*_]+$`, which stripped a literal leading/trailing underscore
|
|
// from values like `_draft.html` (corrupting it to `draft.html`).
|
|
// Narrow the strip to only consume Markdown residue, never literal
|
|
// characters in the value itself.
|
|
it('preserves a literal leading underscore in a plain-label value (e.g. _draft.html)', () => {
|
|
const text = `## Provenance
|
|
|
|
- Project ID: abc-123
|
|
- Design system: alphatrace
|
|
- Current artifact: _draft.html
|
|
- Transcript message count: 7
|
|
- Generated UTC timestamp: 2026-05-08T00:00:00Z
|
|
`;
|
|
const result = parseProvenance(text);
|
|
expect(result).not.toBeNull();
|
|
// The whole filename must survive — no leading underscore strip.
|
|
expect(result!.currentArtifact).toBe('_draft.html');
|
|
});
|
|
|
|
it('preserves a literal trailing underscore in a plain-label id-like value', () => {
|
|
const text = `## Provenance
|
|
|
|
- Project ID: build_id_v1_
|
|
- Design system: alphatrace
|
|
- Current artifact: deck.html
|
|
- Transcript message count: 7
|
|
- Generated UTC timestamp: 2026-05-08T00:00:00Z
|
|
`;
|
|
const result = parseProvenance(text);
|
|
expect(result).not.toBeNull();
|
|
expect(result!.projectId).toBe('build_id_v1_');
|
|
});
|
|
|
|
it('preserves a literal leading underscore even when the label is Markdown-bold', () => {
|
|
const text = `## Provenance
|
|
|
|
- **Project ID:** abc-123
|
|
- **Design system:** alphatrace
|
|
- **Current artifact:** _draft.html
|
|
- **Transcript message count:** 7
|
|
- **Generated UTC timestamp:** 2026-05-08T00:00:00Z
|
|
`;
|
|
const result = parseProvenance(text);
|
|
expect(result).not.toBeNull();
|
|
// The bold-label residue (`** `) must be stripped, but the literal
|
|
// leading underscore on the filename must remain.
|
|
expect(result!.currentArtifact).toBe('_draft.html');
|
|
});
|
|
|
|
it('strips a balanced **value** wrap (residue case, no preceding bold-label residue)', () => {
|
|
const text = `## Provenance
|
|
|
|
- Project ID: **wrapped-id**
|
|
- Design system: alphatrace
|
|
- Current artifact: deck.html
|
|
- Transcript message count: 7
|
|
- Generated UTC timestamp: 2026-05-08T00:00:00Z
|
|
`;
|
|
const result = parseProvenance(text);
|
|
expect(result).not.toBeNull();
|
|
// **X** is unambiguously Markdown emphasis residue per the issue
|
|
// spec; strip the balanced wrap.
|
|
expect(result!.projectId).toBe('wrapped-id');
|
|
});
|
|
|
|
it('still treats "none" as the null sentinel after the bold-label prefix is stripped', () => {
|
|
const text = `## Provenance
|
|
|
|
- **Project ID:** abc-123
|
|
- **Design system:** none
|
|
- **Current artifact:** none
|
|
- **Transcript message count:** 7
|
|
- **Generated UTC timestamp:** 2026-05-08T00:00:00Z
|
|
`;
|
|
const result = parseProvenance(text);
|
|
expect(result).not.toBeNull();
|
|
expect(result!.projectId).toBe('abc-123');
|
|
// NONE_SENTINEL must trip on the value after emphasis is stripped,
|
|
// otherwise "** none" leaks through as a real design-system id.
|
|
expect(result!.designSystemId).toBeNull();
|
|
expect(result!.currentArtifact).toBeNull();
|
|
expect(result!.transcriptMessageCount).toBe(7);
|
|
expect(result!.generatedAt).not.toBeNull();
|
|
});
|
|
});
|