Skip to content

fix: normalize room ID case in Matrix config lookup#19294

Closed
MisterGuy420 wants to merge 1 commit intoopenclaw:mainfrom
MisterGuy420:fix/issue-19278
Closed

fix: normalize room ID case in Matrix config lookup#19294
MisterGuy420 wants to merge 1 commit intoopenclaw:mainfrom
MisterGuy420:fix/issue-19278

Conversation

@MisterGuy420
Copy link
Copy Markdown
Contributor

@MisterGuy420 MisterGuy420 commented Feb 17, 2026

Fixes #19278

What changed

  • Normalize room ID and aliases to lowercase in resolveMatrixRoomConfig() before building lookup candidates
  • This ensures case-insensitive matching between the room ID from Matrix events and the config entries

AI-assisted contribution

  • This fix was generated by an AI agent (OpenClaw cron: gh-issues-fix)
  • Testing depth: validated with pnpm build && pnpm check && pnpm test
  • The fix addresses the root cause described in the issue by normalizing the room ID to lowercase before lookup, ensuring config entries stored in any case can be found regardless of the case used in Matrix events

Greptile Summary

Normalized Matrix room IDs and aliases to lowercase before config lookup to enable case-insensitive matching between Matrix events and configuration entries.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The change is minimal, well-commented, and addresses a specific bug. The normalization to lowercase is consistent with Matrix server behavior and other parts of the codebase that normalize Matrix IDs.
  • No files require special attention

Last reviewed commit: a1357d8

@vincentkoc
Copy link
Copy Markdown
Member

you have been detected be spamming with unwarranted prs and issues and your issues and prs have been automatically closed. please read contributing guide Contributing.md.

@vincentkoc vincentkoc closed this Feb 22, 2026
@MisterGuy420 MisterGuy420 deleted the fix/issue-19278 branch February 22, 2026 21:32
@MisterGuy420
Copy link
Copy Markdown
Contributor Author

Cleaning up: deleted the branch from my fork as part of repository maintenance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: matrix Channel integration: matrix size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Matrix: per-room requireMention/autoReply config ignored due to room ID case mismatch

3 participants