Skip to content

[Inspectorv2] Add metadata properties #16975

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

Merged
merged 9 commits into from
Aug 13, 2025
Original file line number Diff line number Diff line change
@@ -0,0 +1,160 @@
import type { FunctionComponent } from "react";

import type { Nullable } from "core/types";
import { SwitchPropertyLine } from "shared-ui-components/fluent/hoc/propertyLines/switchPropertyLine";
import { TextAreaPropertyLine } from "shared-ui-components/fluent/hoc/propertyLines/textAreaPropertyLine";
import { TextPropertyLine } from "shared-ui-components/fluent/hoc/propertyLines/textPropertyLine";
import { BoundProperty } from "./boundProperty";

enum MetadataTypes {
NULL = "null",
STRING = "string",
OBJECT = "Object",
JSON = "JSON",
}

/**
* Checks if the input is a string.
* @param input - any input to check
* @returns boolean - true if the input is a string, false otherwise
*/
function IsString(input: any): boolean {
return typeof input === "string" || input instanceof String;
}

/**
* Parses a string and returns a JSON object if the string is valid JSON, otherwise returns null
* @param string - any string
* @returns JSON object or null if the string is not valid JSON
*/
// function ParseString(string: string): JSON | null {
// try {
// return JSON.parse(string);
// // eslint-disable-next-line @typescript-eslint/no-unused-vars
// } catch (error) {
// return null;
// }
// }

/**
* Checks recursively for functions on an object and returns `false` if any are found.
* @param o any object, string or number
* @returns boolean
*/
function ObjectCanSafelyStringify(o: object | string | number | boolean): boolean {
if (typeof o === "function") {
return false;
}
if (o === null || o === true || o === false || typeof o === "number" || IsString(o)) {
return true;
}

if (typeof o === "object") {
if (Object.values(o).length === 0) {
return true;
}
return Object.values(o as Record<string, any>).every((value) => ObjectCanSafelyStringify(value));
}

if (Array.isArray(o)) {
return o.every((value) => ObjectCanSafelyStringify(value));
}

return false;
}

export interface IMetadataContainer {
metadata: any;
}

class MetadataUtils {
private _editedMetadata: Nullable<string> = null;

static PrettyJSON = false;

static PreventObjectCorruption = true;

constructor(public readonly entity: IMetadataContainer) {}

get editedMetadata(): string {
if (!this._editedMetadata) {
this._editedMetadata = this.parsedMetadata;
}
return this._editedMetadata;
}

set editedMetadata(value: string) {
this._editedMetadata = value;
}

get entityType(): MetadataTypes {
const metadata = this.entity.metadata;

if (IsString(metadata)) {
return MetadataTypes.STRING;
}
if (metadata === null) {
return MetadataTypes.NULL;
}
if (!ObjectCanSafelyStringify(metadata)) {
return MetadataTypes.OBJECT;
}

return MetadataTypes.JSON;
}

/**
* @returns whether the entity's metadata can be parsed as JSON.
*/
get isParsable(): boolean {
if (!this.entity.metadata) {
return false;
}

try {
return !!JSON.parse(JSON.stringify(this.entity.metadata));
} catch (error) {
return false;
}
}

get isReadonly(): boolean {
return this.entityType === MetadataTypes.OBJECT && MetadataUtils.PreventObjectCorruption;
}

get parsedMetadata(): string {
const metadata = this.entity.metadata;

if (this.isParsable) {
return JSON.stringify(metadata, undefined, MetadataUtils.PrettyJSON ? 2 : undefined);
}

if (IsString(metadata)) {
return metadata;
}

return String(metadata);
}
}

/**
* Component to display metadata properties of an entity.
* @param props - The properties for the component.
* @returns A React component that displays metadata properties.
*/
export const MetadataProperties: FunctionComponent<{ entity: IMetadataContainer }> = (props) => {
const { entity } = props;

const metadataUtils = new MetadataUtils(entity);

return (
<>
<BoundProperty component={TextPropertyLine} label={"Property type"} target={metadataUtils} propertyKey="entityType" />
<BoundProperty component={SwitchPropertyLine} label={"Prevent Object corruption"} target={MetadataUtils} propertyKey="PreventObjectCorruption" />
<BoundProperty component={SwitchPropertyLine} label={"Pretty JSON"} target={MetadataUtils} propertyKey="PrettyJSON" />
<BoundProperty disabled={metadataUtils.isReadonly} component={TextAreaPropertyLine} label={"Data"} target={metadataUtils} propertyKey="editedMetadata" />
{/* TODO: Update component when settings change. Toggling PrettyJSON and PreventObjectCorruption should update the text area, but they don't right now. */}
{/* TODO: Add buttons. See metadataPropertyGridComponent.tsx for v1 implementation. */}
</>
);
};
1 change: 1 addition & 0 deletions packages/dev/inspector-v2/src/inspector.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ import { FrameGraphPropertiesServiceDefinition } from "./services/panes/properti
import { LightPropertiesServiceDefinition } from "./services/panes/properties/lightPropertiesServices";
import { MaterialPropertiesServiceDefinition } from "./services/panes/properties/materialPropertiesService";
import { NodePropertiesServiceDefinition } from "./services/panes/properties/nodePropertiesService";
import { MetadataPropertiesServiceDefinition } from "./services/panes/properties/metadataPropertiesService";
import { ParticleSystemPropertiesServiceDefinition } from "./services/panes/properties/particleSystemPropertiesService";
import { PhysicsPropertiesServiceDefinition } from "./services/panes/properties/physicsPropertiesService";
import { PostProcessPropertiesServiceDefinition } from "./services/panes/properties/postProcessPropertiesService";
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
import type { IMetadataContainer } from "../../../components/properties/metadataProperties";
import type { ServiceDefinition } from "../../../modularity/serviceDefinition";
import type { IPropertiesService } from "./propertiesService";

import { MetadataProperties } from "../../../components/properties/metadataProperties";
import { PropertiesServiceIdentity } from "./propertiesService";

function IsMetadataContainer(entity: unknown): entity is IMetadataContainer {
return (entity as IMetadataContainer).metadata !== undefined;
}

export const MetadataPropertiesServiceDefinition: ServiceDefinition<[], [IPropertiesService]> = {
friendlyName: "Metadata Properties",
consumes: [PropertiesServiceIdentity],
factory: (propertiesService) => {
const contentRegistration = propertiesService.addSectionContent({
key: "Metadata Properties",
// TransformNode and Bone don't share a common base class, but both have the same transform related properties.
predicate: (entity: unknown) => IsMetadataContainer(entity),
content: [
{
section: "Metadata",
component: ({ context }) => <MetadataProperties entity={context} />,
},
],
});

return {
dispose: () => {
contentRegistration.dispose();
},
};
},
};
37 changes: 37 additions & 0 deletions packages/dev/inspector-v2/test/app/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,42 @@ function createTestBoxes() {
boxInstance.position = new Vector3(0, 0, -0.5);
}

function createTestMetadata() {
const materialMeta = new StandardMaterial("material.meta", scene);
materialMeta.emissiveColor = Color3.Red();
materialMeta.metadata = {
test: "test string",
description: "Material JSON metadata.",
someNumber: 73,
};

const defaultMeta = MeshBuilder.CreateBox("default.metadata", { size: 0.15 }, scene);

const undefinedMeta = defaultMeta.clone("undefined.metadata");
undefinedMeta.material = materialMeta;
undefinedMeta.metadata = undefined;

const jsonMeta = defaultMeta.clone("json.metadata");
jsonMeta.material = materialMeta;
jsonMeta.metadata = {
test: "test string",
description: "JSON metadata.",
someNumber: 42,
};

const nullMeta = defaultMeta.clone("null.metadata");
nullMeta.material = materialMeta;
nullMeta.metadata = null;

const stringMeta = defaultMeta.clone("string.metadata");
stringMeta.material = materialMeta;
stringMeta.metadata = "String metadata.";

const objectMeta = defaultMeta.clone("object.metadata");
objectMeta.material = materialMeta;
objectMeta.metadata = jsonMeta;
}

(async () => {
let assetContainer = await LoadAssetContainerAsync("https://assets.babylonjs.com/meshes/Demos/optimized/acrobaticPlane_variants.glb", scene);
assetContainer.addAllToScene();
Expand All @@ -84,6 +120,7 @@ function createTestBoxes() {
await createPhysics();

createTestBoxes();
createTestMetadata();

engine.runRenderLoop(() => {
scene.render();
Expand Down
Loading