Skip to content

feat(auth): persist per-person Git identity across sessions - #4

Draft
cjavad wants to merge 5 commits into
feature/javad/git-identity-signingfrom
feature/johan/git-identity
Draft

cjavad wants to merge 5 commits into
feature/javad/git-identity-signingfrom
feature/johan/git-identity

Conversation

@cjavad

@cjavad cjavad commented Sep 23, 2026

Copy link
Copy Markdown
Owner

Adds stable identity subjects, separates pairing labels from people, and persists the Git identity used for commits. Depends on #3; review this PR against that feature branch to exclude its signing changes.

The commits retain Javad Asgari Shafique and Johan Ohly as their recorded authors. Prepared as a review PR with GPT-6 Sol in Codex.

cjavad and others added 5 commits September 21, 2026 10:40
Agent commits were authored by whoever was connected when the provider
process happened to start. A scheduled run, a thread resumed after a
restart, or a colleague pressing "continue" therefore committed under the
wrong name, and a thread whose owner had no profile fell through to no
identity at all.

The thread now carries it. Its first genuine user message claims the
identity for its sender and later messages keep it, whoever they come
from; creation claims nothing, so forks and imported sessions stay
unassigned until somebody speaks in them. Thread Details shows it and can
reassign it, which takes access-management permission when the target is
another user, since their signing key and GitHub token come with it.

Resolution moved to ProviderSessionManager.open, the single point every
spawn passes through, and reads the thread rather than the socket.
Reassigning detaches live sessions: a running process keeps the
environment it started with, so the next run has to be a new one.
Credential-resolution failures are logged instead of being folded into
"no identity configured".
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XXL labels Sep 23, 2026
@github-actions

Copy link
Copy Markdown

Thread transfer impact

⚠️ The latest CI run did not produce a thread transfer result for cd14d15.

This comment will update automatically after the next completed run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants