fix(watch): dont log rerunning on first run

privatenumber · Nov 23, 2023 · 5a71a87 · 5a71a87
1 parent fb61588
commit 5a71a87
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 12 deletions.
diff --git a/src/watch/index.ts b/src/watch/index.ts
@@ -112,18 +112,21 @@ export const watchCommand = command({
 			return;
 		}
 
-		// If running process
-		if (runProcess?.exitCode === null) {
-			log('restarting');
-			waitingExits = true;
-			await killProcess(runProcess);
-			waitingExits = false;
-		} else {
-			log('rerunning');
-		}
+		// If not first run
+		if (runProcess) {
+			// If process still running
+			if (runProcess.exitCode === null) {
+				log('restarting');
+				waitingExits = true;
+				await killProcess(runProcess);
+				waitingExits = false;
+			} else {
+				log('rerunning');
+			}
 
-		if (options.clearScreen) {
-			process.stdout.write(clearScreen);
+			if (options.clearScreen) {
+				process.stdout.write(clearScreen);
+			}
 		}
 
 		runProcess = spawnProcess();

diff --git a/tests/specs/watch.ts b/tests/specs/watch.ts
@@ -69,7 +69,8 @@ export default testSuite(async ({ describe }) => {
 
 			tsxProcess.kill();
 
-			await tsxProcess;
+			const { all } = await tsxProcess;
+			expect(all!.startsWith('hello world\n')).toBe(true);
 		}, 10_000);
 
 		test('suppresses warnings & clear screen', async () => {
@@ -100,6 +101,7 @@ export default testSuite(async ({ describe }) => {
 			const { all } = await tsxProcess;
 			expect(all).not.toMatch('Warning');
 			expect(all).toMatch('\u001Bc');
+			expect(all!.startsWith('["')).toBeTruthy();
 		}, 10_000);
 
 		test('passes flags', async () => {