Skip to content

test: verify session ID injection works#102

Open
cbarber wants to merge 6 commits intospr/main/75e4b699from
spr/main/dfcac694
Open

test: verify session ID injection works#102
cbarber wants to merge 6 commits intospr/main/75e4b699from
spr/main/dfcac694

Conversation

@cbarber
Copy link
Owner

@cbarber cbarber commented Feb 2, 2026

Session-scoped pr-poll daemon needs to know which session triggered it. Prepending export to bash commands makes session ID available to shell scripts.

Authored By: opencode (claude-sonnet-4-5)

commit-id:295171f4
Session-scoped daemons know exactly which session to notify. Bypassing anvil discovery for explicit session targets reduces latency and ensures correct routing.

Authored By: opencode (claude-sonnet-4-5)

commit-id:4da15748
After creating/updating PRs, agents need automatic monitoring without manual daemon management. Starting session-scoped daemon on spr update ensures timely PR notifications.

Authored By: opencode (claude-sonnet-4-5)

commit-id:5cd74256
After creating PRs via forge, agents need automatic monitoring like spr update provides. Starting session-scoped daemon ensures consistent PR notification behavior.

Authored By: opencode (claude-sonnet-4-5)

commit-id:69e04b20
Global daemon caused multi-session race conditions and died with first shell. Session-scoped daemons started by spr/forge provide better isolation and lifecycle management.

Authored By: opencode (claude-sonnet-4-5)

commit-id:75e4b699
@cbarber cbarber force-pushed the spr/main/75e4b699 branch 8 times, most recently from cb3411c to e264c9f Compare February 15, 2026 05:32
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

Comments