@luxalgo/vela 0.7.4 → 0.7.6

package/dist/widget.cjs

@@ -5115,6 +5115,17 @@ function hasOwnCandlePaint(style) {
   for (const t of chartTypes()) if (t.id === style) return (t.basePainting ?? "candles") === "candles";
   return false;
 }
+var CANDLE_OVERRIDE_KEYS = [
+  "candleUpColor",
+  "candleDownColor",
+  "candleBodyVisible",
+  "candleBorderVisible",
+  "candleBorderUpColor",
+  "candleBorderDownColor",
+  "candleWickVisible",
+  "candleWickUpColor",
+  "candleWickDownColor"
+];
 function candleOverrideFor(style, bags) {
   if (!hasOwnCandlePaint(style)) return null;
   const bag = bags[style] ?? {};
@@ -5186,7 +5197,7 @@ function clampLevel(v) {
 function clampSpacing(v) {
   return v < 0.1 ? 0.1 : v > 10 ? 10 : v;
 }
-function factoryResetConfig(factory) {
+function factoryResetConfig(factory, priceStyle = factory.series.style) {
   const bag = {};
   for (const t of chartTypes()) {
     const section = t.settings;
@@ -5208,10 +5219,16 @@ function factoryResetConfig(factory) {
     if (!section.instances) addRows(section.rows);
     bag[t.id] = defaults2;
   }
+  for (const t of chartTypes()) {
+    if (!hasOwnCandlePaint(t.id)) continue;
+    const defaults2 = bag[t.id] ?? {};
+    for (const key of CANDLE_OVERRIDE_KEYS) defaults2[key] = null;
+    bag[t.id] = defaults2;
+  }
   for (const [typeId, vals] of Object.entries(factory.chartTypes)) {
     bag[typeId] = { ...bag[typeId] ?? {}, ...vals };
   }
-  return { ...factory, chartTypes: bag };
+  return { ...factory, chartTypes: bag, series: { ...factory.series, style: priceStyle } };
 }
 function mergeConfig(base, patch) {
   const p = asObject(patch);
@@ -18443,10 +18460,18 @@ var IndicatorRegistry = class {
     this.records = /* @__PURE__ */ new Map();
     this.counter = 0;
   }
-  /** Allocate a unique, stable per-instance id. */
-  nextId(prefix = "ind") {
-    this.counter += 1;
-    return `${prefix}-${this.counter}`;
+  /**
+   * Mint a unique per-instance id. Host-supplied ids share the namespace, so a minted
+   * id skips anything already recorded — and anything `taken` reports live elsewhere
+   * (a handle the orchestrator holds outside the records, e.g. a fail-soft native).
+   */
+  nextId(prefix = "ind", taken = () => false) {
+    let id;
+    do {
+      this.counter += 1;
+      id = `${prefix}-${this.counter}`;
+    } while (this.records.has(id) || taken(id));
+    return id;
   }
   add(record) {
     this.records.set(record.id, record);
@@ -18653,6 +18678,42 @@ function presetToRange(preset, bars) {
   return { from: Math.max(first, from), to: last };
 }
 
+// src/core/marks/visibility.ts
+function markGroupOwnVisible(choices, groupId, groups) {
+  const chosen = choices[groupId];
+  if (typeof chosen === "boolean") return chosen;
+  return groups.find((g) => g.id === groupId)?.visible !== false;
+}
+function markGroupVisible(choices, groupId, groups) {
+  if (groupId === void 0) return true;
+  const seen = /* @__PURE__ */ new Set();
+  let id = groupId;
+  while (id !== void 0 && !seen.has(id)) {
+    seen.add(id);
+    if (!markGroupOwnVisible(choices, id, groups)) return false;
+    const parent = groups.find((g) => g.id === id)?.parent;
+    id = parent !== void 0 && groups.some((g) => g.id === parent) ? parent : void 0;
+  }
+  return true;
+}
+function markGroupRows(groups) {
+  const ids = new Set(groups.map((g) => g.id));
+  const out = [];
+  const placed = /* @__PURE__ */ new Set();
+  const place = (group, depth) => {
+    if (placed.has(group.id)) return;
+    placed.add(group.id);
+    out.push({ group, depth });
+    for (const child of groups) if (child.parent === group.id && child.id !== group.id) place(child, depth + 1);
+  };
+  for (const g of groups) {
+    const parentKnown = g.parent !== void 0 && ids.has(g.parent) && g.parent !== g.id;
+    if (!parentKnown) place(g, 0);
+  }
+  for (const g of groups) place(g, 0);
+  return out;
+}
+
 // src/core/marks/MarksController.ts
 var MarksController = class {
   constructor(renderer, events) {
@@ -18697,6 +18758,7 @@ var MarksController = class {
   defineGroup(group) {
     if (!group || typeof group.id !== "string" || group.id.length === 0) throw new Error("[vela] marks.defineGroup: `id` must be a non-empty string");
     if (typeof group.label !== "string") throw new Error(`[vela] marks.defineGroup: group "${group.id}" needs a string \`label\``);
+    if (group.parent !== void 0 && (typeof group.parent !== "string" || group.parent.length === 0)) throw new Error(`[vela] marks.defineGroup: group "${group.id}" has a \`parent\` that is not a group id`);
     this.groups.set(group.id, { ...group });
     this.sync();
   }
@@ -18716,12 +18778,16 @@ var MarksController = class {
     }
     this.renderer.applyFeature("marks", { groups: { [id]: visible } });
   }
-  /** A group's effective visibility: the user's (persisted) choice, else the group's declared default, else visible. */
+  /**
+   * A group's effective visibility: its own switch — the user's (persisted) choice, else
+   * the declared default, else visible — AND every ancestor's, so a child under a
+   * switched-off parent reads hidden whatever its own choice says.
+   */
   isGroupVisible(id) {
     const state = this.renderer.readFeature("marks");
-    const chosen = state?.groups?.[id];
-    if (typeof chosen === "boolean") return chosen;
-    return this.groups.get(id)?.visible !== false;
+    const groups = {};
+    for (const [gid, v] of Object.entries(state?.groups ?? {})) if (typeof v === "boolean") groups[gid] = v;
+    return markGroupVisible(groups, id, [...this.groups.values()]);
   }
   destroy() {
     for (const unsub of this.subs) unsub();
@@ -19788,7 +19854,7 @@ var EngineOrchestrator = class _EngineOrchestrator {
     this.engines.set(language, engine);
   }
   addIndicator(source, options = {}) {
-    const id = this.registry.nextId();
+    const id = this.claimIndicatorId(options.id);
     const title = options.title ?? "Indicator";
     const handle = new IndicatorHandleImpl(id, title, this, source);
     this.handles.set(id, handle);
@@ -19810,7 +19876,7 @@ var EngineOrchestrator = class _EngineOrchestrator {
       const existing = this.registry.all().find((r) => r.native?.type === type);
       if (existing) return this.handles.get(existing.id) ?? new IndicatorHandleImpl(existing.id, existing.title, this, void 0, type);
     }
-    const id = this.registry.nextId("native");
+    const id = this.registry.nextId("native", (candidate) => this.handles.has(candidate));
     const title = descriptor?.title ?? type;
     const handle = new IndicatorHandleImpl(id, title, this, void 0, type);
     this.handles.set(id, handle);
@@ -20088,6 +20154,24 @@ var EngineOrchestrator = class _EngineOrchestrator {
     this.events.clear();
   }
   // ── internals ───────────────────────────────────────────────
+  /**
+   * The id a new script indicator runs under: the host's when supplied, else a minted
+   * one. A host id is opaque but must be a non-empty string, and it must not be live
+   * on this chart — a duplicate is a programming error surfaced synchronously, never
+   * renamed behind the caller's back (the whole point of supplying one is that
+   * `handle.id` equals what was passed). "Live" reads both the records and the handle
+   * map: a fail-soft handle never enters the registry but still owns its id.
+   */
+  claimIndicatorId(requested) {
+    if (requested === void 0) return this.registry.nextId("ind", (candidate) => this.handles.has(candidate));
+    if (typeof requested !== "string" || requested.length === 0) {
+      throw new TypeError("[vela] addIndicator: `id` must be a non-empty string");
+    }
+    if (this.handles.has(requested) || this.registry.get(requested)) {
+      throw new Error(`[vela] addIndicator: indicator id "${requested}" is already live on this chart`);
+    }
+    return requested;
+  }
   async startIndicator(id, source, options, handle) {
     try {
       await this.readyPromise;
@@ -25710,11 +25794,8 @@ function mergeMarksState(base, patch) {
   for (const [id, v] of Object.entries(g)) if (typeof v === "boolean") groups[id] = v;
   return { visible: typeof p.visible === "boolean" ? p.visible : base.visible, groups };
 }
-function markGroupVisible(state, groupId, groups) {
-  if (groupId === void 0) return true;
-  const chosen = state.groups[groupId];
-  if (typeof chosen === "boolean") return chosen;
-  return groups.find((g) => g.id === groupId)?.visible !== false;
+function markGroupVisible2(state, groupId, groups) {
+  return markGroupVisible(state.groups, groupId, groups);
 }
 
 // src/renderers/native/core/SceneGraph.ts
@@ -27890,13 +27971,29 @@ function paintMarkLane(ctx, layout, deps) {
       const img = deps.icons.get(mark.glyph.icon, ink, symbolPx, deps.dpr);
       if (img) ctx.drawImage(img, center.x - symbolPx / 2, center.y - symbolPx / 2, symbolPx, symbolPx);
     } else if (mark.glyph.letter) {
+      const letter = mark.glyph.letter.slice(0, 2);
       ctx.fillStyle = ink;
-      ctx.font = `600 ${Math.round(size * 0.58)}px ${deps.fontFamily}`;
-      ctx.fillText(mark.glyph.letter.slice(0, 2), center.x, center.y + 0.5);
+      let px = letterFontPx(size, letter);
+      ctx.font = `600 ${px}px ${deps.fontFamily}`;
+      if (letter.length > 1) {
+        px = fitLetterPx(px, ctx.measureText(letter).width, symbolInnerWidth(shape, size));
+        ctx.font = `600 ${px}px ${deps.fontFamily}`;
+      }
+      ctx.fillText(letter, center.x, center.y + 0.5);
     }
   }
   ctx.restore();
 }
+function letterFontPx(size, letter) {
+  return Math.round(size * (letter.length > 1 ? 0.38 : 0.58));
+}
+function symbolInnerWidth(shape, size) {
+  return (shape === "pin" ? size * 0.82 : size) - 3;
+}
+function fitLetterPx(px, width, inner) {
+  if (!(width > inner) || !(width > 0)) return px;
+  return Math.max(4, Math.floor(px * inner / width));
+}
 function traceShape(ctx, shape, x, y, size) {
   const r = size / 2;
   ctx.beginPath();
@@ -28098,7 +28195,7 @@ var ChromeRenderer = class {
     this.markLayout = layoutMarkLane({
       marks: scene.timelineMarks,
       groups: scene.markGroups,
-      hidden: (groupId) => !markGroupVisible(scene.marks, groupId, scene.markGroups),
+      hidden: (groupId) => !markGroupVisible2(scene.marks, groupId, scene.markGroups),
       barTimes: this.barTimes(scene),
       intervalMs: coords.barInterval,
       xOf: (bar) => coords.logicalToX(bar),
@@ -28788,6 +28885,10 @@ ${overlayScrollbarCss(".vela-sd-pane")}
    muted and non-interactive. Applied to each row's children so it survives display:contents;
    !important beats the inline opacity on labels. */
 .vela-sd-soft>*{opacity:0.4 !important;pointer-events:none !important;}
+/* A mark group nested under a parent group on the Events tab: indented one step per level
+   (--vela-sd-depth). The indent rides the first child (the switch) as a MARGIN \u2014 padding
+   would push the switch's own tick out of its box. */
+.vela-sd-nested>*:first-child{margin-left:calc(var(--vela-sd-depth,1)*24px);}
 /* \u2500\u2500 mobile presentation (.vela-sd-mobile on the scrim; structural sizes are inline in open()) \u2500\u2500
    The tab rail becomes a burger-opened overlay sidebar; the group TOC becomes a sticky
    row of horizontally scrollable tabs; the instance strip scrolls instead of wrapping;
@@ -28836,6 +28937,8 @@ var SettingsDialog = class {
     this.hostSections = [];
     /** The timeline-mark groups (defined + named by marks) — one checkbox each on the Events tab. */
     this.markGroups = [];
+    /** A group's OWN switch (what its checkbox shows) — a child under an off parent keeps its own state. */
+    this.markGroupOwnVisible = () => true;
     this.markGroupVisible = () => true;
     /** The Canvas → Theme row: current app theme + where a pick is raised. The row is a
      *  host callback, NOT a config patch — the app theme stays out of the persisted
@@ -28853,6 +28956,9 @@ var SettingsDialog = class {
     this.activeSection = null;
     /** Mobile chrome: fullscreen card, burger-opened section sidebar, TOC as top tabs. */
     this.mobileLayout = false;
+    /** Opens/closes the mobile section sidebar of the CURRENT build (the burger in the
+     *  shell header outlives pane rebuilds, so it reaches the rail through here). */
+    this.toggleRail = null;
     /** The visibility policy: setting ids hidden by the host (subtree semantics). */
     this.hiddenSettings = /* @__PURE__ */ new Set();
     if (getComputedStyle(container).position === "static") container.style.position = "relative";
@@ -28862,7 +28968,8 @@ var SettingsDialog = class {
     this.hostSections = sections;
   }
   /** The timeline-mark groups and their current visibility — the Events tab's rows on next open. */
-  setMarkGroups(groups, visible) {
+  setMarkGroups(groups, visible, ownVisible = visible) {
+    this.markGroupOwnVisible = ownVisible;
     this.markGroups = groups;
     this.markGroupVisible = visible;
   }
@@ -28928,7 +29035,6 @@ var SettingsDialog = class {
     this.onReset = onReset ?? null;
     ensureControlStyles();
     const mobile = this.mobileLayout;
-    let toggleRail = null;
     let burger;
     if (mobile) {
       burger = document.createElement("button");
@@ -28936,8 +29042,83 @@ var SettingsDialog = class {
       burger.className = "vela-sd-burger";
       burger.innerHTML = iconAt("burger", 16);
       burger.title = "Sections";
-      burger.addEventListener("click", () => toggleRail?.());
+      burger.addEventListener("click", () => this.toggleRail?.());
+    }
+    const ui = new Dialog({
+      host: this.container,
+      title: "Chart settings",
+      // Non-modal: a live-edit dialog must leave the page interactive — a modal
+      // machine locks pointer events on the whole body, killing the chart, the
+      // legend, and the body-portaled popovers (color picker, select lists).
+      modal: false,
+      contained: true,
+      align: "top",
+      draggable: !mobile,
+      flush: true,
+      className: "vela-dialog--settings",
+      headerStart: burger,
+      closeOnBackdrop: true,
+      footer: (foot) => {
+        foot.style.cssText = `padding:10px 14px;display:flex;align-items:center;justify-content:flex-start;gap:8px;`;
+        const resetBtn = document.createElement("button");
+        resetBtn.type = "button";
+        resetBtn.textContent = "Reset defaults";
+        resetBtn.className = "vela-sd-btn";
+        resetBtn.addEventListener("click", () => this.onReset?.());
+        foot.appendChild(resetBtn);
+      },
+      // A close signal may only close ITS OWN dialog: the machine reports the exit
+      // asynchronously, so a close-then-reopen in one tick would otherwise see the
+      // old instance's signal tear down the freshly opened one.
+      onOpenChange: (open2) => {
+        if (!open2 && this.ui === ui) this.close();
+      }
+    });
+    if (mobile) ui.positioner.classList.add("vela-sd-mobile");
+    ui.positioner.style.paddingTop = mobile ? "0" : "8vh";
+    this.root = ui.positioner;
+    this.ui = ui;
+    const panes = this.buildContent(ui, config, section, mobile);
+    ui.show();
+    this.layoutPanes(panes);
+  }
+  /**
+   * Re-seed every control of an OPEN dialog from `config`, in place: the shell stays
+   * (no close/open transition), only the tab rail and panes rebuild, landing back on
+   * the current tab. The reset path — the restored values must show without the
+   * dialog re-entering. No-op while closed.
+   */
+  refresh(config) {
+    const ui = this.ui;
+    if (!ui) return;
+    closeOpenPopovers();
+    closeWidthPopover();
+    for (const dispose of this.hintTips) dispose();
+    this.hintTips = [];
+    this.config = config;
+    ui.body.replaceChildren();
+    this.layoutPanes(this.buildContent(ui, config, this.activeSection ?? void 0, this.mobileLayout));
+  }
+  /** Structured chart-type panes (instance strip / group TOC) own their layout and
+   *  tag their rows hosts instead; each host gets its own field grid. Runs on a SHOWN
+   *  dialog — the grids measure their labels. */
+  layoutPanes(panes) {
+    for (const el of panes) {
+      const hosts = [...el.querySelectorAll("[data-sd-rows-host]")];
+      if (hosts.length === 0) {
+        this.layoutSettingsGrids(el);
+        continue;
+      }
+      for (const h of hosts) this.layoutSettingsGrids(h);
     }
+  }
+  /**
+   * Build the tab rail + one pane per section into `ui.body` (the linear `body` of
+   * section markers and rows is split afterwards), select `section` (the active
+   * style's own tab when none is asked for), and return the pane elements for
+   * {@link layoutPanes}.
+   */
+  buildContent(ui, config, section, mobile) {
     const body = document.createElement("div");
     body.style.cssText = "display:flex;flex-direction:column;gap:0;";
     const sid = (el, id) => {
@@ -29141,9 +29322,28 @@ var SettingsDialog = class {
     if (this.markGroups.length > 0) {
       body.append(sid(this.section("Events"), MARKS_SETTINGS_ID));
       body.append(sid(this.sectionTitle("Visible events"), MARKS_GROUPS_SETTINGS_ID));
-      for (const g of this.markGroups) {
-        body.append(sid(this.boolRow(g.label, this.markGroupVisible(g.id), (v) => this.emit({ marks: { groups: { [g.id]: v } } })), markGroupSettingsId(g.id)));
+      const rowsById = /* @__PURE__ */ new Map();
+      const rows = markGroupRows(this.markGroups);
+      const refreshDimming = () => {
+        for (const { group } of rows) {
+          const el = rowsById.get(group.id);
+          if (!el || group.parent === void 0) continue;
+          el.classList.toggle("vela-sd-soft", !this.markGroupVisible(group.parent));
+        }
+      };
+      for (const { group: g, depth } of rows) {
+        const row = this.boolRow(g.label, this.markGroupOwnVisible(g.id), (v) => {
+          this.emit({ marks: { groups: { [g.id]: v } } });
+          refreshDimming();
+        });
+        if (depth > 0) {
+          row.classList.add("vela-sd-nested");
+          row.style.setProperty("--vela-sd-depth", String(depth));
+        }
+        rowsById.set(g.id, row);
+        body.append(sid(row, markGroupSettingsId(g.id)));
       }
+      refreshDimming();
     }
     renderChartTypeSections("end");
     renderHostSections("end");
@@ -29171,8 +29371,8 @@ var SettingsDialog = class {
     if (mobile) {
       railScrim = document.createElement("div");
       railScrim.className = "vela-sd-railscrim";
-      railScrim.addEventListener("click", () => toggleRail?.());
-      toggleRail = (open2) => {
+      railScrim.addEventListener("click", () => this.toggleRail?.());
+      this.toggleRail = (open2) => {
         const on = open2 ?? !rail.classList.contains("open");
         rail.classList.toggle("open", on);
         railScrim?.classList.toggle("open", on);
@@ -29212,35 +29412,6 @@ var SettingsDialog = class {
       });
       if (hidActive) activate(0);
     };
-    const ui = new Dialog({
-      host: this.container,
-      title: "Chart settings",
-      // Non-modal: a live-edit dialog must leave the page interactive — a modal
-      // machine locks pointer events on the whole body, killing the chart, the
-      // legend, and the body-portaled popovers (color picker, select lists).
-      modal: false,
-      contained: true,
-      align: "top",
-      draggable: !mobile,
-      flush: true,
-      className: "vela-dialog--settings",
-      headerStart: burger,
-      closeOnBackdrop: true,
-      footer: (foot) => {
-        foot.style.cssText = `padding:10px 14px;display:flex;align-items:center;justify-content:flex-start;gap:8px;`;
-        const resetBtn = document.createElement("button");
-        resetBtn.type = "button";
-        resetBtn.textContent = "Reset defaults";
-        resetBtn.className = "vela-sd-btn";
-        resetBtn.addEventListener("click", () => this.onReset?.());
-        foot.appendChild(resetBtn);
-      },
-      onOpenChange: (open2) => {
-        if (!open2) this.close();
-      }
-    });
-    if (mobile) ui.positioner.classList.add("vela-sd-mobile");
-    ui.positioner.style.paddingTop = mobile ? "0" : "8vh";
     const activate = (idx) => {
       panes.forEach((p, i) => {
         p.el.style.display = i === idx ? "block" : "none";
@@ -29249,7 +29420,7 @@ var SettingsDialog = class {
       this.activeSection = panes[idx]?.title ?? null;
       if (mobile) {
         ui.titleEl.textContent = panes[idx]?.title ?? "Chart settings";
-        toggleRail?.(false);
+        this.toggleRail?.(false);
       }
     };
     panes.forEach((p, i) => {
@@ -29264,17 +29435,7 @@ var SettingsDialog = class {
     shell.append(rail, paneHost);
     if (railScrim) shell.append(railScrim);
     ui.body.appendChild(shell);
-    this.root = ui.positioner;
-    this.ui = ui;
-    ui.show();
-    for (const p of panes) {
-      const hosts = [...p.el.querySelectorAll("[data-sd-rows-host]")];
-      if (hosts.length === 0) {
-        this.layoutSettingsGrids(p.el);
-        continue;
-      }
-      for (const h of hosts) this.layoutSettingsGrids(h);
-    }
+    return panes.map((p) => p.el);
   }
   close() {
     closeOpenPopovers();
@@ -29282,6 +29443,7 @@ var SettingsDialog = class {
     const ui = this.ui;
     this.ui = null;
     this.root = null;
+    this.toggleRail = null;
     this.tabs = [];
     for (const dispose of this.hintTips) dispose();
     this.hintTips = [];
@@ -36600,7 +36762,11 @@ var NativeRenderer = class {
     }
     this.settingsDialog.setTheme(this.theme);
     this.settingsDialog.setHostSections(this.hostSettingsSections);
-    this.settingsDialog.setMarkGroups(this.markGroupsInUse(), (id) => markGroupVisible(this.scene.marks, id, this.scene.markGroups));
+    this.settingsDialog.setMarkGroups(
+      this.markGroupsForEventsTab(),
+      (id) => markGroupVisible2(this.scene.marks, id, this.scene.markGroups),
+      (id) => markGroupOwnVisible(this.scene.marks.groups, id, this.scene.markGroups)
+    );
     this.settingsDialog.setHiddenSettings(this.hiddenSettings);
     this.syncThemeControl();
     this.settingsDialog.toggle(
@@ -36608,13 +36774,24 @@ var NativeRenderer = class {
       (patch) => this.applyConfig(patch),
       (json) => this.applyConfig(json),
       () => {
-        if (this.factoryConfig) this.applyConfig(factoryResetConfig(this.factoryConfig));
-        this.settingsDialog?.close();
-        this.openSettingsDialog();
+        if (this.factoryConfig) this.applyConfig(this.factoryResetDocument(this.factoryConfig));
+        this.settingsDialog?.refresh(this.getConfig());
       },
       section
     );
   }
+  /**
+   * The document "Reset defaults" applies: every setting back to its first-run value,
+   * with two things that are NOT settings held or resolved here — the price style
+   * stays the one the user is looking at, and the timeline-mark groups (an additive
+   * merge, like the type bags) are named back to their host-declared visibility.
+   */
+  factoryResetDocument(factory) {
+    const doc = factoryResetConfig(factory, this.scene.priceStyle);
+    const groups = { ...doc.marks.groups };
+    for (const g of this.markGroupsInUse()) groups[g.id] = g.visible !== false;
+    return { ...doc, marks: { ...doc.marks, groups } };
+  }
   /** Close the in-chart dialogs (indicator settings + chart-settings gear). No-op when none are open. */
   closeDialogs() {
     this.inputsUI?.closeOpenDialog();
@@ -37677,6 +37854,15 @@ var NativeRenderer = class {
   markGroupsInUse() {
     return effectiveMarkGroups(this.scene.timelineMarks, this.scene.markGroups);
   }
+  /**
+   * The groups as the Events tab lists them: nested only under a DEFINED parent. The
+   * painter's visibility chain resolves parents against the defined groups alone, so a
+   * parent that marks merely name must not nest (and dim) a child the painter still shows.
+   */
+  markGroupsForEventsTab() {
+    const defined = new Set(this.scene.markGroups.map((g) => g.id));
+    return this.markGroupsInUse().map((g) => g.parent !== void 0 && !defined.has(g.parent) ? { ...g, parent: void 0 } : g);
+  }
   onViewportChange(cb) {
     this.viewportCbs.add(cb);
     return () => this.viewportCbs.delete(cb);
@@ -41817,6 +42003,7 @@ function registerClassicIndicators() {
 }
 
 // src/Vela.ts
+var asError = (err) => err instanceof Error ? err : new Error(String(err));
 var Vela = class {
   constructor(container, options = {}, deps = {}) {
     registerBuiltinChartTypes();
@@ -41940,7 +42127,12 @@ var Vela = class {
    * fetch from — the same relationship `script:run` has to `context:changed`.
    */
   runScript(source, options) {
-    const handle = this.addIndicator(source, options);
+    let handle;
+    try {
+      handle = this.addIndicator(source, options);
+    } catch (err) {
+      return Promise.resolve({ ok: false, run: null, error: asError(err), onUpdate: () => () => void 0, remove: () => void 0 });
+    }
     const updates = /* @__PURE__ */ new Set();
     const drop = () => {
       try {
@@ -41984,7 +42176,12 @@ var Vela = class {
     });
   }
   runIndicator(source, options) {
-    const handle = this.addIndicator(source, options);
+    let handle;
+    try {
+      handle = this.addIndicator(source, options);
+    } catch (err) {
+      return Promise.resolve({ ok: false, handle: null, error: asError(err), context: null });
+    }
     return new Promise((resolve) => {
       const offReady = handle.on("ready", () => {
         offReady();
@@ -43569,6 +43766,11 @@ var ChartCell = class {
       if (typeof zone === "string" && normalizeTimezone(zone) !== normalizeTimezone(this.deps.timezone())) {
         this.deps.setTimezone(normalizeTimezone(zone));
       }
+      const style = this.priceStyle;
+      if (style !== (this.state.priceStyle ?? "candles")) {
+        this.state.priceStyle = style;
+        this.deps.onPriceStyleChanged(this.id);
+      }
       this.syncStatuslineColors();
       this.syncPlotOverlayTokens();
     });
@@ -44208,16 +44410,19 @@ var ChartCell = class {
    * a persistence handler's `restore` runs silently, a user-driven call records.
    */
   addExternalIndicator(entry) {
+    const { id, inputs, props, hidden, ...script } = entry;
     this.addManifestInstance(
-      { ...entry, enabled: true },
-      { external: true, ...entry.inputs ? { inputs: entry.inputs } : {}, ...entry.props ? { props: entry.props } : {}, ...entry.hidden ? { hidden: true } : {} }
+      { ...script, enabled: true },
+      { external: true, ...id !== void 0 ? { id } : {}, ...inputs ? { inputs } : {}, ...props ? { props } : {}, ...hidden ? { hidden: true } : {} }
     );
   }
   /** Add ONE instance of a manifest entry (repeatable — duplicates are legitimate). */
   addManifestInstance(entry, opts = {}) {
     if (this.destroyed) return;
     const values = opts.inputs || opts.props ? { inputs: opts.inputs, props: opts.props } : void 0;
-    const it = { entry, handle: this.addToChart(entry, values), ...opts.external ? { external: true } : {}, ...values ? { values } : {} };
+    const handle = this.addToChart(entry, values, opts.id);
+    if (!handle) return;
+    const it = { entry, handle, id: handle.id, ...opts.external ? { external: true } : {}, ...values ? { values } : {} };
     if (opts.hidden) it.handle?.setVisible(false);
     this.instances.push(it);
     this.deps.onIndicatorsChanged(this.id);
@@ -44226,7 +44431,7 @@ var ChartCell = class {
     this.history.push({
       undo: () => this.dropInstance(snapshot),
       redo: () => {
-        snapshot.handle = this.addToChart(snapshot.entry, snapshot.values);
+        snapshot.handle = this.addToChart(snapshot.entry, snapshot.values, snapshot.id);
         this.instances.push(snapshot);
         this.deps.onIndicatorsChanged(this.id);
       }
@@ -44239,7 +44444,7 @@ var ChartCell = class {
     const snapshot = it;
     this.history.push({
       undo: () => {
-        snapshot.handle = this.addToChart(snapshot.entry, snapshot.values);
+        snapshot.handle = this.addToChart(snapshot.entry, snapshot.values, snapshot.id);
         this.instances.push(snapshot);
         this.deps.onIndicatorsChanged(this.id);
       },
@@ -44329,9 +44534,10 @@ var ChartCell = class {
       this.deps.onIndicatorsChanged(this.id);
     });
   }
-  addToChart(entry, values) {
+  addToChart(entry, values, id) {
     try {
       return this.inner?.addIndicator(entry.script, {
+        ...id !== void 0 ? { id } : {},
         ...entry.language !== void 0 ? { language: entry.language } : {},
         ...values?.inputs ? { inputs: values.inputs } : {},
         ...values?.props ? { props: values.props } : {}

package/dist/widget.d.cts

@@ -1,10 +1,10 @@
-import { V as Vela, W as WidgetContext, K as SidePanelButton } from './contributions-DpZeu7KP.cjs';
-export { C as CellStateContext, aA as DEFAULT_PANEL_ORDER, E as ExternalIndicatorEntry, O as OVERRIDABLE_TOPBAR_IDS, Q as SidePanelDescriptor, T as SidePanelHandle, X as StatePersistenceHandler, $ as SymbolRankingHook, a4 as WidgetActionDescriptor, a5 as WidgetActionTarget, a6 as WidgetAttachment, ag as registerSidePanel, ah as registerStatePersistence, ai as registerSymbolRanking, aj as registerWidgetAction, ak as registerWidgetAttachment, am as sidePanels, an as statePersistenceHandlers, ao as symbolRanking, ap as topbarActionOverride, au as unregisterSidePanel, av as unregisterStatePersistence, aw as unregisterWidgetAction, ax as unregisterWidgetAttachment, ay as widgetActions, az as widgetAttachments } from './contributions-DpZeu7KP.cjs';
-import { b as VelaOptions, d as MarketSession, T as ThemeName, c as VelaTheme, v as DataWindowReadout } from './options-XVpfr9HV.cjs';
-import { c as VelaShellOptions, W as WidgetHistory, b as RangePreset, f as WorkspaceState, j as TopbarComposition, a as StatuslinePart, P as PanelsState } from './statusline-model-Bd4qKuD6.cjs';
-export { B as Bottombar, k as BottombarOptions, C as CellState, g as ChartState, I as IndicatorLoader, l as IndicatorManifest, m as IndicatorManifestEntry, n as RANGE_PRESETS, R as ResolvedIndicator, o as ResolvedTopbarComposition, p as TOPBAR_BUILTIN_IDS, q as TOPBAR_DEFAULT_LEFT, r as TOPBAR_DEFAULT_RIGHT, V as VelaStorage, t as WidgetStorage, h as decodeState, i as encodeState, u as localStorageAdapter, v as pinnedTopbarActionIds, w as resolveIndicators, x as resolveTopbarComposition, s as sanitizeState, y as topbarHas } from './statusline-model-Bd4qKuD6.cjs';
+import { V as Vela, W as WidgetContext, K as SidePanelButton } from './contributions-D6p1RB38.cjs';
+export { C as CellStateContext, aA as DEFAULT_PANEL_ORDER, E as ExternalIndicatorEntry, O as OVERRIDABLE_TOPBAR_IDS, Q as SidePanelDescriptor, T as SidePanelHandle, X as StatePersistenceHandler, $ as SymbolRankingHook, a4 as WidgetActionDescriptor, a5 as WidgetActionTarget, a6 as WidgetAttachment, ag as registerSidePanel, ah as registerStatePersistence, ai as registerSymbolRanking, aj as registerWidgetAction, ak as registerWidgetAttachment, am as sidePanels, an as statePersistenceHandlers, ao as symbolRanking, ap as topbarActionOverride, au as unregisterSidePanel, av as unregisterStatePersistence, aw as unregisterWidgetAction, ax as unregisterWidgetAttachment, ay as widgetActions, az as widgetAttachments } from './contributions-D6p1RB38.cjs';
+import { b as VelaOptions, d as MarketSession, T as ThemeName, c as VelaTheme, v as DataWindowReadout } from './options-D1AJKsn_.cjs';
+import { c as VelaShellOptions, W as WidgetHistory, b as RangePreset, f as WorkspaceState, j as TopbarComposition, a as StatuslinePart, P as PanelsState } from './statusline-model-gWHrsOy3.cjs';
+export { B as Bottombar, k as BottombarOptions, C as CellState, g as ChartState, I as IndicatorLoader, l as IndicatorManifest, m as IndicatorManifestEntry, n as RANGE_PRESETS, R as ResolvedIndicator, o as ResolvedTopbarComposition, p as TOPBAR_BUILTIN_IDS, q as TOPBAR_DEFAULT_LEFT, r as TOPBAR_DEFAULT_RIGHT, V as VelaStorage, t as WidgetStorage, h as decodeState, i as encodeState, u as localStorageAdapter, v as pinnedTopbarActionIds, w as resolveIndicators, x as resolveTopbarComposition, s as sanitizeState, y as topbarHas } from './statusline-model-gWHrsOy3.cjs';
 import { K as KeymapManager } from './keymap-CGOz5F5f.cjs';
-import { a as SymbolDescriptor } from './DataProvider-RF1axuy1.cjs';
+import { a as SymbolDescriptor } from './DataProvider-CGa4KHRa.cjs';
 import { d as SidePanel } from './side-panel-HF0IAzwf.cjs';
 export { D as DEFAULT_PANEL_MAX_WIDTH, a as DEFAULT_PANEL_MIN_WIDTH, b as DEFAULT_PANEL_WIDTH, S as SidePanelOptions, c as clampPanelWidth } from './side-panel-HF0IAzwf.cjs';