@luxalgo/vela 0.7.5 → 0.7.6

package/dist/widget.cjs

@@ -18460,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);
@@ -19846,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);
@@ -19868,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);
@@ -20146,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;
@@ -41977,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();
@@ -42100,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 {
@@ -42144,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();
@@ -44373,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);
@@ -44391,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);
       }
@@ -44404,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);
       },
@@ -44494,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-DYtiRtES.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-DYtiRtES.cjs';
-import { b as VelaOptions, d as MarketSession, T as ThemeName, c as VelaTheme, v as DataWindowReadout } from './options-DNBoMKkX.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-gA__yaog.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-gA__yaog.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-Dut6lXGM.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';
 

package/dist/widget.d.ts

@@ -1,10 +1,10 @@
-import { V as Vela, W as WidgetContext, K as SidePanelButton } from './contributions-CEkgJKTU.js';
-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-CEkgJKTU.js';
-import { b as VelaOptions, d as MarketSession, T as ThemeName, c as VelaTheme, v as DataWindowReadout } from './options-DNBoMKkX.js';
-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-DypGXjtr.js';
-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-DypGXjtr.js';
+import { V as Vela, W as WidgetContext, K as SidePanelButton } from './contributions-C7Wsm_S9.js';
+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-C7Wsm_S9.js';
+import { b as VelaOptions, d as MarketSession, T as ThemeName, c as VelaTheme, v as DataWindowReadout } from './options-D1AJKsn_.js';
+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-xBiVsKoE.js';
+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-xBiVsKoE.js';
 import { K as KeymapManager } from './keymap-CGOz5F5f.js';
-import { a as SymbolDescriptor } from './DataProvider-DLeFs1gE.js';
+import { a as SymbolDescriptor } from './DataProvider-CmWUtDVL.js';
 import { d as SidePanel } from './side-panel-HF0IAzwf.js';
 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.js';
 

package/dist/widget.js

@@ -1,6 +1,6 @@
-import { VelaWorkspace } from './chunk-MNG5XPLV.js';
-export { Bottombar, ChartContextMenu, DataWindow, IndicatorPicker, ObjectTree, PanelDock, RANGE_PRESETS, ShortcutsHelp, Statusline, SymbolPicker, TimeframeQuick, Topbar, Watermark, dataWindowSections, decimalsFor, decodeState, encodeState, filterSymbols, fmtChange, fmtPrice, localStorageAdapter, parseTimeframe, priceStyleLabel, resolveIndicators, sanitizeState, timeframeLabel, timeframeMs } from './chunk-MNG5XPLV.js';
-export { TIMEZONES, normalizeTimezone, tzButtonLabel, tzMenuLabel, tzOffset } from './chunk-MCLI6B3S.js';
+import { VelaWorkspace } from './chunk-EMB53WNQ.js';
+export { Bottombar, ChartContextMenu, DataWindow, IndicatorPicker, ObjectTree, PanelDock, RANGE_PRESETS, ShortcutsHelp, Statusline, SymbolPicker, TimeframeQuick, Topbar, Watermark, dataWindowSections, decimalsFor, decodeState, encodeState, filterSymbols, fmtChange, fmtPrice, localStorageAdapter, parseTimeframe, priceStyleLabel, resolveIndicators, sanitizeState, timeframeLabel, timeframeMs } from './chunk-EMB53WNQ.js';
+export { TIMEZONES, normalizeTimezone, tzButtonLabel, tzMenuLabel, tzOffset } from './chunk-JSVCWYBF.js';
 export { DEFAULT_PANEL_MAX_WIDTH, DEFAULT_PANEL_MIN_WIDTH, DEFAULT_PANEL_ORDER, DEFAULT_PANEL_WIDTH, OVERRIDABLE_TOPBAR_IDS, SidePanel, TOPBAR_BUILTIN_IDS, TOPBAR_DEFAULT_LEFT, TOPBAR_DEFAULT_RIGHT, clampPanelWidth, pinnedTopbarActionIds, registerSidePanel, registerStatePersistence, registerSymbolRanking, registerWidgetAction, registerWidgetAttachment, resolveTopbarComposition, sidePanels, statePersistenceHandlers, symbolRanking, topbarActionOverride, topbarHas, unregisterSidePanel, unregisterStatePersistence, unregisterWidgetAction, unregisterWidgetAttachment, widgetActions, widgetAttachments } from './chunk-BFA32GOU.js';
 import './chunk-ZADTVRUO.js';
 import './chunk-BKHSQ4YM.js';

package/dist/workspace.cjs

@@ -18392,10 +18392,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);
@@ -19778,7 +19786,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);
@@ -19800,7 +19808,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);
@@ -20078,6 +20086,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;
@@ -41909,6 +41935,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();
@@ -42032,7 +42059,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 {
@@ -42076,7 +42108,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();
@@ -44305,16 +44342,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);
@@ -44323,7 +44363,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);
       }
@@ -44336,7 +44376,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);
       },
@@ -44426,9 +44466,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/workspace.d.cts

@@ -1,9 +1,9 @@
-import { O as OHLCV, V as VisibleRangePreset, a as VisibleRange, b as VelaOptions, I as InputValue, c as VelaTheme, N as NativeBackend, d as MarketSession } from './options-DNBoMKkX.cjs';
-import { S as SyncSetting, V as VelaStorage, C as CellState, W as WidgetHistory, R as ResolvedIndicator, a as StatuslinePart, b as RangePreset, T as TrackSizes, c as VelaShellOptions, d as SyncOptions, e as SyncKind, f as WorkspaceState } from './statusline-model-gA__yaog.cjs';
-export { g as ChartState, P as PanelsState, h as decodeState, i as encodeState, s as sanitizeState } from './statusline-model-gA__yaog.cjs';
+import { O as OHLCV, V as VisibleRangePreset, a as VisibleRange, b as VelaOptions, I as InputValue, c as VelaTheme, N as NativeBackend, d as MarketSession } from './options-D1AJKsn_.cjs';
+import { S as SyncSetting, V as VelaStorage, C as CellState, W as WidgetHistory, R as ResolvedIndicator, a as StatuslinePart, b as RangePreset, T as TrackSizes, c as VelaShellOptions, d as SyncOptions, e as SyncKind, f as WorkspaceState } from './statusline-model-gWHrsOy3.cjs';
+export { g as ChartState, P as PanelsState, h as decodeState, i as encodeState, s as sanitizeState } from './statusline-model-gWHrsOy3.cjs';
 import { K as KeymapManager } from './keymap-CGOz5F5f.cjs';
-import { I as IndicatorHandle, S as ScriptingEngine, W as WidgetContext, V as Vela, E as ExternalIndicatorEntry, a as ScriptRun } from './contributions-DYtiRtES.cjs';
-import { M as MarketDataFeed } from './DataProvider-Dut6lXGM.cjs';
+import { I as IndicatorHandle, S as ScriptingEngine, W as WidgetContext, V as Vela, E as ExternalIndicatorEntry, a as ScriptRun } from './contributions-D6p1RB38.cjs';
+import { M as MarketDataFeed } from './DataProvider-CGa4KHRa.cjs';
 
 /** The cells that follow `originId` under `setting` — PURE (never includes the origin). */
 declare function syncTargets(originId: string, setting: SyncSetting | undefined, cellIds: readonly string[]): string[];
@@ -133,6 +133,10 @@ interface CellDeps {
 interface CellInstance {
     entry: ResolvedIndicator;
     handle: IndicatorHandle | null;
+    /** The id the indicator runs under on the chart — recorded on the first add (host-
+     *  supplied or minted) so an undo/redo resurrection re-adds it under the SAME id:
+     *  whatever a host keyed on it (a document, an editor tab) keeps pointing at it. */
+    id?: string;
     external?: boolean;
     values?: {
         inputs?: Record<string, InputValue>;
@@ -359,6 +363,7 @@ declare class ChartCell {
     addManifestInstance(entry: ResolvedIndicator, opts?: {
         record?: boolean;
         external?: boolean;
+        id?: string;
         inputs?: Record<string, InputValue>;
         props?: Record<string, InputValue>;
         hidden?: boolean;

package/dist/workspace.d.ts

@@ -1,9 +1,9 @@
-import { O as OHLCV, V as VisibleRangePreset, a as VisibleRange, b as VelaOptions, I as InputValue, c as VelaTheme, N as NativeBackend, d as MarketSession } from './options-DNBoMKkX.js';
-import { S as SyncSetting, V as VelaStorage, C as CellState, W as WidgetHistory, R as ResolvedIndicator, a as StatuslinePart, b as RangePreset, T as TrackSizes, c as VelaShellOptions, d as SyncOptions, e as SyncKind, f as WorkspaceState } from './statusline-model-DypGXjtr.js';
-export { g as ChartState, P as PanelsState, h as decodeState, i as encodeState, s as sanitizeState } from './statusline-model-DypGXjtr.js';
+import { O as OHLCV, V as VisibleRangePreset, a as VisibleRange, b as VelaOptions, I as InputValue, c as VelaTheme, N as NativeBackend, d as MarketSession } from './options-D1AJKsn_.js';
+import { S as SyncSetting, V as VelaStorage, C as CellState, W as WidgetHistory, R as ResolvedIndicator, a as StatuslinePart, b as RangePreset, T as TrackSizes, c as VelaShellOptions, d as SyncOptions, e as SyncKind, f as WorkspaceState } from './statusline-model-xBiVsKoE.js';
+export { g as ChartState, P as PanelsState, h as decodeState, i as encodeState, s as sanitizeState } from './statusline-model-xBiVsKoE.js';
 import { K as KeymapManager } from './keymap-CGOz5F5f.js';
-import { I as IndicatorHandle, S as ScriptingEngine, W as WidgetContext, V as Vela, E as ExternalIndicatorEntry, a as ScriptRun } from './contributions-CEkgJKTU.js';
-import { M as MarketDataFeed } from './DataProvider-DLeFs1gE.js';
+import { I as IndicatorHandle, S as ScriptingEngine, W as WidgetContext, V as Vela, E as ExternalIndicatorEntry, a as ScriptRun } from './contributions-C7Wsm_S9.js';
+import { M as MarketDataFeed } from './DataProvider-CmWUtDVL.js';
 
 /** The cells that follow `originId` under `setting` — PURE (never includes the origin). */
 declare function syncTargets(originId: string, setting: SyncSetting | undefined, cellIds: readonly string[]): string[];
@@ -133,6 +133,10 @@ interface CellDeps {
 interface CellInstance {
     entry: ResolvedIndicator;
     handle: IndicatorHandle | null;
+    /** The id the indicator runs under on the chart — recorded on the first add (host-
+     *  supplied or minted) so an undo/redo resurrection re-adds it under the SAME id:
+     *  whatever a host keyed on it (a document, an editor tab) keeps pointing at it. */
+    id?: string;
     external?: boolean;
     values?: {
         inputs?: Record<string, InputValue>;
@@ -359,6 +363,7 @@ declare class ChartCell {
     addManifestInstance(entry: ResolvedIndicator, opts?: {
         record?: boolean;
         external?: boolean;
+        id?: string;
         inputs?: Record<string, InputValue>;
         props?: Record<string, InputValue>;
         hidden?: boolean;

package/dist/workspace.js

@@ -1,5 +1,5 @@
-export { ChartCell, GRID_PICKER_MAX, VelaWorkspace, activeAfterLayout, decodeState, encodeState, ensureLayout, evenTracks, gridStyles, layoutDefinition, layoutForGrid, layoutShape, layouts, memoryStorageAdapter, rangesWithin, registerBuiltinLayouts, registerLayout, resizeTracks, sanitizeState, syncTargets, trackOffsets, unregisterLayout } from './chunk-MNG5XPLV.js';
-import './chunk-MCLI6B3S.js';
+export { ChartCell, GRID_PICKER_MAX, VelaWorkspace, activeAfterLayout, decodeState, encodeState, ensureLayout, evenTracks, gridStyles, layoutDefinition, layoutForGrid, layoutShape, layouts, memoryStorageAdapter, rangesWithin, registerBuiltinLayouts, registerLayout, resizeTracks, sanitizeState, syncTargets, trackOffsets, unregisterLayout } from './chunk-EMB53WNQ.js';
+import './chunk-JSVCWYBF.js';
 import './chunk-BFA32GOU.js';
 import './chunk-ZADTVRUO.js';
 import './chunk-BKHSQ4YM.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@luxalgo/vela",
-  "version": "0.7.5",
+  "version": "0.7.6",
   "description": "Open-source charting library with a native high-performance renderer, drawing tools, pluggable chart types and pluggable scripting engines.",
   "license": "Apache-2.0",
   "homepage": "https://luxalgo.com/vela",