close: pod tool surface cleanup
This commit is contained in:
parent
97484091b6
commit
ae6eb52f35
|
|
@ -2,12 +2,12 @@
|
||||||
id: 20260531-022821-pod-tool-surface-restore-list
|
id: 20260531-022821-pod-tool-surface-restore-list
|
||||||
slug: pod-tool-surface-restore-list
|
slug: pod-tool-surface-restore-list
|
||||||
title: Pod tools: unify pod listing and rename restore operation
|
title: Pod tools: unify pod listing and rename restore operation
|
||||||
status: open
|
status: closed
|
||||||
kind: task
|
kind: task
|
||||||
priority: P2
|
priority: P2
|
||||||
labels: [pod, tools, orchestration]
|
labels: [pod, tools, orchestration]
|
||||||
created_at: 2026-05-31T02:28:21Z
|
created_at: 2026-05-31T02:28:21Z
|
||||||
updated_at: 2026-05-31T02:59:12Z
|
updated_at: 2026-05-31T03:00:15Z
|
||||||
assignee: null
|
assignee: null
|
||||||
legacy_ticket: null
|
legacy_ticket: null
|
||||||
---
|
---
|
||||||
|
|
@ -0,0 +1 @@
|
||||||
|
Simplified the LLM-facing Pod tool surface: `ListPods` now uses visibility/state-backed listing semantics, `ListVisiblePods` and `InspectPod` are removed from the active tool/protocol surface, and `AttachOrRestorePod` is renamed to `RestorePod`. TUI/protocol/docs/tests were updated, external review approved, and main workspace validation passed.
|
||||||
|
|
@ -76,4 +76,30 @@ Non-blocking follow-ups:
|
||||||
- A small assertion that removed tool names are absent from the registry would satisfy the ticket wording more literally, but reviewer did not consider it blocking.
|
- A small assertion that removed tool names are absent from the registry would satisfy the ticket wording more literally, but reviewer did not consider it blocking.
|
||||||
|
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
<!-- event: implementation_report author: hare at: 2026-05-31T03:00:14Z -->
|
||||||
|
|
||||||
|
## Implementation report
|
||||||
|
|
||||||
|
Main workspace validation after merge:
|
||||||
|
|
||||||
|
- `cargo fmt --check` passed
|
||||||
|
- `cargo check -p pod -p protocol -p tui` passed with pre-existing dead-code warnings in `llm-worker` and `tui`
|
||||||
|
- `cargo test -p protocol pod_discovery` passed
|
||||||
|
- `cargo test -p pod state_backed_visibility_and_restore_planning` passed
|
||||||
|
- `cargo test -p pod --test pod_comm_tools_test` passed
|
||||||
|
- `./tickets.sh doctor` passed
|
||||||
|
- `git diff --check` passed
|
||||||
|
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
<!-- event: close author: hare at: 2026-05-31T03:00:15Z status: closed -->
|
||||||
|
|
||||||
|
## Closed
|
||||||
|
|
||||||
|
Simplified the LLM-facing Pod tool surface: `ListPods` now uses visibility/state-backed listing semantics, `ListVisiblePods` and `InspectPod` are removed from the active tool/protocol surface, and `AttachOrRestorePod` is renamed to `RestorePod`. TUI/protocol/docs/tests were updated, external review approved, and main workspace validation passed.
|
||||||
|
|
||||||
|
|
||||||
---
|
---
|
||||||
Loading…
Reference in New Issue
Block a user