Skip to content

Shared Tree: Persisted Schema Format v2 with persisted metadata support #24590

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 40 commits into from
Closed
Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
40 commits
Select commit Hold shift + click to select a range
bd8406a
Initial change.
TommyBrosman May 12, 2025
3165fe3
persistedMetadata is now backed by an object instead of a string.
TommyBrosman May 13, 2025
9709ede
Updated codec.spec.ts to cover FormatV2.
TommyBrosman May 13, 2025
ba058e1
Removed extraneous whitespace change.
TommyBrosman May 13, 2025
b819742
- Changed TreeNodeStoredSchema implementations to take the write vers…
TommyBrosman May 13, 2025
0b5c842
- Changed the stored schema implementation to handle multiple schema …
TommyBrosman May 13, 2025
ecdca7a
Merge branch 'main' into metadata-schema-3
TommyBrosman May 13, 2025
1117c2c
Apply suggestions from code review
TommyBrosman May 14, 2025
94a5c39
- Switched to using the SchemaCodecVersion in public APIs. Eventually…
TommyBrosman May 14, 2025
302cb21
Merge branch 'metadata-schema-3' of https://github.com/TommyBrosman/F…
TommyBrosman May 14, 2025
0ba34d1
- Removed the schema version constant and replaced it with SchemaCode…
TommyBrosman May 14, 2025
3275ee0
Minor: reverted changes to FluidClientVersion utils.
TommyBrosman May 15, 2025
7c48b9d
Switched back to using min client version for importCompressed.
TommyBrosman May 15, 2025
a2f4195
Minor: reverted accidental change.
TommyBrosman May 15, 2025
4786dcc
Apply suggestions from code review
TommyBrosman May 15, 2025
6fa8836
Made persisted metadata schema field naming consistent.
TommyBrosman May 15, 2025
e4064c7
- Created a separate storedSchemaDecodeDispatcher for v2 schemas.
TommyBrosman May 16, 2025
ab1b882
- Refactored the node kind dispatch.
TommyBrosman May 19, 2025
1027daa
Updated snapshots.
TommyBrosman May 19, 2025
e527eba
- Removed minimum client version changes. Equivalent changes will be …
TommyBrosman May 19, 2025
3eae1f6
Removed more minimumClientVersion glue.
TommyBrosman May 19, 2025
98e1b76
Updated API files.
TommyBrosman May 19, 2025
ecc77c9
Rename: SchemaCodecVersion -> SchemaVersion.
TommyBrosman May 20, 2025
37e900f
Merge branch 'main' into metadata-schema-3
TommyBrosman May 20, 2025
737633b
Removed top-level persisted metadata.
TommyBrosman May 21, 2025
4bb136c
Fixed comments.
TommyBrosman May 22, 2025
0bcb90f
Exposed persistedMetadata as a JsonCompatibleReadOnlyObject.
TommyBrosman May 22, 2025
a675338
Changeset.
TommyBrosman May 22, 2025
1d1ef5f
Wired up toStoredSchema. Still needs tests.
TommyBrosman May 22, 2025
b9898e8
- Updated an old snapshot that didn't include the metadata field.
TommyBrosman May 22, 2025
6ca189a
Refactor: persistedMetadata -> metadata on persisted types.
TommyBrosman May 27, 2025
85240dc
Apply suggestions from code review
TommyBrosman May 27, 2025
8883d45
- Updated the changeset description
TommyBrosman May 28, 2025
9c2c13f
Reverted unnecessary change.
TommyBrosman May 28, 2025
c62f7ec
Wired up node schema metadata persistence. Currently errors on tests …
TommyBrosman May 29, 2025
2f6b519
Reverted unneeded change.
TommyBrosman May 29, 2025
ae1779b
Removed tests for the alpha API I removed in a previous revision.
TommyBrosman May 29, 2025
7eff8ab
This revision adds simple-tree persistence for node and field schema …
TommyBrosman May 30, 2025
f3f572c
- Fixed missing persistedMetadata on fields. toStoredSchema.ts and sh…
TommyBrosman Jun 2, 2025
faaf2f7
Merge branch 'main' into metadata-schema-3
TommyBrosman Jun 2, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion packages/dds/tree/api-report/tree.alpha.api.md
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,8 @@ export enum FluidClientVersion {
v2_0 = "v2_0",
v2_1 = "v2_1",
v2_2 = "v2_2",
v2_3 = "v2_3"
v2_3 = "v2_3",
v2_4 = "v2_4"
}

// @alpha
Expand Down
2 changes: 2 additions & 0 deletions packages/dds/tree/src/codec/codec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -352,6 +352,8 @@ export enum FluidClientVersion {
v2_2 = "v2_2",
/** Fluid Framework Client 2.3 and newer. */
v2_3 = "v2_3",
/** Fluid Framework Client 2.4 and newer. */
v2_4 = "v2_4",
}

/**
Expand Down
5 changes: 4 additions & 1 deletion packages/dds/tree/src/core/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -138,15 +138,18 @@ export {
storedEmptyFieldSchema,
type StoredSchemaCollection,
schemaFormatV1,
schemaFormatV2,
LeafNodeStoredSchema,
ObjectNodeStoredSchema,
MapNodeStoredSchema,
decodeFieldSchema,
encodeFieldSchema,
encodeFieldSchemaV1,
encodeFieldSchemaV2,
storedSchemaDecodeDispatcher,
type SchemaAndPolicy,
Multiplicity,
type SchemaPolicy,
SchemaCodecVersion,
} from "./schema-stored/index.js";

export {
Expand Down
108 changes: 108 additions & 0 deletions packages/dds/tree/src/core/schema-stored/formatV2.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,108 @@
/*!
* Copyright (c) Microsoft Corporation and contributors. All rights reserved.
* Licensed under the MIT License.
*/

import { type ObjectOptions, type Static, Type } from "@sinclair/typebox";

import { unionOptions } from "../../codec/index.js";
import {
type Brand,
brandedStringType,
JsonCompatibleReadOnlySchema,
} from "../../util/index.js";

export const version = 2 as const;

/**
* Key (aka Name or Label) for a field which is scoped to a specific TreeNodeStoredSchema.
*
* Stable identifier, used when persisting data.
*/
export type FieldKey = Brand<string, "tree.FieldKey">;

/**
* TypeBox Schema for encoding {@link FieldKey} in persisted data.
*/
export const FieldKeySchema = brandedStringType<FieldKey>();

/**
* Identifier for a TreeNode schema.
* Also known as "Definition"
*
* Stable identifier, used when persisting data.
*/
export type TreeNodeSchemaIdentifier<TName extends string = string> = Brand<
TName,
"tree.TreeNodeSchemaIdentifier"
>;

/**
* Identifier for a FieldKind.
* Refers to an exact stable policy (ex: specific version of a policy),
* for how to handle (ex: edit and merge edits to) fields marked with this kind.
* Persisted in documents as part of stored schema.
*/
export type FieldKindIdentifier = Brand<string, "tree.FieldKindIdentifier">;
export const FieldKindIdentifierSchema = brandedStringType<FieldKindIdentifier>();

/**
* TypeBox Schema for encoding {@link TreeNodeSchemaIdentifiers} in persisted data.
*/
export const TreeNodeSchemaIdentifierSchema = brandedStringType<TreeNodeSchemaIdentifier>();

export const PersistedMetadataFormat = Type.Optional(JsonCompatibleReadOnlySchema);

const FieldSchemaFormatBase = Type.Object({
kind: FieldKindIdentifierSchema,
types: Type.Array(TreeNodeSchemaIdentifierSchema),
persistedMetadata: PersistedMetadataFormat,
});

const noAdditionalProps: ObjectOptions = { additionalProperties: false };

export const FieldSchemaFormat = Type.Composite([FieldSchemaFormatBase], noAdditionalProps);

/**
* Persisted version of {@link ValueSchema}.
*/
export enum PersistedValueSchema {
Number,
String,
Boolean,
FluidHandle,
Null,
}

/**
* Discriminated union content of tree node schema.
*
* See {@link DiscriminatedUnionDispatcher} for more information on this pattern.
*/
export const TreeNodeSchemaDataFormat = Type.Object(
{
/**
* Object node union member.
*/
object: Type.Optional(Type.Record(Type.String(), FieldSchemaFormat)),
/**
* Map node union member.
*/
map: Type.Optional(FieldSchemaFormat),
/**
* Leaf node union member.
*/
leaf: Type.Optional(Type.Enum(PersistedValueSchema)),
/**
* Persisted metadata for this node.
*/
persistedMetadata: PersistedMetadataFormat,
},
unionOptions,
);

export type TreeNodeSchemaDataFormat = Static<typeof TreeNodeSchemaDataFormat>;

export type FieldSchemaFormat = Static<typeof FieldSchemaFormat>;

export type PersistedMetadataFormat = Static<typeof PersistedMetadataFormat>;
6 changes: 5 additions & 1 deletion packages/dds/tree/src/core/schema-stored/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,12 @@ export {
ObjectNodeStoredSchema,
MapNodeStoredSchema,
decodeFieldSchema,
encodeFieldSchema,
encodeFieldSchemaV1,
encodeFieldSchemaV2,
storedSchemaDecodeDispatcher,
type SchemaAndPolicy,
type SchemaPolicy,
SchemaCodecVersion,
} from "./schema.js";
export {
type TreeStoredSchemaSubscription,
Expand All @@ -36,3 +38,5 @@ export type { TreeNodeSchemaIdentifier, FieldKey, FieldKindIdentifier } from "./

import * as schemaFormatV1 from "./formatV1.js";
export { schemaFormatV1 };
import * as schemaFormatV2 from "./formatV2.js";
export { schemaFormatV2 };
92 changes: 78 additions & 14 deletions packages/dds/tree/src/core/schema-stored/schema.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,26 @@ import { type MakeNominal, brand, invertMap } from "../../util/index.js";
import {
type FieldKey,
type FieldKindIdentifier,
type FieldSchemaFormat,
type FieldSchemaFormat as FieldSchemaFormatV1,
PersistedValueSchema,
type TreeNodeSchemaDataFormat,
type TreeNodeSchemaDataFormat as TreeNodeSchemaDataFormatV1,
type TreeNodeSchemaIdentifier,
} from "./formatV1.js";
import type {
FieldSchemaFormat as FieldSchemaFormatV2,
PersistedMetadataFormat,
TreeNodeSchemaDataFormat as TreeNodeSchemaDataFormatV2,
} from "./formatV2.js";
import type { Multiplicity } from "./multiplicity.js";

export enum SchemaCodecVersion {
v1 = 1,
// Adds persisted metadata to the schema.
v2 = 2,
}

type FieldSchemaFormat = FieldSchemaFormatV1 | FieldSchemaFormatV2;

/**
* Schema for what {@link TreeLeafValue} is allowed on a Leaf node.
* @privateRemarks
Expand Down Expand Up @@ -119,6 +132,13 @@ export interface TreeFieldStoredSchema {
* If not specified, types are unconstrained.
*/
readonly types: TreeTypeSet;

/**
* Portion of the metadata which can be persisted.
* @remarks
* Discarded when encoding to {@link SchemaFormatVersion.V1}.
*/
persistedMetadata?: PersistedMetadataFormat;
}

/**
Expand All @@ -142,6 +162,7 @@ export const storedEmptyFieldSchema: TreeFieldStoredSchema = {
kind: brand(forbiddenFieldKindIdentifier),
// This type set also forces the field to be empty not not allowing any types as all.
types: new Set(),
persistedMetadata: undefined,
};

/**
Expand All @@ -160,7 +181,9 @@ export abstract class TreeNodeStoredSchema {
* This is uses an opaque type to avoid leaking these types out of the package,
* and is runtime validated by the codec.
*/
public abstract encode(): TreeNodeSchemaDataFormat;
public abstract encode(
schemaWriteVersion: SchemaCodecVersion,
): TreeNodeSchemaDataFormatV1 | TreeNodeSchemaDataFormatV2;

/**
* Returns the schema for the provided field.
Expand All @@ -185,18 +208,35 @@ export class ObjectNodeStoredSchema extends TreeNodeStoredSchema {
super();
}

public override encode(): TreeNodeSchemaDataFormat {
public override encode(
schemaWriteVersion: SchemaCodecVersion,
): TreeNodeSchemaDataFormatV1 | TreeNodeSchemaDataFormatV2 {
const fieldsObject: Record<string, FieldSchemaFormat> = Object.create(null);
// Sort fields to ensure output is identical for for equivalent schema (since field order is not considered significant).
// This makes comparing schema easier, and ensures chunk reuse for schema summaries isn't needlessly broken.
for (const key of [...this.objectNodeFields.keys()].sort()) {
let value: FieldSchemaFormatV1 | FieldSchemaFormatV2 | undefined;

switch (schemaWriteVersion) {
case SchemaCodecVersion.v1:
value = encodeFieldSchemaV1(
this.objectNodeFields.get(key) ?? fail(0xae6 /* missing field */),
);
break;
case SchemaCodecVersion.v2:
value = encodeFieldSchemaV2(
this.objectNodeFields.get(key) ?? fail(0xae7 /* missing field */),
);
break;
default:
fail(`Cannot decode schema version ${schemaWriteVersion}`);
}

Object.defineProperty(fieldsObject, key, {
enumerable: true,
configurable: true,
writable: true,
value: encodeFieldSchema(
this.objectNodeFields.get(key) ?? fail(0xae7 /* missing field */),
),
value,
});
}
return {
Expand Down Expand Up @@ -224,10 +264,23 @@ export class MapNodeStoredSchema extends TreeNodeStoredSchema {
super();
}

public override encode(): TreeNodeSchemaDataFormat {
return {
map: encodeFieldSchema(this.mapFields),
};
public override encode(
schemaWriteVersion: SchemaCodecVersion,
): TreeNodeSchemaDataFormatV1 | TreeNodeSchemaDataFormatV2 {
switch (schemaWriteVersion) {
case SchemaCodecVersion.v1: {
return {
map: encodeFieldSchemaV1(this.mapFields),
};
}
case SchemaCodecVersion.v2: {
return {
map: encodeFieldSchemaV2(this.mapFields),
};
}
default:
fail(`Cannot decode schema version ${schemaWriteVersion}`);
}
}

public override getFieldSchema(field: FieldKey): TreeFieldStoredSchema {
Expand All @@ -254,7 +307,9 @@ export class LeafNodeStoredSchema extends TreeNodeStoredSchema {
super();
}

public override encode(): TreeNodeSchemaDataFormat {
public override encode(
schemaWriteVersion: SchemaCodecVersion,
): TreeNodeSchemaDataFormatV1 | TreeNodeSchemaDataFormatV2 {
return {
leaf: encodeValueSchema(this.leafValue),
};
Expand All @@ -266,7 +321,7 @@ export class LeafNodeStoredSchema extends TreeNodeStoredSchema {
}

export const storedSchemaDecodeDispatcher: DiscriminatedUnionDispatcher<
TreeNodeSchemaDataFormat,
TreeNodeSchemaDataFormatV1,
[],
TreeNodeStoredSchema
> = new DiscriminatedUnionDispatcher({
Expand Down Expand Up @@ -301,11 +356,20 @@ function decodeValueSchema(inMemory: PersistedValueSchema): ValueSchema {
return valueSchemaDecode.get(inMemory) ?? fail(0xae9 /* missing ValueSchema */);
}

export function encodeFieldSchema(schema: TreeFieldStoredSchema): FieldSchemaFormat {
export function encodeFieldSchemaV1(schema: TreeFieldStoredSchema): FieldSchemaFormatV1 {
return {
kind: schema.kind,
// Types are sorted by identifier to improve stability of persisted data to increase chance of schema blob reuse.
types: [...schema.types].sort(),
};
}

export function encodeFieldSchemaV2(schema: TreeFieldStoredSchema): FieldSchemaFormatV2 {
return {
kind: schema.kind,
// Types are sorted by identifier to improve stability of persisted data to increase chance of schema blob reuse.
types: [...schema.types].sort(),
persistedMetadata: schema.persistedMetadata,
};
}

Expand Down
1 change: 0 additions & 1 deletion packages/dds/tree/src/feature-libraries/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ export {
encodeTreeSchema,
makeSchemaCodec,
makeSchemaCodecs,
SchemaCodecVersion,
} from "./schema-index/index.js";
export {
stackTreeNodeCursor,
Expand Down
Loading
Loading