fix: emit warning if input.json is modified during run and prefilled with defaults (#672)

Author: vladfrangu

src/commands/run.ts

@@ -357,6 +357,21 @@ export class RunCommand extends ApifyCommand<typeof RunCommand> {
 		} finally {
 			if (storedInputResults) {
 				if (storedInputResults.existingInput) {
+					// Check if the input file was modified since we modified it. If it was, we abort the re-overwrite and warn the user
+					const stats = await stat(storedInputResults.inputFilePath);
+
+					const mtime = Math.trunc(stats.mtimeMs);
+
+					// If its in a 5ms range, we assume the file was modified (realistically impossible)
+					if (mtime - storedInputResults.writtenAt >= 5) {
+						warning({
+							message: `The "${storedInputResults.inputFilePath}" file was overwritten during the run. The CLI will not undo the setting of missing default fields from your input schema.`,
+						});
+
+						// eslint-disable-next-line no-unsafe-finally -- we do not return anything in the commands anyways
+						return;
+					}
+
 					// Overwrite with the original input
 					await writeFile(storedInputResults.inputFilePath, storedInputResults.existingInput.body);
 				} else {
@@ -398,6 +413,7 @@ export class RunCommand extends ApifyCommand<typeof RunCommand> {
 			return {
 				existingInput,
 				inputFilePath,
+				writtenAt: Date.now(),
 			};
 		}
 
@@ -457,6 +473,7 @@ export class RunCommand extends ApifyCommand<typeof RunCommand> {
 			return {
 				existingInput,
 				inputFilePath,
+				writtenAt: Date.now(),
 			};
 		}
 
@@ -468,6 +485,7 @@ export class RunCommand extends ApifyCommand<typeof RunCommand> {
 			return {
 				existingInput,
 				inputFilePath,
+				writtenAt: Date.now(),
 			};
 		}
 
@@ -501,6 +519,7 @@ export class RunCommand extends ApifyCommand<typeof RunCommand> {
 			return {
 				existingInput,
 				inputFilePath,
+				writtenAt: Date.now(),
 			};
 		}