Skip to content

Add git-ai author config override#1555

Merged
svarlamov merged 1 commit into
mainfrom
feat/author-config
Jun 14, 2026
Merged

Add git-ai author config override#1555
svarlamov merged 1 commit into
mainfrom
feat/author-config

Conversation

@svarlamov

@svarlamov svarlamov commented Jun 14, 2026

Copy link
Copy Markdown
Member

Summary

  • Add author.name and author.email config support, including CLI get/set/unset flows.
  • Centralize git-ai effective author identity resolution so config fields partially override the Git committer identity.
  • Use the effective author for authorship metadata, status/blame user metadata, daemon rewrite/carryover paths, API identity headers, and git-ai debug output.
  • Add unit and integration coverage for full and partial author overrides plus debug effective-author formatting.

Validation

  • task fmt
  • task build
  • task test TEST_FILTER=author_config
  • task test TEST_FILTER=append_git_committer_identity
  • task lint
  • task test

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@svarlamov svarlamov force-pushed the feat/author-config branch from 78f687b to 8bd81ac Compare June 14, 2026 20:20
@svarlamov svarlamov merged commit b16cabe into main Jun 14, 2026
22 of 23 checks passed
@svarlamov svarlamov deleted the feat/author-config branch June 14, 2026 20:42
svarlamov added a commit that referenced this pull request Jun 14, 2026
…n sites

The rebase onto origin/main pulled in the author-config feature (#1555), which
replaced `git_author_identity()` with `effective_author_identity()` (the former
plus git-ai's author.name/author.email override) at every author-resolution
site. Three of main's four daemon.rs change sites fell inside blocks our rewrite
deletes (stale carryover/dead-machinery paths), so the override intent did not
reach our surviving attribution sites.

Apply the same change to the surviving sites: the 7 daemon.rs author
resolutions feeding post_commit_*/restore_working_log_carryover and the squash
author in rewrite.rs now use effective_author_identity(), so a configured author
override flows into authorship notes and working-log records. blame.rs/status.rs
kept main's change via clean auto-merge; the internal call inside
effective_author_identity() itself is left as git_author_identity().

Verified by test_author_config_overrides_session_and_known_human_records.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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