Skip to content

remove export default {};#317

Merged
makrsmark merged 1 commit intoairframesio:masterfrom
makrsmark:feature/no-export-default
Feb 2, 2026
Merged

remove export default {};#317
makrsmark merged 1 commit intoairframesio:masterfrom
makrsmark:feature/no-export-default

Conversation

@makrsmark
Copy link
Collaborator

@makrsmark makrsmark commented Feb 2, 2026

Summary by CodeRabbit

  • Refactor
    • Internal module export structure updated across the decoder plugin system. No user-facing functionality changes.

@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026

Walkthrough

The PR systematically removes module-level default exports (empty objects) from 68 TypeScript files across the library, plugin, and utility directories. All named exports and internal logic remain unchanged. One plugin file also has a helper function removed.

Changes

Cohort / File(s) Summary
Core Library Files
lib/DecoderPlugin.ts, lib/DecoderPluginInterface.ts, lib/IcaoDecoder.ts, lib/MessageDecoder.ts
Removed default exports from core library modules while preserving named exports and class definitions.
Label Plugin Files (Labels 10-24)
lib/plugins/Label_10_*.ts, lib/plugins/Label_12_*.ts, lib/plugins/Label_13Through18_Slash.ts, lib/plugins/Label_15*.ts, lib/plugins/Label_16_*.ts, lib/plugins/Label_1*.ts, lib/plugins/Label_20_*.ts, lib/plugins/Label_21_*.ts, lib/plugins/Label_22_*.ts, lib/plugins/Label_24_*.ts
Removed default exports from 40+ label plugin files. Special attention: Label_13Through18_Slash.ts also removes the helper function getMsgType in addition to the default export.
Label Plugin Files (Labels 2P-Label_H)
lib/plugins/Label_2P_*.ts, lib/plugins/Label_30_*.ts, lib/plugins/Label_4*.ts, lib/plugins/Label_5Z_*.ts, lib/plugins/Label_83.ts, lib/plugins/Label_8E.ts, lib/plugins/Label_B6.ts, lib/plugins/Label_ColonComma.ts, lib/plugins/Label_H*.ts
Removed default exports from remaining label plugin files across various label ranges and types.
Utility Files
lib/utils/miam.ts
Removed default export while preserving named utility exports.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested reviewers

  • kevinelliott

Poem

🐰 The default export fields now lay barren and bare,
Sixty-eight modules freed from empty object care,
With named exports standing tall in the light,
The codebase now cleaner—a rabbit's delight!
No logic was harmed, just tidied with grace, 💚

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title 'remove export default {};' directly and accurately describes the primary change across all modified files—the systematic removal of default exports from 57 files in the codebase.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@makrsmark makrsmark merged commit 4190232 into airframesio:master Feb 2, 2026
6 checks passed
@makrsmark makrsmark deleted the feature/no-export-default branch February 2, 2026 12:04
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