Merge branch 'main' into changeset-release/main

Author: elkaix

.changeset/managed-provider-kimi-code.md

@@ -0,0 +1,8 @@
+---
+"@pythoughts/pythinker-code": minor
+"@pythoughts/pythinker-code-sdk": minor
+---
+
+Name the managed OAuth provider after the platform that serves it. It is reached over `auth.kimi.com` and `api.kimi.com`, but it was registered as `managed:pythinker-code`, which read as a first-party service in a client that talks to several providers. The provider id is now `managed:kimi-code`, its models are aliased `kimi-code/*`, and its credentials are stored under `oauth/kimi-code`.
+
+This is a breaking change for an existing config: the previous entries are not rewritten, so run `pythinker login` once to provision the managed provider under its current name, then remove the stale `managed:pythinker-code` entry.

.changeset/session-free-skill-listing.md

@@ -0,0 +1,5 @@
+---
+"@pythoughts/pythinker-code": minor
+---
+
+Resolve a workspace's skills without opening a session, so an editor panel can list them before its first message.

.changeset/shared-catalog-provider-import.md

@@ -0,0 +1,6 @@
+---
+"@pythoughts/pythinker-code-sdk": minor
+"@pythoughts/pythinker-code": patch
+---
+
+Add an SDK routine that imports a catalog provider and its models into the persisted config, and use it for the CLI provider import so both entry points preserve existing defaults the same way.

.changeset/tui-render-gate.md

@@ -0,0 +1,5 @@
+---
+"@pythoughts/pythinker-code": patch
+---
+
+Stop the fixed-layout TUI anchoring its first frames to the shell cursor, which pushed the panel border into scrollback.

.changeset/vscode-chat-logo-and-slash-insert.md

@@ -0,0 +1,5 @@
+---
+"pythinker-code": patch
+---
+
+Show the assistant logo beside replies without breaking the step timeline, complete a picked slash command in the input instead of sending it on its own, ship a decodable Marketplace icon, and retry transient registry failures when publishing.

README.md

@@ -10,7 +10,7 @@
 <br />
 
 [![npm version](https://img.shields.io/npm/v/@pythoughts/pythinker-code?style=for-the-badge&logo=npm&logoColor=white&color=CB3837&label=pythinker-code)](https://www.npmjs.com/package/@pythoughts/pythinker-code)
-[![Downloads](https://img.shields.io/npm/dm/@pythoughts/pythinker-code?style=for-the-badge&logo=npm&logoColor=white&color=16a34a&label=downloads)](https://www.npmjs.com/package/@pythoughts/pythinker-code)
+[![Downloads](https://img.shields.io/npm/dm/@pythoughts/pythinker-code?style=for-the-badge&logo=npm&logoColor=white&color=2b89ff&label=downloads)](https://www.npmjs.com/package/@pythoughts/pythinker-code)
 [![Node.js](https://img.shields.io/badge/Node.js-26%2B-339933?style=for-the-badge&logo=nodedotjs&logoColor=white)](https://github.com/Pythoughts-labs/pythinker-code/blob/main/package.json)
 [![License: MIT](https://img.shields.io/badge/License-MIT-16a34a.svg?style=for-the-badge)](https://github.com/Pythoughts-labs/pythinker-code/blob/main/LICENSE)
 [![CI](https://img.shields.io/github/actions/workflow/status/Pythoughts-labs/pythinker-code/ci.yml?branch=main&label=CI&style=for-the-badge&logo=githubactions&logoColor=white)](https://github.com/Pythoughts-labs/pythinker-code/actions/workflows/ci.yml?query=branch%3Amain)

apps/pythinker-code/package.json

@@ -95,7 +95,7 @@
     "web-tree-sitter": "0.25.10"
   },
   "devDependencies": {
-    "@earendil-works/pi-tui": "^0.81.1",
+    "@earendil-works/pi-tui": "^0.83.0",
     "@pythoughts/acp-adapter": "workspace:^",
     "@pythoughts/dashboard-server": "workspace:^",
     "@pythoughts/dashboard-web": "workspace:*",

apps/pythinker-code/src/cli/sub/provider.ts

@@ -17,17 +17,16 @@ import {
   CustomRegistryApiError,
   fetchCustomRegistry,
   type CustomRegistrySource,
-  type ManagedPythinkerConfigShape,
+  type ManagedKimiConfigShape,
 } from '@pythoughts/pythinker-code-oauth';
 import {
-  applyCatalogProvider,
-  catalogBaseUrl,
   catalogConnectionWire,
   catalogProviderModels,
   CatalogFetchError,
   createPythinkerHarness,
   DEFAULT_CATALOG_URL,
   fetchCatalog,
+  importCatalogProvider,
   inferWireType,
   type Catalog,
   type CatalogProviderEntry,
@@ -366,74 +365,20 @@ export async function handleCatalogAdd(
   }
 
   const harness = deps.getHarness();
-  await harness.ensureConfigFile();
-
-  let config = await harness.getConfig();
-
-  // Capture defaults BEFORE `removeProvider`, because that call clears
-  // `defaultModel` when it points at one of this provider's aliases (see
-  // `core-impl.ts removePythinkerProvider`). Without this, re-importing an
-  // already-configured provider would lose the user's previously-set default
-  // even when `--default-model` is not supplied.
-  const previousDefaultProvider = config.defaultProvider;
-  const previousDefaultModel = config.defaultModel;
-  const previousDefaultThinking = config.defaultThinking;
-
-  if (config.providers[providerId] !== undefined) {
-    config = await harness.removeProvider(providerId);
-  }
-
-  const baseUrl = catalogBaseUrl(entry, wire);
-  // `applyCatalogProvider` always overwrites both `defaultModel` and
-  // `defaultThinking`. The values we pass here are temporary; we restore
-  // a consistent state in the post-apply block below.
-  applyCatalogProvider(config, {
-    providerId,
-    catalogUrl: url,
-    wire,
-    baseUrl,
-    apiKey: useEnvVar ? undefined : literalApiKey,
-    apiKeyEnvVar: useEnvVar ? apiKeyEnvVar : undefined,
-    models,
-    selectedModelId: opts.defaultModel ?? '',
-    thinking: false,
-  });
-
-  // Resolve the final `defaultModel`:
-  //   - If the caller asked for one, `applyCatalogProvider` already set it.
-  //   - Else, restore the previous default ONLY when its alias still resolves
-  //     after the catalog refresh; the catalog may have dropped the old
-  //     model, in which case restoring would point default_model at a
-  //     non-existent alias and break the next session.
-  if (opts.defaultModel === undefined) {
-    const stillResolves =
-      previousDefaultModel !== undefined &&
-      config.models?.[previousDefaultModel] !== undefined;
-    config.defaultModel = stillResolves ? previousDefaultModel : undefined;
+  try {
+    await importCatalogProvider(harness, {
+      providerId,
+      entry,
+      catalogUrl: url,
+      apiKey: useEnvVar ? undefined : literalApiKey,
+      apiKeyEnvVar: useEnvVar ? apiKeyEnvVar : undefined,
+      defaultModel: opts.defaultModel,
+    });
+  } catch (error) {
+    deps.stderr.write(`${errorMessage(error)}\n`);
+    deps.exit(1);
   }
 
-  // Mirror the defaultModel handling for defaultProvider: `removeProvider`
-  // cleared it, and the catalog refresh may have dropped the provider id.
-  config.defaultProvider =
-    previousDefaultProvider !== undefined && config.providers[previousDefaultProvider] !== undefined
-      ? previousDefaultProvider
-      : undefined;
-
-  // Always restore `defaultThinking` from what was there before — including
-  // `undefined`. Persisting `false` when the user never set it would make
-  // `resolveThinkingLevel` (agent-core/src/agent/config/thinking.ts) treat
-  // it as an explicit "off" request and silently disable thinking, even
-  // for thinking-capable models.
-  config.defaultThinking = previousDefaultThinking;
-
-  await harness.setConfig({
-    providers: config.providers,
-    models: config.models,
-    defaultProvider: config.defaultProvider,
-    defaultModel: config.defaultModel,
-    defaultThinking: config.defaultThinking,
-  });
-
   const displayName = entry.name ?? providerId;
   deps.stdout.write(
     `Imported ${displayName} (${providerId}) with ${String(models.length)} model${models.length === 1 ? '' : 's'} from ${url}.\n`,
@@ -572,8 +517,8 @@ function resolveApiKey(flag: string | undefined, env: NodeJS.ProcessEnv): string
   return undefined;
 }
 
-function asManaged(config: PythinkerConfig): ManagedPythinkerConfigShape {
-  return config as unknown as ManagedPythinkerConfigShape;
+function asManaged(config: PythinkerConfig): ManagedKimiConfigShape {
+  return config as unknown as ManagedKimiConfigShape;
 }
 
 function providerSourceLabel(provider: PythinkerConfig['providers'][string]): string {

apps/pythinker-code/src/cli/telemetry.ts

@@ -1,4 +1,4 @@
-import { createPythinkerDeviceId, PYTHINKER_CODE_PROVIDER_NAME } from '@pythoughts/pythinker-code-oauth';
+import { createPythinkerDeviceId, KIMI_CODE_PROVIDER_NAME } from '@pythoughts/pythinker-code-oauth';
 import {
   PythinkerAuthFacade,
   loadRuntimeConfigSafe,
@@ -55,7 +55,7 @@ export function initializeCliTelemetry(options: InitializeCliTelemetryOptions):
     uiMode: options.uiMode,
     model: options.model ?? options.config.defaultModel,
     getAccessToken: async () =>
-      (await options.harness.auth.getCachedAccessToken(PYTHINKER_CODE_PROVIDER_NAME)) ?? null,
+      (await options.harness.auth.getCachedAccessToken(KIMI_CODE_PROVIDER_NAME)) ?? null,
   });
   if (options.bootstrap.firstLaunch) {
     options.harness.track('first_launch');
@@ -102,7 +102,7 @@ export function initializeServerTelemetry(
     version: options.version,
     uiMode: WEB_UI_MODE,
     model: config.defaultModel,
-    getAccessToken: async () => (await auth.getCachedAccessToken(PYTHINKER_CODE_PROVIDER_NAME)) ?? null,
+    getAccessToken: async () => (await auth.getCachedAccessToken(KIMI_CODE_PROVIDER_NAME)) ?? null,
   });
 
   return {

apps/pythinker-code/src/constant/app.ts

@@ -34,7 +34,7 @@ export const PYTHINKER_CODE_BANNER_DIR_NAME = 'banner';
 export const PYTHINKER_CODE_BANNER_STATE_FILE_NAME = 'state.json';
 
 // Managed Pythinker auth provider key shared with OAuth/SDK config.
-export const DEFAULT_OAUTH_PROVIDER_NAME = 'managed:pythinker-code';
+export { KIMI_CODE_PROVIDER_NAME as DEFAULT_OAUTH_PROVIDER_NAME } from '@pythoughts/pythinker-code-oauth';
 
 // SDK/core error code that tells the TUI to show a login-required startup
 // notice. Derived from sdk's ErrorCodes so a future rename in core