Skip to content

Update graphql to support new models - #32

Merged
hungpham10 merged 2 commits into
mainfrom
feaature/update-graphql-to-support-new-models
Sep 13, 2026
Merged

Update graphql to support new models#32
hungpham10 merged 2 commits into
mainfrom
feaature/update-graphql-to-support-new-models

Conversation

@hungpham10

@hungpham10 hungpham10 commented Sep 13, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Documentation

    • Updated MCP tool references to use the current names across guides, specifications, and installation instructions.
    • Revised documentation for symbol search, symbol lookup, and graph-code file access.
    • Removed references to the deprecated exploration tool.
  • API Changes

    • Renamed GraphQL queries and mutations with consistent graphcode_ and graphdoc_ prefixes.
    • Updated simulated response tool names to match the standardized naming scheme.

@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 9e3b98a1-fb3b-40c7-a84a-959e6872cfb1

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 0e93080b-1f18-4dea-b37b-d48849645641

📥 Commits

Reviewing files that changed from the base of the PR and between f00967f and 87f6029.

📒 Files selected for processing (10)
  • .agents/AGENTS.md
  • crates/codegraph-api/src/lib.rs
  • crates/codegraph-api/src/tools.rs
  • crates/codegraph-graphql/src/mutation.rs
  • crates/codegraph-graphql/src/query.rs
  • crates/codegraph-installer/src/instructions-template.md
  • crates/codegraph-installer/src/targets/antigravity.rs
  • crates/codegraph-mcp/src/tools.rs
  • crates/codegraph/src/main.rs
  • docs/specs/07-mcp-server.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The pull request renames GraphQL resolvers and simulation payload tools with domain-prefixed names. It updates MCP references, installer guidance, agent instructions, CLI documentation, and the MCP specification.

Changes

Tool namespace alignment

Layer / File(s) Summary
GraphQL resolver namespace updates
crates/codegraph-graphql/src/mutation.rs, crates/codegraph-graphql/src/query.rs
GraphQL query and mutation resolvers now use graphcode_* and graphdoc_* names. Resolver bodies remain unchanged.
API and MCP name references
crates/codegraph-api/src/lib.rs, crates/codegraph-api/src/tools.rs, crates/codegraph-mcp/src/tools.rs, crates/codegraph/src/main.rs
Simulation payloads and documentation now reference the renamed tools.
Guidance and specification updates
.agents/AGENTS.md, crates/codegraph-installer/src/instructions-template.md, crates/codegraph-installer/src/targets/antigravity.rs, docs/specs/07-mcp-server.md
Tool-selection guidance and the MCP specification use the renamed tools. The specification removes codegraph_explore.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: cleboost

Merge Risk: ⚪ Minimal · up to 87f60

The namespace updates align GraphQL, MCP dispatch, and simulation payload names without an established merge-blocking defect.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title identifies a GraphQL update, which is related to the changes. It is imprecise because the pull request mainly renames GraphQL resolvers and MCP tools rather than adding support for new model…
Docstring Coverage ✅ Passed Docstring coverage is 94.74% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 19 functions across 7 files. (3 skipped: 3 …
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feaature/update-graphql-to-support-new-models

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit hops through names anew
Graphcode tools wear prefixes blue
Graphdoc paths align in rows
MCP guidance clearly flows
The code keeps its logic true

Comment @coderabbitai help to get the list of available commands.

@hungpham10
hungpham10 merged commit 580722a into main Sep 13, 2026
16 of 19 checks passed
@hungpham10
hungpham10 deleted the feaature/update-graphql-to-support-new-models branch September 13, 2026 08:30
@codspeed-hq

codspeed-hq Bot commented Sep 13, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 28 untouched benchmarks


Comparing feaature/update-graphql-to-support-new-models (87000b2) with main (2e7f071)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (f00967f) during the generation of this report, so 2e7f071 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

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