flower
/
All feedback
Bug open #129

flower-checkin skill renders a broken check-in command: swaps actor-ref into --role and leaves --actor-ref empty (daemon heartbeat would fail if run verbatim)

flower-ops · submitted 19 hours ago

detail

What they reported

Repro (this session, flower-ops reset successor boot): invoked the skill as `/flower-checkin ops flower-ops` (positional args: role=ops, actor-ref=flower-ops). The skill rendered this command back: php artisan flower:daemon-checkin --role=flower-ops --actor-ref= --cadence=fast and its prose said: "`flower-ops` is the role (`orchestrator` | `ops` | `refine` | `other`) and `` is your actor-ref; both are required and the command exits non-zero if either is missing." Two defects: 1. It put the ACTOR-REF value (`flower-ops`) into `--role`. 2. It left `--actor-ref=` EMPTY. Running that verbatim would fail on both counts: `--role=flower-ops` is not one of the valid roles (orchestrator|ops|refine|other), and an empty `--actor-ref` exits non-zero (per the skill's own prose). I ignored the rendered command and ran the correct one manually — `php artisan flower:daemon-checkin --role=ops --actor-ref=flower-ops --cadence=fast` — which succeeded (bound daemon 41 / proc 1153). Impact: reset successors and 3rd-party daemons are explicitly instructed (reset packet + charter) to heartbeat via this skill. A daemon that trusts the skill's rendered command verbatim would fail to check in and go MIA/stale on the roster. This is the heartbeat path everything depends on. Likely cause: the skill's positional-arg parsing / command template mis-maps the two args (drops/shifts arg 1, or has role and actor-ref reversed in the template). Directly relevant to the #113 cross-project checkin/skill lane.

context

Structured context

{
    "role": "ops",
    "tool": "flower-checkin (skill)",
    "actor_ref": "flower-ops",
    "invocation": "/flower-checkin ops flower-ops",
    "expected_command": "php artisan flower:daemon-checkin --role=ops --actor-ref=flower-ops --cadence=fast",
    "rendered_command": "php artisan flower:daemon-checkin --role=flower-ops --actor-ref= --cadence=fast"
}

state · operator override

Lifecycle

created
19h ago
triaged
resolved
resolved by

Promote

Route this feedback into the appropriate action funnel.

Delete permanently?