Fix AvaTax app eslint errors (#1580)

.changeset/afraid-dancers-cross.md

@@ -0,0 +1,5 @@
+---
+"app-avatax": patch
+---
+
+Fix Eslint issues. It won't affect AvaTax app functionality.

apps/avatax/.eslintrc.cjs

@@ -0,0 +1,13 @@
+module.exports = {
+ root: true,
+ extends: ["saleor"],
+ plugins: ["neverthrow"],
+ rules: {
+ "turbo/no-undeclared-env-vars": ["error"],
+ },
+ parserOptions: {
+ project: "tsconfig.json",
+ tsconfigRootDir: __dirname,
+ },
+ ignorePatterns: ["**/generated/graphql.ts"],
+};

apps/avatax/scripts/run-webhooks-migration.ts

@@ -35,7 +35,7 @@ const runMigrations = async () => {
  logger.info("Fetching environments from the saleor-cloud APL", { dryRun });
 
  const saleorCloudEnv = await saleorAPL.getAll().catch((error) => {
- logger.error("Could not fetch instances from the Cloud APL", { error, dryRun });
+ logger.error("Could not fetch instances from the Cloud APL", { error: error, dryRun });
 
  process.exit(1);
  });

apps/avatax/src/modules/avatax/ui/tax-code-select.tsx

@@ -87,6 +87,7 @@ const useTaxCodeAutocomplete = ({ taxClassId }: { taxClassId: string }) => {
  prevValueRef.current = debouncedValue;
  updateMutation({ saleorTaxClassId: taxClassId, avataxTaxCode: debouncedValue });
  }
+ // eslint-disable-next-line react-hooks/exhaustive-deps
  }, [debouncedValue, isTouched, taxCodes, taxClassId, updateMutation]);
 
  React.useEffect(() => {

apps/avatax/src/pages/api/webhooks/order-calculate-taxes.ts

@@ -123,6 +123,7 @@ export default wrapWithLoggerContext(
  discountStrategy,
  );
 
+ // eslint-disable-next-line @saleor/saleor-app/logger-leak
  logger.info("Taxes calculated", { calculatedTaxes });
 
  return res.status(200).json(ctx.buildResponse(calculatedTaxes));

apps/avatax/src/pages/api/webhooks/order-confirmed.ts

@@ -172,7 +172,7 @@ export default wrapWithLoggerContext(
 
  return res.status(200).end();
  } catch (error) {
- logger.debug("Error confirming order", { error });
+ logger.debug("Error confirming order", { error: error });
 
  switch (true) {
  case error instanceof TaxBadPayloadError: {
@@ -192,7 +192,7 @@ export default wrapWithLoggerContext(
  }
  }
  Sentry.captureException(error);
- logger.error("Unhandled error executing webhook", { error });
+ logger.error("Unhandled error executing webhook", { error: error });
 
  return res.status(500).json({ message: "Unhandled error" });
  }
@@ -217,7 +217,7 @@ export default wrapWithLoggerContext(
  }
  } catch (error) {
  Sentry.captureException(error);
- logger.error("Unhandled error executing webhook", { error });
+ logger.error("Unhandled error executing webhook", { error: error });
 
  return res.status(500).json({ message: "Unhandled error" });
  }

apps/avatax/tsconfig.json

@@ -19,6 +19,6 @@
  "@/*": ["src/*"]
  }
  },
- "include": ["next-env.d.ts", "**/*.ts", "**/*.tsx", "next.config.js"],
+ "include": ["next-env.d.ts", "**/*.ts", "**/*.tsx", "next.config.js", ".eslintrc.cjs"],
  "exclude": ["node_modules"]
 }

cspell.json

@@ -97,7 +97,7 @@
  "apps/*/src/**",
  "apps/*/e2e/data/**",
  "**/*.bru",
- "**/.eslintrc",
+ "**/.eslintrc*",
  ".github/**",
  "Dockerfile*"
  ]