feat: surface authored pulls in My Work (Closes #1346)

Closes #1324

.gitea/workflows/ci.yml

@@ -57,7 +57,7 @@ jobs:
           pip install -r requirements-e2e.txt
           python3 -m playwright install --with-deps chromium
       - name: Exercise packaged mobile work journeys
-        run: python3 -m pytest tests/e2e/test_mobile_offline_issue_release.py tests/e2e/test_mobile_search_preview_navigation.py tests/e2e/test_mobile_search_week_plan.py tests/e2e/test_mobile_find_work_release.py tests/e2e/test_mobile_home_bootstrap_release.py tests/e2e/test_mobile_sign_out_release.py tests/e2e/test_mobile_today_handoff_release.py tests/e2e/test_mobile_today_wrap_up_release.py tests/e2e/test_mobile_today_summary_release.py tests/e2e/test_mobile_tomorrow_conflict_release.py tests/e2e/test_mobile_week_ahead_release.py tests/e2e/test_mobile_today_week_reschedule_release.py tests/e2e/test_mobile_wrap_up_handoff_release.py tests/e2e/test_mobile_following_release.py tests/e2e/test_mobile_detail_watch_release.py -q
+        run: python3 -m pytest tests/e2e/test_mobile_offline_issue_release.py tests/e2e/test_mobile_search_preview_navigation.py tests/e2e/test_mobile_search_week_plan.py tests/e2e/test_mobile_find_work_release.py tests/e2e/test_mobile_home_bootstrap_release.py tests/e2e/test_mobile_sign_out_release.py tests/e2e/test_mobile_today_handoff_release.py tests/e2e/test_mobile_today_wrap_up_release.py tests/e2e/test_mobile_today_summary_release.py tests/e2e/test_mobile_tomorrow_conflict_release.py tests/e2e/test_mobile_week_ahead_release.py tests/e2e/test_mobile_today_week_reschedule_release.py tests/e2e/test_mobile_wrap_up_handoff_release.py tests/e2e/test_mobile_following_release.py tests/e2e/test_mobile_detail_watch_release.py tests/e2e/test_mobile_pull_reviewer_status_release.py tests/e2e/test_mobile_pull_reviewer_feedback_release.py -q tests/e2e/test_mobile_address_review_feedback_release.py tests/e2e/test_mobile_cancel_pull_review_request_release.py tests/e2e/test_mobile_authored_pull_queue_release.py
 
   release-candidate:
     runs-on: ubuntu-latest

frontend/dashboard.css

@@ -958,11 +958,12 @@ textarea { resize: vertical; min-height: 120px; }
 .issue-sheet-actions #watch-issue-detail, .pull-sheet-actions #watch-pull-detail { min-width:0; min-height:44px; }
 .detail-watch-status:not(:empty) { margin-top:8px; overflow-wrap:anywhere; }
 .issue-sheet-actions a { border:1px solid #60a5fa; border-radius:10px; font-weight:700; }
-.issue-handoff, .pull-ownership { margin-top:14px; padding:12px; border:1px solid #2a496e; border-radius:12px; }
-.issue-handoff > div, .pull-ownership > div { display:grid; gap:8px; margin-top:10px; }
-.issue-handoff select, .pull-ownership select { width:100%; max-width:100%; padding:8px; border:1px solid #1f3a5f; border-radius:8px; background:#0b1526; color:var(--text); }
+.issue-handoff, .pull-ownership, .pull-review-request { margin-top:14px; padding:12px; border:1px solid #2a496e; border-radius:12px; }
+.issue-handoff > div, .pull-ownership > div, .pull-review-request > div { display:grid; gap:8px; margin-top:10px; }
+.issue-handoff select, .pull-ownership select, .pull-review-request select { width:100%; max-width:100%; padding:8px; border:1px solid #1f3a5f; border-radius:8px; background:#0b1526; color:var(--text); }
 .issue-handoff select, .issue-handoff button { min-height:44px; }
 .pull-ownership select, .pull-ownership button { min-height:44px; }
+.pull-review-request select, .pull-review-request button { min-height:44px; }
 .issue-retry { min-height:44px; width:100%; margin-top:10px; }
 .new-issue { min-height:44px; }
 .find-work-action { min-height:44px; }
@@ -1042,6 +1043,12 @@ textarea { resize: vertical; min-height: 120px; }
 .search-preview-comment { min-width:0; padding:10px 0; border-bottom:1px solid #1b2d45; overflow-wrap:anywhere; }
 .search-preview-comment.new-since-review { padding-left:10px; border-left:3px solid var(--accent); background:#10233a; }
 .search-preview-conversation button { min-height:44px; width:100%; }
+.search-preview-review { display:grid; gap:8px; min-width:0; padding-top:8px; border-top:1px solid #2a496e; }
+.search-preview-review[hidden] { display:none; }
+.search-preview-review h2 { margin:0; font-size:1rem; }
+.search-preview-review button { min-height:44px; width:100%; }
+.search-preview-file { min-width:0; margin:8px 0; padding:10px; border:1px solid #203a5c; border-radius:10px; overflow:hidden; }
+.search-preview-file > strong, .search-preview-file > .small { display:block; overflow-wrap:anywhere; }
 .search-preview-reply { display:grid; gap:8px; padding-top:8px; border-top:1px solid #2a496e; }
 .search-preview-reply[hidden] { display:none; }
 .search-preview-reply h2 { margin:0; font-size:1rem; }
@@ -1056,8 +1063,10 @@ textarea { resize: vertical; min-height: 120px; }
 #keep-following-status { display:block; min-height:20px; overflow-wrap:anywhere; }
 .search-preview-actions button, .search-preview-actions a { min-height:44px; box-sizing:border-box; display:flex; align-items:center; justify-content:center; }
 .search-preview-primary-actions { display:grid; grid-template-columns:repeat(2,minmax(0,1fr)); gap:8px; }
+.search-preview-primary-actions.following-disposition-mode { grid-template-columns:repeat(3,minmax(0,1fr)); }
 @media (max-width:420px) {
   .search-preview-primary-actions { grid-template-columns:1fr; }
+  .search-preview-primary-actions.following-disposition-mode { grid-template-columns:1fr; }
 }
 @media (max-width:600px) {
   .search-preview-panel { width:100%; border-left:0; padding:14px; padding-top:max(12px,env(safe-area-inset-top)); }
@@ -1081,7 +1090,7 @@ textarea { resize: vertical; min-height: 120px; }
     padding-bottom:calc(6px + env(safe-area-inset-bottom));
     background:rgba(11,21,38,.98); border-block:1px solid #2a496e;
   }
-  #search-preview-overview, #search-preview-conversation,
+  #search-preview-overview, #search-preview-conversation, #search-preview-review,
   #search-preview-reply-workspace, #search-preview-actions { scroll-margin-top:72px; }
   #search-preview-actions { position:static; }
 }
@@ -1250,6 +1259,34 @@ textarea { resize: vertical; min-height: 120px; }
 .pull-diff-empty { margin:8px 0; padding:10px; border:1px dashed #4e6b8a; border-radius:8px; }
 .pull-review-tools { display:flex; align-items:center; justify-content:space-between; gap:8px; margin:8px 0; }
 .pull-review-tools button { min-height:44px; }
+.pull-reviewer-summary { margin:10px 0; padding:10px; border:1px solid #203a5c; border-radius:10px; }
+.pull-reviewer-summary h3 { margin:0 0 6px; font-size:14px; }
+.pull-reviewer-status { padding:8px 0; border-top:1px solid #203a5c; overflow-wrap:anywhere; }
+.pull-reviewer-status:first-child { border-top:0; }
+.pull-reviewer-status-heading { display:flex; justify-content:space-between; gap:10px; }
+.pull-reviewer-status-heading span { text-align:right; }
+.cancel-review-request { min-height:44px; width:100%; margin-top:8px; }
+.pull-review-feedback { margin-top:8px; border:1px solid #2a496e; border-radius:8px; padding:0 8px 8px; }
+.pull-review-feedback > summary { min-height:44px; display:flex; align-items:center; cursor:pointer; }
+.pull-review-feedback-summary { margin:4px 0 10px; white-space:pre-wrap; }
+.pull-review-feedback-file { margin-top:8px; padding-top:8px; border-top:1px solid #203a5c; }
+.pull-review-feedback-file h4 { margin:0 0 4px; overflow-wrap:anywhere; }
+.pull-review-feedback-file ul { margin:0; padding-left:20px; }
+.pull-review-feedback-file li { margin:6px 0; }
+.pull-review-feedback-file small { display:block; color:#93c5fd; }
+.pull-review-feedback-file button { min-height:44px; width:100%; margin-top:4px; }
+.address-review-feedback, [data-load-review-feedback] { min-height:44px; width:100%; margin-top:8px; }
+.pull-feedback-pass { margin-top:10px; padding:10px; border:1px solid #3b82f6; border-radius:10px; overflow-wrap:anywhere; }
+.pull-feedback-pass-heading { display:flex; align-items:start; justify-content:space-between; gap:8px; }
+.pull-feedback-pass-heading h4 { margin:2px 0 8px; }
+.pull-feedback-pass p { white-space:pre-wrap; }
+.pull-feedback-pass textarea { width:100%; min-height:88px; box-sizing:border-box; }
+.pull-feedback-dispositions { display:grid; grid-template-columns:1fr; gap:6px; margin:10px 0; padding:8px; }
+.pull-feedback-dispositions button[aria-pressed="true"] { border-color:#60a5fa; background:#173b63; }
+.pull-feedback-navigation { display:grid; grid-template-columns:1fr 1fr; gap:8px; margin-top:8px; }
+.pull-feedback-pass button { min-height:44px; }
+#finish-pull-feedback, #request-feedback-review { width:100%; margin-top:8px; }
+#request-updated-pull-review { min-height:44px; width:100%; margin-top:8px; }
 .pull-review { margin-top:14px; overflow:hidden; border:1px solid #2a496e; border-radius:10px; padding:0 10px 10px; }
 .pull-review summary { min-height:44px; display:flex; align-items:center; cursor:pointer; }
 .pull-review summary h2 { margin:0; font-size:16px; }

frontend/dashboard.js

@@ -3,12 +3,12 @@
   const qs = (s, el=document) => el.querySelector(s);
   const announceWork = message => qs('#my-work-action-status').textContent = message;
   const fmt = (d) => new Date(d).toLocaleString();
-  const mobileAppBadge = createMobileAppBadge({
+  const appBadge = createMobileAppBadge({
     control:qs('#app-badge-control'), status:qs('#app-badge-status'),
     container:qs('#app-badge-setting'), navigator, storage:localStorage,
     serviceWorker:navigator.serviceWorker,
   });
-  mobileAppBadge.start();
+  appBadge.start();
   const cardPlanning = createCardPlanning(document);
   const mobileComposerViewport = createMobileComposerViewport({
     viewport: window.visualViewport,
@@ -103,21 +103,22 @@
     const result = await (authored ? authoredOutbox : issueOutbox).retry(item.outbox_id, activeFlushLogin);
     (authored ? applyAuthoredOutboxResult : applyOutboxResult)(result);
   }
-  let mobileQueueCounts = {};
-  let mobilePreparationItems = {};
+  let queueCounts = {};
+  let preparationItems = {};
   const followingQueue = attachFollowing(item => {
     searchPreviewReturnKind = 'following';
     return searchPreview.open(item);
   }, {
     onCount:(count, items) => {
-      mobileQueueCounts.following = count;
-      mobileQueueCounts.followingUnavailable = false;
-      mobilePreparationItems.following = items.filter(item => item.has_unseen_change === true);
+      appBadge.reconcile('following', count, true);
+      queueCounts.following = count;
+      queueCounts.followingUnavailable = false;
+      preparationItems.following = items.filter(item => item.has_unseen_change === true);
       mobileStartDay.render();
     },
     onStatus:status => {
       if (status === 'loading') return;
-      mobileQueueCounts.followingUnavailable = status === 'error';
+      queueCounts.followingUnavailable = status === 'error';
       mobileStartDay.render();
     },
     onReviewComplete:() => mobileStartDay.completePhase('following'),
@@ -166,7 +167,7 @@
     selectFilter: selectMobileQueue,
     firstAction: name => qs('#my-work-list .my-work-card-main, #my-work-list .draft-resume, #my-work-list .draft-continue, #my-work-list .draft-edit'),
     announce: announceWork,
-    getCounts: () => mobileQueueCounts,
+    getCounts: () => queueCounts,
     openFindWork: () => qs('#find-work').click(),
   });
   function openMobileStartDay() {
@@ -201,14 +202,14 @@
     findWork: () => qs('#find-work').click(),
   });
   const mobileStartDay = createMobileStartDay({
-    getCounts: () => mobileQueueCounts,
-    getPhaseItems: () => mobilePreparationItems,
+    getCounts: () => queueCounts,
+    getPhaseItems: () => preparationItems,
     getLogin: () => confirmedOwnerLogin,
     openQueue: name => {
       const sheet = qs('#mobile-queue-sheet');
       if (sheet.open) sheet.close();
       return name === 'find' ? qs('#find-work').click() :
-        name === 'following' ? (mobileQueueCounts.followingUnavailable ? followingQueue.open() :
+        name === 'following' ? (queueCounts.followingUnavailable ? followingQueue.open() :
           followingQueue.review()) : mobileQueueLauncher.open(name);
     },
     onHandoff: current => {
@@ -327,7 +328,7 @@
   let launchFilterResolved = false;
   try {
     const savedFilter = sessionStorage.getItem(WORK_FILTER_KEY);
-    if (['all', 'today', 'agenda', 'attention', 'filed', 'issue', 'pull', 'review', 'update', 'later', 'draft'].includes(savedFilter)) {
+    if (['all', 'today', 'agenda', 'attention', 'filed', 'authored', 'issue', 'pull', 'review', 'update', 'later', 'draft'].includes(savedFilter)) {
       selectedWorkFilter = savedFilter;
       savedWorkFilter = savedFilter;
       launchFilterResolved = true;
@@ -1286,7 +1287,8 @@
           if (continuing) return await completeOwnershipExitToday(item);
           paintMyWork(lastContextSnapshot);
           return false;
-        });
+        }, ()=>selectedPullDetail, ()=>confirmedOwnerLogin);
+
       }
       attachReleaseReceipt();
       if (!reviewController) reviewController = createReviewController({ fetchJson: fetchReviewJson, storage: localStorage });
@@ -1526,7 +1528,7 @@
     },
     onPagination: pagination => {
       notificationPagination = pagination;
-      mobileAppBadge.reconcile(pagination.total, {authoritative:true});
+      appBadge.reconcile('updates', pagination.total, true);
       const loaded = Math.min(pagination.total, pagination.page * 50);
       qs('#notification-page-status').textContent = pagination.total ?
         loaded + ' of ' + pagination.total + ' unread updates loaded.' : '';
@@ -3429,7 +3431,7 @@
     counts.later = laterMyWork.length;
     counts.draft = lastDrafts.length;
     counts.delivery = draftInbox.partition(lastDrafts).actionable;
-    mobilePreparationItems = {
+    preparationItems = {
       delivery:draftInbox.partition(lastDrafts).deliveries,
       agenda:agendaMyWork(activeMyWork),
       attention:activeMyWork.filter(item => item.needs_attention),
@@ -3450,7 +3452,7 @@
       const element = qs('[data-work-count="' + filter + '"]');
       if (element) element.textContent = count;
     });
-    mobileQueueCounts = counts;
+    queueCounts = counts;
     mobileStartDay.reconcile({
       authoritative:authoritativeMyWorkRefresh,
       authoritativePhases:['delivery'],
@@ -3496,10 +3498,10 @@
     if (selectedWorkFilter === 'agenda') return ['issue'];
     if (selectedWorkFilter === 'attention') return ['issue', 'pull', 'review'];
     if (selectedWorkFilter === 'filed') return ['filed'];
-    if (selectedWorkFilter === 'issue') return ['issue'];
-    if (selectedWorkFilter === 'pull') return ['pull'];
-    if (selectedWorkFilter === 'review') return ['review'];
-    if (selectedWorkFilter === 'all') return ['issue', 'filed', 'pull', 'review'];
+    if (['issue', 'pull', 'review', 'authored'].includes(selectedWorkFilter)) {
+      return [selectedWorkFilter];
+    }
+    if (selectedWorkFilter === 'all') return ['issue', 'filed', 'pull', 'review', 'authored'];
     return [];
   }
 
@@ -3745,7 +3747,7 @@
   }
 
   function updateWorkPaginationControls() {
-    const labels = { issue: 'issues', filed: 'filed issues', pull: 'pull requests', review: 'review requests' };
+    const labels = { issue: 'issues', filed: 'filed issues', pull: 'pull requests', review: 'review requests', authored: 'pull requests' };
     const streams = activeWorkStreams();
     const incomplete = streams.filter(stream => workPagination[stream]?.has_more);
     const summaries = streams.flatMap(stream => {
@@ -4535,7 +4537,7 @@
     qs('#pull-review-progress').textContent = pullReviewState.total ?
       pullReviewState.reviewed.length + ' of ' + pullReviewState.total + ' files reviewed' : 'No changed files to review';
     qs('#next-unreviewed-pull-file').disabled = pullReviewState.complete;
-    const eligibility = createPullSheet.mergeEligibility(detail, pullReviewState);
+    const eligibility = createPullSheet.review(detail, pullReviewState, document);
     qs('#pull-merge-state').textContent = eligibility.reason;
     qs('#merge-pull').disabled = !eligibility.allowed;
     qs('#pull-files').innerHTML = (detail.files || []).length ? detail.files.map((file, index) =>
@@ -4572,9 +4574,7 @@
       return;
     }
     selectedPullDetail = { ...selectedPullDetail, ...status };
-    const eligibility = createPullSheet.mergeEligibility(selectedPullDetail, pullReviewState);
-    qs('#pull-merge-state').textContent = eligibility.reason;
-    qs('#merge-pull').disabled = !eligibility.allowed;
+    renderPullReview(selectedPullDetail);
     qs('#pull-review-status').textContent = 'Checks refreshed for the current head.';
   }
 
@@ -4631,6 +4631,7 @@
     qs('#pull-sheet-title').textContent = item.title || 'Assigned pull request';
     qs('#pull-sheet-status').textContent = 'Loading pull request…';
     qs('#pull-sheet-body').textContent = '';
+
     qs('#pull-files').textContent = '';
     qs('#pull-review').open = false;
     qs('#pull-review-status').textContent = 'Expand to load changed files and merge readiness.';
@@ -4646,6 +4647,7 @@
     qs('#pull-checks-summary').textContent = 'Not loaded';
     qs('#pull-check-list').textContent = '';
     qs('#pull-checks').open = false;
+
     qs('#pull-merge-state').textContent = 'Review data not loaded';
     qs('#merge-pull').disabled = true;
     qs('#merge-pull').textContent = workSession.active() ? 'Merge & next' : 'Merge';
@@ -4658,9 +4660,11 @@
       const detail = offlineDetail || await pullController.load(item);
       if (selectedPull !== item) return;
       selectedPullDetail = detail;
+      pullController.setReviewDetail(detail);
       pullConversation = pullController.conversation(item, detail.conversation);
       qs('#pull-sheet-title').textContent = detail.title || 'Assigned pull request';
       qs('#pull-sheet-body').innerHTML = renderMarkdown(detail.body || 'No description provided.');
+
       renderPullConversation(pullConversation.snapshot());
       qs('#open-pull-gitea').href = detail.url || item.url || '#';
       qs('#pull-sheet-status').textContent = 'Pull request ready · by ' + (detail.author || 'unknown author');
@@ -5692,7 +5696,7 @@
     watchButton.hidden = true;
     watchButton.disabled = false;
     shareButton.disabled = true;
-    renderSearchPreviewReply(state, null, searchPreview, document);
+    renderSearchPreviewWorkspaces(state,null,searchPreview,document,escapeHtml);
 
     if (state.status === 'loading') {
       searchReplyAttachmentTarget = { ...state.item };
@@ -5715,6 +5719,7 @@
     const detail = state.detail;
     if (!detail) return;
     searchPreviewDetail = detail;
+
     shareButton.disabled = state.status === 'sharing';
     qs('#search-preview-key').textContent = detail.repository + ' #' + detail.number;
     qs('#search-preview-title').textContent = detail.title || 'Untitled work item';
@@ -5725,7 +5730,7 @@
       (detail.assignees?.length ? ' · assigned to ' + detail.assignees.join(', ') : '');
     qs('#search-preview-body').innerHTML = renderMarkdown(detail.body || 'No description provided.');
     renderSearchConversation(state.conversation);
-    renderSearchPreviewReply(state, detail, searchPreview, document);
+    renderSearchPreviewWorkspaces(state,detail,searchPreview,document,escapeHtml);
     qs('#open-search-result-gitea').href = safeSearchUrl(detail.url) || '#';
     claimButton.hidden = !(detail.claimable || detail.assigned_to_me);
     claimButton.textContent = detail.assigned_to_me ? 'Open in My Work' : 'Assign to me';
@@ -5746,6 +5751,7 @@
       (detail.assigned_to_me ? 'Start in Today' : 'Assign & start');
     startButton.disabled = state.status === 'claiming' || state.status === 'reopening';
     renderSearchPreviewWatch(detail, state, watchButton);
+    followingQueue.preview(state);
     const shareStatus = {
       sharing:'Opening share options…', shared:'Search result shared.', copied:'Search result link copied.',
       'share-canceled':'Share canceled.', 'share-error':'Could not share this result. Try again.',
@@ -5763,13 +5769,10 @@
   const searchSubscription = searchPreviewSubscriptionOptions(fetchReviewJson);
   const searchPreview = createSearchPreview({
     fetchJson:searchSubscription.preview,
-    fetchConversation:(item,page)=>fetchReviewJson(searchPreviewConversationPath(item,page),{
-      headers:{Accept:'application/json'},
-    }),
-    mutate: (detail, action) => fetchReviewJson(
-      'api/v1/repos/' + detail.repository.split('/').map(encodeURIComponent).join('/') +
-      '/issues/' + encodeURIComponent(detail.number) + '/' + action,
-      { method:'PATCH', headers:{ Accept:'application/json' } }
+    fetchConversation:(item,page)=>fetchReviewJson(searchPreviewConversationPath(item,page)),
+    fetchReview:searchSubscription.review,
+    mutate:(detail,action)=>fetchReviewJson(
+      searchPreviewPath(detail).replace(/\?.*$/, '') + '/' + action, {method:'PATCH'}
     ),
     watch:(detail,watching) => searchSubscription.watch(detail, watching).then(result =>
       followingQueue.load().catch(() => {}).then(() => result)),
@@ -5816,7 +5819,7 @@
   createSearchWeekPlanUI({planner:searchWeekPlan,document,window,escapeHtml,escapeAttribute:escAttr,
     getDetail:()=>searchPreviewDetail,
     announce:message=>{qs('#search-preview-status').textContent=message;},
-    onPlanned:item=>{searchPreviewDetail={...item,assigned_to_me:true,claimable:false};},
+    onPlanned:item=>(searchPreviewDetail=item,item.following&&next()),
   });
   searchBatchPlanning = mountSearchBatchPlanning(
     document, createBatchFindWork, todayWork, ()=>planningOwnerLogin, fetchReviewJson,
@@ -7242,6 +7245,7 @@
     if (event.currentTarget.open) loadPullReview();
   });
   qs('#pull-review-retry').addEventListener('click', loadPullReview);
+
   qs('#refresh-pull-checks').addEventListener('click', async () => {
     if (!selectedPull || !selectedPullDetail?.head_sha) return;
     const item = selectedPull;
@@ -8384,8 +8388,8 @@
       protectStorage:() => deviceStorage.requestPersistence(),
       notificationReadiness:() => pushController?.notificationReadiness()
         || {state:'unavailable', detail:'Update notifications are unavailable.'},
-      appBadgeReadiness:() => mobileAppBadge.readiness(),
-      enableAppBadge:() => mobileAppBadge.enable(),
+      appBadgeReadiness:() => appBadge.readiness(),
+      enableAppBadge:() => appBadge.enable(),
       enablePush:async () => {
         const controller = await pushControllerReady;
         if (!controller) return;

frontend/following.js

@@ -3,11 +3,33 @@
   if (typeof module === 'object' && module.exports) {
     module.exports = exports.createFollowing;
     module.exports.attachFollowing = exports.attachFollowing;
+    module.exports.planningDisposition = exports.planningDisposition;
   } else {
     root.createFollowing = exports.createFollowing;
     root.attachFollowing = exports.attachFollowing;
+    root.followingPlanningDisposition = exports.planningDisposition;
   }
 })(typeof self !== 'undefined' ? self : this, function () {
+  function planningDisposition(detail, controls) {
+    const active = detail?.following === true && detail.kind === 'issue' && detail.state === 'open' &&
+      Boolean(detail.claimable || detail.assigned_to_me);
+    const result = active ? {active:true,today:'Add to Today & next',later:'Later & next',week:'Week Ahead & next'} : {active:false};
+    if (!controls && globalThis.document) {
+      const query = selector => globalThis.document.querySelector(selector);
+      controls = {root:query('.search-preview-primary-actions'),claim:query('#claim-search-result'),
+        today:query('#queue-search-result'),later:query('#defer-search-result'),week:query('#plan-search-result'),
+        start:query('#start-search-result'),watch:query('#watch-search-result')};
+    }
+    controls?.root?.classList.toggle('following-disposition-mode', active);
+    if (active && controls) {
+      controls.claim.hidden = controls.start.hidden = controls.watch.hidden = true;
+      controls.today.textContent = result.today;
+      controls.later.textContent = result.later;
+      controls.week.textContent = result.week;
+    }
+    return result;
+  }
+
   function createFollowing(options) {
     let generation = 0;
     let snapshot = {revision:0, items:[]};
@@ -248,7 +270,8 @@
       keepForLater:feature.keepForLater,
       retire:feature.retire,
       preview(state) {
-        keepButton.hidden = state?.item?.following !== true;
+        const disposition = planningDisposition(state?.detail || state?.item);
+        keepButton.hidden = state?.item?.following !== true || disposition.active;
         keepButton.disabled = false;
         if (keepButton.hidden || state.status === 'loading') query('#keep-following-status').textContent = '';
       },
@@ -265,5 +288,5 @@
     };
   }
 
-  return {createFollowing, attachFollowing};
+  return {createFollowing, attachFollowing, planningDisposition};
 });

frontend/index.html

@@ -197,6 +197,7 @@
             <button class="work-filter" data-work-filter="agenda" aria-pressed="false">Agenda <span data-work-count="agenda">0</span></button>
             <button class="work-filter" data-work-filter="attention" aria-pressed="false">Attention <span data-work-count="attention">0</span></button>
             <button class="work-filter" data-work-filter="filed" aria-pressed="false">Filed <span data-work-count="filed">0</span></button>
+            <button class="work-filter" data-work-filter="authored" aria-pressed="false">My PRs <span data-work-count="authored">0</span></button>
             <button class="work-filter" data-work-filter="issue" aria-pressed="false">Issues <span data-work-count="issue">0</span></button>
             <button class="work-filter" data-work-filter="pull" aria-pressed="false">PRs <span data-work-count="pull">0</span></button>
             <button class="work-filter" data-work-filter="review" aria-pressed="false">Reviews <span data-work-count="review">0</span></button>
@@ -215,7 +216,7 @@
             <label for="keep-work-offline"><input id="keep-work-offline" type="checkbox" /> Keep My Work available offline</label>
             <label for="delivery-receipts"><input id="delivery-receipts" type="checkbox" /> Notify me when queued work finishes</label>
             <span class="app-badge-setting" id="app-badge-setting">
-              <label class="app-badge-control" for="app-badge-control"><input id="app-badge-control" type="checkbox" /> Show unread Updates on the app icon</label>
+              <label class="app-badge-control" for="app-badge-control"><input id="app-badge-control" type="checkbox" /> Show review attention on the app icon</label>
               <span class="small" id="app-badge-status" role="status" aria-live="polite"></span>
             </span>
             <label class="push-update-control" for="push-updates"><input id="push-updates" type="checkbox" /> Notify me about new updates</label>
@@ -886,6 +887,7 @@
     <nav class="mobile-search-preview-nav" aria-label="Search preview sections">
       <button type="button" data-search-preview-section="overview">Overview</button>
       <button type="button" data-search-preview-section="conversation">Conversation</button>
+      <button type="button" data-search-preview-section="changes" hidden>Changes</button>
       <button type="button" data-search-preview-section="reply">Reply</button>
       <button type="button" data-search-preview-section="actions">Act</button>
     </nav>
@@ -901,6 +903,12 @@
       <button id="retry-search-preview-conversation" type="button" hidden>Retry conversation</button>
       <button id="load-older-search-preview-comments" type="button" hidden>Load older messages</button>
     </section>
+    <section id="search-preview-review" class="search-preview-review" aria-labelledby="search-preview-review-title" hidden>
+      <h2 id="search-preview-review-title">Changes</h2>
+      <div id="search-preview-review-status" class="small" aria-live="polite"></div>
+      <div id="search-preview-files"></div>
+      <button id="retry-search-preview-review" type="button" hidden>Retry changes</button>
+    </section>
     <section class="search-preview-reply" id="search-preview-reply-workspace" aria-labelledby="search-preview-reply-title" hidden>
       <h2 id="search-preview-reply-title">Reply</h2>
       <textarea id="search-preview-reply" maxlength="10000" placeholder="Write a reply"></textarea>
@@ -1644,6 +1652,16 @@
       <div id="pull-sheet-status" class="small" aria-live="polite">Choose a pull request.</div>
       <button class="pull-retry" id="retry-pull-load" type="button" hidden>Retry loading pull request</button>
       <div class="pull-sheet-content markdown-content" id="pull-sheet-body"></div>
+      <button id="edit-pull-content" type="button" hidden>Edit review context</button>
+      <form class="issue-edit-form" id="pull-edit-form" hidden>
+        <label>Title<input id="pull-edit-title" maxlength="255" required /></label>
+        <label>Description<textarea id="pull-edit-body" maxlength="10000"></textarea></label>
+        <div class="issue-edit-actions">
+          <button id="save-pull-content" type="submit">Save context</button>
+          <button id="cancel-pull-edit" type="button">Cancel</button>
+        </div>
+        <div id="pull-edit-status" class="small" role="status" aria-live="assertive"></div>
+      </form>
     </div>
     <section id="pull-conversation" tabindex="-1">
       <h2>Full conversation</h2><div id="pull-comments"></div>
@@ -1704,6 +1722,36 @@
       <div id="pull-review-status" class="small" aria-live="polite">Expand to load changed files and merge readiness.</div>
       <button class="pull-retry" id="pull-review-retry" type="button" hidden>Retry review data</button>
       <div class="row"><span class="pill" id="pull-ci-state">CI unknown</span><span class="pill" id="pull-merge-state">Review data not loaded</span></div>
+      <section class="pull-reviewer-summary" aria-labelledby="pull-reviewer-heading">
+        <h3 id="pull-reviewer-heading">Reviewer status</h3>
+        <div id="pull-reviewer-status" class="small" aria-live="polite">Reviewer status not loaded.</div>
+        <div id="pull-reviewer-statuses"></div>
+        <button id="request-updated-pull-review" type="button" hidden>Request updated review</button>
+        <section class="pull-feedback-pass" id="pull-feedback-pass" aria-labelledby="pull-feedback-heading" hidden>
+          <div class="pull-feedback-pass-heading">
+            <div><span class="small" id="pull-feedback-progress"></span><h4 id="pull-feedback-heading">Address review feedback</h4></div>
+            <button id="close-pull-feedback" type="button">Close</button>
+          </div>
+          <p class="small" id="pull-feedback-location"></p>
+          <p id="pull-feedback-body"></p>
+          <div class="small" id="pull-feedback-context" aria-live="polite"></div>
+          <fieldset class="pull-feedback-dispositions">
+            <legend>Disposition</legend>
+            <button type="button" data-feedback-disposition="addressed">Addressed</button>
+            <button type="button" data-feedback-disposition="discussion">Needs discussion</button>
+            <button type="button" data-feedback-disposition="skipped">Skip for now</button>
+          </fieldset>
+          <label for="pull-feedback-note">Response note (optional)</label>
+          <textarea id="pull-feedback-note" maxlength="1000" placeholder="What changed or needs discussion?"></textarea>
+          <div class="pull-feedback-navigation">
+            <button id="previous-pull-feedback" type="button">Previous</button>
+            <button id="next-pull-feedback" type="button">Next</button>
+          </div>
+          <button id="finish-pull-feedback" type="button" disabled>Post feedback response</button>
+          <button id="request-feedback-review" type="button" hidden>Request updated review from this reviewer</button>
+          <div id="pull-feedback-status" class="small" role="status" aria-live="assertive"></div>
+        </section>
+      </section>
       <details class="ci-checks" id="pull-checks">
         <summary><span>Checks</span><span class="small" id="pull-checks-summary">Not loaded</span></summary>
         <button class="ci-check-refresh" id="refresh-pull-checks" type="button">Refresh checks</button>
@@ -1713,6 +1761,18 @@
       <div id="pull-files"></div>
       <button id="merge-pull" type="button" disabled>Merge</button>
     </details>
+    <details class="pull-review-request" id="pull-review-request">
+      <summary>Request review</summary>
+      <div>
+        <button id="publish-draft-pull" type="button" hidden>Ready for review</button>
+        <div id="publish-draft-pull-status" class="small" aria-live="assertive"></div>
+        <button id="load-pull-reviewers" type="button">Choose reviewer</button>
+        <label for="pull-review-recipient" class="small">Eligible repository reviewer</label>
+        <select id="pull-review-recipient" disabled><option value="">Select a teammate</option></select>
+        <button id="confirm-pull-review" type="button" disabled>Request review</button>
+        <div id="pull-review-request-status" class="small" aria-live="assertive">Load teammates to request a review.</div>
+      </div>
+    </details>
     <details class="pull-ownership" id="pull-ownership">
       <summary>Ownership</summary>
       <div>
@@ -2026,6 +2086,7 @@
       <button data-mobile-queue="update" type="button" aria-label="Updates, 0 unread conversations"><span><strong>Updates</strong><small>Unread conversations</small></span><span data-mobile-queue-count="update">0</span></button>
       <button data-mobile-queue="following" type="button" aria-label="Following, 0 unseen changes"><span><strong>Following</strong><small>Issues and pull requests you watch</small></span><span data-mobile-queue-count="following">0</span></button>
       <button data-mobile-queue="filed" type="button"><span><strong>Filed</strong><small>Issues you delegated</small></span><span data-mobile-queue-count="filed">0</span></button>
+      <button data-mobile-queue="authored" type="button"><span><strong>My PRs</strong><small>Pull requests you authored</small></span><span data-mobile-queue-count="authored">0</span></button>
       <button data-mobile-queue="later" type="button"><span><strong>Later</strong><small>Deferred work</small></span><span data-mobile-queue-count="later">0</span></button>
       <button data-mobile-queue="draft" type="button"><span><strong>Drafts</strong><small>Unfiled captures</small></span><span data-mobile-queue-count="draft">0</span></button>
       <button data-mobile-queue="find" type="button"><span><strong>Find Work</strong><small>Claim something new</small></span></button>

frontend/mobile-app-badge.js

@@ -11,9 +11,10 @@
 }) {
   const ENABLED_KEY = 'stackchain.app-badge.enabled.v1';
   let enabled = false;
-  let confirmedCount = 0;
+  const confirmedCounts = {updates:0, following:0};
   let renderedCount = null;
 
+
   function available() {
     return typeof navigator?.setAppBadge === 'function'
       && typeof navigator?.clearAppBadge === 'function';
@@ -29,7 +30,16 @@
     }
   }
 
+  async function syncCount(channel, count) {
+    if (!enabled) return;
+    try {
+      const registration = await serviceWorker?.ready;
+      registration?.active?.postMessage({type:'stackchain-app-badge-count', channel, count});
+    } catch (_error) { /* A later refresh can retry. */ }
+  }
+
   async function render() {
+    const confirmedCount = Math.min(9999, confirmedCounts.updates + confirmedCounts.following);
     if (!enabled || !available() || renderedCount === confirmedCount) return true;
     try {
       if (confirmedCount > 0) await navigator.setAppBadge(confirmedCount);
@@ -48,7 +58,13 @@
     if (enabled) storage?.setItem(ENABLED_KEY, 'true');
     else storage?.removeItem(ENABLED_KEY);
     await syncPreference();
+    if (enabled) {
+      if (confirmedCounts.updates > 0) await syncCount('updates', confirmedCounts.updates);
+      if (confirmedCounts.following > 0) await syncCount('following', confirmedCounts.following);
+    }
     if (!enabled && available()) {
+      confirmedCounts.updates = 0;
+      confirmedCounts.following = 0;
       try {
         await navigator.clearAppBadge();
         renderedCount = null;
@@ -82,8 +98,8 @@
   function readiness() {
     if (!available()) return {state:'unavailable', detail:'App icon badges are unavailable in this browser.'};
     return enabled
-      ? {state:'complete', detail:'The app icon shows the confirmed unread Updates count.'}
-      : {state:'incomplete', detail:'Show the confirmed unread Updates count without opening Stackchain.'};
+      ? {state:'complete', detail:'The app icon shows review attention.'}
+      : {state:'incomplete', detail:'Show review attention without opening Stackchain.'};
   }
 
   async function enable() {
@@ -93,9 +109,11 @@
     return true;
   }
 
-  async function reconcile(count, {authoritative = false} = {}) {
-    if (!authoritative || !Number.isSafeInteger(count) || count < 0) return false;
-    confirmedCount = count;
+  async function reconcile(channel, count, authoritative = false) {
+    if (!Object.hasOwn(confirmedCounts, channel)
+        || authoritative !== true || !Number.isSafeInteger(count) || count < 0 || count > 9999) return false;
+    confirmedCounts[channel] = count;
+    await syncCount(channel, count);
     return render();
   }
 

frontend/mobile-search-preview-nav.js

@@ -98,12 +98,14 @@ function attachMobileSearchPreviewNavigation({document, window}) {
     buttons:{
       overview:bySection('overview'),
       conversation:bySection('conversation'),
+      changes:bySection('changes'),
       reply:bySection('reply'),
       actions:bySection('actions'),
     },
     targets:{
       overview:document.getElementById('search-preview-overview'),
       conversation:document.getElementById('search-preview-conversation'),
+      changes:document.getElementById('search-preview-review'),
       reply,
       actions:document.getElementById('search-preview-actions'),
     },

frontend/my-work.js

@@ -71,6 +71,7 @@ function buildMyWork(data, now = new Date()) {
     const assigned = (item.assignees || []).includes(login);
     const isReview = (item.work_reasons || []).includes('review_requested');
     const isFiled = (item.work_reasons || []).includes('created_by_me');
+    const isAuthored = (item.work_reasons || []).includes('authored_by_me');
     const isCompleted = isFiled && item.state === 'closed';
     const due = item.kind === 'issue' ? issueDueState(item.due_date, now) : null;
     const normalized = {
@@ -78,6 +79,7 @@ function buildMyWork(data, now = new Date()) {
       key: (item.repository || 'unknown') + '#' + item.number,
       is_review: isReview,
       is_filed: isFiled,
+      is_authored: isAuthored,
       is_completed: isCompleted,
       is_assigned: assigned,
       has_update: false,
@@ -85,7 +87,8 @@ function buildMyWork(data, now = new Date()) {
       reason: priorityLabel ? priorityLabel + ' priority' :
         (due && due.priority < 4 ? due.label :
           (isReview ? 'Needs your review' : (isCompleted ? 'Completed · review outcome' :
-            (assigned ? 'Assigned to you' : (isFiled ? 'Filed by you' : 'Open work'))))),
+            (assigned ? 'Assigned to you' : (isFiled ? 'Filed by you' :
+              (isAuthored ? 'Authored by you' : 'Open work')))))),
       _priority: priorityLabel ? 0 :
         (due && due.priority < 4 ? due.priority : (isReview ? 3 : (isCompleted ? 3.5 : (assigned ? 4 : 5)))),
     };
@@ -657,6 +660,7 @@ function filterMyWork(items, selectedFilter, selectedMilestone = 'all') {
   let filtered = items;
   if (selectedFilter === 'attention') filtered = items.filter(needsAttention);
   else if (selectedFilter === 'filed') filtered = items.filter((item) => item.is_filed);
+  else if (selectedFilter === 'authored') filtered = items.filter((item) => item.is_authored);
   else if (selectedFilter === 'review') filtered = items.filter((item) => item.is_review);
   else if (selectedFilter === 'update') filtered = items.filter((item) => item.has_update);
   else if (selectedFilter !== 'all') filtered = items.filter((item) => item.kind === selectedFilter);
@@ -1080,6 +1084,7 @@ function countMyWork(items) {
     all: items.length,
     attention: items.filter(needsAttention).length,
     filed: items.filter((item) => item.is_filed).length,
+    authored: items.filter((item) => item.is_authored).length,
     issue: items.filter((item) => item.kind === 'issue').length,
     pull: items.filter((item) => item.kind === 'pull' && !item.is_review).length,
     review: items.filter((item) => item.is_review).length,

frontend/pull-sheet.js

@@ -12,12 +12,107 @@ function mergeEligibility(detail, reviewState) {
       'CI blocked by ' + blockers.join(', ') : 'CI must succeed before merging' };
   }
   if (!detail.head_sha) return { allowed: false, reason: 'Current head is unavailable' };
+  const reviewer = (detail.reviewers || []).find(item => item?.blocking);
+  if (reviewer?.status === 'waiting') {
+    return { allowed: false, reason: 'Waiting for @' + reviewer.login + '’s review' };
+  }
+  if (reviewer?.status === 'changes_requested') {
+    return { allowed: false, reason: '@' + reviewer.login + ' requested changes' };
+  }
+  if (reviewer?.status === 'outdated') {
+    return { allowed: false, reason: '@' + reviewer.login + '’s review is outdated after new commits' };
+  }
   if (reviewState && !reviewState.complete) {
     return { allowed: false, reason: 'Review every changed file before merging' };
   }
   return { allowed: true, reason: 'Ready to merge' };
 }
 
+function renderReviewerFeedback(reviewer, escapeHtml) {
+  const groups = new Map();
+  (Array.isArray(reviewer?.comments) ? reviewer.comments : []).forEach(comment => {
+    if (!comment?.path || !comment?.body) return;
+    if (!groups.has(comment.path)) groups.set(comment.path, []);
+    groups.get(comment.path).push(comment);
+  });
+  const summary = reviewer?.summary ? '<p class="pull-review-feedback-summary">' +
+    escapeHtml(reviewer.summary) + '</p>' : '';
+  const files = Array.from(groups.entries()).map(([path, comments]) =>
+    '<section class="pull-review-feedback-file"><h4>' + escapeHtml(path) + '</h4><ul>' +
+    comments.map(comment => '<li>' + escapeHtml(comment.body) +
+      (comment.line ? '<small>Line ' + Number(comment.line) + '</small>' : '') + '</li>').join('') +
+    '</ul><button type="button" data-review-feedback-file="' + escapeHtml(path) +
+    '">View in changes</button></section>'
+  ).join('');
+  const loadable = Number.isInteger(reviewer?.review_id) && reviewer.review_id > 0 &&
+    ['commented', 'changes_requested', 'outdated'].includes(reviewer?.status);
+  const feedbackState = Array.isArray(reviewer?.comments) ? 'loaded' : reviewer?.feedback_state;
+  const feedbackAction = !loadable || feedbackState === 'loaded' ? '' :
+    feedbackState === 'loading' ? '<p class="small" role="status">Loading inline feedback…</p>' :
+    '<p class="small" role="status">' + (feedbackState === 'error' ? 'Feedback unavailable. ' : '') +
+      '<button type="button" data-load-review-feedback="' + Number(reviewer.review_id) + '">' +
+      (feedbackState === 'error' ? 'Retry feedback' : 'Load inline feedback') + '</button></p>';
+  const emptyLoaded = loadable && feedbackState === 'loaded' && !files ?
+    '<p class="small">No inline comments on this review.</p>' : '';
+  if (!summary && !files && !feedbackAction && !emptyLoaded) return '';
+  const reviewedHead = reviewer?.status === 'outdated' && reviewer?.head_sha ?
+    '<p class="small">Reviewed head ' + escapeHtml(reviewer.head_sha.slice(0, 8)) + '</p>' : '';
+  const addressable = Number.isInteger(reviewer?.review_id) && reviewer.review_id > 0 &&
+    ['changes_requested', 'outdated'].includes(reviewer?.status) && feedbackState === 'loaded' &&
+    (reviewer.comments || []).some(comment => Number.isInteger(comment?.id) && comment.id > 0);
+  const address = addressable ? '<button type="button" class="address-review-feedback" data-address-review-feedback="' +
+    Number(reviewer.review_id) + '">Address feedback</button>' : '';
+  return '<details class="pull-review-feedback"><summary>Review feedback</summary>' +
+    reviewedHead + summary + feedbackAction + emptyLoaded + files + address + '</details>';
+}
+
+function renderReviewerStatuses(reviewers, escapeHtml) {
+  const labels = {
+    waiting: 'Waiting for review',
+    approved: 'Approved current head',
+    changes_requested: 'Changes requested',
+    commented: 'Commented',
+    outdated: 'Outdated after new commits',
+  };
+  return (Array.isArray(reviewers) ? reviewers : []).map(reviewer => {
+    const status = labels[reviewer?.status] || 'Status unavailable';
+    const cancel = reviewer?.status === 'waiting' && reviewer?.login ?
+      '<button type="button" class="cancel-review-request" data-cancel-review-request="' +
+      escapeHtml(reviewer.login) + '">Cancel request</button>' : '';
+    return '<article class="pull-reviewer-status" data-reviewer-status="' +
+      escapeHtml(reviewer?.status || 'unknown') + '" data-review-id="' +
+      (Number.isInteger(reviewer?.review_id) ? Number(reviewer.review_id) : '') +
+      '"><div class="pull-reviewer-status-heading"><strong>@' +
+      escapeHtml(reviewer?.login || 'unknown') + '</strong><span>' + escapeHtml(status) + '</span></div>' +
+      renderReviewerFeedback(reviewer, escapeHtml) + cancel + '</article>';
+  }).join('');
+}
+
+function renderReviewerPanel(doc, detail, escapeHtml = value => String(value)
+  .replaceAll('&', '&amp;').replaceAll('<', '&lt;').replaceAll('>', '&gt;')) {
+  const reviewers = Array.isArray(detail?.reviewers) ? detail.reviewers : [];
+  const openFeedback = new Set(Array.from(doc.querySelectorAll('.pull-review-feedback[open]'))
+    .map(panel => panel.closest('.pull-reviewer-status')?.dataset.reviewId).filter(Boolean));
+  doc.querySelector('#pull-reviewer-statuses').innerHTML = renderReviewerStatuses(reviewers, escapeHtml);
+  openFeedback.forEach(reviewId => {
+    const panel = doc.querySelector('.pull-reviewer-status[data-review-id="' + reviewId + '"] .pull-review-feedback');
+    if (panel) panel.open = true;
+  });
+  doc.querySelectorAll('[data-review-feedback-file]').forEach(button => {
+    button.addEventListener('click', () => focusPullFile(doc, button.dataset.reviewFeedbackFile));
+  });
+  doc.querySelector('#pull-reviewer-status').textContent = reviewers.length ?
+    reviewers.length + ' reviewer status' + (reviewers.length === 1 ? '' : 'es') + ' for this pull request.' :
+    'No review has been requested; existing merge policy is unchanged.';
+  doc.querySelector('#request-updated-pull-review').hidden =
+    !reviewers.some(reviewer => reviewer.status === 'outdated');
+}
+
+function reviewEligibility(detail, reviewState, doc) {
+  renderReviewerPanel(doc, detail);
+  return mergeEligibility(detail, reviewState);
+}
+
 function renderFile(file, index, reviewed, escapeHtml) {
   const filename = escapeHtml(file.filename || 'Unknown file');
   const panelId = 'pull-diff-' + index;
@@ -41,18 +136,22 @@ function renderFile(file, index, reviewed, escapeHtml) {
     '" aria-pressed="' + String(reviewed) + '">' + (reviewed ? 'Reviewed' : 'Mark reviewed') + '</button></article>';
 }
 
-function focusNextUnreviewed(doc, detail, state, controller) {
-  if (!detail || !state) return;
-  const filename = controller.nextUnreviewed(detail, state);
+function focusPullFile(doc, filename) {
   const article = Array.from(doc.querySelectorAll('#pull-files .pull-file'))
     .find(file => file.dataset.pullFilename === filename);
   const toggle = article?.querySelector('.pull-file-toggle');
   const panel = toggle && doc.getElementById(toggle.getAttribute('aria-controls'));
-  if (!toggle || !panel) return;
+  if (!toggle || !panel) return false;
   toggle.setAttribute('aria-expanded', 'true');
   panel.hidden = false;
   toggle.scrollIntoView({ block: 'center', behavior: 'smooth' });
   toggle.focus();
+  return true;
+}
+
+function focusNextUnreviewed(doc, detail, state, controller) {
+  if (!detail || !state) return;
+  focusPullFile(doc, controller.nextUnreviewed(detail, state));
 }
 
 function removeFromSnapshot(data, item) {
@@ -99,9 +198,374 @@ function resetOwnershipControls(doc, item, checkpointed) {
   qs('#release-pull').disabled = false;
   qs('#release-pull').textContent = checkpointed(item) ? 'Release & next' : 'Release assignment';
   qs('#pull-handoff-status').textContent = 'Load teammates to transfer ownership.';
+  qs('#edit-pull-content').hidden = true;
+  qs('#pull-edit-form').hidden = true;
+  qs('#pull-edit-status').textContent = '';
 }
 
-function bindOwnershipControls(doc, controller, getSelected, finish) {
+function resetReviewRequestControls(doc, detail) {
+  const qs = selector => doc.querySelector(selector);
+  qs('#pull-review-request').open = false;
+  qs('#pull-review-request').dataset.headSha = detail?.head_sha || '';
+  qs('#pull-review-recipient').innerHTML = '<option value="">Select a teammate</option>';
+  qs('#pull-review-recipient').disabled = true;
+  qs('#confirm-pull-review').disabled = true;
+  qs('#load-pull-reviewers').disabled = !detail?.head_sha;
+  qs('#pull-review-request-status').textContent = detail?.head_sha ?
+    'Load teammates to request a review.' : 'Load the current pull request before requesting review.';
+  qs('#pull-reviewer-status').textContent = 'Reviewer status not loaded.';
+  qs('#pull-reviewer-statuses').textContent = '';
+  qs('#request-updated-pull-review').hidden = true;
+  qs('#pull-feedback-pass').hidden = true;
+  qs('#pull-feedback-status').textContent = '';
+}
+
+function bindReviewRequestControls(doc, controller, getSelected, getDetail, getLogin) {
+  const qs = selector => doc.querySelector(selector);
+  const load = qs('#load-pull-reviewers');
+  if (load.dataset.reviewRequestBound === 'true') return;
+  load.dataset.reviewRequestBound = 'true';
+  controller.setReviewDetail = detail => {
+    resetReviewRequestControls(doc, detail);
+    controller.edit?.setDetail(detail, Boolean(detail?.saved_at));
+    const publishable = detail?.state === 'open' && detail?.draft === true &&
+      detail?.author === getLogin?.() && detail?.head_sha;
+    qs('#publish-draft-pull').hidden = !publishable;
+    if (publishable) qs('#pull-review-request').open = true;
+    qs('#publish-draft-pull-status').textContent = '';
+  };
+  resetReviewRequestControls(doc, null);
+  qs('#request-updated-pull-review').addEventListener('click', () => {
+    qs('#pull-review-request').open = true;
+    qs('#pull-review-request').scrollIntoView({ block:'center', behavior:'smooth' });
+    load.focus();
+  });
+  controller.onReviewRequested = result => {
+    const detail = getDetail?.();
+    if (!detail || result.head_sha !== detail.head_sha) return;
+    detail.reviewers = (detail.reviewers || []).filter(reviewer => reviewer.login !== result.reviewer);
+    detail.reviewers.push({login:result.reviewer, status:'waiting', head_sha:result.head_sha, blocking:true});
+    renderReviewerPanel(doc, detail);
+    qs('#pull-merge-state').textContent = 'Waiting for @' + result.reviewer + '’s review';
+    qs('#merge-pull').disabled = true;
+  };
+  qs('#publish-draft-pull').addEventListener('click', async () => {
+    const selected = getSelected?.();
+    const detail = getDetail?.();
+    if (!selected || !detail?.head_sha || detail.draft !== true ||
+        !globalThis.confirm('Publish ' + selected.key + ' as ready for review?')) return;
+    const button = qs('#publish-draft-pull');
+    button.disabled = true;
+    qs('#publish-draft-pull-status').textContent = 'Publishing draft…';
+    try {
+      const result = await controller.publishReady(selected, detail.head_sha);
+      if (getSelected?.() !== selected || getDetail?.() !== detail) return;
+      Object.assign(detail, result);
+      selected.title = result.title;
+      qs('#pull-sheet-title').textContent = result.title;
+      controller.setReviewDetail(detail);
+      qs('#publish-draft-pull-status').textContent = 'Published and ready for review.';
+      qs('#pull-review-request').open = true;
+      load.focus();
+    } catch (error) {
+      qs('#publish-draft-pull-status').textContent = error.message + ' The pull request remains a draft; retry.';
+      button.disabled = false;
+      button.focus();
+    }
+  });
+  qs('#pull-reviewer-statuses').addEventListener('click', async event => {
+    const button = event.target.closest?.('[data-cancel-review-request]');
+    if (!button || button.disabled) return;
+    const selected = getSelected?.();
+    const detail = getDetail?.();
+    const reviewer = button.dataset.cancelReviewRequest;
+    if (!selected || !detail?.head_sha || !reviewer ||
+        !globalThis.confirm('Cancel @' + reviewer + '’s review request for ' + selected.key + '?')) return;
+    button.disabled = true;
+    qs('#pull-reviewer-status').textContent = 'Cancelling @' + reviewer + '’s review request…';
+    try {
+      const result = await controller.cancelReview(selected, reviewer, detail.head_sha);
+      if (result?.head_sha !== detail.head_sha || result?.requested_reviewers?.includes(reviewer)) {
+        throw new Error('Cancellation was not confirmed.');
+      }
+      detail.reviewers = (detail.reviewers || []).filter(candidate =>
+        candidate.login !== reviewer || candidate.status !== 'waiting');
+      renderReviewerPanel(doc, detail);
+      const eligibility = mergeEligibility(detail, controller.reviewState(selected, detail));
+      qs('#pull-merge-state').textContent = eligibility.reason;
+      qs('#merge-pull').disabled = !eligibility.allowed;
+      qs('#pull-review-request').open = true;
+      qs('#pull-review-request-status').textContent = 'Review request cancelled. Choose a replacement reviewer.';
+      load.disabled = false;
+      load.focus();
+    } catch (error) {
+      qs('#pull-reviewer-status').textContent = error.message + ' Refresh review data before retrying.';
+      button.disabled = false;
+      button.focus();
+    }
+  });
+  load.addEventListener('click', async () => {
+    const selected = getSelected();
+    if (!selected) return;
+    const select = qs('#pull-review-recipient');
+    load.disabled = true;
+    qs('#pull-review-request-status').textContent = 'Loading eligible reviewers…';
+    try {
+      const count = renderHandoffCandidates(select, await controller.loadReviewCandidates(selected), doc);
+      select.disabled = !count;
+      qs('#confirm-pull-review').disabled = true;
+      qs('#pull-review-request-status').textContent = count ?
+        'Choose a teammate to review the current head.' : 'No eligible reviewers were found.';
+      if (count) select.focus();
+      else load.disabled = false;
+    } catch (error) {
+      qs('#pull-review-request-status').textContent = error.message + ' Retry loading reviewers.';
+      load.disabled = false;
+      load.focus();
+    }
+  });
+  qs('#pull-review-recipient').addEventListener('change', event => {
+    qs('#confirm-pull-review').disabled = !event.target.value;
+  });
+  qs('#confirm-pull-review').addEventListener('click', async () => {
+    const selected = getSelected();
+    const detail = { head_sha: qs('#pull-review-request').dataset.headSha };
+    const reviewer = qs('#pull-review-recipient').value;
+    if (!selected || !detail?.head_sha || !reviewer ||
+        !globalThis.confirm('Request review of ' + selected.key + ' at ' + detail.head_sha.slice(0, 8) + ' from @' + reviewer + '?')) return;
+    const button = qs('#confirm-pull-review');
+    button.disabled = true;
+    qs('#pull-review-request-status').textContent = 'Requesting review…';
+    try {
+      await controller.requestReview(selected, reviewer, detail.head_sha);
+      qs('#pull-review-request-status').textContent = 'Review requested from @' + reviewer + '.';
+    } catch (error) {
+      qs('#pull-review-request-status').textContent = error.message + ' Selection kept; retry the request.';
+      button.disabled = false;
+      button.focus();
+    }
+  });
+}
+
+function bindFeedbackControls(doc, controller, getSelected, getDetail, getLogin) {
+  const qs = selector => doc.querySelector(selector);
+  const panel = qs('#pull-feedback-pass');
+  if (panel.dataset.feedbackBound === 'true') return;
+  panel.dataset.feedbackBound = 'true';
+  let active = null;
+  const comments = () => (active?.reviewer?.comments || [])
+    .filter(comment => Number.isInteger(comment?.id) && comment.id > 0);
+  const current = () => comments()[active?.state?.index || 0];
+  const save = () => {
+    if (active) controller.saveFeedbackPass(active.item, active.detail, active.reviewer, active.login, active.state);
+  };
+  const render = () => {
+    if (!active) return;
+    const items = comments();
+    active.state.index = Math.max(0, Math.min(Number(active.state.index) || 0, items.length - 1));
+    const comment = current();
+    const response = active.state.responses[String(comment.id)] || {};
+    qs('#pull-feedback-progress').textContent = 'Comment ' + (active.state.index + 1) + ' of ' + items.length;
+    qs('#pull-feedback-heading').textContent = 'Address @' + active.reviewer.login + '’s feedback';
+    qs('#pull-feedback-location').textContent = comment.path + (comment.line ? ' · line ' + Number(comment.line) : '');
+    qs('#pull-feedback-body').textContent = comment.body;
+    qs('#pull-feedback-note').value = response.note || '';
+    panel.querySelectorAll('[data-feedback-disposition]').forEach(button => {
+      button.setAttribute('aria-pressed', String(button.dataset.feedbackDisposition === response.disposition));
+    });
+    qs('#previous-pull-feedback').disabled = active.state.index === 0;
+    qs('#next-pull-feedback').disabled = active.state.index >= items.length - 1;
+    qs('#finish-pull-feedback').disabled = active.state.posted === true ||
+      items.some(item => !active.state.responses[String(item.id)]?.disposition);
+    qs('#request-feedback-review').hidden = active.state.posted !== true;
+    const found = focusPullFile(doc, comment.path);
+    qs('#pull-feedback-context').textContent = found ?
+      'Matching change opened below.' : 'Matching change is unavailable in this preview.';
+    panel.scrollIntoView({ block:'center', behavior:'smooth' });
+  };
+  qs('#pull-reviewer-statuses').addEventListener('click', async event => {
+    const loadButton = event.target.closest?.('[data-load-review-feedback]');
+    if (loadButton && !loadButton.disabled) {
+      const detail = getDetail?.();
+      const item = getSelected?.();
+      const reviewId = Number(loadButton.dataset.loadReviewFeedback);
+      const reviewer = (detail?.reviewers || []).find(candidate => candidate.review_id === reviewId);
+      if (!detail || !item || !reviewer) return;
+      const refresh = reviewer.feedback_state === 'error';
+      reviewer.feedback_state = 'loading';
+      renderReviewerPanel(doc, detail);
+      try {
+        const result = await controller.loadFeedback(item, detail, reviewer, { refresh });
+        if (getDetail?.() !== detail || getSelected?.() !== item ||
+            result.head_sha !== detail.head_sha || Number(result.review_id) !== reviewId) return;
+        reviewer.comments = Array.isArray(result.comments) ? result.comments : [];
+        reviewer.feedback_state = 'loaded';
+        renderReviewerPanel(doc, detail);
+        doc.querySelector('[data-address-review-feedback="' + reviewId + '"]')?.focus();
+      } catch (error) {
+        if (getDetail?.() !== detail || getSelected?.() !== item) return;
+        reviewer.feedback_state = 'error';
+        renderReviewerPanel(doc, detail);
+        doc.querySelector('[data-load-review-feedback="' + reviewId + '"]')?.focus();
+      }
+      return;
+    }
+    const button = event.target.closest?.('[data-address-review-feedback]');
+    if (!button) return;
+    const detail = getDetail?.();
+    const item = getSelected?.();
+    const reviewId = Number(button.dataset.addressReviewFeedback);
+    const reviewer = (detail?.reviewers || []).find(candidate => candidate.review_id === reviewId);
+    if (!detail || !item || !reviewer) return;
+    const login = getLogin?.() || '';
+    active = { item, detail, reviewer, login,
+      state:controller.loadFeedbackPass(item, detail, reviewer, login) };
+    panel.hidden = false;
+    qs('#pull-feedback-status').textContent = active.state.posted ?
+      'Response already posted. You can request an updated review.' : 'Progress saves on this device.';
+    render();
+  });
+  panel.querySelector('.pull-feedback-dispositions').addEventListener('click', event => {
+    const disposition = event.target.dataset?.feedbackDisposition;
+    const comment = current();
+    if (!active || !comment || !['addressed', 'discussion', 'skipped'].includes(disposition)) return;
+    const key = String(comment.id);
+    active.state.responses[key] = { ...(active.state.responses[key] || {}), disposition };
+    save();
+    render();
+  });
+  qs('#pull-feedback-note').addEventListener('input', event => {
+    const comment = current();
+    if (!active || !comment) return;
+    const key = String(comment.id);
+    active.state.responses[key] = { ...(active.state.responses[key] || {}), note:event.target.value.slice(0, 1000) };
+    save();
+  });
+  qs('#previous-pull-feedback').addEventListener('click', () => {
+    if (!active) return;
+    active.state.index -= 1; save(); render();
+  });
+  qs('#next-pull-feedback').addEventListener('click', () => {
+    if (!active) return;
+    active.state.index += 1; save(); render();
+  });
+  qs('#close-pull-feedback').addEventListener('click', () => {
+    panel.hidden = true;
+    doc.querySelector('[data-address-review-feedback="' + active?.reviewer?.review_id + '"]')?.focus();
+  });
+  qs('#finish-pull-feedback').addEventListener('click', async () => {
+    if (!active || qs('#finish-pull-feedback').disabled) return;
+    const latest = getDetail?.();
+    if (!latest || latest.head_sha !== active.state.headSha) {
+      qs('#pull-feedback-status').textContent = 'The pull request changed. Refresh review data before posting.';
+      return;
+    }
+    if (!globalThis.confirm('Post one feedback response to ' + active.item.key + '?')) return;
+    qs('#finish-pull-feedback').disabled = true;
+    qs('#pull-feedback-status').textContent = 'Posting feedback response…';
+    try {
+      await controller.postFeedbackResponse(active.item, active.detail, active.reviewer, active.login, active.state);
+      active.state.posted = true;
+      save();
+      qs('#pull-feedback-status').textContent = 'Response posted. Request an updated review when ready.';
+      render();
+    } catch (error) {
+      qs('#pull-feedback-status').textContent = error.message + ' Progress kept; retry without duplicating the response.';
+      qs('#finish-pull-feedback').disabled = false;
+    }
+  });
+  qs('#request-feedback-review').addEventListener('click', async () => {
+    if (!active?.state?.posted) return;
+    const button = qs('#request-feedback-review');
+    button.disabled = true;
+    qs('#pull-feedback-status').textContent = 'Requesting updated review…';
+    try {
+      await controller.requestReview(active.item, active.reviewer.login, active.detail.head_sha);
+      qs('#pull-feedback-status').textContent = 'Response posted; updated review requested from @' + active.reviewer.login + '.';
+      button.hidden = true;
+    } catch (error) {
+      qs('#pull-feedback-status').textContent = 'Response posted; review not requested. ' + error.message;
+      button.disabled = false;
+    }
+  });
+}
+
+function bindContextEditor(doc, controller, getSelected, getDetail, getLogin) {
+  const qs = selector => doc.querySelector(selector);
+  const form = qs('#pull-edit-form');
+  const controls = () => form.querySelectorAll('input,textarea,button');
+  const draft = () => ({
+    title:qs('#pull-edit-title').value.trim(), body:qs('#pull-edit-body').value,
+    expectedHeadSha:getDetail()?.head_sha || '',
+  });
+  qs('#edit-pull-content').addEventListener('click', () => {
+    const item = getSelected(), detail = getDetail();
+    if (!item || !detail) return;
+    const saved = controller.loadEditDraft(item) || {
+      title:detail.title || '', body:detail.body || '', expectedHeadSha:detail.head_sha || '',
+    };
+    qs('#pull-edit-title').value = saved.title;
+    qs('#pull-edit-body').value = saved.body;
+    form.hidden = false;
+    qs('#pull-edit-status').textContent = '';
+    qs('#pull-edit-title').focus();
+  });
+  qs('#cancel-pull-edit').addEventListener('click', () => {
+    form.hidden = true;
+    qs('#edit-pull-content').focus();
+  });
+  ['#pull-edit-title', '#pull-edit-body'].forEach(selector => qs(selector).addEventListener('input', () => {
+    const item = getSelected();
+    if (item && getDetail()) controller.saveEditDraft(item, draft());
+  }));
+  form.addEventListener('submit', async event => {
+    event.preventDefault();
+    const item = getSelected(), update = draft();
+    if (!item || !getDetail()) return;
+    if (!update.title) {
+      qs('#pull-edit-status').textContent = 'Title is required.';
+      qs('#pull-edit-title').focus();
+      return;
+    }
+    controls().forEach(control => { control.disabled = true; });
+    qs('#pull-edit-status').textContent = 'Saving review context…';
+    try {
+      const result = await controller.updateContent(item, update);
+      if (getSelected() !== item) return;
+      const detail = getDetail();
+      Object.assign(detail, result);
+      item.title = result.title;
+      qs('#pull-sheet-title').textContent = result.title;
+      qs('#pull-sheet-body').innerHTML = renderMarkdown(result.body || 'No description provided.');
+      controller.setReviewDetail(detail);
+      qs('#pull-sheet-status').textContent = 'Review context updated · ready to request review';
+      form.hidden = true;
+      qs('#pull-edit-status').textContent = '';
+      qs('#edit-pull-content').focus();
+    } catch (error) {
+      qs('#pull-edit-status').textContent = error.message + ' Your change is still here; retry.';
+      qs('#pull-edit-title').focus();
+    } finally {
+      controls().forEach(control => { control.disabled = false; });
+    }
+  });
+  return {
+    reset() {
+      qs('#edit-pull-content').hidden = true;
+      form.hidden = true;
+      qs('#pull-edit-status').textContent = '';
+    },
+    setDetail(detail, offline = false) {
+      const editable = !offline && detail?.state === 'open' && detail.author === getLogin();
+      qs('#edit-pull-content').hidden = !editable;
+    },
+  };
+}
+
+function bindOwnershipControls(doc, controller, getSelected, finish, getDetail, getLogin) {
+  bindReviewRequestControls(doc, controller, getSelected, getDetail, getLogin);
+  bindFeedbackControls(doc, controller, getSelected, getDetail, getLogin);
+  controller.edit = bindContextEditor(doc, controller, getSelected, getDetail, getLogin);
   const qs = selector => doc.querySelector(selector);
   const load = qs('#load-pull-handoff');
   if (load.dataset.ownershipBound === 'true') return;
@@ -172,13 +636,24 @@ function createPullSheet({ fetchJson, storage, createConversationPager = globalT
   let checkRequest = null;
   let ownershipRequest = null;
   let candidateRequest = null;
+  let reviewCandidateRequest = null;
+  let reviewRequestMutation = null;
+  let reviewCancelMutation = null;
+  let readyMutation = null;
+  let editRequest = null;
+  let feedbackRequest = null;
   const reviewRequests = new Map();
   const reviewCache = new Map();
+  const feedbackRequests = new Map();
+  const feedbackCache = new Map();
   const pathFor = item => 'api/v1/repos/' + String(item.repository || '').split('/')
     .map(encodeURIComponent).join('/') + '/pulls/' + encodeURIComponent(item.number);
   const draftKey = item => 'stackchain.pull-comment.v1:' + item.repository + '#' + item.number;
   const operationKey = item => draftKey(item) + ':operation';
+  const editDraftKey = item => 'stackchain.pull-edit.v1:' + item.repository + '#' + item.number;
   const reviewKey = (item, detail) => 'stackchain.pull-review.v1:' + item.repository + '#' + item.number + ':' + detail.head_sha;
+  const feedbackKey = (item, detail, reviewer, login) => 'stackchain.feedback-pass.v1:' +
+    item.repository + '#' + item.number + ':' + reviewer.review_id + ':' + reviewer.head_sha + ':' + detail.head_sha + ':' + login;
   const fileNames = detail => (detail?.files || []).map(file => file.filename).filter(Boolean);
 
   function reviewState(item, detail) {
@@ -210,6 +685,25 @@ function createPullSheet({ fetchJson, storage, createConversationPager = globalT
       reviewRequests.set(key, request);
       return request;
     },
+    loadFeedback(item, detail, reviewer, { refresh = false } = {}) {
+      const reviewId = Number(reviewer?.review_id);
+      const headSha = detail?.head_sha || '';
+      const key = item.repository + '#' + item.number + ':' + headSha + ':' + reviewId;
+      if (!refresh && feedbackCache.has(key)) return Promise.resolve(feedbackCache.get(key));
+      if (feedbackRequests.has(key)) return feedbackRequests.get(key);
+      const request = fetchJson(pathFor(item) + '/reviews/' + encodeURIComponent(reviewId) +
+        '/feedback?expected_head_sha=' + encodeURIComponent(headSha), {
+        headers: { Accept: 'application/json' },
+      }).then(result => {
+        if (result?.head_sha !== headSha || Number(result?.review_id) !== reviewId) {
+          throw new Error('Review feedback changed. Reload review data before retrying.');
+        }
+        feedbackCache.set(key, result);
+        return result;
+      }).finally(() => feedbackRequests.delete(key));
+      feedbackRequests.set(key, request);
+      return request;
+    },
     loadChecks(item) {
       if (checkRequest) return checkRequest;
       checkRequest = fetchJson(pathFor(item) + '/checks', {
@@ -224,6 +718,129 @@ function createPullSheet({ fetchJson, storage, createConversationPager = globalT
       }).finally(() => { candidateRequest = null; });
       return candidateRequest;
     },
+    loadReviewCandidates(item) {
+      if (reviewCandidateRequest) return reviewCandidateRequest;
+      reviewCandidateRequest = fetchJson(pathFor(item) + '/review-candidates', {
+        headers: { Accept: 'application/json' },
+      }).finally(() => { reviewCandidateRequest = null; });
+      return reviewCandidateRequest;
+    },
+    requestReview(item, reviewer, expectedHeadSha) {
+      if (reviewRequestMutation) return reviewRequestMutation;
+      reviewRequestMutation = fetchJson(pathFor(item) + '/request-review', {
+        method: 'POST',
+        headers: { Accept: 'application/json', 'Content-Type': 'application/json' },
+        body: JSON.stringify({ reviewer, expected_head_sha: expectedHeadSha }),
+      }).then(result => {
+        this.onReviewRequested?.(result);
+        return result;
+      }).finally(() => { reviewRequestMutation = null; });
+      return reviewRequestMutation;
+    },
+    cancelReview(item, reviewer, expectedHeadSha) {
+      if (reviewCancelMutation) return reviewCancelMutation;
+      reviewCancelMutation = fetchJson(pathFor(item) + '/request-review', {
+        method: 'DELETE',
+        headers: { Accept: 'application/json', 'Content-Type': 'application/json' },
+        body: JSON.stringify({ reviewer, expected_head_sha: expectedHeadSha }),
+      }).then(result => {
+        this.onReviewCancelled?.(result);
+        return result;
+      }).finally(() => { reviewCancelMutation = null; });
+      return reviewCancelMutation;
+    },
+    publishReady(item, expectedHeadSha) {
+      if (readyMutation) return readyMutation;
+      readyMutation = fetchJson(pathFor(item) + '/ready', {
+        method:'PATCH',
+        headers:{ Accept:'application/json', 'Content-Type':'application/json' },
+        body:JSON.stringify({ expected_head_sha:expectedHeadSha }),
+      }).then(result => {
+        if (result?.number !== item.number || result?.head_sha !== expectedHeadSha ||
+            result?.state !== 'open' || result?.draft !== false || !result?.title) {
+          throw new Error('Draft publication was not confirmed.');
+        }
+        return result;
+      }).finally(() => { readyMutation = null; });
+      return readyMutation;
+    },
+    loadEditDraft(item) {
+      try {
+        const value = JSON.parse(storage?.getItem(editDraftKey(item)) || 'null');
+        return value && typeof value.title === 'string' && typeof value.body === 'string' &&
+          typeof value.expectedHeadSha === 'string' ? value : null;
+      } catch (_error) { return null; }
+    },
+    saveEditDraft(item, draft) {
+      try { storage?.setItem(editDraftKey(item), JSON.stringify(draft)); }
+      catch (_error) { /* The edit fields remain the fallback. */ }
+    },
+    loadFeedbackPass(item, detail, reviewer, login) {
+      const fresh = { headSha:detail.head_sha, index:0, responses:{}, posted:false };
+      try {
+        const saved = JSON.parse(storage?.getItem(feedbackKey(item, detail, reviewer, login)) || 'null');
+        return saved && saved.headSha === detail.head_sha && saved.responses && typeof saved.responses === 'object'
+          ? { ...fresh, ...saved } : fresh;
+      } catch (_error) { return fresh; }
+    },
+    saveFeedbackPass(item, detail, reviewer, login, state) {
+      try { storage?.setItem(feedbackKey(item, detail, reviewer, login), JSON.stringify(state)); }
+      catch (_error) { /* The visible pass remains usable without storage. */ }
+    },
+    feedbackSummary(reviewer, state) {
+      const labels = { addressed:'Addressed', discussion:'Needs discussion', skipped:'Skip for now' };
+      const lines = (reviewer.comments || []).filter(comment => Number.isInteger(comment?.id) && comment.id > 0)
+        .map(comment => {
+          const response = state.responses[String(comment.id)] || {};
+          const path = String(comment.path || '').replaceAll('`', "'").replaceAll('\n', ' ').slice(0, 300);
+          const body = String(comment.body || '').replaceAll('\n', ' ').slice(0, 180);
+          const note = String(response.note || '').replaceAll('\n', ' ').slice(0, 1000);
+          return '- `' + path + '` · comment #' + comment.id + ' · **' + (labels[response.disposition] || 'Unresolved') +
+            '** — ' + body + (note ? '\n  - Response: ' + note : '');
+        });
+      return ('Addressed review feedback from @' + reviewer.login + ' (review #' + reviewer.review_id + ').\n\n' +
+        lines.join('\n')).slice(0, 10000);
+    },
+    postFeedbackResponse(item, detail, reviewer, login, state) {
+      if (feedbackRequest) return feedbackRequest;
+      const key = feedbackKey(item, detail, reviewer, login);
+      let operationId;
+      try {
+        operationId = storage?.getItem(key + ':operation') || String(createOperationId()).slice(0, 128);
+        storage?.setItem(key + ':operation', operationId);
+      } catch (_error) { operationId = String(createOperationId()).slice(0, 128); }
+      feedbackRequest = fetchJson(pathFor(item) + '/comments', {
+        method:'POST',
+        headers:{ Accept:'application/json', 'Content-Type':'application/json', 'Idempotency-Key':operationId },
+        body:JSON.stringify({ body:this.feedbackSummary(reviewer, state) }),
+      }).then(result => {
+        try { storage?.removeItem(key + ':operation'); } catch (_error) { /* Confirmed upstream. */ }
+        return result;
+      }).finally(() => { feedbackRequest = null; });
+      return feedbackRequest;
+    },
+    updateContent(item, draft) {
+      if (editRequest) return editRequest;
+      this.saveEditDraft(item, draft);
+      editRequest = fetchJson(pathFor(item) + '/content', {
+        method: 'PATCH',
+        headers: { Accept: 'application/json', 'Content-Type': 'application/json' },
+        body: JSON.stringify({
+          title: draft.title,
+          body: draft.body,
+          expected_head_sha: draft.expectedHeadSha,
+        }),
+      }).then(result => {
+        if (result?.number !== item.number || result?.title !== draft.title ||
+            result?.body !== draft.body || result?.head_sha !== draft.expectedHeadSha) {
+          throw new Error('Pull request update was not confirmed.');
+        }
+        try { storage?.removeItem(editDraftKey(item)); }
+        catch (_error) { /* Confirmed upstream content is authoritative. */ }
+        return result;
+      }).finally(() => { editRequest = null; });
+      return editRequest;
+    },
     handoff(item, recipient) {
       if (ownershipRequest) return ownershipRequest;
       ownershipRequest = fetchJson(pathFor(item) + '/handoff', {
@@ -314,6 +931,8 @@ function createPullSheet({ fetchJson, storage, createConversationPager = globalT
 }
 
 createPullSheet.mergeEligibility = mergeEligibility;
+createPullSheet.review = reviewEligibility;
+createPullSheet.renderReviewerStatuses = renderReviewerStatuses;
 createPullSheet.renderFile = renderFile;
 createPullSheet.focusNextUnreviewed = focusNextUnreviewed;
 createPullSheet.removeFromSnapshot = removeFromSnapshot;
@@ -323,4 +942,8 @@ createPullSheet.renderHandoffCandidates = renderHandoffCandidates;
 createPullSheet.ownershipExitMessage = ownershipExitMessage;
 createPullSheet.resetOwnershipControls = resetOwnershipControls;
 createPullSheet.bindOwnershipControls = bindOwnershipControls;
+createPullSheet.bindContextEditor = bindContextEditor;
+createPullSheet.resetReviewRequestControls = resetReviewRequestControls;
+createPullSheet.bindReviewRequestControls = bindReviewRequestControls;
+createPullSheet.bindFeedbackControls = bindFeedbackControls;
 if (typeof module !== 'undefined' && module.exports) module.exports = createPullSheet;

frontend/search-preview.js

@@ -30,6 +30,10 @@
       return 'api/v1/repos/' + repository + '/issues/' + encodeURIComponent(item.number) +
         '/preview/conversation?' + query.toString();
     };
+    root.followingPullReviewPath = item => {
+      const repository = String(item.repository || '').split('/').map(encodeURIComponent).join('/');
+      return 'api/v1/following/' + repository + '/pulls/' + encodeURIComponent(item.number) + '/review-data';
+    };
     root.searchPreviewReplyPath = item => root.searchPreviewPath(item).replace(/\?.*$/, '') +
       '/comments?kind=' + encodeURIComponent(item.kind);
     root.searchPreviewSubscriptionPath = item => root.searchPreviewPath(item).replace(/\?.*$/, '') +
@@ -47,6 +51,7 @@
       watch:(detail,watching) => fetchJson(root.searchPreviewSubscriptionPath(detail), {
         method:watching ? 'PUT' : 'DELETE', headers:{Accept:'application/json'},
       }),
+      review:item => fetchJson(root.followingPullReviewPath(item)),
       };
       options.preview = async item => options.load({...item, ...await fetchJson(root.searchPreviewPath(item), {
         headers:{Accept:'application/json'},
@@ -165,9 +170,58 @@
       else if (state.status === 'reply-error') status.textContent =
         state.error?.message || 'Reply failed. Your draft is safe; retry when ready.';
     };
+    root.renderSearchPreviewReview = (review, document, escapeHtml) => {
+      const section = document.querySelector('#search-preview-review');
+      const status = document.querySelector('#search-preview-review-status');
+      const files = document.querySelector('#search-preview-files');
+      const retry = document.querySelector('#retry-search-preview-review');
+      section.hidden = !review;
+      retry.hidden = review?.status !== 'error';
+      files.innerHTML = '';
+      if (!review) {
+        status.textContent = '';
+        return;
+      }
+      if (review.status === 'loading') {
+        status.textContent = 'Loading CI and changed files…';
+        return;
+      }
+      if (review.status === 'error') {
+        status.textContent = 'Changes unavailable. This revision has not been marked reviewed.';
+        return;
+      }
+      const data = review.data || {};
+      const changed = Array.isArray(data.files) ? data.files : [];
+      const ci = ({success:'CI passed', failure:'CI failed', error:'CI failed', pending:'CI pending'})[
+        data.ci_state
+      ] || 'CI status unavailable';
+      status.textContent = ci + ' · ' + changed.length + ' changed ' +
+        (changed.length === 1 ? 'file.' : 'files.');
+      files.innerHTML = changed.map(file => {
+        const lines = (Array.isArray(file.diff_lines) ? file.diff_lines : []).map(raw => {
+          const line = String(raw);
+          const kind = line.startsWith('@@') ? 'hunk' : line.startsWith('+') ? 'added' :
+            line.startsWith('-') ? 'removed' : 'context';
+          return '<span class="pull-diff-line ' + kind + '">' + escapeHtml(line) + '</span>';
+        }).join('');
+        const diff = file.diff_available
+          ? '<pre class="pull-diff">' + lines +
+            (file.diff_truncated ? '<span class="pull-diff-note">Preview truncated · open in Gitea for the full diff.</span>' : '') + '</pre>'
+          : '<div class="pull-diff-empty">' +
+            (file.diff_binary ? 'Binary file · preview unavailable.' : 'Diff preview unavailable.') + '</div>';
+        return '<article class="search-preview-file"><strong>' + escapeHtml(file.filename || 'Unknown file') +
+          '</strong><span class="small">' + escapeHtml(file.status || 'changed') + ' · +' +
+          Number(file.additions || 0) + ' / −' + Number(file.deletions || 0) + '</span>' + diff + '</article>';
+      }).join('');
+    };
+    root.renderSearchPreviewWorkspaces = (state, detail, preview, document, escapeHtml) => {
+      root.renderSearchPreviewReply(state, detail, preview, document);
+      root.renderSearchPreviewReview(state.review, document, escapeHtml);
+      document.querySelector('[data-search-preview-section="changes"]').hidden = !state.review;
+    };
   }
 })(typeof globalThis !== 'undefined' ? globalThis : this, function () {
-  return function createSearchPreview({ fetchJson, fetchConversation, mutate, watch, share, postReply, queueReply, prepareReply, afterReply, afterUnwatch, hasAttachments, clearAttachments, storage, createOperationId, session, getSession, loadMore, onNavigate, onOpened, navigationRoot, onState }) {
+  return function createSearchPreview({ fetchJson, fetchConversation, fetchReview, mutate, watch, share, postReply, queueReply, prepareReply, afterReply, afterUnwatch, hasAttachments, clearAttachments, storage, createOperationId, session, getSession, loadMore, onNavigate, onOpened, navigationRoot, onState }) {
     if (Array.isArray(session)) {
       getSession = session[0];
       loadMore = () => session[1].loadMore();
@@ -182,6 +236,7 @@
     let replyRequest = null;
     let watchRequest = null;
     let conversation = null;
+    let review = null;
     let openedRevision = null;
 
     function sameItem(left, right) {
@@ -228,6 +283,10 @@
     }
 
     function publish(state) {
+      if (conversation && !Object.prototype.hasOwnProperty.call(state, 'conversation')) {
+        state = {...state, conversation};
+      }
+      if (review && !Object.prototype.hasOwnProperty.call(state, 'review')) state = {...state, review};
       const position = navigation(state.item || current);
       if (navigationRoot) {
         const bar = navigationRoot.querySelector('.search-preview-navigation');
@@ -293,6 +352,37 @@
       });
     }
 
+    function loadReview(detail, requestGeneration) {
+      if (!(detail?.following === true && detail?.kind === 'pull' && typeof fetchReview === 'function')) {
+        review = null;
+        return Promise.resolve(null);
+      }
+      review = {status:'loading', data:null};
+      publish({status:'ready', item:current, detail:current, conversation, review});
+      return fetchReview(detail).then(data => {
+        if (requestGeneration === generation) {
+          review = {status:'ready', data};
+          publish({status:'ready', item:current, detail:current, conversation, review});
+        }
+        return data;
+      }).catch(error => {
+        if (requestGeneration === generation) {
+          review = {status:'error', data:null, error};
+          publish({status:'ready', item:current, detail:current, conversation, review});
+        }
+        return null;
+      });
+    }
+
+    async function acknowledgeWhenContextReady(requestGeneration) {
+      const conversationReady = typeof fetchConversation !== 'function' || conversation?.status === 'ready';
+      const reviewRequired = current?.following === true && current?.kind === 'pull' &&
+        typeof fetchReview === 'function';
+      if (conversationReady && (!reviewRequired || review?.status === 'ready')) {
+        await notifyOpened(current, requestGeneration);
+      }
+    }
+
     const api = {
       hasReplyAttachments() {
         return Boolean(hasAttachments?.());
@@ -303,6 +393,7 @@
         const requestGeneration = generation;
         current = { ...item };
         conversation = null;
+        review = null;
         openedRevision = null;
         publish({ status: 'loading', item: current });
         return fetchJson(current).then(async detail => {
@@ -311,8 +402,9 @@
             if (typeof fetchConversation === 'function') {
               const context = loadConversation(current, requestGeneration);
               if (current.following === true) {
-                await context;
-                if (conversation?.status === 'ready') await notifyOpened(current, requestGeneration);
+                const reviewContext = loadReview(current, requestGeneration);
+                await Promise.all([context, reviewContext]);
+                await acknowledgeWhenContextReady(requestGeneration);
               } else {
                 await notifyOpened(current, requestGeneration);
               }
@@ -371,9 +463,14 @@
         if (!current) return null;
         const requestGeneration = generation;
         const result = await loadConversation(current, requestGeneration);
-        if (current?.following === true && conversation?.status === 'ready') {
-          await notifyOpened(current, requestGeneration);
-        }
+        if (current?.following === true) await acknowledgeWhenContextReady(requestGeneration);
+        return result;
+      },
+      async retryReview() {
+        if (!current) return null;
+        const requestGeneration = generation;
+        const result = await loadReview(current, requestGeneration);
+        await acknowledgeWhenContextReady(requestGeneration);
         return result;
       },
       loadOlderConversation() {
@@ -482,6 +579,7 @@
       navigationRoot.querySelector('#previous-search-result').addEventListener('click', () => api.previous().catch(() => {}));
       navigationRoot.querySelector('#next-search-result').addEventListener('click', () => api.next().catch(() => {}));
       navigationRoot.querySelector('#retry-search-preview-conversation').addEventListener('click', () => api.retryConversation());
+      navigationRoot.querySelector('#retry-search-preview-review').addEventListener('click', () => api.retryReview());
       navigationRoot.querySelector('#load-older-search-preview-comments').addEventListener('click', () => api.loadOlderConversation());
       const reply = navigationRoot.querySelector('#search-preview-reply');
       reply?.addEventListener('input', event => {

frontend/search-week-plan.js

@@ -42,7 +42,7 @@ function createSearchWeekPlan({week,claim,accept=item=>item,identity,maxItems=5}
         await week.flush();
         return {status:'planned',date:options.date,estimate,item};
       } catch(_error) {
-        return {status:'planned-pending',date:options.date,estimate};
+        return {status:'planned-pending',date:options.date,estimate,item};
       }
     };
     running=perform().finally(()=>{running=null;});
@@ -99,8 +99,8 @@ function createSearchWeekPlanUI({planner,document,window,getDetail,onPlanned=()=
     else if(outcome.status==='day-full')status.textContent='That day already has '+outcome.limit+' items. Choose another day.';
     else if(outcome.status==='already-planned')status.textContent='Already planned for '+outcome.date+'. Choose Move to change the day.';
     else if(outcome.status==='assigned-not-planned')status.textContent='Assigned, not planned. Find it in My Work and retry.';
-    else if(outcome.status==='planned-pending'){announce('Planned for '+date+' · sync pending.');onPlanned(detail,true);close(true);}
-    else if(outcome.status==='planned'){announce('Planned for '+date+'.');onPlanned(detail,false);close(true);}
+    else if(outcome.status==='planned-pending'){announce('Planned for '+date+' · sync pending.');await onPlanned(outcome.item||detail,true,outcome);close(true);}
+    else if(outcome.status==='planned'){announce('Planned for '+date+'.');await onPlanned(outcome.item||detail,false,outcome);close(true);}
     else status.textContent='Could not plan this issue. Retry.';
     button.disabled=false;
   }

frontend/service-worker.js

@@ -1,7 +1,7 @@
 const BASE = new URL('./', self.location.href).pathname;
 importScripts(BASE + 'static/private-data-registry.js');
 importScripts(BASE + 'static/background-issue-sync.js');
-const CACHE = 'stackchain-dashboard-shell-v136';
+const CACHE = 'stackchain-dashboard-shell-v138';
 const OFFLINE_LEASE_URL = new URL(BASE + '__offline-session-lease', self.location.origin).href;
 const OUTAGE_STATUSES = new Set([500, 502, 503, 504]);
 const NAVIGATION_TIMEOUT_MS = self.__STACKCHAIN_NAVIGATION_TIMEOUT_MS || 4000;
@@ -40,22 +40,55 @@ function createAppBadgePreference() {
       });
       database.close();
     },
+    async getCounts() {
+      const database = await open();
+      const counts = {};
+      for (const channel of ['updates', 'following']) {
+        counts[channel] = await new Promise((resolve, reject) => {
+          const request = database.transaction(storeName, 'readonly').objectStore(storeName).get('count:' + channel);
+          request.onsuccess = () => resolve(Number.isSafeInteger(request.result) ? request.result : 0);
+          request.onerror = () => reject(request.error || new Error('App badge count unavailable.'));
+        });
+      }
+      database.close();
+      return counts;
+    },
+    async setCount(channel, count) {
+      const database = await open();
+      await new Promise((resolve, reject) => {
+        const transaction = database.transaction(storeName, 'readwrite');
+        transaction.objectStore(storeName).put(count, 'count:' + channel);
+        transaction.oncomplete = resolve;
+        transaction.onerror = () => reject(transaction.error || new Error('App badge count could not be saved.'));
+        transaction.onabort = transaction.onerror;
+      });
+      database.close();
+    },
+    async clearCounts() {
+      await this.setCount('updates', 0);
+      await this.setCount('following', 0);
+    },
   };
 }
 const appBadgePreference = self.__STACKCHAIN_APP_BADGE_PREFERENCE || createAppBadgePreference();
 let renderedBackgroundBadgeCount = null;
 
-async function reconcileBackgroundAppBadge(count) {
-  if (!Number.isSafeInteger(count) || count < 0 || count > 9999
+async function reconcileBackgroundAppBadge(channel, count) {
+  if (!['updates', 'following'].includes(channel)
+      || !Number.isSafeInteger(count) || count < 0 || count > 9999
       || typeof self.registration.setAppBadge !== 'function'
       || typeof self.registration.clearAppBadge !== 'function') return false;
   let enabled = false;
   try { enabled = await appBadgePreference.get(); } catch (_error) { return false; }
-  if (!enabled || renderedBackgroundBadgeCount === count) return false;
+  if (!enabled) return false;
   try {
-    if (count > 0) await self.registration.setAppBadge(count);
+    await appBadgePreference.setCount(channel, count);
+    const counts = await appBadgePreference.getCounts();
+    const total = Math.min(9999, counts.updates + counts.following);
+    if (renderedBackgroundBadgeCount === total) return false;
+    if (total > 0) await self.registration.setAppBadge(total);
     else await self.registration.clearAppBadge();
-    renderedBackgroundBadgeCount = count;
+    renderedBackgroundBadgeCount = total;
     return true;
   } catch (_error) { return false; }
 }
@@ -576,6 +609,21 @@ async function updateTodayLockScreen(active, running, rawActionToken = '', rawBr
 }
 
 self.addEventListener('message', event => {
+  if (event.data?.type === 'stackchain-app-badge-count') {
+    event.waitUntil((async () => {
+      if (!String(event.source?.url || '').startsWith(self.location.origin + BASE)) return;
+      const channel = event.data.channel;
+      const count = event.data.count;
+      if (!['updates', 'following'].includes(channel)
+          || !Number.isSafeInteger(count) || count < 0 || count > 9999) return;
+      try {
+        if (await appBadgePreference.get()) {
+          await appBadgePreference.setCount(channel, count);
+          renderedBackgroundBadgeCount = null;
+        }
+      } catch (_error) { /* A later authoritative refresh can restore the count. */ }
+    })());
+  }
   if (event.data?.type === 'stackchain-app-badge-preference') {
     event.waitUntil((async () => {
       if (!String(event.source?.url || '').startsWith(self.location.origin + BASE)) return;
@@ -584,6 +632,7 @@ self.addEventListener('message', event => {
         await appBadgePreference.set(enabled);
         renderedBackgroundBadgeCount = null;
         if (!enabled && typeof self.registration.clearAppBadge === 'function') {
+          await appBadgePreference.clearCounts();
           await self.registration.clearAppBadge();
         }
       } catch (_error) { /* Page preference remains authoritative on next launch. */ }
@@ -693,14 +742,15 @@ self.addEventListener('push', event => {
     && followingCount > 0
     && followingCount <= 50
   ) {
-    event.waitUntil(self.registration.showNotification(
-      followingCount + ' watched item' + (followingCount === 1 ? '' : 's') + ' changed',
-      {
+    event.waitUntil(Promise.all([
+      reconcileBackgroundAppBadge('following', followingCount),
+      self.registration.showNotification(
+        followingCount + ' watched item' + (followingCount === 1 ? '' : 's') + ' changed', {
         body: 'Open Following to review the latest activity.',
         tag,
         data: {route},
-      }
-    ));
+      }),
+    ]));
     return;
   }
   if (
@@ -711,7 +761,7 @@ self.addEventListener('push', event => {
     && updateCount <= 50
   ) {
     event.waitUntil(Promise.all([
-      reconcileBackgroundAppBadge(unreadCount),
+      reconcileBackgroundAppBadge('updates', unreadCount),
       self.registration.showNotification(updateCount + ' new work updates', {
         body: 'Tap to review them in Stackchain.', tag, data: {route},
       }),
@@ -736,7 +786,7 @@ self.addEventListener('push', event => {
     options.data.notificationId = notificationId;
   }
   event.waitUntil(Promise.all([
-    reconcileBackgroundAppBadge(unreadCount),
+    reconcileBackgroundAppBadge('updates', unreadCount),
     self.registration.showNotification('New work update', options),
   ]));
 });

frontend/work-route.js

@@ -6,7 +6,7 @@
   'use strict';
 
   const repositoryPart = /^[A-Za-z0-9_.-]+$/;
-  const queueFilters = ['today', 'agenda', 'attention', 'filed', 'update', 'later', 'draft', 'following'];
+  const queueFilters = ['today', 'agenda', 'attention', 'filed', 'authored', 'update', 'later', 'draft', 'following'];
   const sections = {
     issue: ['overview', 'conversation', 'reply', 'actions'],
     filed: ['overview', 'conversation', 'reply', 'actions'],

src/gitea_proxy.py

@@ -370,6 +370,7 @@ WORK_SEARCHES = {
     "filed": ("created=true", "issues", "created_by_me"),
     "pull": ("assigned=true", "pulls", "assigned_to_me"),
     "review": ("review_requested=true", "pulls", "review_requested"),
+    "authored": ("created=true", "pulls", "authored_by_me"),
 }
 
 
@@ -1962,6 +1963,182 @@ async def pull_handoff_candidates(repository: str) -> list[dict]:
     ][:25]
 
 
+async def pull_review_candidates(repository: str, number: int) -> list[dict]:
+    pull, candidates = await asyncio.gather(
+        fetch(f"repos/{repository}/pulls/{number}"),
+        pull_handoff_candidates(repository),
+    )
+    if not isinstance(pull, dict):
+        raise ValueError("Gitea pull request response was not an object")
+    excluded = {
+        item["login"].casefold()
+        for item in (pull.get("requested_reviewers") or [])
+        if isinstance(item, dict) and isinstance(item.get("login"), str)
+    }
+    author = pull.get("user") if isinstance(pull.get("user"), dict) else {}
+    if isinstance(author.get("login"), str):
+        excluded.add(author["login"].casefold())
+    return [
+        item for item in candidates
+        if item["login"].casefold() not in excluded
+    ][:25]
+
+
+async def request_assigned_pull_review(
+    repository: str, number: int, reviewer: str, expected_head_sha: str
+) -> dict:
+    login, pull = await _current_login_and_target(
+        f"repos/{repository}/pulls/{number}"
+    )
+    head = pull.get("head") if isinstance(pull.get("head"), dict) else {}
+    if (
+        pull.get("state") != "open"
+        or pull.get("merged") is True
+        or not _login_in_users(login, pull.get("assignees"))
+        or head.get("sha") != expected_head_sha
+    ):
+        raise IssueNotAvailableError("Pull request is no longer eligible for review request")
+    eligible = {
+        item["login"] for item in await pull_review_candidates(repository, number)
+    }
+    if reviewer not in eligible:
+        raise IssueNotAvailableError("Reviewer is not eligible")
+    response = await _get_client().post(
+        f"/api/v1/repos/{repository}/pulls/{number}/requested_reviewers",
+        headers=_auth(),
+        json={"reviewers": [reviewer]},
+    )
+    response.raise_for_status()
+    confirmed_response = await _get_client().get(
+        f"/api/v1/repos/{repository}/pulls/{number}", headers=_auth()
+    )
+    confirmed_response.raise_for_status()
+    confirmed = confirmed_response.json()
+    requested = [
+        item["login"] for item in (confirmed.get("requested_reviewers") or [])
+        if isinstance(item, dict) and isinstance(item.get("login"), str)
+    ] if isinstance(confirmed, dict) else []
+    if reviewer not in requested:
+        raise ValueError("Gitea did not confirm the requested reviewer")
+    return {
+        "repository": repository,
+        "number": number,
+        "head_sha": expected_head_sha,
+        "requested_reviewers": requested,
+        "reviewer": reviewer,
+    }
+
+
+async def publish_authored_assigned_pull(
+    repository: str, number: int, expected_head_sha: str
+) -> dict:
+    login, pull = await _current_login_and_target(
+        f"repos/{repository}/pulls/{number}"
+    )
+    author = pull.get("user") if isinstance(pull.get("user"), dict) else {}
+    head = pull.get("head") if isinstance(pull.get("head"), dict) else {}
+    title = pull.get("title") if isinstance(pull.get("title"), str) else ""
+    ready_title = re.sub(
+        r"^(?:WIP\s*:|\[WIP\]|Draft\s*:|\[Draft\])\s*",
+        "",
+        title,
+        count=1,
+        flags=re.IGNORECASE,
+    ).strip()
+    if (
+        pull.get("state") != "open"
+        or pull.get("merged") is True
+        or pull.get("draft") is not True
+        or author.get("login", "").casefold() != login.casefold()
+        or not _login_in_users(login, pull.get("assignees"))
+        or head.get("sha") != expected_head_sha
+        or not ready_title
+        or ready_title == title
+    ):
+        raise IssueNotAvailableError("Draft pull request is no longer publishable")
+
+    response = await _get_client().patch(
+        f"/api/v1/repos/{repository}/pulls/{number}",
+        headers=_auth(),
+        json={"title": ready_title},
+    )
+    response.raise_for_status()
+    confirmed_response = await _get_client().get(
+        f"/api/v1/repos/{repository}/pulls/{number}", headers=_auth()
+    )
+    confirmed_response.raise_for_status()
+    confirmed = confirmed_response.json()
+    confirmed_head = (
+        confirmed.get("head")
+        if isinstance(confirmed, dict) and isinstance(confirmed.get("head"), dict)
+        else {}
+    )
+    if (
+        not isinstance(confirmed, dict)
+        or confirmed.get("number") != number
+        or confirmed.get("state") != "open"
+        or confirmed.get("draft") is not False
+        or confirmed.get("title") != ready_title
+        or confirmed_head.get("sha") != expected_head_sha
+    ):
+        raise ValueError("Gitea did not confirm the pull request is ready")
+    return {
+        "repository": repository,
+        "number": number,
+        "title": ready_title,
+        "head_sha": expected_head_sha,
+        "state": "open",
+        "draft": False,
+    }
+
+
+async def cancel_assigned_pull_review(
+    repository: str, number: int, reviewer: str, expected_head_sha: str
+) -> dict:
+    login, pull = await _current_login_and_target(
+        f"repos/{repository}/pulls/{number}"
+    )
+    head = pull.get("head") if isinstance(pull.get("head"), dict) else {}
+    requested = [
+        item["login"] for item in (pull.get("requested_reviewers") or [])
+        if isinstance(item, dict) and isinstance(item.get("login"), str)
+    ]
+    if (
+        pull.get("state") != "open"
+        or pull.get("merged") is True
+        or not _login_in_users(login, pull.get("assignees"))
+        or head.get("sha") != expected_head_sha
+        or reviewer not in requested
+    ):
+        raise IssueNotAvailableError("Pending review request changed")
+    response = await _get_client().request(
+        "DELETE",
+        f"/api/v1/repos/{repository}/pulls/{number}/requested_reviewers",
+        headers=_auth(),
+        json={"reviewers": [reviewer]},
+    )
+    response.raise_for_status()
+    confirmed_response = await _get_client().get(
+        f"/api/v1/repos/{repository}/pulls/{number}", headers=_auth()
+    )
+    confirmed_response.raise_for_status()
+    confirmed = confirmed_response.json()
+    remaining = [
+        item["login"] for item in (confirmed.get("requested_reviewers") or [])
+        if isinstance(item, dict) and isinstance(item.get("login"), str)
+    ] if isinstance(confirmed, dict) else []
+    confirmed_head = confirmed.get("head") if isinstance(confirmed, dict) and isinstance(confirmed.get("head"), dict) else {}
+    if reviewer in remaining or confirmed_head.get("sha") != expected_head_sha:
+        raise ValueError("Gitea did not confirm reviewer cancellation")
+    return {
+        "repository": repository,
+        "number": number,
+        "head_sha": expected_head_sha,
+        "requested_reviewers": remaining,
+        "reviewer": reviewer,
+    }
+
+
 async def _change_assigned_pull_owners(
     repository: str, number: int, recipient: str | None
 ) -> dict:
@@ -2592,29 +2769,34 @@ async def is_open_authored_issue(repository: str, number: int) -> bool:
 
 
 async def pull_requests() -> WorkItems:
-    assigned, review_requested = await asyncio.gather(
-        work_page("pull"),
-        work_page("review"),
+    streams = ("pull", "review", "authored")
+    outcomes = await asyncio.gather(
+        *(work_page(stream) for stream in streams),
+        return_exceptions=True,
     )
+    for outcome in outcomes:
+        if isinstance(outcome, asyncio.CancelledError):
+            raise outcome
+    if all(isinstance(outcome, BaseException) for outcome in outcomes):
+        raise outcomes[0]
+
     merged: dict[int, dict] = {}
-    for result in (
-        assigned,
-        review_requested,
-    ):
-        for pull in result["items"]:
+    pagination = {}
+    for stream, outcome in zip(streams, outcomes, strict=True):
+        if isinstance(outcome, BaseException):
+            pagination[stream] = {
+                "page": 1, "total": 0, "has_more": False, "unavailable": True,
+            }
+            continue
+        pagination[stream] = _page_metadata(outcome)
+        for pull in outcome["items"]:
             identity = pull.get("id")
             if identity not in merged:
                 merged[identity] = {**pull, "work_reasons": []}
             for reason in pull.get("work_reasons", []):
                 if reason not in merged[identity]["work_reasons"]:
                     merged[identity]["work_reasons"].append(reason)
-    return WorkItems(
-        list(merged.values()),
-        {
-            "pull": _page_metadata(assigned),
-            "review": _page_metadata(review_requested),
-        },
-    )
+    return WorkItems(list(merged.values()), pagination)
 
 
 async def is_requested_review(repository: str, number: int) -> bool:
@@ -2637,6 +2819,52 @@ async def is_assigned_pull(repository: str, number: int) -> bool:
     )
 
 
+async def update_authored_assigned_pull(
+    repository: str,
+    number: int,
+    title: str,
+    body: str,
+    expected_head_sha: str,
+) -> dict:
+    path = f"repos/{repository}/pulls/{number}"
+    login, pull = await _current_login_and_target(path)
+    author = pull.get("user") if isinstance(pull.get("user"), dict) else {}
+    head = pull.get("head") if isinstance(pull.get("head"), dict) else {}
+    if (
+        pull.get("state") != "open"
+        or pull.get("merged") is True
+        or author.get("login") != login
+        or not _login_in_users(login, pull.get("assignees"))
+    ):
+        raise IssueNotAvailableError("pull request not found")
+    if head.get("sha") != expected_head_sha:
+        raise IssueEditConflictError("pull request head changed upstream")
+    response = await _get_client().patch(
+        f"/api/v1/{path}",
+        headers=_auth(),
+        json={"title": title, "body": body},
+    )
+    response.raise_for_status()
+    confirmed = response.json()
+    confirmed_head = confirmed.get("head") if isinstance(confirmed.get("head"), dict) else {}
+    if (
+        confirmed.get("number") != number
+        or confirmed.get("title") != title
+        or confirmed.get("body", "") != body
+        or confirmed_head.get("sha") != expected_head_sha
+    ):
+        raise ValueError("Gitea did not confirm the pull request update")
+    return {
+        "repository": repository,
+        "number": number,
+        "title": title,
+        "body": body,
+        "head_sha": expected_head_sha,
+        "state": confirmed.get("state", "open"),
+        "url": _safe_gitea_web_url(confirmed.get("html_url")),
+    }
+
+
 async def pull_completion_detail(repository: str, number: int) -> dict:
     base = f"repos/{repository}/pulls/{number}"
     pull = await fetch(base)
@@ -2659,6 +2887,106 @@ async def pull_completion_detail(repository: str, number: int) -> dict:
     }
 
 
+def _normalize_review_comments(comments: object) -> list[dict]:
+    normalized = []
+    for comment in (comments if isinstance(comments, list) else [])[:100]:
+        if not isinstance(comment, dict):
+            continue
+        path = comment.get("path")
+        body = comment.get("body")
+        if not isinstance(path, str) or not path.strip() or not isinstance(body, str) or not body.strip():
+            continue
+        item = {"path": path.strip()[:300], "body": body.strip()[:500]}
+        comment_id = comment.get("id")
+        if isinstance(comment_id, int) and comment_id > 0:
+            item["id"] = comment_id
+        position = comment.get("new_position") or comment.get("old_position")
+        if isinstance(position, int) and position > 0:
+            item["line"] = position
+        normalized.append(item)
+        if len(normalized) == 20:
+            break
+    return normalized
+
+
+def _latest_feedback_review_ids(reviews: object) -> dict[str, int]:
+    latest: dict[str, tuple[int, str]] = {}
+    for review in (reviews if isinstance(reviews, list) else [])[:100]:
+        if not isinstance(review, dict):
+            continue
+        user = review.get("user") if isinstance(review.get("user"), dict) else {}
+        login = user.get("login")
+        review_id = review.get("id")
+        state = review.get("state")
+        if (
+            not isinstance(login, str) or not login or not isinstance(review_id, int)
+            or review_id <= 0 or not isinstance(state, str)
+        ):
+            continue
+        key = login.casefold()
+        if key not in latest or review_id >= latest[key][0]:
+            latest[key] = (review_id, state)
+    feedback = [
+        (key, review_id) for key, (review_id, state) in latest.items()
+        if state in {"REQUEST_CHANGES", "COMMENT"}
+    ]
+    return dict(feedback[:10])
+
+
+def _normalize_reviewer_statuses(pull: dict, reviews: object, head_sha: str) -> list[dict]:
+    """Return one bounded, current decision per reviewer without exposing raw review data."""
+    latest: dict[str, dict] = {}
+    for review in (reviews if isinstance(reviews, list) else [])[:100]:
+        if not isinstance(review, dict):
+            continue
+        user = review.get("user") if isinstance(review.get("user"), dict) else {}
+        login = user.get("login")
+        state = review.get("state")
+        commit_id = review.get("commit_id")
+        if (
+            not isinstance(login, str)
+            or not login
+            or state not in {"APPROVED", "REQUEST_CHANGES", "COMMENT"}
+            or not isinstance(commit_id, str)
+            or not commit_id
+        ):
+            continue
+        previous = latest.get(login.casefold())
+        review_id = review.get("id") if isinstance(review.get("id"), int) else 0
+        if previous is None or review_id >= previous["id"]:
+            body = review.get("body")
+            latest[login.casefold()] = {
+                "id": review_id, "login": login, "state": state, "commit_id": commit_id,
+                "summary": body.strip()[:500] if isinstance(body, str) and body.strip() else "",
+            }
+
+    statuses: dict[str, dict] = {}
+    state_names = {
+        "APPROVED": "approved", "REQUEST_CHANGES": "changes_requested", "COMMENT": "commented",
+    }
+    for key, review in latest.items():
+        current = review["commit_id"] == head_sha
+        status = state_names[review["state"]] if current else "outdated"
+        statuses[key] = {
+            "review_id": review["id"],
+            "login": review["login"],
+            "status": status,
+            "head_sha": review["commit_id"],
+            "blocking": status in {"changes_requested", "outdated"},
+        }
+        if review["summary"]:
+            statuses[key]["summary"] = review["summary"]
+
+    requested = pull.get("requested_reviewers") if isinstance(pull, dict) else []
+    for reviewer in (requested if isinstance(requested, list) else [])[:25]:
+        login = reviewer.get("login") if isinstance(reviewer, dict) else None
+        if isinstance(login, str) and login:
+            statuses[login.casefold()] = {
+                "login": login, "status": "waiting", "head_sha": head_sha, "blocking": True,
+            }
+    return sorted(statuses.values(), key=lambda item: item["login"].casefold())
+
+
 async def pull_completion_review(repository: str, number: int) -> dict:
     base = f"repos/{repository}/pulls/{number}"
     pull = await fetch(base)
@@ -2668,15 +2996,17 @@ async def pull_completion_review(repository: str, number: int) -> dict:
     head: dict = head_value if isinstance(head_value, dict) else {}
     sha_value = head.get("sha")
     sha = sha_value if isinstance(sha_value, str) else ""
-    files, status, diff_result = await asyncio.gather(
+    files, status, diff_result, reviews = await asyncio.gather(
         fetch(f"{base}/files"),
         fetch(f"repos/{repository}/commits/{sha}/status"),
         fetch_text(
             f"repos/{repository}/pulls/{number}.diff", REVIEW_DIFF_MAX_BYTES
         ),
+        fetch(f"{base}/reviews?limit=100"),
     )
     diff, diff_truncated = diff_result
     previews = _diff_previews(diff, diff_truncated)
+    reviewers = _normalize_reviewer_statuses(pull, reviews, sha)
     return {
         "repository": repository,
         "number": number,
@@ -2687,6 +3017,7 @@ async def pull_completion_review(repository: str, number: int) -> dict:
         "merged": pull.get("merged") is True,
         "ci_state": status.get("state", "unknown") if isinstance(status, dict) else "unknown",
         "checks": _normalize_commit_checks(status),
+        "reviewers": reviewers,
         "files": [
             {
                 "filename": item.get("filename", ""),
@@ -2709,6 +3040,40 @@ async def pull_completion_review(repository: str, number: int) -> dict:
     }
 
 
+async def pull_review_feedback(
+    repository: str, number: int, review_id: int, expected_head_sha: str
+) -> dict:
+    """Load optional inline feedback only for a review on the confirmed pull head."""
+    base = f"repos/{repository}/pulls/{number}"
+    pull, reviews = await asyncio.gather(
+        fetch(base),
+        fetch(f"{base}/reviews?limit=100"),
+    )
+    head = pull.get("head") if isinstance(pull, dict) else None
+    head_sha = head.get("sha") if isinstance(head, dict) else None
+    if head_sha != expected_head_sha:
+        raise StaleReviewError("Pull request head changed")
+    review_items = reviews if isinstance(reviews, list) else []
+    latest_ids = set(_latest_feedback_review_ids(review_items).values())
+    review = next(
+        (
+            item for item in review_items
+            if isinstance(item, dict) and item.get("id") == review_id
+        ),
+        None,
+    )
+    if review is None or review_id not in latest_ids:
+        raise StaleReviewError("Review feedback is no longer current")
+    comments = await fetch(f"{base}/reviews/{review_id}/comments")
+    reviewed_head = review.get("commit_id")
+    return {
+        "review_id": review_id,
+        "head_sha": head_sha,
+        "reviewed_head_sha": reviewed_head if isinstance(reviewed_head, str) else "",
+        "comments": _normalize_review_comments(comments),
+    }
+
+
 async def release_receipt_status(repository: str, commit_sha: str) -> dict:
     """Return CI and release evidence for one exact merge commit."""
     status, releases = await asyncio.gather(
@@ -2761,7 +3126,10 @@ async def pull_check_status(repository: str, number: int) -> dict:
     head: dict = head_value if isinstance(head_value, dict) else {}
     sha_value = head.get("sha")
     sha = sha_value if isinstance(sha_value, str) else ""
-    status = await fetch(f"repos/{repository}/commits/{sha}/status")
+    status, reviews = await asyncio.gather(
+        fetch(f"repos/{repository}/commits/{sha}/status"),
+        fetch(f"{base}/reviews?limit=100"),
+    )
     return {
         "repository": repository,
         "number": number,
@@ -2772,6 +3140,7 @@ async def pull_check_status(repository: str, number: int) -> dict:
         "merged": pull.get("merged") is True,
         "ci_state": status.get("state", "unknown") if isinstance(status, dict) else "unknown",
         "checks": _normalize_commit_checks(status),
+        "reviewers": _normalize_reviewer_statuses(pull, reviews, sha),
     }
 
 
@@ -2812,6 +3181,12 @@ async def merge_assigned_pull(
         or ci_state != "success"
     ):
         raise PullNotMergeableError("Pull request is not currently safe to merge")
+    reviews = await fetch(f"{base}/reviews?limit=100")
+    if any(
+        reviewer["blocking"]
+        for reviewer in _normalize_reviewer_statuses(pull, reviews, current_sha)
+    ):
+        raise PullNotMergeableError("Pull request review is not currently resolved")
     response = await _get_client().post(
         f"/api/v1/{base}/merge",
         headers=_auth(),

src/main.py

@@ -1108,6 +1108,37 @@ class IssueHandoff(BaseModel):
     )
 
 
+class PullReviewRequest(BaseModel):
+    reviewer: str = Field(
+        min_length=1, max_length=255, pattern=r"^[A-Za-z0-9_.-]+$"
+    )
+    expected_head_sha: str = Field(
+        min_length=7, max_length=64, pattern=r"^[A-Fa-f0-9]+$"
+    )
+
+
+class PullReadyRequest(BaseModel):
+    expected_head_sha: str = Field(
+        min_length=7, max_length=64, pattern=r"^[A-Fa-f0-9]+$"
+    )
+
+
+class PullContentUpdate(BaseModel):
+    title: str = Field(min_length=1, max_length=255)
+    body: str = Field(default="", max_length=10_000)
+    expected_head_sha: str = Field(
+        min_length=7, max_length=64, pattern=r"^[A-Fa-f0-9]+$"
+    )
+
+    @field_validator("title")
+    @classmethod
+    def strip_title(cls, value: str) -> str:
+        value = value.strip()
+        if not value:
+            raise ValueError("title cannot be blank")
+        return value
+
+
 class IssueReassignment(IssueHandoff):
     expected_assignees: list[str] = Field(min_length=1, max_length=10)
 
@@ -1298,7 +1329,7 @@ def _context_payload(user_data, repo_data, issues_data, prs_data) -> dict:
             user=p["user"].get("login", "") if isinstance(p.get("user"), dict) else "",
             labels=[label.get("name", "") for label in (p.get("labels") or []) if isinstance(label, dict)],
             assignees=[assignee.get("login", "") for assignee in (p.get("assignees") or []) if isinstance(assignee, dict)],
-            work_reasons=[reason for reason in (p.get("work_reasons") or []) if reason in ("assigned_to_me", "review_requested")],
+            work_reasons=[reason for reason in (p.get("work_reasons") or []) if reason in ("assigned_to_me", "review_requested", "authored_by_me")],
             repository=p["repository"].get("full_name", "") if isinstance(p.get("repository"), dict) else "",
             updated_at=p.get("updated_at") or "", url=p["html_url"],
         )
@@ -1339,7 +1370,11 @@ def _normalize_work_items(stream: str, items: list[dict]) -> list[dict]:
             for item in items
             if all(field in item for field in ("id", "number", "title", "state", "html_url"))
         ]
-    reason = "assigned_to_me" if stream == "pull" else "review_requested"
+    reason = {
+        "pull": "assigned_to_me",
+        "review": "review_requested",
+        "authored": "authored_by_me",
+    }[stream]
     return [
         PullRequest(
             id=item["id"], number=item["number"], title=item["title"],
@@ -2753,6 +2788,45 @@ async def keep_following_revision(
         )
 
 
+@app.get("/api/v1/following/{owner}/{repo}/pulls/{number}/review-data")
+async def following_pull_review_data(
+    owner: str, repo: str, number: int = PathParam(gt=0)
+):
+    """Return bounded review context only for a pull in this account's Following list."""
+    repository = f"{owner}/{repo}"
+    login = await _confirmed_login()
+
+    async def load_review():
+        snapshot = await asyncio.to_thread(_following_store().get, login)
+        watched = any(
+            item.get("kind") == "pull"
+            and item.get("repository", "").lower() == repository.lower()
+            and item.get("number") == number
+            for item in snapshot["items"]
+        )
+        if not watched:
+            raise HTTPException(status_code=404, detail="Watched pull request not found")
+        return await gitea_proxy.pull_completion_review(repository, number)
+
+    try:
+        result = await asyncio.wait_for(load_review(), timeout=REVIEW_DETAIL_TIMEOUT_SECONDS)
+    except HTTPException:
+        raise
+    except TimeoutError:
+        return JSONResponse(
+            {"error": "Loading watched pull request changes timed out. Please retry."},
+            status_code=503,
+            headers={"Retry-After": "1"},
+        )
+    except Exception:
+        return JSONResponse(
+            {"error": "Watched pull request changes are temporarily unavailable. Please retry."},
+            status_code=503,
+            headers={"Retry-After": "1"},
+        )
+    return JSONResponse(result, headers={"Cache-Control": "no-store"})
+
+
 @app.get("/api/v1/completed-filed-reviews")
 async def get_completed_filed_reviews(response: Response):
     login = await _confirmed_login()
@@ -4086,7 +4160,7 @@ async def resolve_work_route(
 
 @app.get("/api/v1/work/{stream}")
 async def paged_work(
-    stream: Literal["issue", "filed", "pull", "review"],
+    stream: Literal["issue", "filed", "pull", "review", "authored"],
     page: int = Query(ge=2, le=100),
 ) -> JSONResponse:
     try:
@@ -6510,6 +6584,70 @@ async def assigned_pull_detail(owner: str, repo: str, number: int = PathParam(gt
         )
 
 
+@app.patch("/api/v1/repos/{owner}/{repo}/pulls/{number}/content")
+async def update_authored_assigned_pull_content(
+    update: PullContentUpdate,
+    owner: str,
+    repo: str,
+    number: int = PathParam(gt=0),
+):
+    repository = f"{owner}/{repo}"
+    try:
+        result = await asyncio.wait_for(
+            gitea_proxy.update_authored_assigned_pull(
+                repository,
+                number,
+                update.title,
+                update.body,
+                update.expected_head_sha,
+            ),
+            timeout=ISSUE_ACTION_TIMEOUT_SECONDS,
+        )
+    except gitea_proxy.IssueEditConflictError:
+        return JSONResponse(
+            {"error": "This pull request changed in Gitea. Your draft is safe; reload the latest pull request before saving."},
+            status_code=409,
+        )
+    except gitea_proxy.IssueNotAvailableError:
+        raise HTTPException(status_code=404, detail="Editable pull request not found")
+    except Exception:
+        return JSONResponse(
+            {"error": "The pull request could not be updated. Your draft is safe; please retry."},
+            status_code=503,
+            headers={"Retry-After": "1"},
+        )
+    return JSONResponse(result)
+
+
+@app.patch("/api/v1/repos/{owner}/{repo}/pulls/{number}/ready")
+async def publish_authored_assigned_pull(
+    update: PullReadyRequest,
+    owner: str,
+    repo: str,
+    number: int = PathParam(gt=0),
+):
+    repository = f"{owner}/{repo}"
+    try:
+        result = await asyncio.wait_for(
+            gitea_proxy.publish_authored_assigned_pull(
+                repository, number, update.expected_head_sha
+            ),
+            timeout=ISSUE_ACTION_TIMEOUT_SECONDS,
+        )
+    except gitea_proxy.IssueNotAvailableError:
+        return JSONResponse(
+            {"error": "The draft pull request changed. Reload before publishing."},
+            status_code=409,
+        )
+    except Exception:
+        return JSONResponse(
+            {"error": "The draft pull request could not be published. Please retry."},
+            status_code=503,
+            headers={"Retry-After": "1"},
+        )
+    return JSONResponse(result)
+
+
 @app.get("/api/v1/repos/{owner}/{repo}/pulls/{number}/review-data")
 async def assigned_pull_review_data(
     owner: str, repo: str, number: int = PathParam(gt=0)
@@ -6541,6 +6679,50 @@ async def assigned_pull_review_data(
         )
 
 
+@app.get("/api/v1/repos/{owner}/{repo}/pulls/{number}/reviews/{review_id}/feedback")
+async def assigned_pull_review_feedback(
+    owner: str,
+    repo: str,
+    number: int = PathParam(gt=0),
+    review_id: int = PathParam(gt=0),
+    expected_head_sha: str = Query(min_length=7, max_length=64, pattern=r"^[A-Fa-f0-9]+$"),
+):
+    repository = f"{owner}/{repo}"
+
+    async def load_feedback():
+        if not await gitea_proxy.is_assigned_pull(repository, number):
+            raise HTTPException(status_code=404, detail="Assigned pull request not found")
+        return await gitea_proxy.pull_review_feedback(
+            repository, number, review_id, expected_head_sha
+        )
+
+    try:
+        result = await asyncio.wait_for(
+            load_feedback(), timeout=REVIEW_DETAIL_TIMEOUT_SECONDS
+        )
+    except HTTPException:
+        raise
+    except gitea_proxy.StaleReviewError:
+        return JSONResponse(
+            {"error": "The pull request or review changed. Reload review data before retrying."},
+            status_code=409,
+            headers={"Cache-Control": "no-store"},
+        )
+    except TimeoutError:
+        return JSONResponse(
+            {"error": "Loading review feedback timed out. Please retry."},
+            status_code=503,
+            headers={"Cache-Control": "no-store", "Retry-After": "1"},
+        )
+    except Exception:
+        return JSONResponse(
+            {"error": "Review feedback is temporarily unavailable. Please retry."},
+            status_code=503,
+            headers={"Cache-Control": "no-store", "Retry-After": "1"},
+        )
+    return JSONResponse(result, headers={"Cache-Control": "no-store"})
+
+
 @app.get("/api/v1/repos/{owner}/{repo}/pulls/{number}/checks")
 async def assigned_pull_checks(
     owner: str, repo: str, number: int = PathParam(gt=0)
@@ -6623,6 +6805,85 @@ async def handoff_assigned_pull(
     return JSONResponse(result)
 
 
+@app.get("/api/v1/repos/{owner}/{repo}/pulls/{number}/review-candidates")
+async def pull_review_candidates(
+    owner: str, repo: str, number: int = PathParam(gt=0)
+) -> JSONResponse:
+    repository = f"{owner}/{repo}"
+    try:
+        if not await gitea_proxy.is_assigned_pull(repository, number):
+            raise HTTPException(status_code=404, detail="Assigned pull request not found")
+        result = await asyncio.wait_for(
+            gitea_proxy.pull_review_candidates(repository, number),
+            timeout=ISSUE_ACTION_TIMEOUT_SECONDS,
+        )
+    except HTTPException:
+        raise
+    except Exception:
+        return JSONResponse(
+            {"error": "Reviewers could not be loaded. Please retry."},
+            status_code=503,
+            headers={"Retry-After": "1"},
+        )
+    return JSONResponse(result, headers={"Cache-Control": "no-store"})
+
+
+@app.post("/api/v1/repos/{owner}/{repo}/pulls/{number}/request-review")
+async def request_assigned_pull_review(
+    request: PullReviewRequest,
+    owner: str,
+    repo: str,
+    number: int = PathParam(gt=0),
+) -> JSONResponse:
+    try:
+        result = await asyncio.wait_for(
+            gitea_proxy.request_assigned_pull_review(
+                f"{owner}/{repo}", number, request.reviewer, request.expected_head_sha
+            ),
+            timeout=ISSUE_ACTION_TIMEOUT_SECONDS,
+        )
+    except gitea_proxy.IssueNotAvailableError:
+        return JSONResponse(
+            {"error": "The pull request or reviewer changed. Reload before requesting review."},
+            status_code=409,
+        )
+    except Exception:
+        return JSONResponse(
+            {"error": "The review request could not be confirmed. Please retry."},
+            status_code=503,
+            headers={"Retry-After": "1"},
+        )
+    return JSONResponse(result)
+
+
+@app.delete("/api/v1/repos/{owner}/{repo}/pulls/{number}/request-review")
+async def cancel_assigned_pull_review(
+    request: PullReviewRequest,
+    owner: str,
+    repo: str,
+    number: int = PathParam(gt=0),
+) -> JSONResponse:
+    try:
+        result = await asyncio.wait_for(
+            gitea_proxy.cancel_assigned_pull_review(
+                f"{owner}/{repo}", number, request.reviewer, request.expected_head_sha
+            ),
+            timeout=ISSUE_ACTION_TIMEOUT_SECONDS,
+        )
+    except gitea_proxy.IssueNotAvailableError:
+        return JSONResponse(
+            {"error": "The pull request or reviewer changed. Reload before cancelling the request."},
+            status_code=409,
+        )
+    except Exception:
+        return JSONResponse(
+            {"error": "The cancellation could not be confirmed. Refresh review data before retrying."},
+            status_code=503,
+            headers={"Retry-After": "1"},
+        )
+    return JSONResponse(result)
+
+
 @app.patch("/api/v1/repos/{owner}/{repo}/pulls/{number}/release")
 async def release_assigned_pull(
     owner: str, repo: str, number: int = PathParam(gt=0)

tests/e2e/test_mobile_address_review_feedback_release.py

@@ -0,0 +1,133 @@
+from pathlib import Path
+
+import pytest
+
+
+ROOT = Path(__file__).parents[2]
+
+
+@pytest.mark.parametrize("viewport", [(320, 568), (390, 844)])
+def test_mobile_author_addresses_every_review_comment_and_requests_updated_review(viewport):
+    playwright = pytest.importorskip("playwright.sync_api")
+    html = (ROOT / "frontend" / "index.html").read_text()
+    detail = {
+        "state": "open",
+        "draft": False,
+        "mergeable": True,
+        "merged": False,
+        "ci_state": "success",
+        "head_sha": "current-head",
+        "reviewers": [{
+            "review_id": 42,
+            "login": "sam",
+            "status": "changes_requested",
+            "head_sha": "current-head",
+            "blocking": True,
+            "comments": [
+                {"id": 99, "path": "src/api.py", "body": "Handle the empty state.", "line": 12},
+                {"id": 100, "path": "frontend/app.js", "body": "Explain the retry.", "line": 8},
+            ],
+        }],
+        "files": [
+            {"filename": "src/api.py", "status": "modified", "diff_available": True,
+             "diff_lines": ["@@ -11 +11 @@", "+return empty"]},
+            {"filename": "frontend/app.js", "status": "modified", "diff_available": True,
+             "diff_lines": ["@@ -7 +7 @@", "+showRetry()"]},
+        ],
+    }
+
+    with playwright.sync_playwright() as runtime:
+        try:
+            browser = runtime.chromium.launch(headless=True)
+        except Exception as error:
+            pytest.skip(f"Chromium is not installed: {error}")
+        page = browser.new_page(viewport={"width": viewport[0], "height": viewport[1]})
+        page.set_content(html, wait_until="domcontentloaded")
+        page.add_style_tag(path=ROOT / "frontend" / "dashboard.css")
+        page.add_script_tag(path=ROOT / "frontend" / "pull-sheet.js")
+        page.evaluate(
+            """detail => {
+                globalThis.confirm = () => true;
+                globalThis.feedbackCalls = [];
+                globalThis.feedbackStored = new Map();
+                globalThis.feedbackStorage = {
+                    getItem:key => feedbackStored.has(key) ? feedbackStored.get(key) : null,
+                    setItem:(key, value) => feedbackStored.set(key, String(value)),
+                    removeItem:key => feedbackStored.delete(key),
+                };
+                globalThis.feedbackItem = {repository:'stackchain/api', number:7, key:'stackchain/api#7'};
+                globalThis.feedbackDetail = detail;
+                globalThis.feedbackController = createPullSheet({
+                    storage: feedbackStorage,
+                    createOperationId: () => 'feedback-operation',
+                    fetchJson: async (path, options = {}) => {
+                        feedbackCalls.push({path, options});
+                        if (path.endsWith('/comments')) return {id: 501, body:JSON.parse(options.body).body};
+                        if (path.endsWith('/request-review')) return {
+                            number:7, reviewer:'sam', head_sha:'current-head', requested_reviewers:['sam']
+                        };
+                        throw new Error('unexpected request');
+                    },
+                });
+                document.querySelector('#pull-sheet').classList.add('open');
+                document.querySelector('#pull-review').open = true;
+                document.querySelector('#pull-files').innerHTML = detail.files.map((file, index) =>
+                    createPullSheet.renderFile(file, index, false, value => String(value)
+                        .replaceAll('&', '&amp;').replaceAll('<', '&lt;').replaceAll('>', '&gt;'))
+                ).join('');
+                createPullSheet.bindFeedbackControls(document, feedbackController,
+                    () => feedbackItem, () => feedbackDetail, () => 'timmy');
+                createPullSheet.review(detail, null, document);
+            }""",
+            detail,
+        )
+
+        page.locator(".pull-review-feedback summary").click()
+        page.locator('[data-address-review-feedback="42"]').click()
+        assert page.locator("#pull-feedback-progress").text_content() == "Comment 1 of 2"
+        assert page.locator("#finish-pull-feedback").is_disabled()
+        page.locator('[data-feedback-disposition="addressed"]').click()
+        page.locator("#next-pull-feedback").click()
+        page.locator("#pull-feedback-note").fill("Added a visible retry message.")
+        page.locator('[data-feedback-disposition="discussion"]').click()
+        assert not page.locator("#finish-pull-feedback").is_disabled()
+        page.locator("#finish-pull-feedback").click()
+        page.locator("#request-feedback-review").click()
+
+        page.wait_for_function("feedbackCalls.length === 2")
+        result = page.evaluate(
+            """() => ({
+                calls:feedbackCalls,
+                scrollWidth:document.documentElement.scrollWidth,
+                clientWidth:document.documentElement.clientWidth,
+                controlHeights:Array.from(document.querySelectorAll('#pull-feedback-pass button'))
+                    .filter(button => !button.hidden).map(button => button.getBoundingClientRect().height),
+                status:document.querySelector('#pull-feedback-status').textContent,
+                secondExpanded:document.querySelector('[data-pull-filename="frontend/app.js"] .pull-file-toggle')
+                    .getAttribute('aria-expanded'),
+                persisted:Array.from(feedbackStored.keys()).some(key => key.includes('feedback-pass')),
+            })"""
+        )
+        result["resetHidden"] = page.evaluate(
+            """() => {
+                createPullSheet.resetReviewRequestControls(document, feedbackDetail);
+                return document.querySelector('#pull-feedback-pass').hidden;
+            }"""
+        )
+        browser.close()
+
+    assert result["scrollWidth"] <= result["clientWidth"]
+    assert min(result["controlHeights"]) >= 44
+    assert result["secondExpanded"] == "true"
+    assert result["persisted"] is True
+    assert result["resetHidden"] is True
+    assert result["calls"][0]["path"].endswith("/comments")
+    assert result["calls"][0]["options"]["headers"]["Idempotency-Key"] == "feedback-operation"
+    summary = result["calls"][0]["options"]["body"]
+    assert "Handle the empty state" in summary
+    assert "Addressed" in summary
+    assert "Needs discussion" in summary
+    assert "Added a visible retry message" in summary
+    assert result["calls"][1]["path"].endswith("/request-review")
+    assert '"reviewer":"sam"' in result["calls"][1]["options"]["body"]
+    assert result["status"] == "Response posted; updated review requested from @sam."

tests/e2e/test_mobile_authored_pull_queue_release.py

@@ -0,0 +1,89 @@
+import os
+from pathlib import Path
+
+import pytest
+
+if os.getenv("STACKCHAIN_RUN_RELEASE_E2E") != "1":
+    pytest.skip("packaged mobile My PRs journey runs only in its gated CI job", allow_module_level=True)
+pytest.importorskip("playwright.sync_api")
+from playwright.sync_api import expect, sync_playwright
+
+
+ROOT = Path(__file__).parents[2]
+FRONTEND = ROOT / "frontend"
+
+
+@pytest.mark.parametrize("viewport", [
+    {"width": 320, "height": 568},
+    {"width": 390, "height": 844},
+])
+def test_authored_pull_queue_is_phone_usable_and_opens_the_existing_pull_route(viewport):
+    with sync_playwright() as playwright:
+        browser = playwright.chromium.launch(headless=True)
+        page = browser.new_page(viewport=viewport)
+        page.set_content((FRONTEND / "index.html").read_text())
+        page.add_style_tag(path=FRONTEND / "dashboard.css")
+        page.add_script_tag(path=FRONTEND / "my-work.js")
+        page.add_script_tag(path=FRONTEND / "work-route.js")
+        page.add_script_tag(path=FRONTEND / "mobile-queue-launcher.js")
+        page.evaluate("document.querySelector('#mobile-queue-sheet').showModal()")
+
+        row = page.locator('[data-mobile-queue="authored"]')
+        expect(row).to_have_count(1)
+        expect(row).to_contain_text("My PRs")
+        expect(row).to_contain_text("Pull requests you authored")
+        assert row.bounding_box()["height"] >= 44
+
+        result = page.evaluate("""() => {
+          const work = buildMyWork({
+            user:{login:'timmy'},
+            pull_requests:[{
+              number:41,title:'Ship the authored mobile flow',repository:'stackchain/dashboard',
+              state:'open',user:'timmy',assignees:[],work_reasons:['authored_by_me'],
+              url:'https://forge.example/git/stackchain/dashboard/pulls/41'
+            }]
+          });
+          const authored = filterMyWork(work, 'authored');
+          let selected = '';
+          let route = '';
+          const action = document.createElement('button');
+          action.className = 'my-work-card-main';
+          action.addEventListener('click', () => { route = createWorkRoute.serialize(authored[0]); });
+          const launcher = createMobileQueueLauncher({
+            selectFilter:name => { selected = name; },
+            firstAction:() => action,
+            announce:() => {},
+          });
+          const opened = launcher.open('authored');
+          return {selected, route, opened, count:authored.length};
+        }""")
+        assert result == {
+            "selected": "authored",
+            "route": "#/my-work/pull/stackchain/dashboard/41",
+            "opened": "opened",
+            "count": 1,
+        }
+        assert page.evaluate(
+            "document.documentElement.scrollWidth > document.documentElement.clientWidth"
+        ) is False
+        browser.close()
+
+
+def test_empty_authored_queue_announces_the_standard_empty_message():
+    with sync_playwright() as playwright:
+        browser = playwright.chromium.launch(headless=True)
+        page = browser.new_page(viewport={"width": 320, "height": 568})
+        page.set_content((FRONTEND / "index.html").read_text())
+        page.add_script_tag(path=FRONTEND / "mobile-queue-launcher.js")
+
+        message = page.evaluate("""() => {
+          let announcement = '';
+          const launcher = createMobileQueueLauncher({
+            selectFilter:() => {}, firstAction:() => null,
+            announce:value => { announcement = value; },
+          });
+          launcher.open('authored');
+          return announcement;
+        }""")
+        assert message == "No work is ready to open."
+        browser.close()

tests/e2e/test_mobile_cancel_pull_review_request_release.py

@@ -0,0 +1,88 @@
+import os
+from pathlib import Path
+
+import pytest
+
+
+ROOT = Path(__file__).parents[2]
+
+if os.environ.get("STACKCHAIN_RUN_RELEASE_E2E") != "1":
+    pytest.skip("release browser journey is opt-in", allow_module_level=True)
+
+
+@pytest.mark.parametrize("viewport", [(320, 568), (390, 844)])
+def test_mobile_author_cancels_pending_review_and_opens_replacement_picker(viewport):
+    playwright = pytest.importorskip("playwright.sync_api")
+    html = (ROOT / "frontend" / "index.html").read_text()
+
+    with playwright.sync_playwright() as runtime:
+        try:
+            browser = runtime.chromium.launch(headless=True)
+        except Exception as error:
+            pytest.skip(f"Chromium is not installed: {error}")
+        page = browser.new_page(viewport={"width": viewport[0], "height": viewport[1]})
+        page.set_content(html, wait_until="domcontentloaded")
+        page.add_style_tag(path=ROOT / "frontend" / "dashboard.css")
+        page.add_script_tag(path=ROOT / "frontend" / "pull-sheet.js")
+        page.evaluate(
+            """() => {
+              globalThis.confirm = () => true;
+              globalThis.cancelCalls = [];
+              globalThis.cancelItem = {
+                repository:'stackchain/api', number:7, key:'stackchain/api#7'
+              };
+              globalThis.cancelDetail = {
+                state:'open', draft:false, mergeable:true, merged:false,
+                ci_state:'success', head_sha:'abc1234', files:[],
+                reviewers:[{
+                  login:'sam', status:'waiting', head_sha:'abc1234', blocking:true
+                }],
+              };
+              globalThis.cancelController = createPullSheet({
+                storage:null,
+                fetchJson:async (path, options={}) => {
+                  cancelCalls.push({path, options});
+                  return {
+                    repository:'stackchain/api', number:7, reviewer:'sam',
+                    head_sha:'abc1234', requested_reviewers:[],
+                  };
+                },
+              });
+              createPullSheet.bindReviewRequestControls(
+                document, cancelController, () => cancelItem, () => cancelDetail
+              );
+              createPullSheet.review(cancelDetail, null, document);
+              document.querySelector('#pull-sheet').classList.add('open');
+              document.querySelector('#pull-review').open = true;
+            }"""
+        )
+
+        cancel = page.locator('[data-cancel-review-request="sam"]')
+        bounds = cancel.bounding_box()
+        assert bounds and bounds["height"] >= 44
+        cancel.click()
+        page.wait_for_function("cancelCalls.length === 1")
+        page.wait_for_function(
+            "document.querySelector('#pull-review-request-status').textContent.includes('Choose a replacement reviewer')"
+        )
+        result = page.evaluate(
+            """() => ({
+              calls:cancelCalls,
+              waitingRows:document.querySelectorAll('[data-reviewer-status="waiting"]').length,
+              pickerOpen:document.querySelector('#pull-review-request').open,
+              focused:document.activeElement?.id,
+              scrollWidth:document.documentElement.scrollWidth,
+              clientWidth:document.documentElement.clientWidth,
+            })"""
+        )
+        browser.close()
+
+    assert result["scrollWidth"] <= result["clientWidth"]
+    assert result["waitingRows"] == 0
+    assert result["pickerOpen"] is True
+    assert result["focused"] == "load-pull-reviewers"
+    assert len(result["calls"]) == 1
+    assert result["calls"][0]["options"]["method"] == "DELETE"
+    assert result["calls"][0]["options"]["body"] == (
+        '{"reviewer":"sam","expected_head_sha":"abc1234"}'
+    )

tests/e2e/test_mobile_following_release.py

@@ -58,6 +58,29 @@ def test_following_queue_is_phone_usable_at_narrow_viewport(viewport):
         assert page.locator(".following-card").bounding_box()["height"] >= 44
         assert page.locator("#close-following").bounding_box()["height"] >= 44
 
+        page.evaluate("""() => {
+          document.querySelector('#following-sheet').close();
+          document.querySelector('#search-preview').classList.add('open');
+          renderSearchPreviewReview({status:'ready',data:{ci_state:'success',files:[{
+            filename:'src/a/very/long/mobile/path/that-must-wrap/review.py',status:'modified',
+            additions:1,deletions:1,diff_available:true,
+            diff_lines:['@@ -1 +1 @@','-unsafe long line that scrolls inside the diff only',
+              '+safe long line that scrolls inside the diff only']
+          }]}},document,value=>String(value));
+          document.querySelector('[data-search-preview-section="changes"]').hidden=false;
+        }""")
+        changes = page.locator("#search-preview-review")
+        expect(changes).to_be_visible()
+        expect(changes).to_contain_text("CI passed · 1 changed file.")
+        expect(changes).to_contain_text("review.py")
+        assert page.locator('[data-search-preview-section="changes"]').bounding_box()["height"] >= 44
+        assert page.locator("#search-preview-review .pull-diff").evaluate(
+            "node => node.scrollWidth >= node.clientWidth"
+        ) is True
+        assert page.evaluate(
+            "document.documentElement.scrollWidth > document.documentElement.clientWidth"
+        ) is False
+
         retire = page.locator("#watch-search-result")
         page.evaluate("""() => {
           document.querySelector('#following-sheet').close();

tests/e2e/test_mobile_publish_draft_pull_release.py

@@ -0,0 +1,87 @@
+from pathlib import Path
+
+import pytest
+
+
+ROOT = Path(__file__).parents[2]
+
+
+@pytest.mark.parametrize("viewport", [(320, 568), (390, 844)])
+def test_mobile_author_publishes_draft_and_continues_to_reviewer_picker(viewport):
+    playwright = pytest.importorskip("playwright.sync_api")
+    html = (ROOT / "frontend" / "index.html").read_text()
+    detail = {
+        "number": 7,
+        "title": "WIP: Ship mobile flow",
+        "author": "timmy",
+        "state": "open",
+        "draft": True,
+        "merged": False,
+        "head_sha": "abc1234",
+    }
+
+    with playwright.sync_playwright() as runtime:
+        try:
+            browser = runtime.chromium.launch(headless=True)
+        except Exception as error:
+            pytest.skip(f"Chromium is not installed: {error}")
+        page = browser.new_page(viewport={"width": viewport[0], "height": viewport[1]})
+        page.on("dialog", lambda dialog: dialog.accept())
+        page.set_content(html, wait_until="domcontentloaded")
+        page.add_style_tag(path=ROOT / "frontend" / "dashboard.css")
+        page.add_script_tag(path=ROOT / "frontend" / "pull-sheet.js")
+        page.evaluate(
+            """detail => {
+                window.publishDetail = detail;
+                window.publishItem = {repository:'stackchain/api', number:7, key:'stackchain/api#7'};
+                const controller = createPullSheet({
+                    storage:null,
+                    fetchJson:(path, options) => {
+                        if (!path.endsWith('/ready') || options.method !== 'PATCH') {
+                            return Promise.reject(new Error('unexpected request'));
+                        }
+                        const payload = JSON.parse(options.body);
+                        if (payload.expected_head_sha !== detail.head_sha) {
+                            return Promise.reject(new Error('wrong head'));
+                        }
+                        return Promise.resolve({
+                            repository:'stackchain/api', number:7, title:'Ship mobile flow',
+                            head_sha:detail.head_sha, state:'open', draft:false,
+                        });
+                    },
+                });
+                createPullSheet.bindOwnershipControls(
+                    document, controller, () => window.publishItem, async () => false,
+                    () => window.publishDetail, () => 'timmy'
+                );
+                document.querySelector('#pull-sheet').classList.add('open');
+                controller.setReviewDetail(detail);
+            }""",
+            detail,
+        )
+
+        ready = page.get_by_role("button", name="Ready for review")
+        before = ready.evaluate("button => ({height:button.getBoundingClientRect().height, hidden:button.hidden})")
+        ready.click()
+        page.get_by_text("Published and ready for review.").wait_for()
+        metrics = page.evaluate(
+            """() => ({
+                scrollWidth:document.documentElement.scrollWidth,
+                clientWidth:document.documentElement.clientWidth,
+                readyHidden:document.querySelector('#publish-draft-pull').hidden,
+                requestOpen:document.querySelector('#pull-review-request').open,
+                reviewerFocused:document.activeElement === document.querySelector('#load-pull-reviewers'),
+                title:window.publishDetail.title,
+                draft:window.publishDetail.draft,
+            })"""
+        )
+        browser.close()
+
+    assert before["hidden"] is False
+    assert before["height"] >= 44
+    assert metrics["scrollWidth"] <= metrics["clientWidth"]
+    assert metrics["readyHidden"] is True
+    assert metrics["requestOpen"] is True
+    assert metrics["reviewerFocused"] is True
+    assert metrics["title"] == "Ship mobile flow"
+    assert metrics["draft"] is False

tests/e2e/test_mobile_pull_reviewer_feedback_release.py

@@ -0,0 +1,172 @@
+from pathlib import Path
+
+import pytest
+
+
+ROOT = Path(__file__).parents[2]
+
+
+@pytest.mark.parametrize("viewport", [(320, 568), (390, 844)])
+def test_mobile_received_review_feedback_opens_matching_change(viewport):
+    playwright = pytest.importorskip("playwright.sync_api")
+    html = (ROOT / "frontend" / "index.html").read_text()
+    detail = {
+        "state": "open",
+        "draft": False,
+        "mergeable": True,
+        "merged": False,
+        "ci_state": "success",
+        "head_sha": "current-head",
+        "reviewers": [{
+            "login": "sam",
+            "status": "outdated",
+            "head_sha": "old-head-123456",
+            "blocking": True,
+            "summary": "Please handle the narrow empty state before merging.",
+            "comments": [{
+                "path": "src/a/very/long/mobile/path/review_target.py",
+                "body": "Return before parsing when the payload is empty.",
+                "line": 12,
+            }],
+        }],
+        "files": [{
+            "filename": "src/a/very/long/mobile/path/review_target.py",
+            "status": "modified",
+            "additions": 2,
+            "deletions": 1,
+            "diff_available": True,
+            "diff_lines": ["@@ -10,2 +10,3 @@", "-parse(payload)", "+if not payload: return", "+parse(payload)"],
+        }],
+    }
+
+    with playwright.sync_playwright() as runtime:
+        try:
+            browser = runtime.chromium.launch(headless=True)
+        except Exception as error:
+            pytest.skip(f"Chromium is not installed: {error}")
+        page = browser.new_page(viewport={"width": viewport[0], "height": viewport[1]})
+        page.set_content(html, wait_until="domcontentloaded")
+        page.add_style_tag(path=ROOT / "frontend" / "dashboard.css")
+        page.add_script_tag(path=ROOT / "frontend" / "pull-sheet.js")
+        page.evaluate(
+            """detail => {
+                document.querySelector('#pull-sheet').classList.add('open');
+                document.querySelector('#pull-review').open = true;
+                createPullSheet.review(detail, null, document);
+                document.querySelector('#pull-files').innerHTML = detail.files.map((file, index) =>
+                    createPullSheet.renderFile(file, index, false, value => String(value)
+                        .replaceAll('&', '&amp;').replaceAll('<', '&lt;').replaceAll('>', '&gt;'))
+                ).join('');
+            }""",
+            detail,
+        )
+
+        page.locator(".pull-review-feedback summary").click()
+        jump = page.locator('[data-review-feedback-file]')
+        jump.click()
+        metrics = page.evaluate(
+            """() => {
+                const jump = document.querySelector('[data-review-feedback-file]');
+                const toggle = document.querySelector('.pull-file-toggle');
+                const panel = document.getElementById(toggle.getAttribute('aria-controls'));
+                return {
+                    scrollWidth: document.documentElement.scrollWidth,
+                    clientWidth: document.documentElement.clientWidth,
+                    jumpHeight: jump.getBoundingClientRect().height,
+                    expanded: toggle.getAttribute('aria-expanded'),
+                    panelHidden: panel.hidden,
+                    focused: document.activeElement === toggle,
+                    reviewedHead: document.querySelector('.pull-review-feedback').textContent,
+                    updatedReviewVisible: !document.querySelector('#request-updated-pull-review').hidden,
+                };
+            }"""
+        )
+        browser.close()
+
+    assert metrics["scrollWidth"] <= metrics["clientWidth"]
+    assert metrics["jumpHeight"] >= 44
+    assert metrics["expanded"] == "true"
+    assert metrics["panelHidden"] is False
+    assert metrics["focused"] is True
+    assert "Reviewed head old-head" in metrics["reviewedHead"]
+    assert metrics["updatedReviewVisible"] is True
+
+
+@pytest.mark.parametrize("viewport", [(320, 568), (390, 844)])
+def test_mobile_pull_review_retries_inline_feedback_without_losing_core_review(viewport):
+    playwright = pytest.importorskip("playwright.sync_api")
+    html = (ROOT / "frontend" / "index.html").read_text()
+    detail = {
+        "state": "open", "draft": False, "mergeable": True, "merged": False,
+        "ci_state": "success", "head_sha": "abc1234",
+        "reviewers": [{
+            "login": "sam", "review_id": 8, "status": "changes_requested",
+            "head_sha": "abc1234", "blocking": True,
+            "summary": "Please handle the empty state.",
+        }],
+        "files": [],
+    }
+
+    with playwright.sync_playwright() as runtime:
+        try:
+            browser = runtime.chromium.launch(headless=True)
+        except Exception as error:
+            pytest.skip(f"Chromium is not installed: {error}")
+        page = browser.new_page(viewport={"width": viewport[0], "height": viewport[1]})
+        page.set_content(html, wait_until="domcontentloaded")
+        page.add_style_tag(path=ROOT / "frontend" / "dashboard.css")
+        page.add_script_tag(path=ROOT / "frontend" / "pull-sheet.js")
+        page.evaluate(
+            """detail => {
+                window.lazyFeedbackDetail = detail;
+                window.lazyFeedbackItem = {repository:'stackchain/api', number:7, key:'stackchain/api#7'};
+                let attempts = 0;
+                const controller = createPullSheet({
+                    storage:null,
+                    fetchJson:() => {
+                        attempts += 1;
+                        if (attempts === 1) return Promise.reject(new Error('feedback unavailable'));
+                        return Promise.resolve({
+                            review_id:8, head_sha:'abc1234', reviewed_head_sha:'abc1234',
+                            comments:[{id:99, path:'src/api.py', body:'Return before parsing.', line:12}],
+                        });
+                    },
+                });
+                createPullSheet.bindFeedbackControls(
+                    document, controller, () => window.lazyFeedbackItem,
+                    () => window.lazyFeedbackDetail, () => 'timmy'
+                );
+                document.querySelector('#pull-sheet').classList.add('open');
+                document.querySelector('#pull-review').open = true;
+                const eligibility = createPullSheet.review(detail, null, document);
+                document.querySelector('#pull-merge-state').textContent = eligibility.reason;
+            }""",
+            detail,
+        )
+
+        page.locator(".pull-review-feedback summary").click()
+        load = page.get_by_role("button", name="Load inline feedback")
+        load.click()
+        retry = page.get_by_role("button", name="Retry feedback")
+        retry.wait_for()
+        retry.click()
+        page.get_by_text("Return before parsing.").wait_for()
+        metrics = page.evaluate(
+            """() => {
+                const details = document.querySelector('.pull-review-feedback');
+                const address = document.querySelector('[data-address-review-feedback]');
+                return {
+                    scrollWidth: document.documentElement.scrollWidth,
+                    clientWidth: document.documentElement.clientWidth,
+                    blocker: document.querySelector('#pull-merge-state').textContent,
+                    feedback: details.textContent,
+                    addressHeight: address.getBoundingClientRect().height,
+                };
+            }"""
+        )
+        browser.close()
+
+    assert metrics["scrollWidth"] <= metrics["clientWidth"]
+    assert "requested changes" in metrics["blocker"]
+    assert "Return before parsing." in metrics["feedback"]
+    assert metrics["addressHeight"] >= 44

tests/e2e/test_mobile_pull_reviewer_status_release.py

@@ -0,0 +1,87 @@
+from pathlib import Path
+
+import pytest
+
+
+ROOT = Path(__file__).parents[2]
+
+
+@pytest.mark.parametrize("viewport", [(320, 568), (390, 844)])
+def test_mobile_pull_reviewer_status_wraps_and_keeps_updated_review_reachable(viewport):
+    playwright = pytest.importorskip("playwright.sync_api")
+    html = (ROOT / "frontend" / "index.html").read_text()
+    detail = {
+        "state": "open",
+        "draft": False,
+        "mergeable": True,
+        "merged": False,
+        "ci_state": "success",
+        "head_sha": "current-head",
+        "reviewers": [
+            {
+                "login": "casey-with-a-very-long-mobile-login",
+                "status": "waiting",
+                "head_sha": "current-head",
+                "blocking": True,
+            },
+            {
+                "login": "sam",
+                "status": "changes_requested",
+                "head_sha": "current-head",
+                "blocking": True,
+            },
+            {
+                "login": "lee",
+                "status": "outdated",
+                "head_sha": "old-head",
+                "blocking": True,
+            },
+            {
+                "login": "pat",
+                "status": "approved",
+                "head_sha": "current-head",
+                "blocking": False,
+            },
+        ],
+    }
+
+    with playwright.sync_playwright() as runtime:
+        try:
+            browser = runtime.chromium.launch(headless=True)
+        except Exception as error:
+            pytest.skip(f"Chromium is not installed: {error}")
+        page = browser.new_page(viewport={"width": viewport[0], "height": viewport[1]})
+        page.set_content(html, wait_until="domcontentloaded")
+        page.add_style_tag(path=ROOT / "frontend" / "dashboard.css")
+        page.add_script_tag(path=ROOT / "frontend" / "pull-sheet.js")
+        page.evaluate(
+            """detail => {
+                document.querySelector('#pull-sheet').classList.add('open');
+                document.querySelector('#pull-review').open = true;
+                createPullSheet.review(detail, null, document);
+            }""",
+            detail,
+        )
+
+        metrics = page.evaluate(
+            """() => ({
+                scrollWidth: document.documentElement.scrollWidth,
+                clientWidth: document.documentElement.clientWidth,
+                updatedReviewHeight: document.querySelector('#request-updated-pull-review')
+                    .getBoundingClientRect().height,
+                updatedReviewVisible: !document.querySelector('#request-updated-pull-review').hidden,
+                statuses: Array.from(document.querySelectorAll('.pull-reviewer-status span'))
+                    .map(node => node.textContent),
+            })"""
+        )
+        browser.close()
+
+    assert metrics["scrollWidth"] <= metrics["clientWidth"]
+    assert metrics["updatedReviewHeight"] >= 44
+    assert metrics["updatedReviewVisible"] is True
+    assert metrics["statuses"] == [
+        "Waiting for review",
+        "Changes requested",
+        "Outdated after new commits",
+        "Approved current head",
+    ]

tests/e2e/test_mobile_search_preview_navigation.py

@@ -51,8 +51,10 @@ def test_rendered_mobile_search_preview_navigation_preserves_reading_space_and_d
         assert watch_bounds and watch_bounds["height"] >= 44
         navigation = page.locator(".mobile-search-preview-nav")
         expect(navigation).to_be_visible()
-        buttons = navigation.locator("button")
+        buttons = navigation.locator("button:visible")
+        assert navigation.locator("button").count() == 5
         assert buttons.count() == 4
+        expect(navigation.locator('[data-search-preview-section="changes"]')).to_be_hidden()
         for button in buttons.all():
             bounds = button.bounding_box()
             assert bounds and bounds["height"] >= 44

tests/test_ci_workflow.py

@@ -73,7 +73,9 @@ def test_release_promotion_waits_for_packaged_mobile_journeys():
         "tests/e2e/test_mobile_today_week_reschedule_release.py "
         "tests/e2e/test_mobile_wrap_up_handoff_release.py "
         "tests/e2e/test_mobile_following_release.py "
-        "tests/e2e/test_mobile_detail_watch_release.py -q"
+        "tests/e2e/test_mobile_detail_watch_release.py "
+        "tests/e2e/test_mobile_pull_reviewer_status_release.py "
+        "tests/e2e/test_mobile_pull_reviewer_feedback_release.py -q"
     ) in browser
     assert "needs: [lint, build-release, browser-journey]" in release
 
@@ -83,3 +85,18 @@ def test_browser_job_runs_packaged_today_week_reschedule_journey():
     browser = text[text.index("  browser-journey:") : text.index("  release-candidate:")]
 
     assert "tests/e2e/test_mobile_today_week_reschedule_release.py" in browser
+
+
+def test_browser_job_gates_latest_pull_review_mobile_journeys():
+    text = WORKFLOW.read_text()
+    browser = text[text.index("  browser-journey:") : text.index("  release-candidate:")]
+
+    assert "tests/e2e/test_mobile_address_review_feedback_release.py" in browser
+    assert "tests/e2e/test_mobile_cancel_pull_review_request_release.py" in browser
+
+
+def test_browser_job_gates_authored_pull_queue_journey():
+    text = WORKFLOW.read_text()
+    browser = text[text.index("  browser-journey:") : text.index("  release-candidate:")]
+
+    assert "tests/e2e/test_mobile_authored_pull_queue_release.py" in browser

tests/test_command_palette.py

@@ -1334,7 +1334,7 @@ def test_closed_issue_preview_reopens_then_resumes_through_capacity_guard():
     html = dashboard_bundle_text()
 
     assert "detail.reopenable ? 'Reopen & resume'" in html
-    assert "mutate: (detail, action) => fetchReviewJson(" in html
+    assert "mutate:(detail,action)=>fetchReviewJson(" in html
     handler = html.split("qs('#start-search-result').addEventListener('click'", 1)[1].split(
         "qs('#close-whiteboard')", 1
     )[0]

tests/test_comment_next.py

@@ -303,4 +303,4 @@ async def test_unread_update_offers_reply_mark_read_and_next_independent_of_toda
     assert '.update-reply-actions { display:grid; grid-template-columns:repeat(2,minmax(0,1fr));' in html
     assert '.update-reply-actions button { min-height:44px;' in html
     worker = (Path(__file__).parents[1] / "frontend" / "service-worker.js").read_text()
-    assert "stackchain-dashboard-shell-v136" in worker
+    assert "stackchain-dashboard-shell-v138" in worker

tests/test_following_api.py

@@ -103,6 +103,63 @@ async def test_confirmed_pull_watch_round_trips_through_following(monkeypatch, t
     ]
 
 
+@pytest.mark.anyio
+async def test_watched_pull_can_load_read_only_review_context_without_assignment(monkeypatch, tmp_path):
+    store = FollowingStore(tmp_path / "following.sqlite3", encryption_key=b"r" * 32)
+    store.set_watching("timmy", {
+        "repository": "stackchain/api", "kind": "pull", "number": 84,
+        "title": "Review watched changes", "state": "open",
+        "updated_at": "2026-08-24T05:00:00Z",
+        "url": "https://forge.example/stackchain/api/pulls/84",
+    }, True)
+    review = {
+        "head_sha": "abc123", "ci_state": "success",
+        "files": [{"filename": "src/api.py", "status": "modified", "patch": "+safe"}],
+    }
+
+    async def user():
+        return {"login": "timmy"}
+
+    async def load_review(repository, number):
+        assert (repository, number) == ("stackchain/api", 84)
+        return review
+
+    monkeypatch.setattr(main, "_following_store", lambda: store)
+    monkeypatch.setattr(main, "current_user", user)
+    monkeypatch.setattr(main.gitea_proxy, "pull_completion_review", load_review)
+    transport = httpx.ASGITransport(app=main.app)
+    async with httpx.AsyncClient(transport=transport, base_url="http://test") as client:
+        response = await client.get(
+            "/api/v1/following/stackchain/api/pulls/84/review-data"
+        )
+
+    assert response.status_code == 200
+    assert response.headers["cache-control"] == "no-store"
+    assert response.json() == review
+
+
+@pytest.mark.anyio
+async def test_unwatched_pull_review_context_is_not_exposed(monkeypatch, tmp_path):
+    store = FollowingStore(tmp_path / "following.sqlite3", encryption_key=b"u" * 32)
+
+    async def user():
+        return {"login": "timmy"}
+
+    async def unexpected_review(*_args):
+        raise AssertionError("unwatched pull must be rejected before upstream access")
+
+    monkeypatch.setattr(main, "_following_store", lambda: store)
+    monkeypatch.setattr(main, "current_user", user)
+    monkeypatch.setattr(main.gitea_proxy, "pull_completion_review", unexpected_review)
+    transport = httpx.ASGITransport(app=main.app)
+    async with httpx.AsyncClient(transport=transport, base_url="http://test") as client:
+        response = await client.get(
+            "/api/v1/following/stackchain/api/pulls/84/review-data"
+        )
+
+    assert response.status_code == 404
+
+
 @pytest.mark.anyio
 async def test_full_following_collection_rejects_watch_before_gitea_mutation(monkeypatch, tmp_path):
     store = FollowingStore(tmp_path / "following.sqlite3", encryption_key=b"b" * 32, limit=1)

tests/test_following_frontend.py

@@ -176,6 +176,76 @@ const preview=createSearchPreview({{
     }
 
 
+def test_following_pull_revision_waits_for_changes_and_ci_context():
+    preview_module = ROOT / "frontend" / "search-preview.js"
+    script = f"""
+const createSearchPreview=require({json.dumps(str(preview_module))});
+let unavailable=true;
+const opened=[];
+const states=[];
+const preview=createSearchPreview({{
+  fetchJson:async item=>({{...item,title:'Changed pull request'}}),
+  fetchConversation:async()=>({{comments:[]}}),
+  fetchReview:async()=>{{if(unavailable)throw new Error('diff offline');return {{ci_state:'success',files:[{{filename:'src/api.py',diff_lines:['+safe'],diff_available:true}}]}};}},
+  onOpened:async item=>opened.push(item.updated_at),
+  onState:state=>states.push({{status:state.review?.status,ci:state.review?.data?.ci_state}}),
+}});
+(async()=>{{
+  await preview.open({{
+    repository:'stackchain/api',kind:'pull',number:84,following:true,
+    reviewed_at:'2026-08-23T03:00:00Z',updated_at:'2026-08-23T04:00:00Z'
+  }});
+  const afterFailure=[...opened];
+  unavailable=false;
+  await preview.retryReview();
+  process.stdout.write(JSON.stringify({{afterFailure,opened,last:states.at(-1)}}));
+}})().catch(error=>{{console.error(error);process.exit(1);}});
+"""
+
+    result = json.loads(subprocess.run(
+        ["node", "-e", script], text=True, capture_output=True, check=True
+    ).stdout)
+
+    assert result == {
+        "afterFailure": [],
+        "opened": ["2026-08-23T04:00:00Z"],
+        "last": {"status": "ready", "ci": "success"},
+    }
+
+
+def test_watched_pull_changes_render_bounded_diff_and_ci_state():
+    preview_module = ROOT / "frontend" / "search-preview.js"
+    script = f"""
+require({json.dumps(str(preview_module))});
+const nodes={{
+  '#search-preview-review':{{hidden:true}},
+  '#search-preview-review-status':{{textContent:''}},
+  '#search-preview-files':{{innerHTML:''}},
+  '#retry-search-preview-review':{{hidden:true}},
+}};
+const document={{querySelector:selector=>nodes[selector]}};
+global.renderSearchPreviewReview({{status:'ready',data:{{ci_state:'success',files:[{{
+  filename:'src/api.py',status:'modified',additions:1,deletions:0,
+  diff_available:true,diff_lines:['@@ -1 +1 @@','-unsafe','+safe']
+}}]}}}},document,value=>String(value).replaceAll('<','&lt;'));
+process.stdout.write(JSON.stringify({{
+  hidden:nodes['#search-preview-review'].hidden,
+  status:nodes['#search-preview-review-status'].textContent,
+  html:nodes['#search-preview-files'].innerHTML,
+  retry:nodes['#retry-search-preview-review'].hidden,
+}}));
+"""
+    result = json.loads(subprocess.run(
+        ["node", "-e", script], text=True, capture_output=True, check=True
+    ).stdout)
+
+    assert result["hidden"] is False
+    assert result["status"] == "CI passed · 1 changed file."
+    assert "src/api.py" in result["html"]
+    assert "pull-diff-line added" in result["html"]
+    assert result["retry"] is True
+
+
 def test_following_typed_identity_prevents_issue_pull_collisions():
     script = f"""
 const createFollowing = require({json.dumps(str(MODULE))});
@@ -487,9 +557,43 @@ def test_following_review_controls_are_wired_into_the_phone_preview_flow():
     assert "afterUnwatch:item => followingQueue.retire(item)" in dashboard
     assert "followingQueue.returnToFollowing()" in dashboard
     assert "'Back to Following'" in dashboard
+
+
+def test_following_open_issue_exposes_only_action_and_continue_planning_dispositions():
+    script = f"""
+const createFollowing = require({json.dumps(str(MODULE))});
+const disposition = createFollowing.planningDisposition;
+process.stdout.write(JSON.stringify({{
+  active:disposition({{following:true,kind:'issue',state:'open',claimable:true}}),
+  generic:disposition({{following:false,kind:'issue',state:'open',claimable:true}}),
+  pull:disposition({{following:true,kind:'pull',state:'open',assigned_to_me:true}}),
+}}));
+"""
+    result = json.loads(subprocess.run(
+        ["node", "-e", script], text=True, capture_output=True, check=True
+    ).stdout)
+
+    assert result == {
+        "active": {
+            "active": True,
+            "today": "Add to Today & next",
+            "later": "Later & next",
+            "week": "Week Ahead & next",
+        },
+        "generic": {"active": False},
+        "pull": {"active": False},
+    }
+
+    dashboard = (ROOT / "frontend" / "dashboard.js").read_text()
+    css = (ROOT / "frontend" / "dashboard.css").read_text()
+    service_worker = (ROOT / "frontend" / "service-worker.js").read_text()
+    assert "followingQueue.preview(state);" in dashboard
+    assert "following-disposition-mode" in MODULE.read_text()
+    assert "controls.claim.hidden = controls.start.hidden = controls.watch.hidden = true" in MODULE.read_text()
+    assert ".following-disposition-mode" in css
     assert "if (searchPreviewReturnKind === 'following')" in dashboard
     assert "e.key === 'Escape' && searchPreviewReturnKind === 'following'" in dashboard
-    assert "stackchain-dashboard-shell-v136" in service_worker
+    assert "stackchain-dashboard-shell-v138" in service_worker
 
 
 def test_prepare_today_lazily_refreshes_and_directly_reviews_following():
@@ -504,8 +608,8 @@ def test_prepare_today_lazily_refreshes_and_directly_reviews_following():
     assert "const completed = feature.finishReview()" in following
     assert "if (completed) return 'completed-following'" in following
     assert "followingQueue.load().catch(() => {})" in dashboard
-    assert "mobileQueueCounts.following = count" in dashboard
-    assert "mobileQueueCounts.followingUnavailable = status === 'error'" in dashboard
-    assert "mobilePreparationItems.following = items.filter" in dashboard
-    assert "mobileQueueCounts.followingUnavailable ? followingQueue.open()" in dashboard
+    assert "queueCounts.following = count" in dashboard
+    assert "queueCounts.followingUnavailable = status === 'error'" in dashboard
+    assert "preparationItems.following = items.filter" in dashboard
+    assert "queueCounts.followingUnavailable ? followingQueue.open()" in dashboard
     assert "mobileStartDay.completePhase('following')" in dashboard

tests/test_gitea_work_search.py

@@ -100,6 +100,32 @@ async def test_filed_work_page_loads_open_and_closed_authored_issues_with_a_dist
     }
 
 
+@pytest.mark.anyio
+async def test_authored_pull_page_loads_open_created_pulls_with_a_distinct_reason():
+    requests = []
+
+    def upstream(request):
+        requests.append(str(request.url))
+        return httpx.Response(200, headers={"X-Total-Count": "3"}, json=[{
+            "id": 901, "number": 41, "title": "Ship mobile flow", "state": "open",
+            "user": {"login": "timmy"},
+            "repository": {"full_name": "stackchain/dashboard"},
+            "html_url": "http://127.0.0.1:3000/stackchain/dashboard/pulls/41",
+        }])
+
+    gitea_proxy.start_client(transport=httpx.MockTransport(upstream))
+    try:
+        result = await gitea_proxy.work_page("authored", page=1)
+    finally:
+        await gitea_proxy.stop_client()
+
+    assert requests == [
+        "http://127.0.0.1:3000/api/v1/repos/issues/search?state=open&created=true&type=pulls&limit=50&page=1"
+    ]
+    assert result["total"] == 3
+    assert result["items"][0]["work_reasons"] == ["authored_by_me"]
+
+
 @pytest.mark.anyio
 async def test_issue_context_merges_self_assigned_and_filed_streams_without_duplicates(monkeypatch):
     async def page(stream, page=1, limit=50):
@@ -628,7 +654,7 @@ async def test_confirmed_claim_is_removed_from_retained_available_snapshot(monke
 async def test_initial_work_collections_expose_independent_pagination(monkeypatch):
     async def fake_page(stream, page=1, limit=50):
         assert page == 1
-        totals = {"issue": 84, "filed": 62, "pull": 61, "review": 73}
+        totals = {"issue": 84, "filed": 62, "pull": 61, "review": 73, "authored": 12}
         return {
             "items": [], "page": 1, "total": totals[stream],
             "has_more": True, "stream": stream,
@@ -646,6 +672,7 @@ async def test_initial_work_collections_expose_independent_pagination(monkeypatc
     assert pulls.pagination == {
         "pull": {"page": 1, "total": 61, "has_more": True},
         "review": {"page": 1, "total": 73, "has_more": True},
+        "authored": {"page": 1, "total": 12, "has_more": True},
     }
 
 
@@ -666,11 +693,12 @@ async def test_work_collections_include_supported_review_request_search(monkeypa
         ("filed", 1, 50),
         ("pull", 1, 50),
         ("review", 1, 50),
+        ("authored", 1, 50),
     ]
 
 
 @pytest.mark.anyio
-async def test_pull_requests_merge_assignment_and_review_responsibilities(monkeypatch):
+async def test_pull_requests_merge_assignment_review_and_authored_responsibilities(monkeypatch):
     assigned = {
         "id": 11,
         "number": 7,
@@ -683,10 +711,24 @@ async def test_pull_requests_merge_assignment_and_review_responsibilities(monkey
         "title": "Review mobile",
         "repository": {"full_name": "stackchain/mobile"},
     }
+    authored_only = {
+        "id": 13,
+        "number": 9,
+        "title": "Ship authored flow",
+        "repository": {"full_name": "stackchain/dashboard"},
+    }
 
     async def fake_page(stream, page=1, limit=50):
-        items = [assigned] if stream == "pull" else [assigned.copy(), review_only]
-        reason = "assigned_to_me" if stream == "pull" else "review_requested"
+        items = {
+            "pull": [assigned],
+            "review": [assigned.copy(), review_only],
+            "authored": [assigned.copy(), authored_only],
+        }[stream]
+        reason = {
+            "pull": "assigned_to_me",
+            "review": "review_requested",
+            "authored": "authored_by_me",
+        }[stream]
         return {
             "stream": stream,
             "items": [{**item, "work_reasons": [reason]} for item in items],
@@ -697,9 +739,44 @@ async def test_pull_requests_merge_assignment_and_review_responsibilities(monkey
 
     pulls = await gitea_proxy.pull_requests()
 
-    assert [pull["id"] for pull in pulls] == [11, 12]
-    assert pulls[0]["work_reasons"] == ["assigned_to_me", "review_requested"]
+    assert [pull["id"] for pull in pulls] == [11, 12, 13]
+    assert pulls[0]["work_reasons"] == [
+        "assigned_to_me", "review_requested", "authored_by_me",
+    ]
     assert pulls[1]["work_reasons"] == ["review_requested"]
+    assert pulls[2]["work_reasons"] == ["authored_by_me"]
+    assert pulls.pagination["authored"] == {
+        "page": 1, "total": 2, "has_more": False,
+    }
+
+
+@pytest.mark.anyio
+async def test_authored_pull_failure_keeps_other_pull_work_and_marks_only_that_stream_unavailable(monkeypatch):
+    async def fake_page(stream, page=1, limit=50):
+        if stream == "authored":
+            raise httpx.ConnectError("authored search unavailable")
+        item = {
+            "id": 11 if stream == "pull" else 12,
+            "number": 7 if stream == "pull" else 8,
+            "title": "Assigned" if stream == "pull" else "Review",
+            "repository": {"full_name": "stackchain/dashboard"},
+            "work_reasons": ["assigned_to_me" if stream == "pull" else "review_requested"],
+        }
+        return {
+            "stream": stream, "items": [item], "page": page,
+            "total": 1, "has_more": False,
+        }
+
+    monkeypatch.setattr(gitea_proxy, "work_page", fake_page)
+
+    pulls = await gitea_proxy.pull_requests()
+
+    assert [pull["id"] for pull in pulls] == [11, 12]
+    assert pulls.pagination["pull"].get("unavailable", False) is False
+    assert pulls.pagination["review"].get("unavailable", False) is False
+    assert pulls.pagination["authored"] == {
+        "page": 1, "total": 0, "has_more": False, "unavailable": True,
+    }
 
 
 @pytest.mark.anyio

tests/test_later_sync.py

@@ -435,5 +435,5 @@ async def test_dashboard_syncs_every_later_change_and_exposes_account_status():
 def test_later_sync_ships_atomically_in_the_offline_shell():
     source = (Path(__file__).parents[1] / "frontend" / "service-worker.js").read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/later-sync.js'" in source

tests/test_markdown_renderer.py

@@ -256,4 +256,4 @@ def test_markdown_work_bodies_are_mobile_safe_block_containers():
     assert ".markdown-content { min-width:0; max-width:100%; overflow-wrap:anywhere;" in css
     assert ".markdown-content pre { max-width:100%; overflow-x:auto;" in css
     assert ".markdown-content a { min-height:44px;" in css
-    assert "stackchain-dashboard-shell-v136" in worker
+    assert "stackchain-dashboard-shell-v138" in worker

tests/test_mobile_app_badge.py

@@ -45,12 +45,12 @@ const controller = createMobileAppBadge({
   },
 });
 controller.start();
-await controller.reconcile(7, {authoritative:true});
-await controller.reconcile(12, {authoritative:false});
+await controller.reconcile('updates', 7, true);
+await controller.reconcile('updates', 12, false);
 control.checked = true;
 await listeners.change();
-await controller.reconcile(7, {authoritative:true});
-await controller.reconcile(0, {authoritative:true});
+await controller.reconcile('updates', 7, true);
+await controller.reconcile('updates', 0, true);
 console.log(JSON.stringify({values, clears, saved:storage.get('stackchain.app-badge.enabled.v1'), status:status.textContent}));
 """)
     assert result == {
@@ -61,6 +61,33 @@ console.log(JSON.stringify({values, clears, saved:storage.get('stackchain.app-ba
     }
 
 
+def test_app_badge_combines_authoritative_update_and_following_channels_independently():
+    result = run_badge("""
+const values = [];
+let clears = 0;
+const controller = createMobileAppBadge({
+  control:{checked:true, disabled:false, addEventListener() {}},
+  status:{textContent:''}, container:{hidden:false},
+  navigator:{
+    async setAppBadge(value) { values.push(value); },
+    async clearAppBadge() { clears++; },
+  },
+  storage:{getItem() { return 'true'; }, setItem() {}, removeItem() {}},
+});
+controller.start();
+await controller.reconcile('updates', 3, true);
+await controller.reconcile('following', 2, true);
+await controller.reconcile('following', 0, true);
+await controller.reconcile('following', 4, false);
+await controller.reconcile('updates', 0, true);
+await controller.reconcile('following', 2, true);
+await controller.reconcile('updates', 5, true);
+await controller.reconcile('following', 2, true);
+console.log(JSON.stringify({values, clears}));
+""")
+    assert result == {"values": [3, 5, 3, 2, 7], "clears": 1}
+
+
 def test_app_badge_hides_unsupported_device_control_without_touching_storage():
     result = run_badge("""
 const control = {checked:false, disabled:false, addEventListener() { throw new Error('must not wire'); }};
@@ -92,9 +119,11 @@ def test_app_badge_is_packaged_touch_safe_and_reconciled_from_server_total():
     assert 'id="app-badge-control"' in index
     assert 'id="app-badge-status" role="status"' in index
     assert ".app-badge-control { min-height:44px;" in css
-    assert "const mobileAppBadge = createMobileAppBadge({" in dashboard
-    assert "mobileAppBadge.start();" in dashboard
-    assert "mobileAppBadge.reconcile(pagination.total, {authoritative:true});" in dashboard
+    assert "const appBadge = createMobileAppBadge({" in dashboard
+    assert "appBadge.start();" in dashboard
+    assert "appBadge.reconcile('updates', pagination.total, true);" in dashboard
+    assert "appBadge.reconcile('following', count, true);" in dashboard
+    assert "Show review attention on the app icon" in index
 
 
 def test_app_badge_exposes_device_setup_readiness_and_enable_action():
@@ -113,7 +142,7 @@ const controller = createMobileAppBadge({
   },
 });
 controller.start();
-await controller.reconcile(4, {authoritative:true});
+await controller.reconcile('updates', 4, true);
 const before = controller.readiness();
 await controller.enable();
 const after = controller.readiness();
@@ -122,11 +151,11 @@ console.log(JSON.stringify({before, after, checked:control.checked, values}));
     assert result == {
         "before": {
             "state": "incomplete",
-            "detail": "Show the confirmed unread Updates count without opening Stackchain.",
+            "detail": "Show review attention without opening Stackchain.",
         },
         "after": {
             "state": "complete",
-            "detail": "The app icon shows the confirmed unread Updates count.",
+            "detail": "The app icon shows review attention.",
         },
         "checked": True,
         "values": [4],
@@ -142,8 +171,8 @@ def test_device_setup_offers_app_badge_as_an_optional_supported_step():
     assert 'id="device-setup-app-badge" type="button"' in index
     assert "['appBadge', options.appBadgeButton, options.appBadgeStatus, options.enableAppBadge, 'Enable']" in setup
     assert "appBadge:options.appBadgeReadiness()," in setup
-    assert "appBadgeReadiness:() => mobileAppBadge.readiness()," in dashboard
-    assert "enableAppBadge:() => mobileAppBadge.enable()," in dashboard
+    assert "appBadgeReadiness:() => appBadge.readiness()," in dashboard
+    assert "enableAppBadge:() => appBadge.enable()," in dashboard
 
 
 def test_app_badge_api_failure_is_reported_without_rejecting_live_refresh():
@@ -156,7 +185,7 @@ const controller = createMobileAppBadge({
   storage:{getItem() { return 'true'; }, setItem() {}, removeItem() {}},
 });
 controller.start();
-const reconciled = await controller.reconcile(3, {authoritative:true});
+const reconciled = await controller.reconcile('updates', 3, true);
 console.log(JSON.stringify({reconciled, status:status.textContent}));
 """)
     assert result == {
@@ -210,3 +239,51 @@ console.log(JSON.stringify(messages));
         {"type": "stackchain-app-badge-preference", "enabled": True},
         {"type": "stackchain-app-badge-preference", "enabled": False},
     ]
+
+
+def test_app_badge_synchronizes_authoritative_channel_counts_with_worker():
+    result = run_badge("""
+const messages = [];
+const controller = createMobileAppBadge({
+  control:{checked:true, disabled:false, addEventListener() {}}, status:{textContent:''}, container:{hidden:false},
+  navigator:{async setAppBadge() {}, async clearAppBadge() {}},
+  storage:{getItem() { return 'true'; }, setItem() {}, removeItem() {}},
+  serviceWorker:{ready:Promise.resolve({active:{postMessage(message) { messages.push(message); }}})},
+});
+controller.start();
+await controller.reconcile('updates', 3, true);
+await controller.reconcile('following', 2, true);
+await controller.reconcile('following', 9, false);
+await new Promise(resolve => setImmediate(resolve));
+console.log(JSON.stringify(messages));
+""")
+    assert result == [
+        {"type": "stackchain-app-badge-preference", "enabled": True},
+        {"type": "stackchain-app-badge-count", "channel": "updates", "count": 3},
+        {"type": "stackchain-app-badge-count", "channel": "following", "count": 2},
+    ]
+
+
+def test_enabling_badge_synchronizes_counts_confirmed_while_opted_out():
+    result = run_badge("""
+const listeners = {};
+const messages = [];
+const control = {checked:false, disabled:false, addEventListener(name, callback) { listeners[name] = callback; }};
+const controller = createMobileAppBadge({
+  control, status:{textContent:''}, container:{hidden:false},
+  navigator:{async setAppBadge() {}, async clearAppBadge() {}},
+  storage:{getItem() { return null; }, setItem() {}, removeItem() {}},
+  serviceWorker:{ready:Promise.resolve({active:{postMessage(message) { messages.push(message); }}})},
+});
+controller.start();
+await controller.reconcile('updates', 3, true);
+await controller.reconcile('following', 2, true);
+control.checked = true;
+await listeners.change();
+console.log(JSON.stringify(messages));
+""")
+    assert result == [
+        {"type": "stackchain-app-badge-preference", "enabled": True},
+        {"type": "stackchain-app-badge-count", "channel": "updates", "count": 3},
+        {"type": "stackchain-app-badge-count", "channel": "following", "count": 2},
+    ]

tests/test_mobile_composer_integration.py

@@ -45,7 +45,7 @@ def test_offline_shell_contains_every_local_dashboard_runtime_asset():
     shell_assets = set(re.findall(r"BASE \+ '([^']+)'", worker.split("async function sessionCsrf", 1)[0]))
 
     assert local_assets <= shell_assets, f"Offline shell is missing: {sorted(local_assets - shell_assets)}"
-    assert "stackchain-dashboard-shell-v136" in worker
+    assert "stackchain-dashboard-shell-v138" in worker
 
 
 def test_all_conversation_composers_offer_accessible_mobile_mentions():

tests/test_mobile_device_setup.py

@@ -383,7 +383,7 @@ def test_mobile_dashboard_mounts_phone_safe_device_setup_flow():
     assert "controller.recoverPermission('deadline')" in dashboard
     assert "pushControllerReady.then(ensureDeviceSetup)" in dashboard
     assert "BASE + 'static/mobile-device-setup.js'" in worker
-    assert "stackchain-dashboard-shell-v136" in worker
+    assert "stackchain-dashboard-shell-v138" in worker
     assert ".device-setup-panel" in css
     assert ".device-readiness-card" in css
     assert "overflow-x:hidden" in css

tests/test_mobile_insights.py

@@ -243,5 +243,5 @@ async def test_mobile_home_progressively_discloses_secondary_panels_as_insights(
 def test_mobile_insights_rolls_into_the_offline_shell():
     worker = (CONTROLLER.parent / "service-worker.js").read_text()
 
-    assert "stackchain-dashboard-shell-v136" in worker
+    assert "stackchain-dashboard-shell-v138" in worker
     assert "BASE + 'static/mobile-insights.js'" in worker

tests/test_mobile_start_day.py

@@ -407,14 +407,14 @@ async def test_dashboard_wires_thumb_safe_start_day_briefing_into_offline_mobile
     assert "mobileStartDay.finish()" in html
     assert "openQueue: name =>" in html
     assert "if (sheet.open) sheet.close();" in html
-    assert "mobileQueueCounts.followingUnavailable ? followingQueue.open()" in html
+    assert "queueCounts.followingUnavailable ? followingQueue.open()" in html
     assert "Checking Agenda, Attention, Updates, Filed, and Following" in html
     assert "mobileStartDay.render();" in html
     assert ".mobile-start-day-action { width:100%; min-height:48px;" in html
     assert ".mobile-start-day-finish { min-height:44px;" in html
     assert "max-width:100%; overflow-wrap:anywhere;" in html
     assert "BASE + 'static/mobile-start-day.js'" in service_worker
-    assert "stackchain-dashboard-shell-v136" in service_worker
+    assert "stackchain-dashboard-shell-v138" in service_worker
 
 
 @pytest.mark.anyio
@@ -427,9 +427,9 @@ async def test_primary_mobile_work_action_opens_a_unique_item_prepare_today_brie
     assert "qs('#mobile-queue-heading').textContent = state.active ? 'Resume Prepare Today' : 'Prepare Today';" in html
     assert "qs('#mobile-queue-sheet').showModal();" in html
     assert "qs('#mobile-start-day-action').focus();" in html
-    assert "let mobilePreparationItems = {};" in html
-    assert "getPhaseItems: () => mobilePreparationItems" in html
-    assert "mobilePreparationItems = {" in html
+    assert "let preparationItems = {};" in html
+    assert "getPhaseItems: () => preparationItems" in html
+    assert "preparationItems = {" in html
     assert "delivery:draftInbox.partition(lastDrafts).deliveries" in html
     assert "agenda:agendaMyWork(activeMyWork)" in html
     assert "attention:activeMyWork.filter(item => item.needs_attention)" in html

tests/test_mobile_task_dock.py

@@ -835,7 +835,7 @@ async def test_dashboard_renders_and_wires_mobile_queue_switcher():
     assert "const mobileQueueLauncher = createMobileQueueLauncher({" in html
     assert "firstAction: name => qs('#my-work-list .my-work-card-main, #my-work-list .draft-resume, #my-work-list .draft-continue, #my-work-list .draft-edit')" in html
     assert "name === 'find' ? qs('#find-work').click() :" in html
-    assert "mobileQueueCounts.followingUnavailable ? followingQueue.open()" in html
+    assert "queueCounts.followingUnavailable ? followingQueue.open()" in html
     assert '.mobile-queue-sheet' in html
     assert '.my-work-actions { display:none;' in html
     assert 'padding-bottom:calc(16px + env(safe-area-inset-bottom))' in html
@@ -873,7 +873,7 @@ async def test_dashboard_wires_work_dock_and_app_shortcut_to_live_queue_recommen
     assert "queueLauncher: mobileQueueLauncher" in html
     assert "work: () => mobileWorkEntry.open()" in html
     assert "continueWork: () => mobileWorkEntry.open()" in html
-    assert "mobileQueueCounts = counts;" in html
+    assert "queueCounts = counts;" in html
     assert "mobileTaskDock.updateWork(mobileWorkEntry.mode())" in html
 
 
@@ -895,7 +895,7 @@ async def test_dashboard_renders_and_wires_phone_safe_task_dock():
     assert "createMobileTaskDock({" in html
     assert "work: () => mobileWorkEntry.open()" in html
     assert "name === 'find' ? qs('#find-work').click() :" in html
-    assert "mobileQueueCounts.followingUnavailable ? followingQueue.open()" in html
+    assert "queueCounts.followingUnavailable ? followingQueue.open()" in html
     assert "qs('[data-work-filter=\"' + name + '\"]').click()" in html
     assert "find: () => qs('#find-work').click()" in html
     assert "new: () => qs('#new-issue').click()" in html

tests/test_my_work.py

@@ -100,6 +100,53 @@ process.stdout.write(JSON.stringify(buildMyWork({json.dumps(payload)})));
     assert result[1]["reason"] == "Filed by you"
 
 
+def test_authored_pull_is_classified_for_the_my_prs_queue():
+    payload = {
+        "user": {"login": "timmy"},
+        "pull_requests": [{
+            "number": 41, "title": "Ship mobile flow", "repository": "stackchain/dashboard",
+            "state": "open", "user": "timmy", "assignees": [],
+            "work_reasons": ["authored_by_me"], "updated_at": "2026-08-24T12:00:00Z",
+        }],
+    }
+    script = f"""
+const buildMyWork = require({json.dumps(str(MY_WORK))});
+process.stdout.write(JSON.stringify(buildMyWork({json.dumps(payload)})));
+"""
+    completed = subprocess.run(["node", "-e", script], capture_output=True, text=True)
+
+    assert completed.returncode == 0, completed.stderr
+    result = json.loads(completed.stdout)
+    assert result[0]["is_authored"] is True
+    assert result[0]["reason"] == "Authored by you"
+    assert result[0]["kind"] == "pull"
+
+
+def test_my_prs_filter_and_count_include_only_authored_pulls():
+    script = f"""
+const buildMyWork = require({json.dumps(str(MY_WORK))});
+const items = buildMyWork({{
+  user:{{login:'timmy'}},
+  pull_requests:[
+    {{number:41,title:'Authored',repository:'stackchain/dashboard',state:'open',user:'timmy',
+      assignees:[],work_reasons:['authored_by_me']}},
+    {{number:42,title:'Review',repository:'stackchain/dashboard',state:'open',user:'alex',
+      assignees:[],work_reasons:['review_requested']}},
+  ],
+}});
+process.stdout.write(JSON.stringify({{
+  authored:buildMyWork.filterMyWork(items,'authored').map(item=>item.number),
+  counts:buildMyWork.countMyWork(items),
+}}));
+"""
+    completed = subprocess.run(["node", "-e", script], capture_output=True, text=True)
+
+    assert completed.returncode == 0, completed.stderr
+    result = json.loads(completed.stdout)
+    assert result["authored"] == [41]
+    assert result["counts"]["authored"] == 1
+
+
 def test_completed_child_resolves_only_a_canonical_reciprocal_parent_task():
     script = f"""
 const sheet = require({json.dumps(str(ISSUE_SHEET))});
@@ -1059,6 +1106,24 @@ async def test_agenda_activation_checks_all_issue_pages_before_showing_empty_sta
     assert "selectedWorkFilter === 'agenda' && workPagination.issue?.has_more" in html
 
 
+@pytest.mark.anyio
+async def test_my_prs_is_a_thumb_safe_mobile_queue_and_desktop_filter():
+    html = await dashboard()
+    markup = (Path(__file__).resolve().parents[1] / "frontend" / "index.html").read_text()
+    css = (Path(__file__).resolve().parents[1] / "frontend" / "dashboard.css").read_text()
+
+    assert 'data-work-filter="authored"' in markup
+    assert 'data-work-count="authored"' in markup
+    assert 'data-mobile-queue="authored"' in markup
+    assert 'data-mobile-queue-count="authored"' in markup
+    assert '<strong>My PRs</strong>' in markup
+    assert ".mobile-queue-list button" in css and "min-height:56px" in css
+    assert "['all', 'today', 'agenda', 'attention', 'filed', 'authored'" in html
+    assert "['issue', 'pull', 'review', 'authored'].includes(selectedWorkFilter)" in html
+    assert "authored: 'pull requests'" in html
+
+
+
 @pytest.mark.anyio
 async def test_mobile_queue_finder_is_labeled_thumb_safe_and_offers_older_search():
     html = await dashboard()
@@ -4988,7 +5053,8 @@ process.stdout.write(JSON.stringify(buildMyWork.countMyWork({json.dumps(items)})
     )
 
     assert json.loads(result.stdout) == {
-        "all": 3, "attention": 1, "filed": 0, "issue": 1, "pull": 1, "review": 1, "update": 0
+        "all": 3, "attention": 1, "filed": 0, "authored": 0,
+        "issue": 1, "pull": 1, "review": 1, "update": 0
     }
 
 
@@ -6165,7 +6231,8 @@ process.stdout.write(JSON.stringify({{
     assert output["updates"][1]["kind"] == "update"
     assert output["updates"][1]["update_reason"] == ""
     assert output["counts"] == {
-        "all": 2, "attention": 2, "filed": 0, "issue": 1, "pull": 0, "review": 0, "update": 2
+        "all": 2, "attention": 2, "filed": 0, "authored": 0,
+        "issue": 1, "pull": 0, "review": 0, "update": 2
     }
     assert output["summary"] == "2 unread updates · 0 reviews · 1 assigned"
 
@@ -7424,6 +7491,80 @@ Promise.all([first, duplicate]).then(async comments => {{
     assert len(output["comments"]) == 2 and len(output["merges"]) == 2
 
 
+def test_pull_sheet_preserves_context_draft_until_confirmed_update():
+    script = f"""
+const createPullSheet = require({json.dumps(str(PULL_SHEET))});
+const values = new Map();
+const storage = {{getItem:k => values.get(k) || null, setItem:(k,v) => values.set(k,v), removeItem:k => values.delete(k)}};
+const calls = [];
+let fail = true;
+const controller = createPullSheet({{
+  storage,
+  fetchJson: async (url, options={{}}) => {{
+    const body = JSON.parse(options.body);
+    calls.push({{url, method:options.method, body}});
+    if (fail) {{ fail = false; throw new Error('network down'); }}
+    return {{number:7,title:body.title,body:body.body,head_sha:body.expected_head_sha,state:'open'}};
+  }},
+}});
+const item = {{repository:'stackchain/api',number:7}};
+const draft = {{title:'Clear review brief',body:'Review mobile handoff.',expectedHeadSha:'abc1234'}};
+(async () => {{
+  await controller.updateContent(item, draft).catch(() => null);
+  const kept = controller.loadEditDraft(item);
+  const result = await controller.updateContent(item, draft);
+  process.stdout.write(JSON.stringify({{kept,result,after:controller.loadEditDraft(item),calls}}));
+}})().catch(error => {{ console.error(error); process.exit(1); }});
+"""
+    result = subprocess.run(["node", "-e", script], check=True, capture_output=True, text=True)
+    output = json.loads(result.stdout)
+    assert output["kept"] == {
+        "title": "Clear review brief",
+        "body": "Review mobile handoff.",
+        "expectedHeadSha": "abc1234",
+    }
+    assert output["after"] is None
+    assert output["result"]["head_sha"] == "abc1234"
+    assert output["calls"] == [
+        {
+            "url": "api/v1/repos/stackchain/api/pulls/7/content",
+            "method": "PATCH",
+            "body": {
+                "title": "Clear review brief",
+                "body": "Review mobile handoff.",
+                "expected_head_sha": "abc1234",
+            },
+        },
+        {
+            "url": "api/v1/repos/stackchain/api/pulls/7/content",
+            "method": "PATCH",
+            "body": {
+                "title": "Clear review brief",
+                "body": "Review mobile handoff.",
+                "expected_head_sha": "abc1234",
+            },
+        },
+    ]
+
+
+@pytest.mark.anyio
+async def test_pull_overview_exposes_author_only_context_editor_with_retry_safe_status():
+    html = (Path(__file__).parents[1] / "frontend" / "index.html").read_text()
+    source = await dashboard()
+    pull_source = PULL_SHEET.read_text()
+
+    assert 'id="edit-pull-content"' in html
+    assert 'id="pull-edit-form"' in html
+    assert 'id="pull-edit-title"' in html
+    assert 'id="pull-edit-body"' in html
+    assert 'id="save-pull-content"' in html
+    assert "detail.author === getLogin()" in pull_source
+    assert "controller.loadEditDraft(item)" in pull_source
+    assert "controller.updateContent(item" in pull_source
+    assert "Your change is still here; retry." in pull_source
+    assert "controller.setReviewDetail(detail)" in pull_source
+
+
 def test_pull_sheet_removes_one_pull_from_snapshot_without_touching_other_work():
     script = f"""
 const createPullSheet = require({json.dumps(str(PULL_SHEET))});
@@ -7463,6 +7604,9 @@ const elements = new Map([
   ['#load-pull-handoff', {{disabled:true}}],
   ['#release-pull', {{disabled:true,textContent:''}}],
   ['#pull-handoff-status', {{textContent:''}}],
+  ['#edit-pull-content', {{hidden:false}}],
+  ['#pull-edit-form', {{hidden:false}}],
+  ['#pull-edit-status', {{textContent:'stale'}}],
 ]);
 const select = {{textContent:'stale', appendChild:node => children.push(node)}};
 const doc = {{
@@ -7539,6 +7683,123 @@ const item = {{repository:'stackchain/api',number:7}};
     assert output["released"] == {"assignees": []}
 
 
+def test_pull_sheet_loads_reviewers_and_single_flights_review_request():
+    script = f"""
+const createPullSheet = require({json.dumps(str(PULL_SHEET))});
+const calls = [];
+let finish;
+const controller = createPullSheet({{
+  storage: null,
+  fetchJson: (url, options={{}}) => {{
+    calls.push({{url, method:options.method || 'GET', body:options.body ? JSON.parse(options.body) : null}});
+    if (url.endsWith('/review-candidates')) return Promise.resolve([{{login:'casey',name:'Casey'}}]);
+    return new Promise(resolve => {{ finish = resolve; }});
+  }}
+}});
+const notified = [];
+controller.onReviewRequested = result => notified.push(result);
+const item = {{repository:'stackchain/api',number:7}};
+(async () => {{
+  const candidates = await controller.loadReviewCandidates(item);
+  const first = controller.requestReview(item, 'casey', 'abc1234');
+  const duplicate = controller.requestReview(item, 'casey', 'abc1234');
+  finish({{reviewer:'casey',requested_reviewers:['sam','casey']}});
+  const results = await Promise.all([first, duplicate]);
+  process.stdout.write(JSON.stringify({{calls,candidates,results,same:first===duplicate,notified}}));
+}})();
+"""
+    output = json.loads(subprocess.run(
+        ["node", "-e", script], check=True, capture_output=True, text=True
+    ).stdout)
+
+    assert output == {
+        "calls": [
+            {"url": "api/v1/repos/stackchain/api/pulls/7/review-candidates", "method": "GET", "body": None},
+            {"url": "api/v1/repos/stackchain/api/pulls/7/request-review", "method": "POST", "body": {
+                "reviewer": "casey", "expected_head_sha": "abc1234"
+            }},
+        ],
+        "candidates": [{"login": "casey", "name": "Casey"}],
+        "results": [
+            {"reviewer": "casey", "requested_reviewers": ["sam", "casey"]},
+            {"reviewer": "casey", "requested_reviewers": ["sam", "casey"]},
+        ],
+        "same": True,
+        "notified": [{"reviewer": "casey", "requested_reviewers": ["sam", "casey"]}],
+    }
+
+
+def test_pull_sheet_single_flights_pending_review_cancellation():
+    script = f"""
+const createPullSheet = require({json.dumps(str(PULL_SHEET))});
+const calls = [];
+let finish;
+const controller = createPullSheet({{
+  storage:null,
+  fetchJson:(url, options={{}}) => {{
+    calls.push({{url, method:options.method, body:JSON.parse(options.body)}});
+    return new Promise(resolve => {{ finish = resolve; }});
+  }}
+}});
+const notified = [];
+controller.onReviewCancelled = result => notified.push(result);
+const item = {{repository:'stackchain/api',number:7}};
+(async () => {{
+  const first = controller.cancelReview(item, 'sam', 'abc1234');
+  const duplicate = controller.cancelReview(item, 'sam', 'abc1234');
+  finish({{reviewer:'sam',head_sha:'abc1234',requested_reviewers:['casey']}});
+  const results = await Promise.all([first, duplicate]);
+  process.stdout.write(JSON.stringify({{calls,results,same:first===duplicate,notified}}));
+}})();
+"""
+    output = json.loads(subprocess.run(
+        ["node", "-e", script], check=True, capture_output=True, text=True
+    ).stdout)
+
+    assert output == {
+        "calls": [{
+            "url": "api/v1/repos/stackchain/api/pulls/7/request-review",
+            "method": "DELETE",
+            "body": {"reviewer": "sam", "expected_head_sha": "abc1234"},
+        }],
+        "results": [{"reviewer": "sam", "head_sha": "abc1234", "requested_reviewers": ["casey"]}] * 2,
+        "same": True,
+        "notified": [{"reviewer": "sam", "head_sha": "abc1234", "requested_reviewers": ["casey"]}],
+    }
+
+
+def test_pull_sheet_exposes_touch_safe_review_request_flow():
+    root = PULL_SHEET.parents[1]
+    html = (root / "frontend" / "index.html").read_text()
+    css = (root / "frontend" / "dashboard.css").read_text()
+    dashboard = (root / "frontend" / "dashboard.js").read_text()
+
+    assert 'id="pull-review-request"' in html
+    assert 'id="load-pull-reviewers"' in html
+    assert 'id="pull-review-recipient"' in html
+    assert 'id="confirm-pull-review"' in html
+    assert 'id="pull-review-request-status" class="small" aria-live="assertive"' in html
+    assert '.pull-review-request select, .pull-review-request button { min-height:44px;' in css
+    assert "createPullSheet.bindOwnershipControls(document, pullController" in dashboard
+    assert "pullController.setReviewDetail(detail)" in dashboard
+
+
+def test_pull_sheet_exposes_mobile_reviewer_status_and_updated_review_action():
+    root = PULL_SHEET.parents[1]
+    html = (root / "frontend" / "index.html").read_text()
+    css = (root / "frontend" / "dashboard.css").read_text()
+    dashboard = (root / "frontend" / "dashboard.js").read_text()
+
+    assert 'id="pull-reviewer-statuses"' in html
+    assert 'id="request-updated-pull-review"' in html
+    assert 'id="pull-reviewer-status" class="small" aria-live="polite"' in html
+    assert ".pull-reviewer-status" in css
+    assert "overflow-wrap:anywhere" in css
+    assert "#request-updated-pull-review { min-height:44px;" in css
+    assert "createPullSheet.review(detail, pullReviewState, document)" in dashboard
+    assert "reviewers.some(reviewer => reviewer.status === 'outdated')" in PULL_SHEET.read_text()
+
+
 def test_pull_sheet_surfaces_pending_merge_confirmation_guidance():
     script = f"""
 const createPullSheet = require({json.dumps(str(PULL_SHEET))});
@@ -7585,6 +7846,126 @@ process.stdout.write(JSON.stringify(states.map(createPullSheet.mergeEligibility)
     assert output[3]["allowed"] is False and "conflict" in output[3]["reason"].lower()
 
 
+def test_pull_sheet_renders_reviewer_decisions_and_blocks_unresolved_requests():
+    script = f"""
+const createPullSheet = require({json.dumps(str(PULL_SHEET))});
+const base = {{state:'open', draft:false, mergeable:true, merged:false, ci_state:'success', head_sha:'new'}};
+const states = [
+  [{{login:'casey', status:'waiting', head_sha:'new', blocking:true}}],
+  [{{login:'sam', status:'changes_requested', head_sha:'new', blocking:true}}],
+  [{{login:'lee', status:'outdated', head_sha:'old', blocking:true}}],
+  [{{login:'pat', status:'approved', head_sha:'new', blocking:false}}],
+];
+process.stdout.write(JSON.stringify({{
+  eligibility:states.map(reviewers => createPullSheet.mergeEligibility({{...base, reviewers}})),
+  html:createPullSheet.renderReviewerStatuses(states.flat(), value => String(value)
+    .replaceAll('&', '&amp;').replaceAll('<', '&lt;').replaceAll('>', '&gt;')),
+}}));
+"""
+    output = json.loads(subprocess.run(
+        ["node", "-e", script], check=True, capture_output=True, text=True
+    ).stdout)
+
+    assert output["eligibility"] == [
+        {"allowed": False, "reason": "Waiting for @casey’s review"},
+        {"allowed": False, "reason": "@sam requested changes"},
+        {"allowed": False, "reason": "@lee’s review is outdated after new commits"},
+        {"allowed": True, "reason": "Ready to merge"},
+    ]
+    assert "@casey" in output["html"] and "Waiting for review" in output["html"]
+    assert "Changes requested" in output["html"]
+    assert "Outdated after new commits" in output["html"]
+    assert "Approved current head" in output["html"]
+
+
+def test_pull_sheet_offers_cancel_only_for_pending_review_requests():
+    script = f"""
+const createPullSheet = require({json.dumps(str(PULL_SHEET))});
+const html = createPullSheet.renderReviewerStatuses([
+  {{login:'sam',status:'waiting'}},
+  {{login:'casey',status:'approved'}},
+  {{login:'lee',status:'commented'}},
+], value => String(value));
+process.stdout.write(html);
+"""
+    html = subprocess.run(["node", "-e", script], check=True, capture_output=True, text=True).stdout
+
+    assert html.count('data-cancel-review-request=') == 1
+    assert 'data-cancel-review-request="sam"' in html
+    assert "Cancel request" in html
+    assert 'data-cancel-review-request="casey"' not in html
+    assert 'data-cancel-review-request="lee"' not in html
+
+
+def test_pull_sheet_pending_review_cancellation_is_touch_safe_and_opens_replacement_picker():
+    root = PULL_SHEET.parents[1]
+    css = (root / "frontend" / "dashboard.css").read_text()
+    script = PULL_SHEET.read_text()
+
+    assert ".cancel-review-request { min-height:44px;" in css
+    assert "controller.cancelReview(selected, reviewer, detail.head_sha)" in script
+    assert "Choose a replacement reviewer." in script
+    assert "qs('#pull-review-request').open = true" in script
+
+
+def test_pull_sheet_renders_bounded_feedback_grouped_by_file():
+    script = f"""
+const createPullSheet = require({json.dumps(str(PULL_SHEET))});
+const html = createPullSheet.renderReviewerStatuses([{{
+  login:'sam', status:'changes_requested', head_sha:'abc123', blocking:true,
+  summary:'Please <split> this helper.',
+  comments:[
+    {{path:'src/api.py', body:'Handle <empty>.', line:12}},
+    {{path:'src/api.py', body:'Keep the return explicit.'}},
+    {{path:'frontend/app.js', body:'Show the error.'}},
+  ],
+}}], value => String(value).replaceAll('&', '&amp;').replaceAll('<', '&lt;').replaceAll('>', '&gt;'));
+process.stdout.write(html);
+"""
+    html = subprocess.run(["node", "-e", script], check=True, capture_output=True, text=True).stdout
+
+    assert "Review feedback" in html
+    assert "Please &lt;split&gt; this helper." in html
+    assert html.count('data-review-feedback-file="src/api.py"') == 1
+    assert 'data-review-feedback-file="frontend/app.js"' in html
+    assert "Handle &lt;empty&gt;." in html
+    assert "Line 12" in html
+    assert "View in changes" in html
+    assert "Please <split>" not in html
+
+
+def test_pull_sheet_marks_inline_feedback_as_on_demand_and_retryable():
+    script = f"""
+const createPullSheet = require({json.dumps(str(PULL_SHEET))});
+const render = state => createPullSheet.renderReviewerStatuses([{{
+  login:'sam', review_id:8, status:'changes_requested', head_sha:'abc1234',
+  blocking:true, summary:'Please fix this.', feedback_state:state,
+}}], value => String(value));
+process.stdout.write(JSON.stringify({{idle:render(undefined), failed:render('error')}}));
+"""
+    result = subprocess.run(["node", "-e", script], check=True, capture_output=True, text=True)
+    output = json.loads(result.stdout)
+
+    assert "Load inline feedback" in output["idle"]
+    assert 'data-load-review-feedback="8"' in output["idle"]
+    assert "Feedback unavailable" in output["failed"]
+    assert "Retry feedback" in output["failed"]
+    assert "Address feedback" not in output["idle"]
+
+
+def test_pull_sheet_binds_on_demand_feedback_with_visible_retry_state():
+    script = PULL_SHEET.read_text()
+
+    assert "event.target.closest?.('[data-load-review-feedback]')" in script
+    assert "await controller.loadFeedback(item, detail, reviewer" in script
+    assert "reviewer.feedback_state = 'loading'" in script
+    assert "reviewer.feedback_state = 'loaded'" in script
+    assert "reviewer.feedback_state = 'error'" in script
+    assert "result.head_sha !== detail.head_sha" in script
+    css = (Path(__file__).parents[1] / "frontend" / "dashboard.css").read_text()
+    assert "[data-load-review-feedback] { min-height:44px; width:100%;" in css
+
+
 def test_pull_sheet_persists_head_scoped_file_review_and_gates_merge():
     script = f"""
 const createPullSheet = require({json.dumps(str(PULL_SHEET))});
@@ -7660,6 +8041,37 @@ Promise.allSettled([first, concurrent]).then(async failed => {{
     ]
 
 
+def test_pull_sheet_feedback_is_single_flight_cached_by_head_and_review():
+    script = f"""
+const createPullSheet = require({json.dumps(str(PULL_SHEET))});
+const calls = [];
+let resolveFeedback;
+const sheet = createPullSheet({{storage:null, fetchJson:url => {{
+  calls.push(url);
+  return new Promise(resolve => {{ resolveFeedback = resolve; }});
+}}}});
+const item = {{repository:'stackchain/api', number:7}};
+const detail = {{head_sha:'abc1234'}};
+const reviewer = {{review_id:8, head_sha:'abc1234'}};
+const first = sheet.loadFeedback(item, detail, reviewer);
+const concurrent = sheet.loadFeedback(item, detail, reviewer);
+resolveFeedback({{review_id:8, head_sha:'abc1234', reviewed_head_sha:'abc1234', comments:[{{id:99}}]}});
+Promise.all([first, concurrent]).then(async results => {{
+  const cached = await sheet.loadFeedback(item, detail, reviewer);
+  process.stdout.write(JSON.stringify({{same:first === concurrent, results, cached, calls}}));
+}});
+"""
+    result = subprocess.run(["node", "-e", script], capture_output=True, text=True)
+
+    assert result.returncode == 0, result.stderr
+    output = json.loads(result.stdout)
+    assert output["same"] is True
+    assert output["cached"] == output["results"][0]
+    assert output["calls"] == [
+        "api/v1/repos/stackchain/api/pulls/7/reviews/8/feedback?expected_head_sha=abc1234"
+    ]
+
+
 def test_pull_sheet_refreshes_status_only_without_clearing_review_progress():
     script = f"""
 const createPullSheet = require({json.dumps(str(PULL_SHEET))});
@@ -8135,7 +8547,7 @@ async def test_mobile_filters_wrap_show_counts_and_persist_for_the_session():
     assert 'data-work-count="review"' in html
     assert 'data-work-count="update"' in html
     assert 'data-work-count="later"' in html
-    assert "['all', 'today', 'agenda', 'attention', 'filed', 'issue', 'pull', 'review', 'update', 'later', 'draft'].includes(savedFilter)" in html
+    assert "['all', 'today', 'agenda', 'attention', 'filed', 'authored', 'issue', 'pull', 'review', 'update', 'later', 'draft'].includes(savedFilter)" in html
     assert 'data-work-count="draft"' in html
     assert 'sessionStorage.getItem(WORK_FILTER_KEY)' in html
     assert 'sessionStorage.setItem(WORK_FILTER_KEY, selectedWorkFilter)' in html

tests/test_plan_today.py

@@ -418,7 +418,7 @@ async def test_starting_saved_today_work_closes_a_concurrent_rollover_planner():
 def test_plan_today_controller_is_available_in_the_offline_shell():
     source = SERVICE_WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/plan-today.js'" in source
     assert "BASE + 'static/plan-today-readiness.js'" in source
     assert "BASE + 'static/plan-today-preview.js'" in source

tests/test_pull_api.py

@@ -1,4 +1,5 @@
 import asyncio
+import json
 
 import httpx
 import pytest
@@ -7,6 +8,170 @@ from src import gitea_proxy, main
 from src.security_event_store import SecurityEventStoreError
 
 
+@pytest.mark.anyio
+async def test_pull_author_can_publish_assigned_draft_at_expected_head(monkeypatch):
+    calls = []
+
+    async def publish(repository, number, expected_head_sha):
+        calls.append((repository, number, expected_head_sha))
+        return {
+            "repository": repository,
+            "number": number,
+            "title": "Ship mobile flow",
+            "head_sha": expected_head_sha,
+            "state": "open",
+            "draft": False,
+        }
+
+    monkeypatch.setattr(
+        main.gitea_proxy, "publish_authored_assigned_pull", publish, raising=False
+    )
+    transport = httpx.ASGITransport(app=main.app)
+    async with httpx.AsyncClient(transport=transport, base_url="http://test") as client:
+        response = await client.patch(
+            "/api/v1/repos/stackchain/api/pulls/7/ready",
+            json={"expected_head_sha": "abc1234"},
+        )
+
+    assert response.status_code == 200
+    assert response.json()["draft"] is False
+    assert calls == [("stackchain/api", 7, "abc1234")]
+
+
+@pytest.mark.anyio
+async def test_gitea_publishes_only_authored_assigned_draft_and_verifies_ready_head():
+    requests = []
+    pull = {
+        "number": 7,
+        "title": "WIP: Ship mobile flow",
+        "state": "open",
+        "draft": True,
+        "merged": False,
+        "user": {"login": "alex"},
+        "assignees": [{"login": "alex"}],
+        "head": {"sha": "abc1234"},
+    }
+
+    async def handler(request):
+        requests.append((request.method, request.url.path, request.content))
+        if request.url.path == "/api/v1/user":
+            return httpx.Response(200, json={"login": "alex"})
+        if request.url.path == "/api/v1/repos/stackchain/api/pulls/7" and request.method == "GET":
+            return httpx.Response(200, json=pull)
+        if request.url.path == "/api/v1/repos/stackchain/api/pulls/7" and request.method == "PATCH":
+            assert json.loads(request.content) == {"title": "Ship mobile flow"}
+            pull.update({"title": "Ship mobile flow", "draft": False})
+            return httpx.Response(201, json=pull)
+        raise AssertionError(f"unexpected request: {request.method} {request.url.path}")
+
+    gitea_proxy.start_client(transport=httpx.MockTransport(handler))
+    try:
+        result = await gitea_proxy.publish_authored_assigned_pull(
+            "stackchain/api", 7, "abc1234"
+        )
+    finally:
+        await gitea_proxy.stop_client()
+
+    assert result == {
+        "repository": "stackchain/api",
+        "number": 7,
+        "title": "Ship mobile flow",
+        "head_sha": "abc1234",
+        "state": "open",
+        "draft": False,
+    }
+    assert [item[:2] for item in requests] == [
+        ("GET", "/api/v1/user"),
+        ("GET", "/api/v1/repos/stackchain/api/pulls/7"),
+        ("PATCH", "/api/v1/repos/stackchain/api/pulls/7"),
+        ("GET", "/api/v1/repos/stackchain/api/pulls/7"),
+    ]
+
+
+@pytest.mark.anyio
+async def test_pull_author_can_update_assigned_open_pull_context(monkeypatch):
+    calls = []
+
+    async def update(repository, number, title, body, expected_head_sha):
+        calls.append((repository, number, title, body, expected_head_sha))
+        return {
+            "repository": repository,
+            "number": number,
+            "title": title,
+            "body": body,
+            "head_sha": expected_head_sha,
+            "state": "open",
+        }
+
+    monkeypatch.setattr(
+        main.gitea_proxy, "update_authored_assigned_pull", update, raising=False
+    )
+    transport = httpx.ASGITransport(app=main.app)
+    async with httpx.AsyncClient(transport=transport, base_url="http://test") as client:
+        response = await client.patch(
+            "/api/v1/repos/stackchain/api/pulls/7/content",
+            json={
+                "title": "Clarify mobile handoff",
+                "body": "Explain the reviewer path.",
+                "expected_head_sha": "abc1234",
+            },
+        )
+
+    assert response.status_code == 200
+    assert response.json()["title"] == "Clarify mobile handoff"
+    assert calls == [
+        (
+            "stackchain/api",
+            7,
+            "Clarify mobile handoff",
+            "Explain the reviewer path.",
+            "abc1234",
+        )
+    ]
+
+
+@pytest.mark.anyio
+async def test_gitea_pull_context_update_requires_author_assignment_and_matching_head():
+    requests = []
+
+    async def handler(request):
+        requests.append((request.method, request.url.path, request.content))
+        if request.url.path == "/api/v1/user":
+            return httpx.Response(200, json={"login": "alex"})
+        if request.url.path == "/api/v1/repos/stackchain/api/pulls/7" and request.method == "GET":
+            return httpx.Response(200, json={
+                "number": 7, "title": "Old context", "body": "Old body", "state": "open",
+                "merged": False, "user": {"login": "alex"}, "assignees": [{"login": "alex"}],
+                "head": {"sha": "abc1234"},
+                "html_url": "https://forge.example/stackchain/api/pulls/7",
+            })
+        if request.url.path == "/api/v1/repos/stackchain/api/pulls/7" and request.method == "PATCH":
+            assert json.loads(request.content) == {
+                "title": "Clear review brief", "body": "Review the mobile handoff."
+            }
+            return httpx.Response(200, json={
+                "number": 7, "title": "Clear review brief", "body": "Review the mobile handoff.",
+                "state": "open", "head": {"sha": "abc1234"},
+                "html_url": "https://forge.example/stackchain/api/pulls/7",
+            })
+        raise AssertionError(f"unexpected request: {request.method} {request.url.path}")
+
+    gitea_proxy.start_client(transport=httpx.MockTransport(handler))
+    try:
+        result = await gitea_proxy.update_authored_assigned_pull(
+            "stackchain/api", 7, "Clear review brief", "Review the mobile handoff.", "abc1234"
+        )
+    finally:
+        await gitea_proxy.stop_client()
+
+    assert result["title"] == "Clear review brief"
+    assert [request[:2] for request in requests] == [
+        ("GET", "/api/v1/user"),
+        ("GET", "/api/v1/repos/stackchain/api/pulls/7"),
+        ("PATCH", "/api/v1/repos/stackchain/api/pulls/7"),
+    ]
+
+
 @pytest.mark.anyio
 async def test_assigned_pull_detail_reports_completion_state(monkeypatch):
     async def assigned(repository, number):
@@ -127,7 +292,152 @@ async def test_assigned_pull_review_endpoint_loads_review_payload_on_demand(monk
 
 
 @pytest.mark.anyio
-async def test_gitea_pull_check_status_skips_files_reviews_and_diff():
+async def test_assigned_pull_feedback_endpoint_authorizes_and_returns_one_review(monkeypatch):
+    calls = []
+
+    async def assigned(repository, number):
+        calls.append(("assigned", repository, number))
+        return True
+
+    async def feedback(repository, number, review_id, expected_head_sha):
+        calls.append(("feedback", repository, number, review_id, expected_head_sha))
+        return {
+            "review_id": review_id, "head_sha": expected_head_sha,
+            "reviewed_head_sha": expected_head_sha, "comments": [],
+        }
+
+    monkeypatch.setattr(main.gitea_proxy, "is_assigned_pull", assigned)
+    monkeypatch.setattr(main.gitea_proxy, "pull_review_feedback", feedback, raising=False)
+    transport = httpx.ASGITransport(app=main.app)
+    async with httpx.AsyncClient(transport=transport, base_url="http://test") as client:
+        response = await client.get(
+            "/api/v1/repos/stackchain/api/pulls/7/reviews/8/feedback?expected_head_sha=abc1234"
+        )
+
+    assert response.status_code == 200
+    assert response.headers["cache-control"] == "no-store"
+    assert response.json()["review_id"] == 8
+    assert calls == [
+        ("assigned", "stackchain/api", 7),
+        ("feedback", "stackchain/api", 7, 8, "abc1234"),
+    ]
+
+
+def test_reviewer_statuses_distinguish_waiting_current_and_outdated_decisions():
+    pull = {
+        "requested_reviewers": [{"login": "casey"}],
+        "head": {"sha": "new-head"},
+    }
+    reviews = [
+        {"id": 1, "user": {"login": "casey"}, "state": "APPROVED", "commit_id": "old-head"},
+        {"id": 2, "user": {"login": "sam"}, "state": "REQUEST_CHANGES", "commit_id": "new-head"},
+        {"id": 3, "user": {"login": "lee"}, "state": "COMMENT", "commit_id": "new-head"},
+        {"id": 4, "user": {"login": "pat"}, "state": "APPROVED", "commit_id": "new-head"},
+    ]
+
+    assert gitea_proxy._normalize_reviewer_statuses(pull, reviews, "new-head") == [
+        {"login": "casey", "status": "waiting", "head_sha": "new-head", "blocking": True},
+        {"review_id": 3, "login": "lee", "status": "commented", "head_sha": "new-head", "blocking": False},
+        {"review_id": 4, "login": "pat", "status": "approved", "head_sha": "new-head", "blocking": False},
+        {"review_id": 2, "login": "sam", "status": "changes_requested", "head_sha": "new-head", "blocking": True},
+    ]
+
+    pull["requested_reviewers"] = []
+    assert gitea_proxy._normalize_reviewer_statuses(pull, reviews[:1], "new-head") == [
+        {"review_id": 1, "login": "casey", "status": "outdated", "head_sha": "old-head", "blocking": True},
+    ]
+
+
+def test_reviewer_status_exposes_only_bounded_nonempty_feedback_summary():
+    pull = {"requested_reviewers": []}
+    reviews = [
+        {
+            "id": 8,
+            "user": {"login": "sam"},
+            "state": "REQUEST_CHANGES",
+            "commit_id": "abc123",
+            "body": "Please split this helper. " + ("x" * 900),
+        },
+        {
+            "id": 9,
+            "user": {"login": "lee"},
+            "state": "COMMENT",
+            "commit_id": "abc123",
+            "body": "   ",
+        },
+    ]
+
+    statuses = gitea_proxy._normalize_reviewer_statuses(pull, reviews, "abc123")
+
+    sam = next(item for item in statuses if item["login"] == "sam")
+    lee = next(item for item in statuses if item["login"] == "lee")
+    assert sam["summary"].startswith("Please split this helper.")
+    assert len(sam["summary"]) == 500
+    assert "summary" not in lee
+
+
+def test_review_comments_are_bounded_and_require_a_file_and_body():
+    comments = [
+        {"id": 1, "path": "src/api.py", "body": "Handle the empty state", "new_position": 4},
+        {"id": 2, "path": "src/api.py", "body": "x" * 900, "old_position": 7},
+        {"id": 3, "path": "", "body": "missing file"},
+        {"id": 4, "path": "src/ignored.py", "body": "   "},
+        "malformed",
+    ] + [
+        {"id": index, "path": f"src/{index}.py", "body": "bounded"}
+        for index in range(5, 40)
+    ]
+
+    normalized = gitea_proxy._normalize_review_comments(comments)
+
+    assert len(normalized) == 20
+    assert normalized[0] == {
+        "id": 1, "path": "src/api.py", "body": "Handle the empty state", "line": 4,
+    }
+    assert normalized[1]["line"] == 7
+    assert len(normalized[1]["body"]) == 500
+    assert all(item["path"] and item["body"] for item in normalized)
+
+
+def test_review_feedback_preserves_stable_review_and_comment_identities():
+    pull = {"requested_reviewers": []}
+    reviews = [{
+        "id": 42,
+        "user": {"login": "sam"},
+        "state": "REQUEST_CHANGES",
+        "commit_id": "abc123",
+    }]
+
+    statuses = gitea_proxy._normalize_reviewer_statuses(pull, reviews, "abc123")
+    comments = gitea_proxy._normalize_review_comments([
+        {"id": 99, "path": "src/api.py", "body": "Handle the empty state", "new_position": 4},
+        {"id": -1, "path": "src/other.py", "body": "Invalid identity is omitted"},
+    ])
+
+    assert statuses == [{
+        "review_id": 42,
+        "login": "sam",
+        "status": "changes_requested",
+        "head_sha": "abc123",
+        "blocking": True,
+    }]
+    assert comments[0]["id"] == 99
+    assert "id" not in comments[1]
+
+
+def test_inline_feedback_is_loaded_only_for_each_reviewers_latest_decision():
+    reviews = [
+        {"id": 7, "user": {"login": "sam"}, "state": "REQUEST_CHANGES"},
+        {"id": 8, "user": {"login": "sam"}, "state": "APPROVED"},
+        {"id": 9, "user": {"login": "lee"}, "state": "COMMENT"},
+        {"id": 10, "user": {"login": "pat"}, "state": "PENDING"},
+    ]
+
+    assert gitea_proxy._latest_feedback_review_ids(reviews) == {"lee": 9}
+
+
+@pytest.mark.anyio
+async def test_gitea_pull_check_status_refreshes_reviewer_decisions_without_files_or_diff():
     requests = []
 
     async def handler(request):
@@ -136,7 +446,10 @@ async def test_gitea_pull_check_status_skips_files_reviews_and_diff():
             return httpx.Response(200, json={
                 "state": "open", "draft": False, "mergeable": True, "merged": False,
                 "head": {"sha": "abc123"},
+                "requested_reviewers": [{"login": "casey"}],
             })
+        if request.url.path.endswith("/pulls/7/reviews"):
+            return httpx.Response(200, json=[])
         if request.url.path.endswith("/commits/abc123/status"):
             return httpx.Response(200, json={
                 "state": "success",
@@ -153,12 +466,16 @@ async def test_gitea_pull_check_status_skips_files_reviews_and_diff():
     assert requests == [
         ("GET", "/api/v1/repos/stackchain/api/pulls/7"),
         ("GET", "/api/v1/repos/stackchain/api/commits/abc123/status"),
+        ("GET", "/api/v1/repos/stackchain/api/pulls/7/reviews"),
     ]
     assert status == {
         "repository": "stackchain/api", "number": 7, "head_sha": "abc123",
         "state": "open", "draft": False, "mergeable": True, "merged": False,
         "ci_state": "success",
         "checks": [{"name": "tests", "state": "success", "description": "Passed", "url": ""}],
+        "reviewers": [
+            {"login": "casey", "status": "waiting", "head_sha": "abc123", "blocking": True},
+        ],
     }
 
 
@@ -200,7 +517,12 @@ async def test_gitea_assigned_pull_review_includes_bounded_diff_previews():
                 "state": "open",
                 "mergeable": True,
                 "head": {"sha": "abc123"},
+                "requested_reviewers": [{"login": "casey"}],
             })
+        if path.endswith("/pulls/7/reviews"):
+            return httpx.Response(200, json=[
+                {"id": 8, "user": {"login": "sam"}, "state": "APPROVED", "commit_id": "abc123"},
+            ])
         if path.endswith("/pulls/7/files"):
             return httpx.Response(200, json=[
                 {"filename": "src/api.py", "status": "modified", "additions": 1, "deletions": 1},
@@ -230,9 +552,89 @@ async def test_gitea_assigned_pull_review_includes_bounded_diff_previews():
     assert "+new" in detail["files"][0]["diff_lines"]
     assert detail["files"][1]["diff_binary"] is True
     assert detail["files"][1]["diff_available"] is False
+    assert detail["reviewers"] == [
+        {"login": "casey", "status": "waiting", "head_sha": "abc123", "blocking": True},
+        {"review_id": 8, "login": "sam", "status": "approved", "head_sha": "abc123", "blocking": False},
+    ]
     assert "conversation" not in detail
 
 
+@pytest.mark.anyio
+async def test_assigned_pull_review_does_not_block_on_inline_feedback():
+    requests = []
+
+    async def handler(request):
+        path = request.url.path
+        requests.append(path)
+        if path.endswith("/pulls/7"):
+            return httpx.Response(200, json={
+                "state": "open", "mergeable": True, "head": {"sha": "abc123"},
+                "requested_reviewers": [],
+            })
+        if path.endswith("/pulls/7/reviews"):
+            return httpx.Response(200, json=[
+                {"id": 7, "user": {"login": "sam"}, "state": "COMMENT", "commit_id": "old-head"},
+                {"id": 8, "user": {"login": "sam"}, "state": "REQUEST_CHANGES",
+                 "commit_id": "abc123", "body": "Please handle the empty state."},
+            ])
+        if path.endswith("/pulls/7/files"):
+            return httpx.Response(200, json=[{"filename": "src/api.py", "status": "modified"}])
+        if path.endswith("/commits/abc123/status"):
+            return httpx.Response(200, json={"state": "success"})
+        if path.endswith("/pulls/7.diff"):
+            return httpx.Response(200, text="")
+        raise AssertionError(f"optional feedback must not block core review: {request.method} {path}")
+
+    gitea_proxy.start_client(transport=httpx.MockTransport(handler))
+    try:
+        detail = await gitea_proxy.pull_completion_review("stackchain/api", 7)
+    finally:
+        await gitea_proxy.stop_client()
+
+    assert not any(path.endswith("/comments") for path in requests)
+    assert detail["reviewers"] == [{
+        "review_id": 8, "login": "sam", "status": "changes_requested", "head_sha": "abc123", "blocking": True,
+        "summary": "Please handle the empty state.",
+    }]
+
+
+@pytest.mark.anyio
+async def test_pull_review_feedback_loads_one_current_bounded_review():
+    requests = []
+
+    async def handler(request):
+        path = request.url.path
+        requests.append(path)
+        if path.endswith("/pulls/7"):
+            return httpx.Response(200, json={"head": {"sha": "abc123"}})
+        if path.endswith("/pulls/7/reviews"):
+            return httpx.Response(200, json=[
+                {"id": 7, "user": {"login": "sam"}, "state": "COMMENT", "commit_id": "old-head"},
+                {"id": 8, "user": {"login": "sam"}, "state": "REQUEST_CHANGES", "commit_id": "abc123"},
+            ])
+        if path.endswith("/pulls/7/reviews/8/comments"):
+            return httpx.Response(200, json=[
+                {"id": 99, "path": "src/api.py", "body": "Return before parsing.", "new_position": 12},
+            ])
+        raise AssertionError(f"unexpected request: {request.method} {path}")
+
+    gitea_proxy.start_client(transport=httpx.MockTransport(handler))
+    try:
+        feedback = await gitea_proxy.pull_review_feedback(
+            "stackchain/api", 7, 8, "abc123"
+        )
+    finally:
+        await gitea_proxy.stop_client()
+
+    assert requests[-1].endswith("/pulls/7/reviews/8/comments")
+    assert feedback == {
+        "review_id": 8,
+        "head_sha": "abc123",
+        "reviewed_head_sha": "abc123",
+        "comments": [{"id": 99, "path": "src/api.py", "body": "Return before parsing.", "line": 12}],
+    }
+
+
 @pytest.mark.anyio
 async def test_assigned_pull_conversation_endpoint_reuses_issue_thread_with_pull_authorization(monkeypatch):
     calls = []
@@ -593,6 +995,39 @@ async def test_gitea_merge_rejects_failed_ci_without_upstream_mutation():
     ]
 
 
+@pytest.mark.anyio
+@pytest.mark.parametrize("requested,reviews", [
+    ([{"login": "casey"}], []),
+    ([], [{"id": 9, "user": {"login": "casey"}, "state": "REQUEST_CHANGES", "commit_id": "abc123"}]),
+])
+async def test_gitea_merge_rejects_unresolved_reviewer_status_without_mutation(requested, reviews):
+    requests = []
+
+    async def handler(request):
+        requests.append((request.method, request.url.path))
+        if request.url.path.endswith("/pulls/7"):
+            return httpx.Response(200, json={
+                "number": 7, "state": "open", "draft": False, "mergeable": True,
+                "merged": False, "head": {"sha": "abc123"},
+                "requested_reviewers": requested,
+            })
+        if request.url.path.endswith("/commits/abc123/status"):
+            return httpx.Response(200, json={"state": "success"})
+        if request.url.path.endswith("/pulls/7/reviews"):
+            return httpx.Response(200, json=reviews)
+        raise AssertionError(f"unexpected mutation: {request.method} {request.url.path}")
+
+    gitea_proxy.start_client(transport=httpx.MockTransport(handler))
+    try:
+        with pytest.raises(gitea_proxy.PullNotMergeableError):
+            await gitea_proxy.merge_assigned_pull("stackchain/api", 7, "abc123")
+    finally:
+        await gitea_proxy.stop_client()
+
+    assert requests[-1] == ("GET", "/api/v1/repos/stackchain/api/pulls/7/reviews")
+    assert all(method == "GET" for method, _path in requests)
+
+
 @pytest.mark.anyio
 async def test_assigned_pull_ownership_endpoints_list_handoff_and_release(monkeypatch):
     calls = []
@@ -640,6 +1075,69 @@ async def test_assigned_pull_ownership_endpoints_list_handoff_and_release(monkey
     ]
 
 
+@pytest.mark.anyio
+async def test_assigned_pull_review_request_endpoints_list_and_submit(monkeypatch):
+    calls = []
+
+    async def assigned(repository, number):
+        calls.append(("assigned", repository, number))
+        return True
+
+    async def candidates(repository, number):
+        calls.append(("candidates", repository, number))
+        return [{"login": "casey", "name": "Casey"}]
+
+    async def request_review(repository, number, reviewer, expected_head_sha):
+        calls.append(("request", repository, number, reviewer, expected_head_sha))
+        return {"number": number, "reviewer": reviewer, "requested_reviewers": [reviewer]}
+
+    monkeypatch.setattr(main.gitea_proxy, "is_assigned_pull", assigned)
+    monkeypatch.setattr(main.gitea_proxy, "pull_review_candidates", candidates, raising=False)
+    monkeypatch.setattr(main.gitea_proxy, "request_assigned_pull_review", request_review, raising=False)
+    transport = httpx.ASGITransport(app=main.app)
+    async with httpx.AsyncClient(transport=transport, base_url="http://test") as client:
+        listed = await client.get("/api/v1/repos/stackchain/api/pulls/7/review-candidates")
+        requested = await client.post(
+            "/api/v1/repos/stackchain/api/pulls/7/request-review",
+            json={"reviewer": "casey", "expected_head_sha": "abc1234"},
+        )
+
+    assert listed.status_code == 200
+    assert listed.headers["cache-control"] == "no-store"
+    assert listed.json() == [{"login": "casey", "name": "Casey"}]
+    assert requested.status_code == 200
+    assert requested.json()["requested_reviewers"] == ["casey"]
+    assert calls == [
+        ("assigned", "stackchain/api", 7),
+        ("candidates", "stackchain/api", 7),
+        ("request", "stackchain/api", 7, "casey", "abc1234"),
+    ]
+
+
+@pytest.mark.anyio
+async def test_assigned_pull_cancel_review_endpoint_forwards_head_scoped_request(monkeypatch):
+    calls = []
+
+    async def cancel(repository, number, reviewer, expected_head_sha):
+        calls.append((repository, number, reviewer, expected_head_sha))
+        return {
+            "repository": repository, "number": number, "reviewer": reviewer,
+            "head_sha": expected_head_sha, "requested_reviewers": ["casey"],
+        }
+
+    monkeypatch.setattr(main.gitea_proxy, "cancel_assigned_pull_review", cancel, raising=False)
+    transport = httpx.ASGITransport(app=main.app)
+    async with httpx.AsyncClient(transport=transport, base_url="http://test") as client:
+        response = await client.request(
+            "DELETE", "/api/v1/repos/stackchain/api/pulls/7/request-review",
+            json={"reviewer": "sam", "expected_head_sha": "abc1234"},
+        )
+
+    assert response.status_code == 200
+    assert response.json()["requested_reviewers"] == ["casey"]
+    assert calls == [("stackchain/api", 7, "sam", "abc1234")]
+
+
 @pytest.mark.anyio
 async def test_gitea_pull_handoff_preserves_coassignees_and_confirms_ownership_exit():
     requests = []
@@ -709,6 +1207,124 @@ async def test_pull_handoff_candidates_are_bounded_and_exclude_invalid_or_curren
     ]
 
 
+@pytest.mark.anyio
+async def test_gitea_pull_review_request_filters_candidates_and_confirms_requested_reviewer():
+    requests = []
+    review_requested = False
+
+    async def handler(request):
+        nonlocal review_requested
+        requests.append((request.method, request.url.path, request.content))
+        if request.url.path.endswith("/user"):
+            return httpx.Response(200, json={"login": "timmy"})
+        if request.url.path.endswith("/assignees"):
+            return httpx.Response(200, json=[
+                {"login": "timmy", "full_name": "Timmy"},
+                {"login": "alex", "full_name": "Alexander"},
+                {"login": "sam", "full_name": "Sam"},
+                {"login": "casey", "full_name": "Casey"},
+            ])
+        if request.url.path.endswith("/pulls/7"):
+            requested = [{"login": "sam"}]
+            if review_requested:
+                requested.append({"login": "casey"})
+            return httpx.Response(200, json={
+                "number": 7, "state": "open", "merged": False,
+                "head": {"sha": "abc123"}, "user": {"login": "alex"},
+                "assignees": [{"login": "timmy"}],
+                "requested_reviewers": requested,
+            })
+        if request.method == "POST" and request.url.path.endswith("/requested_reviewers"):
+            review_requested = True
+            return httpx.Response(201, json={})
+        raise AssertionError(f"unexpected request: {request.method} {request.url.path}")
+
+    gitea_proxy.start_client(transport=httpx.MockTransport(handler))
+    try:
+        candidates = await gitea_proxy.pull_review_candidates("stackchain/api", 7)
+        result = await gitea_proxy.request_assigned_pull_review(
+            "stackchain/api", 7, "casey", "abc123"
+        )
+    finally:
+        await gitea_proxy.stop_client()
+
+    assert candidates == [{"login": "casey", "name": "Casey"}]
+    assert result == {
+        "repository": "stackchain/api", "number": 7, "head_sha": "abc123",
+        "requested_reviewers": ["sam", "casey"], "reviewer": "casey",
+    }
+    mutation = next(item for item in requests if item[0] == "POST")
+    assert mutation[1] == "/api/v1/repos/stackchain/api/pulls/7/requested_reviewers"
+    assert mutation[2] == b'{"reviewers":["casey"]}'
+
+
+@pytest.mark.anyio
+async def test_gitea_cancel_pending_pull_review_confirms_authoritative_removal():
+    requests = []
+    cancelled = False
+
+    async def handler(request):
+        nonlocal cancelled
+        requests.append((request.method, request.url.path, request.content))
+        if request.url.path.endswith("/user"):
+            return httpx.Response(200, json={"login": "timmy"})
+        if request.url.path.endswith("/pulls/7"):
+            return httpx.Response(200, json={
+                "number": 7, "state": "open", "merged": False,
+                "head": {"sha": "abc123"}, "assignees": [{"login": "timmy"}],
+                "requested_reviewers": [{"login": "casey"}] if cancelled else [{"login": "sam"}, {"login": "casey"}],
+            })
+        if request.method == "DELETE" and request.url.path.endswith("/requested_reviewers"):
+            assert request.content == b'{"reviewers":["sam"]}'
+            cancelled = True
+            return httpx.Response(204)
+        raise AssertionError(f"unexpected request: {request.method} {request.url.path}")
+
+    gitea_proxy.start_client(transport=httpx.MockTransport(handler))
+    try:
+        result = await gitea_proxy.cancel_assigned_pull_review(
+            "stackchain/api", 7, "sam", "abc123"
+        )
+    finally:
+        await gitea_proxy.stop_client()
+
+    assert result == {
+        "repository": "stackchain/api", "number": 7, "head_sha": "abc123",
+        "requested_reviewers": ["casey"], "reviewer": "sam",
+    }
+    assert [method for method, _path, _body in requests] == ["GET", "GET", "DELETE", "GET"]
+
+
+@pytest.mark.anyio
+async def test_gitea_cancel_pending_pull_review_rejects_head_drift_before_mutation():
+    requests = []
+
+    async def handler(request):
+        requests.append(request.method)
+        if request.url.path.endswith("/user"):
+            return httpx.Response(200, json={"login": "timmy"})
+        if request.url.path.endswith("/pulls/7"):
+            return httpx.Response(200, json={
+                "number": 7, "state": "open", "merged": False,
+                "head": {"sha": "new-head"}, "assignees": [{"login": "timmy"}],
+                "requested_reviewers": [{"login": "sam"}],
+            })
+        if request.method == "DELETE":
+            return httpx.Response(204)
+        raise AssertionError(request.url.path)
+
+    gitea_proxy.start_client(transport=httpx.MockTransport(handler))
+    try:
+        with pytest.raises(gitea_proxy.IssueNotAvailableError):
+            await gitea_proxy.cancel_assigned_pull_review(
+                "stackchain/api", 7, "sam", "old-head"
+            )
+    finally:
+        await gitea_proxy.stop_client()
+
+    assert requests == ["GET", "GET"]
+
+
 @pytest.mark.anyio
 async def test_gitea_pull_release_removes_current_login_case_insensitively():
     async def handler(request):
@@ -745,6 +1361,8 @@ async def test_gitea_merge_returns_the_exact_merge_commit_for_release_tracking()
             })
         if request.url.path.endswith("/commits/abc123/status"):
             return httpx.Response(200, json={"state": "success"})
+        if request.url.path.endswith("/pulls/7/reviews"):
+            return httpx.Response(200, json=[])
         if request.url.path.endswith("/pulls/7/merge"):
             return httpx.Response(200, json={"merged": True, "sha": "merge456"})
         raise AssertionError(request.url.path)

tests/test_search_week_plan.py

@@ -98,7 +98,12 @@ const outcome=await planner.plan({kind:'issue',state:'open',assigned_to_me:true}
 console.log(JSON.stringify(outcome));
 """)
 
-    assert result == {"status": "planned-pending", "date": "2026-08-22", "estimate": 45}
+    assert result == {
+        "status": "planned-pending",
+        "date": "2026-08-22",
+        "estimate": 45,
+        "item": {"kind": "issue", "state": "open", "assigned_to_me": True},
+    }
 
 
 def test_search_week_plan_stages_canonical_details_before_flushing():
@@ -122,3 +127,37 @@ console.log(JSON.stringify({outcome,calls}));
         ["details", "issue:stackchain/dashboard:42:", "Canonical title", "stackchain/dashboard", 42],
         ["flush"],
     ]
+
+
+def test_search_week_plan_waits_for_success_callback_before_closing():
+    result = run_planner("""
+const events=[];
+const nodes={};
+function node(extra={}) { return Object.assign({
+  hidden:true,disabled:false,textContent:'',value:'',dataset:{},
+  addEventListener(){},focus(){events.push('focus');},
+  setAttribute(){},querySelectorAll(){return[];}
+},extra); }
+nodes['#search-week-plan-sheet']=node();
+nodes['#search-week-plan-status']=node();
+nodes['#confirm-search-week-plan']=node();
+nodes['#search-week-plan-copy']=node();
+nodes['#search-week-plan-days']=node({dataset:{selected:'2026-08-22'}});
+nodes['#search-week-plan-estimate']=node({value:'45'});
+nodes['#plan-search-result']=node(); nodes['#cancel-search-week-plan']=node();
+const document={querySelector:selector=>nodes[selector],addEventListener(){}};
+const window={history:{state:{},pushState(value){this.state=value;},back(){events.push('back');nodes['#search-week-plan-sheet'].hidden=true;}},location:{href:'/'},addEventListener(){}};
+const planner={pending:()=>false,preview:async()=>({days:[{date:'2026-08-22',label:'Sat',planned_minutes:0,capacity_minutes:90,ids:[]}]}),
+ plan:async()=>({status:'planned',date:'2026-08-22',estimate:45,item:{number:42}})};
+const ui=createSearchWeekPlan.UI({planner,document,window,getDetail:()=>({title:'Issue',number:42}),
+ escapeHtml:value=>value,escapeAttribute:value=>value,announce:message=>events.push(message),
+ onPlanned:async()=>{events.push('callback-start');await Promise.resolve();events.push('callback-end');}});
+await ui.open(node());
+await ui.confirm();
+events.push(nodes['#search-week-plan-sheet'].hidden?'closed':'open');
+console.log(JSON.stringify({events}));
+""")
+
+    assert result["events"][-4:] == [
+        "callback-start", "callback-end", "back", "closed"
+    ]

tests/test_service_worker.py

@@ -14,7 +14,7 @@ def run_worker_scenario(scenario: str) -> dict:
 const fs = require('fs');
 const vm = require('vm');
 const listeners = {{}};
-const state = {{ added: [], addAttempts: [], individuallyAdded: [], failedAdds: [], deleted: [], deletedDatabases: [], claimed: false, skipped: false, fetches: [], puts: [], migrated: [], activationOrder: [], oldCachedAssets: {{}}, sharedRecords: {{}}, todayCommands: {{}}, failTodayCommandPut: false, failSharedPut: false, backgroundFlushes: 0, backgroundResumes: 0, outboxPurges: 0, outboxLifecycle: [], notifications: [], closedNotifications: 0, focused: [], opened: [], appBadges: [], clearedAppBadges: 0, badgeEnabled: false, failFetch: false, stallFetch: false, lateFetch: false, fetchAborted: false, fetchStatus: 200, fetchRedirected: false, cachedBody: null }};
+const state = {{ added: [], addAttempts: [], individuallyAdded: [], failedAdds: [], deleted: [], deletedDatabases: [], claimed: false, skipped: false, fetches: [], puts: [], migrated: [], activationOrder: [], oldCachedAssets: {{}}, sharedRecords: {{}}, todayCommands: {{}}, failTodayCommandPut: false, failSharedPut: false, backgroundFlushes: 0, backgroundResumes: 0, outboxPurges: 0, outboxLifecycle: [], notifications: [], closedNotifications: 0, focused: [], opened: [], appBadges: [], clearedAppBadges: 0, badgeEnabled: false, badgeCounts: {{updates:0, following:0}}, failFetch: false, stallFetch: false, lateFetch: false, fetchAborted: false, fetchStatus: 200, fetchRedirected: false, cachedBody: null }};
 const storedResponses = new Map();
 storedResponses.set(
   'https://forge.example/dashboard/__offline-session-lease',
@@ -81,6 +81,9 @@ const context = {{
     __STACKCHAIN_APP_BADGE_PREFERENCE: {{
       get: async () => state.badgeEnabled,
       set: async enabled => {{ state.badgeEnabled = enabled; }},
+      getCounts: async () => ({{...state.badgeCounts}}),
+      setCount: async (channel, count) => {{ state.badgeCounts[channel] = count; }},
+      clearCounts: async () => {{ state.badgeCounts = {{updates:0, following:0}}; }},
     }},
     addEventListener: (name, handler) => {{ listeners[name] = handler; }},
     skipWaiting: async () => {{ state.skipped = true; }},
@@ -186,7 +189,7 @@ async function dispatchPush(payload) {{
 def test_week_unplan_undo_rolls_the_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/week-plan.js'" in source
     assert "BASE + 'static/dashboard.css'" in source
 
@@ -194,20 +197,20 @@ def test_week_unplan_undo_rolls_the_offline_shell():
 def test_private_today_action_mailbox_rolls_the_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
 
 
 def test_per_day_week_conflict_ui_rolls_the_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/week-plan.js'" in source
 
 
 def test_resumable_today_session_ships_in_a_new_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/my-work.js'" in source
     assert "BASE + 'static/dashboard.js'" in source
     assert "BASE + 'static/dashboard.css'" in source
@@ -216,7 +219,7 @@ def test_resumable_today_session_ships_in_a_new_offline_shell():
 def test_mobile_conversation_photo_bundles_roll_the_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/dashboard.js'" in source
     assert "BASE + 'static/authored-outbox.js'" in source
     assert "BASE + 'static/background-issue-sync.js'" in source
@@ -225,7 +228,7 @@ def test_mobile_conversation_photo_bundles_roll_the_offline_shell():
 def test_photo_metadata_sanitizer_rolls_the_cached_optimizer_atomically():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/issue-evidence-review.js'" in source
     assert "BASE + 'static/issue-attachment.js'" in source
 
@@ -233,14 +236,14 @@ def test_photo_metadata_sanitizer_rolls_the_cached_optimizer_atomically():
 def test_ownership_exit_runtime_rolls_the_offline_shell_cache():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/dashboard.js'" in source
 
 
 def test_offline_review_next_ships_today_completion_atomically():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/today-completion.js'" in source
     assert "BASE + 'static/dashboard.js'" in source
 
@@ -248,7 +251,7 @@ def test_offline_review_next_ships_today_completion_atomically():
 def test_duplicate_aware_capture_ships_in_a_new_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/create-issue-sheet.js'" in source
     assert "BASE + 'static/dashboard.js'" in source
 
@@ -256,7 +259,7 @@ def test_duplicate_aware_capture_ships_in_a_new_offline_shell():
 def test_inline_checklist_step_flow_rolls_the_offline_shell_atomically():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/issue-sheet.js'" in source
     assert "BASE + 'static/checklist-conflict.js'" in source
     assert "BASE + 'static/dashboard.js'" in source
@@ -266,14 +269,14 @@ def test_inline_checklist_step_flow_rolls_the_offline_shell_atomically():
 def test_exact_later_picker_ships_atomically_in_a_new_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/later-picker.js'" in source
 
 
 def test_navigation_deadline_ships_in_a_new_shell_cache():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/dashboard.css'" in source
     assert "BASE + 'static/dashboard.js'" in source
     assert "BASE + 'static/install-app.js'" in source
@@ -282,21 +285,21 @@ def test_navigation_deadline_ships_in_a_new_shell_cache():
 def test_today_convergence_ships_in_a_new_shell_cache():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/today-sync.js'" in source
 
 
 def test_mobile_search_viewport_ships_in_a_new_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/mobile-search-viewport.js'" in source
 
 
 def test_update_ownership_flow_ships_atomically_in_a_new_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/update-ownership.js'" in source
 
 
@@ -999,6 +1002,55 @@ def test_opted_in_background_updates_reconcile_authoritative_badge_without_churn
     assert len(result["notifications"]) == 4
 
 
+def test_background_badge_combines_following_and_updates_without_channel_overwrite():
+    result = run_worker_scenario(
+        """
+  await dispatchMessage({type:'stackchain-app-badge-preference', enabled:true}, [], {url:'https://forge.example/dashboard/'});
+  await dispatchPush({tag:'stackchain-following-0123456789abcdef', route:'#/my-work/following', following_count:2});
+  await dispatchPush({tag:'stackchain-update-42', route:'#/my-work/update/42', notification_id:42, unread_count:3});
+  await dispatchPush({tag:'stackchain-following-fedcba9876543210', route:'#/my-work/following', following_count:'invalid'});
+  await dispatchPush({tag:'stackchain-following-1111111111111111', route:'#/my-work/following', following_count:1});
+  await dispatchPush({tag:'stackchain-update-digest', route:'#/my-work/updates', update_count:1, unread_count:3});
+  process.stdout.write(JSON.stringify(state));
+"""
+    )
+
+    assert result["appBadges"] == [2, 5, 4]
+    assert result["badgeCounts"] == {"updates": 3, "following": 1}
+    assert result["clearedAppBadges"] == 0
+
+
+def test_worker_accepts_only_authenticated_authoritative_badge_channel_counts():
+    result = run_worker_scenario(
+        """
+  state.badgeEnabled = true;
+  await dispatchMessage({type:'stackchain-app-badge-count', channel:'following', count:2}, [], {url:'https://forge.example/dashboard/'});
+  await dispatchMessage({type:'stackchain-app-badge-count', channel:'updates', count:8}, [], {url:'https://evil.example/dashboard/'});
+  await dispatchMessage({type:'stackchain-app-badge-count', channel:'following', count:-1}, [], {url:'https://forge.example/dashboard/'});
+  await dispatchPush({tag:'stackchain-update-42', route:'#/my-work/update/42', notification_id:42, unread_count:3});
+  process.stdout.write(JSON.stringify(state));
+"""
+    )
+
+    assert result["badgeCounts"] == {"updates": 3, "following": 2}
+    assert result["appBadges"] == [5]
+
+
+def test_foreground_channel_sync_invalidates_worker_render_cache_for_next_push():
+    result = run_worker_scenario(
+        """
+  state.badgeEnabled = true;
+  await dispatchPush({tag:'stackchain-following-0123456789abcdef', route:'#/my-work/following', following_count:2});
+  await dispatchMessage({type:'stackchain-app-badge-count', channel:'following', count:0}, [], {url:'https://forge.example/dashboard/'});
+  await dispatchPush({tag:'stackchain-update-42', route:'#/my-work/update/42', notification_id:42, unread_count:2});
+  process.stdout.write(JSON.stringify(state));
+"""
+    )
+
+    assert result["badgeCounts"] == {"updates": 2, "following": 0}
+    assert result["appBadges"] == [2, 2]
+
+
 def test_disabling_background_badge_clears_it_and_future_updates_leave_it_off():
     result = run_worker_scenario(
         """
@@ -1302,7 +1354,7 @@ def test_one_session_bound_csrf_proof_is_reused_for_a_background_drain():
 def test_queue_today_ships_atomically_in_a_new_offline_shell():
     source = WORKER.read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/queue-today.js'" in source
 
 

tests/test_today_readiness.py

@@ -221,7 +221,7 @@ async def test_today_blocker_opens_existing_preview_and_preserves_readiness_gate
 def test_readiness_runtime_is_available_in_offline_shell():
     service_worker = SERVICE_WORKER.read_text()
 
-    assert "const CACHE = 'stackchain-dashboard-shell-v136';" in service_worker
+    assert "const CACHE = 'stackchain-dashboard-shell-v138';" in service_worker
     assert "BASE + 'static/today-readiness.js'" in service_worker
 
 

tests/test_today_sync.py

@@ -343,7 +343,7 @@ listeners['stackchain:first-task-complete']();
 def test_inflight_today_drain_ships_in_a_new_offline_shell():
     source = (Path(__file__).parents[1] / "frontend" / "service-worker.js").read_text()
 
-    assert "stackchain-dashboard-shell-v136" in source
+    assert "stackchain-dashboard-shell-v138" in source
     assert "BASE + 'static/today-sync.js'" in source
 
 

tests/test_work_pages.py

@@ -90,6 +90,47 @@ async def test_filed_work_page_endpoint_preserves_authored_reason(monkeypatch):
     assert response.json()["items"][0]["work_reasons"] == ["created_by_me"]
 
 
+@pytest.mark.anyio
+async def test_authored_pull_page_endpoint_preserves_authorship_reason(monkeypatch):
+    async def page_loader(stream, page):
+        assert (stream, page) == ("authored", 2)
+        return {
+            "stream": stream, "page": page, "total": 51, "has_more": False,
+            "items": [{
+                "id": 901, "number": 41, "title": "Ship mobile flow", "state": "open",
+                "user": {"login": "timmy"}, "labels": [], "assignees": [],
+                "work_reasons": ["authored_by_me"],
+                "repository": {"full_name": "stackchain/dashboard"},
+                "html_url": "https://forge.example/pulls/41",
+            }],
+        }
+
+    monkeypatch.setattr(main.gitea_proxy, "work_page", page_loader)
+    transport = httpx.ASGITransport(app=main.app)
+    async with httpx.AsyncClient(transport=transport, base_url="http://test") as client:
+        response = await client.get("/api/v1/work/authored?page=2")
+
+    assert response.status_code == 200
+    assert response.json()["items"][0]["work_reasons"] == ["authored_by_me"]
+
+
+def test_initial_context_preserves_authored_pull_reason():
+    payload = main._context_payload(
+        {"id": 1, "login": "timmy"},
+        [],
+        [],
+        [{
+            "id": 901, "number": 41, "title": "Ship mobile flow", "state": "open",
+            "user": {"login": "timmy"}, "labels": [], "assignees": [],
+            "work_reasons": ["authored_by_me"],
+            "repository": {"full_name": "stackchain/dashboard"},
+            "html_url": "https://forge.example/pulls/41",
+        }],
+    )
+
+    assert payload["pull_requests"][0]["work_reasons"] == ["authored_by_me"]
+
+
 @pytest.mark.anyio
 async def test_pwa_assets_expose_root_scoped_share_target_without_caching_api_data():
     transport = httpx.ASGITransport(app=main.app)