chore: update eslint to v9 (#1439)

Closes #1276

Author: B4nan

.eslintignore

@@ -6,9 +6,9 @@ const apifyClient = new ApifyClient({
 const result = await apifyClient
     .requestQueue('<QUEUE ID>')
     .addRequest({
-        'uniqueKey': 'http://example.com',
-        'url': 'http://example.com',
-        'method': 'GET',
+        uniqueKey: 'http://example.com',
+        url: 'http://example.com',
+        method: 'GET',
     });
 
 console.log(result);

apify-api/openapi/code_samples/javascript/requestQueue_requests_post.js

@@ -3,26 +3,6 @@ const remarkStringify = require('remark-stringify');
 const { unified } = require('unified');
 const { visitParents } = require('unist-util-visit-parents');
 
-/**
- * Updates the markdown content for better UX and compatibility with Docusaurus v3.
- * @param {string} changelog The markdown content.
- * @returns {string} The updated markdown content.
- */
-function updateChangelog(changelog) {
-    const pipeline = unified()
-        .use(remarkParse)
-        .use(removeGitCliffMarkers)
-        .use(incrementHeadingLevels)
-        .use(prettifyPRLinks)
-        .use(linkifyUserTags)
-        .use(remarkStringify);
-
-    changelog = pipeline.processSync(changelog).toString();
-    changelog = addFrontmatter(changelog);
-    changelog = escapeMDXCharacters(changelog);
-    return changelog;
-}
-
 /**
  * Bumps the headings levels in the markdown content. This function increases the depth
  * of all headings in the content by 1. This is useful when the content is included in
@@ -60,7 +40,7 @@ const linkifyUserTags = () => (tree) => {
 
         const directParent = parents[parents.length - 1];
 
-        if (!match || directParent.type === 'link') return;
+        if (!match || directParent.type === 'link') return 0;
 
         const nodeIndexInParent = directParent.children.findIndex((x) => x === node);
 
@@ -95,7 +75,7 @@ const prettifyPRLinks = () => (tree) => {
         const prLinkRegex = /https:\/\/github.com\/[^\s]+\/pull\/(\d+)/g;
         const match = prLinkRegex.exec(node.value);
 
-        if (!match) return;
+        if (!match) return 0;
 
         const directParent = parents[parents.length - 1];
         const nodeIndexInParent = directParent.children.findIndex((x) => x === node);
@@ -148,6 +128,26 @@ function escapeMDXCharacters(changelog) {
     });
 }
 
+/**
+ * Updates the markdown content for better UX and compatibility with Docusaurus v3.
+ * @param {string} changelog The markdown content.
+ * @returns {string} The updated markdown content.
+ */
+function updateChangelog(changelog) {
+    const pipeline = unified()
+        .use(remarkParse)
+        .use(removeGitCliffMarkers)
+        .use(incrementHeadingLevels)
+        .use(prettifyPRLinks)
+        .use(linkifyUserTags)
+        .use(remarkStringify);
+
+    changelog = pipeline.processSync(changelog).toString();
+    changelog = addFrontmatter(changelog);
+    changelog = escapeMDXCharacters(changelog);
+    return changelog;
+}
+
 module.exports = {
     updateChangelog,
 };

apify-docs-theme/src/markdown.js

@@ -38,13 +38,14 @@ async function getHash(source) {
     })).json();
 
     if (!res.data || !res.data.encoded) {
-        // eslint-disable-next-line no-console
         console.error(`Signing failed:' ${inspect(res.error) || 'Unknown error'}`, res);
         return 'invalid-token';
     }
 
     cache[cacheKey] = res.data.encoded;
-    await new Promise((resolve) => setTimeout(resolve, 100));
+    await new Promise((resolve) => {
+        setTimeout(resolve, 100);
+    });
 
     return res.data.encoded;
 }
@@ -79,7 +80,6 @@ module.exports = async function (code) {
         return { code, hash: 'fast' };
     }
 
-    // eslint-disable-next-line no-console
     console.log(`Signing ${urlToRequest(this.resourcePath)}...`, { working, queue: queue.length });
     const codeHash = await encodeAndSign(code);
 

apify-docs-theme/src/roa-loader/index.js

@@ -108,7 +108,6 @@ This either means that your Docusaurus setup is misconfigured, or that your GitH
                     await generateChangelogFromGitHubReleases(pathsToCopyChangelog, `${context.siteConfig.organizationName}/${context.siteConfig.projectName}`);
                 }
             } catch (e) {
-                // eslint-disable-next-line no-console
                 console.warn(`Changelog page could not be initialized: ${e.message}`);
             }
         },

apify-docs-theme/src/theme.js

@@ -1,11 +1,11 @@
-import React from 'react';
-// eslint-disable-next-line import/no-extraneous-dependencies
-import clsx from 'clsx';
-import useIsBrowser from '@docusaurus/useIsBrowser';
 import { translate } from '@docusaurus/Translate';
-import IconLightMode from '../Icon/LightMode';
-import IconDarkMode from '../Icon/DarkMode';
+import useIsBrowser from '@docusaurus/useIsBrowser';
+import clsx from 'clsx';
+import React from 'react';
+
 import styles from './styles.module.css';
+import IconDarkMode from '../Icon/DarkMode';
+import IconLightMode from '../Icon/LightMode';
 
 function ColorModeToggle({
     className,

apify-docs-theme/src/theme/ColorModeToggle/index.jsx

@@ -1,12 +1,12 @@
-import React from 'react';
-// eslint-disable-next-line import/no-extraneous-dependencies
-import clsx from 'clsx';
+import isInternalUrl from '@docusaurus/isInternalUrl';
+import Link from '@docusaurus/Link';
+import { isActiveSidebarItem } from '@docusaurus/plugin-content-docs/client';
 import { ThemeClassNames } from '@docusaurus/theme-common';
 import useDocusaurusContext from '@docusaurus/useDocusaurusContext';
-import { isActiveSidebarItem } from '@docusaurus/plugin-content-docs/client';
-import Link from '@docusaurus/Link';
-import isInternalUrl from '@docusaurus/isInternalUrl';
 import IconExternalLink from '@theme/Icon/ExternalLink';
+import clsx from 'clsx';
+import React from 'react';
+
 import styles from './styles.module.css';
 
 export default function DocSidebarItemLink({

apify-docs-theme/src/theme/DocSidebarItem/Link/index.jsx

@@ -1,8 +1,8 @@
-import React from 'react';
-// eslint-disable-next-line import/no-extraneous-dependencies
-import clsx from 'clsx';
 import { useThemeConfig } from '@docusaurus/theme-common';
 import LinkItem from '@theme/Footer/LinkItem';
+import clsx from 'clsx';
+import React from 'react';
+
 import styles from './index.module.css';
 
 function FooterLinksColumn({ column }) {

apify-docs-theme/src/theme/Footer/index.jsx

@@ -1,9 +1,9 @@
-import React from 'react';
+import { useLocation } from '@docusaurus/router';
 // cannot use any of the theme aliases here as it causes a circular dependency :( ideas welcome
 import Layout from '@docusaurus/theme-classic/lib/theme/Layout/index';
-import { usePluginData } from '@docusaurus/useGlobalData';
 import useBaseUrl from '@docusaurus/useBaseUrl';
-import { useLocation } from '@docusaurus/router';
+import { usePluginData } from '@docusaurus/useGlobalData';
+import React from 'react';
 
 export default function LayoutWrapper(props) {
     const { options: { subNavbar } } = usePluginData('@apify/docs-theme');
@@ -13,9 +13,9 @@ export default function LayoutWrapper(props) {
     return (
         <div style={{
             '--ifm-navbar-height': subNavbar && !currentPath.startsWith('api/v2') ? '123px' : '68px',
-            'margin': 0,
-            'padding': 0,
-            'boxSizing': 'border-box',
+            margin: 0,
+            padding: 0,
+            boxSizing: 'border-box',
         }}>
             <Layout {...props} />
         </div>

apify-docs-theme/src/theme/Layout/index.jsx

@@ -1,8 +1,8 @@
-import React from 'react';
 import { useThemeConfig } from '@docusaurus/theme-common';
 import useBaseUrl from '@docusaurus/useBaseUrl';
 import { usePluginData } from '@docusaurus/useGlobalData';
 import NavbarItem from '@theme/NavbarItem';
+import React from 'react';
 
 function useNavbarItems() {
     // TODO temporary casting until ThemeConfig type is improved