@oclif/core 3.18.1 → 3.19.0

package/lib/config/config.d.ts

@@ -93,7 +93,7 @@ export declare class Config implements IConfig {
         force: boolean;
     }): Promise<void>;
     loadThemes(): Promise<{
-        file: string;
+        file: string | undefined;
         theme: Theme | undefined;
     }>;
     protected macosCacheDir(): string | undefined;

package/lib/config/config.js

@@ -360,12 +360,21 @@ class Config {
         }
     }
     async loadThemes() {
-        const file = (0, node_path_1.resolve)(this.configDir, 'theme.json');
-        const themes = await (0, fs_1.safeReadJson)(file);
-        const theme = themes ? (0, theme_1.parseTheme)(themes) : undefined;
+        const defaultThemeFile = this.pjson.oclif.theme
+            ? (0, node_path_1.resolve)(this.root, this.pjson.oclif.theme)
+            : this.pjson.oclif.theme;
+        const userThemeFile = (0, node_path_1.resolve)(this.configDir, 'theme.json');
+        const [defaultTheme, userTheme] = await Promise.all([
+            defaultThemeFile ? await (0, fs_1.safeReadJson)(defaultThemeFile) : undefined,
+            await (0, fs_1.safeReadJson)(userThemeFile),
+        ]);
+        // Merge the default theme with the user theme, giving the user theme precedence.
+        const merged = { ...defaultTheme, ...userTheme };
         return {
-            file,
-            theme,
+            // Point to the user file if it exists, otherwise use the default file.
+            // This doesn't really serve a purpose to anyone but removing it would be a breaking change.
+            file: userTheme ? userThemeFile : defaultThemeFile,
+            theme: Object.keys(merged).length > 0 ? (0, theme_1.parseTheme)(merged) : undefined,
         };
     }
     macosCacheDir() {

package/lib/help/command.js

@@ -150,7 +150,7 @@ class CommandHelp extends formatter_1.HelpFormatter {
                     labels.push(`--${flag.name.trim()}`);
                 }
             }
-            label = labels.join((0, theme_1.colorize)(this.config?.theme?.flagSeparator, flag.char ? ', ' : '  '));
+            label = labels.join(flag.char ? (0, theme_1.colorize)(this.config?.theme?.flagSeparator, ', ') : '  ');
         }
         if (flag.type === 'option') {
             let value = flag.helpValue || (this.opts.showFlagNameInTitle ? flag.name : '<value>');
@@ -163,14 +163,14 @@ class CommandHelp extends formatter_1.HelpFormatter {
                 value = chalk_1.default.underline(value);
             label += `=${value}`;
         }
-        return label;
+        return (0, theme_1.colorize)(this.config.theme?.flag, label);
     }
     flags(flags) {
         if (flags.length === 0)
             return;
         const noChar = flags.reduce((previous, current) => previous && current.char === undefined, true);
         return flags.map((flag) => {
-            let left = (0, theme_1.colorize)(this.config?.theme?.flag, this.flagHelpLabel(flag));
+            let left = this.flagHelpLabel(flag);
             if (noChar)
                 left = left.replace('    ', '');
             let right = flag.summary || flag.description || '';

package/lib/interfaces/pjson.d.ts

@@ -49,15 +49,11 @@ export declare namespace PJSON {
                 identifier?: string;
                 sign?: string;
             };
-            windows?: {
-                homepage?: string;
-                keypath?: string;
-                name?: string;
-            };
             plugins?: string[];
             repositoryPrefix?: string;
             schema?: number;
             state?: 'beta' | 'deprecated' | string;
+            theme?: string;
             topicSeparator?: ' ' | ':';
             topics?: {
                 [k: string]: {
@@ -78,6 +74,11 @@ export declare namespace PJSON {
                 };
                 s3: S3;
             };
+            windows?: {
+                homepage?: string;
+                keypath?: string;
+                name?: string;
+            };
         };
         version: string;
     }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@oclif/core",
   "description": "base library for oclif CLIs",
-  "version": "3.18.1",
+  "version": "3.19.0",
   "author": "Salesforce",
   "bugs": "https://github.com/oclif/core/issues",
   "dependencies": {
@@ -38,20 +38,20 @@
     "@oclif/plugin-help": "^6",
     "@oclif/plugin-plugins": "^4",
     "@oclif/prettier-config": "^0.2.1",
-    "@oclif/test": "^3.0.3",
+    "@oclif/test": "^3.1.12",
     "@types/ansi-styles": "^3.2.1",
     "@types/benchmark": "^2.1.5",
     "@types/chai": "^4.3.11",
     "@types/chai-as-promised": "^7.1.8",
     "@types/clean-stack": "^2.1.1",
-    "@types/color": "^3.0.5",
+    "@types/color": "^3.0.6",
     "@types/debug": "^4.1.10",
     "@types/ejs": "^3.1.5",
     "@types/indent-string": "^4.0.1",
     "@types/js-yaml": "^3.12.7",
     "@types/mocha": "^10.0.6",
     "@types/node": "^18",
-    "@types/node-notifier": "^8.0.2",
+    "@types/node-notifier": "^8.0.5",
     "@types/pnpapi": "^0.0.5",
     "@types/slice-ansi": "^4.0.0",
     "@types/strip-ansi": "^5.2.1",
@@ -59,14 +59,14 @@
     "@types/wordwrap": "^1.0.3",
     "@types/wrap-ansi": "^3.0.0",
     "benchmark": "^2.1.4",
-    "chai": "^4.3.10",
+    "chai": "^4.4.1",
     "chai-as-promised": "^7.1.1",
     "commitlint": "^17.7.2",
     "cross-env": "^7.0.3",
-    "eslint": "^8.50.0",
+    "eslint": "^8.56.0",
     "eslint-config-oclif": "^5.0.0",
     "eslint-config-oclif-typescript": "^3.0.35",
-    "eslint-config-prettier": "^9.0.0",
+    "eslint-config-prettier": "^9.1.0",
     "fancy-test": "^3.0.1",
     "globby": "^11.1.0",
     "husky": "^8",
@@ -77,8 +77,8 @@
     "prettier": "^3.1.1",
     "shx": "^0.3.4",
     "sinon": "^16.1.3",
-    "ts-node": "^10.9.1",
-    "tsd": "^0.29.0",
+    "ts-node": "^10.9.2",
+    "tsd": "^0.30.4",
     "typescript": "^5"
   },
   "engines": {
@@ -92,7 +92,13 @@
   ],
   "homepage": "https://github.com/oclif/core",
   "keywords": [
-    "oclif"
+    "oclif",
+    "cli",
+    "command",
+    "command line",
+    "parser",
+    "args",
+    "argv"
   ],
   "license": "MIT",
   "main": "./lib/index.js",