@oclif/plugin-help 3.2.1 → 3.2.2

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [3.2.2](https://github.com/oclif/plugin-help/compare/v3.2.1...v3.2.2) (2021-02-03)
+
+
+### Bug Fixes
+
+* avoid recomputing topics in loop ([#189](https://github.com/oclif/plugin-help/issues/189)) ([eccb862](https://github.com/oclif/plugin-help/commit/eccb862e725f28d305de5cccde109c15abbb5f98)), closes [#101](https://github.com/oclif/plugin-help/issues/101)
+
 ## [3.2.1](https://github.com/oclif/plugin-help/compare/v3.2.0...v3.2.1) (2020-12-14)
 
 

package/lib/command.js

@@ -1,14 +1,14 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const chalk_1 = require("chalk");
+const chalk = require("chalk");
 const indent = require("indent-string");
 const stripAnsi = require("strip-ansi");
 const list_1 = require("./list");
 const util_1 = require("./util");
-const { underline, bold, } = chalk_1.default;
-let { dim, } = chalk_1.default;
+const { underline, bold, } = chalk;
+let { dim, } = chalk;
 if (process.env.ConEmuANSI === 'ON') {
-    dim = chalk_1.default.gray;
+    dim = chalk.gray;
 }
 const wrap = require('wrap-ansi');
 class CommandHelp {

package/lib/index.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const errors_1 = require("@oclif/errors");
-const chalk_1 = require("chalk");
+const chalk = require("chalk");
 const indent = require("indent-string");
 const stripAnsi = require("strip-ansi");
 const command_1 = require("./command");
@@ -12,7 +12,7 @@ const util_1 = require("./util");
 const util_2 = require("./util");
 exports.getHelpClass = util_2.getHelpClass;
 const wrap = require('wrap-ansi');
-const { bold, } = chalk_1.default;
+const { bold, } = chalk;
 const ROOT_INDEX_CMD_ID = '';
 function getHelpSubject(args) {
     for (const arg of args) {
@@ -44,9 +44,12 @@ class Help extends HelpBase {
      * and this can be removed.
      */
     get _topics() {
-        return this.config.topics.filter((topic) => {
+        // since this.config.topics is a getter that does non-trivial work, cache it outside the filter loop for
+        // performance benefits in the presence of large numbers of topics
+        const topics = this.config.topics;
+        return topics.filter((topic) => {
             // it is assumed a topic has a child if it has children
-            const hasChild = this.config.topics.some(subTopic => subTopic.name.includes(`${topic.name}:`));
+            const hasChild = topics.some(subTopic => subTopic.name.includes(`${topic.name}:`));
             return hasChild;
         });
     }

package/lib/root.js

@@ -1,11 +1,11 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const chalk_1 = require("chalk");
+const chalk = require("chalk");
 const indent = require("indent-string");
 const stripAnsi = require("strip-ansi");
 const util_1 = require("./util");
 const wrap = require('wrap-ansi');
-const { bold, } = chalk_1.default;
+const { bold, } = chalk;
 class RootHelp {
     constructor(config, opts) {
         this.config = config;

package/oclif.manifest.json

@@ -1 +1 @@
-{"version":"3.2.1","commands":{"help":{"id":"help","description":"display help for <%= config.bin %>","pluginName":"@oclif/plugin-help","pluginType":"core","aliases":[],"flags":{"all":{"name":"all","type":"boolean","description":"see all commands in CLI","allowNo":false}},"args":[{"name":"command","description":"command to show help for","required":false}]}}}
+{"version":"3.2.2","commands":{"help":{"id":"help","description":"display help for <%= config.bin %>","pluginName":"@oclif/plugin-help","pluginType":"core","aliases":[],"flags":{"all":{"name":"all","type":"boolean","description":"see all commands in CLI","allowNo":false}},"args":[{"name":"command","description":"command to show help for","required":false}]}}}

package/package.json

@@ -1,14 +1,14 @@
 {
   "name": "@oclif/plugin-help",
   "description": "standard help for oclif",
-  "version": "3.2.1",
+  "version": "3.2.2",
   "author": "Jeff Dickey @jdxcode",
   "bugs": "https://github.com/oclif/plugin-help/issues",
   "dependencies": {
     "@oclif/command": "^1.5.20",
     "@oclif/config": "^1.15.1",
     "@oclif/errors": "^1.2.2",
-    "chalk": "^2.4.1",
+    "chalk": "^4.1.0",
     "indent-string": "^4.0.0",
     "lodash.template": "^4.4.0",
     "string-width": "^4.2.0",
@@ -35,7 +35,7 @@
     "mocha": "^8.2.1",
     "nock": "^13.0.0",
     "sinon": "^9.0.1",
-    "ts-node": "^8.8.2",
+    "ts-node": "^9.1.1",
     "typescript": "3.8.3"
   },
   "engines": {