@pi-harness/pi-harness 0.1.26 → 0.1.28

package/packages/api-gateway/src/marketplace-entries/official/workspace-navigator.json

@@ -1,7 +1,7 @@
 {
   "id": "workspace-navigator",
   "packageName": "@pi-harness/core/plugins/workspace-navigator",
-  "version": "0.1.26",
+  "version": "0.1.28",
   "name": "Workspace Navigator",
   "description": "Show a bounded workspace tree and read-only Git status for a high-signal coding sidebar.",
   "author": "Pi Harness",

package/packages/api-gateway/src/marketplace-entries/official/workspace-search.json

@@ -1,7 +1,7 @@
 {
   "id": "workspace-search",
   "packageName": "@pi-harness/core/plugins/workspace-search",
-  "version": "0.1.26",
+  "version": "0.1.28",
   "name": "Workspace Search",
   "description": "Search bounded text files in the active workspace with ignored directories and binary content excluded.",
   "author": "Pi Harness",

package/packages/api-gateway/src/marketplace-entries/official/yaml-validator.json

@@ -1,7 +1,7 @@
 {
   "id": "yaml-validator",
   "packageName": "@pi-harness/core/plugins/yaml-validator",
-  "version": "0.1.26",
+  "version": "0.1.28",
   "name": "YAML Validator",
   "description": "Validate workspace-contained UTF-8 multi-document YAML through bounded, cancellable reads and line-aware diagnostics.",
   "author": "Pi Harness",

package/packages/api-gateway/test/release-contract.test.ts

@@ -39,7 +39,7 @@ describe("release contract", () => {
     expect(workflow).not.toContain("RELEASE_TAG_EXISTS");
     expect(workflow).not.toContain("was already submitted");
     expect(workflow).not.toContain("tag_exists=$tag_exists");
-    expect(workflow).toContain("npm publish --workspace @pi-harness/core --access public");
+    expect(workflow).toContain('npm publish --workspace "$package_name" --access public');
     expect(workflow).toContain("npm publish --access public");
   });
 
@@ -79,4 +79,22 @@ describe("release contract", () => {
         `${name} (used by ${file}) is declared by neither ${core.name} dependencies nor peerDependencies`,
       ).toBeDefined();
   });
+
+  test("the plugin API declares every package its sources import as a peer", () => {
+    // A plugin author installs this package on its own, so anything it imports has to be something the author is told to install alongside it.
+    const pluginApi = readJson("packages/plugin-api/package.json") as {
+      name: string;
+      dependencies?: Record<string, string>;
+      peerDependencies: Record<string, string>;
+    };
+    const required = new Map<string, string>();
+    for (const file of sourceFiles(resolve(repositoryRoot, "packages/plugin-api/src"), ".ts")) {
+      for (const match of readFileSync(file, "utf8").matchAll(/(?:from|declare module)\s*["']([^"'\s]+)["']/gu))
+        required.set(packageNameOf(match[1] ?? ""), relative(repositoryRoot, file));
+    }
+    const external = [...required].filter(([name]) => !name.startsWith("node:") && !name.startsWith("."));
+    expect(external.length).toBeGreaterThan(0);
+    for (const [name, file] of external) expect(pluginApi.peerDependencies[name], `${name} (used by ${file}) is not a peer of ${pluginApi.name}`).toBeDefined();
+    expect(pluginApi.dependencies, `${pluginApi.name} must not pull a second copy of anything a plugin already installs`).toBeUndefined();
+  });
 });

package/packages/api-gateway/test/release-package.test.ts

@@ -32,12 +32,45 @@ describe("release package", () => {
     expect(clientManifest.private).toBe(true);
 
     const workflow = await readFile(resolve(repositoryRoot, ".github/workflows/release.yml"), "utf8");
-    expect(workflow).toContain("npm publish --workspace @pi-harness/core --access public");
+    expect(workflow).toContain('npm publish --workspace "$package_name" --access public');
     expect(workflow).toContain("npm publish --access public");
     expect(workflow).toContain("Verify package availability");
     expect(workflow).not.toContain("RELEASE_TAG_EXISTS");
   });
 
+  it("publishes the plugin API as its own package before the packages that depend on it", async () => {
+    const rootManifest = await readJson("package.json");
+    const rootDependencies = rootManifest.dependencies as Record<string, string>;
+    const manifest = await readJson("packages/plugin-api/package.json");
+
+    expect(manifest.name).toBe("@pi-harness/plugin-api");
+    expect(manifest.private, "the plugin API is the package a plugin author installs directly").toBeUndefined();
+    expect(manifest.version).toBe(rootManifest.version);
+    expect(manifest.files).toEqual(["dist", "README.md"]);
+    expect((manifest.publishConfig as Record<string, string>).access).toBe("public");
+
+    // The runtimes are peers here for the same reason they are peers in core: a second copy detaches the Context augmentation.
+    expect(manifest.dependencies, "plugin API runtimes must stay peers so a plugin shares the harness copy").toBeUndefined();
+    const peers = manifest.peerDependencies as Record<string, string>;
+    expect(Object.keys(peers).sort()).toEqual([...corePeerRuntimeNames]);
+    for (const name of corePeerRuntimeNames) expect(peers[name], `${name} peer range must pin the version the launcher installs`).toBe(rootDependencies[name]);
+
+    const core = await readJson("packages/core/package.json");
+    expect((core.dependencies as Record<string, string>)["@pi-harness/plugin-api"]).toBe(rootManifest.version);
+
+    const workflow = await readFile(resolve(repositoryRoot, ".github/workflows/release.yml"), "utf8");
+    expect(workflow).toContain("for workspace in packages/plugin-api packages/core .; do");
+    expect(workflow).toContain("for package_name in @pi-harness/plugin-api @pi-harness/core @pi-harness/pi-harness; do");
+  });
+
+  it("creates the GitHub Release before waiting on npm's CDN", async () => {
+    const workflow = await readFile(resolve(repositoryRoot, ".github/workflows/release.yml"), "utf8");
+
+    // Availability is CDN propagation latency, not a publish failure. Ordering the wait first made a slow tarball skip the release step outright, so the tag shipped without one.
+    expect(workflow.indexOf("- name: Create GitHub Release")).toBeLessThan(workflow.indexOf("- name: Verify package availability"));
+    expect(workflow).not.toContain("seq 1 60");
+  });
+
   it("keeps the runtimes in the core type surface as peers so a plugin author resolves one copy", async () => {
     const rootManifest = await readJson("package.json");
     const rootDependencies = rootManifest.dependencies as Record<string, string>;

package/packages/bundle-web-app/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/web-app",
-  "version": "0.1.26",
+  "version": "0.1.28",
   "private": true,
   "description": "Patchable Web application bundle for Pi Harness",
   "homepage": "https://github.com/pi-harness/pi-harness#readme",

package/packages/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/cli",
-  "version": "0.1.26",
+  "version": "0.1.28",
   "private": true,
   "description": "Plugin-first Pi Harness CLI",
   "homepage": "https://github.com/pi-harness/pi-harness#readme",

package/packages/host-webserver/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/host-webserver",
-  "version": "0.1.26",
+  "version": "0.1.28",
   "private": true,
   "description": "Cordis route-registration service for the Pi Harness web host",
   "homepage": "https://github.com/pi-harness/pi-harness#readme",