Skip to content

Fix TypeScript generator node-mavlink import#1179

Merged
peterbarker merged 2 commits into
ArduPilot:masterfrom
vvezre:codex/typescript-generator-import-fix
Mar 30, 2026
Merged

Fix TypeScript generator node-mavlink import#1179
peterbarker merged 2 commits into
ArduPilot:masterfrom
vvezre:codex/typescript-generator-import-fix

Conversation

@vvezre

@vvezre vvezre commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • update the TypeScript generator to import MAVLinkMessage, readInt64LE, and readUInt64LE from @ifrunistuttgart/node-mavlink
  • keep the generated imports aligned with the package referenced by users in the existing issue
  • add a regression test that generates TypeScript output and locks the import path in place

Testing

  • python -m pytest -p no:cacheprovider tests\test_mavgen_typescript.py -q
  • python -m py_compile generator\mavgen_typescript.py tests\test_mavgen_typescript.py

Closes #432

@peterbarker peterbarker merged commit d32ec97 into ArduPilot:master Mar 30, 2026
23 checks passed
@peterbarker

Copy link
Copy Markdown
Contributor

Merged, thanks!

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.

Typescript Mavlink Generator Has Non-Existant Import

2 participants