Improve prepack script (#1747)

* improve prepack script

* remove .npmignore

* make prepack use pkg.files

* fix linter errors

package.json

@@ -5,6 +5,8 @@
   "files": [
     "build",
     "contracts",
+    "!contracts/mocks",
+    "!contracts/examples",
     "test/behaviors"
   ],
   "scripts": {
@@ -18,10 +20,10 @@
     "lint:js": "eslint .",
     "lint:js:fix": "eslint . --fix",
     "lint:sol": "solhint --max-warnings 0 \"contracts/**/*.sol\"",
-    "prepack": "scripts/prepack.sh",
+    "prepack": "node scripts/prepack.js",
     "release": "scripts/release/release.sh",
     "version": "scripts/release/update-changelog-release-date.js && scripts/release/update-ethpm-version.js",
-    "test": "npm run compile && scripts/test.sh"
+    "test": "scripts/test.sh"
   },
   "repository": {
     "type": "git",
@@ -54,6 +56,7 @@
     "ethereumjs-util": "^6.0.0",
     "ganache-cli": "^6.4.1",
     "ganache-cli-coverage": "https://github.com/frangio/ganache-cli/releases/download/v6.4.1-coverage/ganache-cli-coverage-6.4.1.tgz",
+    "micromatch": "^4.0.2",
     "openzeppelin-docsite": "github:OpenZeppelin/openzeppelin-docsite",
     "openzeppelin-test-helpers": "^0.3.2",
     "solhint": "^1.5.0",

scripts/compile.sh

@@ -4,4 +4,7 @@ if [ "$SOLC_NIGHTLY" = true ]; then
   docker pull ethereum/solc:nightly
 fi
 
+# Necessary to avoid an error in Truffle
+rm -rf build/contracts
+
 truffle compile

scripts/prepack.js

@@ -0,0 +1,42 @@
+#!/usr/bin/env node
+
+// This script removes the build artifacts of ignored contracts.
+
+const fs = require('fs');
+const path = require('path');
+const cp = require('child_process');
+const match = require('micromatch');
+
+function exec (cmd, ...args) {
+  cp.execFileSync(cmd, args, { stdio: 'inherit' });
+}
+
+function readJSON (path) {
+  return JSON.parse(fs.readFileSync(path));
+}
+
+exec('npm', 'run', 'compile');
+
+const pkgFiles = readJSON('package.json').files;
+
+// Get only negated patterns.
+const ignorePatterns = pkgFiles
+  .filter(pat => pat.startsWith('!'))
+// Add **/* to ignore all files contained in the directories.
+  .flatMap(pat => [pat, path.join(pat, '**/*')])
+// Remove the negation part. Makes micromatch usage more intuitive.
+  .map(pat => pat.slice(1));
+
+const artifactsDir = 'build/contracts';
+
+for (const artifact of fs.readdirSync(artifactsDir)) {
+  const fullArtifactPath = path.join(artifactsDir, artifact);
+  const { sourcePath: fullSourcePath } = readJSON(fullArtifactPath);
+  const sourcePath = path.relative('.', fullSourcePath);
+
+  const ignore = match.any(sourcePath, ignorePatterns);
+
+  if (ignore) {
+    fs.unlinkSync(fullArtifactPath);
+  }
+}