build(deps-dev): bump @biomejs/biome from 1.9.4 to 2.0.6 (#470)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Eden Zimbelman <[email protected]>

Author: dependabot[bot]

biome.json

@@ -1,7 +1,7 @@
 {
   "$schema": "./node_modules/@biomejs/biome/configuration_schema.json",
   "files": {
-    "ignore": ["*.json"],
+    "includes": ["**", "!**/*.json", "!coverage"],
     "ignoreUnknown": true
   },
   "formatter": {
@@ -10,11 +10,27 @@
   "linter": {
     "enabled": true,
     "rules": {
-      "recommended": true
+      "recommended": true,
+      "style": {
+        "noParameterAssign": "error",
+        "useAsConstAssertion": "error",
+        "useDefaultParameterLast": "error",
+        "useEnumInitializers": "error",
+        "useSelfClosingElements": "error",
+        "useSingleVarDeclarator": "error",
+        "noUnusedTemplateLiteral": "error",
+        "useNumberNamespace": "error",
+        "noInferrableTypes": "error",
+        "noUselessElse": "error"
+      }
     }
   },
-  "organizeImports": {
-    "enabled": true
+  "assist": {
+    "actions": {
+      "source": {
+        "organizeImports": "on"
+      }
+    }
   },
   "vcs": {
     "enabled": true,

package-lock.json

@@ -45,7 +45,7 @@
     "markup-js": "^1.5.21"
   },
   "devDependencies": {
-    "@biomejs/biome": "^1.9.4",
+    "@biomejs/biome": "^2.0.6",
     "@types/chai": "^5.0.1",
     "@types/flat": "^5.0.5",
     "@types/js-yaml": "^4.0.9",

package.json

@@ -13,9 +13,9 @@ export default async function send(core) {
   const config = new Config(core);
   try {
     await post(config);
-    config.core.setOutput("time", Math.floor(new Date().valueOf() / 1000));
+    config.core.setOutput("time", Math.floor(Date.now() / 1000));
   } catch (/** @type {any} */ error) {
-    config.core.setOutput("time", Math.floor(new Date().valueOf() / 1000));
+    config.core.setOutput("time", Math.floor(Date.now() / 1000));
     if (config.inputs.errors) {
       core.setFailed(error);
       throw new SlackError(core, error);

src/send.js

@@ -206,8 +206,8 @@ describe("client", () => {
         );
         assert.equal(mocks.core.setOutput.getCall(5).firstArg, "time");
         assert.equal(mocks.core.setOutput.getCalls().length, 6);
-      } catch (error) {
-        console.error(error);
+      } catch (err) {
+        console.error(err);
         assert.fail("Unexpected error when calling the method");
       }
     });
@@ -244,8 +244,8 @@ describe("client", () => {
         assert.equal(mocks.core.setOutput.getCall(2).lastArg, "C0101010101");
         assert.equal(mocks.core.setOutput.getCall(3).firstArg, "time");
         assert.equal(mocks.core.setOutput.getCalls().length, 4);
-      } catch (error) {
-        console.error(error);
+      } catch (err) {
+        console.error(err);
         assert.fail("Unexpected error when calling the method");
       }
     });
@@ -277,8 +277,8 @@ describe("client", () => {
         );
         assert.equal(mocks.core.setOutput.getCall(2).firstArg, "time");
         assert.equal(mocks.core.setOutput.getCalls().length, 3);
-      } catch (error) {
-        console.error(error);
+      } catch (err) {
+        console.error(err);
         assert.fail("Unexpected error when calling the method");
       }
     });
@@ -305,7 +305,7 @@ describe("client", () => {
         mocks.calls.rejects(errors.requestErrorWithOriginal(response, true));
         await send(mocks.core);
         assert.fail("Expected an error but none was found");
-      } catch (error) {
+      } catch (_err) {
         assert.isTrue(mocks.core.setFailed.called);
         assert.equal(mocks.core.setOutput.getCall(0).firstArg, "ok");
         assert.equal(mocks.core.setOutput.getCall(0).lastArg, false);
@@ -340,7 +340,7 @@ describe("client", () => {
         mocks.calls.rejects(errors.httpErrorFromResponse(response));
         await send(mocks.core);
         assert.fail("Expected an error but none was found");
-      } catch (error) {
+      } catch (_err) {
         assert.isFalse(mocks.core.setFailed.called);
         assert.equal(mocks.core.setOutput.getCall(0).firstArg, "ok");
         assert.equal(mocks.core.setOutput.getCall(0).lastArg, false);
@@ -376,7 +376,7 @@ describe("client", () => {
         mocks.calls.rejects(errors.platformErrorFromResult(response));
         await send(mocks.core);
         assert.fail("Expected an error but none was found");
-      } catch (error) {
+      } catch (_err) {
         assert.isTrue(mocks.core.setFailed.called);
         assert.equal(mocks.core.setOutput.getCall(0).firstArg, "ok");
         assert.equal(mocks.core.setOutput.getCall(0).lastArg, false);
@@ -405,7 +405,7 @@ describe("client", () => {
         mocks.calls.rejects(errors.platformErrorFromResult(response));
         await send(mocks.core);
         assert.fail("Expected an error but none was found");
-      } catch (error) {
+      } catch (_err) {
         assert.isFalse(mocks.core.setFailed.called);
         assert.equal(mocks.core.setOutput.getCall(0).firstArg, "ok");
         assert.equal(mocks.core.setOutput.getCall(0).lastArg, false);
@@ -431,7 +431,7 @@ describe("client", () => {
         mocks.calls.rejects(errors.rateLimitedErrorWithDelay(12));
         await send(mocks.core);
         assert.fail("Expected an error but none was found");
-      } catch (error) {
+      } catch (_err) {
         assert.isFalse(mocks.core.setFailed.called);
         assert.equal(mocks.core.setOutput.getCall(0).firstArg, "ok");
         assert.equal(mocks.core.setOutput.getCall(0).lastArg, false);

test/client.spec.js

@@ -91,10 +91,12 @@ describe("content", () => {
         }
       `);
       const config = new Config(mocks.core);
+      // biome-ignore-start lint/suspicious/noTemplateCurlyInString: GitHub Action YAML variable syntax
       const expected = {
         message: "this matches an existing variable: ${{ github.apiUrl }}",
         channel: "C0123456789",
       };
+      // biome-ignore-end lint/suspicious/noTemplateCurlyInString: https://docs.github.com/en/actions/how-tos/writing-workflows/choosing-what-your-workflow-does/store-information-in-variables#using-contexts-to-access-variable-values
       assert.deepEqual(config.content.values, expected);
     });
 
@@ -206,9 +208,11 @@ describe("content", () => {
      * @see {@link https://github.com/slackapi/slack-github-action/issues/203}
      */
     it("templatizes variables with missing variables", async () => {
+      // biome-ignore-start lint/suspicious/noTemplateCurlyInString: GitHub Action YAML variable syntax
       mocks.core.getInput
         .withArgs("payload")
         .returns("message: What makes ${{ env.TREASURE }} a secret");
+      // biome-ignore-end lint/suspicious/noTemplateCurlyInString: https://docs.github.com/en/actions/how-tos/writing-workflows/choosing-what-your-workflow-does/store-information-in-variables#using-contexts-to-access-variable-values
       mocks.core.getBooleanInput.withArgs("payload-templated").returns(true);
       const config = new Config(mocks.core);
       const expected = {
@@ -366,10 +370,12 @@ describe("content", () => {
         }
       `);
       const config = new Config(mocks.core);
+      // biome-ignore-start lint/suspicious/noTemplateCurlyInString: GitHub Action YAML variable syntax
       const expected = {
         message: "this matches an existing variable: ${{ github.apiUrl }}",
         channel: "C0123456789",
       };
+      // biome-ignore-end lint/suspicious/noTemplateCurlyInString: https://docs.github.com/en/actions/how-tos/writing-workflows/choosing-what-your-workflow-does/store-information-in-variables#using-contexts-to-access-variable-values
       assert.deepEqual(config.content.values, expected);
     });