Skip to content

[codex] fix warnings across targets#8

Open
bobzhang wants to merge 1 commit intooboard:mainfrom
bobzhang:codex/fix-all-warnings
Open

[codex] fix warnings across targets#8
bobzhang wants to merge 1 commit intooboard:mainfrom
bobzhang:codex/fix-all-warnings

Conversation

@bobzhang
Copy link
Copy Markdown
Contributor

@bobzhang bobzhang commented Apr 2, 2026

Summary

  • remove deprecated syntax and lint warnings across native, js, and wasm codepaths
  • add explicit wasm websocket stubs so all-target builds compile cleanly
  • refresh generated .mbti files after moon info

Why

  • the module had deprecation warnings on the default target, and moon check --target all also exposed wasm-only warnings and missing symbols

Impact

  • moon check --target all is now clean
  • unsupported wasm websocket/fetch paths now fail explicitly instead of relying on missing symbols
  • no intended public API changes beyond regenerated interface output formatting

Validation

  • moon check --target all
  • moon test --target all
  • moon info
  • moon fmt

Review

  • a fresh Codex review pass found no actionable findings after the wasm fetch follow-up fix

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