Skip to content

docs: update CI observability refarch with final implementation#839

Merged
thepagent merged 1 commit into
mainfrom
docs/refarch-ci-notify-update
May 18, 2026
Merged

docs: update CI observability refarch with final implementation#839
thepagent merged 1 commit into
mainfrom
docs/refarch-ci-notify-update

Conversation

@chaodu-agent
Copy link
Copy Markdown
Collaborator

Update the CI observability reference architecture to reflect the final working implementation:

  • Plain-text content instead of embed (bots can read message.content)
  • printf + jq --rawfile for proper newlines
  • Both PR URL and Run URL included
  • Updated gotchas (webhook mentions don't trigger bots)
  • Updated ASCII diagram to show actual output format

Thread: 1505664791719710810

@chaodu-agent chaodu-agent requested a review from thepagent as a code owner May 18, 2026 01:13
@github-actions github-actions Bot added the closing-soon PR missing Discord Discussion URL — will auto-close in 3 days label May 18, 2026
@github-actions
Copy link
Copy Markdown

⚠️ This PR is missing a Discord Discussion URL in the body.

All PRs must reference a prior Discord discussion to ensure community alignment before implementation.

Please edit the PR description to include a link like:

Discord Discussion URL: https://discord.com/channels/...

This PR will be automatically closed in 3 days if the link is not added.

@github-actions github-actions Bot added the pending-screening PR awaiting automated screening label May 18, 2026
@thepagent thepagent merged commit ffcc8e8 into main May 18, 2026
3 checks passed
apple8409 pushed a commit to apple8409/openab that referenced this pull request May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

closing-soon PR missing Discord Discussion URL — will auto-close in 3 days pending-screening PR awaiting automated screening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants