Skip to content

Navigation Menu

Appearance settings

View all features
- BY COMPANY SIZE
  Enterprises
  Small and medium teams
  Startups
  Nonprofits
- BY USE CASE
  App Modernization
  DevSecOps
  DevOps
  CI/CD
  View all use cases
- BY INDUSTRY
  Healthcare
  Financial services
  Manufacturing
  Government
  View all industries
View all solutions
- EXPLORE BY TOPIC
  AI
  Software Development
  DevOps
  Security
  View all topics
- EXPLORE BY TYPE
  Customer stories
  Events & webinars
  Ebooks & reports
  Business insights
  GitHub Skills
- SUPPORT & SERVICES
  Documentation
  Customer support
  Community forum
  Trust center
  Partners
- COMMUNITY
  GitHub SponsorsFund open source developers
- PROGRAMS
  Security Lab
  Maintainer Community
  Accelerator
  Archive Program
- REPOSITORIES
  Topics
  Trending
  Collections
- ENTERPRISE SOLUTIONS
  Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  GitHub Advanced SecurityEnterprise-grade security features
  Copilot for BusinessEnterprise-grade AI features
  Premium SupportEnterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

BloopAI / vibe-kanban Public

Notifications You must be signed in to change notification settings
Fork 1.7k
Star 18k

Code
Issues 183
Pull requests 84
Discussions
Actions
Projects
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Discussions
Actions
Projects
Security
Insights

Add session-level review endpoint with StartReviewDialog (Vibe Kanban)#1849

Merged

LSRCT merged 54 commits intomainBloopAI/vibe-kanban:mainfrom

vk/38f8-add-review-endpoBloopAI/vibe-kanban:vk/38f8-add-review-endpoCopy head branch name to clipboard

Jan 12, 2026

Conversation Commits54 (54)Checks Files changed

Merged

Add session-level review endpoint with StartReviewDialog (Vibe Kanban)#1849
LSRCT merged 54 commits intomainfrom
vk/38f8-add-review-endpo

Commits

Commits on Jan 12, 2026

All checks pass. Let me provide a summary of what was implemented:
LSRCT
committed
Backend changes for the `use_all_workspace_commits` flag are complete. Here's what was implemented:
LSRCT
committed
Frontend UI implementation is complete. Here's a summary of what was added:
LSRCT
committed
Done. Changes made:
LSRCT
committed
Done. Here's a summary of the changes:
LSRCT
committed
Codex review support (vibe-kanban e7996a18)
LSRCT
committed
Done. Here's a summary of the changes:
LSRCT
committed
Use custom review target for codex
LSRCT
committed
Done. Here's a summary of the changes:
LSRCT
committed
Done! Here's a summary of all the frontend changes:
LSRCT
committed
Done. The fix adds the `ReviewRequest` case to `latest_executor_profile_for_session()` in `crates/db/src/models/execution_process.rs:688-690`.
LSRCT
committed
Done. Removed `CommitRange` enum entirely. Changes:
LSRCT
committed
Done. Simplified the review context building:
LSRCT
committed
Done. Addressed all 23 review comments:
LSRCT
committed
Done. Added the review toggle button:
LSRCT
committed
Fix duplicate routes
LSRCT
committed
I've analyzed the problem. Here's my understanding:
LSRCT
committed
Done. Here's a summary of the changes:
LSRCT
committed
Cleanup script changes for workspace 38f8b607-9fdc-43dc-afa3-16e8f5e9cbd3
LSRCT
committed
Removed the unused `find_first_commit_for_repo` function. Everything still compiles.
LSRCT
committed
All comments removed and code still compiles.
LSRCT
committed
Fixed. The changes:
LSRCT
committed
Fixed. Now the workspace is re-fetched after `ensure_container_exists` to get the updated `container_ref`.
LSRCT
committed
Fixed. Now using the `container_ref` returned directly from `ensure_container_exists` instead of re-fetching the workspace.
LSRCT
committed
Fixed the naming:
LSRCT
committed
Cleanup script changes for workspace 38f8b607-9fdc-43dc-afa3-16e8f5e9cbd3
LSRCT
committed
Done. I've added an "Include git context" checkbox to the StartReviewDialog:
LSRCT
committed
I've completed the i18n task. Here's a summary of what was done:
LSRCT
committed
Cleanup script changes for workspace 38f8b607-9fdc-43dc-afa3-16e8f5e9cbd3
LSRCT
committed
All checks pass. I've added a description below the "Include git context" checkbox:
LSRCT
committed
Done. Here's a summary of the changes:
LSRCT
committed
Types
LSRCT
committed
Removed the unused `useStartReview` hook and its export from `hooks/index.ts`.
LSRCT
committed
All checks pass. Here's a summary of the changes made:
LSRCT
committed
All checks pass. Done. Here's a summary of the additional change:
LSRCT
committed
All checks pass. Here's a summary of the changes:
LSRCT
committed
All checks pass. Now toolbar actions work identically to navbar actions:
LSRCT
committed
Cleanup script changes for workspace 38f8b607-9fdc-43dc-afa3-16e8f5e9cbd3
LSRCT
committed
Done. The "Start Review" action with the highlighter icon now appears in both the navbar (left side) and the chat input toolbar.
LSRCT
committed
Cleanup script changes for workspace 38f8b607-9fdc-43dc-afa3-16e8f5e9cbd3
LSRCT
committed
Done. Removed `StartReview` from the navbar and added `ToggleChangesMode` to the conversation input toolbar alongside `StartReview`.
LSRCT
committed
Done.
LSRCT
committed
Removed the review-specific branch so the handler is generic, matching other containers. The toolbar action now just respects `requiresTarget`.
LSRCT
committed
Aligned the chatbox to use the shared visibility system, like the navbar, so action availability is handled by the same logic and execution stays generic.
LSRCT
committed
Cleanup script changes for workspace 38f8b607-9fdc-43dc-afa3-16e8f5e9cbd3
LSRCT
committed
Updated the Start Review dialog so it derives the active session and a default executor profile from workspace context + config, which restores preselection and enables the “new session” toggle whe…
LSRCT
committed
Fixed the lint issue and clippy qa-mode compile warning, then reran the requested commands. The Start Review dialog now uses `resolvedSessionId` consistently in its callback, and review log normali…
LSRCT
committed
Renamed the match binding to `_request` so we don’t need the no-op line, and removed the comment you flagged.
LSRCT
committed
Split the `ReviewRequest` arm by cfg so qa-mode uses `_request` and non-qa uses `request` normally—no dummy line needed.
LSRCT
committed
Done. The fix adds `ReviewRequest` to the match statement that creates `CodingAgentTurn` records. Now when a review is started, a `coding_agent_turn` row will be created with the review prompt, and…
LSRCT
committed
Done. Now if a new `ExecutorActionType` variant is added, the compiler will error on this match and force the developer to decide whether it should create a `CodingAgentTurn` record.
LSRCT
committed
Adjusted `sessionsApi.startReview` to return the unwrapped payload like other session calls, and aligned the `handleApiResponse` generic so callers get an `ExecutionProcess` instead of a misleading…
LSRCT
committed
All checks pass. Summary of changes:
LSRCT
committed
Done. Now when a new session is created, the dialog will call `selectSession(targetSessionId)` to switch to the newly created session before closing.
LSRCT
committed

Footer

© 2026 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.