Unify TODO runtime and workflow tooling - #86
Open
moshloop wants to merge 259 commits into
Open
Conversation
…/restart Add TCP port detection for supervised processes using lsof, enabling the UI and CLI to display which ports a process is listening on. Implement a port-watching goroutine that polls for listening ports after process start and records them in the process state. Enhance start/restart UX by: - Showing per-process readiness progress with live task updates (starting → running → listening on :PORT) - Gating processes that previously bound a port as "starting" until the port is re-detected on restart - Displaying listening ports as clickable localhost links in the UI - Polling process status faster (1s vs 3s) while transitions are in flight Extract common utilities (ProcessAlive, TailFile) to a new utils package for reuse across service and procfile modules. Refs: port detection, process lifecycle visibility
Add profile-based process selection and resource monitoring: - Procfile now supports YAML format with per-process configuration (command, default, autoRestart, cpu, mem, profiles, env, maxRestarts) - Processes can declare profiles to auto-start only when active; default:false makes them start on-demand - Supervisor tracks CPU%, memory RSS, and open file descriptors per process - Start/Restart commands accept --profile flag; daemon reuses profile on restart - Control socket allows starting named processes on already-running daemon - Removed terminateGroup function; clicky's SupervisedProcess handles process lifecycle - Refactored supervisor to delegate per-process supervision to clicky instead of custom runLoop - State includes active profile and available profiles from Procfile BREAKING CHANGE: Procfile format changed from simple 'name: command' lines to YAML; Run/Start/Restart functions now require profile parameter
…ment with profiles and resource limits Move author/bot filtering from server-side search config to client-side facet filters, allowing the UI to control bot visibility via @Bots chip. The daemon now fetches all authors and learns bot accounts from results, excluding them at the source on subsequent fetches unless @Bots is active. Add profile support to Procfile supervision: entries with 'profiles' auto-start only when the active profile matches. Replace per-process config overrides in .gavel.yaml with per-process settings in the Procfile itself (command, default, autoRestart, cpu, mem, profiles, env, maxRestarts). Add resource limits (CPU %, memory cap) to both global defaults and per-process config. Refactor tri-state facet filters (include/exclude/neutral) from Set-based to record-based representation, enabling exclude semantics in the UI. Add ProcessManager dropdown showing all supervised workspaces with live metrics (CPU, memory, open files), process trees, log previews, and profile selection. Add theme toggle and search bar to app header. Update config schema and examples to reflect new Procfile structure. Remove SearchControls and RepoSelector components; integrate their functionality into FilterBar and AppShell. BREAKING CHANGE: SearchConfig no longer carries author/any/bots fields; author filtering is now purely client-side. ProcfileConfig.restartPolicy and .processes are removed; use .autoRestart and per-process Procfile entries instead.
Replace npm with pnpm as the package manager across all UI projects (testrunner/ui, pr/ui) and CI workflows. Enable corepack for consistent pnpm version management instead of manual installation. Add pnpm-workspace.yaml files to support linking local @flanksource/clicky-ui sibling checkouts while resolving from npm registry in CI. Update all npm commands to pnpm equivalents and remove npm cache configuration from setup-node actions. Add packageManager field to package.json files and dedupe preact in testrunner/ui vite config to handle workspace resolution correctly. Update Makefile and Taskfile to use corepack pnpm instead of npm. Delete package-lock.json files as they are no longer needed. BREAKING CHANGE: npm is no longer used; pnpm@10.33.0 is now required via corepack
Exclude the compiled PR UI bundle from version control to keep the repository clean and prevent build artifacts from being tracked.
Add /api/proc/favicon endpoint to fetch favicons from localhost services with project+port validation. Consolidate StatusIndicator and SettingsButton into a unified dropdown menu that displays health status, GitHub rate limits, sync controls, and version information. Add process uptime tracking and visualize CPU/memory metrics with charts. Extract ProcessPortLink component to display favicons alongside port links. Changes: - Backend: New handleProcFavicon handler with port discovery validation - Frontend: Merge Summary and SettingsButton functionality into StatusIndicator - Add @tanstack/react-query dependency for future data fetching - Add uptime calculation and CPU/memory visualization components - Improve process table with additional metrics and visual indicators
Replace the runtime iconify-icon custom element with a new GavelIcon component that uses pre-built SVG icons from clicky-ui. This eliminates the external CDN dependency and improves performance. Add support for menubar and processes pages with dedicated views. Implement menubar external link handling via webkit bridge for macOS integration. Refactor process metrics display to show workspace-level CPU and memory aggregates in headers. Add process favicon display and reorganize process table columns. Update pnpm workspace configuration to use catalog mode and add react-grab for development debugging. Refs: Removal of external iconify-icon script dependency, addition of offline icon rendering capability
…nd macOS menubar via Wails Enhance AI fix output with token usage tracking and context window percentage display. Replace caseymrm/menuet with Wails v3 for native macOS menubar integration with better lifecycle management. Key changes: - NewStderrRenderer now accepts model and contextWindow parameters to display `[model X%]` prefix tracking token consumption - Add isTokenLimitError detection to automatically chunk commits by directory when AI analysis exceeds context limits - Migrate menubar from menuet to Wails v3 with proper window management, pointer events, and hide controller - Add gavel test ansi command for capturing and analyzing PTY output with width-aware ANSI settling - Extend ANSI settle logic to support terminal width wrapping and detect wrap-induced redraw bugs - Update go-git and related dependencies Refs: token limit handling, menubar UX improvements
Refactor process startup/restart/stop to track readiness outcomes and stream logs live. Key changes: - Extract procTracker state machine to classify process readiness (ready/warn/failed) across status samples, enabling callers to exit non-zero on startup failures - Add Stream() to tail process logs with per-process prefixes until context cancellation, used by restart/stop operations - Add streamUntilReady() and stopAndStream() to multiplex log streaming with readiness polling - renderProcReadiness() now returns error on startup failures instead of silently succeeding - Add -f/--follow flag to proc restart/stop to keep streaming after processes settle - Add procMetricsLoop() to sample CPU/memory timeseries for process dashboard gauges - Update ProcessTable.tsx to poll recorded metrics from backend instead of one-shot values - Supervisor.Wait() now persists terminal state.json on self-exit (daemon crash) vs clearing on explicit stop - Add comprehensive tests for procTracker, Stream, and metrics endpoint Breaking change: renderProcReadiness signature changed to return error; callers must handle startup failures.
Introduce a Provider interface to abstract TODO storage, enabling support for multiple backends (file-based and Grite issue tracker). Implement FileProvider for local .todos directory and GriteProvider for Grite integration with full CRUD operations, state management, and event history tracking. Key changes: - Add Provider interface with List, Get, UpdateState, UpdateLatestFailure, SaveAttempt methods - Implement FileProvider wrapping existing file-based TODO discovery and persistence - Implement GriteProvider with Grite CLI integration for issue management - Add --provider flag to select between 'grite' (default) and 'todos' backends - Update TODOExecutor to accept and use Provider for state persistence - Add ProviderEvent type to track issue history from external providers - Extend TODO type with ID, ShortID, Provider, ProviderState, Labels, ProviderEvents fields - Extract common TODO filtering logic into filterTODOsByArgs helper - Update fixture parser to support parsing markdown content from non-file sources BREAKING CHANGE: TODOExecutor constructor now accepts optional Provider parameter; executeGroups and executeSingleTODOs functions now require Provider argument
Implement a new todos tab in the dashboard with workspace-scoped todo listing, creation, and management. Add RESTful project CRUD endpoints (/api/projects/{name}) with proper HTTP semantics (201 for create, 409 for conflict, 204 for delete). Introduce todo backend resolution (auto-detect .todos files or Grite), git change counting for workspaces, and OpenAPI/Clicky integration for the projects entity. Refactor projects handler to separate concerns: entity logic in projects_entity.go, Clicky/OpenAPI support in projects_clicky.go, and shared CRUD functions in projects.go. Update frontend routing to support /todos and /activity as top-level SPA tabs with proper history management.
…ject management CLI Implement a multi-faceted enhancement to the todos and projects subsystems: **Grite Caching**: Add GriteCacheStore interface and CachedGriteProvider to persist grite issue projections in the gavel DB with TTL-guarded incremental syncs. Writes pass through to grite (source of truth) and force a sync so reads reflect changes immediately. **Grouping by Repository**: Extend GroupTODOs to support GroupByRepo strategy that uses git root detection to group TODOs by their repository, with fallback to workDir. **CMux Execution Mode**: Add --mode flag (inline|cmux) to todos run command. CMux mode generates dispatch plans for multi-agent execution with --effort directive (low|medium|high) to control reasoning depth. Dry-run displays cmux commands and prompts. **Model Override**: Add --model flag to override LLM model for TODO execution, taking precedence over frontmatter config. **Projects Management**: New projects CLI command with CRUD subcommands (list, get, add, update, delete) to manage workspace projects stored in ~/.config/gavel/projects.json. **Provider Enhancements**: Add Create and Delete methods to Provider interface. Implement for both FileProvider and GriteProvider with proper state management. Breaking change: Provider interface now requires Create and Delete method implementations.
Remove todo documentation files for PR #81 that tracked CI job failures: - lint-go-mod-tidy-check.md: go mod tidy issues resolved - lint-golangci-lint.md: errcheck linting issues resolved - test-test-ubuntu-latest-go-1-25.md: test build failures resolved These tracking files are no longer needed as the underlying issues have been fixed.
Add cmux-based TODO execution alongside Claude, enabling multi-agent dispatch through cmux workspaces. Implement client library for cmux CLI interaction with workspace management and session polling. Add priority field support to todo updates across all providers (Grite, file-based). Extend PATCH API to accept optional priority changes independently of status. Implement client-side filtering UI for todo statuses with persistent localStorage preferences. Hide completed todos by default while preserving full counts in headers. Refactor executor selection into pluggable factory pattern supporting both Claude and cmux backends. Extract effort directives and agent resolution into shared cmux package utilities. Breaks: cmux mode no longer returns "not implemented" error; it now executes via cmux workspaces when available.
…ecutor Implement todo transfer capability allowing users to move todos between project workspaces via API and CLI. Refactor cmux executor to use workspace/surface-based interaction instead of session polling, enabling more reliable agent communication with retry logic and screen stabilization detection. Key changes: - Add Transfer() function to move todos between providers (file/Grite) - Implement /api/todos/transfer endpoint with validation - Add `gavel todos transfer` CLI command - Refactor CmuxExecutor to use EnsureWorkspace, NewSurface, SendSurface, ReadScreen - Add screen polling with backoff and stability detection - Add send retry logic with configurable attempts - Update dashboard TodoView to support deep-linking and todo selection in URL - Remove deprecated SessionStore-based waiting mechanism - Add AgentCommand and AgentWorkspaceName helpers - Extend cmux Client with workspace/surface management methods Breaking change: CmuxExecutor no longer uses SessionStore; callers must remove Store field from CmuxExecutorConfig.
… run support Introduce two new TODO statuses (draft and verified) throughout the system: - Draft status for TODOs being prepared but not ready to run - Verified status for completed TODOs awaiting closure Add comprehensive TODO creation support: - New CLI command `todos create` with title, body, priority, and status flags - New API endpoint `/api/todos/new` supporting JSON, form, and multipart payloads - Automatic attachment handling and summary generation - AutoSave mode that defaults to pending status instead of draft Implement batch TODO execution: - Support running multiple TODOs in a single agent session via `/api/todos/run` - Multi-select UI in workspace groups with checkbox controls - Advanced run options dialog for model, effort, timeout, and cost limits - Dry-run validation before actual execution Enhance TODO listing: - Hide completed TODOs by default; add `--all` flag to show them - Status filter overrides the default hide behavior - Add `todo` alias for `todos` command Improve Grite provider: - Support short ID prefix resolution for todo references - Detect ambiguous prefixes and report errors - Capture label details in provider events - Add draft and verified status label mappings Refactor UI components: - Extract event rendering into dedicated TodoTimeline component - Add run controls to both detail pane and list view - Update status and count displays for new statuses Update documentation and validation to reflect all seven statuses.
…r proc status Implement memory management improvements across the dashboard: - Add menubar blank controller to navigate hidden webview to about:blank after 5min idle, reclaiming unbounded DOM/JS heap growth that caused 26GB bloat - Replace proc status polling with SSE stream (handleProcStatusStream) that adapts cadence based on process transitions and omits resource churn (cpu/mem/tree) to enable efficient change detection - Gate all background work (SSE streams, pollers, re-render ticks) on document visibility so hidden windows pause fetching and re-rendering - Remove client-side peak tracking for CPU/memory gauges; let TimeseriesCoreBars self-size from live metric series instead - Fetch process tree on-demand in expanded rows rather than streaming it, reducing SSE payload churn - Add useDocumentVisible hook to pause activity feed and proc status streams when tab/window is hidden - Fix react-grab dev overlay gate to use import.meta.env.DEV instead of !CI, preventing production bundle bloat - Add session log tailing for Claude cmux executor to track progress via structured events instead of screen-idle detection These changes eliminate the menubar's constant re-render loop and unbounded memory growth while improving responsiveness through adaptive streaming.
Implement comprehensive session management for TODO agent execution: - Track Claude session IDs via session:<id> labels on issues, enabling runs to be resumed with prior conversation context - Add Plan and Resume options to cmux executor for plan-only mode and session continuation - Implement SessionStatsCache to track live in-progress sessions and compute token usage/cost from session logs - Add session stream and stats endpoints for dashboard to follow agent activity live - Refactor TodoView into composable AppShell-integrated components (TodoBodyHeader, TodoBodyActions, TodoFilterToolbar, TodoWorkspaceList, TodoDetailPane) - Add TodoBucketGroup and grouping/density picker UI for organizing todos by severity/age - Persist session IDs immediately before agent launch so interrupted runs remain resumable - Collapse adjacent label changes in provider events for cleaner timeline display - Omit transcripts from issue comments; instead reference session logs via recorded session IDs The session id is resolved up-front and returned to clients so the dashboard can follow the log live. Resume runs reuse prior session context; fresh runs reference prior sessions in their prompt for history. Token usage and cost are computed from session logs and cached by mtime.
…todo session tracking Replace app-level tick state with a shared useNow() hook that allows leaf components to subscribe to a global clock, reducing unnecessary parent re-renders. Timestamps now refresh themselves via RelativeTime component instead of polling the entire tree. Add comprehensive todo session support: track agent runs with live session logs via SSE, display elapsed time and token usage, support resuming prior sessions, and add session-specific UI in the detail pane. Introduce todo list view preferences (density: comfortable/compact, grouping: workspace/severity/age) persisted to localStorage. Add TodoNewPage for creating todos via deep links with pre-filled fields. Enhance TodoTimeline with sorting and improved event rendering. Add new icon mappings (circle-outline, history, list-flat, rows) and refactor job duration rendering in WorkflowView to use the shared clock pattern. Breaking change: timeAgo() and timeAgoShort() moved from utils to be used internally by RelativeTime component; callers should use <RelativeTime/> instead.
Implement hot-module-reload support for the PR UI via `--dev` flag that spawns and reverse-proxies to a Vite dev server. Add todo session tracking with resume capability, allowing agents to continue prior conversations. Introduce `gavel serve` command as a dedicated dashboard entry point. Add session stats and streaming endpoints for monitoring agent execution. Refactor relative timestamp rendering to use a shared clock subscription instead of full-app ticks, reducing main-thread cost in the menubar. Key changes: - New pr_dev.go: Vite dev server lifecycle management (spawn, health check, port detection) - New devproxy.go: Reverse proxy handler for dev mode with export route bypass - New todos_commit.go: Post-execution commit pipeline for agent changes - New serve_dashboard.go: Dedicated `gavel serve` command - Updated todos flags: --resume, --commit, --dev, --dev-dir - Updated vite.config.ts: Dev server config (port 5273, HMR port 24778), conditional NODE_ENV - Updated App.tsx: Refactored todo views, new TodoNewPage, useNow() clock subscription - Updated types.ts: SessionStats, TodoSessionEvent, TodoDensity, TodoGroupBy - New useNow.ts: Shared 1s clock for relative timestamps with visibility gating
…management Implement responsive mobile detection using media queries to fall back to compact menubar layout on narrow screens (<768px). Extract copy feedback state machine into reusable useCopyFeedback hook to reduce duplication across components. Add useTimeoutFlash hook for transient state with automatic cleanup. Introduce mobile-aware route reconciliation logic that respects menubar layout for both native webview and mobile viewports. New features: - useIsMobile hook for viewport-width tracking - useCopyFeedback hook consolidating copy state management - useTimeoutFlash hook for auto-reverting transient values - ReactGrabHelp component for React Grab integration - TodoEditForm and TodoCommentBox components for todo editing - Mobile layout fallback to menubar UI on narrow screens - Session state badge showing agent progress (thinking/working/ask/completed) - Auto-commit toggle in advanced todo run options Refactoring: - Extract copy feedback logic from App.tsx into dedicated hook - Consolidate session timer display into InProgressBadge component - Improve TodoSession styling with terminal-like appearance - Format ProcessTable with consistent quote style Fixes: - Clear pending timers on component unmount to prevent memory leaks - Proper scroll-following behavior in session transcript - Session state persistence across todo switches
Extract the copy state management and timer logic from App and DetailPanel components into a reusable useCopyFeedback hook. This eliminates code duplication and centralizes the copy-button state machine logic (idle → copying → copied/error → idle) with automatic timer cleanup on unmount to prevent memory leaks.
…proved error handling
Enhance the agent execution pipeline with several improvements:
1. **Prompt Preview**: Add PreviewInstruction() to render the exact prompt dispatched to agents without launching, enabling dashboard preview before runs start. Inline small prompts (<10KB) and truncate large ones with file references.
2. **Multi-TODO Numbering**: Number TODO items in group prompts ("1. Fix auth", "2. Fix db") so agents address each in sequence. Single TODOs remain unnumbered.
3. **Session State Tracking**: Derive high-level agent states (thinking/working/ask/completed) from session log events and expose via SessionStats.State for dashboard display.
4. **Improved Error Handling**: Fail runs loudly when claude session logs never appear instead of silently falling back to screen-idle detection, which masks broken agent launches.
5. **Edit and Comment APIs**: Add Provider.Edit() and Provider.Comment() methods to update TODO titles/bodies and append comments, with file and grite implementations.
6. **Auto-Commit Helper**: Add ShouldCommitAfter() to determine when post-run commits should execute based on result success and executor type.
Refs: Improves agent observability and dashboard UX for multi-TODO runs.
Integrate React Grab bookmarklet for capturing UI elements into todos, add todo group-by menu to tab strip, move new-todo button to action header, enable inline todo editing (title/body/comments), add run prompt preview in advanced dialog, and support auto-commit toggle for runs. Key changes: - Embed React Grab plugin script and install page as assets - Serve plugin with origin substitution for cross-origin injection - Move TodoGroupByPicker from filter toolbar to dedicated menu in tab strip - Extract TodoNewButton to action header, leaving only counts in body actions - Support title/body/comment edits in PATCH /api/todos/item - Add /api/todos/run/preview endpoint to preview prompts before execution - Thread commit option through run payload (defaults to true) - Auto-commit after runs via gavel commit pipeline - Wire status badges in TodoCountsBar as filter pills - Pass toggleStatus callback through workspace group for filter integration - Constrain html/body/#root height to fix AppShell scrolling Refs #123
….0.0.0 Change the default interface binding for UI servers (PR dashboard, bench UI, lint UI, test UI) from localhost to 0.0.0.0 to expose dashboards on the LAN by default. Users can now explicitly set localhost to restrict access to the local machine. Updates: - Default addr flag to 0.0.0.0 across bench, lint, pr list, and test commands - Add --addr flag to pr list command to control UI server binding - Update bindUIListener() to accept host parameter for flexible binding - Add tests for bindUIListener() to verify wildcard and loopback binding - Refactor serve_dashboard.go with applyServeDefaults() helper - Add todos edit/comment/reopen commands with body file support - Simplify todos_commit.go by delegating to commitpkg.RunAfterAgent() BREAKING CHANGE: UI servers now bind to 0.0.0.0 by default instead of localhost, making them accessible on the LAN. Set --addr=localhost to restore previous behavior.
Add RunAfterAgent function to stage and commit changes after TODO runs, supporting both CLI and dashboard auto-commit workflows. Refactor stageFiles to respect .gavel.yaml commit.gitignore rules when staging untracked files, fixing the bug where `gavel commit -A` would stage ignored files via `git add -A`. Split staging into separate git add -u (tracked changes) and custom untracked filtering (respecting both .gitignore and .gavel.yaml rules). Add helper functions gitAddUpdate, untrackedFiles, and addUntracked to handle staging with proper ignore semantics. Embedded git repositories are now safely skipped without aborting the entire stage operation. Also includes: - ScanFreePort now accepts configurable host parameter - useTimeoutFlash React hook for transient state management - NumberTicker animation cleanup improvements - Comprehensive test coverage for staging scenarios
Change the default value of the Addr field from 'localhost' to '0.0.0.0' to expose the UI HTTP server on all network interfaces by default. Update the documentation to clarify that 0.0.0.0 is the default and users can set localhost to restrict access to the local machine. BREAKING CHANGE: The --ui HTTP server now binds to all interfaces (0.0.0.0) by default instead of localhost only.
Add SVG app icon, PNG icon variants (192x512), Apple touch icon, and web manifest configuration for the gavel PR dashboard. Enables PWA installation support across iOS and Android platforms with proper icon scaling and theming.
Enhance the todos interface with several improvements: - Add TodoGroupByMenu and TodoFilterMenu components to support grouping by severity/age and filtering by status in the menubar todos tab - Refactor TodoCompose into separate TodoTitleEditor and TodoBodyEditor components with independent edit states - Add inline edit pencils to TodoDetail for title and body, allowing users to edit without a full form modal - Support defaultDir prop in CreateTodoDialog to preselect the current workspace when creating a new todo - Add attachment support to TodoNewPage with screenshot preview display and multipart form submission - Enhance TodoSession and TodoSessionTimer to display error events with HTTP status codes and context compaction metrics - Update MenubarTodos to include grouping/filtering controls and a New button in a compact tab strip - Improve TodoSection header layout to support action controls (edit pencils) outside the toggle button These changes improve the UX by allowing todos to be created, grouped, filtered, and edited without leaving the current view.
Implement Progressive Web App capabilities with home-screen installation support, including manifest and icons. Add screenshot capture to React Grab plugin using getDisplayMedia and Region Capture APIs. Implement attachment persistence and serving with security validation. Fix menu-bar URL resolution to use loopback for WKWebView App Transport Security compliance. Add session error event streaming and improve todo attachment embedding with inline image rendering. BREAKING CHANGE: Attachment summaries now include served URLs and image flags instead of just metadata; attachment bodies embed images inline as markdown rather than plain links.
Download and merge gavel artifacts from pull requests into reusable snapshots. Expose reproduce-locally commands in CLI, terminal, and PR web results so developers can rerun only the failed tests or linters. Claude-Session-Id: cfccd41f-6c5f-437e-a3e8-14382bd4d683
Use all rendered failure signals for PR status exit codes, preventing false-green results when rollups omit artifact or job failures. Drain shutdown hooks before exiting and return success after a completed --ai-fix workflow. Claude-Session-Id: d02a751b-65e2-4b80-b55c-b4d1ec80a1f2
Prefer open PRs when resolving the current branch so checks do not attach to stale merged work. Track physical terminal rows and clear live frames before final output to prevent duplicated or smeared reports. Improve status rendering so repeated workflows, skipped jobs, and closed-PR notices remain readable and actionable. Claude-Session-Id: a6483152-0156-4e23-9216-0d1e51a65ac7
…tion Normalize artifact test paths across shard work directories so merged PR snapshots retain valid repository-relative locations. Fail non-dry runs that execute no tests and report when --failed packages do not match discovery, preventing silent successful no-ops. BREAKING CHANGE: non-dry runs with no executed tests now return ErrNoTestsExecuted Claude-Session-Id: 019feee2-d80b-7870-be04-0dadab43a54b
Add durable GitHub issue links so TODOs can be published, updated safely, and opened from the dashboard while preventing accidental duplicates. Expose linked issue metadata in list and detail responses, and provide status, priority, activity, and link-state filtering with persisted facet selections. Claude-Session-Id: adc45b9a-111a-49ce-a3c7-669f30ea267c
Reuse the dashboard toolbar so menubar todos support the same grouping, sorting, status, priority, issue, and time filters. Replace the narrower menubar-only controls with one responsive toolbar while preserving todo creation and filtering behavior. Claude-Session-Id: adc45b9a-111a-49ce-a3c7-669f30ea267c
Add CLI and dashboard workflows for publishing TODOs as GitHub issues, updating existing links, and recording external aliases. Reuse portable Markdown and rewrite attachment links against a configured absolute origin for external rendering.
Enable TODOs without a selected plan to receive durable markdown and return to review while preserving immutable revision history. Reuse deterministic sessions and keep rejected plans from being executed accidentally. BREAKING CHANGE: rejected plans remain selected with a rejected approval state and require revision or approval before execution
Persist lifecycle notices so TODO transcripts and notifications explain changes made between execution turns, including runs that end with errors
Claude-Session-Id: e5ae4b2b-811e-4d6b-8204-63a7bc4cd633
…ess startup. Introduce distinct timeout budgets for compilation vs other startup stages. Compilation (go build, bundler passes) legitimately requires minutes; other stages now budget 180s. The process tracker now maintains a per-stage clock that resets when transitioning stages, ensuring time spent compiling does not consume budget for subsequent startup phases. Updates timing logic and adds comprehensive test coverage for multi-stage timeouts and stage transitions.: Support per-stage timeouts for process startup Claude-Session-Id: 63589e85-ee7d-4f76-905f-ed59ce16a50e
Claude-Session-Id: 63589e85-ee7d-4f76-905f-ed59ce16a50e
The "backend" naming was imprecise about what it represented: the execution mechanism (api, agent, cli, cmux). Rename to "mode" and update GetBackend() → GetRuntime() throughout to clarify this concept. This improves code readability and aligns terminology with documentation and domain concepts across ai, pr/ui, todos, and commit packages.
…mode from backend Executor now receives models fully resolved with capabilities and mode, not just names. Execution mode is specified explicitly rather than inferred from backend, enabling cleaner configuration layering and better provenance tracking.
|
Important Review skippedToo many files! This PR contains 1260 files, which is 1160 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (12)
📒 Files selected for processing (1260)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
…ection - Discover nested project roots while pruning node_modules and nested git repositories - Skip linting nested projects when their ancestors are already scheduled, eliminating redundant runs - Add workspace-local binary resolution for oxlint with fallback to PATH - Update TypeScript linter to use SolutionBuilder for project references without writing artifacts - Fix logging format strings in ruff and runner
…atterns - Fix logger.Errorf format verbs: %w is only valid for fmt.Errorf wrapping, not for general error logging; use %v instead - Add missing o.Tags parameter to goPreBuildArgs function call - Simplify field access by accessing embedded/direct fields instead of navigating through parent structs (countingConn.Conn → Conn) - Use capturePolicy conversion constructor instead of manual struct init - Fix string trim check on incorrect field (req.Model.Name → req.Name) - Remove unused httpx import
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
2 participants
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.
What
Notes
0.0.0.0default UI binding.