feat: page-building tools enabled, nr-llm 0.31.0, DeepWiki as a public MCP server - #221
Merged
Merged
Conversation
The backend assistant is meant to build demo pages (NEXT-153). Two tools do that in nr-llm: create_page_draft (0.31, ADR-180) and create_content_element_draft (0.29, ADR-146), both hidden-only, both approval-gated. Both ship disabled; the seed enables them the way it enables update_page_metadata, with the same defence written next to it. A state row for a tool the installed nr-llm does not know yet is never read, so the rows are safe ahead of the bump. The MCP Servers module had nothing to show. The seed adds DeepWiki (https://mcp.deepwiki.com/mcp): public, read-only, no credential, data class publicContent, requires_approval switched off by the seed as the operator of this instance — nr-llm's own documentation says to do that per server once its tools are known, and DeepWiki's three read public documentation. Needs nr-llm >= 0.31 (ADR-181) to connect; the catalogue import stays a one-time click in the module until netresearch/t3x-nr-llm#836 lands. Guards at the end of the seed report both rows missing. Assisted-by: claude-code:claude-fable-5 Agent-Session: https://claude.ai/code/session_0124WZK5UoAM9oYQoSYF1Y8z Agent-Host: 32116e Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
0.31.0 carries create_page_draft (ADR-180), the tool the seed now enables so the backend assistant can build the NEXT-153 demo pages. Six installed extensions still require nr-llm ^0.30 (nr-ai-search, nr-landingpage, nr-llm-compat, nr-mcp-agent, nr-repurpose, t3-cowriter), so the same inline alias that bridged 0.30 over the 0.29 consumers (64e63a7 removed it once they had caught up) bridges 0.31 now: "v0.31.0 as v0.30.99". Drop the alias when the consumers widen their constraint. The DeepWiki MCP row waits for the transport fix (nr-llm #835) in the next nr-llm release; until then its connection test answers 406, which the seed comment says. Assisted-by: claude-code:claude-fable-5 Agent-Session: https://claude.ai/code/session_0124WZK5UoAM9oYQoSYF1Y8z Agent-Host: 32116e Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
|
Contributor
Dependency ReviewThe following issues were found:
License Issuescomposer.lock
OpenSSF Scorecard
Scanned Files
|
Member
Author
|
Self-review: b020bf4 The review this pull request demands is unsatisfiable (Copilot quota wall or repeated bot failures on this head). Per the documented fallback, the diff on this head was reviewed by the PR author; this comment is the on-the-record attestation the merge gate reads back. It stops matching on the next push. |
This was referenced Aug 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to NEXT-153: the backend assistant is meant to build demo pages, and the MCP Servers module had nothing to show.
What
v0.31.0 as v0.30.99— the same bridge 64e63a7 removed once the 0.29 consumers had caught up: six installed extensions (nr-ai-search, nr-landingpage, nr-llm-compat, nr-mcp-agent, nr-repurpose, t3-cowriter) still require^0.30. Drop the alias when they widen. 0.31.0 carriescreate_page_draft(ADR-180).create_page_draftandcreate_content_element_draftintx_nrllm_tool_state(section 8b, same defence asupdate_page_metadata), and adds DeepWiki (https://mcp.deepwiki.com/mcp) as atx_nrllm_mcp_serverrow (section 8c): public, read-only, no credential,publicContent,requires_approval = 0with the reasoning written next to it. Two new SEED-PROBLEM guards.Verified
The seed statements and guards ran twice against a throwaway MariaDB 11 with the real nr-llm DDL: idempotent, guards silent with the rows present and firing (both lines) with the rows removed.
composer validate --check-lockpasses; lock resolved in thecomposer:2image.After deploy
Assisted by claude-code:claude-fable-5 — Session