CLOUDP-288860: Require .js extension on imports

Author: lovisaberggren

eslint.config.mjs

@@ -1,8 +1,18 @@
+import { fileURLToPath } from 'node:url';
+import * as path from 'node:path';
 import globals from 'globals';
+import { FlatCompat } from '@eslint/eslintrc';
 import pluginJs from '@eslint/js';
 import pluginJest from 'eslint-plugin-jest';
 import jest from 'eslint-plugin-jest';
 
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+
+const compat = new FlatCompat({
+  baseDirectory: __dirname,
+});
+
 /** @type {import('eslint').Linter.Config[]} */
 export default [
   {
@@ -23,4 +33,9 @@ export default [
     files: ['**/*.test.js'],
     ...jest.configs['flat/recommended'],
   },
+  ...compat.config({
+    plugins: ['require-extensions'],
+    extends: 'plugin:require-extensions/recommended',
+    ignorePatterns: ['**/*.test.js'],
+  }),
 ];

package-lock.json

@@ -29,6 +29,7 @@
         "@eslint/js": "^9.16.0",
         "@jest/globals": "^29.7.0",
         "eslint": "^9.17.0",
+        "eslint-plugin-require-extensions": "^0.1.3",
         "globals": "^15.13.0",
         "jest": "^29.7.0",
         "prettier": "3.4.2"

package.json

@@ -1,5 +1,5 @@
 import { isPathParam } from './utils/pathUtils.js';
-import { hasException } from './utils/exceptions';
+import { hasException } from './utils/exceptions.js';
 
 const RULE_NAME = 'xgen-IPA-102-path-alternate-resource-name-path-param';
 const ERROR_MESSAGE = 'API paths must alternate between resource name and path params.';

tools/spectral/ipa/rulesets/functions/eachPathAlternatesBetweenResourceNameAndPathParam.js

@@ -6,7 +6,7 @@ import {
   isSingletonResource,
   getResourcePaths,
 } from './utils/resourceEvaluation.js';
-import { hasException } from './utils/exceptions';
+import { hasException } from './utils/exceptions.js';
 
 const RULE_NAME = 'xgen-IPA-104-resource-has-GET';
 const ERROR_MESSAGE = 'APIs must provide a get method for resources.';