aep/claude-harness
1
0
Fork 0
Code Issues 13 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

approval UX: request_approval-инструмент вместо ручной команды (решение владельца 25.10) #77

New Issue
Closed
opened 2026-10-25 09:42:07 +03:00 by aep · 1 comment
aep commented 2026-10-25 09:42:07 +03:00
Owner
Copy Link

Владельцу не нравится ручной /approve kind: неочевидный синтаксис, инициатива на стороне человека. Целевой флоу: модель вызывает инструмент request_approval {change_id, kind} → владельцу показывается userQuestions-подтверждение «Утвердить ?» → маркер пишет код бандла. Гейт C не трогаем: approval.md по-прежнему пишет только бандл, модель подделать не может. /approve остаётся ручным фолбэком.

Технические вопросы для дизайна: как бандл регистрирует model-инструмент в DSH 0.2.1-rc.3 (ctx.tools API, схема аргументов — schemastery или plain JSON; ограничение «без @deepseek-ai импортов» из package.test.mjs) — проверить по app.asar; взаимодействие с гейтом B (вызов инструмента сам по себе ничего не пишет, кроме approval.md — должно проходить).

Связано: #59 (гейты), #76 (worktree-резолюция), #43 (первый потребитель нового флоу).

Владельцу не нравится ручной /approve <id> kind: неочевидный синтаксис, инициатива на стороне человека. Целевой флоу: модель вызывает инструмент request_approval {change_id, kind} → владельцу показывается userQuestions-подтверждение «Утвердить <change> <kind>?» → маркер пишет код бандла. Гейт C не трогаем: approval.md по-прежнему пишет только бандл, модель подделать не может. /approve остаётся ручным фолбэком. Технические вопросы для дизайна: как бандл регистрирует model-инструмент в DSH 0.2.1-rc.3 (ctx.tools API, схема аргументов — schemastery или plain JSON; ограничение «без @deepseek-ai импортов» из package.test.mjs) — проверить по app.asar; взаимодействие с гейтом B (вызов инструмента сам по себе ничего не пишет, кроме approval.md — должно проходить). Связано: #59 (гейты), #76 (worktree-резолюция), #43 (первый потребитель нового флоу).
aep commented 2026-10-25 10:20:10 +03:00
Author
Owner
Copy Link

Закрыто: PR #78 (request_approval-инструмент). Merge по команде владельца 25.10. Deferred: T1.3 (dsh-tools.md — в #43 Task 4), T1.5 (формулировка ошибки диалога в спеке — при следующем касании). Осталось владельцу: переустановить бандл и перезапустить DSH.

Закрыто: PR #78 (request_approval-инструмент). Merge по команде владельца 25.10. Deferred: T1.3 (dsh-tools.md — в #43 Task 4), T1.5 (формулировка ошибки диалога в спеке — при следующем касании). Осталось владельцу: переустановить бандл и перезапустить DSH.
aep closed this issue 2026-10-25 10:20:10 +03:00
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
aep
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aep/claude-harness#77
No description provided.
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?