@crustjs/extensions 0.3.0 → 0.3.2

package/dist/index.d.ts

@@ -109,10 +109,11 @@ declare const colorFlags: readonly [{
  *   and hyperlinks keep following TTY detection, per
  *   [no-color.org](https://no-color.org/).
  *
- * Previous values are restored after the command finishes. When overlapping
- * programmatic runs in one process use opposite flags, the later run wins
- * mid-flight (the env is process-global); the ambient values are restored
- * once all runs finish.
+ * Previous values are restored after the command finishes. Overlapping
+ * programmatic runs in one process may share a direction (both `--color` or
+ * both `--no-color`); the ambient values are restored once all runs finish.
+ * An overlapping run with the opposing flag throws a `CrustError` in
+ * `preRun`, because the env is process-global and cannot hold both values.
  */
 export declare const noColor: ExtensionFactory<[], {}, [], typeof colorFlags>;
 //#endregion

package/dist/index.js

@@ -4,6 +4,7 @@ import { CrustError, defineCommand, defineExtension, defineExtensionId } from "@
 import { buildCommandDocumentation, formatDescription, isListed, sectionsFor } from "@crustjs/core/tooling";
 import { stripVTControlCharacters } from "node:util";
 import { bold, cyan, dim, green, padEnd, stringWidth, yellow } from "@crustjs/style";
+import { packageManagerFromUserAgent } from "@crustjs/utils/process";
 //#region src/completion/escape.ts
 /**
 * Per-shell quoting and validation helpers used by the completion
@@ -1467,6 +1468,7 @@ const didYouMean = defineExtension(DID_YOU_MEAN, (options = {}) => {
 //#region src/no-color.ts
 const NO_COLOR = defineExtensionId("crust:no-color");
 let activeRuns = 0;
+let activeFlag;
 let baseForceColor;
 let baseNoColor;
 const colorRuns = /* @__PURE__ */ new WeakSet();
@@ -1489,10 +1491,11 @@ const colorFlags = [{
 *   and hyperlinks keep following TTY detection, per
 *   [no-color.org](https://no-color.org/).
 *
-* Previous values are restored after the command finishes. When overlapping
-* programmatic runs in one process use opposite flags, the later run wins
-* mid-flight (the env is process-global); the ambient values are restored
-* once all runs finish.
+* Previous values are restored after the command finishes. Overlapping
+* programmatic runs in one process may share a direction (both `--color` or
+* both `--no-color`); the ambient values are restored once all runs finish.
+* An overlapping run with the opposing flag throws a `CrustError` in
+* `preRun`, because the env is process-global and cannot hold both values.
 */
 const noColor = defineExtension(NO_COLOR, () => ({
 	flags: colorFlags,
@@ -1500,6 +1503,12 @@ const noColor = defineExtension(NO_COLOR, () => ({
 		preRun(context) {
 			const flagValue = context.flags.color;
 			if (flagValue !== true && flagValue !== false) return;
+			if (activeRuns > 0 && activeFlag !== flagValue) throw new CrustError("DEFINITION", "noColor: cannot start a --color run while a --no-color run is in flight (or vice versa); opposing overlapping runs share process.env", {
+				subject: "extension",
+				name: NO_COLOR,
+				reason: "opposing-overlap"
+			});
+			activeFlag = flagValue;
 			if (activeRuns === 0) {
 				baseForceColor = process.env.FORCE_COLOR;
 				baseNoColor = process.env.NO_COLOR;
@@ -1519,6 +1528,7 @@ const noColor = defineExtension(NO_COLOR, () => ({
 			colorRuns.delete(context);
 			activeRuns--;
 			if (activeRuns === 0) {
+				activeFlag = void 0;
 				if (baseForceColor === void 0) delete process.env.FORCE_COLOR;
 				else process.env.FORCE_COLOR = baseForceColor;
 				if (baseNoColor === void 0) delete process.env.NO_COLOR;
@@ -1528,16 +1538,6 @@ const noColor = defineExtension(NO_COLOR, () => ({
 	}
 }));
 //#endregion
-//#region ../utils/src/process.ts
-/** Parse the package manager from npm's user-agent environment value. */
-function packageManagerFromUserAgent(userAgent) {
-	if (userAgent?.startsWith("bun")) return "bun";
-	if (userAgent?.startsWith("pnpm")) return "pnpm";
-	if (userAgent?.startsWith("yarn")) return "yarn";
-	if (userAgent?.startsWith("npm")) return "npm";
-	return null;
-}
-//#endregion
 //#region src/update-notifier.ts
 const UPDATE_NOTIFIER = defineExtensionId("crust:update-notifier");
 /** Default check interval: 24 hours. */

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@crustjs/extensions",
-	"version": "0.3.0",
+	"version": "0.3.2",
 	"description": "Official Extensions for the Crust CLI framework",
 	"type": "module",
 	"sideEffects": false,
@@ -46,17 +46,17 @@
 		"postpack": "rm -f LICENSE"
 	},
 	"dependencies": {
-		"@crustjs/store": "^0.4.0",
-		"@crustjs/style": "^0.3.2"
+		"@crustjs/store": "^0.4.1",
+		"@crustjs/style": "^0.3.3",
+		"@crustjs/utils": "^0.1.0"
 	},
 	"devDependencies": {
 		"@crustjs/config": "0.0.0",
-		"@crustjs/core": "0.3.0",
-		"@crustjs/utils": "0.0.0",
+		"@crustjs/core": "0.3.2",
 		"tsdown": "^0.23.0"
 	},
 	"peerDependencies": {
-		"@crustjs/core": "^0.3.0",
+		"@crustjs/core": "^0.3.2",
 		"typescript": "^7.0.0"
 	},
 	"peerDependenciesMeta": {