From 752dbce2c65675dc0b1d165bf764976e61d1c4e1 Mon Sep 17 00:00:00 2001 From: baku-ccron Date: Fri, 21 Aug 2026 15:31:36 +0000 Subject: [PATCH] Delete the "load-bearing" filler The phrase rates a finding instead of stating one, and the reader can do the rating. Each of the 4 occurrences is either deleted -- where the sentence around it already named the consequence -- or replaced by the consequence it was standing in for. No substitute rating word was introduced. No behaviour change: comments and prose only. Co-Authored-By: Claude Opus 5 (1M context) --- src/lib/LibRainDeploySnapshot.sol | 8 ++++---- test/src/lib/GeneratedSnapshotShape.t.sol | 4 ++-- test/src/lib/LibRainDeploySnapshot.t.sol | 5 ++--- 3 files changed, 8 insertions(+), 9 deletions(-) diff --git a/src/lib/LibRainDeploySnapshot.sol b/src/lib/LibRainDeploySnapshot.sol index cb8d8d1..454d1b5 100644 --- a/src/lib/LibRainDeploySnapshot.sol +++ b/src/lib/LibRainDeploySnapshot.sol @@ -1282,7 +1282,7 @@ library LibRainDeploySnapshot { /// /// Strictly greater, so the newest tag itself is refused too: equality is /// not "follows". `SnapshotAlreadyFrozen` also refuses that one, and both - /// must hold — neither guard is load bearing alone. + /// must hold — dropping either one leaves cases the other does not refuse. /// /// The record root and the tag are parameters, so the refusal is reachable /// without a record on disk to re-cut or a `foundry.toml` to rewrite, for @@ -1307,9 +1307,9 @@ library LibRainDeploySnapshot { /// in that order, in one call. /// /// Every guard runs, and every byte that will be written is in hand, BEFORE - /// `/` is created. That ordering is load bearing rather than tidy. - /// Filesystem cheatcodes are not undone by a revert, so a throw once the - /// directory exists leaves a partial record behind — and a partial record is + /// `/` is created. Filesystem cheatcodes are not undone by a revert, + /// so a throw once the directory exists leaves a partial record behind — + /// and a partial record is /// a frozen tag, which `SnapshotAlreadyFrozen` then refuses the retry of. /// The only exit from that state is deleting a directory this design calls /// append-only, so the release is wedged by the failure rather than merely diff --git a/test/src/lib/GeneratedSnapshotShape.t.sol b/test/src/lib/GeneratedSnapshotShape.t.sol index d179db1..60168f7 100644 --- a/test/src/lib/GeneratedSnapshotShape.t.sol +++ b/test/src/lib/GeneratedSnapshotShape.t.sol @@ -177,8 +177,8 @@ contract GeneratedSnapshotShapeTest is RegistryDeploySuites, Test { /// selects is the contract the candidate is anchored to. /// /// `artifactPath` is the one field of a suite that nothing derives, and - /// until this assertion nothing checked either. It is load bearing twice - /// over: `LibRainDeploy` prints it as the `forge verify-contract` command a + /// until this assertion nothing checked either. Two things read it: + /// `LibRainDeploy` prints it as the `forge verify-contract` command a /// human runs against a freshly broadcast contract, and /// `candidateContractName` above takes the contract this whole shape spec /// is about out of it. Only the `:` half was ever read by a check — diff --git a/test/src/lib/LibRainDeploySnapshot.t.sol b/test/src/lib/LibRainDeploySnapshot.t.sol index 507cefd..26b1249 100644 --- a/test/src/lib/LibRainDeploySnapshot.t.sol +++ b/test/src/lib/LibRainDeploySnapshot.t.sol @@ -1957,9 +1957,8 @@ contract LibRainDeploySnapshotTest is Test { /// The newest tag ITSELF MUST be refused: equality is not "follows". The /// fail-safe boundary, and the one case `SnapshotAlreadyFrozen` also - /// refuses — both must hold, so neither is load bearing alone and this one - /// holds against a record root whose directories are not where a freeze - /// would look for them. + /// refuses — both must hold, and this one holds against a record root + /// whose directories are not where a freeze would look for them. function testCheckReleaseFollowsRecordRefusesTheNewestTagItself() external { writeFixture(string.concat(EQUAL_FIXTURE_ROOT, "/0_2_0/", FIXTURE_CONTRACT, ".sol"));