fix: gracefully handle skills/list RPC errors instead of rejecting entire probe

When skills/list returns an error (e.g. on older Codex CLI versions that
don't support it), fall back to an empty skills array instead of calling
fail() which would reject the entire discovery promise and break account
probing.

Author: cursoragent

apps/server/src/provider/codexAppServer.ts

@@ -205,12 +205,7 @@ export async function probeCodexDiscovery(input: {
 
       if (response.id === 2) {
         const errorMessage = readErrorMessage(response);
-        if (errorMessage) {
-          fail(new Error(`skills/list failed: ${errorMessage}`));
-          return;
-        }
-
-        skills = parseCodexSkillsResult(response.result, input.cwd);
+        skills = errorMessage ? [] : parseCodexSkillsResult(response.result, input.cwd);
         maybeResolve();
         return;
       }