Using native typescript loader

[RobinTail]

Using Node.js 22.18 and higher no longer requires unrun to execute typescript.
Though, it requires .ts extensions (#3055) and verbatimModuleSyntax to enforce type-only imports where implied.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 19d21ec7-88ad-4e9d-be8f-ab477e40df11

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch native-ts-execution

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[pullfrog]

Review Summary

Reviewed PR #3267: Using native typescript loader

What Changed

• Added verbatimModuleSyntax: true to tsconfig.json
• Converted implicit type imports to explicit import type syntax (78 files)
• Added .ts extensions to relative imports in the example folder
• Updated Node.js version in CI workflow from 22 to 24

Assessment: LGTM

The implementation is correct:

• Type imports properly use import type syntax (required by verbatimModuleSyntax)
• Example code uses .ts extensions for native TypeScript execution
• Library source code omits extensions (correct for build tooling with noEmit: true)
• Changes are consistent across the codebase

No actionable issues found.

  ^{｜ View workflow run ｜ Triggered by Pullfrog ｜ pullfrog.com ｜ 𝕏}

[pullfrog]

Addressed review feedback: Removed the redundant export type { Routing }; re-export from routing-walker.ts. Change pushed and review comment replied to.

  ^{｜ View workflow run ｜ Triggered by Pullfrog ｜ pullfrog.com ｜ 𝕏}

[RobinTail]

okay

[pullfrog]

Fixed the review feedback: Changed the import of Routing in express-zod-api/src/documentation.ts from ./routing-walker to ./routing. The fix has been committed and pushed to the branch.

  ^{｜ View workflow run ｜ Triggered by Pullfrog ｜ pullfrog.com ｜ 𝕏}

[coveralls-official]

coverage: 100.0%. remained the same
when pulling 3121533 on native-ts-execution
into db876a2 on make-v28.

[RobinTail]

reviewed.
this is fine for now.
later can enforce ts extensions globally by removing root tsconfig module customizations (if needed)