Skip to content

fix(review): isolate review sub-task from getRunningTasks() #39

Description

@robotlearning123

Context

PR #37 added cross-agent diff review. The review agent is spawned via this.run(reviewTask, "/tmp"), which inserts the review task into _runningTasks. During review (~60s), API callers see a ghost task in getRunningTasks() that can't be resolved by getTask().

Options

  1. Add a _isReviewTask flag and filter in getRunningTasks()
  2. Use a separate AgentRunner instance for reviews
  3. Accept the brief visibility (document it)

Impact

Low — the ghost task disappears after review completes. But it's confusing for dashboard consumers.

Ref: #37

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions