Skip to content

feat(git-identity): announce confirmed mismatches at session start - #64

Merged
verlyn13 merged 1 commit into
mainfrom
feat/q1b-sessionstart-identity-announcement
Aug 15, 2026
Merged

feat(git-identity): announce confirmed mismatches at session start#64
verlyn13 merged 1 commit into
mainfrom
feat/q1b-sessionstart-identity-announcement

Conversation

@verlyn13

Copy link
Copy Markdown
Member

Summary

Adds the source-side Q1b SessionStart identity announcer for Claude Code and Codex while
retaining the existing PreToolUse identity enforcement as the mutation backstop.

The announcer emits a fixed privacy-safe warning only for a confirmed Git-identity mismatch
and otherwise fails silently. It does not claim that live SessionStart registration or
dispatch is already active.

Scope

  • add git-identity-sessionstart
  • add callable machine evaluation to git-identity-gate without changing legacy CLI behavior
  • extend the focused identity suite
  • document the source/runtime ownership and activation boundary
  • include the focused suite in repository validation
  • update ng-doctor for the new managed target

No live Claude or Codex configuration is changed by this PR.

Validation

  • focused Q1b suite: 114/114 passed before commit
  • classified differential:
    • accepted_controls=2
    • contract_rejected=80
    • checker_error=1
    • conservative_consumer_only=2
    • producer_rejected_machine_confirmed=0
  • repository validation passed at an actual 80-column PTY
  • GitHub push-triggered Repo Validation passed on exact head
    d55cd1519a66dfd41e77c82f06273b1e1d40a0d7

Activation remains pending

After landing, Q1b still requires separate owner-controlled steps:

  1. deploy the exact managed gate, announcer, and doctor targets;
  2. verify deployed bytes/rendered output against landed source;
  3. append the settled SessionStart command separately to Claude and Codex without disturbing
    existing handlers;
  4. complete Codex trust review;
  5. remeasure installed and active hook inventories;
  6. require positive wrong-root warnings and separately verify correct-root silence;
  7. reverify the existing PreToolUse mutation backstop.

Registration alone is not dispatch proof, and correct-root silence is not positive-path
acceptance.

Deliberately out of scope

  • the pre-existing PTY-width test-isolation defect;
  • the stale .github/PULL_REQUEST_TEMPLATE.md;
  • unrelated 1Password documentation work in the main checkout;
  • any general agentic-configuration cleanup.

@verlyn13
verlyn13 requested a review from a team as a code owner August 15, 2026 18:59
@verlyn13
verlyn13 merged commit e9a4885 into main Aug 15, 2026
2 checks passed
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