[masenf]

All Submissions:

• Have you followed the guidelines stated in CONTRIBUTING.md file?
• Have you checked to ensure there aren't any other open Pull Requests for the desired changed?

Type of change

• Bug fix (non-breaking change which fixes an issue)

Description

LiteralVar.create rendered a set as an array but rejected a frozenset:

>>> LiteralVar.create(frozenset({1, 2, 3}))
TypeError: Unsupported type <class 'frozenset'> for LiteralVar. ...

A frozen set now behaves like a set at runtime: frozenset is registered alongside set on ArrayVar, so the literal renders as [1, 2, 3]; figure_out_type infers frozenset[T] like set[T], and the array index helper reads the element type out of it, so rx.foreach and typed array operations such as .map(lambda item: item.upper()) work over it. Surfaced while reviewing #7606, which treats frozen sets as containers.

Not changed here: the static array bound (ARRAY_VAR_TYPE is bound to Sequence) and the LiteralVar.create overloads, which don't cover set today either, and Sequence-typed props, which reject a set var as well. Those are a broader typing change than this parity fix.

Changes To Core Features:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your core changes, as applicable?
• Have you successfully ran tests with your changes locally?

The array rendering parametrization in tests/units/test_var.py gains a frozenset case, and test_frozenset_literal_types_its_elements checks the inferred type, a typed element on indexing, and a typed .map callback. ruff, pyright on the touched files, and pytest tests/units/test_var.py pass.

🤖 Generated with Claude Code

https://claude.ai/code/session_01RqJ6TNt8AXzbDVm5kLb2nD

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; the previous callback typing issue is fixed.

Summary

Adds frozenset support to array literals and preserves element types for indexing and callbacks.

• Frozen sets render as arrays and keep their element types.

_{Reviews (2) · Last reviewed commit: "fix(vars): infer a frozenset literal's e..." · Reviewed by Greptile}

[cubic-dev-ai]

All reported issues were addressed across 3 files

<sub>Reply with feedback, questions, or to request a fix.

View guided diff | Turn on auto-fix | Re-trigger cubic</sub>

[codspeed]

Merging this PR will not alter performance

⚠️ 16 benchmarks spent significant time in system calls

System calls cannot be consistently instrumented, so they are not included in the measure, which understates the real cost. Please switch to the Walltime instrument to accurately measure system calls.

Measurement and system calls

• test_get_state[memory]
• test_cli_startup[deploy_help]
• test_state_manager_memory_read_only_modify
• test_event_shutdown_cancel
• test_modify_state[disk]
• test_state_manager_memory_cold_get
• test_set_state[disk]
• test_state_manager_memory_warm_get
• test_import_cli
• test_get_state[disk]
• …and 6 more

✅ 148 untouched benchmarks

---

_{Comparing claude/literal-frozenset (da9cc7f) with main (4be8b4d)}