Skip to content

Added function caller info for RunAiQuery#367

Open
satti-hari-krishna-reddy wants to merge 3 commits intoShuffle:mainfrom
satti-hari-krishna-reddy:observ
Open

Added function caller info for RunAiQuery#367
satti-hari-krishna-reddy wants to merge 3 commits intoShuffle:mainfrom
satti-hari-krishna-reddy:observ

Conversation

@satti-hari-krishna-reddy
Copy link
Copy Markdown
Collaborator

@satti-hari-krishna-reddy satti-hari-krishna-reddy commented Apr 8, 2026

What this PR does

I've added Context-Based Caller Tracking to bring visibility and observability into which features/functions are actually using RunAiQuery.

  • Explicitly requiring a caller in the context forces future features to include it. This means they automatically get tracked and prevents "silent" new AI calls.
  • Updates other functions to propagate context

cachedChatMd5 := md5.Sum([]byte(systemMessage))
// Does this mean that two same orgs that has same system message results in the same Md5 id ?

cachedChatMd5 := md5.Sum([]byte(systemMessage+org))
@frikky
Copy link
Copy Markdown
Member

frikky commented Apr 9, 2026

When you add this many context changes, please describe them and why. I don't get it :)

Fundmental changes need describing

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.

3 participants