@systemfsoftware/oxlint-plugin-test-discipline 4.0.1 → 5.0.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @systemfsoftware/oxlint-plugin-test-discipline
 
+## 5.0.0
+
+### Major Changes
+
+- Rules recognize effect 4.0.0 module paths: `effect/Arbitrary` (and the root `Arbitrary` name), `effect/process/ChildProcessSpawner`, and the `effect/encoding` codecs. A module is judged the same whether it is imported from `effect`, from an `effect/<area>` barrel, or by its own subpath. `schema-file-imports-pure-modules-only` admits `Base64`, `Base64Url`, `Hex` and `EncodingError` from `effect/encoding` in place of the removed `Encoding` module.
+
+- Require `effect` `^4` (stable 4.0.0). Release candidates are no longer supported: imports follow 4.0.0's module layout (`effect/http`, `effect/rpc`, `effect/http-api`, `effect/cluster`, `effect/process`, `effect/socket`, `effect/persistence`, `effect/reactivity`, `effect/encoding`, `effect/Arbitrary`), which does not exist in any 4.0.0 RC.
+
+## 4.1.0
+
+### Minor Changes
+
+- The `contract` test suffix is retired. `test-suffix-outside-src` now reports a test file named with the `contract` suffix, and the behaviour-test rules (`behaviour-test-requires-gherkin`, `behaviour-one-feature-per-file`, `behaviour-exercises-use-case`, `no-pseudo-gherkin-unit-tests`) apply to the `integration` suffix alone. Rename a suite that drives a real medium to the `integration` suffix.
+
 ## 4.0.1
 
 ### Patch Changes

package/dist/index.mjs

@@ -81,6 +81,7 @@ const SANCTIONED_TEST_DIRS = /* @__PURE__ */ new Set(["tests"]);
 const NESTED_TEST_DIR = "__tests__";
 const TEST_TREE_DIRS = /* @__PURE__ */ new Set([...SANCTIONED_TEST_DIRS, NESTED_TEST_DIR]);
 const PROPERTY_SUFFIX = ".property.test.ts";
+const INTEGRATION_SUFFIX = ".integration.test.ts";
 const DIFFERENTIAL_SUFFIX = ".differential.test.ts";
 /**
 * Forbidden outright. A schema's laws are generated, so a hand-written
@@ -152,7 +153,7 @@ const isInTestsImportScope = (filename) => {
 };
 const isInConfiguredTestDir = (filename, dirs) => directoriesOf(filename).some((segment) => dirs.includes(segment));
 const isTestFile = (basename) => TEST_BASENAME.test(basename);
-const isBehaviourBasename = (basename) => basename.endsWith(".integration.test.ts") || basename.endsWith(".contract.test.ts");
+const isBehaviourBasename = (basename) => basename.endsWith(INTEGRATION_SUFFIX);
 const CELL_SUFFIX = /\.([^.]+)\.[cm]?tsx?$/;
 const cellOf = (basename) => CELL_SUFFIX.exec(basename)?.[1];
 //#endregion
@@ -1369,7 +1370,7 @@ const IO_SPECIFIERS = {
 	"node:https": true,
 	"node:dns": true,
 	"effect/FileSystem": true,
-	"effect/unstable/process/ChildProcessSpawner": true,
+	"effect/process/ChildProcessSpawner": true,
 	"@effect/platform/FileSystem": true,
 	"@effect/platform/CommandExecutor": true,
 	"@effect/platform-node": true,
@@ -2187,7 +2188,7 @@ const dynamicEdgeOf = (declarator, init, name) => {
 };
 const vocabularyOf = (edge) => {
 	if (SCHEMA_SOURCE_PATTERN.test(edge.source)) return "schema";
-	if (edge.source.startsWith("effect/unstable/arbitrary")) return "schema";
+	if (edge.source === "effect/Arbitrary") return "schema";
 	if (edge.source === "effect" && edge.imported !== null) {
 		if (SCHEMA_NAMESPACE_NAMES[edge.imported] === true) return "schema";
 		if (FASTCHECK_NAMESPACE_NAMES[edge.imported] === true) return "handBuilt";
@@ -2219,7 +2220,7 @@ var Provenance = class {
 		if (resolveLocal$1(name, node, this.getScope).kind !== "import") return false;
 		const edge = this.imports.get(name);
 		if (edge === void 0) return false;
-		if (edge.source.startsWith("effect/unstable/arbitrary")) return true;
+		if (edge.source === "effect/Arbitrary") return true;
 		return edge.source === "effect" && edge.imported === "Arbitrary";
 	}
 	classifyCall(name, node) {
@@ -3105,9 +3106,9 @@ const testFileOutsideTestsDir = defineRule({
 });
 //#endregion
 //#region src/rules/test-suffix-outside-src.config.ts
-const UNSANCTIONED_SUFFIX_EXPECTED = "*.integration.test.ts, *.contract.test.ts, *.differential.test.ts, *.trace.test.ts, or *.conformance.test.ts";
+const UNSANCTIONED_SUFFIX_EXPECTED = "*.integration.test.ts, *.differential.test.ts, *.trace.test.ts, or *.conformance.test.ts";
 const UNSANCTIONED_SUFFIX_ACTUAL = "another test suffix outside src/";
-const UNSANCTIONED_SUFFIX_FIX = "public surface -> *.integration.test.ts; real-oracle medium -> *.contract.test.ts (contract lane); parity -> *.differential.test.ts; span graph -> *.trace.test.ts; concurrent or stateful behaviour -> *.conformance.test.ts; pure-cell property -> in-source; restated literal -> delete";
+const UNSANCTIONED_SUFFIX_FIX = "public surface or real medium -> *.integration.test.ts; parity -> *.differential.test.ts; span graph -> *.trace.test.ts; concurrent or stateful behaviour -> *.conformance.test.ts; pure-cell property -> in-source; restated literal -> delete";
 //#endregion
 //#region src/rules/test-suffix-outside-src.ts
 const testSuffixOutsideSrc = defineRule({

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@systemfsoftware/oxlint-plugin-test-discipline",
   "license": "Apache-2.0",
-  "version": "4.0.1",
+  "version": "5.0.0",
   "author": "Ryan Lee <drdgvhbh@gmail.com>",
   "repository": {
     "type": "git",
@@ -56,7 +56,7 @@
     "@systemfsoftware/vitest-config": "^0.1.0",
     "@types/node": "^26",
     "@vitest/coverage-v8": "^5",
-    "effect": "4.0.0-rc.117",
+    "effect": "^4",
     "fast-check": "^4",
     "oxlint": "~1.82.0",
     "rimraf": "^6.1.3",
@@ -65,7 +65,7 @@
     "vitest": "^5"
   },
   "peerDependencies": {
-    "effect": "4.0.0-rc.117",
+    "effect": "^4",
     "typescript": ">=5.0.0"
   },
   "publishConfig": {