dsh-plugin-toolkit 0.9.2 → 0.9.3

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## 0.9.3
+
+- The panel needed one more thing than `dsh.client`: the host resolves the bundle
+  through `exports["./client"]` (`clientExportOf`), and drops the row when that
+  subpath is missing. 0.9.2 declared `dsh.client` but its `exports` map had no
+  `./client`, so the tab was still absent — this release exports `./client` and
+  `doctor` now checks the same thing the host does.
+- `init` scaffolds `exports: { '.': './index.js', './client': './client.js' }`.
+  Every plugin generated before this release was missing it, so every generated
+  panel was invisible for the same reason.
+- `doctor` reports a `dsh.client` whose `exports["./client"]` file does not exist,
+  and a `dsh.client` with no `./client` export at all. The `files`-list gap is now
+  caught at the manifest stage instead of at pack time.
+
 ## 0.9.2
 
 - `doctor` reports a shipped client bundle that no `dsh.client` declares. The host

package/lib/doctor.mjs

@@ -122,8 +122,20 @@ export function checkClientDeclaration(dir, manifest) {
   const client = manifest.dsh?.client;
   if (client !== undefined && client !== null) {
     const platform = client.platform;
-    if (platform === undefined || platform === 'web') return null;
-    return `dsh.client.platform is ${JSON.stringify(platform)}, but the harness serves client bundles on web`;
+    if (platform !== undefined && platform !== 'web') {
+      return `dsh.client.platform is ${JSON.stringify(platform)}, but the harness serves client bundles on web`;
+    }
+    // The host reads the bundle path from exports["./client"] only — a
+    // declaration without that subpath makes the host drop the row.
+    const exported = manifest.exports;
+    const subpath = typeof exported === 'string' ? undefined : exportTarget(exported?.['./client']);
+    if (typeof subpath !== 'string') {
+      return 'dsh.client is declared but exports has no "./client" bundle, so the host drops the row and the panel cannot appear';
+    }
+    const file = subpath.replace(/^\.\//u, '');
+    return fs.existsSync(path.join(dir, file))
+      ? null
+      : `exports["./client"] points at ${subpath}, which does not exist`;
   }
   const exported = manifest.exports;
   const subpath = typeof exported === 'string' ? undefined : exportTarget(exported?.['./client']);

package/lib/init.mjs

@@ -33,6 +33,7 @@ function manifest({ name, title, version }) {
       type: 'module',
       engines: { node: '>=18', dsh: '>=0.2.0' },
       main: 'index.js',
+      exports: { '.': './index.js', './client': './client.js' },
       files: ['index.js', 'client.js', 'cordis.patch.yml', 'README.md'],
       scripts: { test: 'node --test', doctor: 'dsh-plugin doctor .' },
       devDependencies: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin-toolkit",
-  "version": "0.9.2",
+  "version": "0.9.3",
   "description": "Preflight, release and test DeepSeek Harness plugins: checks what the host needs, waits for npm to actually serve the version, and keeps the market index honest.",
   "type": "module",
   "bin": {
@@ -8,6 +8,7 @@
   },
   "exports": {
     ".": "./index.js",
+    "./client": "./client.js",
     "./cli": "./lib/cli.mjs",
     "./testkit": "./lib/testkit/index.mjs"
   },