@myapihq/cli 2.16.0 → 2.16.1

package/dist/commands/feedback.js

@@ -20,6 +20,11 @@ export const EXPOSES = [
     'DELETE /feedback/orgs/{org_id}/widgets/{id}',
 ];
 export const SCHEMA = {
+    // Declared because `list` reads flags.limit/offset, --help documents both,
+    // and a truncated page prints "raise --limit" — while the parser refused
+    // them as unknown. Reported by ImmoPilot on 2026-08-19.
+    limit: 'number',
+    offset: 'number',
     trace: 'boolean',
     routes: 'string',
     accent: 'string',
@@ -109,6 +114,12 @@ export async function list(flags) {
             info('  screenshot   yes');
         else if (i.screenshot_asset_id)
             info('  screenshot   yes (link could not be signed)');
+        // A missing picture used to be silent, and silence read as "nobody
+        // attached one" when it often meant "the renderer threw on this page".
+        // The backend records which it was; say so rather than printing nothing.
+        else if (i.screenshot_status === 'failed') {
+            info(`  screenshot   FAILED${i.screenshot_error ? ` — ${i.screenshot_error}` : ''}`);
+        }
         if (flags.trace && trace.length) {
             info('');
             for (const e of trace) {
@@ -219,6 +230,14 @@ export async function widget(sub, arg, flags) {
             // the URL from a key and a base by hand.
             info(`  ${w.script_tag || `<script src="https://api.myapihq.com/feedback/in/${w.key}/widget.js" async></script>`}`);
             info(`  routes:  ${w.routes?.length ? w.routes.join(', ') : '(everywhere)'}`);
+            // The API returns these inside `theme`; printing only name/key/routes
+            // meant the only way to read back a widget's position or label was to
+            // fetch widget.js and read the JavaScript.
+            const th = w.theme ?? {};
+            if (th.accent || th.position || th.label) {
+                info(`  theme:   ${[th.label && `label "${th.label}"`, th.position, th.accent]
+                    .filter(Boolean).join(' · ')}`);
+            }
             info(`  origins: ${w.allowed_origins?.length ? w.allowed_origins.join(', ') : '(any — anyone can post with this key)'}`);
             info('');
         }

package/dist/sdk-feedback-widgets.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/sdk-feedback-widgets.test.js

@@ -0,0 +1,59 @@
+// SDK-level unit tests for feedback.listWidgets — the unwrap that made
+// `myapi feedback widget list` report "No widgets yet" to orgs that had
+// widgets. Mocks global fetch — does NOT hit the network.
+//
+// Reported by ImmoPilot four times before it was traced. The HTTP API was
+// answering correctly throughout: GET /feedback/orgs/{org}/widgets returns
+// { widgets, total }, the same envelope shape as the item list. The SDK typed
+// it Promise<Widget[]> and returned the object unwrapped, so the CLI's
+// `!ws.length` was always true. Nothing tested the SDK against a real response
+// shape, which is the only reason a four-time report was possible.
+import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
+import { feedback } from '@myapihq/sdk';
+const API_KEY = 'myapi_test_abc';
+const ORG = '7f1aa7f1-f6eb-4a6d-87b7-1bbe6584f5d6';
+let fetchMock;
+function ok(data) {
+    return new Response(JSON.stringify({ success: true, data, meta: {} }), {
+        status: 200,
+        headers: { 'content-type': 'application/json' },
+    });
+}
+const WIDGET = {
+    id: '27de19f6-1a18-4074-a347-a14e737c3f47',
+    org_id: ORG,
+    name: 'app',
+    key: 'fw_0123456789abcdef',
+    routes: ['/app/**'],
+    theme: { accent: '#2563eb', position: 'bottom-left', label: 'Feedback' },
+};
+describe('feedback.listWidgets', () => {
+    beforeEach(() => {
+        fetchMock = vi.fn();
+        vi.stubGlobal('fetch', fetchMock);
+    });
+    afterEach(() => vi.unstubAllGlobals());
+    it('unwraps the { widgets, total } envelope the API actually returns', async () => {
+        fetchMock.mockResolvedValueOnce(ok({ widgets: [WIDGET, { ...WIDGET, name: 'marketing' }], total: 2 }));
+        const ws = await feedback.listWidgets(API_KEY, ORG);
+        expect(Array.isArray(ws)).toBe(true);
+        expect(ws).toHaveLength(2);
+        expect(ws[0].key).toBe(WIDGET.key);
+    });
+    it('carries theme through, so position and label can be read back', async () => {
+        // Without this the only way to see what --position and --label were set to
+        // was to fetch widget.js and read the JavaScript.
+        fetchMock.mockResolvedValueOnce(ok({ widgets: [WIDGET], total: 1 }));
+        const [w] = await feedback.listWidgets(API_KEY, ORG);
+        expect(w.theme?.position).toBe('bottom-left');
+        expect(w.theme?.label).toBe('Feedback');
+    });
+    it('returns [] rather than undefined when the org has none', async () => {
+        fetchMock.mockResolvedValueOnce(ok({ widgets: [], total: 0 }));
+        await expect(feedback.listWidgets(API_KEY, ORG)).resolves.toEqual([]);
+    });
+    it('tolerates a bare array, in case the endpoint is ever simplified', async () => {
+        fetchMock.mockResolvedValueOnce(ok([WIDGET]));
+        await expect(feedback.listWidgets(API_KEY, ORG)).resolves.toHaveLength(1);
+    });
+});

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@myapihq/cli",
   "license": "Apache-2.0",
-  "version": "2.16.0",
+  "version": "2.16.1",
   "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.16.0"
+    "@myapihq/sdk": "^2.16.1"
   },
   "devDependencies": {
     "@types/node": "^25.6.0",