chore: lint fixes

Author: fbosch

scripts/benchmarks/run.mjs

@@ -106,13 +106,6 @@ const runCli = async (args) => {
 	});
 };
 
-const runGit = async (args, cwd) => {
-	await execFileAsync("git", args, {
-		cwd,
-		maxBuffer: 1024 * 1024,
-	});
-};
-
 const formatRow = (result) => {
 	const min = result.min ?? 0;
 	const max = result.max ?? 0;

src/init.ts

@@ -8,10 +8,8 @@ import {
 } from "@clack/prompts";
 import {
 	DEFAULT_CACHE_DIR,
-	DEFAULT_CONFIG,
 	DEFAULT_CONFIG_FILENAME,
 	type DocsCacheConfig,
-	resolveConfigPath,
 	writeConfig,
 } from "./config";
 
@@ -46,16 +44,6 @@ export const initConfig = async (
 	const select = deps.select ?? clackSelect;
 	const text = deps.text ?? clackText;
 	const cwd = options.cwd ?? process.cwd();
-	const defaults = {
-		ref: DEFAULT_CONFIG.defaults?.ref ?? "HEAD",
-		targetMode:
-			DEFAULT_CONFIG.defaults?.targetMode ??
-			(process.platform === "win32" ? "copy" : "symlink"),
-		allowHosts: DEFAULT_CONFIG.defaults?.allowHosts ?? [
-			"github.com",
-			"gitlab.com",
-		],
-	};
 	const defaultConfigPath = path.resolve(cwd, DEFAULT_CONFIG_FILENAME);
 	const packagePath = path.resolve(cwd, "package.json");
 	const existingConfigPaths: string[] = [];

src/remove.ts

@@ -2,7 +2,6 @@ import { access, readFile, rm, writeFile } from "node:fs/promises";
 import path from "node:path";
 import {
 	DEFAULT_CONFIG,
-	DEFAULT_CONFIG_FILENAME,
 	type DocsCacheConfig,
 	resolveConfigPath,
 	validateConfig,
@@ -102,7 +101,10 @@ export const removeSources = async (params: {
 		}
 		const resolved = resolveRepoInput(token);
 		if (resolved.repoUrl && sourcesByRepo.has(resolved.repoUrl)) {
-			idsToRemove.add(sourcesByRepo.get(resolved.repoUrl)!.id);
+			const entry = sourcesByRepo.get(resolved.repoUrl);
+			if (entry) {
+				idsToRemove.add(entry.id);
+			}
 			continue;
 		}
 		if (resolved.inferredId && sourcesById.has(resolved.inferredId)) {

tests/edge-cases.test.js

@@ -4,16 +4,7 @@ import { tmpdir } from "node:os";
 import path from "node:path";
 import { test } from "node:test";
 
-import { loadConfig, runSync } from "../dist/api.mjs";
-
-const exists = async (target) => {
-	try {
-		await import("node:fs/promises").then(({ access }) => access(target));
-		return true;
-	} catch {
-		return false;
-	}
-};
+import { loadConfig } from "../dist/api.mjs";
 
 const writeConfig = async (data) => {
 	const tmpRoot = path.join(
@@ -216,7 +207,7 @@ test("config rejects BOM (Byte Order Mark)", async () => {
 	// UTF-8 BOM - this is a real edge case that can happen with Windows editors
 	await writeFile(
 		configPath,
-		"\uFEFF" + JSON.stringify(config, null, 2),
+		`\uFEFF${JSON.stringify(config, null, 2)}`,
 		"utf8",
 	);