Merge pull request #52 from jamesscottbrown/unused-imports

gka · web-flow · commit f55f704582a6 · 2025-05-23T00:39:31.000+02:00
Remove some unused imports
diff --git a/src/lib/core/Facet.svelte b/src/lib/core/Facet.svelte
@@ -1,8 +1,7 @@
 <script lang="ts">
     import { setContext, type Snippet } from 'svelte';
     import { resolveChannel } from '../helpers/resolve.js';
-    import type { BaseMarkProps, DataRecord, PlotScale, RawValue } from '../types.js';
-    import { identity } from '$lib/helpers';
+    import type { BaseMarkProps, DataRecord, RawValue } from '../types.js';
 
     let {
         fx,
diff --git a/src/lib/core/FacetGrid.svelte b/src/lib/core/FacetGrid.svelte
@@ -4,7 +4,7 @@
     marks for each facet domain value 
 -->
 <script lang="ts">
-    import { getContext, setContext, type Snippet } from 'svelte';
+    import { getContext, type Snippet } from 'svelte';
     import type { PlotContext, GenericMarkOptions, Mark } from '../types.js';
     import { scaleBand } from 'd3-scale';
     import Facet from './Facet.svelte';
diff --git a/src/lib/helpers/facets.ts b/src/lib/helpers/facets.ts
@@ -1,4 +1,4 @@
-import type { GenericMarkOptions, Mark, PlotScale, RawValue } from '$lib/types.js';
+import type { GenericMarkOptions, Mark, RawValue } from '$lib/types.js';
 import { resolveChannel } from './resolve.js';
 
 /**
diff --git a/src/lib/helpers/index.test.ts b/src/lib/helpers/index.test.ts
@@ -1,4 +1,4 @@
-import { coalesce, isObject, pick, omit } from './index';
+import { coalesce, isObject, omit } from './index';
 import { describe, it, expect } from 'vitest';
 
 describe('coalesce', () => {
diff --git a/src/lib/helpers/index.ts b/src/lib/helpers/index.ts
@@ -1,5 +1,4 @@
 import type {
-    BaseMarkProps,
     ChannelAccessor,
     ChannelName,
     DataRecord,
diff --git a/src/lib/helpers/projection.ts b/src/lib/helpers/projection.ts
@@ -1,6 +1,6 @@
 import { geoClipRectangle, geoPath, geoTransform } from 'd3-geo';
 
-import { constant, isObject, isValid } from './index.js';
+import { constant, isObject } from './index.js';
 
 const identity = constant({ stream: (stream) => stream });
 
diff --git a/src/lib/helpers/reduce.test.ts b/src/lib/helpers/reduce.test.ts
@@ -1,6 +1,6 @@
 import { describe, it, expect } from 'vitest';
 import { reduceOutputs, type ReducerName } from './reduce.js';
-import type { ChannelAccessor, ChannelName, RawValue } from '$lib/types.js';
+import type { ChannelAccessor, ChannelName } from '$lib/types.js';
 
 describe('reduceOutputs', () => {
     it('should correctly reduce outputs', () => {
diff --git a/src/lib/helpers/reduce.ts b/src/lib/helpers/reduce.ts
@@ -1,12 +1,11 @@
 import type {
-    ChannelAccessor,
     ChannelName,
     Channels,
     DataRecord,
     DataRow,
     RawValue
 } from '$lib/types.js';
-import { min, max, mode, sum, mean, median, variance, deviation, quantile, range } from 'd3-array';
+import { min, max, mode, sum, mean, median, variance, deviation, quantile } from 'd3-array';
 import { resolveChannel } from './resolve.js';
 import { POSITION_CHANNELS } from './index.js';
 
diff --git a/src/lib/helpers/resolve.ts b/src/lib/helpers/resolve.ts
@@ -3,7 +3,6 @@ import isDataRecord from '$lib/helpers/isDataRecord.js';
 import isRawValue from '$lib/helpers/isRawValue.js';
 import type { MarkStyleProps, PlotState, ScaledDataRecord } from '$lib/types.js';
 import { isValid } from './isValid.js';
-import { pick } from 'es-toolkit';
 
 import type {
     ScaleName,
diff --git a/src/lib/helpers/scales.ts b/src/lib/helpers/scales.ts
@@ -1,4 +1,4 @@
-import { extent, range as d3Range, ascending } from 'd3-array';
+import { extent, ascending } from 'd3-array';
 
 import {
     isColorOrNull,
diff --git a/src/lib/marks/Arrow.svelte b/src/lib/marks/Arrow.svelte
@@ -9,7 +9,6 @@
         BaseMarkProps,
         ConstantAccessor,
         ChannelAccessor,
-        FacetContext,
         RawValue
     } from '../types.js';
     import { resolveChannel, resolveProp, resolveStyles } from '../helpers/resolve.js';
diff --git a/src/lib/marks/AxisY.svelte b/src/lib/marks/AxisY.svelte
@@ -6,11 +6,9 @@
         PlotContext,
         BaseMarkProps,
         RawValue,
-        DataRecord,
         FacetContext,
         DefaultOptions
     } from '../types.js';
-    import getBaseStyles from '$lib/helpers/getBaseStyles.js';
     import autoTimeFormat from '$lib/helpers/autoTimeFormat.js';
     import type { ConstantAccessor } from '$lib/types.js';
     import { autoTicks } from '$lib/helpers/autoTicks.js';
diff --git a/src/lib/marks/CellX.svelte b/src/lib/marks/CellX.svelte
@@ -1,6 +1,6 @@
 <script lang="ts">
     import Cell from './Cell.svelte';
-    import { recordizeX, recordizeY } from '$lib/index.js';
+    import { recordizeY } from '$lib/index.js';
     import type { BaseMarkProps, DataRow, RectMarkProps } from '../types.js';
     import type { ChannelAccessor } from '$lib/types.js';
 
diff --git a/src/lib/marks/ColorLegend.svelte b/src/lib/marks/ColorLegend.svelte
@@ -1,7 +1,7 @@
 <script lang="ts">
     import { getContext } from 'svelte';
     import { Plot, AxisX, Frame } from '$lib/index.js';
-    import { symbol as d3Symbol, symbol } from 'd3-shape';
+    import { symbol as d3Symbol } from 'd3-shape';
     import { range as d3Range, extent } from 'd3-array';
     import { maybeSymbol } from '$lib/helpers/symbols.js';
 
diff --git a/src/lib/marks/DifferenceY.svelte b/src/lib/marks/DifferenceY.svelte
@@ -2,7 +2,6 @@
     import type {
         BaseMarkProps,
         ChannelAccessor,
-        ConstantAccessor,
         CurveName,
         DataRecord,
         PlotContext
diff --git a/src/lib/marks/Dot.svelte b/src/lib/marks/Dot.svelte
@@ -15,7 +15,7 @@
     import { sort } from '$lib/index.js';
     import Mark from '../Mark.svelte';
     import DotCanvas from './helpers/DotCanvas.svelte';
-    import { maybeData, testFilter, isValid } from '$lib/helpers/index.js';
+    import { maybeData, isValid } from '$lib/helpers/index.js';
     import { recordizeXY } from '$lib/transforms/recordize.js';
     import { addEventHandlers } from './helpers/events.js';
 
diff --git a/src/lib/marks/Geo.svelte b/src/lib/marks/Geo.svelte
@@ -4,9 +4,7 @@
         DataRecord,
         PlotContext,
         BaseMarkProps,
-        FacetContext,
-        ConstantAccessor,
-        UsedScales
+        ConstantAccessor
     } from '../types.js';
     import Mark from '../Mark.svelte';
     import { geoPath } from 'd3-geo';
diff --git a/src/lib/marks/GridY.svelte b/src/lib/marks/GridY.svelte
@@ -1,7 +1,7 @@
 <script lang="ts">
     import { getContext } from 'svelte';
     import Mark from '../Mark.svelte';
-    import type { PlotContext, BaseMarkProps, RawValue, DataRecord } from '../types.js';
+    import type { PlotContext, BaseMarkProps, RawValue } from '../types.js';
     import { resolveChannel, resolveStyles } from '../helpers/resolve.js';
     import { autoTicks } from '$lib/helpers/autoTicks.js';
     import { testFilter } from '$lib/helpers/index.js';
diff --git a/src/lib/marks/Rect.svelte b/src/lib/marks/Rect.svelte
@@ -6,16 +6,13 @@
     import Mark from '../Mark.svelte';
     import { getContext } from 'svelte';
     import { intervalX, intervalY } from '$lib/index.js';
-    import { resolveProp, resolveStyles } from '../helpers/resolve.js';
-    import { coalesce, maybeNumber } from '../helpers/index.js';
     import type {
         PlotContext,
         DataRecord,
         BaseMarkProps,
         BaseRectMarkProps,
         ChannelAccessor
     } from '../types.js';
-    import { addEventHandlers } from './helpers/events.js';
     import GroupMultiple from './helpers/GroupMultiple.svelte';
     import RectPath from './helpers/RectPath.svelte';
 
diff --git a/src/lib/marks/helpers/BaseAxisY.svelte b/src/lib/marks/helpers/BaseAxisY.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-    import { getContext, tick, untrack } from 'svelte';
+    import { getContext, untrack } from 'svelte';
     import { randomId, testFilter } from '$lib/helpers/index.js';
     import { resolveProp, resolveStyles } from '$lib/helpers/resolve.js';
     import { max } from 'd3-array';
diff --git a/src/lib/marks/helpers/DotCanvas.svelte b/src/lib/marks/helpers/DotCanvas.svelte
@@ -6,7 +6,6 @@
         ScaledDataRecord,
         PlotContext
     } from '$lib/types.js';
-    import { CSS_VAR } from '$lib/constants.js';
     import { resolveProp } from '$lib/helpers/resolve.js';
     import { maybeSymbol } from '$lib/helpers/symbols.js';
     import { symbol as d3Symbol } from 'd3-shape';
diff --git a/src/lib/marks/helpers/GeoCanvas.svelte b/src/lib/marks/helpers/GeoCanvas.svelte
@@ -8,7 +8,7 @@
     } from '$lib/types.js';
     import { CSS_VAR } from '$lib/constants.js';
     import { resolveProp, resolveScaledStyleProps } from '$lib/helpers/resolve.js';
-    import { getContext, untrack } from 'svelte';
+    import { getContext } from 'svelte';
     import { type GeoPath } from 'd3-geo';
     import CanvasLayer from './CanvasLayer.svelte';
     import type { Attachment } from 'svelte/attachments';
diff --git a/src/lib/marks/helpers/events.ts b/src/lib/marks/helpers/events.ts
@@ -1,6 +1,6 @@
-import type { BaseMarkProps, DataRecord, PlotScale, PlotScales, PlotState } from '$lib/types.js';
+import type { BaseMarkProps, DataRecord, PlotScale, PlotState } from '$lib/types.js';
 import type { MouseEventHandler } from 'svelte/elements';
-import { invert, pick } from 'es-toolkit';
+import { pick } from 'es-toolkit';
 import { RAW_VALUE } from '$lib/transforms/recordize.js';
 import { INDEX } from '$lib/constants.js';
 
diff --git a/src/lib/transforms/bin.ts b/src/lib/transforms/bin.ts
@@ -5,13 +5,12 @@ import { maybeInterval } from '$lib/helpers/autoTicks.js';
 import {
     bin as d3Bin,
     extent,
-    groups as d3Groups,
     thresholdFreedmanDiaconis,
     thresholdScott,
     thresholdSturges,
     type ThresholdCountGenerator
 } from 'd3-array';
-import { Reducer, reduceOutputs, type ReducerName } from '$lib/helpers/reduce.js';
+import { reduceOutputs, type ReducerName } from '$lib/helpers/reduce.js';
 import { groupFacetsAndZ } from '$lib/helpers/group.js';
 import { isDate } from '$lib/helpers/typeChecks.js';
 
diff --git a/src/lib/transforms/interval.test.ts b/src/lib/transforms/interval.test.ts
@@ -1,4 +1,4 @@
-import { describe, it, expect, vi } from 'vitest';
+import { describe, it, expect } from 'vitest';
 import { intervalX, intervalY } from './interval.js';
 
 describe('intervalX', () => {
diff --git a/src/lib/transforms/rename.test.ts b/src/lib/transforms/rename.test.ts
@@ -1,8 +1,6 @@
 import { describe, it, expect } from 'vitest';
-import { renameChannels, renameChannelsAndData, replaceChannels } from './rename.js';
+import { renameChannels } from './rename.js';
 import type { DataRecord } from '$lib/types.js';
-import { csvParse } from 'd3-dsv';
-import { readFileSync } from 'fs';
 
 const inputData: DataRecord[] = [
     { year: 2000, facet: 'A', value: 10 },
diff --git a/src/lib/transforms/shift.ts b/src/lib/transforms/shift.ts
@@ -1,6 +1,6 @@
 import { resolveChannel } from '$lib/helpers/resolve.js';
 import { maybeTimeInterval } from '$lib/helpers/time.js';
-import type { ChannelName, DataRecord, TransformArg } from '$lib/types.js';
+import type { DataRecord, TransformArg } from '$lib/types.js';
 
 type RequireAtLeastOne<T, Keys extends keyof T = keyof T> = Pick<T, Exclude<keyof T, Keys>> &
     {
diff --git a/src/lib/transforms/window.ts b/src/lib/transforms/window.ts
@@ -1,6 +1,6 @@
 import { maybeInterval } from '$lib/helpers/autoTicks.js';
 import { isValid } from '$lib/helpers/index.js';
-import { Reducer, mayberReducer, type ReducerName } from '$lib/helpers/reduce.js';
+import { mayberReducer, type ReducerName } from '$lib/helpers/reduce.js';
 import { resolveChannel } from '$lib/helpers/resolve.js';
 import type { DataRecord, ScaledChannelName, TransformArg } from '$lib/types.js';
 import { groups as d3Groups } from 'd3-array';
diff --git a/src/routes/marks/bar/BarPlot.svelte b/src/routes/marks/bar/BarPlot.svelte
@@ -1,6 +1,5 @@
 <script lang="ts">
     import { Plot, BarY, RuleY } from 'svelteplot';
-    import { getContext } from 'svelte';
 
     import { page } from '$app/stores';
     let { alphabet } = $derived($page.data.data);
diff --git a/src/routes/marks/bar/StackedBarPlot.svelte b/src/routes/marks/bar/StackedBarPlot.svelte
@@ -1,6 +1,5 @@
 <script lang="ts">
     import { Plot, BarX } from 'svelteplot';
-    import { getContext } from 'svelte';
     import { rollups } from 'd3-array';
 
     import { page } from '$app/stores';
diff --git a/src/routes/marks/line/CO2Decades.svelte b/src/routes/marks/line/CO2Decades.svelte
@@ -1,6 +1,6 @@
 <script lang="ts">
     import { Plot, Line, Pointer, Text, RectY, RuleY, Dot, selectLast } from '$lib/index.js';
-    import { groups, range } from 'd3-array';
+    import { range } from 'd3-array';
     import { page } from '$app/stores';
 
     let { co2 } = $derived($page.data.data);
diff --git a/src/routes/marks/rule/__+page.svelte b/src/routes/marks/rule/__+page.svelte
@@ -1,16 +1,12 @@
 <script lang="ts">
-    import { Plot, Frame, Line, RuleX, RuleY } from '$lib/index.js';
+    import { Plot, Line, RuleX, RuleY } from '$lib/index.js';
     import type { Datasets } from '$lib/types.js';
-    import { range } from 'd3-array';
     import { getContext } from 'svelte';
     import SineRules from './SineRules.svelte';
     import BarcodeExample from './BarcodeExample.svelte';
     import CandlestickExample from './CandlestickExample.svelte';
-    import { randomNormal, randomLcg } from 'd3-random';
     import Code from '../../Code.svelte';
 
-    const seededNormal = randomNormal.source(randomLcg(0.1234));
-
     const { aapl } = getContext<Datasets>('data');
 
     type AAPL = (typeof aapl)[0];
diff --git a/src/routes/tests/marks/ForceExample.svelte b/src/routes/tests/marks/ForceExample.svelte
@@ -1,7 +1,7 @@
 <script lang="ts">
-    import { Plot, Arrow, Dot, Text, LineY, RuleY } from '$lib/index.js';
+    import { Plot, Arrow, Dot, LineY, RuleY } from '$lib/index.js';
     import { json } from 'd3-fetch';
-    import { range } from 'd3-array';
+    // import { range } from 'd3-array';
     import { noise } from './noise.js';
     import { forceSimulation, forceLink, forceManyBody, forceCenter } from 'd3-force';
 
diff --git a/src/tests/line.test.ts b/src/tests/line.test.ts
@@ -1,4 +1,4 @@
-import { describe, it, expect, test } from 'vitest';
+import { describe, it, expect } from 'vitest';
 import { render } from '@testing-library/svelte';
 import LineTest from './line.test.svelte';
 
diff --git a/src/tests/plot.test.ts b/src/tests/plot.test.ts
@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest';
-import { render, screen } from '@testing-library/svelte';
+import { render } from '@testing-library/svelte';
 import PlotTest from './plot.test.svelte';
 import ResizeObserver from 'resize-observer-polyfill';
 

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import type { GenericMarkOptions, Mark, PlotScale, RawValue } from '$lib/types.js';`
	`1`	`+import type { GenericMarkOptions, Mark, RawValue } from '$lib/types.js';`
`2`	`2`	`import { resolveChannel } from './resolve.js';`
`3`	`3`
`4`	`4`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import { coalesce, isObject, pick, omit } from './index';`
	`1`	`+import { coalesce, isObject, omit } from './index';`
`2`	`2`	`import { describe, it, expect } from 'vitest';`
`3`	`3`
`4`	`4`	`describe('coalesce', () => {`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,4 @@`
`1`	`1`	`import type {`
`2`		`- BaseMarkProps,`
`3`	`2`	`ChannelAccessor,`
`4`	`3`	`ChannelName,`
`5`	`4`	`DataRecord,`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import { extent, range as d3Range, ascending } from 'd3-array';`
	`1`	`+import { extent, ascending } from 'd3-array';`
`2`	`2`
`3`	`3`	`import {`
`4`	`4`	`isColorOrNull,`