Normalized message

[lao]

Summary by CodeRabbit

• New Features

  • Added platform-agnostic message fields such as timestamps, reply references, and mentioned users for better consistency across bots.
  • Added access to raw platform data and platform-specific clients when normalized fields aren’t enough.

• Bug Fixes

  • Improved message conversion across Discord, Slack, Telegram, and CLI so more message details are preserved reliably.
  • Fixed attachment handling and mention parsing in several platforms.

• Documentation

  • Updated docs and examples to show the new message fields and raw access patterns.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4ecf5ebd-eb79-4294-a9f0-c908b98889fc

📥 Commits

Reviewing files that changed from the base of the PR and between f96f3c0 and e7519d0.

📒 Files selected for processing (17)

• .gitignore
• CLAUDE.md
• README.md
• botbooter.go
• botbooter_test.go
• examples/v1/main.go
• internal/cli/cli.go
• internal/cli/cli_test.go
• internal/core/core.go
• internal/core/core_test.go
• internal/core/imports_test.go
• internal/discord/discord.go
• internal/discord/discord_test.go
• internal/slack/slack.go
• internal/slack/slack_test.go
• internal/telegram/telegram.go
• internal/telegram/telegram_test.go

---

📝 Walkthrough

Walkthrough

The PR normalizes message data into core.Message.Raw plus shared fields, moves Discord, Slack, Telegram, and CLI adapters to adapter-backed raw/client accessors, and updates the facade, docs, tests, and example logging to use the new access path.

Changes

Raw payload normalization and facade accessors

Layer / File(s)	Summary
Normalized message contract internal/core/core.go, internal/core/core_test.go, internal/core/imports_test.go	Message now uses normalized fields and Raw any; Bot keeps an adapter; AdapterAs[T] is added and covered by tests and the core import check.
Discord adapter mapping internal/discord/discord.go, internal/discord/discord_test.go, botbooter_test.go	Discord messages now dispatch through toMessage, store the raw event in Raw, read attachments from Raw, expose Session/RawEvent, and update attachment tests to use Raw.
Slack adapter mapping internal/slack/slack.go, internal/slack/slack_test.go, botbooter_test.go	Slack messages now dispatch through toMessage, parse timestamps and mentions, read attachments from Raw, expose client/session/raw helpers, and update attachment tests to use Raw.
Telegram adapter mapping internal/telegram/telegram.go, internal/telegram/telegram_test.go	Telegram messages now dispatch through toMessage, store the raw update in Raw, read attachments from Raw, expose Client/RawUpdate, and extend message and mention tests.
CLI raw payload handling internal/cli/cli.go, internal/cli/cli_test.go	CLI dispatch now stores parsed input in Raw; Attachments and RawData read the raw CLI message through a type assertion, and the new accessor is tested.
Facade accessors and docs botbooter.go, botbooter_test.go, README.md, CLAUDE.md, examples/v1/main.go	Facade helpers re-export raw-event and client/session accessors, the tests call those accessors, and the README, CLAUDE.md, and example log the normalized fields and raw access path.

Sequence Diagram(s)

sequenceDiagram
  participant "botbooter.DiscordRawEvent" as DR
  participant "internal.discord.RawEvent" as IR
  participant "botbooter.DiscordSession" as DS
  participant "core.AdapterAs[T]" as AA
  participant "internal.discord.Session" as IS

  DR->>IR: inspect m.Raw
  IR-->>DR: *discordgo.MessageCreate, ok
  DS->>AA: recover adapter from *core.Bot
  AA-->>DS: concrete adapter, ok
  DS->>IS: return session or nil
  IS-->>DS: *discordgo.Session

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Possibly related PRs

• lao/botbooter#2: Both PRs work in the same facade/internal split and adapter-based access path around botbooter.go and internal/core.
• lao/botbooter#6: Both PRs touch the Telegram adapter and the shared message-shape changes that move platform data into Raw.

Poem

Hop, I ferried the Raw payload home,
with timestamps tucked in a carrot-patch dome.
Discord, Slack, Telegram, and CLI all hum,
while AdapterAs[T] twitches like a drum.
The bot burrows deep, and the docs go yum. 🐰

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch worktree-normalized-message

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}