Skip to content

fix(cli): warn on native Git drift - #122

Merged
freezscholte merged 1 commit into
mainfrom
codex/explain-native-git-drift
Jul 5, 2026
Merged

freezscholte merged 1 commit into
mainfrom
codex/explain-native-git-drift

Conversation

@freezscholte

@freezscholte freezscholte commented Jul 5, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add a best-effort native-backend warning when Forge changed_paths includes paths that are clean in Git
  • surface the warning from both save and propose so stale native-base drift is visible before accept
  • add a regression covering a Git-only commit plus an actual attempt edit

Verification

  • rtk cargo fmt --all --check
  • rtk cargo test --workspace
  • rtk cargo clippy --workspace --all-targets -- -D warnings
  • rtk bash scripts/check-rust-line-count.sh
  • rtk bash scripts/e2e-eval.sh
  • real-binary smoke: native save reports README.md + git_only.txt and warns only for git_only.txt

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@freezscholte
freezscholte merged commit d480e94 into main Jul 5, 2026
1 check passed
@freezscholte
freezscholte deleted the codex/explain-native-git-drift branch July 5, 2026 13:49
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.

1 participant