-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Bug]: MSTeams provider-prefixed explicit target IDs miss resolver gate #104381
Copy link
Copy link
Open
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:message-lossChannel message delivery can be lost, duplicated, or misrouted.Channel message delivery can be lost, duplicated, or misrouted.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:message-lossChannel message delivery can be lost, duplicated, or misrouted.Channel message delivery can be lost, duplicated, or misrouted.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Type
Fields
Priority
None yet
Bug type
Behavior bug (incorrect output/state without crash)
Beta release blocker
No
Summary
Provider-prefixed Microsoft Teams explicit targets such as
teams:user:<aad-guid>andmsteams:user:<aad-guid>are normalized likeuser:<aad-guid>, but the MSTeams target-ID detector did not classify the prefixed forms as explicit IDs.Steps to reproduce
user:40a1a0ed-4ff2-4164-a219-55518990c197.teams:user:40a1a0ed-4ff2-4164-a219-55518990c197andmsteams:user:40a1a0ed-4ff2-4164-a219-55518990c197.maybeResolvePluginMessagingTarget({ requireIdLike: true })and inputteams:user:40a1a0ed-4ff2-4164-a219-55518990c197.Expected behavior
Provider-prefixed explicit MSTeams user IDs should be classified consistently with their normalized
user:<aad-guid>form, allowing the explicit-target resolver path to run.Actual behavior
The detector only trimmed the raw input before checking known MSTeams ID formats, so the prefixed forms were not recognized as explicit IDs and could be stopped by the id-like resolver gate.
OpenClaw version
main branch before #104364
Operating system
Windows 11 local checkout; behavior is string parsing and not OS-specific.
Install method
Local repository checkout with targeted Vitest validation.
Model
NOT_ENOUGH_INFO
Provider / routing chain
MSTeams outbound target normalization and plugin target resolution.
Additional provider/model setup details
This is limited to MSTeams target parsing/resolution. No live Microsoft Graph or Teams credentials are required for the focused reproduction.
Logs
Screenshots, recordings, and evidence
Proposed fix and focused behavior proof: #104364
The added behavior-level test proves
teams:user:<aad-guid>passes throughmaybeResolvePluginMessagingTarget({ requireIdLike: true }), calls the plugin resolver, and reaches it as normalizeduser:<aad-guid>. It also provesteams:user:John Smithremains outside the id-like gate.Impact and severity
Affected: MSTeams users/operators who use provider-prefixed explicit targets.
Severity: Low to medium; explicit IDs can be classified inconsistently depending on whether the local provider prefix is present.
Frequency: Always for provider-prefixed explicit user IDs before the fix.
Consequence: Explicit target resolution paths that require id-like input can reject a prefixed target even though the normalized target is valid.
Additional information
This issue is intentionally narrow. It does not claim that directory lookup, approval authorization, or live Microsoft Graph behavior is broadly broken.