oxlint-plugin-effect 0.5.0 → 0.5.1

package/README.md

@@ -37,7 +37,7 @@ If the project uses JSON configuration, copy the exported rule map into `rules`;
 | `effect/noDynamicImports`     | Allows import() only behind a named lazy-loading boundary; bans require()                 |
 | `effect/noEffectDo`           | Bans Effect.Do                                                                            |
 | `effect/noEffectBind`         | Bans Effect.bind                                                                          |
-| `effect/noGlobals`            | Bans ambient capabilities with direct Effect replacements                                 |
+| `effect/noGlobals`            | Bans ambient capabilities with direct Effect replacements; allows `process.std*.isTTY`    |
 | `effect/noNodeBuiltinImport`  | Bans fully replaced Node modules and replaced operations from partial modules             |
 
 The preset intentionally allows `Effect.as`, `Option.as`, `Effect.never`, `Effect.async`, ordinary `if` and `switch` statements, and runtime runners at explicit application boundaries.

package/dist/rules/no-globals.js

@@ -148,6 +148,16 @@ const isUnshadowedGlobal = (ctx, node, name) => Option.match(findVariableUp(ctx.
 	onSome: (variable) => variable.defs.length === 0
 });
 const staticMember = (node) => Option.getOrUndefined(memberNames(node));
+const processStreams = new Set([
+	"stderr",
+	"stdin",
+	"stdout"
+]);
+const isTtyRead = (node) => {
+	const parent = node.parent;
+	if (parent?.type !== "MemberExpression" || parent.computed) return false;
+	return parent.object === node && parent.property.type === "Identifier" && parent.property.name === "isTTY";
+};
 const isCryptoDigest = (node) => {
 	if (node.computed || node.property.type !== "Identifier" || node.property.name !== "digest") return false;
 	const object = node.object;
@@ -175,7 +185,10 @@ const noGlobals = define({
 				if (names === void 0) return Effect.void;
 				const [object, property] = names;
 				if (!isUnshadowedGlobal(ctx, memberExpression, object)) return Effect.void;
-				for (const [bannedObject, properties, alternative] of memberBans) if (object === bannedObject && properties.has(property)) return report(memberExpression, `${object}.${property}`, alternative);
+				for (const [bannedObject, properties, alternative] of memberBans) if (object === bannedObject && properties.has(property)) {
+					if (object === "process" && processStreams.has(property) && isTtyRead(memberExpression)) return Effect.void;
+					return report(memberExpression, `${object}.${property}`, alternative);
+				}
 				return Effect.void;
 			},
 			CallExpression: (node) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oxlint-plugin-effect",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "oxlint plugin with Effect-specific lint rules",
   "license": "MIT",
   "repository": {