Remove unused imports and unused vars

Tone/component/analysis/DCMeter.test.ts

@@ -1,6 +1,6 @@
 import { expect } from "chai";
 
-import { BasicTests, warns } from "../../../test/helper/Basic.js";
+import { BasicTests } from "../../../test/helper/Basic.js";
 import { PassAudio } from "../../../test/helper/PassAudio.js";
 import { Signal } from "../../signal/Signal.js";
 import { DCMeter } from "./DCMeter.js";

Tone/component/analysis/Meter.test.ts

@@ -2,10 +2,8 @@ import { expect } from "chai";
 
 import { BasicTests, warns } from "../../../test/helper/Basic.js";
 import { PassAudio } from "../../../test/helper/PassAudio.js";
-import { Signal } from "../../signal/Signal.js";
 import { Oscillator } from "../../source/oscillator/Oscillator.js";
 import { Merge } from "../channel/Merge.js";
-import { Panner } from "../channel/Panner.js";
 import { Meter } from "./Meter.js";
 
 describe("Meter", () => {

Tone/component/envelope/AmplitudeEnvelope.test.ts

@@ -2,7 +2,6 @@ import { expect } from "chai";
 
 import { BasicTests } from "../../../test/helper/Basic.js";
 import { CompareToFile } from "../../../test/helper/CompareToFile.js";
-import { connectFrom, connectTo } from "../../../test/helper/Connect.js";
 import { Offline } from "../../../test/helper/Offline.js";
 import { Signal } from "../../signal/Signal.js";
 import { Oscillator } from "../../source/oscillator/Oscillator.js";

Tone/component/filter/EQ3.test.ts

@@ -1,7 +1,6 @@
 import { expect } from "chai";
 
 import { BasicTests } from "../../../test/helper/Basic.js";
-import { connectFrom } from "../../../test/helper/Connect.js";
 import { PassAudio } from "../../../test/helper/PassAudio.js";
 import { EQ3 } from "./EQ3.js";
 

Tone/core/clock/Ticker.ts

@@ -105,6 +105,7 @@ export class Ticker {
 		if (this._type === "worker") {
 			try {
 				this._createWorker();
+				// eslint-disable-next-line unused-imports/no-unused-vars
 			} catch (e) {
 				// workers not supported, fallback to timeout
 				this._type = "timeout";

Tone/core/clock/TransportRepeatEvent.ts

@@ -86,7 +86,7 @@ export class TransportRepeatEvent extends TransportEvent {
 	 */
 	invoke(time: Seconds): void {
 		// create more events if necessary
-		this._createEvents(time);
+		this._createEvents();
 		// call the super class
 		super.invoke(time);
 	}
@@ -107,9 +107,7 @@ export class TransportRepeatEvent extends TransportEvent {
 	/**
 	 * Push more events onto the timeline to keep up with the position of the timeline
 	 */
-	private _createEvents(time: Seconds): void {
-		// schedule the next event
-		// const ticks = this.transport.getTicksAtTime(time);
+	private _createEvents(): void {
 		// if the next tick is within the bounds set by "duration"
 		if (
 			LT(this._nextTick + this._interval, this.floatTime + this.duration)

Tone/core/context/Param.test.ts

@@ -1,18 +1,11 @@
 import { expect } from "chai";
 
 import { BasicTests, testAudioContext } from "../../../test/helper/Basic.js";
-import { Compare, Plot } from "../../../test/helper/compare/index.js";
+import { Plot } from "../../../test/helper/compare/index.js";
 import { atTime, Offline } from "../../../test/helper/Offline.js";
 import { Signal } from "../../signal/Signal.js";
 import { getContext } from "../Global.js";
 import {
-	BPM,
-	Decibels,
-	Frequency,
-	Positive,
-	Seconds,
-	Time,
-	Unit,
 	UnitName,
 } from "../type/Units.js";
 import { Param } from "./Param.js";

Tone/core/context/ToneWithContext.ts

@@ -1,4 +1,3 @@
-import type { TransportClass } from "../clock/Transport.js";
 import { getContext } from "../Global.js";
 import { Tone } from "../Tone.js";
 import { FrequencyClass } from "../type/Frequency.js";

Tone/core/util/Draw.test.ts

@@ -3,13 +3,24 @@ import { expect } from "chai";
 import { DrawClass } from "./Draw.js";
 
 describe("Draw", () => {
+	const originalRAF = window.requestAnimationFrame;
+	before(async () => {
+		window.requestAnimationFrame = (callback) => {
+			return setTimeout(callback, 10);
+		};
+	});
+
+	after(() => {
+		window.requestAnimationFrame = originalRAF;
+	});
+
 	it("can schedule a callback at a AudioContext time", (done) => {
 		const draw = new DrawClass();
 		const scheduledTime = draw.now() + 0.2;
 		draw.schedule(() => {
-			done();
 			expect(draw.context.currentTime).to.be.closeTo(scheduledTime, 0.05);
 			draw.dispose();
+			done();
 		}, scheduledTime);
 	});
 

Tone/event/PatternGenerator.ts

@@ -18,7 +18,7 @@ export type PatternName =
 /**
  * Start at the first value and go up to the last
  */
-function* upPatternGen<T>(numValues: number): IterableIterator<number> {
+function* upPatternGen(numValues: number): IterableIterator<number> {
 	let index = 0;
 	while (index < numValues) {
 		index = clamp(index, 0, numValues - 1);
@@ -30,7 +30,7 @@ function* upPatternGen<T>(numValues: number): IterableIterator<number> {
 /**
  * Start at the last value and go down to 0
  */
-function* downPatternGen<T>(numValues: number): IterableIterator<number> {
+function* downPatternGen(numValues: number): IterableIterator<number> {
 	let index = numValues - 1;
 	while (index >= 0) {
 		index = clamp(index, 0, numValues - 1);
@@ -42,7 +42,7 @@ function* downPatternGen<T>(numValues: number): IterableIterator<number> {
 /**
  * Infinitely yield the generator
  */
-function* infiniteGen<T>(
+function* infiniteGen(
 	numValues: number,
 	gen: typeof upPatternGen
 ): IterableIterator<number> {
@@ -54,7 +54,7 @@ function* infiniteGen<T>(
 /**
  * Alternate between two generators
  */
-function* alternatingGenerator<T>(
+function* alternatingGenerator(
 	numValues: number,
 	directionUp: boolean
 ): IterableIterator<number> {
@@ -79,7 +79,7 @@ function* alternatingGenerator<T>(
 /**
  * Starting from the bottom move up 2, down 1
  */
-function* jumpUp<T>(numValues: number): IterableIterator<number> {
+function* jumpUp(numValues: number): IterableIterator<number> {
 	let index = 0;
 	let stepIndex = 0;
 	while (index < numValues) {
@@ -93,7 +93,7 @@ function* jumpUp<T>(numValues: number): IterableIterator<number> {
 /**
  * Starting from the top move down 2, up 1
  */
-function* jumpDown<T>(numValues: number): IterableIterator<number> {
+function* jumpDown(numValues: number): IterableIterator<number> {
 	let index = numValues - 1;
 	let stepIndex = 0;
 	while (index >= 0) {
@@ -107,7 +107,7 @@ function* jumpDown<T>(numValues: number): IterableIterator<number> {
 /**
  * Choose a random index each time
  */
-function* randomGen<T>(numValues: number): IterableIterator<number> {
+function* randomGen(numValues: number): IterableIterator<number> {
 	while (true) {
 		const randomIndex = Math.floor(Math.random() * numValues);
 		yield randomIndex;
@@ -117,7 +117,7 @@ function* randomGen<T>(numValues: number): IterableIterator<number> {
 /**
  * Randomly go through all of the values once before choosing a new random order
  */
-function* randomOnce<T>(numValues: number): IterableIterator<number> {
+function* randomOnce(numValues: number): IterableIterator<number> {
 	// create an array of indices
 	const copy: number[] = [];
 	for (let i = 0; i < numValues; i++) {
@@ -134,7 +134,7 @@ function* randomOnce<T>(numValues: number): IterableIterator<number> {
 /**
  * Randomly choose to walk up or down 1 index
  */
-function* randomWalk<T>(numValues: number): IterableIterator<number> {
+function* randomWalk(numValues: number): IterableIterator<number> {
 	// randomly choose a starting index
 	let index = Math.floor(Math.random() * numValues);
 	while (true) {
@@ -157,12 +157,10 @@ function* randomWalk<T>(numValues: number): IterableIterator<number> {
  * according to the passed in pattern that can be used as indexes into an array of size numValues.
  * @param numValues The size of the array to emit indexes for
  * @param pattern The name of the pattern use when iterating over
- * @param index Where to start in the offset of the values array
  */
 export function* PatternGenerator(
 	numValues: number,
-	pattern: PatternName = "up",
-	index = 0
+	pattern: PatternName = "up"
 ): Iterator<number> {
 	// safeguards
 	assert(numValues >= 1, "The number of values must be at least one");

Tone/index.ts

@@ -1,6 +1,5 @@
-export { getContext, setContext } from "./core/Global.js";
-import { Context } from "./core/context/Context.js";
 export * from "./classes.js";
+export { getContext, setContext } from "./core/Global.js";
 export * from "./version.js";
 import { ToneAudioBuffer } from "./core/context/ToneAudioBuffer.js";
 import { getContext } from "./core/Global.js";

Tone/signal/Abs.test.ts

@@ -1,5 +1,4 @@
 import { BasicTests } from "../../test/helper/Basic.js";
-import { connectFrom, connectTo } from "../../test/helper/Connect.js";
 import { ConstantOutput } from "../../test/helper/ConstantOutput.js";
 import { Abs } from "./Abs.js";
 import { Signal } from "./Signal.js";

Tone/signal/AudioToGain.test.ts

@@ -1,7 +1,6 @@
 import { expect } from "chai";
 
 import { BasicTests } from "../../test/helper/Basic.js";
-import { connectFrom, connectTo } from "../../test/helper/Connect.js";
 import { ConstantOutput } from "../../test/helper/ConstantOutput.js";
 import { Offline } from "../../test/helper/Offline.js";
 import { Oscillator } from "../source/oscillator/Oscillator.js";

Tone/signal/GainToAudio.test.ts

@@ -1,5 +1,4 @@
 import { BasicTests } from "../../test/helper/Basic.js";
-import { connectFrom, connectTo } from "../../test/helper/Connect.js";
 import { ConstantOutput } from "../../test/helper/ConstantOutput.js";
 import { GainToAudio } from "./GainToAudio.js";
 import { Signal } from "./Signal.js";

Tone/signal/Signal.test.ts

@@ -1,11 +1,9 @@
 import { expect } from "chai";
 
 import { BasicTests } from "../../test/helper/Basic.js";
-import { connectFrom, connectTo } from "../../test/helper/Connect.js";
 import { ConstantOutput } from "../../test/helper/ConstantOutput.js";
 import { Offline } from "../../test/helper/Offline.js";
 import { Gain } from "../core/context/Gain.js";
-import { Decibels, Frequency, Time } from "../core/type/Units.js";
 import { Signal } from "./Signal.js";
 
 describe("Signal", () => {

Tone/signal/SyncedSignal.ts

@@ -1,4 +1,5 @@
-import type { TransportClass } from "../core/clock/Transport.js";
+import "../core/clock/Transport.js";
+
 import { OutputNode } from "../core/context/ToneAudioNode.js";
 import { TransportTimeClass } from "../core/type/TransportTime.js";
 import {

eslint.config.mjs

@@ -1,10 +1,11 @@
 // @ts-check
-import tseslint from "typescript-eslint";
 import stylisticJs from "@stylistic/eslint-plugin-js";
 import stylisticTs from "@stylistic/eslint-plugin-ts";
-import jsdoc from "eslint-plugin-jsdoc";
 import html from "eslint-plugin-html";
+import jsdoc from "eslint-plugin-jsdoc";
 import simpleImportSort from "eslint-plugin-simple-import-sort";
+import unusedImports from "eslint-plugin-unused-imports";
+import tseslint from "typescript-eslint";
 
 /** @type {import("typescript-eslint").ConfigWithExtends} */
 const customConfig = {
@@ -15,6 +16,7 @@ const customConfig = {
 		jsdoc,
 		html,
 		"simple-import-sort": simpleImportSort,
+		"unused-imports": unusedImports,
 	},
 	rules: {
 		"@typescript-eslint/array-type": "off",
@@ -86,6 +88,16 @@ const customConfig = {
 				},
 			},
 		],
+		"unused-imports/no-unused-imports": "error",
+		"unused-imports/no-unused-vars": [
+			"warn",
+			{
+				vars: "all",
+				varsIgnorePattern: "^_",
+				args: "after-used",
+				argsIgnorePattern: "^_",
+			},
+		],
 	},
 };
 
@@ -96,6 +108,7 @@ export default tseslint.config(
 		files: ["**/*.test.ts", "./test/**/*.ts"],
 		rules: {
 			"@typescript-eslint/no-unused-expressions": "off",
+			"unused-imports/no-unused-vars": "off",
 		},
 	},
 	{

package.json

@@ -75,6 +75,7 @@
 		"eslint-plugin-html": "^8.1.2",
 		"eslint-plugin-jsdoc": "^50.6.11",
 		"eslint-plugin-simple-import-sort": "^12.1.1",
+		"eslint-plugin-unused-imports": "^4.1.4",
 		"fft-windowing": "^0.1.4",
 		"fourier-transform": "^1.1.2",
 		"fs-fixture": "^2.7.1",

test/helper/Connect.ts

@@ -1,5 +1,4 @@
 import { Gain } from "../../Tone/core/context/Gain.js";
-import { ToneAudioNode } from "../../Tone/core/context/ToneAudioNode.js";
 
 export function connectFrom(): Gain {
 	return new Gain();

test/helper/MonophonicTests.ts

@@ -1,6 +1,5 @@
 import { expect } from "chai";
 
-import { Monophonic } from "../../Tone/instrument/Monophonic.js";
 import { Offline } from "./Offline.js";
 
 export function MonophonicTest(Constr, note, constrArg?): void {

test/helper/SourceTests.ts

@@ -1,7 +1,7 @@
 // import APITest from "helper/APITest";
 import { expect } from "chai";
 
-import { connectFrom, connectTo } from "./Connect.js";
+import { connectTo } from "./Connect.js";
 import { Offline } from "./Offline.js";
 import { OutputAudio } from "./OutputAudio.js";
 

test/web-test-runner.config.js

@@ -2,7 +2,6 @@ import { resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 
 import rollupCommonjs from "@rollup/plugin-commonjs";
-import { esbuildPlugin } from "@web/dev-server-esbuild";
 import { fromRollup } from "@web/dev-server-rollup";
 import { puppeteerLauncher } from "@web/test-runner-puppeteer";