Skip to content

Fix principal-owned session reads#179

Merged
chubes4 merged 1 commit into
mainfrom
fix/principal-session-get
May 17, 2026
Merged

Fix principal-owned session reads#179
chubes4 merged 1 commit into
mainfrom
fix/principal-session-get

Conversation

@chubes4
Copy link
Copy Markdown
Contributor

@chubes4 chubes4 commented May 17, 2026

Summary

  • Add an optional owner-aware single-session read contract for principal-owned transcript stores.
  • Use the owner-aware read path when loading one conversation session, so stores with hashed owner keys can verify ownership internally.
  • Extend the conversation session smoke test to cover matching-owner loads and cross-owner denial.

Testing

  • php tests/agents-conversation-session-abilities-smoke.php
  • homeboy lint --changed-since origin/main
  • homeboy test --changed-since origin/main

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Diagnosed the anonymous session load ownership gap, drafted the contract/test change, and ran scoped verification; Chris remains responsible for review and merge.

@chubes4 chubes4 merged commit da44775 into main May 17, 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