Microsoft/bcquality
1
0
Fork 0
mirror of https://github.com/microsoft/BCQuality.git synced 2026-10-05 14:46:55 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 4
bcquality/community/knowledge/agents/analyze-message-error-stops-warning-forces-review.md
Jesper Schulz-Wedde 2b5550c346
Some checks failed
Validate knowledge index / validate-index (push) Has been cancelled
Details
Validate AL review fixtures / validate-review-fixtures (push) Has been cancelled
Details
Validate frontmatter and structure / validate (push) Has been cancelled
Details
Improve partner onboarding and documentation navigation (#174) 
Lead with a complete plugin quick start and add task-oriented usage, troubleshooting, customization, and contribution guides. Preserve the broader plugin framing, correct conflicting contract guidance, support Agents folder reviews, and align repository validation. Convert existing sample references to clickable links without changing knowledge rules.

Co-authored-by: Jesper Schulz-Wedde <jesper.schulzwedde@microsoft.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
2026-09-09 17:31:03 +02:00

1.4 KiB
Raw Permalink Blame History

bc-version domain keywords technologies countries application-area
27..
agents
analyzeagenttaskmessage
agent-annotation
error
warning
setrequiresreview
al
w1
all

AnalyzeAgentTaskMessage: Error stops the task; Warning still requires review

Description

IAgentTaskExecution.AnalyzeAgentTaskMessage runs on inbound and outbound messages. An Error annotation stops processing. A Warning annotation requests user intervention. If analysis returns Warning, the platform still requires approval even when the incoming message used SetRequiresReview(false). Output text can be rewritten here (signature, redaction).

Best Practice

Validate inbound payloads in analysis: Error when the task must not run; Warning when a human must confirm. For outbound messages, adjust text in this method rather than in a later subscriber. Do not rely on skip-review to bypass warnings.

See sample: analyze-message-error-stops-warning-forces-review.good.al.

Anti Pattern

Ignoring analysis entirely, or emitting Warning while documenting that SetRequiresReview(false) means unattended run. Detection signal: empty AnalyzeAgentTaskMessage plus skip-review on external input.

See sample: analyze-message-error-stops-warning-forces-review.bad.al.