Skip to content

fix: Standardize SC4S MCP Tool Naming and Set Up MCP Server#3129

Open
digrajpu wants to merge 4 commits into
feat/mcp-feature-branchfrom
fix/mcp_tooling_name_changes
Open

fix: Standardize SC4S MCP Tool Naming and Set Up MCP Server#3129
digrajpu wants to merge 4 commits into
feat/mcp-feature-branchfrom
fix/mcp_tooling_name_changes

Conversation

@digrajpu

@digrajpu digrajpu commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@digrajpu digrajpu requested a review from rjha-splunk as a code owner July 1, 2026 16:58
Comment thread docs/mcp_server/tools.md Outdated
| `sc4s_health()` | Returns the health payload from the SC4S management API. Use this first when troubleshooting. |
| `sc4s_get_env()` | Reads the current `env_file` from the running SC4S instance. |
| `sc4s_set_env(env_file_content)` | Uploads a new `env_file`. The SC4S API backs up the previous file, applies the new one, and restarts `syslog-ng`. On validation failure the previous file is restored. |
| `health()` | Returns the health payload from the SC4S management API. Use this first when troubleshooting. |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

here sc4s_health

Comment thread sc4s_mcp/prompts/workflows.py Outdated
1. First call `sc4s_health` to check the instance status.
2. Call `sc4s_get_env` to review the current configuration.
3. Call `sc4s_list_custom_parsers` to see deployed custom parsers.
1. First call `health` to check the instance status.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

sc4s_health

Comment thread sc4s_mcp/tests/test_config_tools.py Outdated
sc4s_delete_parser,
sc4s_list_custom_parsers,
sc4s_get_custom_parser,
health,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

sc4s_health

Comment thread sc4s_mcp/tests/test_config_tools.py Outdated
def test_sc4s_health(mock_req):
sc4s_health()
def test_health(mock_req):
health()

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

sc4s_health

Comment thread sc4s_mcp/tests/test_config_tools.py Outdated
@patch("tools.configuration_tools._sc4s_request")
def test_sc4s_health(mock_req):
sc4s_health()
def test_health(mock_req):

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

here as well

Comment thread sc4s_mcp/tools/configuration_tools.py Outdated

@mcp.tool
def sc4s_health() -> dict:
def health() -> dict:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

sc4s_health

@digrajpu digrajpu requested a review from Kawron July 2, 2026 07:36
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.

2 participants