pennyrouter 0.3.12 → 0.3.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pennyrouter",
-  "version": "0.3.12",
+  "version": "0.3.14",
   "description": "Install and manage PennyRouter local coding-agent integrations.",
   "homepage": "https://pennyrouter.com",
   "bugs": {

package/src/cli.js

@@ -174,7 +174,7 @@ async function profiles(flags) {
   }
   if (requested) {
     // Resolve before saving so an unknown name fails without changing anything.
-    const selected = resolvePennyProfile(config, requested);
+    const selected = resolvePennyProfile(config, requested, { requireExisting: true });
     await savePennyConfig({ ...selected, active_profile: requested });
     console.log(`Active profile is now "${requested}" (${selected.gateway_base_url}).`);
     return;

package/src/launch.js

@@ -151,16 +151,19 @@ export function activePennyProfile(config) {
  * `gateway_base_url`. Every consumer reads those two fields, so resolving here keeps the
  * profile lookup in one place rather than threading a profile through each caller.
  *
- * An unconfigured machine resolves to a profile with no credentials rather than throwing:
- * `install` has to run against a config that does not exist yet.
+ * A name that does not exist yet resolves to a profile with no credentials rather than
+ * throwing, because `install --profile private` has to read the profile it is about to
+ * create. Only an explicit `requireExisting` lookup — selecting a profile to launch or
+ * switch to — treats an unknown name as an error.
  */
-export function resolvePennyProfile(config, name = null) {
+export function resolvePennyProfile(config, name = null, { requireExisting = false } = {}) {
   const requested = name || activePennyProfile(config);
   const profiles = config.profiles || {};
   const entry = profiles[requested];
   if (!entry) {
-    if (Object.keys(profiles).length === 0) return { ...config, profile: requested };
-    const known = Object.keys(profiles).join(", ");
+    if (!requireExisting) return { ...config, profile: requested };
+    const names = Object.keys(profiles);
+    const known = names.length ? names.join(", ") : "none configured";
     throw new Error(`Unknown PennyRouter profile "${requested}". Available: ${known}.`);
   }
   return {
@@ -183,13 +186,13 @@ export function setPennyProfile(config, name, { apiKey, gatewayBaseUrl }) {
   };
 }
 
-export async function loadPennyConfig({ profile = null } = {}) {
+export async function loadPennyConfig({ profile = null, requireExisting = false } = {}) {
   const config = (await readJsonFile(pennyConfigPath(), null)) || {
     version: CONFIG_VERSION,
     profiles: {},
     harnesses: {},
   };
-  return resolvePennyProfile(config, profile);
+  return resolvePennyProfile(config, profile, { requireExisting });
 }
 
 /**
@@ -1039,7 +1042,10 @@ export async function runPenny(argv = process.argv.slice(2)) {
   }
   const selection = extractProfileArgs(argv.slice(1));
   argv = [harness, ...selection.args];
-  const config = await loadPennyConfig({ profile: selection.profile });
+  const config = await loadPennyConfig({
+    profile: selection.profile,
+    requireExisting: Boolean(selection.profile),
+  });
   if (harness === "claude" && argv[1]?.toLowerCase() === "desktop") {
     const option = argv[2]?.toLowerCase();
     if (option === "--restore" || option === "restore") {

package/src/launch.test.js

@@ -284,6 +284,10 @@ async function main() {
     });
     assert.ok(paid.includes('env_key = "PENNYROUTER_API_KEY"'));
     assert.ok(!paid.includes("requires_openai_auth"));
+
+    // Inside the fixture: testProfiles persists a config, which must land in the
+    // temporary HOME and never in the developer's real ~/.config/pennyrouter.
+    await testProfiles();
   } finally {
     for (const key of ENV_KEYS) {
       if (previous[key] === undefined) delete process.env[key];
@@ -292,7 +296,6 @@ async function main() {
     fs.rmSync(root, { recursive: true, force: true });
   }
 
-  await testProfiles();
   console.log("launch.test.js: all checks passed");
 }
 
@@ -333,7 +336,21 @@ async function testProfiles() {
     resolvePennyProfile(config, "cloud").gateway_base_url,
     "https://api.pennyrouter.com",
   );
-  assert.throws(() => resolvePennyProfile(config, "nope"), /Unknown PennyRouter profile/);
+  // A not-yet-created profile resolves empty: `install --profile private` must be able to
+  // read the profile it is about to write, even when other profiles already exist.
+  const pending = resolvePennyProfile(config, "brand-new");
+  assert.equal(pending.profile, "brand-new");
+  assert.equal(pending.api_key, undefined);
+  // Selecting one to launch or switch to is the opposite: an unknown name must fail loudly
+  // rather than silently launching with no credentials.
+  assert.throws(
+    () => resolvePennyProfile(config, "nope", { requireExisting: true }),
+    /Unknown PennyRouter profile "nope"\. Available: cloud, private\./,
+  );
+  await assert.rejects(
+    loadPennyConfig({ profile: "nope", requireExisting: true }),
+    /Unknown PennyRouter profile/,
+  );
 
   // A resolved config must not persist one profile's credentials at the top level,
   // where they would shadow whichever profile is selected on the next load.