@toptal/davinci-monorepo 10.0.1-alpha-PF-add-update-picasso-command-8498a7ec.4 → 10.1.0

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @toptal/davinci-monorepo
 
+## 10.1.0
+
+### Minor Changes
+
+- [#2533](https://github.com/toptal/davinci/pull/2533) [`d79f895f`](https://github.com/toptal/davinci/commit/d79f895fbf90cf41720cea37702f4a759cea9379) Thanks [@vedrani](https://github.com/vedrani)!
+  Add update-picasso command:
+  - add command to manage Picasso dependencies across all packages
+
 ## 10.0.0
 
 ### Major Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toptal/davinci-monorepo",
-  "version": "10.0.1-alpha-PF-add-update-picasso-command-8498a7ec.4+8498a7ec",
+  "version": "10.1.0",
   "keywords": [
     "lint"
   ],
@@ -27,7 +27,7 @@
   "dependencies": {
     "@nodelib/fs.walk": "^1.2.6",
     "@oclif/core": "^1.16.1",
-    "@toptal/davinci-cli-shared": "2.5.2-alpha-PF-add-update-picasso-command-8498a7ec.28+8498a7ec",
+    "@toptal/davinci-cli-shared": "^2.5.1",
     "chalk": "^4.1.2",
     "codeowners": "5.1.1",
     "dependency-cruiser": "^16.3.0",
@@ -50,6 +50,5 @@
   },
   "publishConfig": {
     "access": "public"
-  },
-  "gitHead": "8498a7ecb702af81884d1541abed64e24f4f3661"
+  }
 }

package/src/commands/__snapshots__/update-picasso.test.js.snap

@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`createUpdatePicassoCommand has the correct command structure 1`] = `
+exports[`createUpdatePicassoCommand creates a command with the correct structure 1`] = `
 "{
   "_events": {},
   "_eventsCount": 1,

package/src/commands/update-picasso.js

@@ -1,6 +1,28 @@
 import execa from 'execa'
 import { print } from '@toptal/davinci-cli-shared'
 
+const updateDependencies = async dryRun => {
+  await execa(
+    'yarn',
+    [
+      'lerna',
+      'exec',
+      '--',
+      'ncu',
+      '--',
+      '--filter="/^@toptal\\/picasso/"',
+      ...(dryRun ? [] : ['-u']),
+    ],
+    { stdio: 'inherit' }
+  )
+
+  if (!dryRun) {
+    print.cyan('Installing updated dependencies...')
+    await execa('yarn', [], { stdio: 'inherit' })
+    print.success('Successfully updated Picasso packages!')
+  }
+}
+
 /**
  * Updates Picasso dependencies across all packages in the monorepo
  * @param {Object} options - Command options
@@ -10,41 +32,12 @@ const updatePicassoCommand = async ({ dryRun }) => {
   print.header('Updating Picasso packages')
 
   try {
-    // Update Picasso dependencies using locally installed npm-check-updates
     print.cyan(
       dryRun
         ? 'Checking for Picasso updates (dry run)...'
         : 'Updating Picasso dependencies...'
     )
-
-    const { stdout: updateOutput } = await execa('yarn', [
-      'lerna',
-      'exec',
-      '--',
-      'ncu',
-      '--',
-      '--filter="/^@toptal\\/picasso/"',
-      ...(dryRun ? [] : ['-u']),
-    ])
-
-    // Log the update results
-    if (updateOutput) {
-      print.cyan('Update check results:')
-      print.cyan(updateOutput)
-    }
-
-    if (!dryRun) {
-      // Install updated dependencies
-      print.cyan('Installing updated dependencies...')
-      const { stdout: installOutput } = await execa('yarn')
-
-      if (installOutput) {
-        print.cyan('Installation details:')
-        print.cyan(installOutput)
-      }
-
-      print.success('Successfully updated Picasso packages!')
-    }
+    await updateDependencies(dryRun)
   } catch (error) {
     print.red('Failed to update Picasso packages:')
     print.red(error.message || error)

package/src/commands/update-picasso.test.js

@@ -16,13 +16,13 @@ describe('createUpdatePicassoCommand', () => {
     jest.clearAllMocks()
   })
 
-  it('has the correct command structure', () => {
+  it('creates a command with the correct structure', () => {
     const command = createUpdatePicassoCommand(program)
 
     expect(JSON.stringify(command, null, 2)).toMatchSnapshot()
   })
 
-  it('creates a command with the correct parameters', () => {
+  it('creates a command with the correct name and description', () => {
     const command = createUpdatePicassoCommand(program)
 
     expect(command._name).toBe('update-picasso')
@@ -31,7 +31,7 @@ describe('createUpdatePicassoCommand', () => {
     )
   })
 
-  it('has the dry-run option', () => {
+  it('has the dry-run option with correct defaults', () => {
     const command = createUpdatePicassoCommand(program)
     const dryRunOption = command.options.find(
       option => option.long === '--dry-run'