Skip to content

[DRAFT EVIDENCE] stacked governance-test patch from #934 - #953

Closed
groupthinking with Copilot wants to merge 15 commits into
copilot/developfrom
copilot/fix-copilot-failures
Closed

[DRAFT EVIDENCE] stacked governance-test patch from #934#953
groupthinking with Copilot wants to merge 15 commits into
copilot/developfrom
copilot/fix-copilot-failures

Conversation

Copilot AI commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Controller disposition — noncanonical draft evidence

This proposal is preserved for review but is not an executable unit. It is stacked on quarantined #934 (copilot/develop), links no focused child issue, declares issue 0, and has no trusted terminal artifact. Canonical governance remains #905 / #899. Do not merge or count this proposal as progress.


Summary

Coverage run 29933825793 failed with 3 regressions in tests/unit/test_agent_completion_gate.py while the rest of the suite was green (7753 passed).
This PR updates stale test expectations to match current pr-checks.yml behavior without changing production or workflow logic.

  • Workflow parser expectations
    • legacyRunId is now asserted as two intentional script-local declarations (scheduled sweep + collector), each validated against the same parsing contract.
  • Fallback contract behavior
    • Updated policy assertions for missing headings to reflect manifest/PR-body fallback behavior.
    • Added explicit no-manifest scenario to preserve coverage for missing_agent_login / missing_agent_run_id.
  • Scheduled identity helper alignment
    • Updated embedded JS assertions to the current scheduledContractIdentity(selectedIssueNumber, selectedIssue, pull, currentApplicable) call shape.
    • Clarified fixture naming/comments around isolated github-script scope and null PR context.
// embedded assertion fixture in test_agent_completion_gate.py
const noPrContext = null;
const identityA = scheduledContractIdentity(870, issueA, noPrContext, true);

Linked issue

Verification

  • Focused tests
  • Required CI
  • Review threads resolved

Agent provenance

The declared agent publishes a result comment on the linked issue or PR with the exact run ID and current 40-character head SHA. Replace agent-lock-event-example with agent-lock-event only when publishing real evidence.

@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 Ready Ready Preview, Comment, Open in v0 Jul 23, 2026 5:18am

@github-actions

Copy link
Copy Markdown

🔍 PR Validation

⚠️ PR title should follow conventional commits format

@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown

Agent Completion Truth Gate: BLOCKED

Reasons: invalid_payload

Machine-readable verdict
{
  "details": {
    "invalid_fields": [
      "issue.number",
      "policy.agent_login",
      "policy.run_id"
    ]
  },
  "reasons": [
    "invalid_payload"
  ],
  "verdict": "blocked"
}

Workflow evidence

@groupthinking groupthinking changed the title [WIP] Fix Copilot-related test failures [DRAFT EVIDENCE] stacked governance-test patch from #934 Jul 23, 2026
@groupthinking groupthinking added ci/cd duplicate This issue or pull request already exists security labels Jul 23, 2026 — with ChatGPT Codex Connector
Copilot AI changed the title [DRAFT EVIDENCE] stacked governance-test patch from #934 Align completion-gate workflow tests with current parser/fallback behavior Jul 23, 2026
Copilot AI requested a review from groupthinking July 23, 2026 05:18

Copy link
Copy Markdown
Owner

Controller quarantine — 2026-07-23T05:20:17.120Z

PR #953 is preserved as draft evidence, not an executable unit.

Execution receipt:

No active implementation or delegated completion is claimed.

@groupthinking groupthinking changed the title Align completion-gate workflow tests with current parser/fallback behavior [DRAFT EVIDENCE] stacked governance-test patch from #934 Jul 23, 2026
@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 duplicate This issue or pull request already exists security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants