Luxxy/beegoon
Watch
1
0
Fork
You've already forked beegoon
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix RefreshInventory always failing with CommandNotSupported #7

Merged
Luxxy merged 1 commit from phase-2-3-inventory-operations into main 2026-08-02 22:36:01 +00:00
Conversation 0 Commits 1 Files changed 1 +215 -19
Luxxy commented 2026-08-02 22:35:57 +00:00
Owner
Copy link

The agent answered a RefreshInventory command with an InventoryReport but
then unconditionally fell through to the generic command executor, which
has no case for it and always returned CommandNotSupported. This went
unnoticed because nothing ever dispatched RefreshInventory until the new
automatic post-pull refresh and the manual admin refresh button started
exercising it, both surfacing as a failed operation despite the report
having gone out successfully.

Adds two regression tests exercising handle_message against a mock socket:
a healthy runtime reports success with the inventory report, and an
unreachable runtime fails explicitly with RuntimeUnavailable.

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com

The agent answered a RefreshInventory command with an InventoryReport but then unconditionally fell through to the generic command executor, which has no case for it and always returned CommandNotSupported. This went unnoticed because nothing ever dispatched RefreshInventory until the new automatic post-pull refresh and the manual admin refresh button started exercising it, both surfacing as a failed operation despite the report having gone out successfully. Adds two regression tests exercising handle_message against a mock socket: a healthy runtime reports success with the inventory report, and an unreachable runtime fails explicitly with RuntimeUnavailable. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Fix RefreshInventory always failing with CommandNotSupported
Some checks failed
Agent CI / Against a real Incus daemon (pull_request) Has been skipped
Details
Agent CI / Test the agent (pull_request) Failing after 2s
Details
Security / Advisories, licences and sources (pull_request) Failing after 2s
Details
Security / No secrets or placeholder credentials committed (pull_request) Failing after 2s
Details
Security / Frontend dependency audit (pull_request) Failing after 38s
Details
Agent CI / The workspace still compiles off Linux (pull_request) Has been cancelled
Details
797e8a5b8c 
The agent answered a RefreshInventory command with an InventoryReport but
then unconditionally fell through to the generic command executor, which
has no case for it and always returned CommandNotSupported. This went
unnoticed because nothing ever dispatched RefreshInventory until the new
automatic post-pull refresh and the manual admin refresh button started
exercising it, both surfacing as a failed operation despite the report
having gone out successfully.

Adds two regression tests exercising handle_message against a mock socket:
a healthy runtime reports success with the inventory report, and an
unreachable runtime fails explicitly with RuntimeUnavailable.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Luxxy merged commit 6b54b1469e into main 2026-08-02 22:36:01 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
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
Luxxy/beegoon!7
No description provided.