The pin moves to rc.21, and the four Group 2 expectations move with it - #103
Closed
BTCBoyd wants to merge 2 commits into
Closed
The pin moves to rc.21, and the four Group 2 expectations move with it#103BTCBoyd wants to merge 2 commits into
BTCBoyd wants to merge 2 commits into
Conversation
ONE COMMIT, because they cannot land separately. Measured, not assumed: with the
expectations changed and the lockfile still at rc.12, verify-published-credentials
returns 1, because rc.12 cannot verify the four. An expectation is a statement
about the version being run, so it moves when the version does.
THE FOUR
verify-samples/ppp-verdict-released.json
verify-samples/ppp-verdict-denied-ceiling.json
verify-samples/ppp-probe-a-verdict-release-above-escalation.json
verify-samples/ppp-probe-b-verdict-release-above-ceiling.json
before allow: false
reasonContains "(op.evaluation.verdict.v3); the record states
construction op.evaluation.verdict.v4"
knownIssue VERDICT-PAYLOAD-VERSION-PIN
after allow: true
reasonContains "signature verifies over the payload this build
rebuilds (op.evaluation.verdict.v4)"
knownIssue removed
THE EXPECTATIONS CHANGED BECAUSE THE VERSION AT latest CHANGED, NOT BECAUSE THE
ENGINE CHANGED. The records are signed under op.evaluation.verdict.v4 and were
always sound. rc.12 rebuilds v3 and cannot verify them; rc.21 rebuilds v4 and
does. Nothing about the artifacts moved.
The reasonContains strings named both version tokens deliberately so the pin
could not move in either direction without turning this check red. It moved, the
check went red, and this is the answer to it: the control working, not a finding
about the engine.
POPULATION SEARCHED: every tracked file with dist/ excluded, grepped for the
package name adjacent to a semver, plus every data-engine-version="current" span
in tracked HTML.
scripts/package.json 1.0.0-rc.12 -> 1.0.0-rc.21
scripts/package-lock.json 1.0.0-rc.12 -> 1.0.0-rc.21
eight "current" spans verify.html 3, index.html 2, docs.html 2,
check.html 1
EIGHT, NOT THE FIVE THE EARLIER SURVEY FOUND. That survey excluded verify.html's
three as parked; they are on master now, so the population grew by exactly those
three. The difference is not a correction to the earlier count, it is the same
method over a population that changed underneath it.
A NINTH THING, WHICH NO INSTRUMENT FOUND. verify-samples/README.md said "What the
pin does, and what it does not: scripts/package.json still pins 1.0.0-rc.12 ...
and credential-expectations.json still records the four allow:false outcomes."
This commit falsifies that paragraph. Rewritten here.
sentences-at-risk.mjs was run before committing and found NOTHING, correctly and
uselessly: it keys off results/ values and this change moves a pin and eight
spans, touching no results/ file. The population it derives is not the population
this change touches, and the README paragraph is in Markdown, which it does not
read at all. Its scope is one class of change, and this is a different one.
verify.html, index.html, docs.html and check.html all moved their git date with the version spans. make-sitemap.py is one commit behind by construction, so this follows rather than rides with them. Caught by tools/hooks/pre-push, which refused the push. First time that hook has stopped a real one rather than a probe.
✅ Deploy Preview for observerprotocol ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This was referenced Aug 24, 2026
Member
Author
|
Closing in favour of #104, which carries this branch's commits unchanged. Ruling from Boyd: #101 and #103 are combined because they are one change to the world. A repository state with one landed and not the other is a state that should not exist, so they land together or not at all. The branch protection is correct and was not relaxed. The measurement that forced it: each was red for the other's reasons.
Neither could go green first, so neither could merge first. Combined, the merge is clean and the suite passes. Nothing from this branch is dropped; its commits are in #104's history. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
One commit, because they cannot land separately. Measured: with the expectations changed and the lockfile still at rc.12,
verify-published-credentialsreturns 1, because rc.12 cannot verify the four. An expectation is a statement about the version being run, so it moves when the version does.The four
allowfalsetruereasonContains(op.evaluation.verdict.v3); the record states construction op.evaluation.verdict.v4signature verifies over the payload this build rebuilds (op.evaluation.verdict.v4)knownIssueVERDICT-PAYLOAD-VERSION-PINppp-verdict-released,ppp-verdict-denied-ceiling,ppp-probe-a-verdict-release-above-escalation,ppp-probe-b-verdict-release-above-ceiling.The expectations changed because the version at
latestchanged, not because the engine changed. The records are signed under v4 and were always sound. rc.12 rebuilds v3 and cannot verify them; rc.21 rebuilds v4 and does. Nothing about the artifacts moved.The
reasonContainsstrings named both version tokens deliberately so the pin could not move in either direction without turning this check red. It moved, the check went red, and this is the answer — the control working, not a finding about the engine.Population searched
Every tracked file,
dist/excluded, grepped for the package name adjacent to a semver, plus everydata-engine-version="current"span in tracked HTML.scripts/package.jsonscripts/package-lock.jsoncurrentspansverify.html3,index.html2,docs.html2,check.html1Eight, not the five the earlier survey found. That survey excluded
verify.html's three as parked; they are on master now, so the population grew by exactly those three. Not a correction to the earlier count — the same method over a population that changed underneath it.A ninth thing, which no instrument found
verify-samples/README.mdsaid "scripts/package.jsonstill pins1.0.0-rc.12… andcredential-expectations.jsonstill records the fourallow:falseoutcomes." This commit falsifies that paragraph. Rewritten here.sentences-at-risk.mjswas run before committing and found nothing — correctly and uselessly. It keys offresults/values, and this change moves a pin and eight spans, touching noresults/file. The README paragraph is Markdown, which it does not read at all. Its scope is one class of change and this is a different one.Verified, not predicted
13/13 checks green with this branch merged into #101's content locally.
sync-engine-versiongoes from 1 to 0 — it was the only thing failing on #101.This branch alone leaves
check-measured-figuresred, for #101's reasons: the stalenpmLatestinresults/and the two section-02 markers. Those are #101's to fix and it does.The pre-push hook caught this
It refused the first push: four pages moved their git dates and the sitemap was stale. Regenerated in its own commit. First time that hook has stopped a real push rather than a probe.