Repository navigation
feat(bot): route events to forum topics by category - #733
harmondconsultant-png wants to merge 2 commits into
Conversation
Closes mimir-stellar#455 Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
3b2505b to
b83d607
Compare
|
Warning Possible malicious code: please review before merging. An automated security scan flagged files in this pull request that appear to contain malicious code. Code like this can run automatically when a project is opened in an editor or its dependencies are installed, so it could compromise the machine of anyone who checks out this branch. Automated scans can be wrong, so this may be a false positive. Maintainers: please review the flagged files carefully before merging, on GitHub in your browser rather than by checking out the branch. The malicious part is often hidden far off to the right of an ordinary-looking line, so scroll sideways through each file. @harmondconsultant-png: this pull request adds or changes these files, possibly without your knowledge. Malware like this spreads by adding itself to repositories on machines it has infected, so your machine might be compromised. Please undo these changes in this pull request, and don't open or run other projects on that machine until you've checked it. Flagged files at commit Only merge once you're confident these files are safe. ℹ️ Posted automatically by Drips Wave. Questions? Contact support@drips.network. |
Closes #455
Adds
src/notifications/topics.ts(parseclaims:12,resolutions:15,squad:20, resolve category to thread id, derive category from event name), a validatedTELEGRAM_TOPIC_MAPsetting (invalid mappings fail config load), andmessage_thread_idfromcreateNotifieronly when mapped (otherwise payload unchanged). The poller setsextra.category(resolutionsfor claim_resolved/market_settled/resolved,claimsfor other claim_* events, else the contract source). Key registered in config-cli and provenance. Tests:tests/forum-topics.test.mjs.Verification (honest):
npm cisucceeded on Node 20.npm run typecheckFAILS with 42 syntax errors insrc/bot.tsandsrc/notifications/format.ts(e.g. a duplicatedcase "claim_created"at format.ts:406-408), files this PR does not touch, somainappears already broken. Because the build fails, tests were not run. No errors are reported in files changed here, but syntax errors may mask semantic checks.?? Generated with Claude Code