@displaydev/cli 0.6.0 → 0.7.0

package/README.md

@@ -99,6 +99,38 @@ dsp mcp
 
 Visibility and share flags **default to "keep current"** on update: omitting `--public`/`--company` leaves visibility unchanged, and omitting `--share`/`--clear-shares` leaves the guest list unchanged. `--public`/`--company` are mutually exclusive; `--share`/`--clear-shares` are mutually exclusive.
 
+## Testing
+
+The CLI suite uses three distinct fetch-faking strategies, each pinned to a
+layer of the stack. New contributors pick the one that matches the layer
+they're testing — adding a fourth is almost always wrong.
+
+- **Unit-level fetch fake (`stubFetchJson(...)`)** — for exercising
+  `ApiClient` and anything that composes it purely in-process. Low ceremony,
+  full control over responses, no cross-process boundary. Use the
+  `stubFetchJson(body, init?)` helper from
+  [`cli/src/test-helpers.ts`](src/test-helpers.ts) — it stubs `global.fetch`
+  with a fresh JSON `Response` per call and returns the `vi.fn()` so tests
+  can still inspect `fetchMock.mock.calls[...]`. Fall back to raw
+  `vi.stubGlobal('fetch', vi.fn().mockRejectedValue(...))` only for network-
+  failure cases. Representative example:
+  [`cli/src/api-client.spec.ts`](src/api-client.spec.ts) and the integration
+  wire-up assertion in
+  [`cli/src/mcp-mode-selection.spec.ts`](src/mcp-mode-selection.spec.ts).
+- **Hand-rolled `createMockApiClient()`** — for MCP tool specs that verify
+  argument-routing from an MCP tool handler to the right `ApiClient` method.
+  No HTTP in the loop at all; the fake is a plain object of `vi.fn()`s.
+  Representative example: [`cli/src/mcp-server.spec.ts`](src/mcp-server.spec.ts)
+  and [`cli/src/mcp-publish-file-read.spec.ts`](src/mcp-publish-file-read.spec.ts).
+- **Real localhost HTTP fixture (`createHttpFixture` + `spawnCli`)** — for
+  `program.action` E2E specs that need to exercise commander argv parsing,
+  the stdout/stderr split, exit codes, and the full HTTP round-trip. Drives
+  the CLI as a real child process. Representative examples:
+  [`cli/src/cli-mode-selection.spec.ts`](src/cli-mode-selection.spec.ts),
+  [`cli/src/cli-subcommands.spec.ts`](src/cli-subcommands.spec.ts),
+  [`cli/src/login-otp.spec.ts`](src/login-otp.spec.ts),
+  [`cli/src/login-otp-send.spec.ts`](src/login-otp-send.spec.ts).
+
 ## MCP
 
 The CLI doubles as an MCP server over stdio. Add to your MCP client config:

package/dist/api-client.js

@@ -1,7 +1,13 @@
 /**
  * HTTP client for the display.dev API.
  * Used by the stdio MCP server to proxy tool calls to the REST API.
- */ function asyncGeneratorStep(gen, resolve, reject, _next, _throw, key, arg) {
+ */ function _assert_this_initialized(self) {
+    if (self === void 0) {
+        throw new ReferenceError("this hasn't been initialised - super() hasn't been called");
+    }
+    return self;
+}
+function asyncGeneratorStep(gen, resolve, reject, _next, _throw, key, arg) {
     try {
         var info = gen[key](arg);
         var value = info.value;
@@ -30,11 +36,32 @@ function _async_to_generator(fn) {
         });
     };
 }
+function _call_super(_this, derived, args) {
+    derived = _get_prototype_of(derived);
+    return _possible_constructor_return(_this, _is_native_reflect_construct() ? Reflect.construct(derived, args || [], _get_prototype_of(_this).constructor) : derived.apply(_this, args));
+}
 function _class_call_check(instance, Constructor) {
     if (!(instance instanceof Constructor)) {
         throw new TypeError("Cannot call a class as a function");
     }
 }
+function _construct(Parent, args, Class) {
+    if (_is_native_reflect_construct()) {
+        _construct = Reflect.construct;
+    } else {
+        _construct = function construct(Parent, args, Class) {
+            var a = [
+                null
+            ];
+            a.push.apply(a, args);
+            var Constructor = Function.bind.apply(Parent, a);
+            var instance = new Constructor();
+            if (Class) _set_prototype_of(instance, Class.prototype);
+            return instance;
+        };
+    }
+    return _construct.apply(null, arguments);
+}
 function _defineProperties(target, props) {
     for(var i = 0; i < props.length; i++){
         var descriptor = props[i];
@@ -62,6 +89,25 @@ function _define_property(obj, key, value) {
     }
     return obj;
 }
+function _get_prototype_of(o) {
+    _get_prototype_of = Object.setPrototypeOf ? Object.getPrototypeOf : function getPrototypeOf(o) {
+        return o.__proto__ || Object.getPrototypeOf(o);
+    };
+    return _get_prototype_of(o);
+}
+function _inherits(subClass, superClass) {
+    if (typeof superClass !== "function" && superClass !== null) {
+        throw new TypeError("Super expression must either be null or a function");
+    }
+    subClass.prototype = Object.create(superClass && superClass.prototype, {
+        constructor: {
+            value: subClass,
+            writable: true,
+            configurable: true
+        }
+    });
+    if (superClass) _set_prototype_of(subClass, superClass);
+}
 function _instanceof(left, right) {
     "@swc/helpers - instanceof";
     if (right != null && typeof Symbol !== "undefined" && right[Symbol.hasInstance]) {
@@ -70,6 +116,9 @@ function _instanceof(left, right) {
         return left instanceof right;
     }
 }
+function _is_native_function(fn) {
+    return Function.toString.call(fn).indexOf("[native code]") !== -1;
+}
 function _object_spread(target) {
     for(var i = 1; i < arguments.length; i++){
         var source = arguments[i] != null ? arguments[i] : {};
@@ -85,6 +134,57 @@ function _object_spread(target) {
     }
     return target;
 }
+function _possible_constructor_return(self, call) {
+    if (call && (_type_of(call) === "object" || typeof call === "function")) {
+        return call;
+    }
+    return _assert_this_initialized(self);
+}
+function _set_prototype_of(o, p) {
+    _set_prototype_of = Object.setPrototypeOf || function setPrototypeOf(o, p) {
+        o.__proto__ = p;
+        return o;
+    };
+    return _set_prototype_of(o, p);
+}
+function _type_of(obj) {
+    "@swc/helpers - typeof";
+    return obj && typeof Symbol !== "undefined" && obj.constructor === Symbol ? "symbol" : typeof obj;
+}
+function _wrap_native_super(Class) {
+    var _cache = typeof Map === "function" ? new Map() : undefined;
+    _wrap_native_super = function wrapNativeSuper(Class) {
+        if (Class === null || !_is_native_function(Class)) return Class;
+        if (typeof Class !== "function") {
+            throw new TypeError("Super expression must either be null or a function");
+        }
+        if (typeof _cache !== "undefined") {
+            if (_cache.has(Class)) return _cache.get(Class);
+            _cache.set(Class, Wrapper);
+        }
+        function Wrapper() {
+            return _construct(Class, arguments, _get_prototype_of(this).constructor);
+        }
+        Wrapper.prototype = Object.create(Class.prototype, {
+            constructor: {
+                value: Wrapper,
+                enumerable: false,
+                writable: true,
+                configurable: true
+            }
+        });
+        return _set_prototype_of(Wrapper, Class);
+    };
+    return _wrap_native_super(Class);
+}
+function _is_native_reflect_construct() {
+    try {
+        var result = !Boolean.prototype.valueOf.call(Reflect.construct(Boolean, [], function() {}));
+    } catch (_) {}
+    return (_is_native_reflect_construct = function() {
+        return !!result;
+    })();
+}
 function _ts_generator(thisArg, body) {
     var f, y, t, _ = {
         label: 0,
@@ -184,6 +284,20 @@ function _ts_generator(thisArg, body) {
         };
     }
 }
+export var ApiError = /*#__PURE__*/ function(Error1) {
+    "use strict";
+    _inherits(ApiError, Error1);
+    function ApiError(message, status) {
+        _class_call_check(this, ApiError);
+        var _this;
+        _this = _call_super(this, ApiError, [
+            message
+        ]), _define_property(_this, "status", void 0), _this.status = status;
+        _this.name = 'ApiError';
+        return _this;
+    }
+    return ApiError;
+}(_wrap_native_super(Error));
 export var ApiClient = /*#__PURE__*/ function() {
     "use strict";
     function ApiClient(config) {
@@ -210,6 +324,36 @@ export var ApiClient = /*#__PURE__*/ function() {
                 }).call(this);
             }
         },
+        {
+            key: "publishPublic",
+            value: /**
+   * Anonymous (no API key) publish. Returns a claimable preview URL plus
+   * the one-shot claim URL the user must click to own the artifact.
+   */ function publishPublic(params) {
+                return _async_to_generator(function() {
+                    var ext, blob, form;
+                    return _ts_generator(this, function(_state) {
+                        ext = params.format === 'md' ? '.md' : '.html';
+                        blob = new Blob([
+                            params.content
+                        ], {
+                            type: 'application/octet-stream'
+                        });
+                        form = new FormData();
+                        form.append('file', blob, "content".concat(ext));
+                        if (params.name) {
+                            form.append('name', params.name);
+                        }
+                        return [
+                            2,
+                            this.doFetch('POST', '/v1/public/artifacts', form, {
+                                'X-Client-Type': this.clientType
+                            })
+                        ];
+                    });
+                }).call(this);
+            }
+        },
         {
             key: "publishVersion",
             value: function publishVersion(shortId, params) {
@@ -223,6 +367,22 @@ export var ApiClient = /*#__PURE__*/ function() {
                 }).call(this);
             }
         },
+        {
+            key: "share",
+            value: /**
+   * Delta-update an artifact's visibility or sharedWith list without
+   * re-publishing content (spec §5 share endpoint).
+   */ function share(shortId, params) {
+                return _async_to_generator(function() {
+                    return _ts_generator(this, function(_state) {
+                        return [
+                            2,
+                            this.request('POST', "/v1/artifacts/".concat(shortId, "/share"), params)
+                        ];
+                    });
+                }).call(this);
+            }
+        },
         {
             key: "buildPublishForm",
             value: function buildPublishForm(params) {
@@ -248,7 +408,7 @@ export var ApiClient = /*#__PURE__*/ function() {
                     try {
                         for(var _iterator = params.share[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true){
                             var email = _step.value;
-                            form.append('guestEmails', email);
+                            form.append('sharedWith', email);
                         }
                     } catch (err) {
                         _didIteratorError = true;
@@ -266,7 +426,7 @@ export var ApiClient = /*#__PURE__*/ function() {
                     }
                 }
                 if (params.clearShares) {
-                    form.append('clearGuestEmails', 'true');
+                    form.append('clearSharedWith', 'true');
                 }
                 if (params.showBranding) {
                     form.append('showBranding', params.showBranding);
@@ -378,6 +538,70 @@ export var ApiClient = /*#__PURE__*/ function() {
                 }).call(this);
             }
         },
+        {
+            key: "exportSource",
+            value: /**
+   * Fetch the raw source bytes of an artifact (the markdown for `.md`,
+   * the served HTML for `.html`). Returns the body plus the response's
+   * Content-Type so callers can disambiguate. Throws `ApiError` on
+   * non-2xx (the CLI maps 404 to a clean message + exit 4).
+   */ function exportSource(shortId, version) {
+                return _async_to_generator(function() {
+                    var _res_headers_get, qs, url, res, _parsed_message, text, parsed, arrayBuffer;
+                    return _ts_generator(this, function(_state) {
+                        switch(_state.label){
+                            case 0:
+                                qs = version !== undefined ? "?v=".concat(version) : '';
+                                url = "".concat(this.baseUrl, "/v1/artifacts/").concat(shortId, "/source").concat(qs);
+                                return [
+                                    4,
+                                    fetch(url, {
+                                        method: 'GET',
+                                        headers: {
+                                            'Authorization': "Bearer ".concat(this.apiKey),
+                                            'X-Client-Type': this.clientType
+                                        }
+                                    })
+                                ];
+                            case 1:
+                                res = _state.sent();
+                                if (!!res.ok) return [
+                                    3,
+                                    3
+                                ];
+                                return [
+                                    4,
+                                    res.text()
+                                ];
+                            case 2:
+                                text = _state.sent();
+                                try {
+                                    parsed = JSON.parse(text);
+                                } catch (unused) {
+                                    parsed = {
+                                        message: text
+                                    };
+                                }
+                                throw new ApiError((_parsed_message = parsed.message) !== null && _parsed_message !== void 0 ? _parsed_message : "API error ".concat(res.status), res.status);
+                            case 3:
+                                return [
+                                    4,
+                                    res.arrayBuffer()
+                                ];
+                            case 4:
+                                arrayBuffer = _state.sent();
+                                return [
+                                    2,
+                                    {
+                                        data: Buffer.from(arrayBuffer),
+                                        contentType: (_res_headers_get = res.headers.get('content-type')) !== null && _res_headers_get !== void 0 ? _res_headers_get : 'application/octet-stream'
+                                    }
+                                ];
+                        }
+                    });
+                }).call(this);
+            }
+        },
         {
             key: "delete",
             value: function _delete(shortId) {
@@ -634,7 +858,7 @@ export var ApiClient = /*#__PURE__*/ function() {
                                         message: text
                                     };
                                 }
-                                throw new Error((_parsed_message = parsed.message) !== null && _parsed_message !== void 0 ? _parsed_message : "API error ".concat(res.status));
+                                throw new ApiError((_parsed_message = parsed.message) !== null && _parsed_message !== void 0 ? _parsed_message : "API error ".concat(res.status), res.status);
                             case 3:
                                 return [
                                     2,

package/dist/config.js

@@ -0,0 +1,220 @@
+function asyncGeneratorStep(gen, resolve, reject, _next, _throw, key, arg) {
+    try {
+        var info = gen[key](arg);
+        var value = info.value;
+    } catch (error) {
+        reject(error);
+        return;
+    }
+    if (info.done) {
+        resolve(value);
+    } else {
+        Promise.resolve(value).then(_next, _throw);
+    }
+}
+function _async_to_generator(fn) {
+    return function() {
+        var self = this, args = arguments;
+        return new Promise(function(resolve, reject) {
+            var gen = fn.apply(self, args);
+            function _next(value) {
+                asyncGeneratorStep(gen, resolve, reject, _next, _throw, "next", value);
+            }
+            function _throw(err) {
+                asyncGeneratorStep(gen, resolve, reject, _next, _throw, "throw", err);
+            }
+            _next(undefined);
+        });
+    };
+}
+function _ts_generator(thisArg, body) {
+    var f, y, t, _ = {
+        label: 0,
+        sent: function() {
+            if (t[0] & 1) throw t[1];
+            return t[1];
+        },
+        trys: [],
+        ops: []
+    }, g = Object.create((typeof Iterator === "function" ? Iterator : Object).prototype), d = Object.defineProperty;
+    return d(g, "next", {
+        value: verb(0)
+    }), d(g, "throw", {
+        value: verb(1)
+    }), d(g, "return", {
+        value: verb(2)
+    }), typeof Symbol === "function" && d(g, Symbol.iterator, {
+        value: function() {
+            return this;
+        }
+    }), g;
+    function verb(n) {
+        return function(v) {
+            return step([
+                n,
+                v
+            ]);
+        };
+    }
+    function step(op) {
+        if (f) throw new TypeError("Generator is already executing.");
+        while(g && (g = 0, op[0] && (_ = 0)), _)try {
+            if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
+            if (y = 0, t) op = [
+                op[0] & 2,
+                t.value
+            ];
+            switch(op[0]){
+                case 0:
+                case 1:
+                    t = op;
+                    break;
+                case 4:
+                    _.label++;
+                    return {
+                        value: op[1],
+                        done: false
+                    };
+                case 5:
+                    _.label++;
+                    y = op[1];
+                    op = [
+                        0
+                    ];
+                    continue;
+                case 7:
+                    op = _.ops.pop();
+                    _.trys.pop();
+                    continue;
+                default:
+                    if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) {
+                        _ = 0;
+                        continue;
+                    }
+                    if (op[0] === 3 && (!t || op[1] > t[0] && op[1] < t[3])) {
+                        _.label = op[1];
+                        break;
+                    }
+                    if (op[0] === 6 && _.label < t[1]) {
+                        _.label = t[1];
+                        t = op;
+                        break;
+                    }
+                    if (t && _.label < t[2]) {
+                        _.label = t[2];
+                        _.ops.push(op);
+                        break;
+                    }
+                    if (t[2]) _.ops.pop();
+                    _.trys.pop();
+                    continue;
+            }
+            op = body.call(thisArg, _);
+        } catch (e) {
+            op = [
+                6,
+                e
+            ];
+            y = 0;
+        } finally{
+            f = t = 0;
+        }
+        if (op[0] & 5) throw op[1];
+        return {
+            value: op[0] ? op[1] : void 0,
+            done: true
+        };
+    }
+}
+import { readFile, mkdir, writeFile, rename, chmod } from 'node:fs/promises';
+import { homedir } from 'node:os';
+import { dirname, join } from 'node:path';
+export function defaultConfigIo() {
+    return {
+        path: join(homedir(), '.displaydev', 'config.json')
+    };
+}
+export function loadConfig() {
+    return _async_to_generator(function() {
+        var io, raw, unused;
+        var _arguments = arguments;
+        return _ts_generator(this, function(_state) {
+            switch(_state.label){
+                case 0:
+                    io = _arguments.length > 0 && _arguments[0] !== void 0 ? _arguments[0] : defaultConfigIo();
+                    _state.label = 1;
+                case 1:
+                    _state.trys.push([
+                        1,
+                        3,
+                        ,
+                        4
+                    ]);
+                    return [
+                        4,
+                        readFile(io.path, 'utf-8')
+                    ];
+                case 2:
+                    raw = _state.sent();
+                    return [
+                        2,
+                        JSON.parse(raw)
+                    ];
+                case 3:
+                    unused = _state.sent();
+                    return [
+                        2,
+                        null
+                    ];
+                case 4:
+                    return [
+                        2
+                    ];
+            }
+        });
+    }).apply(this, arguments);
+}
+export function saveConfig(_0) {
+    return _async_to_generator(function(config) {
+        var io, tmpPath;
+        var _arguments = arguments;
+        return _ts_generator(this, function(_state) {
+            switch(_state.label){
+                case 0:
+                    io = _arguments.length > 1 && _arguments[1] !== void 0 ? _arguments[1] : defaultConfigIo();
+                    return [
+                        4,
+                        mkdir(dirname(io.path), {
+                            recursive: true
+                        })
+                    ];
+                case 1:
+                    _state.sent();
+                    tmpPath = "".concat(io.path, ".tmp");
+                    return [
+                        4,
+                        writeFile(tmpPath, "".concat(JSON.stringify(config, null, 2), "\n"), {
+                            mode: 384
+                        })
+                    ];
+                case 2:
+                    _state.sent();
+                    return [
+                        4,
+                        rename(tmpPath, io.path)
+                    ];
+                case 3:
+                    _state.sent();
+                    return [
+                        4,
+                        chmod(io.path, 384)
+                    ];
+                case 4:
+                    _state.sent();
+                    return [
+                        2
+                    ];
+            }
+        });
+    }).apply(this, arguments);
+}