Issue #432: Migrate linting and formatting to Biome by Priyanshubhartistm · Pull Request #452 · cameri/nostream

Priyanshubhartistm · 2026-04-10T21:30:37Z

Description

Migrated the linting and formatting toolchain to Biome.
Added Biome configuration for linting and formatting behavior.
Updated npm scripts for lint, lint report, lint fix, format, and format check to use Biome.
Updated CI lint pipeline to run Biome.
Updated project documentation to reference Biome workflow.
Removed legacy ESLint configuration and ESLint dependencies.
Updated lockfile after dependency migration.

Related Issue

Closes#432

Motivation and Context

Use a single tool for linting and formatting.
Simplify tooling and maintenance.
Improve performance compared to separate lint/format tools.

How Has This Been Tested?

npm run lint
npm run build:check

Screenshots (if appropriate):

N/A

Types of changes

Non-functional change (docs, style, minor refactor)
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

My code follows the code style of this project.
My change requires a change to the documentation.
I have updated the documentation accordingly.
I have read the CONTRIBUTING document.
I have added tests to cover my code changes.
All new and existing tests passed.

Performance & Disk Usage

Metric	Before (ESLint + Prettier)	After (Biome)	Change
Lint time	7.55 seconds	0.97 seconds	7.8x faster (-87%)
`node_modules` size	268 MB	348 MB	+80 MB (+30%)

Summary:
Linting speed improved dramatically (~7.8 times faster).
Note: node_modules size increased slightly due to dependency changes during migration.

socket-security · 2026-04-10T21:35:36Z

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	pg-query-stream@4.3.0
	ramda@0.28.0
	pg@8.9.0
	@biomejs/biome@2.4.12

View full report

socket-security · 2026-04-10T21:35:38Z

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

Copilot

Pull request overview

Migrates the project’s JavaScript/TypeScript linting + formatting workflow from ESLint (and related tooling) to Biome, updating developer docs and CI to use the new toolchain.

Changes:

Added biome.json and @biomejs/biome dev dependency; updated npm scripts to run Biome for linting/formatting.
Updated GitHub Actions CI checks to run the new lint command.
Updated documentation and removed legacy ESLint config/ignore files.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
README.md	Documents the new Biome lint/format commands under the Tests section.
package.json	Replaces ESLint scripts with Biome scripts; adds Biome as a dev dependency; removes ESLint-related deps.
CONTRIBUTING.md	Updates contributor guidance to reference Biome for code-quality checks.
biome.json	Introduces Biome configuration (formatter + linter rules).
.github/workflows/checks.yml	Updates the lint job step name and runs `npm run lint` (now Biome-based).
.eslintrc.js	Removed legacy ESLint configuration.
.eslintignore	Removed legacy ESLint ignore file.

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

…ration # Conflicts: # package-lock.json # package.json

Priyanshubhartistm · 2026-04-16T14:00:02Z

hi @cameri , @phoenix-server
could you please review this PR whenever you are free
Thank you

cameri · 2026-04-18T11:51:50Z

@Priyanshubhartistm since we've upgraded to latest version of ESLint, do you mind addressing conflicts?

…ration # Conflicts: # .eslintrc.js # package-lock.json # package.json

cameri · 2026-04-18T14:14:48Z

@Priyanshubhartistm could you capture the speed and dependency disk cost improvements by migrating to Biome? I'm interested in the absolute values before/after and the % relative change to understand the impact.

If you do, could you update the PR description with it or add it as a comment to the issue? Either works!

…ration # Conflicts: # package.json

* chore(tooling): migrate npm tooling scripts to biome * chore(biome): add biome configuration * ci: switch lint workflow to biome * docs: update contribution and readme for biome * chore(lint): remove legacy eslint config files * docs: update lint tooling wording to biome * fix(lint): address Copilot review feedback

Priyanshubhartistm · 2026-04-18T15:46:54Z

@cameri Here are the before/after measurements:

Performance & Disk Usage

Metric	Before (ESLint + Prettier)	After (Biome)	Change
Lint time	7.55 seconds	0.97 seconds	7.8x faster (-87%)
`node_modules` size	268 MB	348 MB	+80 MB (+30%)

Summary:
Linting speed improved dramatically (~7.8 times faster).
Note: node_modules size increased slightly due to dependency changes during migration.

Linting is now ~7.8x faster. node_modules size increased a bit, likely due to dependency updates. Let me know if anything else is needed!

cameri · 2026-04-18T15:49:10Z

@Priyanshubhartistm thank you!

Priyanshubhartistm added 6 commits April 11, 2026 02:44

chore(tooling): migrate npm tooling scripts to biome
1cdedb6

chore(biome): add biome configuration
cc457d8

ci: switch lint workflow to biome
1f190ca

docs: update contribution and readme for biome
409a397

chore(lint): remove legacy eslint config files
ff5078d

docs: update lint tooling wording to biome
3f7778e

cameri requested a review from Copilot April 11, 2026 02:11

CopilotAI reviewed Apr 11, 2026
View reviewed changes

Comment threadpackage.json Outdated
Comment threadbiome.json Outdated

Priyanshubhartistm added 2 commits April 14, 2026 14:53

fix(lint): address Copilot review feedback
9e81533

Merge remote-tracking branch 'upstream/main' into issue-432-biome-mig…
a2125dc
…ration # Conflicts: # package-lock.json # package.json

Merge remote-tracking branch 'upstream/main' into issue-432-biome-mig…
ec3379c
…ration # Conflicts: # .eslintrc.js # package-lock.json # package.json

Merge remote-tracking branch 'upstream/main' into issue-432-biome-mig…
211b668
…ration # Conflicts: # package.json

cameri reviewed Apr 18, 2026
View reviewed changes

Comment threadCONTRIBUTING.md Outdated

chore: merge upstream main and resolve lockfile conflict
fa2d97f

cameri approved these changes Apr 18, 2026
View reviewed changes

cameri merged commit 842b78e into cameri:mainApr 18, 2026
9 checks passed

cameri mentioned this pull request Apr 22, 2026
chore: release new version 🚀 #560
Merged

This was referenced Apr 22, 2026
chore: release new version 🚀 Mahmoud-s-Khedr/nostream#2
Open
chore: release new version 🚀 archief2910/nostream#1
Open
chore: release new version 🚀 vikashsiwach/nostream#1
Open
chore: release new version 🚀 CKodidela/nostream#1
Open

github-actionsBot mentioned this pull request Apr 30, 2026
chore: release new version 🚀 YashIIT0909/nostream#1
Open

Navigation Menu

Issue #432: Migrate linting and formatting to Biome - #452

Issue #432: Migrate linting and formatting to Biome#452
cameri merged 11 commits into
cameri:mainfrom
Priyanshubhartistm:issue-432-biome-migration

Priyanshubhartistm commented Apr 10, 2026•
edited
Loading

Uh oh!

socket-securityBot commented Apr 10, 2026•
edited
Loading

Uh oh!

socket-securityBot commented Apr 10, 2026•
edited
Loading

Uh oh!

CopilotAI left a comment

Uh oh!

Uh oh!

Uh oh!

Priyanshubhartistm commented Apr 16, 2026

Uh oh!

cameri commented Apr 18, 2026

Uh oh!

cameri commented Apr 18, 2026

Uh oh!

Uh oh!

Uh oh!

Priyanshubhartistm commented Apr 18, 2026

Uh oh!

cameri commented Apr 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Priyanshubhartistm commented Apr 10, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

Checklist:

Performance & Disk Usage

Uh oh!

socket-securityBot commented Apr 10, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

socket-securityBot commented Apr 10, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Priyanshubhartistm commented Apr 16, 2026

Uh oh!

cameri commented Apr 18, 2026

Uh oh!

cameri commented Apr 18, 2026

Uh oh!

Uh oh!

Uh oh!

Priyanshubhartistm commented Apr 18, 2026

Performance & Disk Usage

Uh oh!

cameri commented Apr 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Priyanshubhartistm commented Apr 10, 2026•
edited
Loading

socket-securityBot commented Apr 10, 2026•
edited
Loading

socket-securityBot commented Apr 10, 2026•
edited
Loading