@stigmer/react 3.5.2 → 3.5.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stigmer/react",
-  "version": "3.5.2",
+  "version": "3.5.3",
   "description": "React provider and client hook for the Stigmer platform SDK",
   "license": "Apache-2.0",
   "type": "module",
@@ -37,7 +37,7 @@
   "dependencies": {
     "@bufbuild/protovalidate": "^1.1.1",
     "@dagrejs/dagre": "^1.1.4",
-    "@stigmer/theme": "3.5.2",
+    "@stigmer/theme": "3.5.3",
     "diff": "^8.0.3",
     "fflate": "^0.8.2",
     "hast-util-to-jsx-runtime": "^2.3.6",
@@ -60,8 +60,8 @@
     "@codemirror/state": "^6.0.0",
     "@codemirror/view": "^6.0.0",
     "@lezer/highlight": "^1.0.0",
-    "@stigmer/protos": "3.5.2",
-    "@stigmer/sdk": "3.5.2",
+    "@stigmer/protos": "3.5.3",
+    "@stigmer/sdk": "3.5.3",
     "@tanstack/react-table": "^8.20.0",
     "@xyflow/react": "^12.0.0",
     "elkjs": "^0.9.0",

package/src/execution/__tests__/PlanArtifactCard.test.tsx

@@ -1,5 +1,11 @@
-import { describe, it, expect, vi, afterEach } from "vitest";
-import { render, screen, fireEvent, cleanup } from "@testing-library/react";
+import { describe, it, expect, vi, afterEach, beforeEach } from "vitest";
+import {
+  render,
+  screen,
+  fireEvent,
+  cleanup,
+  waitFor,
+} from "@testing-library/react";
 import type { ReactNode } from "react";
 import { create } from "@bufbuild/protobuf";
 import type { Stigmer } from "@stigmer/sdk";
@@ -47,6 +53,17 @@ function renderCard(ui: ReactNode) {
 
 afterEach(cleanup);
 
+// The download flow ends in a transient-anchor click with target="_blank".
+// Left unstubbed, happy-dom treats that click as a real popup navigation and
+// fetches the fixture URL over the actual network — after this test has
+// already returned, so the DNS failure gets attributed to whichever test the
+// worker is running by then (the flake in issue #334).
+beforeEach(() => {
+  vi.spyOn(HTMLAnchorElement.prototype, "click")
+    .mockImplementation(() => {})
+    .mockClear();
+});
+
 describe("PlanArtifactCard — compact document card", () => {
   it("is an accessible region with the plan's title, filename, and size", () => {
     renderCard(
@@ -178,6 +195,7 @@ describe("PlanArtifactCard — compact document card", () => {
 
   it("downloads the artifact on demand via a freshly minted URL", async () => {
     const { stigmer, getArtifactDownloadUrl } = createStigmerMock();
+    const anchorClick = vi.spyOn(HTMLAnchorElement.prototype, "click");
     render(
       withStigmer(
         <PlanArtifactCard executionId="aex_1" artifact={planArtifact} org="acme" />,
@@ -191,6 +209,9 @@ describe("PlanArtifactCard — compact document card", () => {
     const req = getArtifactDownloadUrl.mock.calls[0][0];
     expect(req.executionId).toBe("aex_1");
     expect(req.storageKey).toBe("artifacts/aex_1/plan.md");
+    // Wait for the mint-URL → anchor-click continuation INSIDE the test body;
+    // returning while it is still in flight is what let it escape.
+    await waitFor(() => expect(anchorClick).toHaveBeenCalledTimes(1));
   });
 });