feat: remove any querystring from imports (#67)

Author: joakimbeng

package.json

@@ -42,6 +42,7 @@
     "test:multipleTsconfigs": "eslint --ext ts,tsx tests/multipleTsconfigs",
     "test:withJsExtension": "node tests/withJsExtension/test.js && eslint --ext ts,tsx tests/withJsExtension",
     "test:withPaths": "eslint --ext ts,tsx tests/withPaths",
+    "test:withQuerystring": "eslint --ext ts,tsx tests/withQuerystring",
     "test:withoutPaths": "eslint --ext ts,tsx tests/withoutPaths",
     "type-coverage": "type-coverage --cache --detail --ignore-catch --strict --update"
   },

src/index.ts

@@ -51,6 +51,8 @@ export function resolve(
 
   log('looking for:', source)
 
+  source = removeQuerystring(source)
+
   // don't worry about core node modules
   if (isCore(source)) {
     log('matched core:', source)
@@ -136,6 +138,15 @@ function tsResolve(id: string, opts?: SyncOpts): string {
   }
 }
 
+/** Remove any trailing querystring from module id. */
+function removeQuerystring(id: string) {
+  const querystringIndex = id.lastIndexOf('?')
+  if (querystringIndex >= 0) {
+    return id.slice(0, querystringIndex)
+  }
+  return id
+}
+
 /** Remove .js or .jsx extension from module id. */
 function removeJsExtension(id: string) {
   return id.replace(/\.jsx?$/, '')

tests/withQuerystring/.eslintrc.js

@@ -0,0 +1 @@
+module.exports = require('../baseEslintConfig')(__dirname)

tests/withQuerystring/image.svg

@@ -0,0 +1,7 @@
+// import svg without querystring
+import './image.svg'
+import './subfolder/image.svg'
+
+// import svg with querystring
+import './image.svg?raw'
+import './subfolder/image.svg?raw'

tests/withQuerystring/index.ts

@@ -0,0 +1,4 @@
+{
+  "compilerOptions": {},
+  "files": ["index.ts"]
+}