@rentaltide/cli 0.1.0 → 0.1.1

package/dist/api.js

@@ -52,11 +52,28 @@ function safeJson(text) {
         return { message: text.slice(0, 300) };
     }
 }
+/**
+ * Fetch one app.
+ *
+ * The endpoint answers `{ success, app }`, not the app — reading it flat gives
+ * an object whose every field is undefined, which is not an error anywhere: the
+ * diff simply shows every value being cleared. Unwrapped in one place so no
+ * command can get it wrong again.
+ */
+export async function getApp(appId) {
+    const data = await request(`/developer/apps/${appId}`);
+    const app = data.app ?? data;
+    if (!app?.appId)
+        fail('That app could not be loaded.', 'Run `rentaltide link` to pick another.');
+    return app;
+}
 /** Project config → the field names the API expects. */
 export function configToAppPayload(config) {
     return {
         appName: config.name,
-        appSlug: config.slug,
+        // `appSlug` is deliberately absent: the API cannot change a slug after
+        // creation, so sending one is a silent no-op that shows in a diff as a
+        // change that never happens.
         developerName: config.developerName,
         description: config.description,
         category: config.category,

package/dist/commands/dev.js

@@ -14,7 +14,7 @@
  *  - It restores the previous embed URL on exit, including Ctrl-C, and says so
  *    if it could not.
  */
-import { request } from '../api.js';
+import { getApp, request } from '../api.js';
 import { requireAppConfig } from '../config.js';
 import { bold, cyan, dim, fail, ok, say, step, warn, yellow } from '../ui.js';
 const DEFAULT_DEV_URL = 'http://localhost:5173';
@@ -24,7 +24,7 @@ export async function dev(args) {
         fail('This project is not linked yet.', 'Run `rentaltide link`.');
     const flagIndex = args.findIndex((a) => a === '--url');
     const devUrl = (flagIndex >= 0 ? args[flagIndex + 1] : undefined) || config.dev?.embedUrl || DEFAULT_DEV_URL;
-    const app = await request(`/developer/apps/${config.appId}`);
+    const app = await getApp(config.appId);
     if (app.status === 'approved') {
         fail(`${app.appName} is approved and installed by real merchants.`, 'Repointing its embed URL would point all of them at your laptop. Work on a draft copy instead.');
     }
@@ -38,6 +38,15 @@ export async function dev(args) {
     if (!sandbox)
         fail('Could not create your sandbox.');
     await request(`/developer/apps/${config.appId}/sandbox-install`, { method: 'POST' }).catch(() => undefined);
+    // Is anything actually listening? A sandbox pointed at a dead port renders a
+    // blank frame, which looks like the platform failing rather than a dev server
+    // that was never started.
+    try {
+        await fetch(devUrl, { signal: AbortSignal.timeout(2000) });
+    }
+    catch {
+        warn(`Nothing is answering at ${devUrl} yet — start your dev server.`);
+    }
     step(`Pointing ${app.appName} at ${bold(devUrl)}`);
     await request(`/developer/apps/${config.appId}`, {
         method: 'PUT',
@@ -74,7 +83,14 @@ export async function dev(args) {
     say();
     say(dim('Open the console from the partner portal → App Sandbox → Open sandbox console.'));
     say(dim('Edit your app and refresh the page. No redeploy, no reinstall.'));
-    say();
+    if (devUrl.startsWith('http://')) {
+        // http://localhost is a potentially-trustworthy origin, so Chrome and
+        // Firefox embed it inside an https page. Safari does not, and shows an
+        // empty frame with the reason only in its console.
+        say(dim('Chrome and Firefox will embed a plain-http localhost URL. Safari will not —\n' +
+            '  use --url https://localhost:5173 with a local certificate there.'));
+        say();
+    }
     say(yellow('Leave this running.') + dim(' Ctrl-C restores the embed URL.'));
     // Nothing to poll: the page reloads from the developer's own dev server.
     // This process exists to hold the restore handler.

package/dist/commands/link.js

@@ -1,5 +1,5 @@
 /** Attach this directory to one of your apps, and pull its config down. */
-import { appToConfig, request } from '../api.js';
+import { appToConfig, getApp, request } from '../api.js';
 import { readAppConfig, requireAppConfig, writeAppConfig, CONFIG_FILENAME } from '../config.js';
 import { ask, bold, dim, fail, ok, say } from '../ui.js';
 async function listApps() {
@@ -27,7 +27,7 @@ export async function pull() {
     const config = requireAppConfig();
     if (!config.appId)
         fail('This project is not linked yet.', 'Run `rentaltide link`.');
-    const app = await request(`/developer/apps/${config.appId}`);
+    const app = await getApp(config.appId);
     writeAppConfig(appToConfig(app, config));
     ok(`Pulled ${app.appName} into ${CONFIG_FILENAME}.`);
 }

package/dist/commands/push.js

@@ -5,7 +5,7 @@
  * may read and where it renders; applying a change to them silently because a
  * file moved is not a thing a tool should do.
  */
-import { appToConfig, configToAppPayload, request } from '../api.js';
+import { appToConfig, configToAppPayload, getApp, request } from '../api.js';
 import { requireAppConfig } from '../config.js';
 import { bold, confirm, dim, fail, green, ok, red, say } from '../ui.js';
 function flatten(value) {
@@ -19,7 +19,7 @@ export async function push(args) {
     const config = requireAppConfig();
     if (!config.appId)
         fail('This project is not linked yet.', 'Run `rentaltide link`.');
-    const current = await request(`/developer/apps/${config.appId}`);
+    const current = await getApp(config.appId);
     const before = appToConfig(current);
     const fields = [
         ['name', before.name, config.name],

package/dist/index.js

@@ -15,7 +15,7 @@ import { init } from './commands/init.js';
 import { webhook } from './commands/webhook.js';
 import { open } from './commands/open.js';
 import { bold, dim, say } from './ui.js';
-const VERSION = '0.1.0';
+const VERSION = '0.1.1';
 function usage() {
     say(`${bold('rentaltide')} ${dim(VERSION)} — build apps for RentalTide`);
     say();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rentaltide/cli",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Build RentalTide apps: scaffold, run against your sandbox, push config, trigger webhooks.",
   "license": "MIT",
   "author": "RentalTide Inc.",