Skip to content

docs: fix incorrect tab reference for Claude Desktop setup#852

Merged
Scriptwonder merged 1 commit into
CoplayDev:betafrom
Lulubellelll:fix/claude-config-text
Mar 3, 2026
Merged

docs: fix incorrect tab reference for Claude Desktop setup#852
Scriptwonder merged 1 commit into
CoplayDev:betafrom
Lulubellelll:fix/claude-config-text

Conversation

@Lulubellelll

@Lulubellelll Lulubellelll commented Mar 2, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

Description

Corrected a misleading instruction in the Claude Desktop configuration UI. The text previously directed users to the "Advanced Settings" tab for transport configuration, but these options are located in the "Connect" tab in the current version.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • Documentation update

Changes Made

  • Modified the instructional text within the Claude Desktop "Manual Configuration" dropdown.
  • Changed "Advanced Settings tab" to "Connect tab" to accurately reflect the UI layout.
  • Updated the specific action text to: "In Connect tab, change the Transport option from HTTP to stdio, then regenerate."

Testing/Screenshots/Recordings

Screenshot 2026-03-02 at 12 45 28

Documentation Updates

  • I have added/removed/modified tools or resources
  • If yes, I have updated all documentation files using:
    • The LLM prompt at tools/UPDATE_DOCS_PROMPT.md (recommended)
    • Manual updates following the guide at tools/UPDATE_DOCS.md

Related Issues

Fixes #850

Additional Notes

This was observed and tested on version 9.4.7 using Unity 6.3 LTS.

Summary by Sourcery

Documentation:

  • Update the Claude Desktop manual configuration snippet to direct users to the Connect tab for changing the transport from HTTP to stdio.

Summary by CodeRabbit

  • Documentation
    • Updated configuration instructions for Claude Desktop to provide clearer guidance on transport settings.

@sourcery-ai

sourcery-ai Bot commented Mar 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Updates the manual configuration guidance string for Claude Desktop to reference the correct UI tab and action for switching transport from HTTP to stdio.

File-Level Changes

Change Details Files
Corrected Claude Desktop manual configuration instructions to match the current UI layout and transport settings flow.
  • Updated the manual configuration snippet returned when HTTP transport is enabled to reference the Connect tab instead of Advanced Settings.
  • Rephrased the guidance to explicitly instruct changing the Transport option from HTTP to stdio and then regenerating.
  • Preserved the existing fallback to the base manual snippet when HTTP transport is not used.
 MCPForUnity/Editor/Clients/Configurators/ClaudeDesktopConfigurator.cs

Assessment against linked issues

Issue Objective Addressed Explanation
#850 Update the Claude Desktop manual configuration instructions so that the transport settings location matches the current UI (referencing the Connect tab instead of the Advanced tab). ✅

Possibly linked issues

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@coderabbitai

coderabbitai Bot commented Mar 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 80e522f and d18e1aa.

📒 Files selected for processing (1)
  • MCPForUnity/Editor/Clients/Configurators/ClaudeDesktopConfigurator.cs
📝 Walkthrough

Walkthrough

A single text update corrects the manual configuration instructions for Claude Desktop, changing the referenced tab location from "Advanced Settings" to "Connect" for the transport option selection.

Changes

Cohort / File(s) Summary
Documentation Update
MCPForUnity/Editor/Clients/Configurators/ClaudeDesktopConfigurator.cs		 Updated GetManualSnippet() guidance text to reference the "Connect" tab instead of "Advanced Settings" for the HTTP transport configuration option.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tab was missed, a label wrong,
Connect, not Advanced all along!
With one small line, the path is clear,
Configuration's location, shining here.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: correcting an incorrect tab reference in Claude Desktop setup documentation.
Description check ✅ Passed The description fully covers required sections including a clear description of the issue, type of change (bug fix and documentation update), specific changes made, screenshots, and related issue reference.
Linked Issues check ✅ Passed The pull request directly addresses issue #850 by updating the instructional text from 'Advanced Settings tab' to 'Connect tab' and specifying the exact action needed, fulfilling the primary objective.
Out of Scope Changes check ✅ Passed All changes are directly related to fixing the incorrect tab reference identified in issue #850; no out-of-scope modifications are present.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

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

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

sourcery-ai[bot]
sourcery-ai Bot reviewed Mar 2, 2026
View reviewed changes

@sourcery-ai sourcery-ai 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.

Hey - I've left some high level feedback:

  • Consider tweaking the message string to read more naturally (e.g., "In the Connect tab, change the Transport option…") so the instruction is grammatically clear to users.
Prompt for AI Agents 
Please address the comments from this code review:

## Overall Comments
- Consider tweaking the message string to read more naturally (e.g., "In the Connect tab, change the Transport option…") so the instruction is grammatically clear to users.
Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@Scriptwonder Scriptwonder merged commit f375986 into CoplayDev:beta Mar 3, 2026
2 checks passed
@Lulubellelll Lulubellelll deleted the fix/claude-config-text branch March 3, 2026 15:17
@coderabbitai coderabbitai Bot mentioned this pull request Mar 11, 2026
Closed
@coderabbitai coderabbitai Bot mentioned this pull request May 22, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sourcery-ai sourcery-ai[bot] sourcery-ai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[UI/Docs] Transport settings location is in 'Connect' tab, not 'Advanced'

2 participants

@Lulubellelll @Scriptwonder