Skip to content

fix: scope V2 task recovery to plugin location - #58

Merged
danyel117 merged 1 commit into
mainfrom
fix/issue-57-lazy-task-recovery
Sep 26, 2026
Merged

danyel117 merged 1 commit into
mainfrom
fix/issue-57-lazy-task-recovery

Conversation

@danyel117

Copy link
Copy Markdown
Contributor

Closes #57

Summary

  • reuse the existing V2 session-ownership check before loading restart-recovery transcripts
  • skip goals owned by other directories/workspaces while preserving legacy behavior when the host does not expose a location
  • add regression coverage proving only the local goal transcript is read
  • rebuild the tracked server bundle

Validation

  • bun run lint
  • bun run typecheck
  • bun test — 296 passed, 0 failed
  • bun run build
  • bun run pack:dry-run

Audit

Claude Code CLI with Opus, audit 1: no high or medium findings. Four low observations were noted (diagnostic logging on ownership lookup failure, one harmless post-dispose request window, serial ownership lookups, and two optional test gaps); none block correctness or safety. No correction cycle was required.

AI assistance

Implementation: Codex (GPT-5). Review: Claude Code CLI with Opus.

@danyel117
danyel117 merged commit e7c3560 into main Sep 26, 2026
5 checks passed
@danyel117
danyel117 deleted the fix/issue-57-lazy-task-recovery branch September 26, 2026 06:10
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.

[Bug]: recoverTrackedTasks re-reads every open goal's full transcript on each plugin instance start

1 participant