docs: update CI visibility refarch to content-only approach#840
docs: update CI visibility refarch to content-only approach#840chaodu-agent wants to merge 1 commit into
Conversation
|
All PRs must reference a prior Discord discussion to ensure community alignment before implementation. Please edit the PR description to include a link like: This PR will be automatically closed in 3 days if the link is not added. |
OpenAB PR ScreeningThis is auto-generated by the OpenAB project-screening flow for context collection and reviewer handoff.
Screening reportscreening complete.GitHub comment: #840 (comment) IntentPR #840 updates the CI observability Discord reference architecture to match the implementation that actually worked in testing. The operator-visible problem is that CI notifications must be readable by Discord bots, preserve multiline formatting, route to the right thread, and reliably ping the intended user or bot. FeatDocs work. The PR changes Who It ServesMaintainers and deployers. It also helps agent runtime operators who rely on Discord notifications being visible to bot frameworks rather than hidden inside embeds. Rewritten PromptUpdate the CI observability Discord reference architecture doc so it describes the verified content-only webhook approach. Remove guidance that depends on Discord embeds for CI metadata. Show a reusable GitHub Actions notification workflow that builds the Discord message as real multiline text, reads it into JSON with Merge PitchThis should move forward as a low-risk documentation correction. Main reviewer concern: the sample must be copy-paste accurate, especially around whether mentions support users only or roles too. Best-Practice ComparisonOpenClaw and Hermes Agent mostly do not apply here because this is documentation for CI-to-Discord webhook notifications, not scheduling, persisted execution, or gateway runtime architecture. RecommendationUse the balanced path: merge the doc correction after quick technical review, then follow up with a tested reusable workflow/template so the reference architecture does not drift again. |
Summary
Update the CI visibility reference architecture doc to reflect the final working implementation.
Changes
contentfield so bots can read itjq --rawfile— guarantees real newlines in JSON payloadallowed_mentions— required for webhook mentions to actually pingWhy
We discovered through testing that:
jq --argwith shell variables loses newlinesallowed_mentionsto ping