[SOA] Enhance attachment handling in agent task messages with ignored reason support#7786
Conversation
…ored reason support
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
AL Documentation Audit
Documentation gaps were detected in the following apps:
- Agent: 0% documentation coverage
To generate documentation, run /al-docs init or /al-docs update using GitHub Copilot CLI or Claude Code.
This review is for awareness to help keep documentation in sync with code changes. It is okay to dismiss this request.
…son based on attachment status
…ed_Reason_for_Attachment
There was a problem hiding this comment.
Pull request overview
Adds support in the Agent task message pipeline to capture and persist an “Ignored Reason” for message attachments, enabling richer diagnostics/auditing when attachments are intentionally skipped by the agent.
Changes:
- Extend the public
Agent Task Message BuilderAPI with anAddAttachment(...; Ignored; IgnoredReason)overload. - Track ignored reasons per attachment in the internal message builder implementation and forward them during message creation.
- Persist ignored reason onto the
"Agent Task Message Attachment"record when linking attachments to messages.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/System Application/App/Agent/Interaction/Internal/AgentTaskMsgBuilderImpl.Codeunit.al | Tracks per-attachment ignored reasons and forwards them into attachment creation flow. |
| src/System Application/App/Agent/Interaction/Internal/AgentMessageImpl.Codeunit.al | Extends attachment creation to accept and persist an ignored reason. |
| src/System Application/App/Agent/Interaction/AgentTaskMessageBuilder.Codeunit.al | Exposes a new builder overload that accepts an ignored reason for ignored attachments. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…mentsList and GlobalIgnoredReasonByFileId
…ed_Reason_for_Attachment
Summary
This PR adds support for tracking and populating the ignored reason when handling attachments in agent task messages.
Changes:
AgentTaskMessageBuilder.Codeunit.alto expose ignored reason functionalityAgentMessageImpl.Codeunit.alto handle ignored reason in attachment processingAgentTaskMsgBuilderImpl.Codeunit.alto populate ignored reason for attachmentsWork Item(s)
Fixes AB#629496