Skip to content

Users/ishagautam/add inbound request triage template - #14

Open
ishagautamMSFT wants to merge 15 commits into
mainfrom
users/ishagautam/add-inbound-request-triage-template
Open

ishagautamMSFT wants to merge 15 commits into
mainfrom
users/ishagautam/add-inbound-request-triage-template

Conversation

@ishagautamMSFT

Copy link
Copy Markdown
Collaborator
image image

@adrianatruji

Copy link
Copy Markdown
Collaborator

Review feedback

1. Classification buckets, consider collapsing to two.
Since Noise and Other both effectively capture "anything that isn't an actionable request," and both lead to the same outcome (input deemed not relevant for any action/triage), would it make sense to keep only two classification buckets? The distinction between Noise and Other doesn't seem to drive any differentiated downstream behavior.

2. request-handler agent node, truncated instruction.
The instruction "After successfully sending both emails, return ONLY this valid JSON:" has no content following it. Not sure if something was meant to be added here, but I believe a reference to the structured output fields is all that's needed to close it off.

@adrianatruji

Copy link
Copy Markdown
Collaborator

Thanks for reviewing , ishagautamMSFT !
Quick note, I see request-handler still instruction still ends at "After successfully sending both emails, return ONLY this valid JSON:" with nothing after it. Could you remove this? Thanks!

@ishagautamMSFT
ishagautamMSFT enabled auto-merge (squash) September 22, 2026 10:40
@ishagautamMSFT

Copy link
Copy Markdown
Collaborator Author

Thanks for reviewing , ishagautamMSFT ! Quick note, I see request-handler still instruction still ends at "After successfully sending both emails, return ONLY this valid JSON:" with nothing after it. Could you remove this? Thanks!

Adriana Trujillo (@adrianatruji) Can you please double check since i didn't find any mention in the latest workflow itself

@adrianatruji

Copy link
Copy Markdown
Collaborator

Thanks for reviewing , ishagautamMSFT ! Quick note, I see request-handler still instruction still ends at "After successfully sending both emails, return ONLY this valid JSON:" with nothing after it. Could you remove this? Thanks!

Adriana Trujillo (Adriana Trujillo (@adrianatruji)) Can you please double check since i didn't find any mention in the latest workflow itself

Mixed it with an older version, apologies!

This is good to merge :) adilei

This branch has not been deployed

No deployments
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