fix: pass the issue title+body to the dev agent so agent/dev works without a plan #20

Merged

ellen merged 1 commit from fix/dev-prompt-includes-issue into main 2026-07-31 18:54:37 +00:00

Conversation 0 Commits 1 Files changed 1 +22 -8

cloudcli commented 2026-07-31 18:35:37 +00:00

Member

Copy link

Problem

dev.yaml builds the model prompt from two sources only: the
## Implementation plan (agent) comment and the tester's ## Test findings
comment. The issue's own title and body are never passed.

That is fine on the designed path (agent/plan → plan/approved → implement),
but it makes agent/dev unusable as a standalone trigger. Label a fresh issue
agent/dev and $PLAN is empty, so the agent is told "implement the approved
plan below" with nothing below it and no other statement of the task. It
produces no commit, the progress=no branch fires, and the issue is parked to
needs/human.

This matters for queueing work ahead of a usage reset: agent/dev is the label
you would reach for to queue a small, already-understood change without spending
a planning run on it.

Change

• Fetch the issue title + body and always include them in the prompt as an
  === ISSUE #N === section, ahead of the plan.
• When $PLAN is empty, substitute an explicit instruction to work from the
  issue text and keep the change tightly scoped, instead of emitting an empty
  section.
• CHECKPOINT now logs the issue title, so the run log shows what the agent was
  actually given.

No behaviour change on the plan/approved path — the plan is still the primary
instruction there, now with the issue as framing.

Merge order

Cut from main. It touches only the build step's prompt region, so it merges
cleanly with feat/runner-injected-credentials (which touches the credentials
region of the same file) in either order. Note that until that branch is merged,
no route runs at all — every route file on main still carries the secrets:
block under on.workflow_call that makes the file unparseable.

### Problem `dev.yaml` builds the model prompt from two sources only: the `## Implementation plan (agent)` comment and the tester's `## Test findings` comment. The issue's own title and body are never passed. That is fine on the designed path (`agent/plan` → `plan/approved` → implement), but it makes `agent/dev` unusable as a standalone trigger. Label a fresh issue `agent/dev` and `$PLAN` is empty, so the agent is told *"implement the approved plan below"* with nothing below it and no other statement of the task. It produces no commit, the `progress=no` branch fires, and the issue is parked to `needs/human`. This matters for queueing work ahead of a usage reset: `agent/dev` is the label you would reach for to queue a small, already-understood change without spending a planning run on it. ### Change - Fetch the issue title + body and always include them in the prompt as an `=== ISSUE #N ===` section, ahead of the plan. - When `$PLAN` is empty, substitute an explicit instruction to work from the issue text and keep the change tightly scoped, instead of emitting an empty section. - `CHECKPOINT` now logs the issue title, so the run log shows what the agent was actually given. No behaviour change on the `plan/approved` path — the plan is still the primary instruction there, now with the issue as framing. ### Merge order Cut from `main`. It touches only the build step's prompt region, so it merges cleanly with `feat/runner-injected-credentials` (which touches the credentials region of the same file) in either order. Note that until that branch is merged, no route runs at all — every route file on `main` still carries the `secrets:` block under `on.workflow_call` that makes the file unparseable.

cloudcli added 1 commit 2026-07-31 18:35:37 +00:00

fix: pass the issue title+body to the dev agent so agent/dev works without a plan 55fb22eb40

ellen merged commit dd99e6a2da into main 2026-07-31 18:54:37 +00:00

ellen deleted branch fix/dev-prompt-includes-issue 2026-07-31 18:54:37 +00:00

ellen referenced this pull request from a commit 2026-07-31 18:54:38 +00:00

Merge pull request 'fix: pass the issue title+body to the dev agent so agent/dev works without a plan' (#20) from fix/dev-prompt-includes-issue into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

  

agent

coordinate

Summon coordinator on an epic (one-shot; agent removes on pickup)

  

agent

dev

Summon developer (one-shot; agent removes on pickup)

  

agent

plan

Summon planner (one-shot; agent removes on pickup)

  

agent

reflect

Summon reflection agent (one-shot; agent removes on pickup)

  

agent

test

Summon tester on a PR (one-shot; agent removes on pickup)

  

agent

triage

Summon triage (one-shot; agent removes on pickup)

  

ai/off

Agents ignore this issue entirely

  

blocked

night

Queued: waiting for night window

  

blocked

usage

Queued: usage window exhausted (resume time in state comment)

  

loop

1

Dev↔test bounce 1 of 3 at current tier

  

loop

2

Dev↔test bounce 2 of 3 at current tier

  

loop

3

Dev↔test bounce 3 of 3 — cap; next is escalate or needs/human

  

needs/approval

Agent → you: plan awaits approval

  

needs/human

Agent → you: stuck / escalated out (see summary comment)

  

needs/input

Agent → you: blocked on a question only you can answer

  

needs/review

Agent → you: PR passed tests, awaits human review/merge

  

plan/approved

Human approval gate — honored only when applied by APPROVER

  

run/night

Start only inside NIGHT_WINDOW (default 00:00–06:00 Europe/Helsinki)

  

state

blocked

Machine state: blocked (see needs/* or blocked/*)

  

state

building

Machine state (workflow-managed)

  

state

passed

Machine state: tests passed, awaiting merge

  

state

planning

Machine state (workflow-managed)

  

state

testing

Machine state (workflow-managed)

  

tier

fable

Model rung: Fable 5 — HUMAN-APPLIED ONLY

  

tier

haiku

Model rung: Haiku 4.5

  

tier

local

Model rung: local vLLM (no quota)

  

tier

opus

Model rung: Opus 4.8 (hard dev, default plan)

  

tier

sonnet

Model rung: Sonnet 5 (default dev/test)

No labels

agent

coordinate

agent

dev

agent

plan

agent

reflect

agent

test

agent

triage

ai/off

blocked

night

blocked

usage

loop

1

loop

2

loop

3

needs/approval

needs/human

needs/input

needs/review

plan/approved

run/night

state

blocked

state

building

state

passed

state

planning

state

testing

tier

fable

tier

haiku

tier

local

tier

opus

tier

sonnet

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

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

valssi-public/agent-workflows!20

No description provided.