Lint fixes.

Author: kitschpatrol

.github/workflows/ci.yml

@@ -4,6 +4,7 @@ on:
   push:
     branches: [main]
   pull_request: {}
+  workflow_dispatch: {}
 
 permissions:
   contents: read

cspell.config.ts

@@ -2,5 +2,5 @@ import { cspellConfig } from '@kitschpatrol/cspell-config'
 
 export default cspellConfig({
 	ignorePaths: ['test/fixtures/anki-data-folder/**/*'],
-	words: ['afmt', 'bafmt', 'bfont', 'bqfmt', 'flds', 'odid', 'odue', 'pycache', 'sched', 'tmpls'],
+	words: ['afmt', 'bafmt', 'bfont', 'bqfmt', 'flds', 'odid', 'odue', 'pkill', 'sched', 'tmpls'],
 })

eslint.config.ts

@@ -5,6 +5,12 @@ export default eslintConfig(
 		ignores: ['test/fixtures/anki-data-folder/**/*'],
 		ts: {
 			overrides: {
+				'depend/ban-dependencies': [
+					'error',
+					{
+						allowed: ['execa'],
+					},
+				],
 				'ts/naming-convention': [
 					'error',
 					{

knip.config.ts

@@ -1,5 +1,6 @@
 import { knipConfig } from '@kitschpatrol/knip-config'
 
 export default knipConfig({
+	ignoreBinaries: ['anki', 'awk', 'launchctl', 'open', 'osascript', 'pkill', 'taskkill'],
 	ignoreDependencies: ['@kitschpatrol/typescript-config', 'playwright'],
 })

test/utilities/anki-process.ts

@@ -16,7 +16,17 @@ const platform =
  */
 function findAnkiWindows(): string {
 	const candidates = [
-		path.join(os.homedir(), 'scoop', 'apps', 'anki', 'current', 'programfiles', '.venv', 'Scripts', 'anki.exe'),
+		path.join(
+			os.homedir(),
+			'scoop',
+			'apps',
+			'anki',
+			'current',
+			'programfiles',
+			'.venv',
+			'Scripts',
+			'anki.exe',
+		),
 		path.join('C:', 'Program Files', 'Anki', 'anki.exe'),
 	]
 
@@ -157,42 +167,50 @@ export async function closeAnki(): Promise<void> {
 					ankiPid = undefined
 				}
 
-				await execa('pkill', ['-x', 'anki']).catch(() => {})
+				await execa('pkill', ['-x', 'anki']).catch(() => {
+					// Ignore
+				})
 				break
 			}
 
 			case 'mac': {
-				if (ankiPid !== undefined) {
+				if (ankiPid === undefined) {
+					await execa('osascript', ['-e', 'tell application "Anki" to quit']).catch(async () => {
+						await execa('sh', [
+							'-c',
+							"launchctl stop $(launchctl list | grep ankiweb | awk '{print $3}')",
+						]).catch(() => {
+							// Ignore
+						})
+					})
+				} else {
 					try {
 						process.kill(-ankiPid, 'SIGKILL')
 					} catch {
 						// Ignore if process group already exited
 					}
 
 					ankiPid = undefined
-				} else {
-					await execa('osascript', ['-e', 'tell application "Anki" to quit']).catch(
-						async () => {
-							await execa('sh', [
-								'-c',
-								"launchctl stop $(launchctl list | grep ankiweb | awk '{print $3}')",
-							]).catch(() => {})
-						},
-					)
 				}
 
 				// Fallback for pip-installed Anki in worker processes where ankiPid is lost
-				await execa('pkill', ['-9', '-f', 'bin/anki']).catch(() => {})
+				await execa('pkill', ['-9', '-f', 'bin/anki']).catch(() => {
+					// Ignore
+				})
 				break
 			}
 
 			case 'windows': {
 				if (ankiPid !== undefined) {
-					await execa('taskkill', ['/PID', String(ankiPid), '/T', '/F']).catch(() => {})
+					await execa('taskkill', ['/PID', String(ankiPid), '/T', '/F']).catch(() => {
+						// Ignore
+					})
 					ankiPid = undefined
 				}
 
-				await execa('taskkill', ['/IM', 'anki.exe', '/T', '/F']).catch(() => {})
+				await execa('taskkill', ['/IM', 'anki.exe', '/T', '/F']).catch(() => {
+					// Ignore
+				})
 				break
 			}
 		}