Skip to content

Commit bef68a2

Browse files
committed
Satisfy receipt style rules without changing qualification behavior
1 parent cfe8c16 commit bef68a2

2 files changed

Lines changed: 14 additions & 17 deletions

File tree

‎tests/KeyLoad.SiteTests/Features/BenchmarkComparisons/SiteIsolatedGitHubArchiveReceipt.cs‎

Lines changed: 9 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -5,29 +5,21 @@ namespace KeyLoad.SiteTests.Features.BenchmarkComparisons;
55

66
internal sealed record SiteIsolatedGitHubArchiveFile(string Path, long Bytes, string Sha256);
77

8-
internal sealed class SiteIsolatedGitHubArchiveReceipt
8+
internal sealed class SiteIsolatedGitHubArchiveReceipt(string capture, string receiptPath,
9+
JsonObject value, SiteIsolatedGitHubArchiveFile[] files)
910
{
10-
private readonly JsonObject originalValue;
11-
private readonly SiteIsolatedGitHubArchiveFile[] originalFiles;
11+
private JsonObject OriginalValue { get; } = value.DeepClone().AsObject();
12+
private SiteIsolatedGitHubArchiveFile[] OriginalFiles { get; } = [.. files];
1213

13-
public SiteIsolatedGitHubArchiveReceipt(string capture, string receiptPath,
14-
JsonObject value, SiteIsolatedGitHubArchiveFile[] files)
15-
{
16-
Capture = capture;
17-
ReceiptPath = receiptPath;
18-
originalValue = value.DeepClone().AsObject();
19-
originalFiles = [.. files];
20-
}
21-
22-
public string Capture { get; }
14+
public string Capture { get; } = capture;
2315

24-
public string ReceiptPath { get; }
16+
public string ReceiptPath { get; } = receiptPath;
2517

26-
public JsonObject Value => originalValue.DeepClone().AsObject();
18+
public JsonObject Value => OriginalValue.DeepClone().AsObject();
2719

28-
public SiteIsolatedGitHubArchiveFile[] Files => [.. originalFiles];
20+
public SiteIsolatedGitHubArchiveFile[] Files => [.. OriginalFiles];
2921

30-
internal bool Matches(JsonNode? value) => JsonNode.DeepEquals(originalValue, value);
22+
internal bool Matches(JsonNode? value) => JsonNode.DeepEquals(OriginalValue, value);
3123
}
3224

3325
internal static class SiteIsolatedGitHubReceiptReader

‎zonetree-orleans-wal.plan.md‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,3 +127,8 @@ Root source/branch/lifetime/privacy review, normal enabled Release builds, scope
127127

128128

129129
Final prerequisite source review keeps all RPC outcomes/privacy, native Neo4j mismatch/restore/report assertions and original RF3 budgets. The enabled comparison Release build passes0warnings/0errors after sole-client ownership transfer and layered await-using scopes; the six owned files pass required formatter verification, and static governance/diff checks pass. The full dirty solution's Unit/Integration/Comparison projects build; final solution result fails only two concurrent SiteIsolatedGitHubArchiveAuthorityTests MatchesBytes diagnostics outside this scope. No local tests ran, and this is not full runtime qualification. The nine-file stable WAL gate checkpoint excludes every concurrent Site/isolated-comparison file; complete new-source GitHub build/unit/scalar/recovery/RF3 remains required, with the Series safe response diagnostic retained for any repeated failure.
130+
131+
132+
## Mechanical Site build prerequisite
133+
134+
Exactcfe8c1608/run37083535329 enabled GitHub build fails only SiteIsolatedGitHubArchiveReceipt.cs IDE0032/IDE0290; native analyzer118 gate passes and RF3 runs independently. TASK-WAL-CI-STYLE under WAL005 changes only that committed receipt class to the required primary constructor/get-only backing properties, preserving constructor parameter order, exact DeepClone/copy-on-access and Matches behavior. Root prepares an exactHEAD index patch so the active owner's different receipt/authority/MatchesBytes work stays visible and unstaged. No behavior/API/security/test assertion changes, new runtime tests or ADR: N/A for mechanical style. Source/formatter/governance review and full next-source GitHub gate remain mandatory; skipped unit/recovery/comparison after failed build are not passing evidence.

0 commit comments

Comments
 (0)