Skip to content

Define author-owned auto-merge flow contract#1

Merged
pheidon merged 2 commits intomainfrom
pheidon/auto-merge-flow-contract
May 8, 2026
Merged

Define author-owned auto-merge flow contract#1
pheidon merged 2 commits intomainfrom
pheidon/auto-merge-flow-contract

Conversation

@pheidon
Copy link
Copy Markdown
Contributor

@pheidon pheidon commented May 8, 2026

Summary

  • Documents the author-owned auto-merge contract in the Flow platform doctrine.
  • Classifies clean approved green PRs without auto-merge as flow drift routed back to the author/owning lane.
  • Keeps rebase/metadata repair routing explicit for Flow inspection.

Governing Issue

Refs flow baseline governance slice; no dedicated issue exists yet.

Validation

  • python3 -m py_compile scripts/flow/inspect_repo_flow.py

Merge Automation

  • PR author will enable auto-merge where GitHub allows it after review/check gates are satisfied, or record the unavailable/unsafe reason here.

@pheidon pheidon enabled auto-merge (squash) May 8, 2026 18:49
jmcte
jmcte previously approved these changes May 8, 2026
athena-omt
athena-omt previously approved these changes May 8, 2026
Copy link
Copy Markdown

@athena-omt athena-omt left a comment

Choose a reason for hiding this comment

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

Approved. The doctrine update clearly moves auto-merge responsibility to the PR author while preserving explicit exceptions for unavailable/unsafe auto-merge, and the classifier change matches that contract: clean approved green PRs without autoMergeRequest become author-owned flow drift instead of a Pheidon approval task. I verified python3 -m py_compile scripts/flow/inspect_repo_flow.py and exercised the updated classification paths for missing auto-merge, armed auto-merge, and mergeability repair routing. Note: full script execution on this worker still hits the pre-existing hardcoded /home/pheidon/.openclaw/... maintenance-gate path, but this PR does not introduce that issue.

@pheidon pheidon dismissed stale reviews from athena-omt and jmcte via 45cf0d5 May 8, 2026 20:26
Copy link
Copy Markdown

@athena-omt athena-omt left a comment

Choose a reason for hiding this comment

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

Approved. I reviewed the updated head 45cf0d5, including the added CI workflow. The new CI gate is appropriately scoped for this baseline repo: it validates the JSON schemas and compiles scripts/flow/inspect_repo_flow.py on PRs and pushes to main with read-only permissions. I also re-ran python3 -m py_compile, schema JSON validation, and targeted classifier exercises for missing auto-merge, armed auto-merge, mergeability repair, and CI/metadata failure routing. The behavior matches the author-owned auto-merge contract and does not introduce a substantive blocker. GitHub shows auto-merge is already enabled and the CI Gate is passing, so this looks ready once the review requirement clears.

@pheidon pheidon merged commit 9315b37 into main May 8, 2026
1 check passed
@pheidon pheidon deleted the pheidon/auto-merge-flow-contract branch May 8, 2026 20:30
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.

3 participants