dreampiler/kandev
1
0
Fork 0
Code Issues 43 Pull Requests Actions 6 Packages Projects Releases Wiki Activity

[상류 3단계] route-action 대기 queue guard #205

New Issue
Closed
opened 2026-10-29 12:33:59 +09:00 by dreampiler · 5 comments
dreampiler commented 2026-10-29 12:33:59 +09:00
Owner
Copy Link
Copy Source

원 요구·승인

소유자 최신 실행 승인(sent/codex_upstream_execute_approved_sequence_1029.md) 3단계 '독립적인 작은 수정' 중 route-action 대기 queue guard 1건의 단일 source writer. 93 결정 (가). p320·공개 PR에 기존 담당 없음(8e 확인).

원 source (운영 root 12:11 읽기 전용 확인)

운영 source 사본 D:/260720_Ai_Workspace_Tools/kandev-local-main-20261022 (codex/production f742d87b14cf, git-common-dir D:/260720_Ai_Workspace_Tools/kandev-upstream-native-20261015/.git). 파일 apps/backend/internal/orchestrator/queue_dynamic_route_block.go(94줄): action_required + no executor에서 자동 queue claim의 무한 실패를 막고, 이미 기록된 userMessage·예약 identity를 보존하며, orphaned starting만 명시적 recovery까지 restore. 소비자: event_handlers_workflow.go:5850, task_operations.go:10358(queueDrainBlocked), event_handlers_agent.go:2281(retainQueuedMessageForDynamicRouteAction). 행번호는 current main에서 달라질 수 있음. container commit dfd27cb1e059/f742 통째 이식 0.

할 일 (시작 뒤)

  1. 자기 Native 새 사본(base 현재 upstream main), 운영 사본은 읽기만.
  2. 현재 main과 열린 dreampiler PR(#4710 queue resume, #4603 explicit retry 등)을 대조해 동등 수정이 있으면 그 결과를 근거로 기록하고 끝낸다.
  3. 빠졌으면 실제 entrypoint로 무한 claim 실패 재현 → 최소 포팅, 기존 test 파일 최소 회귀, 새 commit, normal hooks, 바뀐 delta만 gate rv-run.
  4. 첫 결과·검토된 head·영문 PR 초안을 8e/current 96e9d7f4-0840-479a-b7b6-3e18d45b4da4에 각 1회. 공개는 93→5d 새 좁은 PR. push/PR/댓글 0.

시작 조건(선행 대기)

공통 task_operations.go 등을 materialized profile guard 단위(KANDEV-UPSTREAM-S3-MATERIALIZED-PROFILE-GUARD-20261029)와 동시에 쓰지 않기 위해, 그 단위가 변경 경로를 확정하면 8e가 겹침을 대조해 이 task를 대기 칸으로 옮긴다. 운영 변경 0.

## 원 요구·승인 소유자 최신 실행 승인(sent/codex_upstream_execute_approved_sequence_1029.md) 3단계 '독립적인 작은 수정' 중 route-action 대기 queue guard 1건의 단일 source writer. 93 결정 (가). p320·공개 PR에 기존 담당 없음(8e 확인). ## 원 source (운영 root 12:11 읽기 전용 확인) 운영 source 사본 D:/260720_Ai_Workspace_Tools/kandev-local-main-20261022 (codex/production f742d87b14cf, git-common-dir D:/260720_Ai_Workspace_Tools/kandev-upstream-native-20261015/.git). 파일 apps/backend/internal/orchestrator/queue_dynamic_route_block.go(94줄): action_required + no executor에서 자동 queue claim의 무한 실패를 막고, 이미 기록된 userMessage·예약 identity를 보존하며, orphaned starting만 명시적 recovery까지 restore. 소비자: event_handlers_workflow.go:5850, task_operations.go:10358(queueDrainBlocked), event_handlers_agent.go:2281(retainQueuedMessageForDynamicRouteAction). 행번호는 current main에서 달라질 수 있음. container commit dfd27cb1e059/f742 통째 이식 0. ## 할 일 (시작 뒤) 1. 자기 Native 새 사본(base 현재 upstream main), 운영 사본은 읽기만. 2. 현재 main과 열린 dreampiler PR(#4710 queue resume, #4603 explicit retry 등)을 대조해 동등 수정이 있으면 그 결과를 근거로 기록하고 끝낸다. 3. 빠졌으면 실제 entrypoint로 무한 claim 실패 재현 → 최소 포팅, 기존 test 파일 최소 회귀, 새 commit, normal hooks, 바뀐 delta만 gate rv-run. 4. 첫 결과·검토된 head·영문 PR 초안을 8e/current 96e9d7f4-0840-479a-b7b6-3e18d45b4da4에 각 1회. 공개는 93→5d 새 좁은 PR. push/PR/댓글 0. ## 시작 조건(선행 대기) 공통 task_operations.go 등을 materialized profile guard 단위(KANDEV-UPSTREAM-S3-MATERIALIZED-PROFILE-GUARD-20261029)와 동시에 쓰지 않기 위해, 그 단위가 변경 경로를 확정하면 8e가 겹침을 대조해 이 task를 대기 칸으로 옮긴다. 운영 변경 0.
dreampiler commented 2026-10-29 15:49:18 +09:00
Author
Owner
Copy Link
Copy Source

8e receiving acceptance: ACCEPT stage-3 route-action waiting queue guard source unit (task 6649f0f8-e9b4-4856-ad8e-f44430fca632, single writer). 8e read-only checks: worktree kandev-upstream-s3-r_r8j1q157 branch feature/kandev-upstream-s3-r-kua HEAD 724b891020eae0e4331e3ce59368f339d9f856af, clean, 2 commits on upstream main dc2d77359f101fe398255112346b60c617477162 (17d8917d4ffa fix, 724b891020ea docs package + test move; no amend/force), 10 files +346/-3: new queue_dynamic_route_block.go, edits in event_handlers_workflow.go, task_operations.go (drainQueuedMessageForPromptableSessionLockedForIdentity), event_handlers_agent.go, dynamic_policy_recovery.go, the two focused cases moved into existing dynamic_stranded_starting_route_test.go (no new test file), AC-AGENTS-DYNAMIC-AGENT-ROUTING-001.15/.16, dynamic-agent-routing-02 note, docs/plans/route-action-waiting-queue-guard plan.md + task-01. 8e ran upstream pr-docs.cjs validateCoverage: covered, errors 0. RV: implementation FORMAL-rv-delta-review.md-1793254244766847800-23123-mid-1 APPROVE findings 0; new delta FORMAL-rv-delta-review-3.md-1793256333951190400-23959-mid-1 (CALL-20261029-e7dd5d3deb96) APPROVE findings 0 (8e read both). Source-reported: RED->GREEN through the real drain entrypoint, focused tests pass, gofmt/lint-go-changed 0, list-docs/lint-spec, hooks pass. Duplicate check: no equivalent on main or open PRs (#4710/#4603/#4604 distinct). Shared spec files with #4939 (materialized guard, AC .23): different sections, but both insert after AC .14, so whichever merges second needs a normal rebase of adjacent lines. No push/PR/comment. Publication: 93->5d new narrow PR from PR-DRAFT.md.

8e receiving acceptance: ACCEPT stage-3 route-action waiting queue guard source unit (task 6649f0f8-e9b4-4856-ad8e-f44430fca632, single writer). 8e read-only checks: worktree kandev-upstream-s3-r_r8j1q157 branch feature/kandev-upstream-s3-r-kua HEAD 724b891020eae0e4331e3ce59368f339d9f856af, clean, 2 commits on upstream main dc2d77359f101fe398255112346b60c617477162 (17d8917d4ffa fix, 724b891020ea docs package + test move; no amend/force), 10 files +346/-3: new queue_dynamic_route_block.go, edits in event_handlers_workflow.go, task_operations.go (drainQueuedMessageForPromptableSessionLockedForIdentity), event_handlers_agent.go, dynamic_policy_recovery.go, the two focused cases moved into existing dynamic_stranded_starting_route_test.go (no new test file), AC-AGENTS-DYNAMIC-AGENT-ROUTING-001.15/.16, dynamic-agent-routing-02 note, docs/plans/route-action-waiting-queue-guard plan.md + task-01. 8e ran upstream pr-docs.cjs validateCoverage: covered, errors 0. RV: implementation FORMAL-rv-delta-review.md-1793254244766847800-23123-mid-1 APPROVE findings 0; new delta FORMAL-rv-delta-review-3.md-1793256333951190400-23959-mid-1 (CALL-20261029-e7dd5d3deb96) APPROVE findings 0 (8e read both). Source-reported: RED->GREEN through the real drain entrypoint, focused tests pass, gofmt/lint-go-changed 0, list-docs/lint-spec, hooks pass. Duplicate check: no equivalent on main or open PRs (#4710/#4603/#4604 distinct). Shared spec files with #4939 (materialized guard, AC .23): different sections, but both insert after AC .14, so whichever merges second needs a normal rebase of adjacent lines. No push/PR/comment. Publication: 93->5d new narrow PR from PR-DRAFT.md.
dreampiler commented 2026-10-29 15:51:21 +09:00
Author
Owner
Copy Link
Copy Source

2026-10-29 upstream preparation complete and reviewed; publication pending 93 -> 5d. Task 6649f0f8-e9b4-4856-ad8e-f44430fca632. Branch feature/kandev-upstream-s3-r-kua. Base dc2d77359f101fe398255112346b60c617477162. Head 724b891020eae0e4331e3ce59368f339d9f856af (2 commits, clean). Commits: fix(orchestrator): hold queued delivery while a dynamic route awaits action; docs(agents): add route-action queue guard delivery package. Change: queue_dynamic_route_block.go; drain hooks in event_handlers_workflow.go and task_operations.go; retention in event_handlers_agent.go; shared orphan reason in dynamic_policy_recovery.go; focused tests in existing dynamic_stranded_starting_route_test.go; AC 001.15/.16 and design note; docs/plans/route-action-waiting-queue-guard work order. Verification: focused tests PASS; gofmt clean; lint-go-changed 0 issues; list-docs/lint-spec pass; commit hooks pass; validateCoverage(dc2d773..724b891) = covered, no errors. Formal RV: production CALL-20261029-7b992f23dee8 APPROVE 5/5 findings 0; delta CALL-20261029-e7dd5d3deb96 APPROVE 5/5 findings 0, id FORMAL-rv-delta-review-3.md-1793256333951190400-23959-mid-1. Artifacts: _UserHub/_task/kandev-upstream-s3-route-action-queue-guard-20261029/. Push/PR/comment: 0. Flow stays open pending publication/readback.

2026-10-29 upstream preparation complete and reviewed; publication pending 93 -> 5d. Task 6649f0f8-e9b4-4856-ad8e-f44430fca632. Branch feature/kandev-upstream-s3-r-kua. Base dc2d77359f101fe398255112346b60c617477162. Head 724b891020eae0e4331e3ce59368f339d9f856af (2 commits, clean). Commits: fix(orchestrator): hold queued delivery while a dynamic route awaits action; docs(agents): add route-action queue guard delivery package. Change: queue_dynamic_route_block.go; drain hooks in event_handlers_workflow.go and task_operations.go; retention in event_handlers_agent.go; shared orphan reason in dynamic_policy_recovery.go; focused tests in existing dynamic_stranded_starting_route_test.go; AC 001.15/.16 and design note; docs/plans/route-action-waiting-queue-guard work order. Verification: focused tests PASS; gofmt clean; lint-go-changed 0 issues; list-docs/lint-spec pass; commit hooks pass; validateCoverage(dc2d773..724b891) = covered, no errors. Formal RV: production CALL-20261029-7b992f23dee8 APPROVE 5/5 findings 0; delta CALL-20261029-e7dd5d3deb96 APPROVE 5/5 findings 0, id FORMAL-rv-delta-review-3.md-1793256333951190400-23959-mid-1. Artifacts: _UserHub/_task/kandev-upstream-s3-route-action-queue-guard-20261029/. Push/PR/comment: 0. Flow stays open pending publication/readback.
dreampiler commented 2026-10-29 18:03:08 +09:00
Author
Owner
Copy Link
Copy Source

2026-10-29 PR #4940 review fixes complete and reviewed. Task 6649f0f8. Branch feature/kandev-upstream-s3-r-kua. Base dc2d77359f101fe398255112346b60c617477162. Head f0e526744a4b35d6bb2d7f767e9000b17c7ab1d7. Commits: 19120a1 fix(orchestrator): release retained queue rows for dynamic route recovery; f0e5267 test(orchestrator): prove the route action waits for the queue release. P1 Greptile 4409517703 durable reserved row release in place; P3 CodeRabbit 4409585411 route-action lock ordering; P2 Greptile 4409517710 comment invariants. Tests in existing dynamic_stranded_starting_route_test.go; focused 1.020s; neighbour 2.514s; lock-order test pass; gofmt clean; golangci-lint 0; spec validators pass; validateCoverage covered; hooks pass. Final RV CALL-20261029-7545eb31bb71 APPROVE 5/5 findings 0 id FORMAL-rv-delta-review-5.md-1793263836979995700-5926-mid-1. Prior RV CALL-20261029-f58133bb3e28 REQUEST_CHANGES resolved by f0e5267. Result sent to 93 (SID d6347506). Push/PR/comment: 0. Flow stays open pending 93 -> 5d publication/readback.

2026-10-29 PR #4940 review fixes complete and reviewed. Task 6649f0f8. Branch feature/kandev-upstream-s3-r-kua. Base dc2d77359f101fe398255112346b60c617477162. Head f0e526744a4b35d6bb2d7f767e9000b17c7ab1d7. Commits: 19120a1 fix(orchestrator): release retained queue rows for dynamic route recovery; f0e5267 test(orchestrator): prove the route action waits for the queue release. P1 Greptile 4409517703 durable reserved row release in place; P3 CodeRabbit 4409585411 route-action lock ordering; P2 Greptile 4409517710 comment invariants. Tests in existing dynamic_stranded_starting_route_test.go; focused 1.020s; neighbour 2.514s; lock-order test pass; gofmt clean; golangci-lint 0; spec validators pass; validateCoverage covered; hooks pass. Final RV CALL-20261029-7545eb31bb71 APPROVE 5/5 findings 0 id FORMAL-rv-delta-review-5.md-1793263836979995700-5926-mid-1. Prior RV CALL-20261029-f58133bb3e28 REQUEST_CHANGES resolved by f0e5267. Result sent to 93 (SID d6347506). Push/PR/comment: 0. Flow stays open pending 93 -> 5d publication/readback.
dreampiler commented 2026-10-29 18:45:03 +09:00
Author
Owner
Copy Link
Copy Source

8e acceptance and publication readback: route-action waiting queue guard is published upstream as kdlbs/kandev#4940 and its three review threads are fixed. Source 6649f0f8 final head f0e526744a4b35d6bb2d7f767e9000b17c7ab1d7, descendant of the first public head 724b891020ea (8e git check; 4 files vs 724b: queue_dynamic_route_block.go, existing dynamic_stranded_starting_route_test.go, task-01 work order, dynamic-agent-routing-02 note; no new file). Final formal RV FORMAL-rv-delta-review-5.md-1793263836979995700-5926-mid-1 (CALL-20261029-7545eb31bb71) completed APPROVE findings 0 (8e read the record; earlier barrier/design REQUEST_CHANGES rounds were fixed before it). 8e gh readback: #4940 OPEN, MERGEABLE, head f0e526744a4b, unresolved review threads 0. Root/5d readback: xcjqx answered 4410414029 (durable rows released via ReleaseQueuedDeliveryForSession, only removed reservations restored), xcjq0 answered 4410415222 (invariant comments), xct0t answered 4410416553 (existing per-session operation lock plus barrier identity/visibility assertions; bot agreed 4410449164). Checks on f0e5267: 13 check-run records at 93 read (5 success, 2 in progress, 6 skipped, 0 failed) plus a separate PR documentation coverage status success; root later read all terminal with 0 failed. Not a whole-CI-green or merge claim. Maintainer review and merge remain external.

8e acceptance and publication readback: route-action waiting queue guard is published upstream as kdlbs/kandev#4940 and its three review threads are fixed. Source 6649f0f8 final head f0e526744a4b35d6bb2d7f767e9000b17c7ab1d7, descendant of the first public head 724b891020ea (8e git check; 4 files vs 724b: queue_dynamic_route_block.go, existing dynamic_stranded_starting_route_test.go, task-01 work order, dynamic-agent-routing-02 note; no new file). Final formal RV FORMAL-rv-delta-review-5.md-1793263836979995700-5926-mid-1 (CALL-20261029-7545eb31bb71) completed APPROVE findings 0 (8e read the record; earlier barrier/design REQUEST_CHANGES rounds were fixed before it). 8e gh readback: #4940 OPEN, MERGEABLE, head f0e526744a4b, unresolved review threads 0. Root/5d readback: xcjqx answered 4410414029 (durable rows released via ReleaseQueuedDeliveryForSession, only removed reservations restored), xcjq0 answered 4410415222 (invariant comments), xct0t answered 4410416553 (existing per-session operation lock plus barrier identity/visibility assertions; bot agreed 4410449164). Checks on f0e5267: 13 check-run records at 93 read (5 success, 2 in progress, 6 skipped, 0 failed) plus a separate PR documentation coverage status success; root later read all terminal with 0 failed. Not a whole-CI-green or merge claim. Maintainer review and merge remain external.
dreampiler commented 2026-10-29 18:46:26 +09:00
Author
Owner
Copy Link
Copy Source

인계·정리 완료.

  • 보존: feature/kandev-upstream-s3-r-kua @ f0e526744a4b (base dc2d77359f1, 4 commits, clean, not pushed)
  • 병합·push·PR·댓글: 0 (상류 준비 단위, branch 보존)
  • 최종 RV: CALL-20261029-7545eb31bb71 APPROVE, 5/5, findings 0
  • 93 인계: 새 head·RV·validateCoverage·PR 초안 전달됨 (d6347506, sent)
  • flow 기록: CALL-20261029-46bde24a14f8, receipt FLOW-NOTE-...-20261029T090319563976Z, queue는 publication 대기로 open
  • 남은 것: 93/5d의 공개 #4940을 f0e5267로 갱신·readback, 이후 담당 writer가 flow close
인계·정리 완료. - **보존**: `feature/kandev-upstream-s3-r-kua` @ `f0e526744a4b` (base `dc2d77359f1`, 4 commits, clean, not pushed) - **병합·push·PR·댓글**: 0 (상류 준비 단위, branch 보존) - **최종 RV**: `CALL-20261029-7545eb31bb71` **APPROVE**, 5/5, findings 0 - **93 인계**: 새 head·RV·validateCoverage·PR 초안 전달됨 (`d6347506`, `sent`) - **flow 기록**: `CALL-20261029-46bde24a14f8`, receipt `FLOW-NOTE-...-20261029T090319563976Z`, queue는 publication 대기로 open - **남은 것**: 93/5d의 공개 #4940을 `f0e5267`로 갱신·readback, 이후 담당 writer가 flow close
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dreampiler (dreampiler)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dreampiler/kandev#205
Delete Branch "%!s()"

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?