@myapihq/cli 2.27.1 → 2.27.2

package/dist/errors.js

@@ -125,6 +125,28 @@ export function friendlyError(err) {
         out += '\n  Current numbers: myapi account sending';
         return withOrgContext(out, err);
     }
+    // A failure on the platform's side, with no message of its own. The handler
+    // that raises it calls envelope.Error(w, r, 500, "internal_error") — a bare
+    // string, so there is no `message` to fall through to and the caller was left
+    // looking at the word `internal_error` alone: no cause, no next step, and
+    // nothing to quote at anybody.
+    //
+    // The one genuinely useful thing about a 500 is its request id, which the
+    // platform returns on every response and the SDK used to discard.
+    //
+    // Deliberately claims nothing about what did or did not happen. A 500 can
+    // land before or after the write, and inventing "nothing was changed" here
+    // would be the kind of reassuring guess this codebase keeps having to undo.
+    if (err.status >= 500 || err.code === 'internal_error') {
+        const lines = ['The platform failed on this request — this is our fault, not a bad command.'];
+        if (err.detail && err.detail !== err.code)
+            lines.push(`  ${err.detail}`);
+        lines.push(err.requestId
+            ? `  Quote this when reporting it: ${err.requestId}`
+            : '  No request id came back, so quote the exact command and the time instead.');
+        lines.push('  Whether the change went through is not knowable from here — check before retrying.');
+        return withOrgContext(lines.join('\n'), err);
+    }
     const base = ERROR_MESSAGES[err.code] || err.code;
     // Generic fallback: append the backend's `message` when it adds info beyond
     // the friendly mapping. Future per-service detail fields can be added here.

package/dist/errors.test.js

@@ -68,3 +68,42 @@ describe('paused sending points at the CLI, not an HTTP route', () => {
         expect(msg).toContain('myapi account resume-sending');
     });
 });
+describe('a 500 must be reportable', () => {
+    // `envelope.Error(w, r, 500, "internal_error")` sends a bare string: no
+    // message to fall through to. The caller used to see the word
+    // `internal_error` alone — no cause, no next step, nothing to quote. Hit it
+    // for real on 2026-08-23 running `funnel push` with an empty --funnel.
+    //
+    // The one useful thing about a 500 is the request id, which the platform
+    // returns on every response and the SDK discarded.
+    function fail(code, status, requestId, detail) {
+        const e = new MyApiError(code, status, detail);
+        if (requestId)
+            e.requestId = requestId;
+        return e;
+    }
+    it('quotes the request id so it can be reported and traced', () => {
+        const out = friendlyError(fail('internal_error', 500, 'req_1adc0b171e187b45'));
+        expect(out).toContain('req_1adc0b171e187b45');
+        expect(out).toMatch(/our fault/i);
+    });
+    it('says what to quote when no request id came back', () => {
+        const out = friendlyError(fail('internal_error', 500));
+        expect(out).toMatch(/exact command/i);
+    });
+    it('never claims the write did not happen', () => {
+        // A 500 can land either side of the write. Saying "nothing was changed"
+        // would be a reassuring guess, and this codebase keeps having to undo those.
+        const out = friendlyError(fail('internal_error', 500, 'req_x'));
+        expect(out).not.toMatch(/nothing was (changed|written|charged)/i);
+        expect(out).toMatch(/not knowable|check before retrying/i);
+    });
+    it('applies to any 5xx, not only the internal_error code', () => {
+        expect(friendlyError(fail('BAD_GATEWAY', 502, 'req_y'))).toContain('req_y');
+    });
+    it('leaves 4xx alone — those have their own guidance', () => {
+        const out = friendlyError(fail('INVALID_SLUG', 422, 'req_z', 'slug must look like /about'));
+        expect(out).toContain('slug must look like /about');
+        expect(out).not.toMatch(/our fault/i);
+    });
+});

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@myapihq/cli",
   "license": "Apache-2.0",
-  "version": "2.27.1",
+  "version": "2.27.2",
   "description": "MyAPI command-line interface",
   "repository": {
     "type": "git",
@@ -46,7 +46,7 @@
     "lint:skills:strict": "node scripts/copy-skills.js && node scripts/lint-skills.js --strict"
   },
   "dependencies": {
-    "@myapihq/sdk": "^2.27.1"
+    "@myapihq/sdk": "^2.27.2"
   },
   "devDependencies": {
     "@types/node": "^25.6.0",