Skip to content

[DRAFT EVIDENCE] duplicate of canonical governance PR #899 - #954

Closed
google-labs-jules[bot] wants to merge 3 commits into
mainfrom
agent/autonomous-repository-governance-11618302243648674141
Closed

[DRAFT EVIDENCE] duplicate of canonical governance PR #899#954
google-labs-jules[bot] wants to merge 3 commits into
mainfrom
agent/autonomous-repository-governance-11618302243648674141

Conversation

@google-labs-jules

@google-labs-jules google-labs-jules Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Controller disposition — noncanonical duplicate

Controller receipt


Original proposal

Re-applies governance work already owned by canonical #899 and originally claimed Fixes #905.

Re-apply the exact verified head cbae911 for PR #899:
- Add PR Governance check workflow to validate canonical issues on pr.head.sha
- Add non-destructive Repository Reconciliation loop to generate drift reports
- Introduce focused unit tests to enforce governance and reconciliation behavior
- Document workflows in AUDIT.md and README.md, and update pull_request_template.md
@google-labs-jules

Copy link
Copy Markdown
Contributor Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v0-uvai Canceled Canceled Jul 23, 2026 6:07am

@github-actions github-actions Bot added ci/cd documentation Improvements or additions to documentation python labels Jul 23, 2026
@github-actions

Copy link
Copy Markdown

🔍 PR Validation

⚠️ PR title should follow conventional commits format
⚠️ Large PR detected (570 lines changed)

@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.
See the Details below.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 0fc63fe.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

License Issues

.github/workflows/repository-reconciliation.yml

PackageVersionLicenseIssue Type
actions/github-script8.*.*NullUnknown License
Allowed Licenses: MIT, Apache-2.0, BSD-2-Clause, BSD-3-Clause, ISC, Python-2.0, BlueOak-1.0.0, MPL-2.0, CC0-1.0, 0BSD
Excluded from license check: pkg:npm/caniuse-lite, pkg:npm/@sentry/nextjs, pkg:npm/@sentry/cli, pkg:npm/@sentry/cli-darwin, pkg:npm/@sentry/cli-linux-arm, pkg:npm/@sentry/cli-linux-arm64, pkg:npm/@sentry/cli-linux-i686, pkg:npm/@sentry/cli-linux-x64, pkg:npm/@sentry/cli-win32-arm64, pkg:npm/@sentry/cli-win32-i686, pkg:npm/@sentry/cli-win32-x64, pkg:npm/@sentry/bundler-plugin-core, pkg:npm/@sentry/babel-plugin-component-annotate

OpenSSF Scorecard

PackageVersionScoreDetails
actions/actions/github-script 8.*.* 🟢 6.6
Details
CheckScoreReason
Maintained⚠️ 00 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 0
Code-Review🟢 10all changesets reviewed
Binary-Artifacts🟢 10no binaries found in the repo
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Token-Permissions🟢 9detected GitHub workflow tokens with excessive permissions
Pinned-Dependencies⚠️ 1dependency not pinned by hash detected -- score normalized to 1
License🟢 10license file detected
Fuzzing⚠️ 0project is not fuzzed
Signed-Releases⚠️ -1no releases found
Security-Policy🟢 9security policy file detected
SAST🟢 10SAST tool is run on all commits
Branch-Protection🟢 5branch protection is not maximal on development and all release branches

Scanned Files

  • .github/workflows/repository-reconciliation.yml

@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown

Agent Completion Truth Gate: BLOCKED

Reasons: evidence_collection_failed, missing_agent_result, missing_copilot_current_head_review, missing_copilot_rabbit_label, required_checks_failed, draft_pr, invalid_pr_title, focused_tests_failed

Machine-readable verdict
{
  "details": {
    "collection_errors": [
      "incomplete_linked_issue_contract",
      "missing_closing_issue_reference",
      "missing_intent_snapshot",
      "agent_run_id_mismatch",
      "agent_login_mismatch"
    ],
    "focused_test_failures": [
      "tests/unit/test_pr_governance_workflow.py",
      "tests/unit/test_repository_reconciliation_workflow.py"
    ],
    "identity_projection": {
      "agent_login": "groupthinking",
      "issue_number": 905,
      "run_id": "eventrelay-governance-repair-20260721-0215z"
    }
  },
  "reasons": [
    "evidence_collection_failed",
    "missing_agent_result",
    "missing_copilot_current_head_review",
    "missing_copilot_rabbit_label",
    "required_checks_failed",
    "draft_pr",
    "invalid_pr_title",
    "focused_tests_failed"
  ],
  "verdict": "blocked"
}

Workflow evidence

Re-apply the exact verified head cbae911 for PR #899:
- Add PR Governance check workflow to validate canonical issues on pr.head.sha
- Add non-destructive Repository Reconciliation loop to generate drift reports
- Introduce focused unit tests to enforce governance and reconciliation behavior
- Document workflows in AUDIT.md and README.md, and update pull_request_template.md

@groupthinking groupthinking left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Review — PR #954 (Activate exact-head repository governance)

Reviewed at head e1adcd7. Scope is coherent: it rewrites the PR template into a canonical-delivery contract, adds two governance workflows (pr-governance.yml, repository-reconciliation.yml) with focused unit tests, and prunes doc rows. Overall this is a sound, well-tested change.

What's good

  • Security posture is correct. Both new workflows use pull_request_target but only read PR metadata via the API (pr.body, issue/PR lists) — they never check out or execute PR-head code, and permissions are minimal (checks:write, contents:read, issues:read, pull-requests:read for governance; issues:write only for the reconciliation upsert). This avoids the classic pull_request_target code-injection trap.
  • Reconciliation logic is defensive: issue references are validated via the Issues API before forming duplicate groups, activeHeads is filtered to same-repo PRs to avoid fork branch-name collisions, drafts are excluded, and the drift report is upserted idempotently. Tests cover each of these invariants.

Finding — doc/reality drift (actionable)

The diff removes the gh-aw Validation rows from both .github/workflows/AUDIT.md and .github/workflows/README.md, but .github/workflows/gh-aw-validation.yml still exists and is active on this head (it runs on push/PR to main for *.md/*.lock.yml changes, and recent commits #924/#927 made the gh-aw canary authoritative). After this PR the workflows index would no longer list a live workflow. Please either restore those two rows, or land the workflow removal in the same PR so docs and reality stay in sync.

Merge blocker (not fixable in this PR)

The required Agent completion enforcement check is failing closed by design: per this PR's own AUDIT.md, .github/agent-lock/trusted-publishers.json ships with empty allowlists and blocks until a repository administrator provisions the dedicated GitHub App and trusted actor identities through protected review. No code change on this branch can turn that check green — it needs an admin provisioning step, then the ruleset must require the check plus one independent approval and resolved conversations. (agent-completion/truth-gate remains advisory and is re-running; core CI — test, build, coverage — was still in progress at review time.)

Recommendation: fix the gh-aw doc drift, let core CI settle, then this is a human/admin merge decision on protected main — provision the trusted App (or accept that enforcement stays red by design) before merging. Not approving automatically given it targets protected main and the enforcement gate is intentionally red.


Generated by Claude Code

@groupthinking
groupthinking marked this pull request as draft July 23, 2026 05:39
@groupthinking groupthinking added the duplicate This issue or pull request already exists label Jul 23, 2026 — with ChatGPT Codex Connector
@groupthinking groupthinking changed the title Activate exact-head repository governance (PR #899) [DRAFT EVIDENCE] duplicate of canonical governance PR #899 Jul 23, 2026
Re-apply the exact verified head cbae911 for PR #899:
- Add PR Governance check workflow to validate canonical issues on pr.head.sha
- Add non-destructive Repository Reconciliation loop to generate drift reports
- Introduce focused unit tests to enforce governance and reconciliation behavior
- Document workflows in AUDIT.md and README.md, and update pull_request_template.md
@groupthinking

Copy link
Copy Markdown
Owner

Closing as duplicate/noncanonical draft evidence — cleanup pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd documentation Improvements or additions to documentation duplicate This issue or pull request already exists python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant