fix: scope the placeholder size passthrough to expanded nodes with a recorded size

Guard the new `size` passthrough on two cases review surfaced:

- `last_serialization` with no `size` (schema v0.4 / partial entries): the
  unconditional override wrote the LGraphNode constructor default [140, 60]
  into the saved workflow, a dimension the source file never had, pinning
  the real node to it once the pack is installed. Reverts the v0.4 snapshot
  churn along with it.
- Collapsed placeholders: Vue nodes mode clears the CSS width/height floors
  on collapse, so the ResizeObserver -> layoutStore -> useLayoutSync chain
  writes the collapsed card's measurement back to `node.size`. Since
  `flags` is still replayed from the file, persisting that would pair
  `collapsed: false` with collapsed dimensions.

Tests now drive geometry through `setPos`/`setSize` (the production
mutators `LGraphCanvas` resizing uses) rather than raw field assignment.

Author: mattmillerai

src/lib/litegraph/src/LGraphNode.test.ts

@@ -101,12 +101,15 @@ describe('LGraphNode', () => {
   })
 
   describe('missing-node placeholder serialization', () => {
-    function createPlaceholder(): LGraphNode {
+    function createPlaceholder(
+      overrides: Partial<ISerialisedNode> = {}
+    ): LGraphNode {
       const lastSerialization = getMockISerialisedNode({
         type: 'UninstalledNodeType',
         pos: [100, 100],
         size: [140, 60],
-        widgets_values: [512]
+        widgets_values: [512],
+        ...overrides
       })
       const placeholder = new LGraphNode('')
       placeholder.last_serialization = lastSerialization
@@ -117,8 +120,8 @@ describe('LGraphNode', () => {
 
     test('carries the live pos and size through to the replayed serialization', () => {
       const placeholder = createPlaceholder()
-      placeholder.pos = [999, 888]
-      placeholder.size = [777, 666]
+      placeholder.setPos(999, 888)
+      placeholder.setSize([777, 666])
 
       const serialized = placeholder.serialize()
 
@@ -128,13 +131,33 @@ describe('LGraphNode', () => {
 
     test('still replays the fields it has no definition to regenerate', () => {
       const placeholder = createPlaceholder()
-      placeholder.size = [777, 666]
+      placeholder.setSize([777, 666])
 
       const serialized = placeholder.serialize()
 
       expect(serialized.type).toEqual('UninstalledNodeType')
       expect(serialized.widgets_values).toEqual([512])
     })
+
+    test('does not invent a size when the recorded serialization has none', () => {
+      const placeholder = createPlaceholder()
+      delete (placeholder.last_serialization as Partial<ISerialisedNode>).size
+
+      const serialized = placeholder.serialize()
+
+      expect(serialized).not.toHaveProperty('size')
+    })
+
+    test('keeps the recorded size while collapsed, since flags are replayed from the file', () => {
+      const placeholder = createPlaceholder({ flags: { collapsed: false } })
+      placeholder.flags.collapsed = true
+      placeholder.setSize([80, 30])
+
+      const serialized = placeholder.serialize()
+
+      expect(serialized.size).toEqual([140, 60])
+      expect(serialized.flags).toEqual({ collapsed: false })
+    })
   })
 
   test('should configure inputs correctly', () => {

src/lib/litegraph/src/LGraphNode.ts

@@ -1060,13 +1060,24 @@ export class LGraphNode
     }
 
     // special case for when there were errors
-    if (this.constructor === LGraphNode && this.last_serialization)
+    if (this.constructor === LGraphNode && this.last_serialization) {
+      // Only an expanded placeholder whose file recorded a size has a live
+      // size worth keeping. Vue nodes mode drops the CSS width/height floors
+      // on collapse, so a collapsed node's size is a measurement of the
+      // collapsed card, and `flags` is replayed from the file — keeping both
+      // would pair `collapsed: false` with collapsed dimensions. With no
+      // recorded size there is no definition to compute a real one from, so
+      // keeping the live size would invent a dimension the file never had.
+      const carriesLiveSize =
+        this.last_serialization.size != null && !this.flags.collapsed
+
       return {
         ...this.last_serialization,
         mode: o.mode,
         pos: o.pos,
-        size: o.size
+        ...(carriesLiveSize && { size: o.size })
       }
+    }
 
     if (this.inputs)
       o.inputs = this.inputs.map((input) => inputAsSerialisable(input))

src/lib/litegraph/src/__snapshots__/LGraph.test.ts.snap

@@ -34,10 +34,6 @@ exports[`LGraph > supports schema v0.4 graphs > oldSchemaGraph 1`] = `
         10,
         10,
       ],
-      "size": [
-        140,
-        60,
-      ],
     },
   ],
   "revision": 0,