Skip to content

TEST deletion of AI lines in the next commit#194

Closed
AtnesNess wants to merge 6 commits intogit-ai-project:mainfrom
AtnesNess:e2e/deleting-ai-lines-in-next-commits
Closed

TEST deletion of AI lines in the next commit#194
AtnesNess wants to merge 6 commits intogit-ai-project:mainfrom
AtnesNess:e2e/deleting-ai-lines-in-next-commits

Conversation

@AtnesNess
Copy link
Copy Markdown
Contributor

@AtnesNess AtnesNess commented Nov 6, 2025

#193


Note

Adds regression tests for correct attribution when AI deletes its own lines and introduces detailed checkpoint debug logging.

  • Tests:
    • Add e2e test tests/e2e/user-scenarios.bats verifying stats when AI deletes one of its own lines and both human and AI add lines in the same commit (human_additions: 2, ai_additions: 2, total_ai_deletions: 1).
    • Add Rust test tests/simple_additions.rs::test_ai_deletion_with_human_checkpoint_in_same_commit asserting correct line-by-line blame and stats for the same scenario.
  • Debugging:
    • Add verbose println! debugging in src/commands/checkpoint.rs within get_checkpoint_entry_for_file and make_entry_for_file to log checkpoint kind, author, data source, previous/current content, previous/updated attributions, line attributions, and skip reasons.

Written by Cursor Bugbot for commit c64697a. This will update automatically on new commits. Configure here.

cursor[bot]

This comment was marked as off-topic.

svarlamov added a commit that referenced this pull request Nov 21, 2025
@svarlamov
Copy link
Copy Markdown
Member

Closing this PR as I've added the test in ba35c5c

as part of the work in #228

@svarlamov svarlamov closed this Nov 21, 2025
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.

3 participants