Skip to content

language/runtime: preserve computed property-key conversion side effects #3559

Description

@andrewtdiz

Summary

Computed key expressions are evaluated and converted in source order, including side effects from number/object key conversion.

Affected Test262 rows

Perry actual behavior from report

  • 2 x runtime-fail: Uncaught exception: Actual [] and expected [0] should have the same contents. order set for key1
  • 2 x runtime-fail: Uncaught exception: The value of counter is 2 Expected SameValue(«0», «2») to be true

Node / ECMAScript expected behavior

Computed key expressions are evaluated and converted in source order, including side effects from number/object key conversion.

Likely subsystem

HIR evaluation ordering / ToPropertyKey runtime

Acceptance criteria

  • The to-name-side-effects rows record the same side-effect order as Node.
  • Class and object literal paths share the same ToPropertyKey ordering semantics.
  • No eager/dropped conversion skips side effects.

Validation

Run the Test262 differential against the affected area and confirm these rows leave the gap report:

scripts/test262_subset.py --root vendor/test262 --dir language/computed-property-names/to-name-side-effects --sample-cap 99999 --report /tmp/COMPUTED_KEY_EFFECTS.json

This is Perry language/runtime semantics triage, not a Node core API compatibility issue.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions