chore: enable isolatedDeclarations

Author: johnsoncodehk

src/computed.ts

@@ -19,7 +19,7 @@ export class Computed<T = any> implements IComputed {
 	deps: Link | undefined = undefined;
 	depsTail: Link | undefined = undefined;
 	trackId = 0;
-	dirtyLevel = DirtyLevels.Dirty;
+	dirtyLevel: DirtyLevels = DirtyLevels.Dirty;
 	canPropagate = false;
 
 	constructor(
@@ -45,7 +45,7 @@ export class Computed<T = any> implements IComputed {
 		return this.cachedValue!;
 	}
 
-	update() {
+	update(): void {
 		const prevSub = Subscriber.startTrack(this);
 		const oldValue = this.cachedValue;
 		let newValue: T;

src/effect.ts

@@ -1,7 +1,7 @@
 import { activeEffectScope } from './effectScope.js';
 import { Dependency, DirtyLevels, IEffect, Link, Subscriber, System } from './system.js';
 
-export function effect(fn: () => void) {
+export function effect(fn: () => void): Effect<void> {
 	const e = new Effect(fn);
 	e.run();
 	return e;
@@ -18,7 +18,7 @@ export class Effect<T = any> implements IEffect {
 	deps: Link | undefined = undefined;
 	depsTail: Link | undefined = undefined;
 	trackId = 0;
-	dirtyLevel = DirtyLevels.Dirty;
+	dirtyLevel: DirtyLevels = DirtyLevels.Dirty;
 	canPropagate = false;
 
 	constructor(
@@ -37,7 +37,7 @@ export class Effect<T = any> implements IEffect {
 		}
 	}
 
-	notify() {
+	notify(): void {
 		let dirtyLevel = this.dirtyLevel;
 		if (dirtyLevel > DirtyLevels.None) {
 			if (dirtyLevel === DirtyLevels.MaybeDirty) {
@@ -53,7 +53,7 @@ export class Effect<T = any> implements IEffect {
 		}
 	}
 
-	run() {
+	run(): T {
 		const prevSub = Subscriber.startTrack(this);
 		try {
 			return this.fn();
@@ -62,7 +62,7 @@ export class Effect<T = any> implements IEffect {
 		}
 	}
 
-	stop() {
+	stop(): void {
 		if (this.deps !== undefined) {
 			Subscriber.clearTrack(this.deps);
 			this.deps = undefined;

src/effectScope.ts

@@ -2,26 +2,26 @@ import { DirtyLevels, Link, Subscriber, System } from './system.js';
 
 export let activeEffectScope: EffectScope | undefined = undefined;
 
-export function effectScope() {
+export function effectScope(): EffectScope {
 	return new EffectScope();
 }
 
 export class EffectScope implements Subscriber {
 	// Subscriber
 	deps: Link | undefined = undefined;
 	depsTail: Link | undefined = undefined;
-	trackId = -(++System.lastTrackId);
-	dirtyLevel = DirtyLevels.None;
+	trackId: number = -(++System.lastTrackId);
+	dirtyLevel: DirtyLevels = DirtyLevels.None;
 	canPropagate = false;
 
-	notify() {
+	notify(): void {
 		if (this.dirtyLevel !== DirtyLevels.None) {
 			this.dirtyLevel = DirtyLevels.None;
 			Subscriber.runInnerEffects(this.deps);
 		}
 	}
 
-	run<T>(fn: () => T) {
+	run<T>(fn: () => T): T {
 		const prevSub = activeEffectScope;
 		activeEffectScope = this;
 		this.trackId = Math.abs(this.trackId);
@@ -33,7 +33,7 @@ export class EffectScope implements Subscriber {
 		}
 	}
 
-	stop() {
+	stop(): void {
 		if (this.deps !== undefined) {
 			Subscriber.clearTrack(this.deps);
 			this.deps = undefined;

src/signal.ts

@@ -20,7 +20,7 @@ export class Signal<T = any> implements Dependency {
 		public currentValue: T
 	) { }
 
-	get() {
+	get(): NonNullable<T> {
 		const activeTrackId = System.activeTrackId;
 		if (activeTrackId !== 0) {
 			const subsTail = this.subsTail;
@@ -31,7 +31,7 @@ export class Signal<T = any> implements Dependency {
 		return this.currentValue!;
 	}
 
-	set(value: T) {
+	set(value: T): void {
 		if (this.currentValue !== (this.currentValue = value)) {
 			const subs = this.subs;
 			if (subs !== undefined) {

src/system.ts

@@ -48,11 +48,11 @@ export namespace System {
 	export let queuedEffectsTail: IEffect | undefined = undefined;
 }
 
-export function startBatch() {
+export function startBatch(): void {
 	System.batchDepth++;
 }
 
-export function endBatch() {
+export function endBatch(): void {
 	System.batchDepth--;
 	if (System.batchDepth === 0) {
 		while (System.queuedEffects !== undefined) {
@@ -73,7 +73,7 @@ export function endBatch() {
 export namespace Link {
 	let pool: Link | undefined = undefined;
 
-	export function get(dep: Dependency, sub: Subscriber, nextDep: Link | undefined) {
+	export function get(dep: Dependency, sub: Subscriber, nextDep: Link | undefined): Link {
 		if (pool !== undefined) {
 			const newLink = pool;
 			pool = newLink.nextDep;
@@ -94,7 +94,7 @@ export namespace Link {
 		}
 	}
 
-	export function release(link: Link) {
+	export function release(link: Link): void {
 		const dep = link.dep;
 		const nextSub = link.nextSub;
 		const prevSub = link.prevSub;
@@ -131,11 +131,11 @@ export namespace Dependency {
 	/**
 	 * @deprecated Use `startTrack` instead.
 	 */
-	export function linkSubscriber(dep: Dependency, sub: Subscriber) {
+	export function linkSubscriber(dep: Dependency, sub: Subscriber): void {
 		return link(dep, sub);
 	}
 
-	export function link(dep: Dependency, sub: Subscriber) {
+	export function link(dep: Dependency, sub: Subscriber): void {
 		const depsTail = sub.depsTail;
 		const old = depsTail !== undefined
 			? depsTail.nextDep
@@ -166,7 +166,7 @@ export namespace Dependency {
 		}
 	}
 
-	export function propagate(subs: Link) {
+	export function propagate(subs: Link): void {
 		let link: Link | undefined = subs;
 		let dep = subs.dep;
 		let dirtyLevel = DirtyLevels.Dirty;
@@ -282,7 +282,7 @@ export namespace Subscriber {
 
 	const system = System;
 
-	export function runInnerEffects(link: Link | undefined) {
+	export function runInnerEffects(link: Link | undefined): void {
 		while (link !== undefined) {
 			const dep = link.dep;
 			if ('notify' in dep) {
@@ -292,7 +292,7 @@ export namespace Subscriber {
 		}
 	}
 
-	export function resolveMaybeDirty(sub: IComputed | IEffect, depth = 0) {
+	export function resolveMaybeDirty(sub: IComputed | IEffect, depth = 0): void {
 		let link = sub.deps;
 
 		while (link !== undefined) {
@@ -327,7 +327,7 @@ export namespace Subscriber {
 		}
 	}
 
-	export function resolveMaybeDirtyNonRecursive(sub: IComputed | IEffect) {
+	export function resolveMaybeDirtyNonRecursive(sub: IComputed | IEffect): void {
 		let link = sub.deps;
 		let remaining = 0;
 
@@ -402,18 +402,18 @@ export namespace Subscriber {
 	/**
 	 * @deprecated Use `startTrack` instead.
 	 */
-	export function startTrackDependencies(sub: Subscriber) {
+	export function startTrackDependencies(sub: Subscriber): Subscriber | undefined {
 		return startTrack(sub);
 	}
 
 	/**
 	 * @deprecated Use `endTrack` instead.
 	 */
-	export function endTrackDependencies(sub: Subscriber, prevSub: Subscriber | undefined) {
+	export function endTrackDependencies(sub: Subscriber, prevSub: Subscriber | undefined): void {
 		return endTrack(sub, prevSub);
 	}
 
-	export function startTrack(sub: Subscriber) {
+	export function startTrack(sub: Subscriber): Subscriber | undefined {
 		const newTrackId = system.lastTrackId + 1;
 		const prevSub = system.activeSub;
 
@@ -428,7 +428,7 @@ export namespace Subscriber {
 		return prevSub;
 	}
 
-	export function endTrack(sub: Subscriber, prevSub: Subscriber | undefined) {
+	export function endTrack(sub: Subscriber, prevSub: Subscriber | undefined): void {
 		if (prevSub !== undefined) {
 			system.activeSub = prevSub;
 			system.activeTrackId = prevSub.trackId;
@@ -450,7 +450,7 @@ export namespace Subscriber {
 		sub.trackId = -sub.trackId;
 	}
 
-	export function clearTrack(link: Link) {
+	export function clearTrack(link: Link): void {
 		do {
 			const dep = link.dep;
 			const nextDep = link.nextDep;

src/unstable/computedSet.ts

@@ -1,6 +1,6 @@
 import { computed, ISignal } from '../index.js';
 
-export function computedSet<T>(source: ISignal<Set<T>>) {
+export function computedSet<T>(source: ISignal<Set<T>>): ISignal<Set<T>> {
 	return computed<Set<T>>(
 		(oldValue) => {
 			const newValue = source.get();

tsconfig.json

@@ -8,6 +8,7 @@
 		"noUnusedLocals": true,
 		"noUnusedParameters": true,
 		"skipLibCheck": true,
+		"isolatedDeclarations": true,
 		"rootDir": "src",
 		"outDir": "cjs",
 	},