@intentius/behold 0.27.0 → 0.28.0

package/dist/cli.js

@@ -9133,6 +9133,11 @@ var init_workspace_kinds_schema = __esm({
             },
             fields: {
               $ref: "#/$defs/fields"
+            },
+            reader: {
+              type: "string",
+              pattern: "^[a-z][a-z0-9-]{0,39}$",
+              description: "A link reader chant ships, by name, that derives a member's links from the member's own files (#3932): dbt-project reads a dbt project's sources. The reader is chant's code and runs offline, parsing files as data; the package supplies only the name. A name this chant doesn't have keeps the kind without a reader, and the load reports it. A chant older than this field fails a kinds file that uses it."
             }
           },
           patternProperties: {
@@ -15776,7 +15781,9 @@ function estateFace(irIn, boxesIn, opts = {}) {
   const face = cardFaces(irIn, boxesIn, opts.prefixes ? { prefixes: opts.prefixes } : {});
   const ir = face.ir;
   if (Object.keys(face.overrides).length) faceOverrides.set(ir, face.overrides);
-  const boxes = boxesIn ? Object.fromEntries(Object.entries(boxesIn).map(([key, ids]) => [key, ids.map((id) => face.display.get(id) ?? id)])) : void 0;
+  const toDisplay = (record) => record ? Object.fromEntries(Object.entries(record).map(([key, ids]) => [key, ids.map((id) => face.display.get(id) ?? id)])) : void 0;
+  const boxes = toDisplay(boxesIn);
+  const subgroups = boxes ? toDisplay(opts.subgroups) : void 0;
   const realBand = moduleBandOf(irIn);
   const toReal = new Map(ir.nodes.map((n, i) => [n.id, irIn.nodes[i].id]));
   const bandOf = realBand ? (id) => realBand(toReal.get(id) ?? id) : void 0;
@@ -15786,13 +15793,14 @@ function estateFace(irIn, boxesIn, opts = {}) {
     ...opts.nodesep !== void 0 ? { nodesep: opts.nodesep } : {},
     ...opts.ranksep !== void 0 ? { ranksep: opts.ranksep } : {}
   }) : { ir, grids: /* @__PURE__ */ new Map() };
-  return { ir, overrides: face.overrides, boxes, display: face.display, bandOf, chains, restore: face.restore };
+  return { ir, overrides: face.overrides, boxes, subgroups, display: face.display, bandOf, chains, restore: face.restore };
 }
 function packEstateBoxes(layout, face, opts = {}) {
   const boxes = face.boxes;
   if (!boxes) return;
-  const boxGrids = packBoxComponents(layout, face.ir, boxes, face.bandOf);
-  if (opts.members) packMemberBoxes(layout, face.ir, boxes);
+  const boxGrids = packBoxComponents(layout, face.ir, boxes, face.bandOf, face.subgroups);
+  if (opts.members) packMemberBoxes(layout, face.ir, { ...topLevelSubgroups(layout.groups, face.subgroups), ...boxes });
+  if (face.subgroups) fitNestedBoxes(layout, face.ir, boxes, face.subgroups);
   addBandBoxes(layout, new Map([...boxGrids, ...face.chains.grids]), footprints(face.ir, face.overrides));
   applyBadges(layout.groups, opts.badges, opts.marks);
 }
@@ -15826,17 +15834,27 @@ function renderGraph(irIn, opts = {}) {
   const groupsIn = irIn.groups;
   const pipelines = groupsIn.byPipeline && Object.keys(groupsIn.byPipeline).length ? groupsIn.byPipeline : void 0;
   const boxKey = groupsIn.byWave ? "byWave" : opts.boxes ?? (pipelines ? "byPipeline" : void 0);
-  const face = estateFace(irIn, boxKey ? groupsIn[boxKey] : void 0);
+  const nested = boxKey === "byStack" ? pipelines : void 0;
+  const face = estateFace(irIn, boxKey ? groupsIn[boxKey] : void 0, nested ? { subgroups: nested } : {});
   const ir = face.ir;
   const boxes = face.boxes;
   const chains = face.chains;
-  const layout = layoutIr(chains.ir, { fit: true, overrides: face.overrides, ...boxes ? { groups: boxes } : {} });
+  const layout = layoutIr(chains.ir, {
+    fit: true,
+    overrides: face.overrides,
+    ...boxes ? { groups: boxes } : {},
+    ...face.subgroups ? { subgroups: face.subgroups } : {}
+  });
   if (opts.radial && !boxes) radializeLayout(layout, groupKeyByNode(ir), footprints(ir, face.overrides));
   else if (!boxes) packComponents(layout, ir);
   packEstateBoxes(layout, face, { members: boxKey === "byStack", ...opts.groupBadges ? { badges: opts.groupBadges } : {} });
-  if (boxKey === "byPipeline" && pipelines) {
+  if ((boxKey === "byPipeline" || nested) && pipelines) {
     const titles = pipelineTitles(irIn, pipelines);
-    for (const box of layout.groups ?? []) if (box.id !== void 0 && titles[box.id] !== void 0) box.title = titles[box.id];
+    for (const box of layout.groups ?? []) {
+      if (box.id === void 0 || titles[box.id] === void 0) continue;
+      const title = titles[box.id];
+      box.title = title === box.id && box.parent !== void 0 && title.startsWith(`${box.parent}/`) ? title.slice(box.parent.length + 1) : title;
+    }
   }
   const svg = renderSvg(ir, layout, {
     fit: true,
@@ -15849,9 +15867,16 @@ function renderGraph(irIn, opts = {}) {
 }
 function pipelineTitles(ir, byPipeline) {
   const nodes = new Map(ir.nodes.map((n) => [n.id, n]));
+  const workflowOf = (ids) => {
+    const workflows = ids.flatMap((id) => {
+      const n = nodes.get(id);
+      return n && /::Workflow$/.test(n.kind) ? [n] : [];
+    });
+    return workflows.length === 1 ? workflows[0] : void 0;
+  };
   return Object.fromEntries(
-    Object.keys(byPipeline).map((key) => {
-      const name = nodes.get(key)?.attrs?.["name"];
+    Object.entries(byPipeline).map(([key, ids]) => {
+      const name = (nodes.get(key) ?? workflowOf(ids))?.attrs?.["name"];
       return [key, typeof name === "string" && name.trim() ? name : key];
     })
   );
@@ -15901,6 +15926,39 @@ function addBandBoxes(layout, grids, sizes) {
   boxes.push(...added);
 }
 var BAND_PAD = 14;
+var NEST_PAD_X = 50;
+var NEST_PAD_Y = 36;
+var NEST_OUTER = 36;
+function topLevelSubgroups(boxes, subgroups) {
+  if (!subgroups) return {};
+  return Object.fromEntries(
+    (boxes ?? []).flatMap((b) => b.parent === void 0 && b.id !== void 0 && subgroups[b.id] ? [[b.id, subgroups[b.id]]] : [])
+  );
+}
+function fitNestedBoxes(layout, ir, boxes, subgroups, only2) {
+  const at2 = new Map(layout.nodes.map((n) => [n.id, n]));
+  const ownerOf2 = /* @__PURE__ */ new Map();
+  for (const [key, ids] of Object.entries(boxes)) for (const id of ids) if (at2.has(id)) ownerOf2.set(id, key);
+  const sizes = footprints(ir);
+  for (const box of layout.groups ?? []) {
+    if (box.parent === void 0 || box.id === void 0 || only2 !== void 0 && box.parent !== only2) continue;
+    const cards = (subgroups[box.id] ?? []).flatMap((id) => {
+      const p = at2.get(id);
+      if (!p || ownerOf2.get(id) !== box.parent) return [];
+      const s = sizes.get(id) ?? { w: NODE_W, h: NODE_H };
+      return [{ x: p.x, y: p.y, w: s.w, h: s.h }];
+    });
+    if (!cards.length) continue;
+    const x0 = Math.min(...cards.map((c) => c.x - c.w / 2)) - NEST_PAD_X;
+    const x1 = Math.max(...cards.map((c) => c.x + c.w / 2)) + NEST_PAD_X;
+    const y0 = Math.min(...cards.map((c) => c.y - c.h / 2)) - NEST_PAD_Y;
+    const y1 = Math.max(...cards.map((c) => c.y + c.h / 2)) + NEST_PAD_Y;
+    box.x = (x0 + x1) / 2;
+    box.y = (y0 + y1) / 2;
+    box.w = x1 - x0;
+    box.h = y1 - y0;
+  }
+}
 function clampInside(r, box) {
   const inset = 6;
   const left = box.x - box.w / 2 + inset;
@@ -16216,7 +16274,8 @@ function packComponents(layout, ir) {
   layout.height = Math.max(...nodes.map((n) => n.y + halfH(n))) + pad;
 }
 function packMemberBoxes(layout, ir, members) {
-  const boxes = layout.groups ?? [];
+  const allBoxes = layout.groups ?? [];
+  const boxes = allBoxes.filter((b) => b.parent === void 0);
   if (boxes.length < 4) return;
   const nodes = layout.nodes;
   const size = footprints(ir);
@@ -16332,7 +16391,7 @@ function gridComponent(one, sizes) {
   const rows = Math.ceil(order.length / cols);
   return { local, w: Math.min(cols, order.length) * (cell.w + sep12.x) - sep12.x, h: rows * (cell.h + sep12.y) - sep12.y };
 }
-function packBoxComponents(layout, ir, boxes, bandOf) {
+function packBoxComponents(layout, ir, boxes, bandOf, subgroups) {
   const grids = /* @__PURE__ */ new Map();
   const groupBoxes = layout.groups ?? [];
   const nodes = layout.nodes;
@@ -16363,6 +16422,10 @@ function packBoxComponents(layout, ir, boxes, bandOf) {
       const b = idxOf.get(e.to);
       if (a !== void 0 && b !== void 0) parent[find(a)] = find(b);
     }
+    for (const ids2 of Object.values(subgroups ?? {})) {
+      const inside = ids2.flatMap((id) => idxOf.get(id) ?? []);
+      for (const i of inside.slice(1)) parent[find(i)] = find(inside[0]);
+    }
     const comps = /* @__PURE__ */ new Map();
     own.forEach((n, i) => (comps.get(find(i)) ?? comps.set(find(i), []).get(find(i))).push(n));
     if (comps.size < 2) continue;
@@ -16375,12 +16438,19 @@ function packBoxComponents(layout, ir, boxes, bandOf) {
         edges: ir.edges.filter((e) => inside.has(e.from) && inside.has(e.to)),
         groups: {}
       };
-      let one = layoutIr(sub, { fit: true });
+      const nest = Object.fromEntries(
+        Object.entries(subgroups ?? {}).flatMap(([k, ids2]) => {
+          const mine = ids2.filter((id) => inside.has(id));
+          return mine.length ? [[k, mine]] : [];
+        })
+      );
+      const nests = Object.keys(nest).length > 0;
+      let one = layoutIr(sub, { fit: true, ...nests ? { groups: nest } : {} });
       if (one.width > one.height * BOX_ASPECT_MAX) {
-        const sideways = layoutIr(sub, { fit: true, rankdir: "LR" });
+        const sideways = layoutIr(sub, { fit: true, rankdir: "LR", ...nests ? { groups: nest } : {} });
         if (offAspect(sideways) < offAspect(one)) one = sideways;
       }
-      const placed = offAspect(one) > GRID_COMPONENT ? gridComponent(one, size) : { local: new Map(one.nodes.map((n) => [n.id, { x: n.x, y: one.height - n.y }])), w: one.width, h: one.height };
+      const placed = !nests && offAspect(one) > GRID_COMPONENT ? gridComponent(one, size) : { local: new Map(one.nodes.map((n) => [n.id, { x: n.x, y: one.height - n.y }])), w: one.width, h: one.height };
       laid.set(root, { ...placed, ns });
     }
     const order = [void 0];
@@ -16396,8 +16466,9 @@ function packBoxComponents(layout, ir, boxes, bandOf) {
     const all = [...laid.values()];
     const area = all.reduce((s, c) => s + (c.w + COMPONENT_GAP) * (c.h + COMPONENT_GAP), 0);
     const targetW = Math.max(Math.max(...all.map((c) => c.w)), Math.sqrt(area * ROW_ASPECT));
-    const originX = box.x - box.w / 2 + BAND_PAD;
-    const originY = box.y - box.h / 2 + TITLE;
+    const holdsNested = groupBoxes.some((b) => b.parent === key);
+    const originX = box.x - box.w / 2 + (holdsNested ? NEST_OUTER : BAND_PAD);
+    const originY = box.y - box.h / 2 + (holdsNested ? NEST_OUTER : TITLE);
     let shelfX = 0;
     let shelfY = 0;
     let shelfH = 0;
@@ -16429,10 +16500,18 @@ function packBoxComponents(layout, ir, boxes, bandOf) {
       }
       bands.push({ key: band, ids: ids2 });
     }
-    const x0 = Math.min(...own.map((n) => n.x - halfW(n))) - BAND_PAD;
-    const x1 = Math.max(...own.map((n) => n.x + halfW(n))) + BAND_PAD;
-    const y0 = Math.min(...own.map((n) => n.y - halfH(n))) - TITLE;
-    const y1 = Math.max(...own.map((n) => n.y + halfH(n))) + BAND_PAD;
+    if (subgroups) fitNestedBoxes(layout, ir, boxes, subgroups, key);
+    const inner = groupBoxes.filter((b) => b.parent === key);
+    let x0 = Math.min(...own.map((n) => n.x - halfW(n))) - BAND_PAD;
+    let x1 = Math.max(...own.map((n) => n.x + halfW(n))) + BAND_PAD;
+    let y0 = Math.min(...own.map((n) => n.y - halfH(n))) - TITLE;
+    let y1 = Math.max(...own.map((n) => n.y + halfH(n))) + BAND_PAD;
+    for (const b of inner) {
+      x0 = Math.min(x0, b.x - b.w / 2 - NEST_OUTER);
+      x1 = Math.max(x1, b.x + b.w / 2 + NEST_OUTER);
+      y0 = Math.min(y0, b.y - b.h / 2 - NEST_OUTER);
+      y1 = Math.max(y1, b.y + b.h / 2 + NEST_OUTER);
+    }
     box.x = (x0 + x1) / 2;
     box.y = (y0 + y1) / 2;
     box.w = x1 - x0;
@@ -19486,6 +19565,17 @@ async function ledgerIn(root, dir) {
   return { members, prefix: memberLedgerPrefix(members) };
 }
 
+// node_modules/@intentius/chant/src/lifecycle/no-record.ts
+var NO_RECORD_ENV = "CHANT_NO_RECORD";
+function ledgerRecordingOff(env = process.env) {
+  const value = env[NO_RECORD_ENV]?.trim().toLowerCase();
+  return value !== void 0 && value !== "" && value !== "0" && value !== "false";
+}
+var overlays = /* @__PURE__ */ new Map();
+function overlayTip(key) {
+  return overlays.get(key)?.tip;
+}
+
 // node_modules/@intentius/chant/src/lifecycle/git.ts
 var STATE_BRANCH = "chant/lifecycle";
 async function ledgerDir(dir, opts) {
@@ -19502,7 +19592,7 @@ async function readBlobFromPath(environment, filename, opts) {
   const rt = getRuntime();
   const dir = await ledgerDir(environment, opts);
   const result = await rt.spawn(
-    ["git", "show", `${STATE_BRANCH}:${dir}/${filename}`],
+    ["git", "show", `${await ledgerRevision(opts?.cwd)}:${dir}/${filename}`],
     { cwd: opts?.cwd }
   );
   if (result.exitCode !== 0) return null;
@@ -19527,7 +19617,7 @@ async function listLedgerTrees(opts) {
   const rt = getRuntime();
   const base = await ledgerDir("", opts);
   const rootResult = await rt.spawn(
-    ["git", "ls-tree", "--full-tree", base === "" ? STATE_BRANCH : `${STATE_BRANCH}:${base}`],
+    ["git", "ls-tree", "--full-tree", base === "" ? tip : `${tip}:${base}`],
     { cwd: opts?.cwd }
   );
   if (rootResult.exitCode !== 0) return [];
@@ -19538,7 +19628,12 @@ async function listLedgerTrees(opts) {
   }
   return trees;
 }
+var C_LOCALE_ENV = { LC_ALL: "C", LANGUAGE: "" };
 async function getStateBranchTip(cwd) {
+  if (ledgerRecordingOff()) {
+    const tip = overlayTip(await overlayKey(cwd));
+    if (tip !== void 0) return tip;
+  }
   const rt = getRuntime();
   const result = await rt.spawn(
     ["git", "rev-parse", "--verify", `refs/heads/${STATE_BRANCH}`],
@@ -19547,6 +19642,20 @@ async function getStateBranchTip(cwd) {
   if (result.exitCode !== 0) return null;
   return result.stdout.trim();
 }
+async function ledgerRevision(cwd) {
+  if (!ledgerRecordingOff()) return STATE_BRANCH;
+  return overlayTip(await overlayKey(cwd)) ?? STATE_BRANCH;
+}
+var overlayKeys = /* @__PURE__ */ new Map();
+async function overlayKey(cwd) {
+  const from = cwd ?? process.cwd();
+  const cached = overlayKeys.get(from);
+  if (cached !== void 0) return cached;
+  const result = await getRuntime().spawn(["git", "rev-parse", "--path-format=absolute", "--git-common-dir"], { cwd: from, env: C_LOCALE_ENV });
+  const key = result.exitCode === 0 && result.stdout.trim() !== "" ? result.stdout.trim() : from;
+  overlayKeys.set(from, key);
+  return key;
+}
 
 // node_modules/@intentius/chant/src/content-digest.ts
 import { createHash as createHash6 } from "node:crypto";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@intentius/behold",
-  "version": "0.27.0",
+  "version": "0.28.0",
   "license": "Apache-2.0",
   "type": "module",
   "description": "behold — a live control plane on chant. See your whole estate (every substrate in one graph), coloured by drift; act through delegated, gated Ops.",
@@ -41,9 +41,9 @@
   },
   "dependencies": {
     "@hono/node-server": "^2.0.5",
-    "@intentius/chant": "^0.124.20",
-    "@intentius/chant-k8s-client": "^0.124.20",
-    "@intentius/pinhole": "^0.4.0",
+    "@intentius/chant": "^0.124.24",
+    "@intentius/chant-k8s-client": "^0.124.24",
+    "@intentius/pinhole": "^0.5.0",
     "ajv": "^8.20.0",
     "hono": "^4.12.34"
   },

package/web/app.js

@@ -21,7 +21,7 @@ import { initTerragucciEstate } from "./terragucci-estate.js";
 import { auditLines as yrAuditLines, cornerText as yrCorner, datedWords as yrDated, environmentLines as yrEnvLines, joinMarks as yrJoin, legend as yrLegend, markLine as yrMarkLine } from "./yodel-report.js";
 import { cardProgress as tgCardProgress, progressLines as tgProgressLines, progressSection as tgProgressSection, stampProgress as tgStampProgress } from "./terragucci-progress.js";
 import { seconds, whyQuery, whyView } from "./why.js";
-import { edgeLabel, edgeLegend, edgeTitle, isCiKind } from "./edge-kind.js";
+import { edgeKey, edgeLabel, edgeLegend, edgeTitle, isCiKind, viaIndex } from "./edge-kind.js";
 import { readCostLine } from "./read-cost.js";
 import { mountHistoryPicker } from "./history-picker.js";
 import { applyMemberFrame, pendingLine, stillPending } from "./pending.js";
@@ -4457,21 +4457,16 @@ function relabelEdgeChip(chip, kind, via) {
 function wireEdgeHighlight(svgEl, ir) {
   const edges = [...svgEl.querySelectorAll("g[data-edge-from]")];
   if (!edges.length) return;
-  // Reason per edge, keyed by from|to (and its reverse, since pinhole may paint
-  // an edge in either direction relative to the IR). pinhole stamps
-  // data-edge-via itself; this fills it in where it did not.
-  const viaOf = new Map();
-  for (const e of ir.edges || []) {
-    if (!e.viaAttr) continue;
-    viaOf.set(`${e.from}|${e.to}`, e.viaAttr);
-    viaOf.set(`${e.to}|${e.from}`, e.viaAttr);
-  }
+  // Reason per edge, keyed by from, to and kind (pinhole's edgeKey, #564), so
+  // two kinds on one pair keep their own reason. pinhole stamps data-edge-via
+  // itself; this fills it in where it did not.
+  const viaOf = viaIndex(ir.edges || []);
   for (const g of edges) {
     const from = g.getAttribute("data-edge-from");
     const to = g.getAttribute("data-edge-to");
-    const via = g.getAttribute("data-edge-via") || viaOf.get(`${from}|${to}`);
-    if (via) g.setAttribute("data-edge-via", via); // drives the dashed style
     const kind = g.getAttribute("data-edge-kind") || "ref";
+    const via = g.getAttribute("data-edge-via") || viaOf.get(edgeKey({ from, to, kind }));
+    if (via) g.setAttribute("data-edge-via", via); // drives the dashed style
     const cross = g.getAttribute("data-edge-cross") === "true";
     if (!g.querySelector("path")) relabelEdgeChip(g, kind, via);
     const title = document.createElementNS(SVGNS, "title");
@@ -5280,17 +5275,24 @@ function userRect(el, toUser) {
  */
 function edgeLabelOf(g) {
   if (!(g.getAttribute("data-edge-via") || "").trim()) return null;
-  const from = g.getAttribute("data-edge-from");
-  const to = g.getAttribute("data-edge-to");
+  // The pair and the kind (pinhole#128, #564): two kinds on one pair draw two
+  // lines with a chip each, and each line keeps its own chip.
+  const key = edgeKeyOf(g);
   for (const cand of g.parentNode ? g.parentNode.children : []) {
     if (cand === g || cand.tagName.toLowerCase() !== "g") continue;
-    if (cand.getAttribute("data-edge-from") !== from || cand.getAttribute("data-edge-to") !== to) continue;
+    if (!cand.hasAttribute("data-edge-from") || edgeKeyOf(cand) !== key) continue;
     if (cand.hasAttribute("data-node-id") || cand.querySelector("path")) continue;
     return cand;
   }
   return null;
 }
 
+/** A drawn edge group's identity, pinhole's `edgeKey` over its stamped
+ * attributes (no `data-edge-kind` is a reference). */
+function edgeKeyOf(g) {
+  return edgeKey({ from: g.getAttribute("data-edge-from"), to: g.getAttribute("data-edge-to"), kind: g.getAttribute("data-edge-kind") || "ref" });
+}
+
 /** Index the freshly rendered SVG: node groups, containment boxes, edge paths. */
 function indexLayout(svgEl) {
   const toUser = userSpaceMapper(svgEl);

package/web/edge-kind.js

@@ -32,6 +32,37 @@ export function isCiKind(kind) {
   return !!kind && kind !== "ref";
 }
 
+/**
+ * An edge's identity, the same key pinhole 0.5.0's `edgeKey` gives
+ * (pinhole#128): `from>to` for a reference, `from>to>kind` otherwise. Two
+ * kinds on one pair (a GitLab job that both `needs` and `extends` another)
+ * are two edges, so anything the SPA looks up per edge goes through this
+ * (#564). Read off a drawn edge, pass `data-edge-kind`, which pinhole leaves
+ * off for "ref".
+ */
+export function edgeKey({ from, to, kind } = {}) {
+  return `${from}>${to}${isCiKind(kind) ? `>${kind}` : ""}`;
+}
+
+/**
+ * The `viaAttr` per edge key, for the drawn edges pinhole did not stamp
+ * `data-edge-via` on. The reverse direction is filled in as well, where
+ * nothing already holds it, since pinhole may paint an edge either way round
+ * relative to the IR.
+ */
+export function viaIndex(edges = []) {
+  const via = new Map();
+  for (const e of edges) {
+    if (!e.viaAttr) continue;
+    via.set(edgeKey(e), e.viaAttr);
+  }
+  for (const e of edges) {
+    const back = edgeKey({ from: e.to, to: e.from, kind: e.kind });
+    if (e.viaAttr && !via.has(back)) via.set(back, e.viaAttr);
+  }
+  return via;
+}
+
 /**
  * The chip text, as chant's Mermaid and DOT writers label the edge
  * (graph-mermaid.ts `edgeLabel`): a CI edge by its kind, plus the attribute

package/web/edge-kind.test.js

@@ -1,5 +1,5 @@
 import { describe, expect, it } from "vitest";
-import { edgeLabel, edgeLegend, edgeTitle } from "./edge-kind.js";
+import { edgeKey, edgeLabel, edgeLegend, edgeTitle, viaIndex } from "./edge-kind.js";
 
 // The edges chant 0.124.20's GitHub lexicon writes (lexicons/github/src/
 // graph-edges.ts) and one from its GitLab lexicon.
@@ -68,3 +68,33 @@ describe("edgeLegend (#553)", () => {
     expect(edgeLegend([ref, { from: "a", to: "b", kind: "ref" }])).toEqual([]);
   });
 });
+
+// #564: pinhole 0.5.0 keys an edge on from, to and kind (pinhole#128), so two
+// kinds on one pair are two edges, each with its own chip and tooltip.
+describe("edgeKey and viaIndex (#564)", () => {
+  it("gives the key pinhole's edgeKey gives: a reference keeps from>to", () => {
+    expect(edgeKey({ from: "api", to: "db", kind: "ref" })).toBe("api>db");
+    expect(edgeKey({ from: "api", to: "db" })).toBe("api>db");
+    expect(edgeKey({ from: "test", to: "build", kind: "needs" })).toBe("test>build>needs");
+  });
+
+  it("keeps each kind's reason on a pair joined twice", () => {
+    const via = viaIndex([
+      { from: "test", to: "build", kind: "needs", viaAttr: "stage" },
+      { from: "test", to: "build", kind: "extends", viaAttr: "extends" },
+    ]);
+    expect(via.get(edgeKey({ from: "test", to: "build", kind: "needs" }))).toBe("stage");
+    expect(via.get(edgeKey({ from: "test", to: "build", kind: "extends" }))).toBe("extends");
+  });
+
+  it("fills in the reverse direction without overwriting an edge drawn that way", () => {
+    const via = viaIndex([
+      { from: "a", to: "b", kind: "ref", viaAttr: "role" },
+      { from: "b", to: "a", kind: "ref", viaAttr: "policy_arn" },
+      { from: "c", to: "d", kind: "ref", viaAttr: "zone_id" },
+    ]);
+    expect(via.get("a>b")).toBe("role");
+    expect(via.get("b>a")).toBe("policy_arn");
+    expect(via.get("d>c")).toBe("zone_id");
+  });
+});