@faable/faable 1.30.0 → 1.31.0

package/dist/api/FaableApi.js

@@ -12,6 +12,20 @@ const firstPage = async (res) => {
     const items = (await res).results;
     return items;
 };
+// Walk the cursor to exhaustion. `list()` needs this instead of `firstPage`:
+// a user who sees many apps (admins see all of them) gets a multi-page
+// listing, and matching by repository against a truncated first page made
+// every repo-resolved command answer "No app linked to this repository".
+const allPages = async (fetch_page) => {
+    const items = [];
+    let next;
+    do {
+        const page = await fetch_page(next);
+        items.push(...page.results);
+        next = page.next ?? undefined;
+    } while (next);
+    return items;
+};
 const data = async (res) => {
     const items = (await res).data;
     return items;
@@ -84,7 +98,9 @@ class FaableApi {
         return new FaableApi(config);
     }
     async list() {
-        return firstPage(data(this.client.get(`/app`)));
+        return allPages((next) => data(this.client.get(`/app`, {
+            params: { pageSize: 200, ...(next ? { next } : {}) },
+        })));
     }
     async getBySlug(slug) {
         return data(this.client.get(`/app/slug/${slug}`));

package/dist/commands/deploy/index.js

@@ -19,7 +19,9 @@ import { is_superseded } from './superseded.js';
 
 const deploy = {
     command: 'deploy [app_id]',
-    describe: 'Deploy a faable app',
+    // Name the subcommand groups so `faable --help` makes them discoverable
+    // without digging into `faable deploy --help`.
+    describe: 'Deploy a faable app and manage it (secrets, domains, logs, deployments…)',
     builder: yargs => {
         // Product subcommands live under `deploy` (yargs matches them before the
         // app_id positional, so `faable deploy <app_id>` keeps working).

package/dist/commands/deploy/resolve_app_id.js

@@ -5,10 +5,11 @@ import { log } from '../../log.js';
 // app_id resolution (the user never has to look one up):
 //  1. explicit (positional on `deploy`, --app on subcommands)
 //  2. OIDC in CI — the backend resolves the app from the linked repository
-//  3. locally — the app saved by `faable deploy link` in faable.json
+//  3. locally — a legacy app_id in faable.json (older CLIs wrote it on
+//     `faable deploy link`; the current link only persists in the API)
 //  4. locally — the app whose linked repository matches the git origin remote
 //     of the working directory (repos are connected in the dashboard when the
-//     app is created, so most working copies never ran `link`)
+//     app is created, or via `faable deploy link`)
 const resolve_app_id = async (explicit, ctxAppId, api, workdir = process.cwd()) => {
     const app_id = explicit || ctxAppId || Configuration.instance().app_id;
     if (app_id)

package/dist/commands/link/index.js

@@ -1,7 +1,6 @@
 import { requireApi } from '../../api/context.js';
 import prompts from 'prompts';
 import { log } from '../../log.js';
-import { Configuration } from '../../lib/Configuration.js';
 import { getGitRemoteUrl } from '../../lib/git_remote.js';
 
 const DEPLOY_DOCS_URL = "https://faable.com/docs/deploy/github-actions";
@@ -27,9 +26,27 @@ const link = {
             log.warn('Passing an app_id to "faable link" is deprecated and ignored. ' +
                 'Just run "faable link" and select the app from the list.');
         }
-        const config = Configuration.instance();
-        if (config.app_id) {
-            log.info(`This repository is already linked to app: "${config.app_slug}" (${config.app_id})`);
+        log.info("Checking local git repository...");
+        const gitUrl = await getGitRemoteUrl(workdir);
+        if (!gitUrl) {
+            log.error("No git remote URL detected. Add a GitHub 'origin' remote and try again.");
+            return;
+        }
+        const { api } = await requireApi();
+        const apps = await api.list();
+        if (apps.length === 0) {
+            log.error("No apps found in your account. Create one first at https://faable.com");
+            return;
+        }
+        // The link lives in the API (app.repository) — every command resolves the
+        // app from the git remote, so nothing is written locally. Warn (and
+        // confirm) when the repository is already linked to an app.
+        const alreadyLinked = apps.filter((app) => app.repository === gitUrl);
+        if (alreadyLinked.length > 0) {
+            const names = alreadyLinked
+                .map((app) => `"${app.name}" (${app.id})`)
+                .join(", ");
+            log.info(`This repository is already linked to: ${names}`);
             const { relink } = await prompts({
                 type: "toggle",
                 name: "relink",
@@ -42,14 +59,6 @@ const link = {
                 return;
             }
         }
-        const { api } = await requireApi();
-        log.info("Checking local git repository...");
-        const gitUrl = await getGitRemoteUrl(workdir);
-        const apps = await api.list();
-        if (apps.length === 0) {
-            log.error("No apps found in your account. Create one first at https://faable.com");
-            return;
-        }
         const { selectedApp } = await prompts({
             type: "select",
             name: "selectedApp",
@@ -64,10 +73,6 @@ const link = {
             return;
         }
         log.info(`Linking to "${selectedApp.name}" (${selectedApp.id})...`);
-        if (!gitUrl) {
-            log.error("No git remote URL detected. Add a GitHub 'origin' remote and try again.");
-            return;
-        }
         // The API verifies that the user has a connected GitHub identity AND
         // access to the repository before persisting the link.
         let linked;
@@ -95,9 +100,6 @@ const link = {
             return;
         }
         log.info(`Linked repository ${gitUrl} to ${selectedApp.name}.`);
-        // Save locally for CLI convenience (only after the API confirms the link)
-        Configuration.instance().saveConfig({ app_slug: selectedApp.name, app_id: selectedApp.id });
-        log.info(`Successfully linked local repository to ${selectedApp.name}.`);
         // Deploy v4: push-to-deploy is server-side by default — no workflow to
         // scaffold. A repo that brings its own Faable workflow keeps deploying
         // through it (the API leaves the trigger on the Action in that case).

package/dist/index.js

@@ -50,6 +50,10 @@ yg.scriptName('faable')
     .command(upgrade)
     .command(link_deprecated)
     .demandCommand(1)
+    // Reject unknown (sub)commands loudly. Without this, a typo'd or
+    // not-yet-existing command (`faable secrets list` on a version where it
+    // lived elsewhere) exited silently with just the version banner.
+    .strictCommands()
     .help()
     .fail(function (msg, err) {
     if (err) {
@@ -58,8 +62,11 @@ yg.scriptName('faable')
         return;
     }
     if (msg) {
+        // Validation failure (unknown command, missing subcommand…): show the
+        // help, then fail red — a bad invocation must not exit 0.
         yg.showHelp();
-        log.info(msg);
+        log.error(`❌ ${msg}`);
+        process.exit(1);
     }
 })
     .parse(hideBin(process.argv), {});

package/dist/lib/Configuration.js

@@ -23,12 +23,6 @@ class Configuration {
             }
         }
     }
-    saveConfig(updates) {
-        this.config = { ...this.config, ...updates };
-        const config_path = path__default.join(process.cwd(), this.config_file);
-        fs.writeJSONSync(config_path, this.config, { spaces: 2 });
-        log.info(`Configuration saved to: ${this.config_file}`);
-    }
     static instance() {
         if (!Configuration._instance) {
             Configuration._instance = new Configuration();
@@ -58,9 +52,6 @@ class Configuration {
             next: this.config.next,
         };
     }
-    get app_slug() {
-        return this.config.app_slug;
-    }
     get app_id() {
         return this.config.app_id;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@faable/faable",
-  "version": "1.30.0",
+  "version": "1.31.0",
   "main": "dist/index.js",
   "license": "MIT",
   "author": "Marc Pomar <marc@faable.com>",