docs: plan remediation for July 12 PR review findings #101

Closed

starsetbyte wants to merge 2 commits from docs/pr-review-remediation-plan into main

starsetbyte commented

2026-07-13 01:42:28 +00:00

Owner

Summary

Implementation-ready remediation plan for the confirmed findings in PRs #96–#100.

establishes a shared account-operation epoch before the interaction/moderation fixes;
specifies OAuth successful-nonce and did:web conformance corrections;
defines deterministic race tests and full gates;
resolves the #96/#100 document conflict and defers recreated #95 research until safety work lands;
treats #11 as historical and deliberately excludes it from execution.

Plan artifact: docs/plans/2026-07-12-pr-review-remediation.md.

## Summary Implementation-ready remediation plan for the confirmed findings in PRs #96–#100. - establishes a shared account-operation epoch before the interaction/moderation fixes; - specifies OAuth successful-nonce and did:web conformance corrections; - defines deterministic race tests and full gates; - resolves the #96/#100 document conflict and defers recreated #95 research until safety work lands; - treats #11 as historical and deliberately excludes it from execution. Plan artifact: `docs/plans/2026-07-12-pr-review-remediation.md`.

starsetbyte added 1 commit

2026-07-13 01:42:28 +00:00

docs: plan PR review remediation

peregrine-ci / assembleDebug Build succeeded (37s)

Details

Peregrine CI / Build & Test (JDK 17) (pull_request) Successful in 5m12s

Details

Peregrine CI / Signed Minified Release Build (pull_request) Failing after 2m2s

Details

Peregrine CI / Instrumented Tests (API 29+) (pull_request) Has been skipped

Details

076b380576

starsetbyte added 1 commit

2026-07-13 02:16:44 +00:00

docs: strengthen account epoch remediation design

peregrine-ci / assembleDebug Build succeeded (5s)

Details

Peregrine CI / Build & Test (JDK 17) (pull_request) Successful in 4m35s

Details

Peregrine CI / Signed Minified Release Build (pull_request) Failing after 1m45s

Details

Peregrine CI / Instrumented Tests (API 29+) (pull_request) Has been skipped

Details

5d136e35dd

starsetbyte referenced this pull request from a commit

2026-07-26 03:20:15 +00:00

docs: land PR-review remediation plan as historical record (#101)

starsetbyte commented

2026-07-26 03:22:32 +00:00

Author

Owner

The plan this PR carried is now on main as docs/plans/2026-07-12-pr-review-remediation.md (412 lines), landed via #103 and diffed byte-for-byte against this branch's copy — the only difference is an added status header. That merge is what made deleting branch docs/pr-review-remediation-plan safe; before it, this branch held the only copy. Deleted now that the content is reachable from main.

Reconciled in docs/reviews/2026-07-25-branch-pr-reconciliation.md, which landed on main via #103.

The plan this PR carried is now on `main` as `docs/plans/2026-07-12-pr-review-remediation.md` (412 lines), landed via #103 and diffed byte-for-byte against this branch's copy — the only difference is an added status header. That merge is what made deleting branch `docs/pr-review-remediation-plan` safe; before it, this branch held the only copy. Deleted now that the content is reachable from `main`. Reconciled in `docs/reviews/2026-07-25-branch-pr-reconciliation.md`, which landed on `main` via #103.

starsetbyte closed this pull request

2026-07-26 03:22:32 +00:00