Skip to content

[repo-assist] Add DeepWiki MCP tool + guidance and fix 4 triage bugs (supersedes #50962, #50963) [skip ci] - #50964

Merged
blozano-tt merged 3 commits into
mainfrom
feat/repo-assist-deepwiki-and-triage-fixes
Jul 24, 2026
Merged

blozano-tt merged 3 commits into
mainfrom
feat/repo-assist-deepwiki-and-triage-fixes

Conversation

@blozano-tt

@blozano-tt blozano-tt commented Jul 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This PR combines and supersedes two open repo-assist PRs so there is a single coherent repo-assist.md source and one clean gh aw compile to test against:

Why combine them

.github/workflows/repo-assist.lock.yml is a deterministic function of the whole .github/workflows/repo-assist.md source (both its frontmatter_hash and body_hash are recomputed on every compile). Each of the two PRs changed both hashes, so merging them independently would have one PR's lock regeneration clobber the other's. Per @wilder's request, they are merged here into one source file and compiled once so the workflow can be manually tested end-to-end against a single lock.

What's included

DeepWiki MCP tool + guidance (from #50962)

  • Adds a read-only DeepWiki MCP server (https://mcp.deepwiki.com/mcp) scoped to read_wiki_structure, read_wiki_contents, ask_question. No new write permissions — purely an additional read-only tool.
  • New body section "Using DeepWiki (cross-repo grounding)": use it to orient across the sibling repos the runner can't easily clone (tt-metal, tt-umd, tt-kmd, tt-isa-documentation), with a strict verify-before-asserting discipline — anything load-bearing must be confirmed against current code via bash/gh, never asserted from DeepWiki alone.
  • Memory guidance: cache pointers, not frozen facts — store dated, re-derivable notes ("ask DeepWiki about X; last checked ") rather than pasting DeepWiki answers that drift.

Triage-logic fixes (from #50963)

  1. New-contributor welcome (Task 7) now requires verified zero merged PRs and zero prior issues/comments (via GitHub search) before welcoming — no longer inferring "new" from local context/memory.
  2. Firewall boilerplate — adds an instruction to strip the benign ⚠️ Firewall blocked … awmgmcpg block from public comments. awmgmcpg is gh-aw's own internal MCP-gateway sidecar; it cannot be silenced via network.allowed at this compiler version (confirmed dead end, documented in a frontmatter NOTE), so it's handled at the instruction level.
  3. Fresh-issue scoping (Task 1/2) — first-time comment+label triage is restricted to issues active in ~90 days, and [HELP]/question issues no longer default to the bug label.
  4. Comment dedup (Task 7 + Anti-spam) — when both a triage comment (Task 2) and a welcome (Task 7) apply to the same issue in one run, they are combined into a single comment instead of two.

Compile / testing notes

  • repo-assist.lock.yml was regenerated once with gh aw compile on the combined source using gh-aw v0.82.14 — the exact compiler_version pinned in the base main lock's gh-aw-metadata. Result: 0 errors, 0 warnings, and a second compile is byte-identical (deterministic). The lock diff reflects only real content changes (DeepWiki added to the MCP gateway config + GH_AW_ALLOWED_DOMAINS, updated gh-aw-metadata hashes). The .lock.yml is generated — please don't hand-edit; regenerate via gh aw compile repo-assist.
  • Suggested manual test: trigger /repo-assist <a cross-repo question> (e.g. something about tt-umd device init) on a test issue. One run then exercises both change-sets: the DeepWiki tool for cross-repo grounding, and the fixed contributor-detection / labeling / comment-dedup behavior.

Once this is verified and merged, #50962 and #50963 are being closed in favor of it.

🤖 Generated with Claude Code

CI Status

Auto-generated on every push. Badges update live. Click a badge to filter runs by this branch.

Combines and supersedes #50962 (DeepWiki MCP) and #50963 (triage-logic
fixes) into a single coherent repo-assist.md source and one clean
`gh aw compile` of repo-assist.lock.yml, so the workflow can be manually
tested end-to-end against one lock file rather than two independently
regenerated ones that would clobber each other.

DeepWiki (from #50962):
- Add read-only DeepWiki MCP server (mcp.deepwiki.com/mcp), scoped to
  read_wiki_structure / read_wiki_contents / ask_question.
- Add "Using DeepWiki (cross-repo grounding)" body section: orientation
  only for tt-metal/tt-umd/tt-kmd/tt-isa-documentation, verify-before-
  asserting discipline, and cache pointers-not-facts (dated, re-derivable)
  in repo-memory rather than frozen DeepWiki answers.

Triage fixes (from #50963):
- Task 7: require verified zero merged PRs AND zero prior issues/comments
  (via GitHub search) before welcoming a "new" contributor.
- Guidelines: strip benign "Firewall blocked ... awmgmcpg" boilerplate
  from public comments (network.allowed is a dead end at this compiler
  version; documented in a frontmatter NOTE).
- Task 1/2: scope fresh comment+label triage to issues active in ~90 days
  and stop defaulting [HELP]/question issues to the `bug` label.
- Task 7 + Anti-spam: combine triage + welcome into one comment when both
  apply to the same issue in a single run.

lock.yml regenerated once with gh-aw v0.82.14 (matching base main pin);
0 errors / 0 warnings, deterministic recompile. Do not hand-edit the
lock file.

Co-Authored-By: Claude <noreply@anthropic.com>

Co-Authored-By: BrAIn <brain@tenstorrent.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds DeepWiki cross-repository grounding and refines Repo Assist triage behavior.

Changes:

  • Adds a read-only DeepWiki MCP server and verification guidance.
  • Tightens contributor detection, issue triage, and comment deduplication.
  • Regenerates the compiled workflow lock.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/repo-assist.md Configures DeepWiki and updates triage instructions.
.github/workflows/repo-assist.lock.yml Regenerates the executable workflow configuration.
Comments suppressed due to low confidence (1)

.github/workflows/repo-assist.md:354

  • [🟡] Bug: Prompt text cannot suppress the generated firewall footer

In gh-aw v0.82.14, actions/setup/js/messages_footer.cjs:654-673 reads blocked domains and appends the warning after the agent-produced content. This instruction only controls what the agent submits, so the awmgmcpg warning will still be attached to every public safe output.

Why it matters: the firewall-boilerplate bug advertised as fixed by this PR will reproduce unchanged on the next run. The fix must suppress this internal hostname in gh-aw's footer generation (or upgrade to a release that does) and then regenerate the lock file.

- **Never forward firewall boilerplate into comments**: do not copy or reproduce any `⚠️ Firewall blocked …` warning block (e.g. the benign `awmgmcpg` MCP-gateway notice) into issue/PR comments or descriptions. `awmgmcpg` is gh-aw's own internal MCP Gateway sidecar hostname, not a real missing dependency, and it cannot be silenced via `network.allowed` at the current compiler version (see the NOTE in the frontmatter). Treat any such block as internal-only noise and strip it from anything you post publicly.

Comment thread .github/workflows/repo-assist.md
Addresses Copilot review comment: the 90-day freshness restriction only
lived in Task 2, so Task 1 ("Triage and Label Open Issues") could still
apply first-time labels to long-stale issues — the exact behavior this
PR intends to stop.

- Task 1: scope first-time labelling to issues active in ~90 days,
  mirroring Task 2. First-time labels are not applied cold to stale
  issues; correcting/removing an already-wrong label stays allowed.
- Guidelines/"Systematic": clarify that oldest-first cursor order still
  respects the freshness gate (does not license first-time
  comments/labels on stale issues).
- Regenerated repo-assist.lock.yml via `gh aw compile` (gh-aw v0.82.14).
  frontmatter_hash unchanged, body_hash updated, output deterministic.

Co-Authored-By: BrAIn <brain@tenstorrent.com>
@blozano-tt

Copy link
Copy Markdown
Contributor Author

🤖 Repo Assist — review comments addressed

Went through the review feedback on this PR. 1 unresolved review comment (from the Copilot reviewer); no human inline comments or issue comments were open.

# Reviewer Comment Resolution
1 copilot-pull-request-reviewer 🟡 Apply the 90-day gate to Task 1 labels — the freshness restriction lived only in Task 2, so Task 1 could still first-time-label stale issues Fixed in d60813b3

What changed (.github/workflows/repo-assist.md):

  • Task 1 — Triage and Label Open Issues: first-time labelling is now scoped to issues active in ~90 days, mirroring Task 2. Stale issues (no activity in ~90 days) no longer get a cold first-time label; correcting/removing an already-wrong label stays allowed on any issue.
  • Guidelines → "Systematic": clarified that oldest-first cursor processing still respects the freshness gate, so reaching a stale issue in cursor order does not license a first-time comment or label (this covers the secondary "line 354" pointer in the review).

Lock regeneration: .github/workflows/repo-assist.lock.yml was recompiled with gh aw compile on gh-aw v0.82.14 (the pinned compiler_version). frontmatter_hash unchanged, body_hash updated, and a second compile is byte-identical — deterministic, per the compile discipline in the PR description. Result: 0 errors, 0 warnings.

Nothing was pushed back on — the single comment was a correct catch. The thread has been resolved.

@blozano-tt

Copy link
Copy Markdown
Contributor Author

/codeowners bypass

@blozano-tt
blozano-tt enabled auto-merge July 24, 2026 05:29

@tenstorrent-github-bot tenstorrent-github-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ CodeOwners bypass approval granted by @blozano-tt (metalium-developers-infra team)

@tenstorrent-github-bot

Copy link
Copy Markdown
Contributor

✅ CodeOwners Bypass Approval Granted

This PR has been approved by @blozano-tt (metalium-developers-infra team) using the bypass mechanism.

⚠️ Note: This bypass should only be used for emergency fixes or when standard approval process is blocked.

@blozano-tt blozano-tt changed the title [repo-assist] Add DeepWiki MCP tool + guidance and fix 4 triage bugs (supersedes #50962, #50963) [repo-assist] Add DeepWiki MCP tool + guidance and fix 4 triage bugs (supersedes #50962, #50963) [skip ci] Jul 24, 2026
@blozano-tt
blozano-tt added this pull request to the merge queue Jul 24, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 24, 2026
@blozano-tt
blozano-tt added this pull request to the merge queue Jul 24, 2026
Merged via the queue into main with commit 455bba7 Jul 24, 2026
96 checks passed
@blozano-tt
blozano-tt deleted the feat/repo-assist-deepwiki-and-triage-fixes branch July 24, 2026 08:34
@minaliuTT minaliuTT added pr-complexity:low Complexity Low: Focused, established pattern with simple validation. pr-priority:p3 P3: Low-urgency cleanup, docs, maintenance, or limited current impact. pr-risk:low Risk Low: Localized, reversible, or additive change with limited consequences. labels Sep 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat New feature pr-complexity:low Complexity Low: Focused, established pattern with simple validation. pr-priority:p3 P3: Low-urgency cleanup, docs, maintenance, or limited current impact. pr-risk:low Risk Low: Localized, reversible, or additive change with limited consequences.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants