Skip to content

fix: remove gr2 identity binding (boundary enforcement)#687

Closed
laynepenney wants to merge 11 commits intomainfrom
fix/remove-identity-binding
Closed

fix: remove gr2 identity binding (boundary enforcement)#687
laynepenney wants to merge 11 commits intomainfrom
fix/remove-identity-binding

Conversation

@laynepenney
Copy link
Copy Markdown
Collaborator

Summary

  • Removes gr2 identity binding code from public repo (recall#637)
  • Identity resolution (workspace.toml, agent.toml, g2_ prefix) is premium per boundary policy
  • Preserves SYNAPT_AGENT_ID env var as the neutral OSS seam
  • Code will be migrated to synapt-private in a future sprint

Removed:

  • Gr2Context/Gr1Context/StandaloneContext dataclasses and detection functions
  • gr2 branches in _agent_id() and _resolve_griptree()
  • workspace column from presence schema, migration, and all queries
  • test_gr2_identity.py (416 lines)
  • TestWorktreeInWho from test_channel_worktree.py

Test plan

  • Full test suite: 1961 passed, 0 failed, 19 skipped
  • Team review: Apollo, Atlas, Sentinel confirmed no dependencies

Premium boundary: recall is OSS (channel infrastructure). Identity resolution is premium.

laynepenney and others added 11 commits April 10, 2026 15:37
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Identity resolution (workspace.toml, agent.toml, g2_ prefix) is premium
per boundary policy. Removes ~580 lines from channel.py and tests.
SYNAPT_AGENT_ID env var neutral seam preserved.

1961 tests passing.
@github-actions
Copy link
Copy Markdown
Contributor

Thank you for your contribution! Before we can merge this PR, we need you to sign our Contributor License Agreement.

To sign, please comment on this PR with the following exact text:

I have read the CLA Document and I hereby sign the CLA

You can also re-trigger the CLA check by commenting recheck.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@laynepenney
Copy link
Copy Markdown
Collaborator Author

Obsolete after the recall history rewrite. The public OSS identity-binding path was removed from main and the remaining capability was ported into premium/private code instead of being restored in OSS.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant