Skip to content

Discord message links - #135

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

Discord message links#135
ArnabChatterjee20k wants to merge 7 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 start/end Discord message links to retrieved discussion context and appends a compact reference list to notifier-delivered answers and replies.

  • Adds Discord jump-link formatting with an @me fallback when the guild ID is unavailable.
  • Propagates reference links separately from agent-facing RAG context.
  • Routes Homebrew and APT release publication to fixed abnegate repositories.

Confidence Score: 4/5

The PR is not yet safe to merge because a red-phase agent failure can still bypass the required reproduction gate and later be accepted as green.

The red-phase error branch continues into the fix pipeline without recording or enforcing red confirmation, while later green evaluation only checks that no new test failures remain.

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-list formatting, including tests for guild and missing-guild cases.
crates/claudear-analysis/src/knowledgebase/mod.rs Re-exports the new Discord reference-link formatter.
crates/claudear-engine/src/processing.rs Carries Discord references alongside RAG context and appends them only to notifier deliveries; the previously reported red-green gate failure remains outstanding.
.github/workflows/release.yml Changes Homebrew and APT publication clones to fixed repositories under the abnegate account.

Reviews (3): Last reviewed commit: "Merge remote-tracking branch 'origin/mai..." | 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 and others added 6 commits August 7, 2026 19:51
The `publish-apt` job derived its clone URL from `github.repository_owner`,
which resolves to `appwrite` now that claudear lives under the Appwrite org.
`appwrite/apt-repo` does not exist, so every release failed with:

    fatal: repository 'https://github.com/appwrite/apt-repo.git/' not found

The APT repo is hosted at `abnegate/apt-repo` — the same host the README
tells users to install from (`abnegate.github.io/apt-repo`). The Homebrew
tap has the identical defect: `appwrite/homebrew-tap` 404s while
`abnegate/homebrew-tap` is the tap the README documents (`brew tap
abnegate/tap`). Both clone URLs are now pinned to `abnegate`.

The formula template's `{{REPO_OWNER}}` placeholder is left alone — it
builds release *download* URLs, which do point at the source repo.

Adds tests that pin both workflow push targets to the owner documented in
the README install instructions, so the two cannot drift apart again.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
…wner

fix(release): publish APT repo and Homebrew tap to abnegate
@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.

2 participants