@needle-tools/gltf-build-pipeline 1.1.4 → 1.1.6

package/CHANGELOG.md

@@ -4,6 +4,9 @@ All notable changes to this package will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [1.1.5] - 2023-06-26
+- change: bump gltf-transform dependency to latest 3.4.3
+
 ## [1.1.4] - 2023-06-09
 - fix: progressive texture transform didnt pass mesh compression setting through
 

package/dist/extensions/NEEDLE_opaque/NEEDLE_opaque.d.ts

@@ -10,6 +10,10 @@ export interface IOpaqueExtension {
 export interface IOpaqueProperty {
     /** used by progressive texture extension to pass an extension between documents */
     getExtensionDefinition(): object;
+    /** register a reference so gltf-transform will not remove it in e.g. dedup
+     * https://github.com/donmccurdy/glTF-Transform/issues/980#issuecomment-1585748202
+     */
+    setReference(name: string, value: any): any;
 }
 export interface IExtensionWriter {
     /** used by progressive texture extension to pass an extension between documents */

package/dist/extensions/NEEDLE_opaque/NEEDLE_opaque.js

@@ -199,7 +199,7 @@ export function createOpaqueExtension(name, types, opts) {
                     // attach the property
                     target[index].setExtension(EXTENSION_NAME, prop);
                     // capture all the json pointers in the extension
-                    this.jsonPointerHandler.read(context, target[index], ext);
+                    this.jsonPointerHandler.read(context, target[index], ext, prop);
                     // make sure we write our extension back to the gltf
                     prop.add(new OpaqueWriter(map, gltfField, ext));
                 });
@@ -293,7 +293,11 @@ export function createOpaqueExtension(name, types, opts) {
         extensionName = typeof EXTENSION_NAME;
         parentTypes = ExtensionPropertyTypes;
         propertyType = "NEEDLE_opaque_property";
-        init() { }
+        setReference(name, value) {
+            this.setRef(name, value, { modifyChild: true });
+        }
+        init() {
+        }
         writers = [];
         add(writer) {
             if (typeof writer === "function") {
@@ -495,7 +499,7 @@ class JsonPointerHandler {
         this.document = document;
         this.id = id;
     }
-    read(context, prop, obj, key, fullPath) {
+    read(context, prop, obj, ext, key, fullPath) {
         const value = key !== undefined ? obj[key] : obj;
         // the pointer path contains the full path from the root object to the json pointer
         if (key !== undefined) {
@@ -506,7 +510,7 @@ class JsonPointerHandler {
             case 'object':
                 if (value !== null && value !== undefined) {
                     for (const key of Object.keys(value)) {
-                        this.read(context, prop, value, key, fullPath);
+                        this.read(context, prop, value, ext, key, fullPath);
                     }
                 }
                 break;
@@ -551,6 +555,19 @@ class JsonPointerHandler {
                         else
                             console.warn("WARN: failed registering pointer", fullPath, value);
                     }
+                    if (value.startsWith("/materials/")) {
+                        // set a reference to the material to avoid dedup
+                        // see https://github.com/donmccurdy/glTF-Transform/issues/980#issuecomment-1585748202
+                        if (this.debug)
+                            console.log("Read pointer", value);
+                        const index = parseInt(value.substring("/materials/".length));
+                        const mat = context.materials[index];
+                        if (mat) {
+                            if (ext) {
+                                ext.setReference(key, mat);
+                            }
+                        }
+                    }
                 }
                 break;
         }

package/dist/utils/version.d.ts

@@ -1 +1 @@
-export declare function getVersion(): string;
+export declare function getVersion(marklocal?: boolean): string;

package/dist/utils/version.js

@@ -1,9 +1,11 @@
 import { existsSync, readFileSync } from 'fs';
 let version = undefined;
-export function getVersion() {
+let isLocal = false;
+export function getVersion(marklocal = true) {
     if (version === undefined) {
         version = null;
         const path = process.cwd() + "/package.json";
+        isLocal = !path.includes("node_modules");
         if (existsSync(path)) {
             const packageJson = readFileSync(path, "utf8");
             const packageVersion = JSON.parse(packageJson).version;
@@ -12,5 +14,8 @@ export function getVersion() {
         else
             console.log("Could not find package.json at " + path);
     }
-    return version;
+    let res = version;
+    if (marklocal && isLocal)
+        res += " (local)";
+    return res;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@needle-tools/gltf-build-pipeline",
-  "version": "1.1.4",
+  "version": "1.1.6",
   "description": "Pipeline and tools for optimizing gltf files using gltf-transform and compression settings within glTF extensions",
   "main": "index.mjs",
   "type": "module",
@@ -16,9 +16,9 @@
     "compile": "tsc || exit 1"
   },
   "dependencies": {
-    "@gltf-transform/core": "^3.1.4",
-    "@gltf-transform/extensions": "^3.1.4",
-    "@gltf-transform/functions": "^3.1.4",
+    "@gltf-transform/core": "^3.4.6",
+    "@gltf-transform/extensions": "^3.4.6",
+    "@gltf-transform/functions": "^3.4.6",
     "@squoosh/lib": "^0.4.0",
     "command-exists": "^1.2.9",
     "command-line-args": "^5.2.1",