Refactorings, tidy ups

Author: xeolabs

packages/sdk/src/inspect/sceneModel/Config.ts

@@ -18,7 +18,7 @@ import {clamp} from "../../base/math";
  * The result is a flat keyed bag the inspection's `run` consumes
  * in place of pulling fields off `params` directly.
  *
- * A Studio "Inspection settings" panel can walk every
+ * A host-supplied "Inspection settings" UI can walk every
  * registered inspection's {@link ConfigSchema} and render a form
  * from the field metadata — labels, units, min/max, enumerations
  * — without any per-inspection UI code. Plugins inherit that UI

packages/sdk/src/inspect/sceneModel/DEFAULT_FIX_REGISTRY.ts

@@ -24,13 +24,13 @@ import {dropDuplicateTriangles} from "./fixes/dropDuplicateTriangles";
  * The {@link FixRegistry} {@link applyFixes} reaches for
  * when no `registry` is supplied in its params. Pre-populated with
  * the built-in strategies that ship from
- * {@link demo/sceneModelInspector/fixes}.
+ * {@link studio/sceneModelInspector/fixes}.
  *
  * Plugins are expected to register additional strategies into this
  * singleton on import:
  *
  * ```ts
- * import {DEFAULT_FIX_REGISTRY} from "@xeokit/sdk/studio";
+ * import {DEFAULT_FIX_REGISTRY} from "@xeokit/sdk/inspect/sceneModel";
  *
  * DEFAULT_FIX_REGISTRY.register({
  *   codes: ["MyApp/STALE_PROPERTY_SET"],

packages/sdk/src/inspect/sceneModel/DEFAULT_INSPECTION_REGISTRY.ts

@@ -20,7 +20,7 @@ import {farFromOriginGeometries} from "./inspections/farFromOriginGeometries";
  * The {@link InspectionRegistry} {@link inspectSceneModel} reaches
  * for when no `registry` is supplied in its params. Pre-populated
  * with the built-in inspections that ship from
- * {@link demo/sceneModelInspector/inspections}.
+ * {@link studio/sceneModelInspector/inspections}.
  *
  * Registration order:
  *
@@ -44,7 +44,7 @@ import {farFromOriginGeometries} from "./inspections/farFromOriginGeometries";
  * singleton on import:
  *
  * ```ts
- * import {DEFAULT_INSPECTION_REGISTRY} from "@xeokit/sdk/studio";
+ * import {DEFAULT_INSPECTION_REGISTRY} from "@xeokit/sdk/inspect/sceneModel";
  *
  * DEFAULT_INSPECTION_REGISTRY.register({
  *   codes: ["MyApp/CHECK_NAMING"],

packages/sdk/src/inspect/sceneModel/fixes/index.ts

@@ -11,7 +11,7 @@
  *
  * ```ts
  * // Mutate the singleton — applies to every applyFixes() call.
- * import {DEFAULT_FIX_REGISTRY, Fix} from "@xeokit/sdk/studio";
+ * import {DEFAULT_FIX_REGISTRY, Fix} from "@xeokit/sdk/inspect/sceneModel";
  *
  * const myFix: Fix = {
  *   codes: ["MY_CUSTOM_CODE"],
@@ -23,7 +23,7 @@
  *
  * ```ts
  * // Or build a fresh registry — leaves the default untouched.
- * import {FixRegistry, mergeDuplicateGeometries, applyFixes} from "@xeokit/sdk/studio";
+ * import {FixRegistry, mergeDuplicateGeometries, applyFixes} from "@xeokit/sdk/inspect/sceneModel";
  *
  * const registry = new FixRegistry();
  * registry.register(mergeDuplicateGeometries);   // pick what you want

packages/sdk/src/inspect/sceneModel/index.ts

@@ -168,7 +168,7 @@
  * ```ts
  * // Plug a strategy into the singleton — every applyFixes() call
  * // sees it from now on.
- * import {sceneModelInspector} from "@xeokit/sdk/studio";
+ * import * as sceneModelInspector from "@xeokit/sdk/inspect/sceneModel";
  *
  * sceneModelInspector.DEFAULT_FIX_REGISTRY.register({
  *   codes: ["MyApp/STALE_PROPERTY_SET"],
@@ -183,16 +183,16 @@
  * ```ts
  * // Or build a one-off registry — leaves the default untouched.
  * import {
- *   sceneModelInspector,
  *   FixRegistry,
  *   mergeDuplicateGeometries,
- * } from "@xeokit/sdk/studio";
+ *   applyFixes,
+ * } from "@xeokit/sdk/inspect/sceneModel";
  *
  * const registry = new FixRegistry([
  *   mergeDuplicateGeometries,   // pick the built-ins you want
  *   myFix,
  * ]);
- * sceneModelInspector.applyFixes({sceneModel, report, registry});
+ * applyFixes({sceneModel, report, registry});
  * ```
  *
  * Last registration wins for a given code, so plugins can
@@ -202,7 +202,7 @@
  * ## Built-in inspections
  *
  * One file per inspection under
- * {@link demo/sceneModelInspector/inspections}, all pre-registered into
+ * {@link studio/sceneModelInspector/inspections}, all pre-registered into
  * {@link DEFAULT_INSPECTION_REGISTRY}. Each inspection groups
  * codes by topical concern — one walk, multiple codes — so the
  * file count stays low and the registration order is meaningful.
@@ -226,7 +226,7 @@
  *
  * ## Built-in fixes
  *
- * One file per fix under {@link demo/sceneModelInspector/fixes}, all
+ * One file per fix under {@link studio/sceneModelInspector/fixes}, all
  * pre-registered into {@link DEFAULT_FIX_REGISTRY}.
  *
  * | Fix                              | Codes handled                                                  |
@@ -263,7 +263,7 @@
  * ## Putting it together
  *
  * ```ts
- * import {sceneModelInspector} from "@xeokit/sdk/studio";
+ * import * as sceneModelInspector from "@xeokit/sdk/inspect/sceneModel";
  *
  * const report = sceneModelInspector.inspectSceneModel({
  *   sceneModel,

packages/sdk/src/inspect/sceneModel/inspectSceneModel.ts

@@ -30,7 +30,7 @@ import {getInspectionIndex} from "./internal/getInspectionIndex";
  * ## Custom inspections
  *
  * ```ts
- * import {DEFAULT_INSPECTION_REGISTRY} from "@xeokit/sdk/studio";
+ * import {DEFAULT_INSPECTION_REGISTRY} from "@xeokit/sdk/inspect/sceneModel";
  *
  * DEFAULT_INSPECTION_REGISTRY.register({
  *   codes: ["MyApp/CHECK_NAMING"],
@@ -44,7 +44,7 @@ import {getInspectionIndex} from "./internal/getInspectionIndex";
  * Or for tests / isolated runs, build a fresh registry:
  *
  * ```ts
- * import {InspectionRegistry, geometryDataIntegrity, inspectSceneModel} from "@xeokit/sdk/studio";
+ * import {InspectionRegistry, geometryDataIntegrity, inspectSceneModel} from "@xeokit/sdk/inspect/sceneModel";
  *
  * const registry = new InspectionRegistry([geometryDataIntegrity]);
  * const report = inspectSceneModel({sceneModel, registry});

packages/sdk/src/inspect/sceneModel/inspections/index.ts

@@ -13,7 +13,7 @@
  *
  * ```ts
  * // Mutate the singleton — applies to every inspectSceneModel() call.
- * import {DEFAULT_INSPECTION_REGISTRY, Inspection} from "@xeokit/sdk/studio";
+ * import {DEFAULT_INSPECTION_REGISTRY, Inspection} from "@xeokit/sdk/inspect/sceneModel";
  *
  * const myInspection: Inspection = {
  *   codes: ["MyApp/STALE_PROPERTY_SET"],
@@ -25,7 +25,7 @@
  *
  * ```ts
  * // Or build a fresh registry — leaves the default untouched.
- * import {InspectionRegistry, geometryDataIntegrity, inspectSceneModel} from "@xeokit/sdk/studio";
+ * import {InspectionRegistry, geometryDataIntegrity, inspectSceneModel} from "@xeokit/sdk/inspect/sceneModel";
  *
  * const registry = new InspectionRegistry();
  * registry.register(geometryDataIntegrity);   // pick what you want

packages/sdk/src/inspect/sceneModel/internal/splitGeometryAndRebuildMeshes.ts

@@ -8,7 +8,7 @@ import type {FixApplyResult} from "../Fix";
  * Internal helper shared by every fix strategy that needs to split
  * a {@link model!scene.SceneGeometry | SceneGeometry} in two and fan referencing meshes onto
  * the resulting pieces. Same supported pattern
- * {@link applyIFCMaterials} / {@link demo.materials.MaterialsPalette}
+ * {@link applyIFCMaterials} / {@link studio.materials.MaterialsPalette}
  * use: detach + destroy + recreate + reattach, with a snapshot of
  * each mesh's id / matrix / opacity / parentTransform preserved
  * across the rebuild.