WARNING: edge-case

Author: AugustinMauroy

recipes/fs-truncate-fd-deprecation/README.md

@@ -31,3 +31,8 @@ open('file.txt', 'w', (err, fd) => {
   });
 });
 ```
+
+## Caveats
+
+When using `const fd = fs.openSync('file.txt', 'w');`, the codemod don't care about scope and will always transform `fs.truncate(fd, 10)` to `fs.ftruncateSync(fd, 10)`. So if you have mutiple variables with the same name, you should be careful with the transformation.
+

recipes/fs-truncate-fd-deprecation/tests/expected/edge-case.mjs

@@ -0,0 +1,25 @@
+import fs from 'node:fs';
+
+export default function truncateFile() {
+	fs.open('file.txt', 'w', (err, strangeName) => {
+		if (err) throw err;
+		fs.ftruncate(strangeName, 10, (err) => {
+			if (err) throw err;
+			fs.close(strangeName, () => { });
+		});
+	});
+}
+
+const accesible = fs.openSync('file.txt', 'w');
+
+fs.ftruncateSync(accesible, 10);
+
+fs.closeSync(accesible);
+
+function foo() {
+	truncateFile(unaccessible, 10);
+}
+
+function bar() {
+	const unaccessible = fs.openSync('file.txt', 'w');
+}

recipes/fs-truncate-fd-deprecation/tests/input/edge-case.mjs

@@ -0,0 +1,25 @@
+import fs from 'node:fs';
+
+export default function truncateFile() {
+	fs.open('file.txt', 'w', (err, strangeName) => {
+		if (err) throw err;
+		fs.truncate(strangeName, 10, (err) => {
+			if (err) throw err;
+			fs.close(strangeName, () => { });
+		});
+	});
+}
+
+const accesible = fs.openSync('file.txt', 'w');
+
+fs.truncateSync(accesible, 10);
+
+fs.closeSync(accesible);
+
+function foo() {
+	truncateFile(unaccessible, 10);
+}
+
+function bar() {
+	const unaccessible = fs.openSync('file.txt', 'w');
+}