Target-aware Deep Review reviewer scheduling#559
Draft
limityan wants to merge 14 commits intoGCWing:mainfrom
Draft
Target-aware Deep Review reviewer scheduling#559limityan wants to merge 14 commits intoGCWing:mainfrom
limityan wants to merge 14 commits intoGCWing:mainfrom
Conversation
limityan
force-pushed
the
yanzhn/deepreview
branch
3 times, most recently
from
273097a to
9ebb682
Compare
limityan
force-pushed
the
yanzhn/deepreview
branch
2 times, most recently
from
afef859 to
944ff21
Compare
…S variables Extract shared icon size and text styling into CSS variables in _tool-card-common.scss so that dense-command variants (Terminal, Git) only override spacing, not sizing. This aligns Skill and Terminal tool cards on icon size (16px), font size (14px), line-height (1.65), and font-weight (500). - Add --tool-card-icon-size, --tool-card-action-font-size, --tool-card-action-line-height, --tool-card-action-font-weight - Use variables for SVG width/height and compact-card text - Remove size/font overrides from .compact-tool-card-wrapper--dense-command Generated with BitFun Co-Authored-By: BitFun
…le subcommand When the AI model mistakenly includes the subcommand name in the `args` field (e.g. `operation:
- Set overflow: hidden on .explore-region__content by default - Move overflow-y: auto behind .explore-region--has-scroll class - Preserve auto-scroll behavior during streaming Generated with BitFun Co-Authored-By: BitFun
- Add summary rendering in WebSearchCard when results are empty - Add i18n keys for summary availability in en-US, zh-CN, zh-TW - Refactor expandable logic to include summary-only state
limityan
force-pushed
the
yanzhn/deepreview
branch
from
944ff21 to
a386307
Compare
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.
Summary
/DeepReviewtarget parsing, and git commit/range changed-file resolution.GetFileDiff+Read, share the tooling contract between UI and Review Team execution, and surface invalid reviewers asinvalid_toolingskipped reviewers instead of silently dropping them.Verification
pnpm --dir src/web-ui exec vitest run src/shared/services/reviewTeamService.test.ts src/app/scenes/agents/components/subagentEditorUtils.test.tspnpm run type-check:webpnpm run lint:webpnpm --dir src/web-ui run test:runcargo test -p bitfun-core deep_review -- --nocapturegit diff --check HEAD~4..HEAD