Ruhulalemeen Mulla

Results 4 comments of Ruhulalemeen Mulla

This issue has been resolved in PR #371. I've implemented the missing `ctx.EndInvocation()` call in the `runAfterAgentCallbacks()` function to properly mark the invocation as ended when an after-agent callback returns...

Thanks for the feedback! You raise a good point about comparing with the Python implementation. I understand your concern that since there aren't `ctx.Ended()` validations after the after-agents callback and...

Thanks for the clarification. I’ve updated the PR to remove the EndInvocation() call and added a detailed comment explaining why the invocation state shouldn’t be modified in runAfterAgentCallbacks, matching the...

Thanks for the code review! I've addressed the issue in my latest commit: **Commit: "Fix: Use EndInvocation() method instead of setting non-existent field"** Replaced `ctx.invocationEnded = true` with `ctx.EndInvocation()` to...