Merge pull request #88 from netlify-team-account-1/ignore-node-prefix

[paperwork] take node: prefix as indicator for being a core module

index.js

@@ -163,6 +163,9 @@ precinct.paperwork = function(filename, options = {}) {
 
   if (!options.includeCore) {
     return deps.filter(function(d) {
+      if (d.startsWith('node:')) {
+        return false;
+      }
       return !natives[d];
     });
   }

test/index.js

@@ -338,6 +338,16 @@ describe('node-precinct', function() {
       });
     });
 
+    describe('that imports node-internal with node:-prefix', () => {
+      it('assumes that it exists', () => {
+        var deps = precinct.paperwork(path.join(__dirname, 'internalNodePrefix.js'), {
+          includeCore: false
+        });
+        assert(!deps.includes('node:nonexistant'));
+        assert.deepEqual(deps, ['streams']);
+      });
+    });
+
     describe('that uses dynamic imports', function() {
       it('grabs the dynamic import', function() {
         var es6 = precinct(read('es6DynamicImport.js'));

test/internalNodePrefix.js

@@ -0,0 +1,2 @@
+const someModule = require("node:nonexistant")
+const anotherModule = require("streams")