Skip to content

feat: require repo-scoped verification before Oz results#440

Draft
oz-for-oss[bot] wants to merge 4 commits intomainfrom
oz-agent/implement-issue-439
Draft

feat: require repo-scoped verification before Oz results#440
oz-for-oss[bot] wants to merge 4 commits intomainfrom
oz-agent/implement-issue-439

Conversation

@oz-for-oss
Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot commented May 6, 2026

Closes #439

Summary

  • Add shared repo-scoped verification instructions for webhook-dispatched Oz prompts.
  • Require review, implementation, and PR-comment response agents to verify the webhook repository/ref before final artifacts and disclose command pass/fail/skipped status.
  • Carry PR head repository/SHA metadata into review prompts so verification can target the actual PR head.
  • Add prompt tests covering multiple repository names to confirm the behavior is repo-agnostic.

Validation

  • python3 -m compileall /workspace/oz-for-oss/core /workspace/oz-for-oss/tests
  • git -C /workspace/oz-for-oss --no-pager diff --check

Notes

  • Full unittest/pytest execution was blocked in this sandbox because PyGithub is not installed and Python lacks both pip and ensurepip to install repository requirements.

Co-Authored-By: Safia Abdalla <captainsafia@users.noreply.github.com>

Co-Authored-By: Oz <oz-agent@warp.dev>
Copy link
Copy Markdown
Collaborator

@captainsafia captainsafia left a comment

Choose a reason for hiding this comment

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

@oz-agent One more tweak.

Comment thread core/workflows/repo_verification.py Outdated
@oz-for-oss
Copy link
Copy Markdown
Contributor Author

oz-for-oss Bot commented May 7, 2026

@captainsafia

I'm working on changes requested in this PR (responding to a PR review body).

You can view the conversation on Warp.

I pushed changes to this PR based on the comment.

Replied to and attempted to resolve 1 review comment that this run addressed.

Next steps:

  • Review the changes pushed to this PR.
  • Follow up with another comment if further adjustments are needed.

Powered by Oz

@oz-for-oss
Copy link
Copy Markdown
Contributor Author

oz-for-oss Bot commented May 7, 2026

@captainsafia

I'm working on changes requested in this PR (responding to a PR review body).

You can view the conversation on Warp.

I pushed changes to this PR based on the comment.

Replied to and attempted to resolve 1 review comment that this run addressed.

Next steps:

  • Review the changes pushed to this PR.
  • Follow up with another comment if further adjustments are needed.

Powered by Oz

oz-agent and others added 3 commits May 7, 2026 00:12
Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Safia Abdalla <captainsafia@users.noreply.github.com>
Co-Authored-By: Safia Abdalla <captainsafia@users.noreply.github.com>
Co-Authored-By: Oz <oz-agent@warp.dev>
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.

Run repo-scoped build and sanity verification before reporting Oz results

2 participants