Skip to content

Discord message links - #135

Open
ArnabChatterjee20k wants to merge 3 commits into
feat/grafana-mcpfrom
discord-message-links
Open

Discord message links#135
ArnabChatterjee20k wants to merge 3 commits into
feat/grafana-mcpfrom
discord-message-links

Conversation

@ArnabChatterjee20k

Copy link
Copy Markdown
Member

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@ArnabChatterjee20k
ArnabChatterjee20k changed the base branch from main to feat/grafana-mcp August 7, 2026 13:59
@greptile-apps

greptile-apps Bot commented Aug 7, 2026

Copy link
Copy Markdown

Greptile Summary

The PR adds Discord jump links for retrieved discussions and appends them to user-facing QA and reply notifications while keeping stored answers and ticket comments unchanged.

  • Adds single-message and message-span link formatting, including direct-message links through Discord’s @me route.
  • Returns formatted Discord references alongside retrieval context and appends them only to notifier deliveries.
  • Updates Discord formatting tests for guild, direct-message, empty-result, span, and relevance-score behavior.

Confidence Score: 4/5

The PR should not merge until the outstanding red-phase error path can no longer accept a fix without confirming a failing-test reproduction.

A red-phase agent error still falls through without recording red confirmation, while the later green phase can confirm an unchanged baseline solely because it contains no new failures.

Files Needing Attention: crates/claudear-engine/src/processing.rs

Important Files Changed

Filename Overview
crates/claudear-analysis/src/knowledgebase/discord/mod.rs Adds reusable Discord jump-link and reference-block formatting with focused unit coverage.
crates/claudear-analysis/src/knowledgebase/mod.rs Re-exports the new Discord reference-link formatter.
crates/claudear-engine/src/processing.rs Threads Discord references through retrieval and appends them to notifier deliveries; the previously reported red-green gate defect remains elsewhere in this file.

Reviews (4): Last reviewed commit: "linting" | Re-trigger Greptile

Comment thread crates/claudear-engine/src/processing.rs
@ArnabChatterjee20k
ArnabChatterjee20k changed the base branch from feat/grafana-mcp to main August 7, 2026 14:04
@ArnabChatterjee20k
ArnabChatterjee20k changed the base branch from main to feat/grafana-mcp August 7, 2026 14:07
@ArnabChatterjee20k
ArnabChatterjee20k changed the base branch from feat/grafana-mcp to main August 10, 2026 08:04
@ArnabChatterjee20k
ArnabChatterjee20k changed the base branch from main to feat/grafana-mcp August 11, 2026 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant