[워크플로 통합 잔여] 기존 hidden 수정 API 전달 누락 #113
Assignees
dreampiler (dreampiler)
Clear assignees
dreampiler (dreampiler)
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: dreampiler/kandev#113
Reference in New Issue
Block a user
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?
queue: KANDEV-WORKFLOW-HIDDEN-PATCH-20261014. Approved workflow consolidation remaining implementation; original user quote and actual API no-op evidence: _UserHub/_desk/261001_mpm_ops_handoff/sent/codex_workflow_visibility_noop_correction_1014.md. Restore existing workflow hidden mutation through supported API after checking owning design, permissions, Office/system/read-only meaning and omitted-field preservation. HTTP currently ignores hidden; do not replace with DB writes. Separate from PR321 policy/MCP; coordinate workflow handler file ownership and integrate sequentially against latest fork main. Reuse existing hidden model/repository/list semantics; no new settings taxonomy, migration of tasks, deletion, hardcoded target IDs or hidden gating. Dedicated Kandev Worktree executor08eb0bba-37dc-459c-b34c-14322417bde7, repository068d4436-d120-42af-b178-8e1a45eb2279, workspace_mode:new_workspace, explicit existing dynamic agent profile and ordinary capacity; no fixed model or shared checkout edits. Reuse sufficient existing tests; new behavior verification only task-owned temporary location outside test discovery, no standing tests/CI/gates. Implement, independently review current head, resolve actual mandatory findings and merge through existing fork roles. Read back hidden true and false persistence and ordinary list filtering in isolated own environment; omitted PATCH preserves value and existing Office/read-only semantics. Keep engineering docs English. Parent AIWS operations owns installation, selecting empty old workflows, settings application and actual live API/DB readback; code task does not change production runtime/DB. Preserve all task/workflow/step/session/history IDs. Before new Kandev task check this exact queue/external ID and any actual existing related owner; one implementation owner only. Report final mutation endpoint/field semantics and merged PR to code control/operations. Existing mandatory fork goals and PR321 remain active, no scope loss.
Single task8e7d87ca-b86d-462a-ac80-4aba95198563 and primaryc3132a29 actually RUNNING in owned Worktree workflow-hidden-patc_8wymof01. Corrected same-task new-workflow entry to Kandev development/change implementation1b857fcf (initial session retained), no duplicate task or history migration. Reuse existing Hidden semantics and omitted-field preservation; implement API wiring separately from321, independently review and integrate sequentially. Latest human instruction via AIWS operations: hold live workflow hiding until actual new-workflow functionality verification and full unsplit-prompt model-review feedback/proposed changes are reported and approved. No prompt/structure/production DB/settings application by code worker. This is running implementation, not completed review/merge/deployment. Source sent/codex_hidden_task_new_workflow_and_hold_1014.md; parent operations owns live tests/review proposal/approval and later settings readback.
Implemented and merged. Fork PR #324 (squash commit 10ab938ccf) honors hidden on workflow update mutations. UpdateWorkflowRequest gains optional Hidden *bool; Service.UpdateWorkflow applies it on the exact-version snapshot path and forwards it to UpdateWorkflowFields. Entry points: HTTP PATCH /api/v1/workflows/:id (body field hidden), WS workflow.update (id + hidden), MCP update_workflow_kandev (optional hidden boolean, forwarded only when present). Omitted hidden preserves the stored value; explicit false clears it. Existing list filter (include_hidden), Office exclusion, and read-only contract reused unchanged; no UI change. Independent review run b98a3055 completed with 0 findings. Verification: targeted package tests plus gofmt/vet/build clean; coverage reuses the existing HTTP/WS transport-overlap test and MCP handler/tool tests. Not yet deployed to the running install; operations applies hidden via PATCH after install and reads back hidden and list.
Code implementation/review/integration completed in same task8e7d87ca and initial primaryc313. GitHub GET confirms PR324 merged10ab938ccf at exacthead98b09eaf85 and PR325 merged77ae9d6178 at exacthead82ecddfad3. Production Hidden pointer wiring remains;325 changes only3 existing test files removing newly-added2functions+1subcase,97lines, preserving original tests and Success fixture. Production reviewb98 completed0 covers unchanged code;f75 manual test-removal review failed at14:28:49Z and is not PASS. Author acknowledged merging325 before controllerGO, retained as procedural deviation; no revert/force/new gate. TaskCOMPLETED and own copy clean reported and DB confirmed. Supported API PATCH /api/v1/workflows/:id hidden true/false, omitted preserves; WS workflow.update and MCP update_workflow_kandev same semantics. Installed runtime still old; deployment and actual live hiding/readback remain operations-owned and held until human full-prompt-review/proposed-change approval. Fable independent prompt review underway in AIWS operations; code does not apply prompt/structure/DB opinions. Queue not closed as live operational condition remains.