fix: MiniMax CN authentication and connection test timeout

[Kathie-yu]

Summary

Fix MiniMax CN (minimax-cn) authentication failure and connection test timeout in Pi API Key mode. This serves as a follow-up fix to the MiniMax split introduced in v0.7.3 (Issue #386), addressing underlying SDK routing and timeout issues.

Changes

• Fix provider alias: Removed incorrect minimax-cn → minimax mapping in PI_AUTH_PROVIDER_ALIASES — Pi SDK treats them as separate providers with distinct endpoints (api.minimax.io vs api.minimaxi.com).
• Hide MiniMax in Anthropic mode: Added PI_ONLY_PRESET_KEYS filter so MiniMax presets only appear in Pi API Key mode where they can actually authenticate.
• Fix test timeout: Added lightweight direct HTTP test for Pi providers using the anthropic-messages API, bypassing the heavy subprocess startup that exceeded the 20s timeout limit.
• Add display name: Added minimax-cn entry to PI_AUTH_PROVIDER_DISPLAY_NAMES.

Testing

• Anthropic API Key mode: MiniMax presets no longer appear ✅
• Pi API Key mode → MiniMax CN: connection test passes ✅
• Pi API Key mode → MiniMax CN: chat works in session ✅
• Type checking: all affected packages pass tsc --noEmit ✅

[Copilot]

Pull request overview

Fixes MiniMax CN (minimax-cn) authentication failures and Pi connection test timeouts by correcting provider routing/aliasing and introducing a faster direct HTTP connection test for Anthropic-compatible Pi providers.

Changes:

• Remove incorrect minimax-cn → minimax aliasing and add minimax-cn display name support.
• Hide MiniMax presets in Anthropic API Key mode where Pi SDK-based authentication is required.
• Add a lightweight Anthropic-compatible /v1/messages HTTP connection test path for Pi providers, falling back to the existing subprocess-based test when not applicable.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
packages/shared/src/agent/backend/internal/drivers/pi.ts	Adds direct HTTP connection test for anthropic-messages Pi providers to avoid subprocess startup timeouts.
packages/shared/src/agent/backend/internal/driver-types.ts	Updates testConnection driver contract to allow “decline to handle” via null.
packages/shared/src/agent/backend/factory.ts	Handles null driver results by falling through to the generic subprocess connection test.
packages/server-core/src/domain/connection-setup-logic.ts	Adds display name mapping for minimax-cn.
apps/electron/src/renderer/components/apisetup/submit-helpers.ts	Fixes Pi auth provider aliasing so minimax-cn passes through as its own provider key.
apps/electron/src/renderer/components/apisetup/ApiKeyInput.tsx	Filters out MiniMax presets in Anthropic API Key mode; keeps them for Pi API Key flow.

Comments suppressed due to low confidence (1)

apps/electron/src/renderer/components/apisetup/submit-helpers.ts:27

• resolvePiAuthProviderForSubmit behavior changed for MiniMax variants (only minimax-global is now aliased; minimax-cn should pass through). There are existing bun tests for this helper (ApiKeyInput.test.ts), but they don’t cover these MiniMax cases—please add assertions for minimax-global → minimax and minimax-cn → minimax-cn to prevent regressions.

export function resolvePiAuthProviderForSubmit(
  activePreset: PresetKey,
  lastNonCustomPreset: PresetKey | null
): string | undefined {
  if (activePreset === 'custom') {
    // Pi SDK needs a provider hint for auth header formatting even when
    // the URL is user-provided — default to anthropic as the safest baseline.
    const resolved = lastNonCustomPreset && lastNonCustomPreset !== 'custom'
      ? lastNonCustomPreset
      : 'anthropic'
    return PI_AUTH_PROVIDER_ALIASES[resolved] ?? resolved
  }

  return PI_AUTH_PROVIDER_ALIASES[activePreset] ?? activePreset
}

---

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

You can also share your feedback on Copilot code review. Take the survey.

[rjulius23]

Was my bad, sorey fixing it on main.

[Kathie-yu]

Hi @rjulius23, I've rebased onto v0.7.4, all fixes apply cleanly.

Heads up on a new issue I found: MiniMax CN API (api.minimaxi.com) rejects model IDs with the
MiniMax- prefix — e.g. MiniMax-M2.5-highspeed → 500, but M2.5-highspeed → 200. The Pi SDK
registry uses prefixed IDs, which work on Global but not CN.

I added a 3-line fix in two places (guarded by piAuthProvider === 'minimax-cn', zero impact on
other providers):

• pi-agent-server/src/index.ts — strip prefix in resolvePiModel() for chat path
• shared/.../drivers/pi.ts — strip prefix in testConnection() for connection test

Happy to close if you'd rather fix on main — just flagging since it needs a CN API key to
reproduce.

[rjulius23]

Yes sorry was an oversight from my side fixing the minimax-cn in todays release and introduce webhooks as well.

[Kathie-yu]

Thanks for the heads up! Closing this PR then. Looking forward to the webhooks 🎉