v0.3.0: remove agent service, beef up reef tests, changelog#7
Merged
AlephNotation merged 5 commits intomainfrom Mar 1, 2026
Merged
v0.3.0: remove agent service, beef up reef tests, changelog#7AlephNotation merged 5 commits intomainfrom
AlephNotation merged 5 commits intomainfrom
Conversation
Remove agent service: - services/agent/ deleted — replaced entirely by src/reef.ts - /agent/tasks and /agent/sessions endpoints gone; use POST /reef/submit - Cron agent jobs now post to /reef/submit instead of /agent/tasks Reef tests (8 → 20): - Tree integration: submit creates user node, node is child of main - Continuation: parentId creates child of assistant, reopens task - Validation: 400 for missing/non-string task - Tree API: node + children, ancestor path, 404s - Task listing: filter by status, task detail with path - State: reflects submitted tasks - REEF_DATA_DIR env var for test isolation Changelog: - Document event tree, per-task pi, conversation continuation - New services: store, cron - Feed UI: threaded feed, branch conversations - Agent tools: reef_task_list/read, reef_store_* - Removed: agent service, branch/merge/loop - Developer experience: biome linter, pre-commit hook 277 tests, 0 failures, 0 lint errors
spawn() throws synchronously if the binary doesn't exist. Wrap in try/catch so the task is marked as error instead of crashing the server. Fixes CI where pi is not installed.
- launchTask(task, taskId, userNode, treeContext) owns pi spawn + tree wiring - failTask(task, taskId, error) shared by onError callback and catch block - Submit handler is now: validate → create node → broadcast → launch → respond
spawn() may not throw synchronously — instead the child emits an
'error' event. Without a listener, this crashes the process.
Add child.on('error') handler that calls failTask.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove agent service
services/agent/deleted — it was the old model (/agent/tasksfor fire-and-forget,/agent/sessionsfor RPC chat).src/reef.tsreplaces both:/agent/*)/reef/*)POST /agent/tasks(fire-and-forgetpi -p)POST /reef/submit(per-task RPC with tree integration)POST /agent/sessions+/message(long-lived session)parentIdnodeId/parentIdCron agent jobs updated:
/agent/tasks→/reef/submitReef tests (8 → 20)
parentIdcreates child of assistant, reopens taskREEF_DATA_DIRenv var prevents test pollutionVersion 0.3.0
Full changelog covers everything since 0.2.0: event tree, per-task pi, conversation continuation, store service, cron service, feed UI, biome linter, removed agent service + branch/merge/loop.
277 tests, 0 failures, 0 lint errors.