From 306ea47dc6edf02dd74d4643b1091cf53e16906f Mon Sep 17 00:00:00 2001
From: 2witstudios <2witstudios@gmail.com>
Date: Mon, 22 Jun 2026 12:18:37 -0500
Subject: [PATCH] =?UTF-8?q?fix(onboarding):=20don't=20set=20PAGESPACE=5FMO?=
 =?UTF-8?q?DEL=5FPAGES=20=E2=80=94=20let=20auto-discovery=20run?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Setting PAGESPACE_MODEL_PAGES from onboarding bypassed the extension's
auto-discovery (which only runs when the var is unset). Auto-discovery
fetches agent names; without it config.models has bare {id} objects,
breaking the Shift+Tab cycling which matches by name. Onboarding now
only shows what was discovered for informational output — the extension
handles registration on every launch.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 bin/pagespace.mjs | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/bin/pagespace.mjs b/bin/pagespace.mjs
index 79d7f1b..9528bd9 100755
--- a/bin/pagespace.mjs
+++ b/bin/pagespace.mjs
@@ -417,10 +417,9 @@ async function runOnboarding() {
   const credPath = writeCredentials(buildCredentialRecord({ token, apiUrl: base }));
   if (state.defaultDrive) process.env.PAGESPACE_DRIVE = state.defaultDrive;
   const allModels = state.models ?? [];
-  if (allModels.length > 0) process.env.PAGESPACE_MODEL_PAGES = allModels.map((m) => m.id).join(",");
   console.log(`  ✓ drive: ${state.defaultDrive || "(none)"}`);
   if (allModels.length > 0) {
-    console.log(`  ✓ ${allModels.length} agent model(s): ${allModels.map((m) => m.name).join(", ")}`);
+    console.log(`  ✓ ${allModels.length} agent model(s) discovered: ${allModels.map((m) => m.name).join(", ")}`);
   } else {
     console.log("  · no agent models found — set PAGESPACE_MODEL_PAGES manually if needed.");
   }