Show pony status bar success only after LSP client starts (#14)

Author: orien

.release-notes/14.md

@@ -0,0 +1,9 @@
+## Show pony status bar success only after LSP client starts
+
+Previously, the `Pony LSP ✓` status bar indicator was shown as soon as the
+`pony-lsp` executable was found on disk — before the language client had
+actually connected. If the server failed to start, the status bar would briefly
+flash a false success before switching to `✗`.
+
+The indicator now only shows `✓` once `client.start()` has resolved
+successfully, giving an accurate reflection of the server's readiness.

src/extension.ts

@@ -58,8 +58,6 @@ export async function activate(_context: ExtensionContext) {
     }
     outputChannel.appendLine(`Using ${lspExecutable} from PATH`);
   }
-  showPony(true);
-
   // Set or append to PONYPATH environment variable
   const ponyStdLibPath = config.get<string>('ponyStdLibPath', '');
   const env = { ...process.env };
@@ -108,6 +106,7 @@ export async function activate(_context: ExtensionContext) {
   try {
     await client.start();
     outputChannel.appendLine("Pony language server client ready");
+    showPony(true);
   } catch (reason) {
     window.showWarningMessage(`Pony language server client failed: ${reason}`);
     showPony(false);