agent

core/agent

Fork 0

Commit graph

Author	SHA1	Message	Date
Snider	026b31edf7	fix: address Codex round 2 mediums Some checks failed CI / test (push) Failing after 3s Details - harvest: message says 'ready-for-review' not 'pushed' - sync: timestamp advanced after pulls, not before - sync: accepts main/master/reported branch, not just main - inbox: checks CORE_BRAIN_KEY env before falling back to file - inbox: parses 'from' not 'from_agent', 'messages' not 'data' - queue: strips variant suffix for rate limit lookup (claude:opus → claude) - review_queue: respects ReviewQueueInput.Reviewer instead of hardcoding coderabbit - tests: updated to match real API response structure Co-Authored-By: Virgil <virgil@lethean.io>	2026-03-21 16:05:59 +00:00
Snider	422777580b	fix: address Codex review findings — 2 high, 3 medium Some checks failed CI / test (push) Failing after 3s Details High: Fix missed-notification bug — track completions by workspace name instead of count, so harvest status rewrites don't suppress future notifications. Also tracks blocked/failed terminal states. High: Safety gate fail-closed — check ALL changed files (not just added), reject on git diff failure instead of proceeding. Medium: emitCompletionEvent now passes actual status (completed, failed, blocked) instead of hardcoding "completed". Medium/AX: Harvest no longer auto-pushes to source repos. Sets status to ready-for-review only — pushing happens during explicit review, not silently in the background. Co-Authored-By: Virgil <virgil@lethean.io>	2026-03-21 15:31:29 +00:00
Snider	8eb2430486	test(monitor): add unit tests for completions, inbox, lifecycle Coverage: 23.2% → 86.5% Tests for checkCompletions, checkInbox, monitor loop, Poke, New, Start/Shutdown. Uses httptest for API mocking. Co-Authored-By: Virgil <virgil@lethean.io>	2026-03-21 13:49:23 +00:00

Author

SHA1

Message

Date

Snider

026b31edf7

fix: address Codex round 2 mediums

CI / test (push) Failing after 3s

Details

- harvest: message says 'ready-for-review' not 'pushed'
- sync: timestamp advanced after pulls, not before
- sync: accepts main/master/reported branch, not just main
- inbox: checks CORE_BRAIN_KEY env before falling back to file
- inbox: parses 'from' not 'from_agent', 'messages' not 'data'
- queue: strips variant suffix for rate limit lookup (claude:opus → claude)
- review_queue: respects ReviewQueueInput.Reviewer instead of hardcoding coderabbit
- tests: updated to match real API response structure

Co-Authored-By: Virgil <virgil@lethean.io>

2026-03-21 16:05:59 +00:00

Snider

422777580b

fix: address Codex review findings — 2 high, 3 medium

CI / test (push) Failing after 3s

Details

High: Fix missed-notification bug — track completions by workspace
name instead of count, so harvest status rewrites don't suppress
future notifications. Also tracks blocked/failed terminal states.

High: Safety gate fail-closed — check ALL changed files (not just
added), reject on git diff failure instead of proceeding.

Medium: emitCompletionEvent now passes actual status (completed,
failed, blocked) instead of hardcoding "completed".

Medium/AX: Harvest no longer auto-pushes to source repos. Sets
status to ready-for-review only — pushing happens during explicit
review, not silently in the background.

Co-Authored-By: Virgil <virgil@lethean.io>

2026-03-21 15:31:29 +00:00

Snider

8eb2430486

test(monitor): add unit tests for completions, inbox, lifecycle

Coverage: 23.2% → 86.5%
Tests for checkCompletions, checkInbox, monitor loop,
Poke, New, Start/Shutdown. Uses httptest for API mocking.

Co-Authored-By: Virgil <virgil@lethean.io>

2026-03-21 13:49:23 +00:00

3 commits