Merge master into v1

[TwitchBronBron]

Brings v1 up to date with every master commit through 0.73.3. Conflicts were resolved in v1's favor architecturally, but each feature was re-implemented against v1's APIs rather than dropped.

Ported from master

• continue transpiles to a goto label for firmware below 11.5 (Transpile continue down for firmware below 11.5 #489)
• go-to-definition for file path strings in BRS/BS/XML (Add generic go-to-definition for file path strings in BRS/BS/XML files #1648)
• isTerminal / previousInChain on AstNode (Add isTerminal and previousInChain getters to AstNode #1788)
• nested curly braces in template strings (Fix nested curly braces in template strings #1539)
• regex literals after ${ and , (Recognize regex literals after ${ and , #1789)
• wrong-cased XML tag diagnostic (Better error message for wrong-cased XML tags #1793)
• duplicate and crashing "find all references" fix (Fix duplicate and crashing "find all references" results #1791)
• duplicate sourceMappingURL fix (Avoid emitting a duplicate sourceMappingURL comment #1786)
• findAncestor type-guard inference (Infer node type from findAncestor type-guard matchers #1787)
• lexer token-text interning (Reduce per-Token lexer allocation to cut GC pressure while editing #1712)
• memory-aware default for max worker threads (Modifies default max worker thread logic to be only as much as memory allows #1798)
• js-yaml override bumped to ^4.3.2 (chore: Security enhancements #1796)

Worth a reviewer's attention

• master's whitespace fast-path skipped addToken to avoid allocating a Token it discards. In v1 addToken is also what routes a token into leadingTrivia, so skipping it silently dropped all whitespace trivia (20 test failures). Kept the allocation, took only the interning half.
• Modifies default max worker thread logic to be only as much as memory allows #1798 rewrote WorkerPool around master's simpler worker tracking. v1 has an isDead crashed-worker feature master lacks, so only getDefaultMaxWorkerThreads was ported, on top of v1's tracking — taking master's file wholesale would have deleted that feature.
• @typescript-eslint/no-unsafe-argument is warn here, not error. Master enabled it after cleaning up v0's call sites; v1's rewrites carry ~76 more violations that were never part of that cleanup. TODO left in .eslintrc.js.
• Three tests arrived from master asserting v0 behavior v1 changed deliberately: Comment tokens are trivia rather than emitted, getReferences returns [] rather than null, and the NamespacedVariableNameExpression chain step no longer exists. Updated to v1's contract, reasoning inline.

Breaking changes

• getDefinition returns Array<Location | LocationLink> instead of Location[] (Program, BrsFile, LspProject, Project, ProjectManager, WorkerThreadProject, ProvideDefinitionEvent.definitions). Inherent to Add generic go-to-definition for file path strings in BRS/BS/XML files #1648. Pushing a Location still works; reading results now needs narrowing.
• AliasStatement.value narrowed to VariableExpression | DottedGetExpression — master's abstract clone(): Expression made the old Expression field fail to compile against its own constructor. Only breaks code that was already building invalid AST.
• xml-tag-wrong-case is a new Error-severity diagnostic, so projects with <Children>/<Component> that previously built clean will report errors. Suppressible via diagnosticFilters.

Verification — npm run test:nocover: 4503 passing, 0 failing (v1 baseline was 4373, so all 130 new tests from master run). npx tsc --noEmit and npm run lint clean. Verified by hand that wrong-cased XML round-trips through transpile unchanged, and that continue→goto codegen is correct for nested loops, empty bodies, and continue inside an if.

🤖 Generated with Claude Code