lsm / neokai / 25008225399 / 1
82%
dev: 82%

DEFAULT BRANCH: dev
Ran
Files 29
Run time 4s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 84.825%. Remained the same
25008225399.1

push

github

web-flow 
fix(space): skip redundant merge approval when human already approved (#1668)

* fix(space): skip redundant merge approval when human already approved

When a human approves a task in review, the post-approval merge template
was asking them to approve the merge a second time (step 2 gated on
autonomy_level < 4 alone). Add approval_source == "auto" check so that
human-approved tasks merge directly without the redundant prompt.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* fix(space): use != "human" instead of == "auto" for approval_source check

SpaceApprovalSource is 'human' | 'auto_policy' | 'agent' — there is no
"auto" value. The previous condition would never match, silently skipping
step 2 for all cases. Using != "human" correctly covers both auto_policy
and agent variants.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* fix(space): update stale file header comment for approval_source values

The header claimed 'end_node' | 'human_review' but SpaceApprovalSource
is actually 'human' | 'auto_policy' | 'agent'. Update to match reality.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* fix(space): remove "run tests" from reviewer prompt (gate already requires CI)

The code-ready-gate already requires all CI tests to pass before the
reviewer is activated, so telling the reviewer to run tests is redundant
and wastes time.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

2208 of 2603 relevant lines covered (84.83%)

40.5 hits per line

Source Files on job daemon-0-shared - 25008225399.1