Finish accepted GitHub and co-work outcomes after the manual-override slice
Maintainers usually reply within 1 day
@Zamojski5 is already working on this.
Since Oct 11, 2026.
Assessment
This issue has not been assessed yet.
Description
Parent: #74. This child preserves the remaining accepted GitHub/co-work outcomes after the bounded manual-override/CAS/dormant-export slice in #379. It is tracking of existing v0.1 scope, not a new feature or a deferral. The parent stays open until every criterion below passes.
Acceptance criteria
- AC-1: Authorized project settings bind zero or more repositories using least-privilege GitHub App access. Login, installation, project integration and repo bindings are distinct. Callback state binds user/project; verify current access and selected repositories. No global current repo, automatic new-repository grant expansion or tokens in models/exports. Private repository facts require current Flux and verified repo access. Test two projects, two owners and install/binding revocation.
- AC-2: Verified many-to-many task↔PR links use binding/provider-host/stable repo/object IDs and role; support multi-repository required outputs and related-only links. Verify local CLI-created PRs before linking, preserve source timestamps and original authorship. Linking #42 from another repo/project cannot correlate incorrectly or leak metadata; same-repo bindings never share private Flux context.
- AC-3: Signed webhooks enter a durable inbox before processing; deduplicate, handle out-of-order/redelivery and reconcile gaps/current provider state without an LLM. One binding failure cannot discard another binding delivery. Retry under current permission; uncertain external writes reconcile before retry. Disconnect/outage/import preserve permitted history with truthful unavailable/stale state and no active grants or autoruns. Bridge relevant events/comments of linked PRs into #153's targeted inbox under current access and explicit subscriptions/standing policy; correlate local request and provider event to suppress echoes/duplicate reviews. Gaps reconcile known objects without global model-driven repository scans. Untrusted external comments never expand grants or arbitrarily start work.
- AC-4: Explain and enable standing automation rules for the SAME Flux task under current task authority and optimistic versions. PR/CI can move stages; closed-unmerged is not completion, merge/green checks alone cannot satisfy unmet task/device criteria, all required links matter, related-only links do not drive completion. Manual correction suspends automation until resumed. New SHA invalidates relevant review; no reassignment/audience expansion by incidental webhook. Preserve existing task statuses and separate execution/CI state.
- AC-5: Docker integration tests and a real least-privilege test installation demonstrate binding, PR/CI/review/merge, duplicates/out-of-order events, manual override, access loss and recovery at the tested head. Verify current primary GitHub API/webhook/auth requirements. Preserve required GitHub approvals/checks and distinguish Flux assessment from eligible approval. Code execution stays in local owner clients; optional broker writes need separately scoped authority. CW-4 keeps routine ready/review/fixed coordination in Flux. When required or useful to external collaborators, publish substantive GitHub review once through an eligible authorized identity and link the external review ID to its Flux assessment. Existing GitHub review threads/checks/approvals remain real gates; never substitute a Flux-only verdict.
Already integrated evidence: #168 verified linking/durable delivery; #270 disclosed G-1a rule; #379 delivers only its declared manual override/CAS/migration0068/dormant-export increment. These slices do not certify the complete criteria above.
Remaining work includes real least-privilege App installation/public HTTPS, G-1b polling, G-1c manifest setup, G-1d suggested links; #153 current-recipient bridge, causal echo suppression and eligible once-only external review; and integrated multi-project/current-access/revocation/recovery/provider/client verification at one candidate. Optional UI pins and private-source/import proposals are not silently admitted. Preserve current contracts, final Prostota UI, public permissions and all actual GitHub approval/check gates.
Owner: Zamojski5 (handoff of the remaining #74 work). Evaluator: PelikanFix16. Subsequent bounded implementation resumes within the existing WIP queue. No additional implementation is started by this tracking record.
Ownership handoff, 2026-10-11: at the user's explicit request, remaining implementation is assigned to Zamojski5; independent evaluator is PelikanFix16. Acceptance criteria and required v0.1 scope are unchanged. This queues existing work, not an extra simultaneous implementation beyond the WIP limit. See the pinned handoff comment.
- Dominant language
- TypeScript
- Stars
- 3
- Forks
- 0
- Avg merge
- 19h 9m
- Merged PRs (30d)
- 334
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ColdPhase/flux
-
Research and design full-content E2EE with OIDC, recovery and trusted devicesPossibly taken @PelikanFix16 claimed this 1 day ago. Openagent:planning enhancement
ColdPhase/flux#615 · 1 assignee ·
Maintainers usually reply within 1 day
-
Workspace AI lock: block every agent/assistant and hide all AI interactionPossibly taken @PelikanFix16 claimed this 1 day ago. Openagent:planning enhancement
ColdPhase/flux#614 · 1 assignee ·
Maintainers usually reply within 1 day
-
UI polish: fix project header more menu popover blowout, and reflect ChatGPT/Codex in Settings runtimePossibly taken @PelikanFix16 claimed this 2 days ago. Openagent:review bug enhancement
ColdPhase/flux#606 · 2 comments · 1 assignee ·
Maintainers usually reply within 1 day
-
Conversation: deduplicate 'Create task' in message menu with 'Add to map', and style cited quotesPossibly taken @PelikanFix16 claimed this 2 days ago. Openagent:review bug enhancement
ColdPhase/flux#605 · 3 comments · 1 assignee ·
Maintainers usually reply within 1 day
-
Light motion across the interface: dialogs, panel, views, state changes and press (final design §2)Possibly taken @Zamojski5 claimed this 2 days ago. Openenhancement
ColdPhase/flux#593 · 2 comments · 1 assignee ·
Maintainers usually reply within 1 day
Similar issues
-
e2e-failure ready-to-code
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
redhat-developer/rhdh-plugin-export-overlays#4460 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
Maintainers usually reply within 1 day
-
docs(api): openapi.yaml still documents gif-tools optimize colors as 2-256 instead of 2, 4, 16, 256Openarea: backend documentation priority: low
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
snapotter-hq/SnapOtter#2919 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
obsidianmd/pele#39 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
biomejs/biome#12475 · 1 comment ·
Maintainers usually reply within 1 day