Support Yarn's workspaces being an object with a packages array

[gustavopch]

Problem

Currently, syncpack only understands the following syntax for Yarn Workspaces:

{
  "workspaces": ["packages/*"]
}

Desired behavior

But is should also support:

{
  "workspaces": {
    "packages": ["packages/*"]
  }
}

Proposed solution

AFAICT, the getYarnPatterns function in the get-wrappers.ts file should be modified to something like:

const getYarnPatterns = () => {
  const workspaces = getPatternsFromConfig('package.json', 'workspaces');
  return Array.isArray(workspaces) ? workspaces : workspaces.packages;
};

[gustavopch]

Actually, my proposed solution above won't work because getPatternsFromConfig will return null in case the property is not a non-empty array, which will be the case when workspaces is an object.

Instead, something like the following would do the job:

import get from 'lodash.get'
// ...
const getPatternsFromConfig = (fileName: string, propPath: string): string[] | null => {
  // ...
  const propValue = get(config, propPath);
  const isNonEmptyArray = config && propValue && propValue.length > 0;
  return isNonEmptyArray
    ? [process.cwd()].concat(propValue).map((dirPath: string) => join(dirPath, 'package.json'))
    : null;
};
// ...
const getYarnPatterns = () => getPatternsFromConfig('package.json', 'workspaces') || getPatternsFromConfig('package.json', 'workspaces.packages');

[JamieMason]

Released in 5.5.6