#270 Allow node imports in data-browser

Author: joepio

.eslintrc.js

@@ -52,7 +52,8 @@ module.exports = {
     'comma-dangle': 'off', // https://eslint.org/docs/rules/comma-dangle
     'function-paren-newline': 'off', // https://eslint.org/docs/rules/function-paren-newline
     'global-require': 'off', // https://eslint.org/docs/rules/global-require
-    "import/extensions": ["error", "ignorePackages"],
+    // Turn this on when we have migrated all import paths to use `.js`
+    // "import/extensions": ["error", "ignorePackages"],
     "import/no-unresolved": "off",
     'import/no-dynamic-require': 'off', // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-dynamic-require.md
     'no-inner-declarations': 'off', // https://eslint.org/docs/rules/no-inner-declarations// New rules

data-browser/package.json

@@ -72,5 +72,6 @@
     "test-query": "PWDEBUG=1 npx playwright test \"e2e.spec.ts\" -g",
     "typecheck": "tsc --noEmit"
   },
+  "type": "module",
   "version": "0.34.0"
 }

data-browser/src/helpers/AppSettings.tsx

@@ -1,5 +1,4 @@
-import React, { ReactNode } from 'react';
-import { useContext } from 'react';
+import React, { ReactNode, useContext } from 'react';
 import { DarkModeOption, useDarkMode } from './useDarkMode';
 import {
   useLocalStorage,

data-browser/tsconfig.json

@@ -2,6 +2,8 @@
   "extends": "../tsconfig.build.json",
   "compilerOptions": {
     "composite": true,
+    // We overwrite this so we don't have to use `.js` extensions in imports
+    "moduleResolution": "node",
     "outDir": "./dist",
     "strictNullChecks": true,
   },