refactor(parser): pass SourceType to ModuleRecordBuilder

[Sysix]

Passing SourceType to ModuleRecordBuilder, so it can manage its errors itself.
Before all errors were collected and later checked if this is a typescript file.
Now it will skip the generation of the diagnostic (performance).

This refactoring is needed for TS error 1363 (see downstream PR), where we WANT diagnostics for typescript files

[Sysix]

• feat(parser): add TS1363 error code #17609
• refactor(parser): pass SourceType to ModuleRecordBuilder #17610 👈 (View in Graphite)
• main

---

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• 0-merge - adds this PR to the back of the merge queue
• hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codspeed-hq]

CodSpeed Performance Report

Merging #17610 will not alter performance

_{Comparing 01-03-refactor_parser_pass_sourcetype_to_modulerecordbuilder_ (9394582) with main (3a0c782)}

Summary

✅ 42 untouched
⏩ 3 skipped¹

Footnotes

1. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[Copilot]

Pull request overview

This PR refactors the parser to pass SourceType to ModuleRecordBuilder, enabling it to manage TypeScript-related error filtering internally rather than at the call site. This improves performance by skipping diagnostic generation for TypeScript files early in the process.

• Moved TypeScript checking logic from lib.rs into ModuleRecordBuilder::errors() method
• Updated ModuleRecordBuilder::new() to accept a SourceType parameter
• Simplified error collection logic in the parser by removing conditional TypeScript checks

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
crates/oxc_parser/src/module_record.rs	Added source_type field to ModuleRecordBuilder and implemented early return in errors() method for TypeScript files
crates/oxc_parser/src/lib.rs	Updated constructor call to pass source_type and simplified error collection by removing TypeScript conditional checks
tasks/track_memory_allocations/allocs_parser.snap	Reflects performance improvement with ~51% reduction in system allocations for TypeScript files (cal.com.tsx)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[camc314]

nice 🙂

[graphite-app]

Merge activity

• Jan 5, 8:58 AM UTC: camc314 added this pull request to the Graphite merge queue.
• Jan 5, 9:04 AM UTC: Merged by the Graphite merge queue.