Skip to content

feat: T0.2 batch feature implementation - #5

Merged
alejandro-nexcade merged 3 commits into
mainfrom
alejandro/feature-batch-merge
Jun 30, 2026
Merged

feat: T0.2 batch feature implementation#5
alejandro-nexcade merged 3 commits into
mainfrom
alejandro/feature-batch-merge

Conversation

@alejandro-nexcade

Copy link
Copy Markdown
Collaborator

Summary

  • 12 features built in parallel via 8 subagents + 5 WI plan items (DAG view, card actions, state filters, review source, dispatch wiring)
  • 14 new files, 39 modified files, +6991/-1086 lines
  • 277 tests pass, clippy clean (0 warnings), TypeScript strict clean

Core additions

  • Agent stream JSONL parser (11 event types, 17 tests)
  • Skills system for SKILL.md discovery/injection into rework prompts (14 tests)
  • Workflow hooks for gate state transition side effects (11 tests)
  • Worktree isolation for parallel agent runs
  • ReviewSource enum to differentiate authored vs review-requested vs frontier PRs
  • Expanded config (14 fields: API keys, themes, editor, terminal settings)

App additions

  • Real-time agent activity panel with streaming timeline
  • Monaco editor theme switching (7 themes)
  • Embedded PTY shell backend (portable-pty + xterm.js components, UI wiring pending)
  • Command palette (Cmd+K) with navigation, review jump, and action groups
  • Keyboard shortcuts (Cmd+1-5, R, B, comma)
  • State filter chips with per-gate-state counts
  • Context-aware single-action card buttons (label/style per gate state)
  • DAG frontier view replacing flat list + separate Stacks page
  • Inline comments via Monaco view zones (GitHub-style)
  • Open-in-editor button in DiffView file tree
  • Light/dark theme switching with full CSS variable separation
  • "Submit Review" button for review-requested PRs (mirrors to GitHub with confirmation)
  • Dispatch pipeline wiring (request_changes → prompt assembly → agent spawn → hook server → completion reconciliation)
  • Desktop notifications on agent completion

Docs

  • Gated trait documentation with Mermaid state diagram, transition table, error cases

Test plan

  • cargo clippy --all-targets --all-features -- -D warnings clean
  • cargo test --workspace — 277 tests pass
  • npx tsc --noEmit — clean under strict mode
  • Manual: verify light/dark theme toggle in Settings
  • Manual: verify state filter chips filter correctly
  • Manual: verify single action button per card changes label by state
  • Manual: verify DAG tree rendering in Frontier tab
  • Manual: verify open-in-editor icon in DiffView file tree
  • Manual: verify agent panel timer only starts on agent events
  • Manual: verify Cmd+K opens command palette

🤖 Generated with Claude Code

LifeLex and others added 3 commits June 30, 2026 10:03
Implements 12 features via parallel subagents plus 5 WI plan items:

Core:
- Agent stream parser (JSONL event types, 17 tests)
- Skills system (discover/filter/format SKILL.md files, 14 tests)
- Workflow hooks (TransitionEvent/Rules on gate transitions, 11 tests)
- Worktree isolation (prepare/cleanup for agent runs)
- ReviewSource enum (Authored/ReviewRequested/Frontier)
- Expanded config (14 fields: API keys, theme, editor, terminal)

App:
- Agent activity panel with real-time streaming timeline
- Monaco theme switching (7 themes, settings-driven)
- Embedded PTY shell (portable-pty + xterm.js, not yet wired to UI)
- Command palette (Cmd+K, 3 groups)
- Keyboard shortcuts (Cmd+1-5, R, B, comma)
- State filter chips with counts per gate state
- Context-aware card actions (single button per state)
- DAG frontier view (replaces flat list + separate Stacks page)
- Inline comments via Monaco view zones
- Open-in-editor button in file tree
- Light/dark theme switching with proper CSS variable separation
- Submit Review for review-request PRs
- Dispatch wiring (request_changes spawns agent, hook server, completion)
- Desktop notifications on agent completion

Docs:
- Gated trait documentation with Mermaid state diagram

277 tests pass, clippy clean, TypeScript strict clean.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add light theme CSS variables to :root, move dark to .dark selector
- Set class="dark" on <html> as default, toggle on theme change
- Make comment errors visible in DiffView with error banner
- Let addComment throw so inline form stays open on failure
- Fix navigateToDiff to search all review arrays (authored, requested, frontier)
- Fix agent timer starting on mount with no agent running
- Add open_in_editor Tauri command + file tree button
- Theme-aware scrollbar colors

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…o PRs

navigateToDiff now fetches the review from the backend store instead of
stale frontend arrays, ensuring comments and gate state survive round-trips.
All mutation actions (addComment, openReview, requestChanges, refresh) sync
the updated review back into the frontend arrays so card views stay
consistent.

open_in_editor accepts repoSlug/branch and uses ensure_branch_checkout to
clone cross-repo PRs into ~/.cockpit/repos/ worktrees on demand.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@alejandro-nexcade
alejandro-nexcade merged commit 7f02202 into main Jun 30, 2026
1 of 2 checks passed
@alejandro-nexcade
alejandro-nexcade deleted the alejandro/feature-batch-merge branch June 30, 2026 11:59
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