docs: prune stale backlog sources#45
Merged
Merged
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (7)
💤 Files with no reviewable changes (4)
📝 WalkthroughWalkthroughThis PR consolidates and updates project documentation by removing PR ChangesDocumentation Consolidation and Backlog Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
| Filename | Overview |
|---|---|
| .agents/fork-sync-policy.toml | Removes the postmortem key pointing to the deleted file; the rest of the policy config is unchanged and consistent. |
| docs/README.md | Removes the postmortem link from the maintainer section and adds a backlog pointer in the agent-work section; structure is coherent. |
| docs/backlog.md | Replaces completed-history entries with two current follow-up items; updated preamble clarifies the file's lifecycle intent. |
| docs/maintainers/fork-sync-policy.md | Removes the ## Incident Reference section that linked to the now-deleted postmortem; no other content changed. |
| docs/maintainers/postmortems/pr10-fork-sync.md | Deleted historical document; all references to it have been removed from the rest of the repo. |
| docs/maintainers/security-backlog.md | Adds three new review-surface sections and clarifies partial-handled status on existing items; the Lower Priority section's IPC entry is correctly replaced by the elevated version in the main section. |
| docs/superpowers/plans/2026-05-02-upstream-sync-doc-alignment.md | Deleted completed one-time plan; no references to it remain in the repository. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[docs/README.md] -->|links to| B[docs/backlog.md]
A -->|links to| C[docs/maintainers/fork-sync-policy.md]
A -->|links to| D[docs/maintainers/security-backlog.md]
C -->|was linking to - REMOVED| E[~~postmortems/pr10-fork-sync.md~~]
F[.agents/fork-sync-policy.toml] -->|policy.sync_procedure| C
F -->|policy.canonical_inventory| G[docs/maintainers/fork-divergences.md]
F -->|postmortem key - REMOVED| E
H[~~2026-05-02-upstream-sync-doc-alignment.md~~] -->|deleted one-time plan| Z[archived]
E -->|deleted historical doc| Z
style E fill:#ffcccc,stroke:#cc0000
style H fill:#ffcccc,stroke:#cc0000
style Z fill:#f0f0f0,stroke:#999
Reviews (2): Last reviewed commit: "docs: prune stale backlog sources" | Re-trigger Greptile
There was a problem hiding this comment.
Pull request overview
This PR prunes stale, one-time documentation sources and refreshes the maintainer backlogs so they only track active follow-up work ahead of the planned migration to GitHub Issues.
Changes:
- Remove the obsolete upstream-sync doc alignment plan and the PR #10 fork-sync postmortem, including all inbound references.
- Refresh
docs/backlog.mdand wire it into the docs index as the canonical non-security follow-up list until Issues migration. - Expand and re-prioritize the security backlog to include additional review surfaces (generated-app Electron/IPC, Computer Use desktop control, bundled browser / Chrome native-host, and open-target discovery).
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
docs/superpowers/plans/2026-05-02-upstream-sync-doc-alignment.md |
Removed obsolete one-time plan now that remaining guidance is folded into maintained docs. |
docs/maintainers/postmortems/pr10-fork-sync.md |
Removed PR #10 postmortem document after pruning/relocating remaining live references. |
docs/README.md |
Drops the postmortem link and adds a pointer to the canonical non-security backlog. |
docs/backlog.md |
Replaces the stale “Recently Completed” content with a short, current non-security follow-up list and migration note. |
docs/maintainers/security-backlog.md |
Updates the workflow scope and adds missing/updated security review surfaces and priorities. |
docs/maintainers/fork-sync-policy.md |
Removes the “Incident Reference” section linking to the deleted postmortem. |
.agents/fork-sync-policy.toml |
Removes the postmortem pointer now that the postmortem doc has been deleted. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/maintainers/security-backlog.md`:
- Around line 77-80: The sentence in docs/maintainers/security-backlog.md that
mentions `main`, refreshed Nix SRI hashes, and a separate Apple DMG verification
workflow is a run-on and should be split for clarity: refactor the long sentence
into two (or three) concise sentences so each clause is its own statement—for
example, one sentence describing that the refresh updates `main` and Nix SRI
hashes, a second noting the repository maintains a separate Apple DMG
verification workflow, and a final sentence stating the remaining requirements
(machine-produced upstream version/build and Apple signature/notarization
evidence) before accepting a changed Nix trust root; update the surrounding
punctuation and conjunctions accordingly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: e3521094-72de-450a-a9d9-4056a44def98
📒 Files selected for processing (7)
.agents/fork-sync-policy.tomldocs/README.mddocs/backlog.mddocs/maintainers/fork-sync-policy.mddocs/maintainers/postmortems/pr10-fork-sync.mddocs/maintainers/security-backlog.mddocs/superpowers/plans/2026-05-02-upstream-sync-doc-alignment.md
💤 Files with no reviewable changes (4)
- docs/maintainers/postmortems/pr10-fork-sync.md
- .agents/fork-sync-policy.toml
- docs/maintainers/fork-sync-policy.md
- docs/superpowers/plans/2026-05-02-upstream-sync-doc-alignment.md
Refresh the active non-security and security backlogs, fold one-time sync docs into maintained policy surfaces, and remove obsolete plan/postmortem files. Co-authored-by: Codex <noreply@openai.com>
nisavid
force-pushed
the
nisavid/backlog-doc-cleanup
branch
from
a2e96b6 to
95dc3ec
Compare
This was referenced May 18, 2026
This was referenced Jun 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Verification
git diff --checkrg -n "2026-05-02-upstream-sync-doc-alignment|upstream-sync-doc-alignment|pr10-fork-sync|postmortems/pr10|PR #10 Fork Sync Post-Mortem|postmortem =" .rg -n "Computer Use desktop-control|Chrome native-host|generated-app Electron|open-target discovery|hash-refresh PRs|public package signing" docs/maintainers/security-backlog.md docs/maintainers/threat-model.md README.mdNotes
Docs-only cleanup; no app generation or package build required.
Summary by CodeRabbit
Documentation
Chores