@knowvah/dot-engine 2.6.8 → 2.6.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@knowvah/dot-engine",
-  "version": "2.6.8",
+  "version": "2.6.9",
   "description": "Faithful TypeScript port of Graphviz (dot, neato, fdp, sfdp, circo, twopi, osage, patchwork) — no C: no native binary, no WASM. Oracle-verified against the C implementation, runs in the browser.",
   "keywords": [
     "graphviz",

package/src/common/poly-gencode.ts

@@ -22,7 +22,7 @@ import { FillType } from '../gvc/context.js';
 import { emitHtmlLabel } from './htmltable-emit.js';
 import { transformPoint } from '../gvc/device.js';
 import { nodeAttr } from './poly-init.js';
-import { substObjAnchor, interpretCRNL } from './subst.js';
+import { substObjAnchor, preprocessTooltip } from './subst.js';
 import type { ResolvedFill, PolyStyleFlags } from './style-resolve.js';
 import {
   parseStyleFlags,
@@ -245,7 +245,7 @@ function anchorAttr(n: Node, key: string): string | undefined {
 
 /** Resolve tooltip string: explicit attr, or fall back to label text. */
 function resolveTooltip(n: Node, tooltip: string | undefined): string {
-  if (tooltip !== undefined) return substObjAnchor(interpretCRNL(tooltip), n);
+  if (tooltip !== undefined) return substObjAnchor(preprocessTooltip(tooltip), n);
   const label = n.info.label as TextlabelT | undefined;
   return label?.text ?? '';
 }

package/src/common/subst.ts

@@ -14,6 +14,7 @@ import { Node } from '../model/node.js';
 import { Edge } from '../model/edge.js';
 import { Graph } from '../model/graph.js';
 import type { TextlabelT } from './types.js';
+import { htmlEntityUTF8 } from './html-entities.js';
 
 export type GraphObj = Node | Edge | Graph;
 
@@ -124,3 +125,14 @@ export function interpretCRNL(str: string): string {
   }
   return out;
 }
+
+/**
+ * Tooltip preprocessing: decode HTML entities to UTF-8, then interpret
+ * \n \l \r. C picks latin1ToUTF8 or htmlEntityUTF8 by GD_charset; both decode
+ * entities via htmlEntity, and the port's Latin-1 bytes are already decoded
+ * at input, so one branch serves both (as in make_label).
+ * @see lib/common/emit.c:preprocessTooltip
+ */
+export function preprocessTooltip(s: string): string {
+  return interpretCRNL(htmlEntityUTF8(s));
+}

package/src/gvc/anchor.ts

@@ -17,7 +17,7 @@ import type { Node } from '../model/node.js';
 import type { Point, Box, Spline, Bezier } from '../model/geom.js';
 import type { TextlabelT, PolygonT } from '../common/types.js';
 import type { GraphObj } from '../common/subst.js';
-import { substObjAnchor, interpretCRNL } from '../common/subst.js';
+import { substObjAnchor, preprocessTooltip } from '../common/subst.js';
 import { parseStyleFlags } from '../common/style-resolve.js';
 import { nodeAttr } from '../common/poly-init.js';
 import { bezier } from '../common/bezier.js';
@@ -36,12 +36,13 @@ function subst(s: string, obj: GraphObj): string {
 }
 
 /**
- * preprocessTooltip + strdup_and_subst_obj: interpret \n\l\r escapes, then
- * object substitution — matching the node tooltip path (poly-gencode.ts).
+ * preprocessTooltip + strdup_and_subst_obj: decode entities and interpret
+ * \n\l\r escapes, then object substitution — matching the node tooltip path
+ * (poly-gencode.ts).
  * @see lib/common/emit.c:preprocessTooltip
  */
 function tip(s: string, obj: GraphObj): string {
-  return substObjAnchor(interpretCRNL(s), obj);
+  return substObjAnchor(preprocessTooltip(s), obj);
 }
 
 /** Label text of a textlabel, or null. @see device.ts:labelTextOf */
@@ -71,7 +72,7 @@ function pickEdge(e: Edge, keys: string[]): Picked {
   return pickFirst(e.attrs, keys, (s) => subst(s, e));
 }
 
-/** pickFirst with tooltip preprocessing (interpretCRNL + subst). */
+/** pickFirst with tooltip preprocessing (preprocessTooltip + subst). */
 function pickEdgeTip(e: Edge, keys: string[]): Picked {
   return pickFirst(e.attrs, keys, (s) => tip(s, e));
 }

package/src/render/xml-escape.ts

@@ -1,29 +1,25 @@
 // SPDX-License-Identifier: EPL-2.0
 //
-// XML escaping faithful to graphviz's gv_xml_escape (lib/util/xml.c). '<', '>',
-// '"', and "'" are always escaped; '&' is escaped to &amp; only when it does
-// not begin a valid entity (&name; / &#NNN; / &#xHH;). The text/tooltip variant
-// additionally honors the {raw, dash, nbsp} flags used by the SVG plugin.
+// The SVG renderer's three gv_xml_escape flag sets (lib/util/xml.c), as presets
+// over the faithful port in util/xml.ts. '<', '>', '"' and "'" are always
+// escaped; '&' is escaped only when it does not begin an entity
+// (xml_isentity), except under raw, which escapes every '&'.
 
-// Regexes for quote chars: /"/g and /'/g break Lizard's quote-tracker.
-// Use these named patterns instead to avoid the parser bug.
-const RE_DQUOTE = new RegExp('"', 'g');
-const RE_SQUOTE = new RegExp("'", 'g');
-// A bare '&' that does NOT begin a valid entity (&name; / &#NNN; / &#xHH;).
-// Mirrors xml_isentity (lib/util/xml.c): only such '&' are escaped to &amp;.
-const RE_AMP_NONENTITY = /&(?![A-Za-z]+;|#[0-9]+;|#x[0-9A-Fa-f]+;)/g;
+import { gvXmlEscape, type XmlFlags } from '../util/xml.js';
+
+/** flags {0}. @see plugin/core/gvrender_core_svg.c:svg_begin_anchor (href) */
+const FLAGS_BASE: XmlFlags = { raw: false, dash: false, nbsp: false, utf8: false };
+/** gvputs_xml's flags. @see lib/gvc/gvdevice.c:gvputs_xml */
+const FLAGS_TITLE: XmlFlags = { raw: false, dash: true, nbsp: true, utf8: false };
+/** @see plugin/core/gvrender_core_svg.c:svg_textspan, svg_begin_anchor (tooltip) */
+const FLAGS_TEXT: XmlFlags = { raw: true, dash: true, nbsp: true, utf8: false };
 
 /**
- * Base XML escape (gv_xml_escape with flags {0}): ids, titles, hrefs.
- * @see lib/util/xml.c gv_xml_escape / xml_core / xml_isentity
+ * Base XML escape (gv_xml_escape with flags {0}): ids, hrefs.
+ * @see lib/util/xml.c:gv_xml_escape
  */
 export function escapeXml(s: string): string {
-  let r = s.replace(RE_AMP_NONENTITY, '&amp;');
-  r = r.replace(/</g, '&lt;');
-  r = r.replace(/>/g, '&gt;');
-  r = r.replace(RE_DQUOTE, '&quot;');
-  r = r.replace(RE_SQUOTE, '&#39;');
-  return r;
+  return gvXmlEscape(s, FLAGS_BASE);
 }
 
 /**
@@ -34,23 +30,17 @@ export function escapeXml(s: string): string {
  * @see lib/gvc/gvdevice.c:gvputs_xml, lib/util/xml.c:xml_core
  */
 export function escapeXmlTitle(s: string): string {
-  let r = escapeXml(s);
-  r = r.replace(/-/g, '&#45;');
-  r = r.replace(/ {2,}/g, (m) => ' ' + '&#160;'.repeat(m.length - 1));
-  return r;
+  return gvXmlEscape(s, FLAGS_TITLE);
 }
 
 /**
- * Text/tooltip XML escape (gv_xml_escape with flags {raw, dash, nbsp}):
- * additionally escapes '-'→&#45;, newline/CR→&#10;/&#13;, and 2nd+ consecutive
- * spaces→&#160;. Used for textspans and tooltips.
- * @see plugin/core/gvrender_core_svg.c:546 (svg_textspan flags)
+ * Text/tooltip XML escape (gv_xml_escape with flags {raw, dash, nbsp}): raw
+ * escapes every '&' -- entity-looking or not -- and newline/CR to
+ * &#10;/&#13;; also '-'→&#45; and 2nd+ consecutive spaces→&#160;. Used for
+ * textspans and xlink:title.
+ * @see plugin/core/gvrender_core_svg.c:svg_textspan, svg_begin_anchor
+ * @see lib/util/xml.c:xml_core (`flags.raw || !xml_isentity(s)`)
  */
 export function escapeXmlText(s: string): string {
-  let r = escapeXml(s);
-  r = r.replace(/-/g, '&#45;');
-  r = r.replace(/\n/g, '&#10;');
-  r = r.replace(/\r/g, '&#13;');
-  r = r.replace(/ {2,}/g, (m) => ' ' + '&#160;'.repeat(m.length - 1));
-  return r;
+  return gvXmlEscape(s, FLAGS_TEXT);
 }