@forgeax/engine-geometry 0.1.28 → 0.1.30

package/dist/vertex-attribute-layout.d.ts

@@ -25,6 +25,10 @@ export interface GpuVertexBufferLayoutEntry {
     }>;
     readonly stepMode?: 'vertex' | 'instance' | undefined;
 }
+/** Canonical position/normal/uv/tangent map for ordinary interleaved meshes. */
+export declare const DEFAULT_VERTEX_ATTRIBUTE_MAP: VertexAttributeMap;
+/** Canonical six-attribute map for skinned interleaved meshes. */
+export declare const SKIN_VERTEX_ATTRIBUTE_MAP: VertexAttributeMap;
 /**
  * Build a non-skin `VertexAttributeMap` carrying exactly `uvSetCount` UV sets
  * (set 0 = `uv`, then `uv1..uv{uvSetCount-1}`). `deriveVertexBufferLayout`
@@ -61,6 +65,16 @@ export interface VertexLayoutProjection {
     readonly arrayStride: number;
     readonly digest: string;
 }
+/**
+ * Derive a vertex count from the geometry-owned projection.
+ *
+ * Every upload, morph write, and draw range must use this same byte-level
+ * calculation. Returning `undefined` keeps malformed payloads out of GPU
+ * resource construction instead of silently truncating a fractional count.
+ */
+export declare function deriveVertexCount(vertices: {
+    readonly byteLength: number;
+}, projection: VertexLayoutProjection): number | undefined;
 /** Derive the one canonical immutable layout projection for a vertex map. */
 export declare function deriveVertexLayoutProjection(map: VertexAttributeMap): VertexLayoutProjection;
 export interface PackedVertexAttributes {

package/dist/vertex-attribute-layout.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"vertex-attribute-layout.d.ts","sourceRoot":"","sources":["../src/vertex-attribute-layout.ts"],"names":[],"mappings":"AA+BA,OAAO,EAEL,UAAU,EAIV,KAAK,MAAM,EACX,KAAK,kBAAkB,EACvB,KAAK,yBAAyB,EAE/B,MAAM,uBAAuB,CAAC;AAE/B,QAAA,MAAM,oBAAoB;;;;;;;;;;;;;;;CAehB,CAAC;AAEX,KAAK,YAAY,GAAG,MAAM,OAAO,oBAAoB,CAAC;AAmBtD,MAAM,WAAW,0BAA0B;IACzC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,UAAU,EAAE,aAAa,CAAC;QACjC,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAC;QAChC,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;QACxB,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;KACzB,CAAC,CAAC;IACH,QAAQ,CAAC,QAAQ,CAAC,EAAE,QAAQ,GAAG,UAAU,GAAG,SAAS,CAAC;CACvD;AAuCD;;;;;;;;;;;;GAYG;AACH,wBAAgB,8BAA8B,CAAC,UAAU,EAAE,MAAM,GAAG,kBAAkB,CAYrF;AAED,wBAAgB,wBAAwB,CACtC,GAAG,EAAE,kBAAkB,EACvB,IAAI,CAAC,EAAE;IAAE,gBAAgB,CAAC,EAAE,MAAM,CAAA;CAAE,GACnC,0BAA0B,EAAE,CAsD9B;AAED,4FAA4F;AAC5F,wBAAgB,sCAAsC,CACpD,UAAU,EAAE,sBAAsB,EAClC,IAAI,CAAC,EAAE;IAAE,gBAAgB,CAAC,EAAE,MAAM,CAAA;CAAE,GACnC,0BAA0B,EAAE,CAgD9B;AAED,MAAM,WAAW,+BAA+B;IAC9C,QAAQ,CAAC,GAAG,EAAE,YAAY,CAAC;IAC3B,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAC;IAChC,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,MAAM,EAAE,CAAC,OAAO,oBAAoB,CAAC,CAAC,YAAY,CAAC,CAAC;IAC7D,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;CAC7B;AAED,iFAAiF;AACjF,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,aAAa,EAAE,CAAC,CAAC;IAC1B,QAAQ,CAAC,UAAU,EAAE,SAAS,+BAA+B,EAAE,CAAC;IAChE,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;CACzB;AAmCD,6EAA6E;AAC7E,wBAAgB,4BAA4B,CAAC,GAAG,EAAE,kBAAkB,GAAG,sBAAsB,CAkC5F;AAED,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,UAAU,EAAE,sBAAsB,CAAC;IAC5C,QAAQ,CAAC,QAAQ,EAAE,YAAY,CAAC;CACjC;AAED,qBAAa,wBAAyB,SAAQ,UAAU;IACtD,SAAiB,MAAM,EAAE,yBAAyB,CAAC;gBAEvC,MAAM,EAAE,yBAAyB;CAQ9C;AAED,MAAM,WAAW,+BAA+B;IAC9C,QAAQ,CAAC,IAAI,EAAE,4BAA4B,CAAC;IAC5C,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,MAAM,EAAE;QACf,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;QACtB,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;QAC3B,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;QAC7B,QAAQ,CAAC,MAAM,EAAE,OAAO,GAAG,cAAc,CAAC;KAC3C,CAAC;CACH;AAED,wFAAwF;AACxF,wBAAgB,oCAAoC,CAClD,IAAI,EAAE,MAAM,GACX,MAAM,CAAC,sBAAsB,EAAE,+BAA+B,CAAC,CA4BjE;AAED,uFAAuF;AACvF,wBAAgB,+BAA+B,CAC7C,GAAG,EAAE,kBAAkB,EACvB,WAAW,EAAE,MAAM,GAClB,MAAM,CAAC,sBAAsB,EAAE,wBAAwB,CAAC,CAkF1D"}
+{"version":3,"file":"vertex-attribute-layout.d.ts","sourceRoot":"","sources":["../src/vertex-attribute-layout.ts"],"names":[],"mappings":"AA+BA,OAAO,EAEL,UAAU,EAGV,KAAK,MAAM,EACX,KAAK,kBAAkB,EACvB,KAAK,yBAAyB,EAE/B,MAAM,uBAAuB,CAAC;AAE/B,QAAA,MAAM,oBAAoB;;;;;;;;;;;;;;;CAehB,CAAC;AAEX,KAAK,YAAY,GAAG,MAAM,OAAO,oBAAoB,CAAC;AAmBtD,MAAM,WAAW,0BAA0B;IACzC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,UAAU,EAAE,aAAa,CAAC;QACjC,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAC;QAChC,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;QACxB,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;KACzB,CAAC,CAAC;IACH,QAAQ,CAAC,QAAQ,CAAC,EAAE,QAAQ,GAAG,UAAU,GAAG,SAAS,CAAC;CACvD;AAgBD,gFAAgF;AAChF,eAAO,MAAM,4BAA4B,EAAE,kBAKzC,CAAC;AAEH,kEAAkE;AAClE,eAAO,MAAM,yBAAyB,EAAE,kBAItC,CAAC;AA4BH;;;;;;;;;;;;GAYG;AACH,wBAAgB,8BAA8B,CAAC,UAAU,EAAE,MAAM,GAAG,kBAAkB,CAOrF;AAED,wBAAgB,wBAAwB,CACtC,GAAG,EAAE,kBAAkB,EACvB,IAAI,CAAC,EAAE;IAAE,gBAAgB,CAAC,EAAE,MAAM,CAAA;CAAE,GACnC,0BAA0B,EAAE,CAwC9B;AAED,4FAA4F;AAC5F,wBAAgB,sCAAsC,CACpD,UAAU,EAAE,sBAAsB,EAClC,IAAI,CAAC,EAAE;IAAE,gBAAgB,CAAC,EAAE,MAAM,CAAA;CAAE,GACnC,0BAA0B,EAAE,CAyB9B;AAED,MAAM,WAAW,+BAA+B;IAC9C,QAAQ,CAAC,GAAG,EAAE,YAAY,CAAC;IAC3B,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAC;IAChC,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,MAAM,EAAE,CAAC,OAAO,oBAAoB,CAAC,CAAC,YAAY,CAAC,CAAC;IAC7D,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;CAC7B;AAED,iFAAiF;AACjF,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,aAAa,EAAE,CAAC,CAAC;IAC1B,QAAQ,CAAC,UAAU,EAAE,SAAS,+BAA+B,EAAE,CAAC;IAChE,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;CACzB;AAED;;;;;;GAMG;AACH,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE;IAAE,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAA;CAAE,EACzC,UAAU,EAAE,sBAAsB,GACjC,MAAM,GAAG,SAAS,CAcpB;AAmCD,6EAA6E;AAC7E,wBAAgB,4BAA4B,CAAC,GAAG,EAAE,kBAAkB,GAAG,sBAAsB,CAkC5F;AAED,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,UAAU,EAAE,sBAAsB,CAAC;IAC5C,QAAQ,CAAC,QAAQ,EAAE,YAAY,CAAC;CACjC;AAED,qBAAa,wBAAyB,SAAQ,UAAU;IACtD,SAAiB,MAAM,EAAE,yBAAyB,CAAC;gBAEvC,MAAM,EAAE,yBAAyB;CAQ9C;AAED,MAAM,WAAW,+BAA+B;IAC9C,QAAQ,CAAC,IAAI,EAAE,4BAA4B,CAAC;IAC5C,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,MAAM,EAAE;QACf,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;QACtB,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;QAC3B,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;QAC7B,QAAQ,CAAC,MAAM,EAAE,OAAO,GAAG,cAAc,CAAC;KAC3C,CAAC;CACH;AAED,wFAAwF;AACxF,wBAAgB,oCAAoC,CAClD,IAAI,EAAE,MAAM,GACX,MAAM,CAAC,sBAAsB,EAAE,+BAA+B,CAAC,CA4BjE;AAED,uFAAuF;AACvF,wBAAgB,+BAA+B,CAC7C,GAAG,EAAE,kBAAkB,EACvB,WAAW,EAAE,MAAM,GAClB,MAAM,CAAC,sBAAsB,EAAE,wBAAwB,CAAC,CAkF1D"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forgeax/engine-geometry",
-  "version": "0.1.28",
+  "version": "0.1.30",
   "private": false,
   "type": "module",
   "license": "Apache-2.0",
@@ -22,10 +22,10 @@
     "LICENSE"
   ],
   "dependencies": {
-    "@forgeax/engine-ecs": "0.1.28",
-    "@forgeax/engine-math": "0.1.28",
-    "@forgeax/engine-pack": "0.1.28",
-    "@forgeax/engine-types": "0.1.28"
+    "@forgeax/engine-ecs": "0.1.30",
+    "@forgeax/engine-math": "0.1.30",
+    "@forgeax/engine-pack": "0.1.30",
+    "@forgeax/engine-types": "0.1.30"
   },
   "forgeax": {
     "metrics": {

package/src/__tests__/vertex-attribute-layout-owner.unit.test.ts

@@ -1,9 +1,13 @@
 import {
   buildMeshAttributeMapForUvSets,
+  DEFAULT_VERTEX_ATTRIBUTE_MAP,
   deriveVertexBufferLayout,
+  deriveVertexBufferLayoutFromProjection,
+  deriveVertexCount,
   deriveVertexLayoutProjection,
   deriveVertexLayoutProjectionFromMask,
   packInterleavedVertexAttributes,
+  SKIN_VERTEX_ATTRIBUTE_MAP,
 } from '@forgeax/engine-geometry';
 import type { VertexAttributeMap } from '@forgeax/engine-types';
 import { describe, expect, it } from 'vitest';
@@ -29,6 +33,34 @@ function makeCompleteAttributeMap(): VertexAttributeMap {
 }
 
 describe('vertex attribute layout owner', () => {
+  it('publishes the ordinary and skinned maps from the same geometry owner', () => {
+    expect(Object.keys(DEFAULT_VERTEX_ATTRIBUTE_MAP)).toEqual([
+      'position',
+      'normal',
+      'uv',
+      'tangent',
+    ]);
+    expect(Object.keys(SKIN_VERTEX_ATTRIBUTE_MAP)).toEqual([
+      'position',
+      'normal',
+      'uv',
+      'tangent',
+      'skinIndex',
+      'skinWeight',
+    ]);
+    expect(deriveVertexLayoutProjection(DEFAULT_VERTEX_ATTRIBUTE_MAP).arrayStride).toBe(48);
+    expect(deriveVertexLayoutProjection(SKIN_VERTEX_ATTRIBUTE_MAP).arrayStride).toBe(72);
+  });
+
+  it('derives counts only when vertex bytes match the projected stride', () => {
+    const plain = deriveVertexLayoutProjection(DEFAULT_VERTEX_ATTRIBUTE_MAP);
+    const skinned = deriveVertexLayoutProjection(SKIN_VERTEX_ATTRIBUTE_MAP);
+    expect(deriveVertexCount(new Float32Array(24), plain)).toBe(2);
+    expect(deriveVertexCount(new Float32Array(18), skinned)).toBe(1);
+    expect(deriveVertexCount(new Float32Array(23), plain)).toBeUndefined();
+    expect(deriveVertexCount(new Float32Array(17), skinned)).toBeUndefined();
+  });
+
   it('derives the complete 13-key layout from the format owner', () => {
     expect(deriveVertexBufferLayout(makeCompleteAttributeMap())).toEqual([
       {
@@ -178,4 +210,23 @@ describe('vertex attribute layout owner', () => {
       },
     ]);
   });
+
+  it('aliases sparse UV holes without duplicating authored shader locations', () => {
+    const map = { position: makeBuffer(), uv: makeBuffer(), uv7: makeBuffer() };
+    const projection = deriveVertexLayoutProjection(map);
+    const fromMap = deriveVertexBufferLayout(map, { shaderUvSetCount: 8 });
+    const fromProjection = deriveVertexBufferLayoutFromProjection(projection, {
+      shaderUvSetCount: 8,
+    });
+    expect(fromMap).toEqual(fromProjection);
+    const layout = fromProjection[0];
+    expect(layout?.arrayStride).toBe(28);
+    expect(layout?.attributes).toHaveLength(9);
+    expect(new Set(layout?.attributes.map((entry) => entry.shaderLocation)).size).toBe(9);
+    expect(
+      layout?.attributes
+        .filter((entry) => entry.shaderLocation >= 6)
+        .every((entry) => entry.offset === 20),
+    ).toBe(true);
+  });
 });

package/src/index.ts

@@ -56,13 +56,16 @@ export {
 export { createTorusGeometry } from './torus';
 export {
   buildMeshAttributeMapForUvSets,
+  DEFAULT_VERTEX_ATTRIBUTE_MAP,
   deriveVertexBufferLayout,
   deriveVertexBufferLayoutFromProjection,
+  deriveVertexCount,
   deriveVertexLayoutProjection,
   deriveVertexLayoutProjectionFromMask,
   type GpuVertexBufferLayoutEntry,
   type PackedVertexAttributes,
   packInterleavedVertexAttributes,
+  SKIN_VERTEX_ATTRIBUTE_MAP,
   VertexAttributePackError,
   type VertexLayoutProjection,
   type VertexLayoutProjectionAttribute,

package/src/vertex-attribute-layout.ts

@@ -32,7 +32,6 @@
 import {
   ASSET_ERROR_HINTS,
   AssetError,
-  countUvSets,
   err,
   ok,
   type Result,
@@ -95,6 +94,27 @@ const UV_KEYS: readonly AttributeKey[] = CANONICAL_KEYS.filter(
   (key) => key === 'uv' || key.startsWith('uv'),
 );
 
+// Empty typed-array values are presence sentinels for layout derivation. Keep
+// the common mesh maps here so renderers, recovery, and points/lines cannot
+// grow a second copy of the canonical attribute vocabulary.
+const EMPTY_FLOAT32 = new Float32Array(0);
+const EMPTY_UINT16 = new Uint16Array(0);
+
+/** Canonical position/normal/uv/tangent map for ordinary interleaved meshes. */
+export const DEFAULT_VERTEX_ATTRIBUTE_MAP: VertexAttributeMap = Object.freeze({
+  position: EMPTY_FLOAT32,
+  normal: EMPTY_FLOAT32,
+  uv: EMPTY_FLOAT32,
+  tangent: EMPTY_FLOAT32,
+});
+
+/** Canonical six-attribute map for skinned interleaved meshes. */
+export const SKIN_VERTEX_ATTRIBUTE_MAP: VertexAttributeMap = Object.freeze({
+  ...DEFAULT_VERTEX_ATTRIBUTE_MAP,
+  skinIndex: EMPTY_UINT16,
+  skinWeight: EMPTY_FLOAT32,
+});
+
 type Entry = {
   readonly key: AttributeKey;
   readonly shaderLocation: number;
@@ -102,16 +122,13 @@ type Entry = {
   readonly format: string;
 };
 
-function emitAliasEntries(
-  entries: Entry[],
-  fromIndex: number,
-  toIndex: number,
-  aliasOffset: number,
-  currentStride: number,
-): number {
-  for (let k = fromIndex; k < toIndex && k < UV_KEYS.length; k++) {
+function emitAliasEntries(entries: Entry[], toIndex: number, currentStride: number): number {
+  const lastUv = entries.filter((entry) => UV_KEYS.includes(entry.key)).at(-1);
+  const aliasOffset = lastUv?.offset ?? currentStride;
+  for (let k = 0; k < toIndex && k < UV_KEYS.length; k++) {
     // biome-ignore lint/style/noNonNullAssertion: bounded index on const array
     const uvKey = UV_KEYS[k]!;
+    if (entries.some((entry) => entry.key === uvKey)) continue;
     entries.push({
       key: uvKey,
       shaderLocation: CANONICAL_KEYS.indexOf(uvKey),
@@ -121,7 +138,7 @@ function emitAliasEntries(
   }
   // When meshUvSetCount===0, allocate 8 bytes for the zero UV area.
   // Otherwise no stride increase (aliased to existing offset).
-  return fromIndex === 0 ? currentStride + ATTRIBUTE_BYTE_STRIDE.uv : currentStride;
+  return lastUv === undefined ? currentStride + ATTRIBUTE_BYTE_STRIDE.uv : currentStride;
 }
 
 /**
@@ -138,12 +155,7 @@ function emitAliasEntries(
  * off-screen). uvSetCount <= 1 yields the canonical 4-attribute single-UV map.
  */
 export function buildMeshAttributeMapForUvSets(uvSetCount: number): VertexAttributeMap {
-  const map: Record<string, Float32Array> = {
-    position: new Float32Array(0),
-    normal: new Float32Array(0),
-    uv: new Float32Array(0),
-    tangent: new Float32Array(0),
-  };
+  const map = { ...DEFAULT_VERTEX_ATTRIBUTE_MAP } as unknown as Record<string, Float32Array>;
   for (let set = 1; set < uvSetCount && set < UV_KEYS.length; set++) {
     // biome-ignore lint/style/noNonNullAssertion: bounded index on const array
     map[UV_KEYS[set]!] = new Float32Array(0);
@@ -176,21 +188,7 @@ export function deriveVertexBufferLayout(
 
   // ── clamp-to-last alias (plan-strategy D-1) ──
   if (shaderUvSetCount > 0) {
-    const meshUvSetCount = countUvSets(map);
-
-    if (shaderUvSetCount > meshUvSetCount) {
-      const lastUvIndex = meshUvSetCount - 1;
-      const lastUvKey =
-        lastUvIndex >= 0 && lastUvIndex < UV_KEYS.length ? UV_KEYS[lastUvIndex] : undefined;
-      const aliasOffset = lastUvKey !== undefined ? CANONICAL_KEYS.indexOf(lastUvKey) : -1;
-
-      const aliasByteOffset =
-        aliasOffset >= 0
-          ? (entries.find((e) => e.shaderLocation === aliasOffset)?.offset ?? 0)
-          : offset;
-
-      offset = emitAliasEntries(entries, meshUvSetCount, shaderUvSetCount, aliasByteOffset, offset);
-    }
+    offset = emitAliasEntries(entries, shaderUvSetCount, offset);
   }
 
   if (present === 0 && shaderUvSetCount === 0) return [];
@@ -224,30 +222,7 @@ export function deriveVertexBufferLayoutFromProjection(
   let arrayStride = projection.arrayStride;
   const shaderUvSetCount = opts?.shaderUvSetCount ?? 0;
   if (shaderUvSetCount > 0) {
-    const meshUvSetCount = entries.filter(
-      (entry) => entry.key === 'uv' || entry.key.startsWith('uv'),
-    ).length;
-    if (shaderUvSetCount > meshUvSetCount) {
-      const lastUv = entries
-        .filter((entry) => entry.key === 'uv' || entry.key.startsWith('uv'))
-        .at(-1);
-      const aliasOffset = lastUv?.offset ?? arrayStride;
-      for (
-        let index = meshUvSetCount;
-        index < shaderUvSetCount && index < UV_KEYS.length;
-        index += 1
-      ) {
-        const key = UV_KEYS[index];
-        if (key === undefined) continue;
-        entries.push({
-          key,
-          shaderLocation: CANONICAL_KEYS.indexOf(key),
-          offset: aliasOffset,
-          format: ATTRIBUTE_FORMAT_MAP[key],
-        });
-      }
-      if (meshUvSetCount === 0) arrayStride += ATTRIBUTE_BYTE_STRIDE.uv;
-    }
+    arrayStride = emitAliasEntries(entries, shaderUvSetCount, arrayStride);
   }
   entries.sort((a, b) => a.shaderLocation - b.shaderLocation);
   return projection.attributes.length === 0
@@ -281,6 +256,32 @@ export interface VertexLayoutProjection {
   readonly digest: string;
 }
 
+/**
+ * Derive a vertex count from the geometry-owned projection.
+ *
+ * Every upload, morph write, and draw range must use this same byte-level
+ * calculation. Returning `undefined` keeps malformed payloads out of GPU
+ * resource construction instead of silently truncating a fractional count.
+ */
+export function deriveVertexCount(
+  vertices: { readonly byteLength: number },
+  projection: VertexLayoutProjection,
+): number | undefined {
+  const stride = projection.arrayStride;
+  const byteLength = vertices.byteLength;
+  if (
+    !Number.isSafeInteger(stride) ||
+    stride <= 0 ||
+    !Number.isSafeInteger(byteLength) ||
+    byteLength < 0 ||
+    byteLength % stride !== 0
+  ) {
+    return undefined;
+  }
+  const count = byteLength / stride;
+  return Number.isSafeInteger(count) ? count : undefined;
+}
+
 function bytesForFormat(format: string): number {
   if (format === 'uint16x4' || format === 'float32x2') return 8;
   if (format === 'float32x3') return 12;