Skip to content

fix(estree/tokens): generate tokens for files with BOM#19535

Merged
graphite-app[bot] merged 1 commit intomainfrom
om/02-18-fix_estree_tokens_generate_tokens_for_files_with_bom
Feb 24, 2026
Merged

fix(estree/tokens): generate tokens for files with BOM#19535
graphite-app[bot] merged 1 commit intomainfrom
om/02-18-fix_estree_tokens_generate_tokens_for_files_with_bom

Conversation

@overlookmotel
Copy link
Member

@overlookmotel overlookmotel commented Feb 19, 2026

Fix a bug where we didn't produce any tokens for files which start with a BOM.

As a side effect, re-use the existing Utf8ToUtf16 span converter created for converting spans in the AST again for tokens, rather than creating another one.

Copy link
Member Author

overlookmotel commented Feb 19, 2026


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
Copy link

codspeed-hq bot commented Feb 19, 2026

Merging this PR will not alter performance

✅ 48 untouched benchmarks
⏩ 7 skipped benchmarks1


Comparing om/02-18-fix_estree_tokens_generate_tokens_for_files_with_bom (142e46b) with om/02-18-refactor_estree_tokens_combine_collect_token_context_into_to_estree_tokens_json_ (3507e7a)

Open in CodSpeed

Footnotes

  1. 7 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.

@overlookmotel overlookmotel force-pushed the om/02-18-refactor_estree_tokens_combine_collect_token_context_into_to_estree_tokens_json_ branch from ab13696 to 96294ca Compare February 20, 2026 10:09
@overlookmotel overlookmotel force-pushed the om/02-18-fix_estree_tokens_generate_tokens_for_files_with_bom branch from cd4ce15 to 110cf79 Compare February 20, 2026 10:09
@overlookmotel overlookmotel force-pushed the om/02-18-refactor_estree_tokens_combine_collect_token_context_into_to_estree_tokens_json_ branch from 96294ca to a4a8f76 Compare February 20, 2026 11:43
@overlookmotel overlookmotel force-pushed the om/02-18-fix_estree_tokens_generate_tokens_for_files_with_bom branch from 110cf79 to 443d906 Compare February 20, 2026 11:43
@overlookmotel overlookmotel force-pushed the om/02-18-refactor_estree_tokens_combine_collect_token_context_into_to_estree_tokens_json_ branch from a4a8f76 to bd885e6 Compare February 23, 2026 16:11
@overlookmotel overlookmotel force-pushed the om/02-18-fix_estree_tokens_generate_tokens_for_files_with_bom branch from 443d906 to a4e8469 Compare February 23, 2026 16:11
@overlookmotel overlookmotel changed the base branch from om/02-18-refactor_estree_tokens_combine_collect_token_context_into_to_estree_tokens_json_ to graphite-base/19535 February 23, 2026 22:49
This was referenced Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cli Area - CLI A-linter Area - Linter A-linter-plugins Area - Linter JS plugins A-parser Area - Parser C-bug Category - Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants