chore: lint blocking calls (#180)

Author: sigmaSd

deno.json

@@ -9,7 +9,10 @@
   },
   "lint": {
     "rules": {
-      "exclude": ["ban-types", "no-explicit-any", "no-this-alias"]
+      "exclude": ["ban-types", "no-explicit-any", "no-this-alias"],
+      "include": [
+        "no-sync-fn-in-async-fn"
+      ]
     }
   },
   "exclude": [

mod.test.ts

@@ -1274,7 +1274,7 @@ Deno.test("cp test2", async () => {
   await withTempDir(async (dir) => {
     await $`mkdir -p a/d1`;
     await $`mkdir -p a/d2`;
-    Deno.createSync("a/d1/f").close();
+    await Deno.create("a/d1/f").then((f) => f.close());
     await $`cp a/d1/f a/d2`;
     assert(dir.join("a/d2/f").existsSync());
   });
@@ -1430,12 +1430,12 @@ Deno.test("cd", () => {
 
 Deno.test("cat", async () => {
   await withTempDir(async () => {
-    Deno.writeTextFileSync("hello", "hello world");
+    await Deno.writeTextFile("hello", "hello world");
     assertEquals(
       await $`cat hello`.text(),
       "hello world",
     );
-    Deno.writeTextFileSync("hello2", "hello world2");
+    await Deno.writeTextFile("hello2", "hello world2");
     assertEquals(
       await $`cat hello hello2`.text(),
       "hello worldhello world2",

src/commands/cat.ts

@@ -39,7 +39,7 @@ async function executeCat(context: CommandContext) {
     } else {
       let file;
       try {
-        file = Deno.openSync(pathUtils.join(context.cwd, path), { read: true });
+        file = await Deno.open(pathUtils.join(context.cwd, path), { read: true });
         while (true) {
           // NOTE: rust supports cancellation here
           const size = file.readSync(buf);

src/deps.test.ts

@@ -18,7 +18,7 @@ export { serve } from "https://deno.land/[email protected]/http/server.ts";
  */
 export async function withTempDir(action: (path: PathRef) => Promise<void> | void) {
   const originalDirPath = Deno.cwd();
-  const dirPath = Deno.makeTempDirSync();
+  const dirPath = await Deno.makeTempDir();
   Deno.chdir(dirPath);
   try {
     await action(createPathRef(dirPath).resolve());

src/request.test.ts

@@ -108,7 +108,7 @@ Deno.test("$.request", (t) => {
     });
 
     step("pipeToPath", async () => {
-      const testFilePath = Deno.makeTempFileSync();
+      const testFilePath = await Deno.makeTempFile();
       const originDir = Deno.cwd();
       try {
         {
@@ -121,7 +121,7 @@ Deno.test("$.request", (t) => {
         }
         {
           // test default path
-          Deno.chdir(Deno.makeTempDirSync()); // change path just to not download to the current dir
+          Deno.chdir(await Deno.makeTempDir()); // change path just to not download to the current dir
           const downloadedFilePath = await new RequestBuilder()
             .url(new URL("/text-file", serverUrl))
             .showProgress()
@@ -151,7 +151,7 @@ Deno.test("$.request", (t) => {
         }
         {
           // test downloading to a directory
-          const tempDir = Deno.makeTempDirSync();
+          const tempDir = await Deno.makeTempDir();
           const downloadedFilePath = await new RequestBuilder()
             .url(new URL("/text-file", serverUrl))
             .showProgress()
@@ -162,7 +162,7 @@ Deno.test("$.request", (t) => {
       } finally {
         try {
           Deno.chdir(originDir);
-          Deno.removeSync(testFilePath);
+          await Deno.remove(testFilePath);
         } catch {
           // do nothing
         }