@web/dev-server-rollup - npm Package Compare versions

Comparing version 0.2.11 to 0.2.12

CHANGELOG.md

 # @web/dev-server-rollup
 
+## 0.2.12
+
+### Patch Changes
+
+- d6de058: don't throw on unresolved local imports
+- 6950c7a: improve error message
+
 ## 0.2.11
@@ -4,0 +11,0 @@

dist/fromRollup.js

@@ -7,3 +7,3 @@ "use strict";
     if (typeof rollupPluginFn !== 'function') {
-        throw new Error('fromRollup should be called with a rollup plugin function.');
+        throw new Error(`fromRollup should be called with a rollup plugin function. Received: ${rollupPluginFn}`);
     }
@@ -10,0 +10,0 @@ // return a function wrapper which intercepts creation of the rollup plugin

dist/rollupAdapter.js

@@ -104,3 +104,4 @@ "use strict";
                 if (!resolvedImportPath) {
-                    if (adapterOptions.throwOnUnresolvedImport) {
+                    if (!['/', './', '../'].some(prefix => resolvableImport.startsWith(prefix)) &&
+                        adapterOptions.throwOnUnresolvedImport) {
                         const errorMessage = chalk_1.red(`Could not resolve import ${chalk_1.cyanBright(`"${source}"`)}.`);
@@ -107,0 +108,0 @@ if (typeof code === 'string' &&

package.json

 {
   "name": "@web/dev-server-rollup",
-  "version": "0.2.11",
+  "version": "0.2.12",
   "publishConfig": {
@@ -17,2 +17,8 @@ "access": "public"
   "main": "dist/index.js",
+  "exports": {
+    ".": {
+      "import": "./index.mjs",
+      "require": "./dist/index.js"
+    }
+  },
   "engines": {
@@ -22,4 +28,4 @@ "node": ">=10.0.0"
   "scripts": {
-    "test": "mocha \"test/node/**/*.test.ts\" --require ts-node/register --reporter dot",
-    "test:watch": "mocha \"test/node/**/*.test.ts\" --require ts-node/register --watch --watch-files src,test --reporter dot"
+    "test": "mocha \"test/node/**/*.test.ts\" --require ts-node/register",
+    "test:watch": "mocha \"test/node/**/*.test.ts\" --require ts-node/register --watch --watch-files src,test"
   },
@@ -49,9 +55,9 @@ "files": [
     "parse5": "^6.0.1",
-    "rollup": "^2.33.2",
-    "whatwg-url": "^8.1.0"
+    "rollup": "^2.34.2",
+    "whatwg-url": "^8.4.0"
   },
   "devDependencies": {
-    "@babel/plugin-transform-template-literals": "^7.10.4",
+    "@babel/plugin-transform-template-literals": "^7.12.1",
     "@rollup/plugin-alias": "^3.1.1",
-    "@rollup/plugin-babel": "^5.0.4",
+    "@rollup/plugin-babel": "^5.2.2",
     "@rollup/plugin-commonjs": "^16.0.0",
@@ -61,19 +67,13 @@ "@rollup/plugin-image": "^2.0.5",
     "@rollup/plugin-json": "^4.1.0",
-    "@rollup/plugin-node-resolve": "^8.1.0",
-    "@rollup/plugin-replace": "^2.3.3",
+    "@rollup/plugin-node-resolve": "^10.0.0",
+    "@rollup/plugin-replace": "^2.3.4",
     "@rollup/plugin-url": "^5.0.1",
     "@types/parse5": "^5.0.3",
+    "@types/whatwg-url": "^8.0.0",
     "@web/test-runner-chrome": "^0.7.0",
     "@web/test-runner-core": "^0.8.0",
     "chai": "^4.2.0",
-    "node-fetch": "v3.0.0-beta.9",
-    "rollup-plugin-postcss": "^3.1.2",
-    "sinon": "^9.0.2"
-  },
-  "exports": {
-    ".": {
-      "import": "./index.mjs",
-      "require": "./dist/index.js"
-    }
+    "node-fetch": "3.0.0-beta.9",
+    "rollup-plugin-postcss": "^3.1.8"
   }
 }

README.md

 # Dev Server Rollup
 
-Adapter for using rollup plugins in web dev server and web test runner.
+Adapter for using rollup plugins in Web Dev Server and Web Test Runner.
 
 See [our website](https://modern-web.dev/docs/dev-server/plugins/rollup/) for full documentation.

src/fromRollup.ts

@@ -17,3 +17,5 @@ import { InputOptions, Plugin as RollupPlugin } from 'rollup';
   if (typeof rollupPluginFn !== 'function') {
-    throw new Error('fromRollup should be called with a rollup plugin function.');
+    throw new Error(
+      `fromRollup should be called with a rollup plugin function. Received: ${rollupPluginFn}`,
+    );
   }
@@ -20,0 +22,0 @@

src/rollupAdapter.ts

@@ -153,3 +153,6 @@ /* eslint-disable no-control-regex */
         if (!resolvedImportPath) {
-          if (adapterOptions.throwOnUnresolvedImport) {
+          if (
+            !['/', './', '../'].some(prefix => resolvableImport.startsWith(prefix)) &&
+            adapterOptions.throwOnUnresolvedImport
+          ) {
             const errorMessage = red(`Could not resolve import ${cyanBright(`"${source}"`)}.`);
@@ -156,0 +159,0 @@ if (

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

75318

increased by0.68%

Lines of code

1142

increased by0.53%

Dependency changes

• Updatedrollup@^2.34.2

• Updatedwhatwg-url@^8.4.0