Skip to content

chore: sync protected main into Podman evidence branch - #136

Closed
seonghobae wants to merge 1 commit into
feat/podman-reclaim-desktop-evidencefrom
main
Closed

chore: sync protected main into Podman evidence branch#136
seonghobae wants to merge 1 commit into
feat/podman-reclaim-desktop-evidencefrom
main

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Closed as a branch-protection test and reverse-direction pull request. Its head is main and its base is feat/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.

* 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>
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6ef1b099-0358-4b2f-83f2-00abd001a9f7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@opencode-agent opencode-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 success with required evidence or explicit no-source not-applicable evidence.

  • Regression test: Keep the approval branch checking needs.coverage-evidence.result == success before 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 head 9bcb14ff9719029411591036b249537b22d2f3fd.

  • 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"]
Loading

@opencode-agent

opencode-agent Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

OpenCode Review Overview

  • Head SHA: 9bcb14ff9719029411591036b249537b22d2f3fd
  • Workflow run: 31010943512
  • Workflow attempt: 1
  • Gate result: REQUEST_CHANGES (approval step)

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 success with required evidence or explicit no-source not-applicable evidence.

  • Regression test: Keep the approval branch checking needs.coverage-evidence.result == success before 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 head 9bcb14ff9719029411591036b249537b22d2f3fd.

  • 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"]
Loading

Merge Conflict Guidance

  • Current merge state: DIRTY
  • Base branch: feat/podman-reclaim-desktop-evidence
  • Head branch: main
  • Fix direction: merge or rebase origin/feat/podman-reclaim-desktop-evidence into main, resolve conflict markers in the changed files, rerun the focused checks, then push the same branch.
  • Repair commands:
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

@seonghobae seonghobae closed this Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant