fix: uneval emits valid JS for graphs with more than 65534 repeated references

[youdie006]

Fixes #93.

uneval hoists every value referenced more than once into a parameter of a single IIFE ((function(a,b,c){...}(1,2,3))). A function may have at most 65535 parameters, so a graph with more than ~65534 repeated references produces code that engines reject:

SyntaxError: Too many parameters in function definition (only 65534 allowed)

Repro:

const shared = Array.from({ length: 70000 }, () => ({}));
uneval({ a: shared, b: shared.slice() }); // the output throws when eval'd

Fix: when the parameter count exceeds the limit, pass the hoisted values as a single array argument and destructure them into the placeholder names:

(function(){var[a,b,c]=arguments[0];...}([1,2,3]))

instead of one parameter each. This also avoids the analogous too-many-arguments limit on the call. Graphs below the threshold are byte-for-byte unchanged.

Test: added a case that uneval-serializes a graph with 70,000 repeated references and evals the output, asserting the round-trip and shared identity. Verified red before the fix (Too many parameters in function definition) and green after. npm test (uvu) passes: 723 + 29 + 18.

---

Disclosure: developed with the assistance of Claude Code (AI); reviewed and verified by me.

[changeset-bot]

🦋 Changeset detected

Latest commit: e44e4db

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
devalue	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR