Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

Target-aware Deep Review reviewer scheduling#559

Draft
limityan wants to merge 2 commits intoGCWing:mainfrom
limityan:yanzhn/deepreview
Draft

Target-aware Deep Review reviewer scheduling#559
limityan wants to merge 2 commits intoGCWing:mainfrom
limityan:yanzhn/deepreview

Conversation

@limityan
Copy link
Copy Markdown
Collaborator

Summary

  • Add target-aware Deep Review manifest construction, including frontend/domain classification, explicit /DeepReview target parsing, and git commit/range changed-file resolution.
  • Persist the Deep Review run manifest through Flow Chat session metadata so continuation and child sessions keep the original reviewer scope.
  • Enforce the persisted manifest in the backend Task pipeline so only active reviewers from the run manifest can be launched.
  • Add custom review subagent tooling validation: require GetFileDiff + Read, share the tooling contract between UI and Review Team execution, and surface invalid reviewers as invalid_tooling skipped reviewers instead of silently dropping them.
  • Add token-budget aware extra reviewer handling and skipped-reviewer reporting for disabled, unavailable, not-applicable, budget-limited, and invalid-tooling cases.

Verification

  • pnpm --dir src/web-ui exec vitest run src/shared/services/reviewTeamService.test.ts src/app/scenes/agents/components/subagentEditorUtils.test.ts
  • pnpm run type-check:web
  • pnpm run lint:web
  • pnpm --dir src/web-ui run test:run
  • cargo test -p bitfun-core deep_review -- --nocapture
  • git diff --check HEAD~4..HEAD

@limityan limityan force-pushed the yanzhn/deepreview branch 6 times, most recently from 944ff21 to a386307 Compare April 29, 2026 08:22
@limityan limityan marked this pull request as draft April 29, 2026 15:44
@limityan limityan force-pushed the yanzhn/deepreview branch from a386307 to aa704bf Compare April 30, 2026 09:07
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.

1 participant