@toptal/davinci-monorepo 10.0.1-alpha-PF-add-update-picasso-command-2eb2da4f.2 → 10.0.1-alpha-PF-add-update-picasso-command-cd64a97a.5

package/docs/update-picasso.md

@@ -0,0 +1,55 @@
+# Update Picasso Command
+
+Updates Picasso dependencies across all packages in the monorepo.
+
+## Usage
+
+```bash
+davinci-monorepo update-picasso [options]
+```
+
+## Options
+
+- `--dry-run` - Check for updates without modifying package.json files (default: false)
+
+## Examples
+
+Check for available Picasso updates without making changes:
+
+```bash
+davinci-monorepo update-picasso --dry-run
+```
+
+Update all Picasso dependencies to their latest versions:
+
+```bash
+davinci-monorepo update-picasso
+```
+
+## Description
+
+The `update-picasso` command helps you manage Picasso dependencies across your monorepo. It:
+
+1. Uses `npm-check-updates` (ncu) to find and update Picasso package versions
+2. Updates all packages matching the pattern `@toptal/picasso/*`
+3. Automatically runs `yarn install` after updating dependencies (unless in dry-run mode)
+
+The command will display:
+
+- Which packages are being checked/updated
+- The results of the update check
+- Installation progress and results
+- Detailed error information if something goes wrong
+
+## Error Handling
+
+The command provides detailed error information including:
+
+- The error message
+- Command output (if available)
+- Error details from stderr (if available)
+
+If any part of the process fails, the command will:
+
+1. Display all available error information
+2. Exit with status code 1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toptal/davinci-monorepo",
-  "version": "10.0.1-alpha-PF-add-update-picasso-command-2eb2da4f.2+2eb2da4f",
+  "version": "10.0.1-alpha-PF-add-update-picasso-command-cd64a97a.5+cd64a97a",
   "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-2eb2da4f.26+2eb2da4f",
+    "@toptal/davinci-cli-shared": "2.5.2-alpha-PF-add-update-picasso-command-cd64a97a.29+cd64a97a",
     "chalk": "^4.1.2",
     "codeowners": "5.1.1",
     "dependency-cruiser": "^16.3.0",
@@ -51,5 +51,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "2eb2da4fbd6d2dc07dcd15a04c47728f7d22681b"
+  "gitHead": "cd64a97ac45a72c65e3eee778a0605cc2c1ae820"
 }

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

@@ -0,0 +1,65 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`createUpdatePicassoCommand creates a command with the correct structure 1`] = `
+"{
+  "_events": {},
+  "_eventsCount": 1,
+  "commands": [],
+  "options": [
+    {
+      "flags": "--dry-run",
+      "description": "Check for updates without modifying package.json files",
+      "required": false,
+      "optional": false,
+      "variadic": false,
+      "mandatory": false,
+      "long": "--dry-run",
+      "negate": false,
+      "defaultValue": false,
+      "hidden": false,
+      "conflictsWith": []
+    }
+  ],
+  "parent": null,
+  "_allowUnknownOption": false,
+  "_allowExcessArguments": true,
+  "_args": [],
+  "args": [],
+  "rawArgs": [],
+  "processedArgs": [],
+  "_scriptPath": null,
+  "_name": "update-picasso",
+  "_optionValues": {
+    "dryRun": false
+  },
+  "_optionValueSources": {
+    "dryRun": "default"
+  },
+  "_storeOptionsAsProperties": false,
+  "_executableHandler": false,
+  "_executableFile": null,
+  "_executableDir": null,
+  "_defaultCommandName": null,
+  "_exitCallback": null,
+  "_aliases": [],
+  "_combineFlagAndOptionalValue": true,
+  "_description": "Updates Picasso dependencies across all packages in the monorepo",
+  "_summary": "",
+  "_enablePositionalOptions": false,
+  "_passThroughOptions": false,
+  "_lifeCycleHooks": {},
+  "_showHelpAfterError": false,
+  "_showSuggestionAfterError": true,
+  "_outputConfiguration": {},
+  "_hidden": false,
+  "_hasHelpOption": true,
+  "_helpFlags": "-h, --help",
+  "_helpDescription": "display help for command",
+  "_helpShortFlag": "-h",
+  "_helpLongFlag": "--help",
+  "_helpCommandName": "help",
+  "_helpCommandnameAndArgs": "help [command]",
+  "_helpCommandDescription": "display help for command",
+  "_helpConfiguration": {}
+}"
+`;

package/src/commands/update-picasso.js

@@ -1,38 +1,66 @@
-import { print, runSync } from '@toptal/davinci-cli-shared'
+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
+ * @param {boolean} options.dryRun - Whether to perform a dry run
+ */
 const updatePicassoCommand = async ({ dryRun }) => {
   print.header('Updating Picasso packages')
 
   try {
-    // Update Picasso dependencies using locally installed npm-check-updates
-    print.info(
+    print.cyan(
       dryRun
         ? 'Checking for Picasso updates (dry run)...'
         : 'Updating Picasso dependencies...'
     )
-    runSync('yarn', [
-      'lerna',
-      'exec',
-      '--',
-      'npx',
-      'npm-check-updates',
-      '--filter="/^@toptal\\/picasso/"',
-      ...(dryRun ? [] : ['-u']),
-    ])
-
-    if (!dryRun) {
-      // Install updated dependencies
-      print.info('Installing updated dependencies...')
-      runSync('yarn')
-      print.success('Successfully updated Picasso packages!')
-    }
+    await updateDependencies(dryRun)
   } catch (error) {
     print.red('Failed to update Picasso packages:')
     print.red(error.message || error)
+
+    if (error.stdout) {
+      print.red('Command output:')
+      print.red(error.stdout)
+    }
+
+    if (error.stderr) {
+      print.red('Error details:')
+      print.red(error.stderr)
+    }
+
     process.exit(1)
   }
 }
 
+/**
+ * Creates the update-picasso command
+ * @param {import('commander').Command} program - Commander program instance
+ * @returns {import('commander').Command} The created command
+ */
 export const createUpdatePicassoCommand = program => {
   return program
     .createCommand('update-picasso')

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

@@ -16,7 +16,13 @@ describe('createUpdatePicassoCommand', () => {
     jest.clearAllMocks()
   })
 
-  it('creates a command with the correct parameters', () => {
+  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 name and description', () => {
     const command = createUpdatePicassoCommand(program)
 
     expect(command._name).toBe('update-picasso')
@@ -25,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'