Skip to content

[Types] LLM env layer (emmylua config, type stubs, manual fixes) - #8397

Open
keithharvey wants to merge 1 commit into
migfrom
fmt-llm-source
Open

[Types] LLM env layer (emmylua config, type stubs, manual fixes)#8397
keithharvey wants to merge 1 commit into
migfrom
fmt-llm-source

Conversation

@keithharvey

@keithharvey keithharvey commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Warning

Don't merge this PR by itself — it's one slice of a stacked review.
Merging an intermediate PR breaks the stack. The whole cleanup lands by
merging only the tip, fmt-llm, which pulls in every branch below it.

Part of BAR type-error cleanup. Human-curated env layer that prepares the codebase for the LLM type-fix pass.

This branch carries:

  • .emmyrc.json globals and analyzer config
  • types/* stubs for vendored/generated declarations
  • Explicit type ignores for known dead code
  • CI gate configuration
  • Manual source fixes that require human judgement

The fix recipes the subsequent LLM pass (#8398) uses are catalogued in SKILL.md — same rulebook that guides the subagents.

Branch Topology

All branches in the BAR type-error cleanup stack — see Bulk Migrations for the migration log and how to run just bar::migrate::stylua-cleanup. Regenerated deterministically by just bar::migrate::stylua-cleanup-generate. Generated 2026-08-04 22:18:41 UTC.

Leaves — each isolates one transform's diff vs fmt:

Branch Command Diff vs parent Units
fmt stylua 1429 files, +308036 −194387 ✅ pass
mig-bracket bar-lua-codemod bracket-to-dot 352 files, +8330 −8330 ✅ pass
mig-rename-aliases bar-lua-codemod rename-aliases 178 files, +377 −377 ✅ pass
mig-detach-bar-modules bar-lua-codemod detach-bar-modules 183 files, +1941 −1831 ✅ pass
mig-integration-tests <hand curated> 27 files, +174 −173 ✅ pass
mig-busted-types <hand curated> 18 files, +1539 −92 ✅ pass

Rollups — composite branches stacking the leaves and (for fmt-llm) the env + LLM layers:

Branch Diff vs master Diff vs parent Units
mig 1449 files, +312829 −197631 529 files, +12217 −10668 ✅ pass
👉 fmt-llm-source — you are here 1453 files, +313191 −197778 57 files, +394 −179 ✅ pass
fmt-llm 1454 files, +313241 −197799 32 files, +66 −37 ✅ pass

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Integration Test Results

15 tests  ±0   7 ✅ ±0   4s ⏱️ ±0s
 1 suites ±0   8 💤 ±0 
 1 files   ±0   0 ❌ ±0 

Results for commit d1c6304. ± Comparison against base commit 9357605.

♻️ This comment has been updated with latest results.

Comment thread luarules/gadgets/unit_dynamic_collision_volume.lua
@keithharvey
keithharvey force-pushed the fmt-llm-source branch 2 times, most recently from e9772e3 to 6441fa8 Compare July 29, 2026 22:17
Comment thread .github/workflows/type_check.yml Outdated
Comment thread common/luaUtilities/serpent.lua
@keithharvey
keithharvey force-pushed the fmt-llm-source branch 2 times, most recently from bdef328 to 667ae95 Compare July 30, 2026 01:32
…ixes

One atomic layer commit: folds the emmylua-0.24 dangling-globals and
springFunctions forward-declaration followups. The spring-split alias
adoption (SetupAliases shim + SpringUnsynced call sites) is extracted —
it ships with the engine rename, not the type env.

CI moves to emmylua_check 0.24.0 in lockstep with dev.Containerfile. The
gate reports errors only (severity 1); warnings and hints stay local —
just bar::check for the full output, bar::check-errors for this gate's
exact view.
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.

2 participants