Skip to content

fix: honor explicit human delivery acceptance#222

Open
fujiwaranosai850 wants to merge 1 commit into
devclaw-local-devfrom
issue/218-delivery-phase-review-fixes
Open

fix: honor explicit human delivery acceptance#222
fujiwaranosai850 wants to merge 1 commit into
devclaw-local-devfrom
issue/218-delivery-phase-review-fixes

Conversation

@fujiwaranosai850
Copy link
Copy Markdown
Collaborator

Summary

  • add explicit candidate decision comments so human-routed acceptance has an observable gate
  • keep human-routed promotion gated on explicit candidate records
  • wire delivery-transition heartbeat totals into the aggregated heartbeat logger

Testing

  • npx tsx --test lib/services/delivery-phases.test.ts lib/services/pipeline-delivery.test.ts lib/config/schema.test.ts

Notes

  • npm run check still reports pre-existing workspace issues unrelated to this change (missing vitest/openclaw type deps and existing work-finish test/type errors).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant