| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611 |
- /** Current-tree persistence history rejects uncovered and incorrectly acknowledged type changes. */
- import { spawnSync } from 'node:child_process'
- import { mkdirSync, mkdtempSync, readFileSync, readdirSync, rmSync, writeFileSync } from 'node:fs'
- import { tmpdir } from 'node:os'
- import { join, resolve } from 'node:path'
- import { afterEach, describe, expect, it } from 'vitest'
- import { canonicalizeSchema, schemaDigest } from './persistence-schema-model.ts'
- import type { PersistenceRoot, PersistenceSchemaInventory, SchemaNode, SchemaProperty } from './persistence-schema-model.ts'
- import { extractPersistenceSchema } from './persistence-schema.ts'
- import { persistenceCatalogArtifacts } from './gen-persistence-catalog.ts'
- import {
- classifyPersistenceChange,
- loadPersistenceHistory,
- parseHistoricalPersistenceSnapshot,
- parsePersistenceSnapshot,
- runPersistenceChanges as executePersistenceChanges,
- validatePersistenceHistory,
- verifyPersistenceChanges,
- } from './persistence-changes.ts'
- import type { PersistenceChangeRecord, PersistenceHistoryEntry } from './persistence-changes.ts'
- function runPersistenceChanges(
- args: readonly string[], root: string, extract: (root: string) => PersistenceSchemaInventory,
- ): string {
- return executePersistenceChanges(args, root, extract, (_root, current) => [{
- path: 'docs/persistence-schema.json', content: JSON.stringify(current, null, 2) + '\n',
- }])
- }
- function jsonResult(source: string): { ok: boolean; files: readonly string[] } {
- return JSON.parse(source) as { ok: boolean; files: readonly string[] }
- }
- const AUTHORED_PROSE = {
- en: { summary: 'Adds optional metadata.', compatibility: 'Readers may omit the metadata.', verification: 'The focused tests passed.' },
- zh: { summary: '添加可选元数据。', compatibility: '读取方可省略元数据。', verification: '定向测试通过。' },
- }
- function proseFile(root: string): string {
- const path = join(root, 'prose.json')
- writeFileSync(path, JSON.stringify(AUTHORED_PROSE))
- return path
- }
- const roots: string[] = []
- afterEach(() => { for (const root of roots.splice(0)) rmSync(root, { recursive: true, force: true }) })
- function fixture(): string {
- const root = mkdtempSync(join(tmpdir(), 'dsh-persistence-changes-'))
- roots.push(root)
- mkdirSync(join(root, 'docs/persistence-changes'), { recursive: true })
- return root
- }
- type Shape = 'string' | 'number' | 'boolean' | { readonly [property: string]: Shape }
- function typeRoot(key: string, shape: Shape, version = 3): PersistenceRoot {
- const nodes: SchemaNode[] = []
- function add(shape: Shape): number {
- const index = nodes.length
- nodes.push({ kind: 'primitive', type: 'never' })
- nodes[index] = typeof shape === 'string' ? { kind: 'primitive', type: shape } : {
- kind: 'object', indices: [], properties: Object.entries(shape).map(([name, child]) => ({
- name: name.replace(/\?$/u, ''), optional: name.endsWith('?'), type: add(child),
- })),
- }
- return index
- }
- if (key === 'SessionHeader') {
- const index = add(shape)
- const node = nodes[index]!
- if (node.kind !== 'object') throw new Error('header fixture requires an object')
- nodes[index] = { ...node, properties: [...node.properties, { name: 'version', optional: false, type: nodes.length }] }
- nodes.push({ kind: 'literal', value: version })
- } else if (key.startsWith('event:')) {
- add({ data: shape, type: 'string' })
- const node = nodes[0]!
- if (node.kind !== 'object') throw new Error('event fixture requires an object')
- const type = node.properties.find(item => item.name === 'type')!.type
- nodes[type] = { kind: 'literal', value: key.slice(6) }
- } else add(shape)
- const schema = canonicalizeSchema(nodes, 0)
- return { key, kind: ['SessionHeader', 'JsonlHeaderLine'].includes(key) ? 'header' : key === 'SessionEventEnvelope' ? 'envelope' : 'event',
- ...(key.startsWith('event:') ? { event: key.slice(6), surface: false } : {}), schema, digest: schemaDigest(schema) }
- }
- function inventory(eventShape: Shape = { value: 'string' }, version = 3): PersistenceSchemaInventory {
- return { formatVersion: 1, types: [], roots: [typeRoot('SessionHeader', { id: 'string' }, version), typeRoot('SessionEventEnvelope', { type: 'string' }), typeRoot('event:example/value', eventShape), typeRoot('JsonlHeaderLine', { version: 'number' })] }
- }
- function entry(
- id: string, schema: PersistenceSchemaInventory, previous: string | null,
- baseline = false, decision: 'same-version' | 'version-bump' = 'same-version',
- ): PersistenceHistoryEntry {
- return {
- record: { schemaVersion: 1, id, baseline, changes: schema.roots.map(root => ({
- root: root.key, previous, after: root.digest, decision,
- })) },
- snapshot: schema,
- }
- }
- const BASE_ID = '2026-09-11-baseline'
- const NEXT_ID = '2026-09-11-change'
- function historicalSurface(operation: 'required' | 'optional' | 'absent', event = 'example/value'): PersistenceSchemaInventory {
- const nodes: SchemaNode[] = [
- { kind: 'object', indices: [], properties: [
- { name: 'type', type: 1, optional: false },
- ...(operation === 'absent' ? [] : [{ name: 'surfaceOp', type: 2, optional: operation === 'optional' }]),
- ] },
- { kind: 'literal', value: event },
- { kind: 'primitive', type: 'string' },
- ]
- const schema = canonicalizeSchema(nodes, 0)
- return { formatVersion: 1, types: [], roots: [{
- key: 'event:example/value', kind: 'event', event: 'example/value', surface: true, schema, digest: schemaDigest(schema),
- }] }
- }
- describe('historical persistence snapshot parsing', () => {
- it('preserves source paths and fingerprints in historical inventories', () => {
- const snapshot = historicalSurface('required')
- const root = snapshot.roots[0]!
- const sources = ['packages/core/example/src/types.ts']
- const complete = { ...snapshot, types: [{ schema: root.schema, digest: root.digest, names: ['Example'], sources }] }
- expect(parseHistoricalPersistenceSnapshot(complete)).toEqual(complete)
- })
- it.each([':8', ':8:3', '#L8', '#L8-L12'])('rejects source coordinates %s only in historical inventories', (suffix) => {
- const snapshot = historicalSurface('required')
- const root = snapshot.roots[0]!
- const sources = [`packages/core/example/src/types.ts${suffix}`]
- const complete = { ...snapshot, types: [{ schema: root.schema, digest: root.digest, names: [], sources }] }
- expect(() => parseHistoricalPersistenceSnapshot(complete)).toThrow('historical schema sources must omit line numbers')
- expect(parsePersistenceSnapshot(complete)).toEqual(complete)
- })
- it('admits optional surface operations only through the historical parser', () => {
- const snapshot = historicalSurface('optional')
- expect(() => parsePersistenceSnapshot(snapshot)).toThrow('surface metadata')
- expect(parseHistoricalPersistenceSnapshot(snapshot)).toEqual(snapshot)
- expect(parsePersistenceSnapshot(historicalSurface('required'))).toEqual(historicalSurface('required'))
- })
- it('still rejects absent operations, mismatched event tags, and invalid roots', () => {
- expect(() => parseHistoricalPersistenceSnapshot(historicalSurface('absent'))).toThrow('surface metadata')
- expect(() => parseHistoricalPersistenceSnapshot(historicalSurface('optional', 'wrong/tag'))).toThrow('type does not match')
- const snapshot = historicalSurface('optional')
- expect(() => parseHistoricalPersistenceSnapshot({ ...snapshot, roots: snapshot.roots.map(root => ({ ...root, key: 'event:wrong/key' })) })).toThrow('invalid event root key')
- expect(() => parseHistoricalPersistenceSnapshot({ ...snapshot, roots: snapshot.roots.map(root => ({ ...root, surface: false })) })).toThrow('surface metadata')
- })
- it('preserves graph and digest validation in historical mode', () => {
- const snapshot = historicalSurface('optional')
- expect(() => parseHistoricalPersistenceSnapshot({ ...snapshot, roots: snapshot.roots.map(root => ({ ...root, digest: 'f'.repeat(64) })) })).toThrow('digest mismatch')
- expect(() => parseHistoricalPersistenceSnapshot({ ...snapshot, roots: snapshot.roots.map(root => ({ ...root, schema: { root: 0, nodes: [{ kind: 'array', element: 9 }] } })) })).toThrow('unknown schema node')
- })
- })
- function onlyEvent(schema: PersistenceSchemaInventory): PersistenceSchemaInventory {
- return { ...schema, roots: schema.roots.filter(root => root.kind === 'event') }
- }
- function finishDocuments(root: string, id: string): void {
- for (const suffix of ['.md', '.zh.md']) {
- const path = join(root, 'docs/persistence-changes', id + suffix)
- writeFileSync(path, readFileSync(path, 'utf8').replaceAll('TODO: explain this change.', 'Optional payload metadata preserves the recorded value.')
- .replaceAll('TODO: record validation evidence.', 'The focused persistence-history tests passed.'))
- }
- }
- function commitCurrent(root: string, schema: PersistenceSchemaInventory): void {
- writeFileSync(join(root, 'docs/persistence-schema.json'), JSON.stringify(schema, null, 2) + '\n')
- }
- function baseline(root: string, schema = inventory()): void {
- runPersistenceChanges(['--baseline', BASE_ID], root, () => schema)
- finishDocuments(root, BASE_ID)
- commitCurrent(root, schema)
- }
- function unionBody(arms: readonly (readonly SchemaProperty[])[]): PersistenceRoot {
- const nodes: SchemaNode[] = [
- { kind: 'object', indices: [], properties: [{ name: 'data', type: 1, optional: false }] },
- { kind: 'union', types: arms.map((_, index) => 5 + index) },
- { kind: 'primitive', type: 'string' },
- { kind: 'primitive', type: 'number' },
- { kind: 'primitive', type: 'boolean' },
- ...arms.map(properties => ({ kind: 'object' as const, indices: [], properties })),
- ]
- const schema = canonicalizeSchema(nodes, 0)
- return { ...typeRoot('event:example/value', {}), schema, digest: schemaDigest(schema) }
- }
- describe('persistence change classification', () => {
- it('treats a new optional payload subtree as one additive change even with required descendants', () => {
- const before = typeRoot('event:example/value', { value: 'string' })
- const after = typeRoot('event:example/value', { value: 'string', 'details?': { name: 'string', count: 'number' } })
- expect(classifyPersistenceChange(before, after)).toEqual([{ path: 'event:example/value.data.details', kind: 'optional-property-added', description: 'optional property added', requiresVersionBump: false }])
- })
- it('permits required-to-optional payload properties while rejecting opposite changes, type changes, and removals', () => {
- const required = typeRoot('event:example/value', { value: 'string' })
- const optional = typeRoot('event:example/value', { 'value?': 'string' })
- expect(classifyPersistenceChange(required, optional).every(change => !change.requiresVersionBump)).toBe(true)
- for (const after of [required, typeRoot('event:example/value', { 'value?': 'number' }), typeRoot('event:example/value', {})]) {
- expect(classifyPersistenceChange(optional, after).some(change => change.requiresVersionBump)).toBe(true)
- }
- })
- it('pairs multiple discriminated union arms when a shared payload gains an optional property', () => {
- function unionRoot(optional: boolean): PersistenceRoot {
- const nodes: SchemaNode[] = [
- { kind: 'object', indices: [], properties: [{ name: 'data', type: 1, optional: false }] },
- { kind: 'union', types: [2, 3] },
- { kind: 'object', indices: [], properties: [{ name: 'kind', type: 4, optional: false }, { name: 'payload', type: 6, optional: false }] },
- { kind: 'object', indices: [], properties: [{ name: 'kind', type: 5, optional: false }, { name: 'payload', type: 6, optional: false }] },
- { kind: 'literal', value: 'left' }, { kind: 'literal', value: 'right' },
- { kind: 'object', indices: [], properties: [{ name: 'value', type: 7, optional: false }, ...(optional ? [{ name: 'label', type: 7, optional: true }] : [])] },
- { kind: 'primitive', type: 'string' },
- ]
- const schema = canonicalizeSchema(nodes, 0)
- return { ...typeRoot('event:example/value', {}), schema, digest: schemaDigest(schema) }
- }
- const differences = classifyPersistenceChange(unionRoot(false), unionRoot(true))
- expect(differences.length).toBeGreaterThan(0)
- expect(differences.every(change => !change.requiresVersionBump)).toBe(true)
- })
- it('allows optional additions across undiscriminated union arms', () => {
- const first = [{ name: 'a', type: 2, optional: false }]
- const second = [{ name: 'b', type: 3, optional: false }]
- const extra = { name: 'x', type: 4, optional: true }
- const differences = classifyPersistenceChange(unionBody([first, second]), unionBody([[...first, extra], [...second, extra]]))
- expect(differences).toEqual([expect.objectContaining({ path: 'event:example/value.data.x', requiresVersionBump: false })])
- })
- it('allows required-to-optional fields across undiscriminated union arms', () => {
- const before = unionBody([[{ name: 'a', type: 2, optional: false }], [{ name: 'b', type: 3, optional: false }]])
- const after = unionBody([[{ name: 'a', type: 2, optional: true }], [{ name: 'b', type: 3, optional: true }]])
- const differences = classifyPersistenceChange(before, after)
- expect(differences.map(change => change.path).sort()).toEqual(['event:example/value.data.a', 'event:example/value.data.b'])
- expect(differences.every(change => change.kind === 'property-made-optional' && !change.requiresVersionBump)).toBe(true)
- })
- it('finds a complete matching when one union arm has multiple additive successors', () => {
- const required = [{ name: 'a', type: 2, optional: false }]
- const optional = [{ name: 'a', type: 2, optional: true }]
- const flexible = [{ name: '0', type: 4, optional: true }, ...optional]
- const constrained = [...required, { name: 'z', type: 4, optional: true }]
- const before = unionBody([required, optional])
- for (const after of [unionBody([flexible, constrained]), unionBody([constrained, flexible])]) {
- const differences = classifyPersistenceChange(before, after)
- expect(differences.length).toBeGreaterThan(0)
- expect(differences.every(change => !change.requiresVersionBump)).toBe(true)
- }
- const noCompleteMatching = unionBody([flexible, [{ name: 'different', type: 4, optional: true }]])
- expect(classifyPersistenceChange(before, noCompleteMatching).some(change => change.requiresVersionBump)).toBe(true)
- })
- it('keeps union cardinality, property removal, new required fields, and value types strict', () => {
- const first = [{ name: 'a', type: 2, optional: false }]
- const second = [{ name: 'b', type: 3, optional: false }]
- const third = [{ name: 'c', type: 4, optional: false }]
- const before = unionBody([first, second])
- const variants = [
- unionBody([first, second, third]),
- unionBody([first]),
- unionBody([first, third]),
- unionBody([[...first, { name: 'x', type: 4, optional: false }], [...second, { name: 'x', type: 4, optional: true }]]),
- unionBody([[{ name: 'a', type: 3, optional: true }], [{ name: 'b', type: 3, optional: true }]]),
- ]
- for (const after of variants) expect(classifyPersistenceChange(before, after).some(change => change.requiresVersionBump)).toBe(true)
- })
- it('checks all recursive union arms without retaining provisional successes between candidates', () => {
- const next = { name: 'next', type: 1, optional: true }
- const first = [{ name: 'a', type: 2, optional: false }, next]
- const second = [{ name: 'b', type: 3, optional: false }, next]
- const extra = { name: 'x', type: 4, optional: true }
- const before = unionBody([first, second])
- const additive = unionBody([[...first, extra], [...second, extra]])
- expect(classifyPersistenceChange(before, additive).every(change => !change.requiresVersionBump)).toBe(true)
- const invalid = unionBody([[...first, extra], [{ name: 'b', type: 2, optional: false }, next, extra]])
- expect(classifyPersistenceChange(before, invalid).some(change => change.requiresVersionBump)).toBe(true)
- expect(classifyPersistenceChange(invalid, before).some(change => change.requiresVersionBump)).toBe(true)
- })
- it('preserves body scope through arrays and tuples and rejects a simultaneous value-type change', () => {
- function wrapped(optional: boolean, primitive: 'string' | 'number' = 'string'): PersistenceRoot {
- const nodes: SchemaNode[] = [
- { kind: 'object', indices: [], properties: [{ name: 'data', type: 1, optional: false }] },
- { kind: 'tuple', elements: [{ type: 2, optional: false, rest: false }] },
- { kind: 'array', element: 3 },
- { kind: 'object', indices: [], properties: [{ name: 'value', type: 4, optional }] },
- { kind: 'primitive', type: primitive },
- ]
- const schema = canonicalizeSchema(nodes, 0)
- return { ...typeRoot('event:example/value', {}), schema, digest: schemaDigest(schema) }
- }
- expect(classifyPersistenceChange(wrapped(false), wrapped(true)).every(change => !change.requiresVersionBump)).toBe(true)
- expect(classifyPersistenceChange(wrapped(false), wrapped(true, 'number')).some(change => change.requiresVersionBump)).toBe(true)
- })
- it('keeps optional header/envelope properties and surface event additions strict', () => {
- for (const key of ['SessionHeader', 'JsonlHeaderLine', 'SessionEventEnvelope']) {
- expect(classifyPersistenceChange(typeRoot(key, {}), typeRoot(key, { 'metadata?': 'string' }))[0]?.requiresVersionBump).toBe(true)
- }
- const added = typeRoot('event:example/added', { value: 'string' })
- expect(classifyPersistenceChange(null, added)[0]?.requiresVersionBump).toBe(false)
- expect(classifyPersistenceChange(null, { ...added, surface: true })[0]?.requiresVersionBump).toBe(true)
- expect(classifyPersistenceChange(added, null)[0]?.requiresVersionBump).toBe(true)
- })
- })
- describe('persistence history verification', () => {
- it('accepts a baseline and a successive optional addition without comparing the historical schema to current', () => {
- const before = inventory()
- const after = inventory({ value: 'string', 'label?': 'string' })
- const history = validatePersistenceHistory([entry(BASE_ID, before, null, true), entry(NEXT_ID, onlyEvent(after), BASE_ID)])
- expect(history.tips.get('event:example/value')?.root?.digest).toBe(after.roots[2]?.digest)
- expect(history.tips.get('SessionHeader')?.id).toBe(BASE_ID)
- })
- it('rejects missing predecessors, forks, cycles, duplicates, and missing baseline', () => {
- const base = entry(BASE_ID, inventory(), null, true)
- const next = entry(NEXT_ID, onlyEvent(inventory({ value: 'string', 'label?': 'string' })), BASE_ID)
- expect(() => validatePersistenceHistory([next])).toThrow('exactly one baseline')
- expect(() => validatePersistenceHistory([base, entry(NEXT_ID, next.snapshot, '2026-09-11-missing')])).toThrow('missing predecessor')
- expect(() => validatePersistenceHistory([base, next, entry('2026-09-11-fork', next.snapshot, BASE_ID)])).toThrow('forked')
- expect(() => validatePersistenceHistory([base, base])).toThrow('exactly one baseline')
- const cycle = entry('2026-09-11-cycle', { ...next.snapshot, roots: [typeRoot('event:cycle/event', {})] }, '2026-09-11-cycle')
- expect(() => validatePersistenceHistory([base, cycle])).toThrow('cycle')
- })
- it('binds breaking decisions to the same record header version transition', () => {
- const base = entry(BASE_ID, inventory(), null, true)
- const breaking = inventory({ value: 'number' }, 4)
- expect(() => validatePersistenceHistory([base, entry(NEXT_ID, onlyEvent(breaking), BASE_ID)])).toThrow('requires a format version bump')
- expect(() => validatePersistenceHistory([base, entry(NEXT_ID, onlyEvent(breaking), BASE_ID, false, 'version-bump')])).toThrow("this record's own")
- const touched = { ...breaking, roots: breaking.roots.filter(root => root.key === 'SessionHeader' || root.kind === 'event') }
- expect(validatePersistenceHistory([base, entry(NEXT_ID, touched, BASE_ID, false, 'version-bump')]).tips.get('SessionHeader')?.id).toBe(NEXT_ID)
- const later = entry('2026-09-11-later', onlyEvent(inventory({ value: 'boolean' }, 4)), NEXT_ID, false, 'version-bump')
- expect(() => validatePersistenceHistory([base, entry(NEXT_ID, touched, BASE_ID, false, 'version-bump'), later])).toThrow("this record's own")
- })
- it('retains deletion tombstones and requires re-added roots to continue them', () => {
- const base = entry(BASE_ID, inventory(), null, true)
- const after = inventory({}, 4)
- const header = after.roots[0]!
- const record: PersistenceChangeRecord = { schemaVersion: 1, id: NEXT_ID, baseline: false, changes: [
- { root: 'SessionHeader', previous: BASE_ID, after: header.digest, decision: 'version-bump' },
- { root: 'event:example/value', previous: BASE_ID, after: null, decision: 'version-bump' },
- ] }
- const deletion = { record, snapshot: { ...after, roots: [header] } }
- expect(validatePersistenceHistory([base, deletion]).tips.get('event:example/value')?.root).toBeNull()
- const restored = entry('2026-09-11-restore', onlyEvent(inventory()), NEXT_ID)
- expect(validatePersistenceHistory([base, deletion, restored]).tips.get('event:example/value')?.id).toBe(restored.record.id)
- expect(() => validatePersistenceHistory([base, deletion, entry('2026-09-11-restore', restored.snapshot, null)])).toThrow('forked')
- })
- it('rejects malformed references, unknown schema variants, digest tampering, and extra snapshot roots', () => {
- const schema = inventory()
- const malformed = structuredClone(schema) as unknown as { roots: Array<{ schema: { nodes: unknown[] }; digest: string }> }
- malformed.roots[0]!.schema.nodes = [{ kind: 'array', element: 99 }]
- expect(() => parsePersistenceSnapshot(malformed)).toThrow('unknown schema node')
- malformed.roots[0]!.schema.nodes = [{ kind: 'future' }]
- expect(() => parsePersistenceSnapshot(malformed)).toThrow('unknown schema node kind')
- const tampered = structuredClone(schema)
- Object.assign(tampered.roots[0]!, { digest: '0'.repeat(64) })
- expect(() => parsePersistenceSnapshot(tampered)).toThrow('digest mismatch')
- expect(() => parsePersistenceSnapshot({ ...schema, formatVersion: 2 })).toThrow('normalization version')
- const next = entry(NEXT_ID, onlyEvent(inventory({ value: 'string', 'label?': 'string' })), BASE_ID)
- expect(() => validatePersistenceHistory([entry(BASE_ID, schema, null, true), { ...next, snapshot: inventory() }])).toThrow('snapshot roots')
- })
- })
- describe('persistence changes current-tree commands', () => {
- it('requires completed record prose, rejects stale generated output, and reports unacknowledged paths', () => {
- const root = fixture()
- const before = inventory()
- runPersistenceChanges(['--baseline', BASE_ID], root, () => before)
- expect(() => loadPersistenceHistory(root)).toThrow('complete compatibility')
- finishDocuments(root, BASE_ID)
- rmSync(join(root, 'docs/persistence-schema.json'))
- expect(() => verifyPersistenceChanges(root, before)).toThrow('is missing')
- commitCurrent(root, before)
- const after = inventory({ value: 'string', 'label?': 'string' })
- expect(() => verifyPersistenceChanges(root, after)).toThrow('is stale')
- commitCurrent(root, after)
- expect(() => verifyPersistenceChanges(root, after)).toThrow('event:example/value.data.label: optional property added (same-version allowed)')
- runPersistenceChanges(['--record', NEXT_ID, '--decision', 'same-version'], root, () => after)
- finishDocuments(root, NEXT_ID)
- expect(runPersistenceChanges([], root, () => after)).toContain('4 roots match 2 history records')
- expect(() => runPersistenceChanges(['--baseline', '2026-09-11-reset'], root, () => after)).toThrow('cannot reset history')
- expect(() => runPersistenceChanges(['--record', '2026-09-11-empty', '--decision', 'same-version'], root, () => after)).toThrow('no persistence type changes')
- })
- it('rejects changed bilingual machine declarations and unreferenced snapshots', () => {
- const root = fixture()
- baseline(root)
- const chinese = join(root, 'docs/persistence-changes', `${BASE_ID}.zh.md`)
- const original = readFileSync(chinese, 'utf8')
- writeFileSync(chinese, original.replace('baseline: true', 'baseline: false'))
- expect(() => loadPersistenceHistory(root)).toThrow('bilingual machine records differ')
- writeFileSync(chinese, original)
- writeFileSync(join(root, 'docs/persistence-changes/2026-09-11-orphan.schema.json'), '{}\n')
- expect(() => loadPersistenceHistory(root)).toThrow('unreferenced')
- })
- it('excludes nested historical format documents and schemas from acknowledgements', () => {
- const root = fixture()
- baseline(root)
- const directory = join(root, 'docs/persistence-changes/historical-formats')
- mkdirSync(directory)
- writeFileSync(join(directory, 'v0.md'), '---\nkind: persistence-format\n---\n')
- writeFileSync(join(directory, 'v0.schema.json'), '{}\n')
- expect(loadPersistenceHistory(root).entries.map(entry => entry.record.id)).toEqual([BASE_ID])
- })
- it.each([
- ['optional field', inventory({ value: 'string', 'label?': 'string' })],
- ['required-to-optional field', inventory({ 'value?': 'string' })],
- ['ordinary event addition', { ...inventory(), roots: [...inventory().roots, typeRoot('event:example/added', { count: 'number' })] }],
- ] as const)('infers same-version for %s', (_name, after) => {
- const root = fixture()
- baseline(root)
- runPersistenceChanges(['--record', NEXT_ID, '--prose', proseFile(root)], root, () => after)
- const changes = loadPersistenceHistory(root).entries.find(entry => entry.record.id === NEXT_ID)?.record.changes
- expect(changes?.length).toBeGreaterThan(0)
- expect(changes?.every(change => change.decision === 'same-version')).toBe(true)
- verifyPersistenceChanges(root, after)
- })
- it('infers a version bump when the source includes its own increasing header version', () => {
- const root = fixture()
- baseline(root)
- const after = inventory({ value: 'number' }, 4)
- runPersistenceChanges(['--record', NEXT_ID, '--prose', proseFile(root)], root, () => after)
- const changes = loadPersistenceHistory(root).entries.find(entry => entry.record.id === NEXT_ID)?.record.changes
- expect(changes?.map(change => change.root).sort()).toEqual(['SessionHeader', 'event:example/value'])
- expect(changes?.every(change => change.decision === 'version-bump')).toBe(true)
- verifyPersistenceChanges(root, after)
- })
- it('rejects inferred bumps without a header transition and explicit incorrect assertions before writing', () => {
- const root = fixture()
- baseline(root)
- const prose = proseFile(root)
- const directory = join(root, 'docs/persistence-changes')
- const files = readdirSync(directory).sort()
- const inventoryPath = join(root, 'docs/persistence-schema.json')
- const before = readFileSync(inventoryPath, 'utf8')
- for (const [args, code] of [
- [[], 'version-transition-required'],
- [['--decision', 'same-version'], 'version-bump-required'],
- ] as const) {
- const result = jsonResult(runPersistenceChanges(['--record', NEXT_ID, '--prose', prose, '--json', ...args], root,
- () => inventory({ value: 'number' })))
- expect(result).toMatchObject({ ok: false, code, files: [], changes: [expect.objectContaining({ requiresVersionBump: true })] })
- expect(readFileSync(inventoryPath, 'utf8')).toBe(before)
- expect(readdirSync(directory).sort()).toEqual(files)
- }
- })
- it('authors a complete pair from explicit prose and reports source changes even when generated artifacts are stale', () => {
- const root = fixture()
- baseline(root)
- const after = inventory({ value: 'string', 'label?': 'string' })
- const pending = jsonResult(runPersistenceChanges(['--check', '--json'], root, () => after))
- expect(pending).toMatchObject({ ok: false, operation: 'check', code: 'stale-artifacts',
- roots: [{ root: 'event:example/value', kind: 'event', before: inventory().roots[2]?.digest, after: after.roots[2]?.digest }], changes: [
- { root: 'event:example/value', path: 'event:example/value.data.label', kind: 'optional-property-added', requiresVersionBump: false },
- ] })
- const written = jsonResult(runPersistenceChanges(['--record', NEXT_ID, '--decision', 'same-version', '--prose', proseFile(root), '--json'], root, () => after))
- expect(written).toMatchObject({ ok: true, operation: 'record', recordId: NEXT_ID })
- expect(written.files).toEqual([
- 'docs/persistence-schema.json', `docs/persistence-changes/${NEXT_ID}.md`, `docs/persistence-changes/${NEXT_ID}.zh.md`,
- `docs/persistence-changes/${NEXT_ID}.i18n.yaml`, `docs/persistence-changes/${NEXT_ID}.schema.json`,
- ])
- expect(written).toMatchObject({ roots: [{ root: 'event:example/value', kind: 'event',
- before: inventory().roots[2]?.digest, after: after.roots[2]?.digest }] })
- expect(readFileSync(join(root, `docs/persistence-changes/${NEXT_ID}.md`), 'utf8')).toContain(AUTHORED_PROSE.en.compatibility)
- expect(jsonResult(runPersistenceChanges(['--check', '--json'], root, () => after)).ok).toBe(true)
- })
- it('refreshes only a terminal acknowledgement while preserving its authored prose and predecessor', () => {
- const root = fixture()
- baseline(root)
- const after = inventory({ value: 'string', 'label?': 'string' })
- const prose = proseFile(root)
- runPersistenceChanges(['--record', NEXT_ID, '--decision', 'same-version', '--prose', prose], root, () => after)
- const path = join(root, `docs/persistence-changes/${NEXT_ID}.md`)
- const beforeText = readFileSync(path, 'utf8')
- const refreshed = inventory({ value: 'string', 'label?': 'string', 'extra?': 'number' })
- runPersistenceChanges(['--update', NEXT_ID], root, () => refreshed)
- const stripRecord = (value: string): string => value.replace(/```yaml persistence-change[\s\S]*?```/u, '')
- expect(stripRecord(readFileSync(path, 'utf8'))).toBe(stripRecord(beforeText))
- expect(loadPersistenceHistory(root).tips.get('event:example/value')?.id).toBe(NEXT_ID)
- expect(loadPersistenceHistory(root).entries.find(entry => entry.record.id === NEXT_ID)?.record.changes[0]?.previous).toBe(BASE_ID)
- expect(jsonResult(runPersistenceChanges(['--check', '--json'], root, () => refreshed)).ok).toBe(true)
- const final = inventory({ value: 'string', 'label?': 'string', 'extra?': 'number', 'last?': 'boolean' })
- runPersistenceChanges(['--record', '2026-09-11-successor', '--decision', 'same-version', '--prose', prose], root, () => final)
- expect(() => runPersistenceChanges(['--update', NEXT_ID, '--decision', 'same-version'], root, () => final)).toThrow('with successors')
- expect(() => runPersistenceChanges(['--update', BASE_ID, '--decision', 'same-version'], root, () => final)).toThrow('cannot update the persistence baseline')
- })
- it('completes a scaffold through update and rejects invalid prose or decisions before changing files', () => {
- const root = fixture()
- baseline(root)
- const after = inventory({ value: 'string', 'label?': 'string' })
- runPersistenceChanges(['--record', NEXT_ID, '--decision', 'same-version'], root, () => after)
- const prose = proseFile(root)
- runPersistenceChanges(['--update', NEXT_ID, '--decision', 'same-version', '--prose', prose], root, () => after)
- expect(runPersistenceChanges(['--check'], root, () => after)).toContain('roots match')
- expect(readFileSync(join(root, `docs/persistence-changes/${NEXT_ID}.md`), 'utf8')).not.toContain('TODO:')
- const pairedPaths = ['.md', '.zh.md', '.i18n.yaml'].map(suffix => join(root, `docs/persistence-changes/${NEXT_ID}${suffix}`))
- const completed = pairedPaths.map(path => readFileSync(path, 'utf8'))
- runPersistenceChanges(['--update', NEXT_ID, '--decision', 'same-version', '--prose', prose], root, () => after)
- expect(pairedPaths.map(path => readFileSync(path, 'utf8'))).toEqual(completed)
- const path = join(root, `docs/persistence-changes/${NEXT_ID}.md`)
- const before = readFileSync(path, 'utf8')
- const invalid = inventory({ value: 'number' })
- const result = jsonResult(runPersistenceChanges(['--update', NEXT_ID, '--decision', 'same-version', '--json'], root, () => invalid))
- expect(result).toMatchObject({ ok: false, code: 'version-bump-required', changes: [expect.objectContaining({ kind: 'type-changed', requiresVersionBump: true })] })
- expect(readFileSync(path, 'utf8')).toBe(before)
- for (const value of [
- { ...AUTHORED_PROSE, extra: 'unsupported' },
- { ...AUTHORED_PROSE, en: { ...AUTHORED_PROSE.en, compatibility: ' ' } },
- { ...AUTHORED_PROSE, zh: { ...AUTHORED_PROSE.zh, verification: 'TODO: record validation evidence.' } },
- { ...AUTHORED_PROSE, zh: { ...AUTHORED_PROSE.zh, verification: '```text\nUnpaired code.\n```' } },
- ]) {
- writeFileSync(prose, JSON.stringify(value))
- expect(() => runPersistenceChanges(['--update', NEXT_ID, '--decision', 'same-version', '--prose', prose], root, () => after)).toThrow()
- expect(readFileSync(path, 'utf8')).toBe(before)
- }
- })
- // Six real CLI processes and TypeScript extraction share the Windows coverage test budget.
- it('executes the real CLI against a source-only temporary checkout without Git history', { timeout: 90_000 }, () => {
- const root = fixture()
- const physical = join(root, 'packages/session/session-persistence-jsonl/src')
- mkdirSync(physical, { recursive: true })
- writeFileSync(join(physical, 'format.ts'), "interface HeaderLine { type: 'session'; version: number; id: string; delegationDepth: number }; export {}\n")
- const session = join(root, 'packages/core/session/src')
- mkdirSync(session, { recursive: true })
- writeFileSync(join(session, '../package.json'), JSON.stringify({ name: '@deepseek-ai/dsh-session' }))
- writeFileSync(join(root, 'tsconfig.host.json'), JSON.stringify({ compilerOptions: {
- target: 'ESNext', module: 'ESNext', moduleResolution: 'Bundler', types: [], skipLibCheck: true,
- paths: { '@deepseek-ai/dsh-session/types': ['./packages/core/session/src/types.ts'] },
- } }))
- const source = [
- '/** Stored header. */', 'export interface SessionHeader { version: 3; id: string }',
- '/** Stored event payloads. */', 'export interface SessionEventMap {', '/** Saved value. */',
- "'example/value': { value: string }", '}', '/** Surface event names. */', "export type SurfaceEventType = 'example/value'",
- '/** Persisted event names. */', 'export type SessionEventType = keyof SessionEventMap',
- '/** Surface placement. */', "export type SurfaceOp = 'append'", '/** Persisted event. */',
- 'export type SessionEvent<K extends keyof SessionEventMap = keyof SessionEventMap> =',
- ' { [P in K]: { type: P; seq: number; data: SessionEventMap[P]; surfaceOp: SurfaceOp } }[K]', '',
- ].join('\n')
- writeFileSync(join(session, 'types.ts'), source)
- const script = resolve(import.meta.dirname, 'persistence-changes.ts')
- const cli = (...args: string[]): ReturnType<typeof spawnSync> => spawnSync(process.execPath, ['--import', import.meta.resolve('tsx/esm'), script, '--root', root, ...args], {
- cwd: root, encoding: 'utf8', timeout: 120_000,
- })
- const prose = proseFile(root)
- const initialized = cli('--baseline', BASE_ID, '--prose', prose)
- expect(initialized.error).toBeUndefined()
- expect(initialized.signal).toBeNull()
- expect(initialized.status, String(initialized.stderr)).toBe(0)
- const accepted = cli('--check')
- expect(accepted.error).toBeUndefined()
- expect(accepted.signal).toBeNull()
- expect(accepted.status, String(accepted.stderr)).toBe(0)
- const optional = source.replace('value: string', 'value: string; label?: string')
- writeFileSync(join(session, 'types.ts'), optional)
- const authored = cli('--record', NEXT_ID, '--prose', prose, '--json')
- expect(authored.error).toBeUndefined()
- expect(authored.signal).toBeNull()
- expect(authored.status, String(authored.stderr)).toBe(0)
- expect(JSON.parse(String(authored.stdout)) as unknown).toMatchObject({ ok: true, operation: 'record' })
- const beforeUpdate = readFileSync(join(root, `docs/persistence-changes/${NEXT_ID}.md`), 'utf8')
- writeFileSync(join(session, 'types.ts'), optional.replace('label?: string', 'label?: string; extra?: number'))
- const updated = cli('--update', NEXT_ID, '--json')
- expect(updated.error).toBeUndefined()
- expect(updated.signal).toBeNull()
- expect(updated.status, String(updated.stderr)).toBe(0)
- expect(JSON.parse(String(updated.stdout)) as unknown).toMatchObject({ ok: true, operation: 'update' })
- expect(readFileSync(join(root, `docs/persistence-changes/${NEXT_ID}.md`), 'utf8').replace(/```yaml persistence-change[\s\S]*?```/u, ''))
- .toBe(beforeUpdate.replace(/```yaml persistence-change[\s\S]*?```/u, ''))
- const generated = extractPersistenceSchema(root)
- for (const file of persistenceCatalogArtifacts(root, generated)) expect(readFileSync(join(root, file.path), 'utf8')).toBe(file.content)
- verifyPersistenceChanges(root, generated)
- writeFileSync(join(session, 'types.ts'), optional.replace('label?: string', 'label?: string; extra?: number').replace('value: string', 'value: number'))
- commitCurrent(root, extractPersistenceSchema(root))
- const rejected = cli('--check')
- expect(rejected.error).toBeUndefined()
- expect(rejected.signal).toBeNull()
- expect(rejected.status).toBe(1)
- expect(String(rejected.stderr)).toContain('version-bump required')
- const structured = cli('--check', '--json')
- expect(structured.error).toBeUndefined()
- expect(structured.signal).toBeNull()
- expect(structured.status).toBe(1)
- expect(JSON.parse(String(structured.stdout)) as unknown).toMatchObject({
- ok: false, code: 'unacknowledged-changes', changes: [expect.objectContaining({ kind: 'type-changed', requiresVersionBump: true })],
- })
- })
- })
|