Skip to content

Sync main with tv-labs upstream (through v1.0.2) - #4

Open
smn wants to merge 14 commits into
mainfrom
sync-upstream-main
Open

Sync main with tv-labs upstream (through v1.0.2)#4
smn wants to merge 14 commits into
mainfrom
sync-upstream-main

Conversation

@smn

@smn smn commented Sep 3, 2026

Copy link
Copy Markdown

Catches the fork's main up with tv-labs/lua upstream (13 commits, through the 1.0.2 release and the post-release perf/compiler work).

What's in it

Straight merge of upstream/main into main. Notable upstream changes: the 1.0.2 release, dispatcher/call-convention and compiler peephole perf work, lexer token slicing, one-shot stdlib table build, and the cyclic-table eval-boundary fix.

Conflicts resolved

Four files conflicted — all of them the fork's own fixes that upstream had already merged and refined. Resolved by taking upstream's canonical version in each case:

  • lib/lua/vm/display/table.ex — upstream aliases %Table{} (cosmetic)
  • lib/lua/vm/stdlib/pattern.ex — reworded §6.4 comment (cosmetic)
  • test/lua/vm/cyclic_table_test.exs — upstream has the superset of tests
  • test/lua/vm/stdlib/pattern_anchor_test.exs — upstream has the superset of tests

The fork's dialyzer: track the ancestor set as a plain map change is already how upstream's decode/3 works, so nothing was lost.

Verification

  • mix test — 2870 passed, 0 failures
  • mix format --check-formatted — clean
  • mix dialyzer — 0 errors
  • mix test --only lua53 — 20 passed

🤖 Generated with Claude Code

davydog187 and others added 14 commits July 27, 2026 12:30
…-labs#406)

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Dave Lucia <davelucianyc@gmail.com>
…v-labs#418)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
# Conflicts:
#	lib/lua/vm/display/table.ex
#	lib/lua/vm/stdlib/pattern.ex
#	test/lua/vm/cyclic_table_test.exs
#	test/lua/vm/stdlib/pattern_anchor_test.exs
@smn

smn commented Sep 3, 2026

Copy link
Copy Markdown
Author

@santiagocardo this just catches us up with upstream Lua

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants