Skip to content

fix: replace inline type assertions with proper ast types in closure-analyzer#216

Merged
cs01 merged 1 commit intomainfrom
fix/closure-analyzer-types
Mar 11, 2026
Merged

fix: replace inline type assertions with proper ast types in closure-analyzer#216
cs01 merged 1 commit intomainfrom
fix/closure-analyzer-types

Conversation

@cs01
Copy link
Copy Markdown
Owner

@cs01 cs01 commented Mar 11, 2026

Summary

  • Replace 20+ local interface definitions in closure-analyzer.ts with proper AST types from ast/types.ts
  • Eliminates struct field misalignment risk that caused native compiler SIGSEGV on describe.ts (issue handler registry #22)
  • Reduces file from 449 to 293 lines by removing redundant type definitions

Test plan

  • npm run verify:quick passes (tests + Stage 1 self-hosting)
  • TypeScript type check passes
  • describe.ts test remains @test-skip — needs native compiler rebuild to verify fix

🤖 Generated with Claude Code

@cs01 cs01 merged commit bd15121 into main Mar 11, 2026
12 checks passed
@cs01 cs01 deleted the fix/closure-analyzer-types branch March 12, 2026 05:35
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.

1 participant