@cratis/pi 7.110.0 → 7.112.0

package/package/corpus/harnesses/copilot/instructions/specs.csharp.instructions.md

@@ -48,7 +48,7 @@ public class when_combining_parts : Specification
 | `[Fact] void should_*()` | one assertion per fact | use the `ShouldXxx()` extension methods |
 | `void Destroy()` | teardown after each fact by default | once per eligible class with shared lifecycle opt-in |
 
-The shared lifecycle is the default on xUnit v2 and v3 since 5.0; the package registers the framework, so delete any hand-written `[assembly: Xunit.TestFramework(...)]` (it fails with `CS0579`). Eligible classes derive from `Specification`, have one public parameterless constructor, do not implement `IDisposable`, and contain only plain facts. Construction, Establish/Because and Destroy run once; all facts share the instance and must not mutate it. Initialization failure fails every fact. Theories and `ITestOutputHelper` constructors keep per-fact execution; `[PerFactSpecificationLifecycle]` opts a class out. `CRSPEC0008` (a fact modifies the shared context) is an error by default: move the change into `Establish` or `Because`. See the **cratis-specifications-csharp** skill's shared-lifecycle reference.
+The shared lifecycle is the default on xUnit v2 and v3 since 5.0; the package registers the framework, so delete any hand-written `[assembly: Xunit.TestFramework(...)]` (it fails with `CS0579`). Eligible classes derive from `Specification`, have one public parameterless constructor and contain only plain facts. Construction, Establish/Because and Destroy run once; all facts share the instance and must not mutate it. `IDisposable` specifications also share the lifecycle since Cratis.Specifications 4.4.0: one construction and one `Dispose` after the last fact. Initialization failure fails every fact. Theories and `ITestOutputHelper` constructors keep per-fact execution; `[PerFactSpecificationLifecycle]` opts a class out. `CRSPEC0008` (a fact modifies the shared context) is an error by default: move the change into `Establish` or `Because`. See the **cratis-specifications-csharp** skill's shared-lifecycle reference.
 
 Run specifications as the built executable, not `dotnet test`: restore and build once, then run `bin/<Config>/net10.0/<Project>.Specs`. In a solution that mixes Microsoft.Testing.Platform and VSTest projects, run each project on its own; `dotnet test` on the whole solution does not work there. Filters and the 4.x → 5.0 checklist are in the **cratis-specifications-csharp** skill's [running-specs reference](../../../skills/cratis-specifications-csharp/references/running-specs.md).
 

package/package/corpus/hooks/scripts/cratis-check-pr.mjs

@@ -35,12 +35,14 @@ function readBodyFile(value) {
         return readFileSync(resolved, 'utf8');
     } catch (error) {
         const parent = dirname(resolved);
+        const fileExists = existsSync(resolved);
         const details = [`--body-file value: ${value}`, `path tried: ${resolved}`,
-            `parent directory exists: ${existsSync(parent)}`, `file exists now: ${existsSync(resolved)}`,
+            `parent directory exists: ${existsSync(parent)}`, `file exists now: ${fileExists}`,
             `checker working directory: ${process.cwd()}`,
             ...Object.entries(invocation).map(([name, detail]) => `${name}: ${detail}`),
             `checker script: ${fileURLToPath(import.meta.url)}`];
-        throw new Error(`cannot read the pull-request body file (${error.code || error.message}); ${details.join('; ')}`, { cause: error });
+        const hint = fileExists ? '' : ' If the file is written in the same batch of parallel tool calls as this command, write it in a separate, earlier step; the check runs before a concurrent write completes.';
+        throw new Error(`cannot read the pull-request body file (${error.code || error.message}); ${details.join('; ')}${hint}`, { cause: error });
     }
 }
 const gh = (args, options = {}) => {

package/package/corpus/rules/specs.csharp.md

@@ -47,7 +47,7 @@ public class when_combining_parts : Specification
 | `[Fact] void should_*()` | one assertion per fact | use the `ShouldXxx()` extension methods |
 | `void Destroy()` | teardown after each fact by default | once per eligible class with shared lifecycle opt-in |
 
-The shared lifecycle is the default on xUnit v2 and v3 since 5.0; the package registers the framework, so delete any hand-written `[assembly: Xunit.TestFramework(...)]` (it fails with `CS0579`). Eligible classes derive from `Specification`, have one public parameterless constructor, do not implement `IDisposable`, and contain only plain facts. Construction, Establish/Because and Destroy run once; all facts share the instance and must not mutate it. Initialization failure fails every fact. Theories and `ITestOutputHelper` constructors keep per-fact execution; `[PerFactSpecificationLifecycle]` opts a class out. `CRSPEC0008` (a fact modifies the shared context) is an error by default: move the change into `Establish` or `Because`. See the **cratis-specifications-csharp** skill's shared-lifecycle reference.
+The shared lifecycle is the default on xUnit v2 and v3 since 5.0; the package registers the framework, so delete any hand-written `[assembly: Xunit.TestFramework(...)]` (it fails with `CS0579`). Eligible classes derive from `Specification`, have one public parameterless constructor and contain only plain facts. Construction, Establish/Because and Destroy run once; all facts share the instance and must not mutate it. `IDisposable` specifications also share the lifecycle since Cratis.Specifications 4.4.0: one construction and one `Dispose` after the last fact. Initialization failure fails every fact. Theories and `ITestOutputHelper` constructors keep per-fact execution; `[PerFactSpecificationLifecycle]` opts a class out. `CRSPEC0008` (a fact modifies the shared context) is an error by default: move the change into `Establish` or `Because`. See the **cratis-specifications-csharp** skill's shared-lifecycle reference.
 
 Run specifications as the built executable, not `dotnet test`: restore and build once, then run `bin/<Config>/net10.0/<Project>.Specs`. In a solution that mixes Microsoft.Testing.Platform and VSTest projects, run each project on its own; `dotnet test` on the whole solution does not work there. Filters and the 4.x → 5.0 checklist are in the **cratis-specifications-csharp** skill's [running-specs reference](../skills/cratis-specifications-csharp/references/running-specs.md).
 

package/package/corpus/skills/cratis-specifications-csharp/references/shared-lifecycle.md

@@ -26,10 +26,14 @@ a duplicate fails the build with `CS0579`.
 ## Eligibility
 
 A class is shared only when it derives from `Specification`, has a single public
-parameterless constructor (or only framework-managed fixture arguments), does not
-implement `IDisposable`, and has only plain facts. Theories, `ITestOutputHelper`
-constructors, custom `IAsyncLifetime` implementations, custom fact test cases and plain
-xUnit classes keep the per-fact lifecycle.
+parameterless constructor (or only framework-managed fixture arguments), and has only
+plain facts. Theories, `ITestOutputHelper` constructors, custom `IAsyncLifetime`
+implementations, custom fact test cases and plain xUnit classes keep the per-fact lifecycle.
+
+Requires Cratis.Specifications 4.4.0 or later for `IDisposable` specifications to share
+the lifecycle; earlier versions use per-fact construction. With shared lifecycle enabled,
+construction happens once and `Dispose` runs once after the last fact. Use
+`[PerFactSpecificationLifecycle]` when a fresh instance per fact is needed.
 
 `Specification.InitializeAsync()` is idempotent for a single instance. It does not make
 an ineligible class shared or cache instances across classes.

package/package/profile-catalog.json

@@ -927,7 +927,11 @@
         "cratis-application-slice-specifications",
         "cratis-chronicle-event-specifications",
         "cratis-chronicle-read-model-specifications",
-        "cratis-specifications-csharp"
+        "cratis-specifications-csharp",
+        "coverage-analysis",
+        "filter-syntax",
+        "platform-detection",
+        "run-tests"
       ],
       "languages": [
         "csharp"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cratis/pi",
-  "version": "7.110.0",
+  "version": "7.112.0",
   "description": "Configuration-aware Cratis AI integration for Pi",
   "type": "module",
   "files": [