Skip to content

fix: everything AICodeKing reported#320

Merged
t3dotgg merged 4 commits intomainfrom
feature/git-aware-checkpoints-diff-ux
Mar 7, 2026
Merged

fix: everything AICodeKing reported#320
t3dotgg merged 4 commits intomainfrom
feature/git-aware-checkpoints-diff-ux

Conversation

@t3dotgg
Copy link
Member

@t3dotgg t3dotgg commented Mar 7, 2026

  • Guard non-git projects and improve send and diff UX
  • Skip checkpoint capture/revert and turn diff ingestion for non-git workspaces
  • Normalize project workspace roots and return clearer request/error messages
  • Show "No Git" state in chat/diff UI and enrich work log with commands/files
  • Track send start time so working status shows elapsed duration consistently

Note

Enforce git-repo checks across server and web, gating checkpoint diffs and validating workspaceRoot in wsServer.ts

Add isGitRepository and gate checkpoint capture, diff completion, and UI diff views on git status; validate and expand workspaceRoot on project create/update; surface concise errors; add working timer and richer work log details in chat.

📍Where to Start

Start with server-side gating and path validation in createServer within apps/server/src/wsServer.ts, then follow git-repo checks in apps/server/src/orchestration/Layers/ProviderRuntimeIngestion.ts and apps/server/src/orchestration/Layers/CheckpointReactor.ts.

Macroscope summarized 457b73e.

- Skip checkpoint capture/revert and turn diff ingestion for non-git workspaces
- Normalize project workspace roots and return clearer request/error messages
- Show "No Git" state in chat/diff UI and enrich work log with commands/files
- Track send start time so working status shows elapsed duration consistently
@vercel
Copy link

vercel bot commented Mar 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
t3code-marketing Ready Ready Preview, Comment Mar 7, 2026 10:00am

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Mar 7, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 1cc3a11b-1e4f-431a-88b6-1406541a89d4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • ✅ Review completed - (🔄 Check again to review again)
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/git-aware-checkpoints-diff-ux

Comment @coderabbitai help to get the list of available commands and usage tips.

- add `formatWorkingTimer` for human-friendly durations in ChatView
- show active "Working for" time as `0s`, `1m 5s`, or `2h 3m` instead of ms
@UtkarshUsername
Copy link

Seems to also be the fix for #236

- derive active work start from unsettled turn state instead of always latest turn
- update chat view to use shared deriveActiveWorkStartedAt helper
- make server/ws tests create temporary workspace roots with .git for project creation
@t3dotgg t3dotgg merged commit 0837c8b into main Mar 7, 2026
5 checks passed
@t3dotgg t3dotgg deleted the feature/git-aware-checkpoints-diff-ux branch March 7, 2026 10:06
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