This is an automated email from the ASF dual-hosted git repository.

terrymanu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 3cfde6a2596 Update MCP troubleshooting follow-up items (#38856)
3cfde6a2596 is described below

commit 3cfde6a259663ee2727d5f987bf183f0ee1fb855
Author: Liang Zhang <[email protected]>
AuthorDate: Wed Jun 17 04:15:44 2026 +0800

    Update MCP troubleshooting follow-up items (#38856)
    
    Remove the resolved empty runtimeDatabases startup policy item from the
    ShardingSphere-MCP troubleshooting follow-up code improvement list.
    Keep the remaining unresolved improvement items unchanged.
---
 .../shardingsphere-mcp/troubleshooting.cn.md            | 17 ++++++++---------
 .../shardingsphere-mcp/troubleshooting.en.md            | 17 ++++++++---------
 2 files changed, 16 insertions(+), 18 deletions(-)

diff --git 
a/docs/document/content/user-manual/shardingsphere-mcp/troubleshooting.cn.md 
b/docs/document/content/user-manual/shardingsphere-mcp/troubleshooting.cn.md
index d375dc3718d..a395eb2a266 100644
--- a/docs/document/content/user-manual/shardingsphere-mcp/troubleshooting.cn.md
+++ b/docs/document/content/user-manual/shardingsphere-mcp/troubleshooting.cn.md
@@ -60,15 +60,14 @@ weight = 7
 
 以下内容记录当前文档已明确但仍需要后续源码演进的点。代码完成前,用户仍以现有工具名、错误分类和运行时行为为准。
 
-| 改进项                       | 原因                                               
                                            | 建议                                
                                                                                
    |
-|---------------------------|----------------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------------------------------------------|
-| 连接预检工具命名                  | 现有工具名 
`database_gateway_validate_proxy_connectivity` 容易让用户误以为只能校验 
Proxy,但实际校验的是已配置的运行时数据库连接。 | 后续改为更中性的名称,例如 
`database_gateway_validate_runtime_database` 或 
`database_gateway_validate_connectivity`,并同步调整描述符、测试和文档。 |
-| 运行状态中的错误分类                | 连接错误分类包含 
`database_not_visible`,但运行状态资源的安全分类列表尚未完全对齐。                                    
    | 后续在运行状态资源中补齐该分类,保证排障分类和运行时状态一致。                                           
                                            |
-| HTTP 安全错误分类               | Origin 请求头和会话归属不一致时,目前主要通过 HTTP 状态码和日志定位。        
                                            | 后续增加稳定、非敏感的错误分类,例如 
`origin_not_allowed` 和 `session_attribution_mismatch`,方便 AI 应用和管理员按错误码排查。       
                   |
-| 空 `runtimeDatabases` 启动策略 | 当前启动配置要求 `runtimeDatabases` 非空,缺失或为空会导致启动校验失败。   
                                            | 如果后续希望支持先启动 Server 再补充运行时数据库,需要调整 
YAML 校验、运行状态、启动日志和文档;如果仍要求非空,则保持当前启动失败行为,并继续让错误信息指向配置修正。                        
    |
-| 查询失败错误分类                  | 查询失败可能来自 SQL 
语法、对象不存在、权限不足、连接中断或超时,目前用户仍需要结合错误响应和日志判断。                                       
| 后续为查询执行失败补充稳定、非敏感的分类和处理建议,例如语法错误、对象不可见、权限不足和执行超时,并避免暴露原始连接信息或堆栈。              
                                        |
-| 元数据不可见错误分类                | 看不到数据库、模式、表、列或索引时,用户需要从配置、权限和连接目标中自行排查。          
                                            | 
后续在元数据读取和搜索结果中返回更细的可见性分类,例如数据库不可见、模式不可见、对象不可见或权限不足,并给出下一步检查项。                   
                                      |
-| 敏感值引用机制                   | 规则变更可能需要密钥或凭证,目前由外部受控通道传递。                       
                                            | 后续如需内置支持,应设计密钥引用或解析机制,而不是让 MCP 
Server 直接暴露敏感值。                                                                 
       |
+| 改进项         | 原因                                                             
                              | 建议                                              
                                                                      |
+|-------------|----------------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------------------------------------------|
+| 连接预检工具命名    | 现有工具名 `database_gateway_validate_proxy_connectivity` 
容易让用户误以为只能校验 Proxy,但实际校验的是已配置的运行时数据库连接。 | 后续改为更中性的名称,例如 
`database_gateway_validate_runtime_database` 或 
`database_gateway_validate_connectivity`,并同步调整描述符、测试和文档。 |
+| 运行状态中的错误分类  | 连接错误分类包含 `database_not_visible`,但运行状态资源的安全分类列表尚未完全对齐。          
                              | 后续在运行状态资源中补齐该分类,保证排障分类和运行时状态一致。                 
                                                                      |
+| HTTP 安全错误分类 | Origin 请求头和会话归属不一致时,目前主要通过 HTTP 状态码和日志定位。                      
                              | 后续增加稳定、非敏感的错误分类,例如 `origin_not_allowed` 和 
`session_attribution_mismatch`,方便 AI 应用和管理员按错误码排查。                          |
+| 查询失败错误分类    | 查询失败可能来自 SQL 语法、对象不存在、权限不足、连接中断或超时,目前用户仍需要结合错误响应和日志判断。         
                              | 
后续为查询执行失败补充稳定、非敏感的分类和处理建议,例如语法错误、对象不可见、权限不足和执行超时,并避免暴露原始连接信息或堆栈。                
                                      |
+| 元数据不可见错误分类  | 看不到数据库、模式、表、列或索引时,用户需要从配置、权限和连接目标中自行排查。                        
                              | 
后续在元数据读取和搜索结果中返回更细的可见性分类,例如数据库不可见、模式不可见、对象不可见或权限不足,并给出下一步检查项。                   
                                      |
+| 敏感值引用机制     | 规则变更可能需要密钥或凭证,目前由外部受控通道传递。                                     
                              | 后续如需内置支持,应设计密钥引用或解析机制,而不是让 MCP Server 直接暴露敏感值。  
                                                                      |
 
 ## 提供给管理员或排障人员的信息
 
diff --git 
a/docs/document/content/user-manual/shardingsphere-mcp/troubleshooting.en.md 
b/docs/document/content/user-manual/shardingsphere-mcp/troubleshooting.en.md
index 3c2d12a4ad6..cae0e36f793 100644
--- a/docs/document/content/user-manual/shardingsphere-mcp/troubleshooting.en.md
+++ b/docs/document/content/user-manual/shardingsphere-mcp/troubleshooting.en.md
@@ -60,15 +60,14 @@ When a runtime database or ShardingSphere-Proxy connection 
fails, MCP responses
 
 The following items are documented as known follow-up code improvements. Until 
the code changes are completed, users should rely on the current tool names, 
error categories, and runtime behavior.
 
-| Improvement                             | Reason                             
                                                                                
                                                                                
           | Recommendation                                                     
                                                                                
                                                                                
              [...]
-|-----------------------------------------|---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 [...]
-| Connectivity validation tool name       | The current tool name 
`database_gateway_validate_proxy_connectivity` can imply that only Proxy is 
validated, but the tool actually validates a configured runtime database 
connection.                        | Rename it to a more neutral name, such as 
`database_gateway_validate_runtime_database` or 
`database_gateway_validate_connectivity`, and update descriptors, tests, and 
documentation together.                                                   [...]
-| Error categories in runtime status      | The connection failure categories 
include `database_not_visible`, but the safe category list in the runtime 
status resource is not fully aligned.                                           
                  | Add this category to the runtime status resource so 
troubleshooting categories and runtime status stay consistent.                  
                                                                                
                             [...]
-| HTTP security error categories          | Origin header and session 
attribution failures are currently located mainly through HTTP status codes and 
logs.                                                                           
                    | Add stable, non-sensitive error categories such as 
`origin_not_allowed` and `session_attribution_mismatch`, so AI applications and 
administrators can troubleshoot by category.                                    
                              [...]
-| Empty `runtimeDatabases` startup policy | The current launch configuration 
requires `runtimeDatabases` to be non-empty. A missing or empty value fails 
startup validation.                                                             
                 | If startup without runtime databases should be supported 
later, adjust YAML validation, runtime status, startup logs, and documentation 
together. If the non-empty requirement remains, keep startup failure behavior 
and continue directing use [...]
-| Query failure error categories          | Query failures can come from SQL 
syntax, invisible objects, insufficient privileges, interrupted connections, or 
timeout. Users currently need to combine the error response with logs to 
identify the cause. | Add stable, non-sensitive categories and actions for 
query execution failures, such as syntax error, object not visible, 
insufficient privileges, and execution timeout, without exposing raw connection 
details or stack traces.                [...]
-| Metadata visibility error categories    | When databases, schemas, tables, 
columns, or indexes are not visible, users need to inspect configuration, 
privileges, and connection target manually.                                     
                   | Return finer-grained visibility categories from metadata 
read and search results, such as database not visible, schema not visible, 
object not visible, or insufficient privileges, with the next checks to 
perform.                             [...]
-| Secret reference mechanism              | Rule changes may require keys or 
credentials, which are currently passed through external protected channels.    
                                                                                
             | If built-in support is required later, design a secret reference 
or resolver mechanism instead of exposing sensitive values through the MCP 
Server.                                                                         
                     [...]
+| Improvement                          | Reason                                
                                                                                
                                                                                
        | Recommendation                                                        
                                                                                
                                                                                
       |
+|--------------------------------------|---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
+| Connectivity validation tool name    | The current tool name 
`database_gateway_validate_proxy_connectivity` can imply that only Proxy is 
validated, but the tool actually validates a configured runtime database 
connection.                        | Rename it to a more neutral name, such as 
`database_gateway_validate_runtime_database` or 
`database_gateway_validate_connectivity`, and update descriptors, tests, and 
documentation together.                                               |
+| Error categories in runtime status   | The connection failure categories 
include `database_not_visible`, but the safe category list in the runtime 
status resource is not fully aligned.                                           
                  | Add this category to the runtime status resource so 
troubleshooting categories and runtime status stay consistent.                  
                                                                                
                         |
+| HTTP security error categories       | Origin header and session attribution 
failures are currently located mainly through HTTP status codes and logs.       
                                                                                
        | Add stable, non-sensitive error categories such as 
`origin_not_allowed` and `session_attribution_mismatch`, so AI applications and 
administrators can troubleshoot by category.                                    
                          |
+| Query failure error categories       | Query failures can come from SQL 
syntax, invisible objects, insufficient privileges, interrupted connections, or 
timeout. Users currently need to combine the error response with logs to 
identify the cause. | Add stable, non-sensitive categories and actions for 
query execution failures, such as syntax error, object not visible, 
insufficient privileges, and execution timeout, without exposing raw connection 
details or stack traces.            |
+| Metadata visibility error categories | When databases, schemas, tables, 
columns, or indexes are not visible, users need to inspect configuration, 
privileges, and connection target manually.                                     
                   | Return finer-grained visibility categories from metadata 
read and search results, such as database not visible, schema not visible, 
object not visible, or insufficient privileges, with the next checks to 
perform.                         |
+| Secret reference mechanism           | Rule changes may require keys or 
credentials, which are currently passed through external protected channels.    
                                                                                
             | If built-in support is required later, design a secret reference 
or resolver mechanism instead of exposing sensitive values through the MCP 
Server.                                                                         
                 |
 
 ## Information for Administrators or Troubleshooters
 

Reply via email to