'instrument' required, then optional? Resulting in TypeError: Cannot destructure property 'instrument' of 'options' as it is undefined.

[DonBranson]

This is for fbjs-scripts 3.0.0

fbjs/packages/fbjs-scripts/jest/createCacheKeyFunction.js

Line 35 in d8f0f43

const {instrument, config} = options;

fbjs/packages/fbjs-scripts/jest/createCacheKeyFunction.js

Line 46 in d8f0f43

.update(instrument ? 'instrument' : '')

I have a question about this code, since we're failing on const {instrument, config} = options; because instrument does not exist in options. However, are we not expecting instrument could be missing in .update(instrument ? 'instrument' : '')? If so, could it be optional when destructered from options? We've been looking for a workaround, but nothing works yet.

  return (src, file, configString, options) => {
    const {instrument, config} = options;
    const rootDir = config && config.rootDir;

    return crypto
      .createHash('md5')
      .update(globalCacheKey)
      .update('\0', 'utf8')
      .update(src)
      .update('\0', 'utf8')
      .update(rootDir ? path.relative(config.rootDir, file) : '')
      .update('\0', 'utf8')
      .update(instrument ? 'instrument' : '')
      .digest('hex');
  };
}