@automattic/vip 2.30.0 → 2.31.0-dev1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@automattic/vip",
-  "version": "2.30.0",
+  "version": "2.31.0-dev1",
   "description": "The VIP Javascript library & CLI",
   "main": "index.js",
   "bin": {
@@ -54,7 +54,7 @@
     "vip-wp": "dist/bin/vip-wp.js"
   },
   "scripts": {
-    "test": "npm run lint && npm run flow && jest --coverage --testPathIgnorePatterns __tests__/devenv-e2e/",
+    "test": "npm run lint && npm run flow && npm run check-types && jest --testPathIgnorePatterns __tests__/devenv-e2e/",
     "test:e2e:dev-env": "jest -c __tests__/devenv-e2e/jest/jest.config.js",
     "clean": "rimraf dist",
     "prepare": "npm run clean && npm run build",
@@ -122,6 +122,7 @@
     "@types/xdg-basedir": "^4.0.2",
     "@types/xml2js": "^0.4.11",
     "babel-jest": "^29.5.0",
+    "babel-plugin-explicit-exports-references": "^1.0.2",
     "babel-plugin-module-resolver": "5.0.0",
     "check-node-version": "^4.2.1",
     "dockerode": "^3.3.4",
@@ -129,7 +130,6 @@
     "eslint-plugin-flowtype": "^8.0.3",
     "flow-bin": "^0.206.0",
     "jest": "^29.5.0",
-    "jest-environment-jsdom": "^29.5.0",
     "nock": "13.3.1",
     "prettier": "^2.0.5",
     "publish-please": "5.5.2",
@@ -139,7 +139,7 @@
   "dependencies": {
     "@apollo/client": "3.3.6",
     "@automattic/vip-go-preflight-checks": "^2.0.16",
-    "@automattic/vip-search-replace": "^1.0.15",
+    "@automattic/vip-search-replace": "^1.1.1",
     "args": "5.0.3",
     "chalk": "4.1.2",
     "cli-columns": "^4.0.0",

package/tsconfig.json

@@ -11,10 +11,12 @@
     "checkJs": false,
     // Disallow features that require cross-file information for emit as we're using Babel
     "isolatedModules": true,
+    "resolveJsonModule": true,
     "baseUrl": ".",
     "paths": {
-      "root/*": [
-        "./*"
+      "*": [
+        "./types/*",
+        "./node_modules/*"
       ]
     },
     // It's worth enabling this, I promise!

package/dist/lib/analytics/clients/stub.js

@@ -1,16 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.default = void 0;
-/**
- * Internal dependencies
- */
-class AnalyticsClientStub {
-  // eslint-disable-next-line no-unused-vars
-  trackEvent(name, props) {
-    return Promise.resolve(true);
-  }
-}
-exports.default = AnalyticsClientStub;

package/dist/lib/cli/repo.js

@@ -1,61 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.default = getRepoFromGitConfig;
-var _fs = _interopRequireDefault(require("fs"));
-var _path = _interopRequireDefault(require("path"));
-var _ini = _interopRequireDefault(require("ini"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-/**
- * External dependencies
- */
-async function getRepoFromGitConfig() {
-  const file = await find();
-  if (!file.length) {
-    return '';
-  }
-  const config = _ini.default.parse(_fs.default.readFileSync(file, 'utf-8'));
-
-  // Find the first 'wpcomvip' remote
-  for (const key in config) {
-    if ('remote' !== key.substring(0, 6)) {
-      continue;
-    }
-    if (!config[key].url) {
-      continue;
-    }
-    if (0 > config[key].url.indexOf('wpcomvip/')) {
-      continue;
-    }
-    let repo = config[key].url;
-    repo = repo.replace(/.git$/, '');
-    repo = repo.replace('https://github.com/', '');
-    repo = repo.replace('git@github.com:', '');
-    return repo;
-  }
-}
-async function find(dir) {
-  dir = dir || process.cwd();
-  const test = dir + '/.git/config';
-  if (await exists(test)) {
-    return test;
-  }
-
-  // Bail if we went all the way and didn't find it
-  const directory = _path.default.parse(dir);
-  if (directory.dir === directory.root) {
-    return '';
-  }
-
-  // cd ..
-  const up = dir.split(_path.default.sep);
-  up.pop();
-  return find(up.join(_path.default.sep));
-}
-async function exists(file) {
-  return new Promise(resolve => {
-    _fs.default.access(file, _fs.default.constants.F_OK, err => resolve(!err));
-  });
-}

package/noImplictAnyImportBypass.d.ts

@@ -1,6 +0,0 @@
-// This file allows us to bypass the noImplicitAny rule for importing modules
-// noImplicitAny is too good of a rule to fully disable, so we provide this file to ensure that we can still import modules without types.
-// as that seems to have been a big pain point for everyone.
-// reference: https://www.typescriptlang.org/docs/handbook/modules.html#shorthand-ambient-modules
-
-declare module '*';