feat: workspace support

[psteinroe]

adds workspace support as well as support for extends in the config. This now works:

./postgrestools.jsonc

{ // ...my linter config }

./project_one/postgrestools.jsonc

{ "extends": "../postgrestools.jsonc" // e.g. add specific db config for this project }

Will need to check if client-side changes are required to enable workspace support there.

A few internal changes:

• we now store both the schema cache and the db connection per connection and not globally
• the connections have a idle timeout of a few minutes after which we drop the connection from the cache
• we select the current project when the user opens a file. this is the same pattern biome uses and I think it makes sense.
• EDIT: I want to check if we even need to track the current project since all requests should include the path from which we can find the correct one -> we cannot easily do that. I would go with the current approach for now and maybe do it in a follow up with a better analysis.
• EDIT: Will fix windows path issues for tests 🥲 -> done
• EDIT: Will remove all the windows debug logs 🥲 -> done

ToDo:

• add methods to workspace api
• migrate schema cache manager to store per-connection
• integration ls
• integration cli
• make test green
• implement try_from_payload with extends
• add test for extends config

closes #334

[juleswritescode]

won't there be race conditions if load is called multiple times in parallel and there is no schema_cache in cache? We would SchemaCache::load a couple of times, right?

[psteinroe]

good catch!

[juleswritescode]

Is there state where a user requests diagnostics before or while settings are set up, for example when the file opens?
If so, I think we should return an emtpy response. But if not, it's fine to expect – we do expect the settings to be present for a working LSP right?

[psteinroe]

yeah, makes sense. added that.

[Copilot]

Pull Request Overview

This PR adds workspace folder management and extends support in configuration loading, ensuring per-connection caches and idle timeouts, and updates LSP/CLI integrations.

• Introduce register_project_folder/unregister_project_folder API and per-project data storage
• Implement extends logic in PartialConfiguration with file resolution and merging
• Wire workspace registration in LSP and CLI sessions and update file system resolver integration

Reviewed Changes

Copilot reviewed 34 out of 34 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
crates/pgt_workspace/src/workspace.rs	Add project registration/unregistration and slotmap-based storage
crates/pgt_workspace/src/settings.rs	Manage per-project settings and current-project logic
crates/pgt_workspace/src/lib.rs	Re-export new PartialConfigurationExt trait
crates/pgt_workspace/src/diagnostics.rs	Bridge CantLoadExtendFile into WorkspaceError
crates/pgt_workspace/src/configuration.rs	Replace payload conversion with try_from_payload and implement extends merging
crates/pgt_workspace/Cargo.toml	Add slotmap dependency for project keys
crates/pgt_lsp/src/session.rs	Introduce client info, call register_project_folder during initialization
crates/pgt_lsp/src/server.rs	Handle workspace folder changes and register/unregister methods
crates/pgt_fs/src/fs/os.rs	Integrate oxc_resolver for config resolution
crates/pgt_fs/src/fs/memory.rs	Add stub for resolve_configuration in memory FS
crates/pgt_fs/src/fs.rs	Extend FileSystem trait with resolve_configuration
crates/pgt_fs/Cargo.toml	Add oxc_resolver dependency
crates/pgt_diagnostics/src/adapters.rs	Wrap resolver errors into diagnostics
crates/pgt_diagnostics/Cargo.toml	Add oxc_resolver dependency
crates/pgt_configuration/src/lib.rs	Add extends field and default in partial config
crates/pgt_configuration/src/diagnostics.rs	New diagnostics for extend resolution and load errors
crates/pgt_configuration/Cargo.toml	Add oxc_resolver dependency
crates/pgt_cli/src/diagnostics.rs	Update expected diagnostic size
crates/pgt_cli/src/commands/mod.rs	Register workspace folder in CLI commands
Cargo.toml	Add oxc_resolver and slotmap to workspace

Comments suppressed due to low confidence (2)

crates/pgt_workspace/src/configuration.rs:349

• The detection logic treats only .jsonc as local files; .json entries will be resolved externally. Consider including b"json" in the match to handle .json extends as local paths.

if extend_entry_as_path.starts_with(".") || matches!(extend_entry_as_path.extension().map(OsStr::as_encoded_bytes), Some(b"jsonc")) { 

crates/pgt_lsp/src/session.rs:492

• This call to .await is inside a synchronous function, which will fail to compile. Either make initialize async or spawn a separate async task to log the message.

self.client.log_message(MessageType::ERROR, &error).await; 

[juleswritescode]

Beautiful!!!