stackchain/stackchain-dashboard
1
0
Fork 0
Code Issues 9 Pull Requests 1 Actions Packages Projects Releases 713 Wiki Activity

Apply same-file review suggestions as one atomic mobile fix #1385

Merged
rockachopa merged 1 commits from timmy/1384-atomic-review-suggestion-batch into main 2026-08-25 07:26:01 +00:00
Conversation 0 Commits 1 Files Changed 8 +392 -2
rockachopa commented 2026-08-25 07:15:17 +00:00
Member
Copy Link

Closes #1384

Product slice

  • stage two to eight compatible reviewer suggestions for one file during the mobile feedback pass
  • review the combined bounded fix and commit it once without implicitly marking comments Addressed
  • reject overlaps, stale blob/head state, cross-file partial batching, and unconfirmed mutations
  • preserve the staged batch after retryable failures

The live Gitea 1.22 Swagger contract exposes Git blob/tree/ref APIs as read-only, so this intentionally batches one file through the supported contents API rather than claiming unsupported multi-file atomicity.

Verification

  • pytest -q tests/e2e/test_mobile_address_review_feedback_release.py tests/e2e/test_mobile_apply_review_suggestion_batch_release.py tests/e2e/test_mobile_apply_review_suggestion_release.py tests/test_pull_api.py — 82 passed
  • pytest -q — 2632 passed, 22 skipped
  • rendered at 320×568 and 390×844 with no horizontal overflow and 44px visible controls
Closes #1384 ## Product slice - stage two to eight compatible reviewer suggestions for one file during the mobile feedback pass - review the combined bounded fix and commit it once without implicitly marking comments Addressed - reject overlaps, stale blob/head state, cross-file partial batching, and unconfirmed mutations - preserve the staged batch after retryable failures The live Gitea 1.22 Swagger contract exposes Git blob/tree/ref APIs as read-only, so this intentionally batches one file through the supported contents API rather than claiming unsupported multi-file atomicity. ## Verification - `pytest -q tests/e2e/test_mobile_address_review_feedback_release.py tests/e2e/test_mobile_apply_review_suggestion_batch_release.py tests/e2e/test_mobile_apply_review_suggestion_release.py tests/test_pull_api.py` — 82 passed - `pytest -q` — 2632 passed, 22 skipped - rendered at 320×568 and 390×844 with no horizontal overflow and 44px visible controls
rockachopa added 1 commit 2026-08-25 07:15:18 +00:00
feat: batch mobile review suggestions (Closes #1384)
All checks were successful
CI / lint (pull_request) Successful in 3m38s
Details
CI / build-release (pull_request) Successful in 7s
Details
CI / browser-journey (pull_request) Successful in 6m20s
Details
CI / release-candidate (pull_request) Has been skipped
Details
869543e992
rockachopa merged commit 7bb22e3525 into main 2026-08-25 07:26:01 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
P0

Drop everything

  
P1

High priority

  
P2

Medium priority

  
agent:nightprowl

Assignments routed to the LongCat NightProwl evaluation worker

  
backend

Backend service

  
ci

Continuous integration / actions

  
frontend

Frontend / UI

  
meta

Meta / planning

  
pages

Gitea Pages

  
template

Repo template / standards

No Label
P0
P1
P2
agent:nightprowl
backend
ci
frontend
meta
pages
template
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: stackchain/stackchain-dashboard#1385
No description provided.
Delete Branch "timmy/1384-atomic-review-suggestion-batch"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?