Skip to content

Inject session_id into can_use_tool request#37

Closed
col wants to merge 1 commit intoguess:mainfrom
col:add_session_id_to_can_use_tool
Closed

Inject session_id into can_use_tool request#37
col wants to merge 1 commit intoguess:mainfrom
col:add_session_id_to_can_use_tool

Conversation

@col
Copy link
Contributor

@col col commented Mar 19, 2026

I need to access the session_id within the can_use_tool callbacks and it's not currently available.

This is probably a hack approach to address this but I'm going to try using this workaround in my fork for now.

@guess
Copy link
Owner

guess commented Mar 19, 2026

AH sorry I just saw this. I also took it as an opportunity to clean up port.ex a bit as well. It should be addressed on the latest main branch

@guess guess closed this Mar 19, 2026
@col
Copy link
Contributor Author

col commented Mar 19, 2026

Yep saw your commits. Looks good. I'll test it out tomorrow. Thanks!

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.

2 participants