refactor(rocketchat): move configuration types into rocketchat package

[TheMeier]

Pull Request Checklist

• I have signed-off my commits
• I will follow best practices for contributing to this project

Which user-facing changes does this PR introduce?

NONE

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: a6cd017b-83e6-4bf3-bce1-b616e6e7c296

📥 Commits

Reviewing files that changed from the base of the PR and between 1d89b48 and da7fa2a.

📒 Files selected for processing (5)

• config/config.go
• config/notifiers.go
• notify/rocketchat/config.go
• notify/rocketchat/rocketchat.go
• notify/rocketchat/rocketchat_test.go

💤 Files with no reviewable changes (1)

• config/notifiers.go

🚧 Files skipped from review as they are similar to previous changes (4)

• notify/rocketchat/rocketchat_test.go
• notify/rocketchat/config.go
• config/config.go
• notify/rocketchat/rocketchat.go

---

📝 Walkthrough

Walkthrough

Rocketchat configuration types and YAML parsing move into notify/rocketchat. Receiver inheritance, notifier constructors, payload types, token helpers, attachment creation, and tests are updated to use the package-local definitions.

Changes

Rocketchat configuration migration

Layer / File(s)	Summary
Define local Rocketchat configuration notify/rocketchat/config.go, config/notifiers.go	Rocketchat defaults, configuration fields, attachment types, and token validation are defined in notify/rocketchat; the previous config declarations are removed.
Wire receiver configuration config/config.go	Receiver configuration and YAML inheritance use rocketchat.RocketchatConfig, including nil-entry initialization and global HTTP/API/token values.
Update notifier payload construction notify/rocketchat/rocketchat.go, notify/rocketchat/rocketchat_test.go	Notifier constructors, token helpers, attachment payload construction, and tests use package-local Rocketchat types.

Estimated code review effort: 3 (Moderate) | ~20 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description leaves most required checklist items and issue/release-note details unfilled, so it is not mostly complete.	Fill in the missing checklist sections, link any related issues, and provide the user-facing changes or confirm NONE in the release-notes block.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly states the main refactor: moving Rocketchat configuration types into the rocketchat package.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}