Refactor ClientMetadata to ClientInfo and enhance path parsing

[kunlinglio]

No description provided.

[Copilot]

Pull request overview

This PR refactors how client/server environment metadata is represented (moving from a global ClientMetadata to per-server EditorInfo and a new ServerInfo), and enhances path parsing to recognize more path-like tokens (bare paths and colon-delimited mount/PATH-style segments).

Changes:

• Replace global client environment (ClientMetadata + async global getter/setter) with EditorInfo stored on PathServer via OnceLock, and add ServerInfo for version/mode logging.
• Improve path extraction by recognizing bare path tokens (not just quoted strings) and by splitting colon-delimited tokens to extract volume-mount style paths.
• Update language identifiers and documentation/changelog entries accordingly.

Reviewed changes

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

Show a summary per file

File	Description
src/server.rs	Stores editor/server metadata on the server instance and uses it in feature gating (e.g., document links/hover).
src/server_info.rs	Introduces ServerInfo/ServerMode for version + debug/release mode reporting.
src/providers/document_link.rs	Removes dependence on global client metadata; expects server-side gating for capability checks.
src/parser/path/regex.rs	Extends regex fallback extraction to include bare, whitespace-delimited path tokens.
src/parser/path/mod.rs	Adds colon-delimited splitting to extract paths from docker-compose/PATH-like tokens; adds tests.
src/lib.rs	Updates module layout to remove client and add editor_info/server_info.
src/editor_info.rs	Adds EditorInfo extracted from LSP InitializeParams.
src/document/language.rs	Adds a dockercompose language identifier mapping.
src/client.rs	Removes the global client environment module.
README.md	TODO list updated (needs alignment with implemented behavior).
CHANGELOG.md	Adds Unreleased notes (needs alignment with actual refactor naming).

Comments suppressed due to low confidence (2)

src/server.rs:432

• Same issue as in document_link: .get().expect("Editor info must be initialized") can panic if hover is invoked before initialization. Consider gracefully handling the missing editor_info case instead of panicking.

        let editor_info = self
            .editor_info
            .get()
            .expect("Editor info must be initialized");
        let config = self.get_config().await;

src/parser/path/mod.rs:133

• In this test case, end_byte should match the content length (exclusive end) to keep PathCandidate ranges consistent. The string "./data:/app/data:ro" is 19 bytes long, but end_byte is set to 18.

        let candidate = PathCandidate {
            content: "./data:/app/data:ro".to_string(),
            start_byte: 0,
            end_byte: 18,
        };

---

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