Repository navigation
Complete NuGet package metadata - #94
Conversation
|
Packed (Release): Cratis.CritterStack.Screenplay. Nuspec verified: authors=Cratis, projectUrl=https://www.cratis.io/ (per row), MIT, readme README.md, icon logo.png, tags retained (existing integration-specific tags, per row). Shared props changed; logo.png copied from Arc. Screenplay.CritterStack.slnx Release build OK. |
CodeQL is turned off across the Cratis organization, so its workflow and the shared configuration are removed.
Remove CodeQL code scanning
Update the Cratis AI corpus
Use Specifications 5 with explicit shared lifecycle registration and net10-only spec execution while retaining all library Release target frameworks. Pass the current test cancellation token to Roslyn and preserve ordered diagnostic comparisons across assertion-library equality semantics. Method-by-method parity retains the same three baseline failures.
Build before execution, validate fresh nonempty TRX results, and retain failed results for three days. Cover runner failures with 22 regression tests and document native execution and filtering.
Match the clean package consumer's exact version expectations to the packed dependency graph without removing any checks. Replace retired nullable suffix spellings with canonical optional syntax, retaining explicit command, event, concept and query optionality assertions. All 1458 native Debug specifications pass with unchanged method and case counts.
Reference the canonical executable in both test configurations and compile its six reference-closure facts in Release. Keep all test bodies and assertions unchanged. The exact native CI Release launcher now passes all 1458 methods with the same per-method case counts and outcomes as Debug.
|
Combined candidate: Fresh postmerge checks passed: whole-solution Release build with warnings as errors, package/API validation, metadata and packed framework-asset inspection, and isolated clean-cache package-consumer restore/build/execution. Library Release compilation retains .NET 8, 9 and 10. Consumer checks retain generation, source-range, exact dependency, no-Vogen and API/module-initializer assertions. Native Debug and Release specifications each passed 1,458/1,458 at premerge migration commit Those specification results are reused, not claimed as postmerge runs: the subsequent merge changed only descriptive package metadata and the logo, leaving source/test behavior, runtime dependencies, runner/workflow and compile conditions unchanged. Fresh build/pack/consumer checks cover the changed inputs. Specifications now execute only on .NET 10; .NET 8/9-specific runtime behavior is no longer exercised, although all library targets still compile. The complete combined change passed independent whole-change review with no findings. No local integration/Docker or fresh candidate CI result is claimed; PR CI must pass before merge. |
Reviewer notes — update at 1babff3Fixes the red Canonical Critter Stack Samples verify check (also red on main): the Screenplay printer now emits |
Changed