@toptal/davinci-ci 3.1.1-alpha-fix-bundle-analyze-e1cf8407.3 → 3.1.1-alpha-feature-esm-migration-b7a1c917.27

package/bin/davinci-ci.js

@@ -1,8 +1,7 @@
 #!/usr/bin/env node
+import cliEngine from '@toptal/davinci-cli-shared'
 
-const cliEngine = require('@toptal/davinci-cli-shared')
+import dangerCommand from '../src/commands/danger.js'
 
-const { commands } = require('../src')
-
-cliEngine.loadCommands(commands, 'davinci-ci')
+cliEngine.loadCommands([dangerCommand], 'davinci-ci')
 cliEngine.bootstrap()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toptal/davinci-ci",
-  "version": "3.1.1-alpha-fix-bundle-analyze-e1cf8407.3+e1cf8407",
+  "version": "3.1.1-alpha-feature-esm-migration-b7a1c917.27+b7a1c917",
   "description": "Continuos integrations tools for frontend projects",
   "publishConfig": {
     "access": "public"
@@ -16,7 +16,8 @@
   "bin": {
     "davinci-ci": "./bin/davinci-ci.js"
   },
-  "main": "src/index.js",
+  "type": "module",
+  "exports": "./src/index.cjs",
   "files": [
     "src",
     "CHANGELOG.md"
@@ -26,7 +27,7 @@
     "url": "git+https://github.com/toptal/davinci.git"
   },
   "scripts": {
-    "test": "echo \"Error: run tests from root\" && exit 1"
+    "test": "NODE_OPTIONS='--experimental-vm-modules --no-warnings' yarn jest"
   },
   "bugs": {
     "url": "https://github.com/toptal/davinci/issues"
@@ -34,9 +35,12 @@
   "dependencies": {
     "@commitlint/cli": "^17.4.0",
     "@commitlint/config-conventional": "^17.1.0",
-    "@toptal/davinci-cli-shared": "1.10.3-alpha-fix-bundle-analyze-e1cf8407.5+e1cf8407",
+    "@toptal/davinci-cli-shared": "1.10.3-alpha-feature-esm-migration-b7a1c917.29+b7a1c917",
     "danger": "^11.0.7",
     "markdown-table": "^2.0.0"
   },
-  "gitHead": "e1cf84072ed2102a43bc67c5c725e5c45ff6a36e"
+  "devDependencies": {
+    "@jest/globals": "28.1.0"
+  },
+  "gitHead": "b7a1c917fc2cee48f71652bb6d6eda3ae34c42f5"
 }

package/src/commands/danger.js

@@ -1,10 +1,10 @@
-const {
+import {
   davinciProjectConfig,
   runSync,
   print,
   convertToCLIParameters,
   files,
-} = require('@toptal/davinci-cli-shared')
+} from '@toptal/davinci-cli-shared'
 
 const commandOptions = [
   {
@@ -28,11 +28,11 @@ const dangerCommand = ({ options }) => {
 
   const conventionalCommitsDangerfilePath = files.getPackageFilePath(
     '@toptal/davinci-ci',
-    'src/configs/danger/conventional-commits/dangerfile.js'
+    'src/configs/danger/conventional-commits/dangerfile.cjs'
   )
   const toptalCommitsDangerfilePath = files.getPackageFilePath(
     '@toptal/davinci-ci',
-    'src/configs/danger/toptal/dangerfile.js'
+    'src/configs/danger/toptal/dangerfile.cjs'
   )
 
   let dangerfilePath = dangerfile
@@ -64,4 +64,4 @@ const dangerCommandCreator = {
   options: commandOptions,
 }
 
-module.exports = dangerCommandCreator
+export default dangerCommandCreator

package/src/configs/danger/conventional-commits/dangerfile.cjs

@@ -0,0 +1,22 @@
+const { schedule } = require('danger')
+
+const { conventionalCommits } = require('./plugins/conventional-commits/index.cjs')
+const { conventionalPRTitle } = require('./plugins/conventional-pr-title/index.cjs')
+const { checkAssigneeExist } = require('../plugins/empty-assignee.cjs')
+
+
+schedule(conventionalCommits)
+schedule(conventionalPRTitle)
+
+const setup = async () => {
+  const { davinciProjectConfig } = await import('@toptal/davinci-cli-shared')
+  const {
+    master: { requireAssignee },
+  } = davinciProjectConfig
+
+  if (requireAssignee) {
+    schedule(checkAssigneeExist)
+  }
+}
+
+module.exports.default = setup

package/src/configs/danger/conventional-commits/plugins/conventional-commits/{index.js → index.cjs}

@@ -5,7 +5,7 @@ const { default: lint } = require('@commitlint/lint')
 /* eslint-enable */
 const table = require('markdown-table')
 
-const { WHITELISTED_USERS } = require('../../../toptal/config')
+const { WHITELISTED_USERS } = require('../../../toptal/config.cjs')
 
 const conventionalCommits = async () => {
   if (

package/src/configs/danger/conventional-commits/plugins/conventional-pr-title/{index.js → index.cjs}

@@ -6,7 +6,7 @@ const { default: lint } = require('@commitlint/lint')
 /* eslint-enable */
 const table = require('markdown-table')
 
-const { WHITELISTED_USERS } = require('../../../toptal/config')
+const { WHITELISTED_USERS } = require('../../../toptal/config.cjs')
 
 const conventionalPRTitle = async () => {
   if (!danger.github) {

package/src/configs/danger/conventional-commits/plugins/{index.js → index.cjs}

@@ -1,5 +1,5 @@
-const { conventionalCommits } = require('./conventional-commits')
-const { conventionalPRTitle } = require('./conventional-pr-title')
+const { conventionalCommits } = require('./conventional-commits/index.cjs')
+const { conventionalPRTitle } = require('./conventional-pr-title/index.cjs')
 
 module.exports = {
   conventionalCommits,

package/src/configs/danger/plugins/{empty-assignee.js → empty-assignee.cjs}

@@ -1,13 +1,14 @@
 /// <reference types="danger" />
 /* globals danger, fail */
-const { davinciProjectConfig } = require('@toptal/davinci-cli-shared')
 
-const {
-  master: { requireAssigneeWhiteList = [] },
-} = davinciProjectConfig
+const setup = async () => {
+  const { davinciProjectConfig } = await import('@toptal/davinci-cli-shared')
+  const {
+    master: { requireAssigneeWhiteList = [] },
+  } = davinciProjectConfig
+
+  const WHITELISTED_USERS = requireAssigneeWhiteList
 
-const WHITELISTED_USERS = requireAssigneeWhiteList
-const checkAssigneeExist = () => {
   const isLocalRun = !danger.github
 
   if (isLocalRun || WHITELISTED_USERS.includes(danger.github.pr.user.login)) {
@@ -19,6 +20,10 @@ const checkAssigneeExist = () => {
   }
 }
 
+const checkAssigneeExist = () => {
+  setup()
+}
+
 module.exports = {
   checkAssigneeExist,
 }

package/src/configs/danger/toptal/{dangerfile.js → dangerfile.cjs}

@@ -1,9 +1,9 @@
 const { schedule } = require('danger')
 const { davinciProjectConfig } = require('@toptal/davinci-cli-shared')
 
-const { toptalCommits } = require('./plugins/toptal-commits')
-const { toptalPRTitle } = require('./plugins/toptal-pr-title')
-const { checkAssigneeExist } = require('../plugins/empty-assignee')
+const { toptalCommits } = require('./plugins/toptal-commits/index.cjs')
+const { toptalPRTitle } = require('./plugins/toptal-pr-title/index.cjs')
+const { checkAssigneeExist } = require('../plugins/empty-assignee.cjs')
 
 schedule(toptalCommits)
 schedule(toptalPRTitle)

package/src/configs/danger/toptal/plugins/index.cjs

@@ -0,0 +1,7 @@
+const { toptalCommits } = require('./toptal-commits/index.cjs')
+const { toptalPRTitle } = require('./toptal-pr-title/index.cjs')
+
+module.exports = {
+  toptalCommits,
+  toptalPRTitle,
+}

package/src/configs/danger/toptal/plugins/toptal-commits/{index.js → index.cjs}

@@ -3,7 +3,7 @@ const {
   MAX_COMMIT_LINE_LENGTH,
   VALID_COMMIT_TITLE_REGEX,
   WHITELISTED_USERS,
-} = require('../../config')
+} = require('../../config.cjs')
 
 const validateCommitMessage = message => {
   const errors = []

package/src/configs/danger/toptal/plugins/toptal-commits/toptal-commits.test.js

@@ -1,4 +1,8 @@
-const { toptalCommits } = require('.')
+import { jest } from '@jest/globals'
+
+import toptalCommitsModule from './index.cjs'
+
+const { toptalCommits } = toptalCommitsModule
 
 const prepareCommitsWithMessages = (messages = []) => ({
   commits: messages.map(message => ({

package/src/configs/danger/toptal/plugins/toptal-pr-title/{index.js → index.cjs}

@@ -5,7 +5,7 @@ const {
   WHITELISTED_USERS,
   DEFAULT_PR_TITLE_ERROR_MESSAGE,
   MISSING_TICKET_CODE_ERROR_MESSAGE,
-} = require('../../config')
+} = require('../../config.cjs')
 
 const getTicketCode = text => {
   const ticketCode = text.match(VALID_PR_CODE_REGEX)

package/src/configs/danger/toptal/plugins/toptal-pr-title/toptal-pr-title.test.js

@@ -1,8 +1,10 @@
-const { toptalPRTitle } = require('.')
-const {
+import { jest } from '@jest/globals'
+
+import { toptalPRTitle } from './index.cjs'
+import {
   MISSING_TICKET_CODE_ERROR_MESSAGE,
   DEFAULT_PR_TITLE_ERROR_MESSAGE,
-} = require('../../config')
+} from '../../config.cjs'
 
 const VALID_PR_TITLE = '[ASD-123] Hello world'
 const INVALID_PR_TITLE = '[ASD-123] hello world'

package/src/{index.js → index.cjs}

@@ -1,9 +1,7 @@
-const dangerCommandCreator = require('./commands/danger')
-const toptalPlugins = require('./configs/danger/toptal/plugins')
-const conventionalCommitsPlugins = require('./configs/danger/conventional-commits/plugins')
+const toptalPlugins =  require('./configs/danger/toptal/plugins/index.cjs')
+const conventionalCommitsPlugins = require('./configs/danger/conventional-commits/plugins/index.cjs')
 
 module.exports = {
-  commands: [dangerCommandCreator],
   plugins: {
     conventionalCommit: {
       PRTitle: conventionalCommitsPlugins.conventionalPRTitle,

package/src/configs/danger/conventional-commits/dangerfile.js

@@ -1,17 +0,0 @@
-const { schedule } = require('danger')
-const { davinciProjectConfig } = require('@toptal/davinci-cli-shared')
-
-const { conventionalCommits } = require('./plugins/conventional-commits')
-const { conventionalPRTitle } = require('./plugins/conventional-pr-title')
-const { checkAssigneeExist } = require('../plugins/empty-assignee')
-
-schedule(conventionalCommits)
-schedule(conventionalPRTitle)
-
-const {
-  master: { requireAssignee },
-} = davinciProjectConfig
-
-if (requireAssignee) {
-  schedule(checkAssigneeExist)
-}

package/src/configs/danger/toptal/plugins/index.js

@@ -1,7 +0,0 @@
-const { toptalCommits } = require('./toptal-commits')
-const { toptalPRTitle } = require('./toptal-pr-title')
-
-module.exports = {
-  toptalCommits,
-  toptalPRTitle,
-}