@rollup/plugin-commonjs 23.0.3 → 23.0.4

package/dist/cjs/index.js

@@ -11,7 +11,7 @@ var estreeWalker = require('estree-walker');
 var MagicString = require('magic-string');
 var isReference = require('is-reference');
 
-var version = "23.0.3";
+var version = "23.0.4";
 var peerDependencies = {
 	rollup: "^2.68.0||^3.0.0"
 };

package/dist/es/index.js

@@ -7,7 +7,7 @@ import { walk } from 'estree-walker';
 import MagicString from 'magic-string';
 import isReference from 'is-reference';
 
-var version = "23.0.3";
+var version = "23.0.4";
 var peerDependencies = {
 	rollup: "^2.68.0||^3.0.0"
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rollup/plugin-commonjs",
-  "version": "23.0.3",
+  "version": "23.0.4",
   "publishConfig": {
     "access": "public"
   },

package/types/index.d.ts

@@ -1,5 +1,5 @@
-import { FilterPattern } from '@rollup/pluginutils';
-import { Plugin } from 'rollup';
+import type { FilterPattern } from '@rollup/pluginutils';
+import type { Plugin } from 'rollup';
 
 type RequireReturnsDefaultOption = boolean | 'auto' | 'preferred' | 'namespace';
 type DefaultIsModuleExportsOption = boolean | 'auto';
@@ -109,17 +109,17 @@ interface RollupCommonJSOptions {
    * they should be left unconverted as it requires an optional dependency
    * that may or may not be installed beside the rolled up package.
    * Due to the conversion of `require` to a static `import` - the call is
-   * hoisted to the top of the file, outside of the `try-catch` clause.
+   * hoisted to the top of the file, outside the `try-catch` clause.
    *
-   * - `true`: All `require` calls inside a `try` will be left unconverted.
+   * - `true`: Default. All `require` calls inside a `try` will be left unconverted.
    * - `false`: All `require` calls inside a `try` will be converted as if the
    *   `try-catch` clause is not there.
    * - `remove`: Remove all `require` calls from inside any `try` block.
    * - `string[]`: Pass an array containing the IDs to left unconverted.
-   * - `((id: string) => boolean|'remove')`: Pass a function that control
+   * - `((id: string) => boolean|'remove')`: Pass a function that controls
    *   individual IDs.
    *
-   * @default false
+   * @default true
    */
   ignoreTryCatch?:
     | boolean
@@ -133,14 +133,14 @@ interface RollupCommonJSOptions {
    * NodeJS where ES modules can only import a default export from a CommonJS
    * dependency.
    *
-   * If you set `esmExternals` to `true`, this plugins assumes that all
+   * If you set `esmExternals` to `true`, this plugin assumes that all
    * external dependencies are ES modules and respect the
    * `requireReturnsDefault` option. If that option is not set, they will be
    * rendered as namespace imports.
    *
    * You can also supply an array of ids to be treated as ES modules, or a
-   * function that will be passed each external id to determine if it is an ES
-   * module.
+   * function that will be passed each external id to determine whether it is
+   * an ES module.
    * @default false
    */
   esmExternals?: boolean | ReadonlyArray<string> | ((id: string) => boolean);
@@ -158,7 +158,7 @@ interface RollupCommonJSOptions {
    * import * as foo from 'foo';
    * ```
    *
-   * However there are some situations where this may not be desired.
+   * However, there are some situations where this may not be desired.
    * For these situations, you can change Rollup's behaviour either globally or
    * per module. To change it globally, set the `requireReturnsDefault` option
    * to one of the following values:
@@ -208,7 +208,7 @@ interface RollupCommonJSOptions {
    * Some modules contain dynamic `require` calls, or require modules that
    * contain circular dependencies, which are not handled well by static
    * imports. Including those modules as `dynamicRequireTargets` will simulate a
-   * CommonJS (NodeJS-like)  environment for them with support for dynamic
+   * CommonJS (NodeJS-like) environment for them with support for dynamic
    * dependencies. It also enables `strictRequires` for those modules.
    *
    * Note: In extreme cases, this feature may result in some paths being
@@ -222,7 +222,7 @@ interface RollupCommonJSOptions {
    * To avoid long paths when using the `dynamicRequireTargets` option, you can use this option to specify a directory
    * that is a common parent for all files that use dynamic require statements. Using a directory higher up such as `/`
    * may lead to unnecessarily long paths in the generated code and may expose directory names on your machine like your
-   * home directory name. By default it uses the current working directory.
+   * home directory name. By default, it uses the current working directory.
    */
   dynamicRequireRoot?: string;
 }