@faable/faable 4.21.1 → 4.22.1

package/dist/commands/deploy/apps/name_free.js

@@ -11,8 +11,11 @@ const assert_name_free = async (api, project, name) => {
         limit: 200,
         all: true
     });
-    const wanted = name.toLowerCase();
-    const clash = apps.data.find(a => a.name.toLowerCase() === wanted);
+    // Same comparison as the api (app_name_unique.plugin.ts): trimmed, NFC,
+    // case-insensitive.
+    const key = (s) => s.trim().normalize('NFC').toLowerCase();
+    const wanted = key(name);
+    const clash = apps.data.find(a => key(a.name) === wanted);
     if (clash) {
         throw new CliError('name_taken', `${project} already has an app named "${clash.name}" (${clash.id}). Deploy to it with --app ${clash.id}, or choose another name.`);
     }

package/dist/commands/deploy/remote/follow.js

@@ -31,6 +31,19 @@ const retry_of = (deployment) => {
  * was about to succeed (backlog §379).
  */
 const TAIL_CHARS = 200;
+/**
+ * The build's warnings, after its log (api `deployment.build_warnings`). The
+ * same lines are in the log above, but scrolled away among hundreds of npm
+ * lines; a deploy that ends READY is exactly the one nobody scrolls back on.
+ */
+const format_build_warnings = (warnings) => {
+    if (!warnings?.length)
+        return '';
+    const n = warnings.length;
+    return (`\n⚠️  ${n} build warning${n === 1 ? '' : 's'} — the build worked around ${n === 1 ? 'this' : 'these'}; fix ${n === 1 ? 'it' : 'them'} so it does not have to:\n` +
+        warnings.map(w => `   • ${w.message}\n`).join('') +
+        '\n');
+};
 /**
  * The part of `content` not shown yet, given how much was `printed` and the
  * last characters of it (`seen_tail`).
@@ -111,11 +124,18 @@ out = process.stdout) => {
         // already live (READY) — the promotion poll takes over from here.
         if (phase === 'INITIALIZING' || phase === 'READY') {
             await tail_logs();
-            return { id: deployment_id };
+            const build_warnings = deployment?.build_warnings;
+            const summary = format_build_warnings(build_warnings);
+            if (summary)
+                out.write(summary);
+            return {
+                id: deployment_id,
+                ...(build_warnings?.length ? { build_warnings } : {})
+            };
         }
         await wait(POLL_INTERVAL_MS);
     }
     throw new Error('Timed out waiting for the remote build. Check the dashboard for its status.');
 };
 
-export { follow_remote_build, unseen };
+export { follow_remote_build, format_build_warnings, unseen };

package/dist/commands/deploy/resolve_app_id.js

@@ -21,8 +21,11 @@ in_project) => {
         limit: 200,
         all: true
     });
-    const wanted = ref.toLowerCase();
-    const matches = apps.data.filter(a => a.name.toLowerCase() === wanted || a.slug?.toLowerCase() === wanted);
+    // NFC: an app name is free text, and a decomposed "é" (typed on macOS,
+    // pasted from a file) must match the composed one the api stores.
+    const key = (s) => s?.trim().normalize('NFC').toLowerCase();
+    const wanted = key(ref);
+    const matches = apps.data.filter(a => key(a.name) === wanted || key(a.slug) === wanted);
     if (matches.length === 1)
         return matches[0].id;
     if (matches.length === 0) {

package/dist/mcp/tools.js

@@ -16,6 +16,12 @@ const target = (a) => ['--app', a.app, ...scope(a)];
 const opt = (flag, value) => value === undefined || value === null || value === ''
     ? []
     : [flag, String(value)];
+// Free text that may start with "-" (an app name is any text): as a separate
+// argument yargs reads "--no-deploy" as a flag of its own and the name comes
+// out empty. Glued with "=" it can only be the value.
+const optText = (flag, value) => value === undefined || value === null || value === ''
+    ? []
+    : [`${flag}=${String(value)}`];
 const READ = {
     readOnlyHint: true,
     destructiveHint: false,
@@ -711,7 +717,7 @@ const TOOLS = [
         flags: a => [
             '--repo',
             a.repo,
-            ...opt('--name', a.name),
+            ...optText('--name', a.name),
             ...opt('--branch', a.branch),
             ...(a.deploy === false ? ['--no-deploy'] : []),
             ...(a.wait && a.deploy !== false ? ['--wait', '--timeout', '300'] : []),
@@ -898,7 +904,7 @@ const TOOLS = [
             '--workdir',
             a.workdir,
             ...opt('--app', a.app),
-            ...opt('--create', a.create),
+            ...optText('--create', a.create),
             '--yes',
             ...scope(a)
         ],

package/package.json

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