@363045841yyt/klinechart 0.7.4 → 0.7.5

package/src/__tests__/_mockController.ts

@@ -1,192 +1,192 @@
-/**
- * Mock ChartController for Vue adapter tests.
- *
- * Mirrors the framework-agnostic signal-bearing shape from
- * @363045841yyt/klinechart-core without spinning up the real Chart engine.
- *
- * To keep this test file runnable from the repo root vitest (which does not
- * alias @363045841yyt/klinechart-core), we inline a tiny `Signal` implementation
- * that is shape-compatible with `packages/core/src/reactivity/signal.ts`.
- */
-
-import type { Signal } from '@363045841yyt/klinechart-core/reactivity'
-import type {
-    ActiveIndicator,
-    ChartController,
-    ChartMountOptions,
-    ChartViewport,
-    DrawingController,
-    DrawingState,
-    IndicatorDefinition,
-    IndicatorSelectorController,
-    KLineData,
-    ToolbarController,
-    ToolDefinition,
-    ToolId,
-} from '@363045841yyt/klinechart-core'
-
-// ---------------------------------------------------------------------------
-// Inline mini-signal �?Object.is-equality, sync notify. Drop-in compatible
-// with `@363045841yyt/klinechart-core/reactivity` for shape-only test purposes.
-// ---------------------------------------------------------------------------
-
-function createSignal<T>(initial: T): Signal<T> {
-    let value = initial
-    const subs = new Set<() => void>()
-    const read = (): T => value
-    const peek = (): T => value
-    const set = (next: T): void => {
-        if (Object.is(value, next)) return
-        value = next
-        for (const listener of [...subs]) listener()
-    }
-    const subscribe = (listener: () => void): (() => void) => {
-        subs.add(listener)
-        return () => {
-            subs.delete(listener)
-        }
-    }
-    return Object.assign(read, { peek, set, subscribe }) as Signal<T>
-}
-
-export interface MockChartController extends ChartController {
-    /** spy: how many times `dispose` was called */
-    disposeCalls: () => number
-    /** test-only signal mutators */
-    _setViewport: (vp: ChartViewport) => void
-    _setData: (data: ReadonlyArray<KLineData>) => void
-}
-
-function createMockIndicatorSelector(): IndicatorSelectorController {
-    const catalog = createSignal<ReadonlyArray<IndicatorDefinition>>([])
-    const active = createSignal<ReadonlyArray<ActiveIndicator>>([])
-    const menuOpen = createSignal<boolean>(false)
-    const searchQuery = createSignal<string>('')
-    const filteredMain = createSignal<ReadonlyArray<IndicatorDefinition>>([])
-    const filteredSub = createSignal<ReadonlyArray<IndicatorDefinition>>([])
-
-    return {
-        catalog,
-        active,
-        menuOpen,
-        searchQuery,
-        filteredMain,
-        filteredSub,
-        add: () => null,
-        remove: () => false,
-        updateParams: () => false,
-        reorder: () => false,
-        openMenu: () => menuOpen.set(true),
-        closeMenu: () => menuOpen.set(false),
-        toggleMenu: () => menuOpen.set(!menuOpen.peek()),
-        setSearchQuery: (q: string) => searchQuery.set(q),
-        isActive: () => false,
-        dispose: () => {},
-    }
-}
-
-function createMockToolbar(): ToolbarController {
-    const tools = createSignal<ReadonlyArray<ToolDefinition>>([])
-    const activeTool = createSignal<ToolId | null>(null)
-    const disabledTools = createSignal<ReadonlySet<ToolId>>(new Set())
-    return {
-        tools,
-        activeTool,
-        disabledTools,
-        selectTool: (id) => activeTool.set(id),
-        clearSelection: () => activeTool.set(null),
-        setDisabled: () => {},
-        dispose: () => {},
-    }
-}
-
-function createMockDrawing(): DrawingController {
-    const state = createSignal<DrawingState>({
-        activeTool: null,
-        drawingCount: 0,
-    })
-    return {
-        state,
-        setActiveTool: (tool) =>
-            state.set({ ...state.peek(), activeTool: tool }),
-        clearAll: () => state.set({ ...state.peek(), drawingCount: 0 }),
-        deleteLast: () =>
-            state.set({
-                ...state.peek(),
-                drawingCount: Math.max(0, state.peek().drawingCount - 1),
-            }),
-        dispose: () => {},
-    }
-}
-
-export function createMockChartController(
-    opts: Partial<ChartMountOptions> = {},
-): MockChartController {
-    let disposeCalls = 0
-
-    const viewport = createSignal<ChartViewport>({
-        zoomLevel: opts.initialZoomLevel ?? 3,
-        kWidth: 6,
-        visibleFrom: 0,
-        visibleTo: 0,
-    })
-    const data = createSignal<ReadonlyArray<KLineData>>(opts.data ?? [])
-    const theme = createSignal<'light' | 'dark'>(opts.theme ?? 'light')
-    const indicatorSelector = createMockIndicatorSelector()
-    const toolbar = createMockToolbar()
-    const drawing = createMockDrawing()
-
-    return {
-        viewport,
-        data,
-        theme,
-        indicatorSelector,
-        toolbar,
-        drawing,
-        setData: (next) => data.set(next),
-        appendData: (next) => data.set([...data.peek(), ...next]),
-        updateData: (next) => data.set(next),
-        setTheme: (next) => theme.set(next),
-        zoomToLevel: (level) =>
-            viewport.set({ ...viewport.peek(), zoomLevel: level }),
-        zoomIn: () =>
-            viewport.set({
-                ...viewport.peek(),
-                zoomLevel: viewport.peek().zoomLevel + 1,
-            }),
-        zoomOut: () =>
-            viewport.set({
-                ...viewport.peek(),
-                zoomLevel: viewport.peek().zoomLevel - 1,
-            }),
-        handlePointerEvent: () => false,
-        handleWheelEvent: () => {},
-        handleScrollEvent: () => {},
-        handlePinchZoom: () => {},
-        addIndicator: () => null,
-        removeIndicator: () => false,
-        updateIndicatorParams: () => false,
-        updateRendererConfig: () => {},
-        setDrawingTool: (tool) => drawing.setActiveTool(tool),
-        clearDrawings: () => drawing.clearAll(),
-        removeDrawing: () => {},
-        resizeSubPane: () => false,
-        createSubPane: () => false,
-        clearSubPanes: () => {},
-        updateCustomMarkers: () => {},
-        clearCustomMarkers: () => {},
-        updateSettingsFacade: () => {},
-        updateOptionsFacade: () => {},
-        dispose: () => {
-            disposeCalls += 1
-        },
-        disposeCalls: () => disposeCalls,
-        _setViewport: (vp) => viewport.set(vp),
-        _setData: (next) => data.set(next),
-    }
-}
-
-/** Signal helper used by reactivity bridge tests. */
-export function createTestSignal<T>(initial: T): Signal<T> {
-    return createSignal(initial)
-}
+/**
+ * Mock ChartController for Vue adapter tests.
+ *
+ * Mirrors the framework-agnostic signal-bearing shape from
+ * @363045841yyt/klinechart-core without spinning up the real Chart engine.
+ *
+ * To keep this test file runnable from the repo root vitest (which does not
+ * alias @363045841yyt/klinechart-core), we inline a tiny `Signal` implementation
+ * that is shape-compatible with `packages/core/src/reactivity/signal.ts`.
+ */
+
+import type { Signal } from '@363045841yyt/klinechart-core/reactivity'
+import type {
+    ActiveIndicator,
+    ChartController,
+    ChartMountOptions,
+    ChartViewport,
+    DrawingController,
+    DrawingState,
+    IndicatorDefinition,
+    IndicatorSelectorController,
+    KLineData,
+    ToolbarController,
+    ToolDefinition,
+    ToolId,
+} from '@363045841yyt/klinechart-core'
+
+// ---------------------------------------------------------------------------
+// Inline mini-signal �?Object.is-equality, sync notify. Drop-in compatible
+// with `@363045841yyt/klinechart-core/reactivity` for shape-only test purposes.
+// ---------------------------------------------------------------------------
+
+function createSignal<T>(initial: T): Signal<T> {
+    let value = initial
+    const subs = new Set<() => void>()
+    const read = (): T => value
+    const peek = (): T => value
+    const set = (next: T): void => {
+        if (Object.is(value, next)) return
+        value = next
+        for (const listener of [...subs]) listener()
+    }
+    const subscribe = (listener: () => void): (() => void) => {
+        subs.add(listener)
+        return () => {
+            subs.delete(listener)
+        }
+    }
+    return Object.assign(read, { peek, set, subscribe }) as Signal<T>
+}
+
+export interface MockChartController extends ChartController {
+    /** spy: how many times `dispose` was called */
+    disposeCalls: () => number
+    /** test-only signal mutators */
+    _setViewport: (vp: ChartViewport) => void
+    _setData: (data: ReadonlyArray<KLineData>) => void
+}
+
+function createMockIndicatorSelector(): IndicatorSelectorController {
+    const catalog = createSignal<ReadonlyArray<IndicatorDefinition>>([])
+    const active = createSignal<ReadonlyArray<ActiveIndicator>>([])
+    const menuOpen = createSignal<boolean>(false)
+    const searchQuery = createSignal<string>('')
+    const filteredMain = createSignal<ReadonlyArray<IndicatorDefinition>>([])
+    const filteredSub = createSignal<ReadonlyArray<IndicatorDefinition>>([])
+
+    return {
+        catalog,
+        active,
+        menuOpen,
+        searchQuery,
+        filteredMain,
+        filteredSub,
+        add: () => null,
+        remove: () => false,
+        updateParams: () => false,
+        reorder: () => false,
+        openMenu: () => menuOpen.set(true),
+        closeMenu: () => menuOpen.set(false),
+        toggleMenu: () => menuOpen.set(!menuOpen.peek()),
+        setSearchQuery: (q: string) => searchQuery.set(q),
+        isActive: () => false,
+        dispose: () => {},
+    }
+}
+
+function createMockToolbar(): ToolbarController {
+    const tools = createSignal<ReadonlyArray<ToolDefinition>>([])
+    const activeTool = createSignal<ToolId | null>(null)
+    const disabledTools = createSignal<ReadonlySet<ToolId>>(new Set())
+    return {
+        tools,
+        activeTool,
+        disabledTools,
+        selectTool: (id) => activeTool.set(id),
+        clearSelection: () => activeTool.set(null),
+        setDisabled: () => {},
+        dispose: () => {},
+    }
+}
+
+function createMockDrawing(): DrawingController {
+    const state = createSignal<DrawingState>({
+        activeTool: null,
+        drawingCount: 0,
+    })
+    return {
+        state,
+        setActiveTool: (tool) =>
+            state.set({ ...state.peek(), activeTool: tool }),
+        clearAll: () => state.set({ ...state.peek(), drawingCount: 0 }),
+        deleteLast: () =>
+            state.set({
+                ...state.peek(),
+                drawingCount: Math.max(0, state.peek().drawingCount - 1),
+            }),
+        dispose: () => {},
+    }
+}
+
+export function createMockChartController(
+    opts: Partial<ChartMountOptions> = {},
+): MockChartController {
+    let disposeCalls = 0
+
+    const viewport = createSignal<ChartViewport>({
+        zoomLevel: opts.initialZoomLevel ?? 3,
+        kWidth: 6,
+        visibleFrom: 0,
+        visibleTo: 0,
+    })
+    const data = createSignal<ReadonlyArray<KLineData>>(opts.data ?? [])
+    const theme = createSignal<'light' | 'dark'>(opts.theme ?? 'light')
+    const indicatorSelector = createMockIndicatorSelector()
+    const toolbar = createMockToolbar()
+    const drawing = createMockDrawing()
+
+    return {
+        viewport,
+        data,
+        theme,
+        indicatorSelector,
+        toolbar,
+        drawing,
+        setData: (next) => data.set(next),
+        appendData: (next) => data.set([...data.peek(), ...next]),
+        updateData: (next) => data.set(next),
+        setTheme: (next) => theme.set(next),
+        zoomToLevel: (level) =>
+            viewport.set({ ...viewport.peek(), zoomLevel: level }),
+        zoomIn: () =>
+            viewport.set({
+                ...viewport.peek(),
+                zoomLevel: viewport.peek().zoomLevel + 1,
+            }),
+        zoomOut: () =>
+            viewport.set({
+                ...viewport.peek(),
+                zoomLevel: viewport.peek().zoomLevel - 1,
+            }),
+        handlePointerEvent: () => false,
+        handleWheelEvent: () => {},
+        handleScrollEvent: () => {},
+        handlePinchZoom: () => {},
+        addIndicator: () => null,
+        removeIndicator: () => false,
+        updateIndicatorParams: () => false,
+        updateRendererConfig: () => {},
+        setDrawingTool: (tool) => drawing.setActiveTool(tool),
+        clearDrawings: () => drawing.clearAll(),
+        removeDrawing: () => {},
+        resizeSubPane: () => false,
+        createSubPane: () => false,
+        clearSubPanes: () => {},
+        updateCustomMarkers: () => {},
+        clearCustomMarkers: () => {},
+        updateSettingsFacade: () => {},
+        updateOptionsFacade: () => {},
+        dispose: () => {
+            disposeCalls += 1
+        },
+        disposeCalls: () => disposeCalls,
+        _setViewport: (vp) => viewport.set(vp),
+        _setData: (next) => data.set(next),
+    }
+}
+
+/** Signal helper used by reactivity bridge tests. */
+export function createTestSignal<T>(initial: T): Signal<T> {
+    return createSignal(initial)
+}

package/src/__tests__/contract.test.ts

@@ -1,132 +1,132 @@
-/**
- * Contract test for @363045841yyt/klinechart.
- *
- * Phase 1D agent's brief: make these pass without weakening assertions,
- * preserving the legacy KMapPlugin.install signature.
- */
-
-import { describe, it, expect, vi, afterEach } from 'vitest'
-import { defineComponent, h, nextTick, ref, shallowRef } from 'vue'
-import { mount } from '@vue/test-utils'
-import * as VueAdapter from '../index'
-import { coreSignalToVueRef } from '../index'
-import { createMockChartController, createTestSignal } from './_mockController'
-
-describe('@363045841yyt/klinechart �?public API surface', () => {
-    it('exports createChart, useChart, useIndicatorSelector, KLineChart, KMapPlugin', () => {
-        expect(typeof VueAdapter.createChart).toBe('function')
-        expect(typeof VueAdapter.useChart).toBe('function')
-        expect(typeof VueAdapter.useIndicatorSelector).toBe('function')
-        expect(VueAdapter.KLineChart).toBeDefined()
-        expect(typeof VueAdapter.KMapPlugin.install).toBe('function')
-    })
-
-    it('KMapPlugin.install is callable with a mock app and registers KLineChart', () => {
-        const registered: Record<string, unknown> = {}
-        const mockApp = {
-            component(name: string, comp: unknown) {
-                registered[name] = comp
-            },
-        } as unknown as Parameters<typeof VueAdapter.KMapPlugin.install>[0]
-        VueAdapter.KMapPlugin.install(mockApp)
-        expect(registered.KLineChart).toBe(VueAdapter.KLineChart)
-    })
-})
-
-describe('@363045841yyt/klinechart �?SSR safety', () => {
-    it('module import does not touch window or document', () => {
-        // Import above ran in node env without jsdom. If it touched window, this
-        // file would not have loaded. Test documents the contract.
-        expect(true).toBe(true)
-    })
-})
-
-describe('@363045841yyt/klinechart �?useChart lifecycle', () => {
-    afterEach(() => {
-        // Reset the injected factory so other tests start clean.
-        VueAdapter.__setControllerFactory(null)
-    })
-
-    it('mounts on first render via template ref', async () => {
-        const mockController = createMockChartController({ data: [] })
-        const factorySpy = vi.fn(() => mockController)
-        VueAdapter.__setControllerFactory(factorySpy)
-
-        const HostComponent = defineComponent({
-            name: 'Host',
-            setup() {
-                const containerRef = ref<HTMLElement | null>(null)
-                const { chart } = VueAdapter.useChart(containerRef, { data: [] })
-                return { containerRef, chart }
-            },
-            render() {
-                return h('div', { ref: 'containerRef' })
-            },
-        })
-
-        const wrapper = mount(HostComponent, { attachTo: document.body })
-        await nextTick()
-
-        expect(factorySpy).toHaveBeenCalledTimes(1)
-        const factoryArg = factorySpy.mock.calls[0]?.[0]
-        expect(factoryArg?.container).toBeInstanceOf(HTMLElement)
-        expect(wrapper.vm.chart).toBe(mockController)
-
-        wrapper.unmount()
-    })
-
-    it('disposes on unmount', async () => {
-        const mockController = createMockChartController({ data: [] })
-        VueAdapter.__setControllerFactory(() => mockController)
-
-        const HostComponent = defineComponent({
-            name: 'Host',
-            setup() {
-                const containerRef = ref<HTMLElement | null>(null)
-                const { chart } = VueAdapter.useChart(containerRef, { data: [] })
-                return { containerRef, chart }
-            },
-            render() {
-                return h('div', { ref: 'containerRef' })
-            },
-        })
-
-        const wrapper = mount(HostComponent, { attachTo: document.body })
-        await nextTick()
-
-        expect(mockController.disposeCalls()).toBe(0)
-        wrapper.unmount()
-        // Allow lifecycle hooks to settle.
-        await nextTick()
-        expect(mockController.disposeCalls()).toBe(1)
-    })
-
-    it('reactivity bridge: signal change updates returned ref', async () => {
-        // Mount a tiny scoped component so coreSignalToVueRef can register
-        // its onScopeDispose cleanup. Without a setup scope the ref is still
-        // wired up correctly, but cleanup would not be automatic.
-        const signal = createTestSignal<number>(1)
-        const bridgedRef = shallowRef<{ value: number } | null>(null)
-
-        const HostComponent = defineComponent({
-            name: 'BridgeHost',
-            setup() {
-                const r = coreSignalToVueRef(signal)
-                bridgedRef.value = r as unknown as { value: number }
-                return () => h('div', String(r.value))
-            },
-        })
-
-        const wrapper = mount(HostComponent, { attachTo: document.body })
-        expect(bridgedRef.value?.value).toBe(1)
-        expect(wrapper.text()).toBe('1')
-
-        signal.set(42)
-        await nextTick()
-
-        expect(bridgedRef.value?.value).toBe(42)
-        expect(wrapper.text()).toBe('42')
-
-        wrapper.unmount()
-    })
-})
+/**
+ * Contract test for @363045841yyt/klinechart.
+ *
+ * Phase 1D agent's brief: make these pass without weakening assertions,
+ * preserving the legacy KMapPlugin.install signature.
+ */
+
+import { describe, it, expect, vi, afterEach } from 'vitest'
+import { defineComponent, h, nextTick, ref, shallowRef } from 'vue'
+import { mount } from '@vue/test-utils'
+import * as VueAdapter from '../index'
+import { coreSignalToVueRef } from '../index'
+import { createMockChartController, createTestSignal } from './_mockController'
+
+describe('@363045841yyt/klinechart �?public API surface', () => {
+    it('exports createChart, useChart, useIndicatorSelector, KLineChart, KMapPlugin', () => {
+        expect(typeof VueAdapter.createChart).toBe('function')
+        expect(typeof VueAdapter.useChart).toBe('function')
+        expect(typeof VueAdapter.useIndicatorSelector).toBe('function')
+        expect(VueAdapter.KLineChart).toBeDefined()
+        expect(typeof VueAdapter.KMapPlugin.install).toBe('function')
+    })
+
+    it('KMapPlugin.install is callable with a mock app and registers KLineChart', () => {
+        const registered: Record<string, unknown> = {}
+        const mockApp = {
+            component(name: string, comp: unknown) {
+                registered[name] = comp
+            },
+        } as unknown as Parameters<typeof VueAdapter.KMapPlugin.install>[0]
+        VueAdapter.KMapPlugin.install(mockApp)
+        expect(registered.KLineChart).toBe(VueAdapter.KLineChart)
+    })
+})
+
+describe('@363045841yyt/klinechart �?SSR safety', () => {
+    it('module import does not touch window or document', () => {
+        // Import above ran in node env without jsdom. If it touched window, this
+        // file would not have loaded. Test documents the contract.
+        expect(true).toBe(true)
+    })
+})
+
+describe('@363045841yyt/klinechart �?useChart lifecycle', () => {
+    afterEach(() => {
+        // Reset the injected factory so other tests start clean.
+        VueAdapter.__setControllerFactory(null)
+    })
+
+    it('mounts on first render via template ref', async () => {
+        const mockController = createMockChartController({ data: [] })
+        const factorySpy = vi.fn(() => mockController)
+        VueAdapter.__setControllerFactory(factorySpy)
+
+        const HostComponent = defineComponent({
+            name: 'Host',
+            setup() {
+                const containerRef = ref<HTMLElement | null>(null)
+                const { chart } = VueAdapter.useChart(containerRef, { data: [] })
+                return { containerRef, chart }
+            },
+            render() {
+                return h('div', { ref: 'containerRef' })
+            },
+        })
+
+        const wrapper = mount(HostComponent, { attachTo: document.body })
+        await nextTick()
+
+        expect(factorySpy).toHaveBeenCalledTimes(1)
+        const factoryArg = factorySpy.mock.calls[0]?.[0]
+        expect(factoryArg?.container).toBeInstanceOf(HTMLElement)
+        expect(wrapper.vm.chart).toBe(mockController)
+
+        wrapper.unmount()
+    })
+
+    it('disposes on unmount', async () => {
+        const mockController = createMockChartController({ data: [] })
+        VueAdapter.__setControllerFactory(() => mockController)
+
+        const HostComponent = defineComponent({
+            name: 'Host',
+            setup() {
+                const containerRef = ref<HTMLElement | null>(null)
+                const { chart } = VueAdapter.useChart(containerRef, { data: [] })
+                return { containerRef, chart }
+            },
+            render() {
+                return h('div', { ref: 'containerRef' })
+            },
+        })
+
+        const wrapper = mount(HostComponent, { attachTo: document.body })
+        await nextTick()
+
+        expect(mockController.disposeCalls()).toBe(0)
+        wrapper.unmount()
+        // Allow lifecycle hooks to settle.
+        await nextTick()
+        expect(mockController.disposeCalls()).toBe(1)
+    })
+
+    it('reactivity bridge: signal change updates returned ref', async () => {
+        // Mount a tiny scoped component so coreSignalToVueRef can register
+        // its onScopeDispose cleanup. Without a setup scope the ref is still
+        // wired up correctly, but cleanup would not be automatic.
+        const signal = createTestSignal<number>(1)
+        const bridgedRef = shallowRef<{ value: number } | null>(null)
+
+        const HostComponent = defineComponent({
+            name: 'BridgeHost',
+            setup() {
+                const r = coreSignalToVueRef(signal)
+                bridgedRef.value = r as unknown as { value: number }
+                return () => h('div', String(r.value))
+            },
+        })
+
+        const wrapper = mount(HostComponent, { attachTo: document.body })
+        expect(bridgedRef.value?.value).toBe(1)
+        expect(wrapper.text()).toBe('1')
+
+        signal.set(42)
+        await nextTick()
+
+        expect(bridgedRef.value?.value).toBe(42)
+        expect(wrapper.text()).toBe('42')
+
+        wrapper.unmount()
+    })
+})