chore: sync protected main into Podman evidence branch - #136
Conversation
* feat: bind exact human approval to cloud copies * test: stage live-clock cloud approval freshness repair * ci: validate and publish PR 125 live-clock repair * ci: make PR 125 freshness repair exact-head observable * chore: remove completed PR 125 repair workflow * chore: remove completed PR 125 repair script * chore: add one-shot PR 125 live-clock repair * chore: replace malformed one-shot PR 125 workflow * chore: add audited PR 125 live-clock repair script * chore: run audited PR 125 live-clock repair * ci: isolate PR 125 live-clock validation from publish * ci: retrigger audited PR125 live-clock repair * ci: limit PR125 repair formatting to audited files * ci: scope PR125 formatting verification to repaired files * fix: bind cloud copy approval to live mutation time * ci: repair PR125 coverage cfg mismatch * ci: repair PR 125 coverage helper * ci: trigger PR 125 coverage repair * ci: scope PR125 repair formatting to audited source * ci: supersede stale PR125 repair runs * ci: retrigger exact-head PR125 coverage repair * ci: consolidate PR125 coverage repair workflow * ci: scope PR125 repair token permissions * ci: retrigger exact-head PR 125 coverage repair * ci: publish bounded PR125 coverage repair * fix: compile cloud transfer tests under coverage * test: keep cloud approval fixture available under coverage * ci: add bounded PR 125 approval phrase repair * chore: remove stale PR 125 repair workflow * ci: retry bounded PR 125 phrase ownership repair * test: stage backend-owned approval phrase repair * ci: execute exact PR 125 backend phrase repair * ci: repair PR 125 backend approval phrase contract * ci: securely finalize PR 125 backend phrase repair * chore: remove completed PR 125 repair workflow * chore: remove completed approval phrase repair workflow * chore: remove completed PR 125 repair helper * ci: publish PR 125 backend phrase repair with final-head checks * chore: remove reintroduced completed repair workflow * docs: record cloud copy approval contract * ci: trigger exact-head backend phrase repair * test: add bounded PR 125 backend phrase finalizer * ci: finalize PR 125 backend phrase ownership * ci: allow exact-head PR 125 finalizer run * ci: trigger PR 125 finalizer on ready review * ci: harden PR 125 finalizer changed-file validation * ci: run PR 125 finalizer on exact workflow push * ci: expose exact PR 125 finalizer run * chore: remove completed PR 125 finalize workflow * chore: remove completed PR 125 finalize script * chore: remove completed PR 125 repair workflow * chore: remove completed PR 125 repair script * docs: clarify fail-closed approval phrase behavior * test: require docs for cloud copy approval APIs * chore: stage bounded cloud copy doc repair * ci: apply bounded cloud copy documentation repair * ci: authorize exact-branch one-shot repair * chore: remove superseded one-shot doc repair * ci: stage non-weakened PR 125 documentation repair * ci: run exact-head non-weakened PR 125 documentation repair * docs: restore cloud copy approval API contract * docs: correct approval phrase provenance and preserve regression evidence * ci: stage test-first backend approval phrase repair * ci: execute test-first backend approval phrase repair * ci: fix exact-file validation for PR 125 repair * feat: source cloud approval phrase from Rust plan * docs: describe backend-authored cloud approval phrase --------- Co-authored-by: opencode-agent[bot] <219766164+opencode-agent[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
There was a problem hiding this comment.
Pull request overview
OpenCode cannot approve yet because required coverage evidence did not pass.
Review outcome
1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence
-
Problem: The required coverage-evidence job result was
failure, so OpenCode cannot establish approval sufficiency for this head. -
Root cause: Automated approval is only valid when the same-head coverage-evidence job proves supported repository test suites passed and configured docstring gates passed or were advisory, or reports not applicable because no supported source files or package manifests exist. Missing, failed, skipped, unavailable, or unsupported-tooling test evidence is a blocker.
-
Fix: Install or configure the repository test/docstring evidence tooling when source files or package manifests exist, rerun the current-head coverage-evidence job, and approve only after it reports
successwith required evidence or explicit no-source not-applicable evidence. -
Regression test: Keep the approval branch checking
needs.coverage-evidence.result == successbefore posting APPROVE, and publish REQUEST_CHANGES when coverage-evidence blocker states such as cancelled, skipped, failed, unsupported-tooling, or below-100 evidence are present. -
Result: REQUEST_CHANGES
-
Reason: coverage-evidence result was
failure, so required test/docstring evidence was not proven for current head9bcb14ff9719029411591036b249537b22d2f3fd. -
Head SHA:
9bcb14ff9719029411591036b249537b22d2f3fd -
Workflow run: 31010943512
-
Workflow attempt: 1
Coverage evidence
Coverage evidence job did not run or did not publish coverage evidence.
Changed-File Evidence Map
flowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Changed file (11 files)"]
S1 --> I1["repository behavior"]
I1 --> Conflict["Merge conflict blocks this path"]
Conflict --> V1["required checks"]
OpenCode Review Overview
Pull request overviewOpenCode cannot approve yet because required coverage evidence did not pass. Review outcome1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence
Coverage evidenceCoverage evidence job did not run or did not publish coverage evidence. Changed-File Evidence Mapflowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Changed file (11 files)"]
S1 --> I1["repository behavior"]
I1 --> Conflict["Merge conflict blocks this path"]
Conflict --> V1["required checks"]
Merge Conflict Guidance
gh pr checkout 136 --repo ContextualWisdomLab/disksage
git fetch origin feat/podman-reclaim-desktop-evidence
git merge --no-ff origin/feat/podman-reclaim-desktop-evidence # or: git rebase origin/feat/podman-reclaim-desktop-evidence
git status --short
# resolve files, then git add <resolved-files>
# merge path: git commit
# rebase path: git rebase --continue
git push origin HEAD:main
# rebase path only: git push --force-with-lease origin HEAD:main |
Closed as a branch-protection test and reverse-direction pull request. Its head is
mainand its base isfeat/podman-reclaim-desktop-evidence; merging it would remove the active Podman safety-evidence changes currently under review in #133. This pull request is superseded by #133 and is not a product change.