Skip to content

Conversation

siketyan
Copy link
Member

@siketyan siketyan commented Jul 5, 2025

Added support for the monorepo structure, that is now supported since Biome v2.0, with the root: false and extends: // syntaxes.

The plugin now skips the biome.json(c) files with root: false or extends: // to ensure the workspace root to be where the root configuration exists.

ref: https://biomejs.dev/guides/big-projects/#monorepo

image

@siketyan siketyan requested review from denbezrukov and ematipico July 5, 2025 07:53
@siketyan siketyan self-assigned this Jul 5, 2025
@@ -0,0 +1 @@
console.log('Hello, world!');
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be annotated with an error as noConsole is enabled in the root config.

@@ -0,0 +1 @@
console.log("Hello, world!");
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should NOT be annotated as the rule is disabled in the nested config.

@siketyan
Copy link
Member Author

siketyan commented Jul 5, 2025

I think we can also close #161 as now we need to have only one LSP instance for the monorepo

companion object {
@OptIn(ExperimentalSerializationApi::class)
fun loadFromFile(file: VirtualFile): BiomeConfig? {
val json = Json { ignoreUnknownKeys = true }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this support jsonc files too?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! I added allowComments and allowTrailingComma flags to the Json() initializer.

@siketyan siketyan merged commit 290dbd5 into main Jul 5, 2025
2 checks passed
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.

2 participants