Skip to content

ci: bound open pull-request media revisions - #15861

Merged
teamleaderleo merged 1 commit into
manaflow-ai:mainfrom
teamleaderleo:fix/pr-media-retention
Sep 30, 2026
Merged

teamleaderleo merged 1 commit into
manaflow-ai:mainfrom
teamleaderleo:fix/pr-media-retention

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Follow up #15364 and implement RFC #15841 by bounding unreferenced open pull-request media revisions on pr-media.

  • Keep current-head media, media referenced by open/recent PR bodies or comments, up to three recent revision groups, and flat manual assets.
  • Preserve unknown PRs, incomplete comment pagination, unavailable history, explicit <!-- cmux:pr-media:keep --> markers, and legacy revision groups without reliable ages.
  • Keep the existing 30-day closed-PR policy, while preserving live references from open PRs and maintaining a small timestamp index across squash rewrites.
  • Keep dry-run by default and retain force-with-lease race safety.

Closes #15841.

Validation

  • python3 tests/test_ci_prune_pr_media.py
  • python3 tests/test_ci_workflow_guards_are_wired.py

Summary by cubic

Bounds unreferenced open pull-request media revisions on the pr-media branch, implementing RFC #15841. Previously only media of PRs closed over 30 days ago was pruned; now stale revision groups of open PRs are also dropped, keeping the current head, media referenced in the PR body or comments, up to three recent revision groups, and flat manual assets.

  • Preserves unknown PRs, incomplete comment pagination, unavailable history, explicit <!-- cmux:pr-media:keep --> markers, and legacy revision groups without reliable upload ages.
  • Adds a timestamp index (.cmux-pr-media-retention.json) so upload ages survive the squash that keeps the branch to one commit.
  • Keeps the 30-day closed-PR policy, dry-run default, and force-with-lease race safety.

Written for commit e1ea71e. Summary will update on new commits.

Review in cubic

@cursor

cursor Bot commented Sep 30, 2026

Copy link
Copy Markdown

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 7 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: b8ecc989-5396-4710-b794-e318f36e09ff

📥 Commits

Reviewing files that changed from the base of the PR and between 204b936 and e1ea71e.

📒 Files selected for processing (4)
  • .github/workflows/pr-media-prune.yml
  • scripts/ci/prune_pr_media.py
  • skills/cmux-testing/references/dogfood-scenarios.md
  • tests/test_ci_prune_pr_media.py

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@teamleaderleo
teamleaderleo merged commit 7ed2f6b into manaflow-ai:main Sep 30, 2026
58 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Merge receipt for e1ea71e194: every check was green at merge (12 verified; 16 skipped by policy). Full suite runs on main after merge.

rustybret pushed a commit to rustybret/bmux that referenced this pull request Sep 30, 2026
0e44675 test: bound remote bootstrap subprocess waits (manaflow-ai#15608)
a192a14 fix(agent-chat): show ACP plans as structured step lists (manaflow-ai#15889)
d7f59a3 ci: place attempt 2 like attempt 1, owned minis first (manaflow-ai#15406)
d87c3be feat(agent-chat): register Cursor Agent as an ACP provider (manaflow-ai#15877)
1bd5083 fix: preserve Codex provider for workspace auto-naming (manaflow-ai#15635)
03e1245 fix(worktree-seed): budget each pattern and refuse dangling escapes (manaflow-ai#15860)
5c28fcb fix(agent-chat): stop a disposed ACP session from resurrecting its agent (manaflow-ai#15872)
11216d2 Fix Codex Agent Chat Stop interrupt request (manaflow-ai#15837)
d6b8c15 ci: watch Unix cmux-tui installer changes (manaflow-ai#15874)
0fc35d6 feat(agent-chat): register goose as an ACP provider (manaflow-ai#15871)
7f27bfc cmux ssh: security hardening from the ssh audit (manaflow-ai#15768)
8599250 fix(agent-chat): launch gemini with --experimental-acp (manaflow-ai#15868)
849376a docs: classify contributor issue difficulty (manaflow-ai#15627)
2761cc9 Keep agents with live background work out of hibernation (manaflow-ai#15278)
eae02a6 Cloud: rebake the devbox ladder with cmux-tui 02dac3c (manaflow-ai#15866)
7ed2f6b ci: bound open pull-request media revisions (manaflow-ai#15861)
13c417c Notify on SubagentStop in the notifications hook docs (manaflow-ai#15854)
5cfc6a6 fix: make cmux-tui installs immutable across release uploads (manaflow-ai#15859)
4eee1b1 fix: preserve longest Claude upstream cooldown (manaflow-ai#15856)
204b936 Pin Cloud panes to the daemon's terminal grid (manaflow-ai#15792)
fc13b7c cmux-tui: fix the replay row scroll and stale hook fence tests breaking the full gate (manaflow-ai#15240)
87d66af Add Cloud to the menu bar extra and a main-menu Cloud menu (manaflow-ai#15822)

# Conflicts:
#	.github/workflows/ci-failure-attribution.yml
#	.github/workflows/ci-macos.yml
#	.github/workflows/ci-owned-pool-rescue.yml
#	.github/workflows/ci-queue-janitor.yml
#	.github/workflows/ci.yml
#	.github/workflows/cmux-tui-artifacts.yml
#	.github/workflows/cmux-tui-build-package.yml
#	.github/workflows/cmux-tui-sdks.yml
#	.github/workflows/pr-media-prune.yml
#	.github/workflows/remote-daemon.yml
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.

RFC: bound open pull-request media revisions

1 participant