Reconcile card editor permission and archive recovery - #3048
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8582b7ac38
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Delivery is coordinated through integration PR #3050, which preserves this PR's exact source head and commits. It qualifies all reviewed product slices against main1931da65d after the Smart CI repair; no source merge-resolution changes were required. Local source evidence remains scoped to its recorded head, and the integration's hosted checks and bounded interaction review qualify the combined delivery. Existing review feedback is triaged once; this source PR will be reconciled after the integration lands. |
1 participant
The open card editor now reconciles board permission after every confirmed write403, including assignments, card fields, comments and lifecycle actions. A single bounded server read updates all editor gates, preserves unsaved drafts, and provides an explicit retry when permission cannot be confirmed; restored Writers recover in place and Viewers remain read-only. Archive failures after Escape restore unusable focus without stealing focus from another control or card. Failed writes after mounted card switches produce a persistent generic notice.
Final head
8582b7ac3: typecheck/build, 175 focused tests, and three native Chromium focus cases pass. The combined full frontend suite passed 6,776 tests with three existing skips at8ab19e7ef, before the bounded ordinary-write callback fix; final scoped tests and independent fix-diff verification cover that fix. Relative documentation links pass. Backend permission integration and screen-reader output were not tested.Canonical delivery documentation is coordinated with the integration PR. The A-to-B-to-A pending archive busy-ownership residual remains tracked under #3033; this PR fixes its native-focus and stale-notice portions. Broader #3023 remains open.
Closes #3021
Closes #3037
Closes #3042
Refs #3033