Skip to content

feat(discovery): pursue bounded follow-up research#60

Merged
drewstone merged 1 commit into
mainfrom
feat/discovery-loop
Jul 12, 2026
Merged

feat(discovery): pursue bounded follow-up research#60
drewstone merged 1 commit into
mainfrom
feat/discovery-loop

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

Problem

KnowledgeDiscoveryWorker can return follow-up tasks, but the package only dispatches one batch. Products must otherwise rebuild recursive scheduling, limits, cancellation accounting, and task identity checks.

Solution

Add runDiscoveryLoop, a domain-neutral breadth-first runner that dispatches follow-ups with explicit round/task limits, retains pending work when stopped, returns partial work on cancellation, prevents cycles, and rejects incomplete dispatcher output. Search providers, agents, source verification, and candidate schemas remain product-owned worker concerns.

Verification

  • pnpm typecheck
  • pnpm lint
  • pnpm test (218/218 passing; 10 network tests intentionally skipped)
  • pnpm build
  • Independent Opus review: APPROVE; cancellation-loss finding fixed and covered

@drewstone drewstone marked this pull request as ready for review July 12, 2026 02:54

@tangletools tangletools left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Auto-approved drewstone PR — e5d19ae0

This PR was opened by the trusted drewstone account.
The full PR reviewer audit still runs separately and will publish findings if it detects issues.

tangletools · auto-approval · reason: drewstone_author · 2026-07-12T02:54:08Z

@drewstone drewstone merged commit 5e90f38 into main Jul 12, 2026
1 check passed
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.

2 participants