[masenf]

Requested by Masen · project thread

Type of change

• Revert

Description

Reverts #7622, which registered frozenset on ArrayVar alongside set so a frozen set literal rendered as a JS array.

Treating a set as an ArrayVar is the wrong direction. A set is not a sequence, and the operations a user wants on one (membership, add, remove, union, intersection) are not array operations. Rather than widening the array types further, sets should get their own SetVar backed by the JS Set object; that is now tracked in #7623.

Net effect after the revert is the pre-#7622 state: frozenset is rejected by LiteralVar.create, and set is still routed to ArrayVar as before. #7622 was never released, so there is no changelog entry.

Related: #7623

---

Generated by Claude Code

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

Reverts the unreleased support for treating frozenset values as arrays.

• Frozen sets no longer become array variables.

No actionable new issues were found.

_{Reviews (2) · Last reviewed commit: "Merge branch 'main' into claude/project-..." · Reviewed by Greptile}

[codspeed]

Merging this PR will not alter performance

1 warning

⚠️ 19 benchmarks are dominated by system calls

System calls cannot be simulated consistently and are left out of the measure. The reported value understates the real cost. Switch to the Walltime instrument to measure them.

Learn more about system calls

See affected benchmarks

✅ 148 untouched benchmarks

---

_{Comparing claude/project-thread-t07m06 (4482e25) with main (7f89e48)}

[masenf]

CodSpeed reports a -4.69% regression on test_var_access[mutable_dataclass_list] in tests/benchmarks/test_state_proxy.py. That benchmark exercises the state proxy, which this diff does not touch: the revert only removes frozenset from ArrayVar's registered types and from figure_out_type. CodSpeed's own report notes base and head ran on different runner types, so the delta is runner noise rather than this change. CodSpeed is not a required check, so not acknowledging it on the dashboard or re-running.

---

Generated by Claude Code