feat: mock record/replay and mcp support by anjupathak03 · Pull Request #3714 · keploy/keploy

anjupathak03 · 2026-02-03T17:14:19Z

Describe the changes that are made

This pull request introduces a new mcp (Model Context Protocol) command to the CLI, enabling Keploy to be used as an MCP server for AI assistant integrations (e.g., VS Code, Claude Desktop). It also refactors the mock command structure, adds dedicated mock record and mock test subcommands, and improves command flag handling, logging, and MCP-specific behaviors for robust stdio-based operation.

The most important changes are:

MCP Server Integration

Added a new cli/mcp.go file implementing the mcp command with a serve subcommand, which starts a Keploy MCP server over stdio using JSON-RPC 2.0, with robust logging to stderr and log files, and explicit handling to prevent stdout pollution. This enables integration with AI assistants that support MCP.
Introduced a new logger (newMCPLogger) that outputs only structured JSON logs to stderr (and optionally to file), with no ANSI color codes, to comply with MCP protocol requirements.

Mock Command Enhancements

Refactored the mock command to add dedicated mock record and mock test subcommands, each with their own flag handling and logic, making it easier to record and replay mocks via CLI. [1] [2]
Added detailed implementations for MockRecord and MockTest commands, including argument parsing, service initialization, and structured logging of results.

Command Flag and Validation Improvements

Enhanced CmdConfigurator to add flags contextually for the new mock record and mock test subcommands, and improved normalization for new flags (e.g., mock-name). [1] [2]
Introduced the isMCPStdioCommand helper and MCPStdioAnnotationKey constant to identify MCP commands and adjust validation, error handling, and logger configuration accordingly. [1] [2] [3]
Ensured that ANSI color output and logo printing are disabled for MCP commands to avoid corrupting the JSON-RPC stream, and that logger behavior is adjusted for MCP and non-MCP commands. [1] [2]

Logging and Output Handling

Improved logger configuration to ensure that, for MCP commands, all logs go to stderr in structured JSON format, and that the logger is properly initialized at the correct level based on debug flags. [1] [2] [3]
Added logic to save both stdout (JSON-RPC messages) and stderr (logs) to timestamped log files for debugging, which is crucial when integrating with AI assistants that consume these streams.

General Codebase Improvements

Updated import statements and code structure to support new features and maintain code clarity.

These changes significantly enhance Keploy's CLI by enabling robust AI assistant integrations, improving mock management, and ensuring reliable, protocol-compliant operation for MCP scenarios.

Links & References

Closes: #[issue number that will be closed through this PR]

NA (if very small change like typo, linting, etc.)

🔗 Related PRs

NA

🐞 Related Issues

NA

📄 Related Documents

NA

What type of PR is this? (check all applicable)

Added e2e test pipeline?

👍 yes
🙅 no, because they aren't needed
🙋 no, because I need help

Added comments for hard-to-understand areas?

👍 yes
🙅 no, because the code is self-explanatory

Added to documentation?

📜 README.md
📓 Wiki
🙅 no documentation needed

Are there any sample code or steps to test the changes?

👍 yes, mentioned below
🙅 no, because it is not needed

Self Review done?

✅ yes
❌ no, because I need help

Any relevant screenshots, recordings or logs?

NA

🧠 Semantics for PR Title & Branch Name

Please ensure your PR title and branch name follow the Keploy semantics:

📌 PR Semantics Guide
📌 Branch Semantics Guide

Examples:

PR Title: fix: patch MongoDB document update bug
Branch Name: feat/#1-login-flow (You may skip mentioning the issue number in the branch name if the change is small and the PR description clearly explains it.)

Additional checklist:

Have you read the Contributing Guidelines on issues?
Have you followed the PR Semantics guide for naming this PR?
Have you followed the Branch Semantics guide for naming your branch?

commit 1e8a6aa Author: Anju Pathak <anjupathak9810@gmail.com> Date: Tue Feb 3 16:48:19 2026 +0000 fix: build Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit ddc79d6 Merge: aca4f56 3abdd5f Author: Anju Pathak <anjupathak9810@gmail.com> Date: Tue Feb 3 16:47:21 2026 +0000 Merge branch 'main' into mcp-server-pipeline commit aca4f56 Author: Anju <anjupathak9810@gmail.com> Date: Fri Jan 30 05:41:52 2026 +0530 updated the automatic pipeline creation Signed-off-by: Anju <anjupathak9810@gmail.com> commit 3af87bf Author: Anju <anjupathak9810@gmail.com> Date: Thu Jan 29 16:23:15 2026 +0530 feat(mcp): add keploy_manager tool with pipeline generation Signed-off-by: Anju <anjupathak9810@gmail.com> commit 4ed8cc9 Merge: da2d32a fdf5d57 Author: Anju <anjupathak9810@gmail.com> Date: Tue Jan 27 23:33:20 2026 +0530 Merge branch 'fix-mysql-parser' into mock-record-replay-mcp commit da2d32a Merge: 726de11 5bcc7ab Author: Anju <anjupathak9810@gmail.com> Date: Tue Jan 27 23:30:19 2026 +0530 Merge branch 'main' into mock-record-replay-mcp commit fdf5d57 Author: Anju <anjupathak9810@gmail.com> Date: Tue Jan 27 17:49:35 2026 +0530 fix: lint Signed-off-by: Anju <anjupathak9810@gmail.com> commit 51842f2 Author: Anju <anjupathak9810@gmail.com> Date: Tue Jan 27 17:46:27 2026 +0530 fix: lint Signed-off-by: Anju <anjupathak9810@gmail.com> commit 59d3f07 Author: Anju <anjupathak9810@gmail.com> Date: Tue Jan 27 17:37:11 2026 +0530 fix: date/time binary length handling Signed-off-by: Anju <anjupathak9810@gmail.com> commit 272110c Author: Anju <anjupathak9810@gmail.com> Date: Tue Jan 27 17:34:38 2026 +0530 fix: HandshakeResponse41 auth-response decoding Signed-off-by: Anju <anjupathak9810@gmail.com> commit da9ac85 Author: Anju <anjupathak9810@gmail.com> Date: Tue Jan 27 17:26:53 2026 +0530 fix: Correctly decode 0x02 as auth or COM_INIT_DB based on context Signed-off-by: Anju <anjupathak9810@gmail.com> commit 726de11 Author: Anju <anjupathak9810@gmail.com> Date: Fri Jan 23 16:05:49 2026 +0530 fix: MCP path resolution and use independent context for server lifecycle Signed-off-by: Anju <anjupathak9810@gmail.com> commit a4433bb Author: Anju <anjupathak9810@gmail.com> Date: Fri Jan 23 15:39:36 2026 +0530 fix: MySQL read timeout and og actual SQL for prepared statements Signed-off-by: Anju <anjupathak9810@gmail.com> commit 151ba6b Author: Anju <anjupathak9810@gmail.com> Date: Wed Jan 21 18:13:21 2026 +0530 fix: improved the debugging experience during mock replay Signed-off-by: Anju <anjupathak9810@gmail.com> commit 672ec6c Author: Anju <anjupathak9810@gmail.com> Date: Wed Jan 21 18:10:12 2026 +0530 fix: console output interleaving Signed-off-by: Anju <anjupathak9810@gmail.com> commit afef355 Author: Anju <anjupathak9810@gmail.com> Date: Wed Jan 21 18:07:50 2026 +0530 fix: --duration flag working for the mock record command and duplicate logo Signed-off-by: Anju <anjupathak9810@gmail.com> commit 35ffa13 Author: Anju <anjupathak9810@gmail.com> Date: Wed Jan 21 18:01:26 2026 +0530 fix: Suppress Windows sharing violation on delete, avoid logging to a closed log file. Signed-off-by: Anju <anjupathak9810@gmail.com> commit f7832f5 Author: Anju <anjupathak9810@gmail.com> Date: Wed Jan 21 17:58:49 2026 +0530 fix: Treat TLS close‑notify and use of closed network connection as benign. Signed-off-by: Anju <anjupathak9810@gmail.com> commit c163133 Author: Anju <anjupathak9810@gmail.com> Date: Wed Jan 21 17:43:05 2026 +0530 fix: No goroutine blocking, Both success and error cases are communicated through the channel Signed-off-by: Anju <anjupathak9810@gmail.com> commit 0525b8b Author: Anju <anjupathak9810@gmail.com> Date: Wed Jan 21 17:31:43 2026 +0530 fix: resolved Windows incoming traffic interception issue Signed-off-by: Anju <anjupathak9810@gmail.com> commit 4671660 Author: Anju Pathak <anjupathak9810@gmail.com> Date: Tue Jan 20 13:56:18 2026 +0530 fix: merge conflict Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit 9952088 Merge: b82f4d9 36d1a98 Author: Anju <anjupathak9810@gmail.com> Date: Tue Jan 20 13:34:53 2026 +0530 Merge branch 'main' into mock-record-replay-mcp commit b82f4d9 Author: Anju Pathak <anjupathak9810@gmail.com> Date: Thu Jan 15 11:57:06 2026 +0530 Revert "fix(windows): app exit signal not propagating to agent" This reverts commit 2fcf4f2. commit 12c6c1f Author: Anju <anjupathak9810@gmail.com> Date: Wed Jan 7 16:40:14 2026 +0530 fix(mcp): add log files, optional params & fs-based mock listing Signed-off-by: Anju <anjupathak9810@gmail.com> commit 86e8cbc Author: Anju <anjupathak9810@gmail.com> Date: Tue Jan 6 14:01:33 2026 +0530 fix: mcp serve command, suppress logo output for JSON-RPC Signed-off-by: Anju <anjupathak9810@gmail.com> commit 157ae0f Author: Anju Pathak <anjupathak9810@gmail.com> Date: Mon Jan 5 07:48:26 2026 +0000 docs(mcp): Update API reference, architecture, and user guide for protocol version and path descriptions Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit 2fcf4f2 Author: Anju Pathak <anjupathak9810@gmail.com> Date: Mon Jan 5 07:41:57 2026 +0000 fix(windows): app exit signal not propagating to agent Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit e2573d3 Author: Anju Pathak <anjupathak9810@gmail.com> Date: Mon Jan 5 07:21:36 2026 +0000 feat(mcp): Enhance MCP server for JSON-RPC compliance and add list mocks tool - Updated MCP server to ensure JSON-RPC 2.0 compliance by enforcing output to stderr and avoiding ANSI codes. - Introduced a new logger specifically for MCP that adheres to the compliance requirements. - Added tool to list available mock sets, improving user interaction and mock discovery. - Enhanced mock record and replay tools with better configuration handling and user feedback. - Refactored output handling in the agent to support MCP stdio mode. - Removed outdated README files for mock recording and replay services. - Added utility functions to manage MCP stdio mode state. Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit 4bb3df9 Author: Anju Pathak <anjupathak9810@gmail.com> Date: Mon Jan 5 04:39:09 2026 +0000 Refactor mock replay and recording services - Simplified the replayer service by removing unnecessary dependencies and integrating runtime configuration. - Introduced a new Runtime interface to provide shared state access for replay operations. - Removed the AgentService and MockDB interfaces from the replayer, consolidating functionality. - Streamlined the Replay method to focus on essential operations, delegating mock loading and command execution. - Added a new command package to infer container names from Docker commands. - Removed the mock_record.go file and replaced its functionality with a more modular approach in the record service. - Enhanced the Start method in the Recorder to support flexible recording options and improved error handling. - Updated the service interfaces to reflect the changes in the recording and replay logic. - Introduced new options for recording sessions, allowing for better customization and control over the recording process. Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit 0c11e0b Author: Anju Pathak <anjupathak9810@gmail.com> Date: Mon Jan 5 01:05:51 2026 +0000 refactor to reuse record with a mock‑only mode. Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit cc63787 Merge: 29324ff 1623cb1 Author: Anju Pathak <anjupathak9810@gmail.com> Date: Fri Jan 2 07:18:10 2026 +0000 Merge branch 'feat/windows-redirect' into mock-record-replay-mcp commit 29324ff Author: Anju Pathak <anjupathak9810@gmail.com> Date: Fri Jan 2 07:11:23 2026 +0000 feat: Implement mock recording and replay services - Added `mockrecord` service for capturing and storing outgoing calls during application execution. - Introduced `MockRecordInput` and `MockRecordOutput` types for handling mock recording requests and responses. - Implemented `handleMockRecord` function to manage the recording process, including command execution and mock storage. - Created `mockreplay` service for replaying recorded mocks during testing. - Added `MockReplayInput` and `MockReplayOutput` types for managing mock replay requests and responses. - Implemented `handleMockReplay` function to handle the replay process, including loading mocks and executing application commands. - Developed metadata extraction logic for contextual naming of recorded mocks. - Added YAML parsing for loading mocks from files, supporting both file and directory paths. - Enhanced logging for better traceability during recording and replay processes. - Updated README files for both services to provide usage instructions and architectural overview. Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit 1623cb1 Author: Anju Pathak <anjupathak9810@gmail.com> Date: Mon Dec 22 00:19:07 2025 +0000 fix: add private parsers Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit 0a66246 Author: Anju Pathak <anjupathak9810@gmail.com> Date: Sun Dec 21 20:30:04 2025 +0000 fix: use local mongo Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit 1c85b49 Author: Anju <anjupathak9810@gmail.com> Date: Mon Dec 22 01:47:26 2025 +0530 fix(windows): resolve path issues Signed-off-by: Anju <anjupathak9810@gmail.com> commit 09e1d6a Author: Anju <anjupathak9810@gmail.com> Date: Sun Dec 21 22:40:12 2025 +0530 fix(windows): resolve path issues for native tests Signed-off-by: Anju <anjupathak9810@gmail.com> commit 587421c Author: Anju <anjupathak9810@gmail.com> Date: Sun Dec 21 22:13:26 2025 +0530 fix(windows): resolve path issues for native tests Signed-off-by: Anju <anjupathak9810@gmail.com> commit f102417 Author: Anju <anjupathak9810@gmail.com> Date: Sun Dec 21 21:46:09 2025 +0530 fix(windows): resolve path issues for native tests Signed-off-by: Anju <anjupathak9810@gmail.com> commit 5cfbacd Author: Anju <anjupathak9810@gmail.com> Date: Sun Dec 21 21:18:39 2025 +0530 fix: pre-build to avoid timeout Signed-off-by: Anju <anjupathak9810@gmail.com> commit 4ba0ab1 Author: Anju <anjupathak9810@gmail.com> Date: Sun Dec 21 21:00:31 2025 +0530 fix: pre-build to avoid timeout Signed-off-by: Anju <anjupathak9810@gmail.com> commit 809edc9 Merge: d21e555 51ff0e5 Author: Anju <anjupathak9810@gmail.com> Date: Sun Dec 21 20:01:31 2025 +0530 Merge branch 'main' into feat/windows-redirect commit d21e555 Author: Anju <anjupathak9810@gmail.com> Date: Sat Dec 20 03:06:43 2025 +0530 fix(ci): enhance gin-mongo Windows test script for GitHub Actions Signed-off-by: Anju <anjupathak9810@gmail.com> commit a27cbce Author: Anju <168076172+anjupathak03@users.noreply.github.com> Date: Fri Dec 19 04:16:16 2025 +0530 Update pkg/agent/hooks/windows/redirector.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Anju <168076172+anjupathak03@users.noreply.github.com> commit 0d55523 Author: Anju <anjupathak9810@gmail.com> Date: Fri Dec 19 04:05:59 2025 +0530 fix: Windows workflow syntax errors Signed-off-by: Anju <anjupathak9810@gmail.com> commit 48038c9 Author: Anju <anjupathak9810@gmail.com> Date: Fri Dec 19 03:43:31 2025 +0530 fix: Windows workflow syntax errors Signed-off-by: Anju <anjupathak9810@gmail.com> commit 70a90ac Author: Anju <anjupathak9810@gmail.com> Date: Fri Dec 19 03:40:48 2025 +0530 fix: Windows workflow syntax errors Signed-off-by: Anju <anjupathak9810@gmail.com> commit 9ad4a87 Author: Anju <anjupathak9810@gmail.com> Date: Fri Dec 19 03:31:48 2025 +0530 feat: added pipeline native Windows support Signed-off-by: Anju <anjupathak9810@gmail.com> commit 620ea47 Author: Anju <168076172+anjupathak03@users.noreply.github.com> Date: Fri Dec 19 02:47:07 2025 +0530 Remove Docker image pull step from workflow Removed the step to pull Docker images from the workflow. Signed-off-by: Anju <168076172+anjupathak03@users.noreply.github.com> commit a3e443a Author: Anju <anjupathak9810@gmail.com> Date: Fri Dec 19 02:32:01 2025 +0530 fix: Windows workflow syntax errors Signed-off-by: Anju <anjupathak9810@gmail.com> commit a1885f6 Author: Anju <anjupathak9810@gmail.com> Date: Fri Dec 19 01:20:40 2025 +0530 fix: Windows workflow syntax errors Signed-off-by: Anju <anjupathak9810@gmail.com> commit b6cae49 Author: Anju <anjupathak9810@gmail.com> Date: Fri Dec 19 01:10:37 2025 +0530 fix: Windows workflow syntax errors Signed-off-by: Anju <anjupathak9810@gmail.com> commit d77f79d Author: Anju <anjupathak9810@gmail.com> Date: Fri Dec 19 00:53:35 2025 +0530 fix: Windows workflow syntax errors Signed-off-by: Anju <anjupathak9810@gmail.com> commit b4f2a6b Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 20:39:16 2025 +0530 fix: Windows workflow syntax errors Signed-off-by: Anju <anjupathak9810@gmail.com> commit 2affc7c Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 20:05:48 2025 +0530 fix: Windows workflow syntax errors Signed-off-by: Anju <anjupathak9810@gmail.com> commit eb88853 Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 19:48:27 2025 +0530 fix: typo error (gin-mongo) Signed-off-by: Anju <anjupathak9810@gmail.com> commit f9643a4 Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 19:41:54 2025 +0530 ci: add native Windows workflow (gin-mongo) Signed-off-by: Anju <anjupathak9810@gmail.com> commit 6cef159 Merge: c223d17 c832f0e Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 17:14:57 2025 +0530 Merge remote changes from feat/windows-redirect commit c223d17 Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 17:10:38 2025 +0530 fix: Added MinGW gcc to PATH Signed-off-by: Anju <anjupathak9810@gmail.com> commit c832f0e Merge: 9841dc2 d99ebae Author: Anju <168076172+anjupathak03@users.noreply.github.com> Date: Thu Dec 18 17:00:56 2025 +0530 Merge branch 'main' into feat/windows-redirect commit 9841dc2 Merge: 8212ced 53d5c11 Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 16:59:22 2025 +0530 Merge remote-tracking branch 'origin/feat/windows-redirect' into feat/windows-redirect commit 8212ced Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 16:56:01 2025 +0530 fix: add CGO toolchain sanity check Signed-off-by: Anju <anjupathak9810@gmail.com> commit 53d5c11 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Thu Dec 18 15:57:21 2025 +0530 chore: remove integrations repo and sys file from the root Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit c4cb270 Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 15:06:40 2025 +0530 ci: added native Windows testing workflow on GitHub-hosted runners Signed-off-by: Anju <anjupathak9810@gmail.com> commit a7bae92 Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 03:47:14 2025 +0530 feat: Added Windows certificate installation support using certutil Signed-off-by: Anju <anjupathak9810@gmail.com> commit 8f0719b Merge: 0815003 ac9e9c2 Author: Anju <anjupathak9810@gmail.com> Date: Thu Dec 18 03:41:37 2025 +0530 Merge remote-tracking branch 'origin/feat/windows-redirect' into feat/windows-redirect commit ac9e9c2 Merge: ef6b7ba ae5d167 Author: Sarthak Shyngle <50234097+Sarthak160@users.noreply.github.com> Date: Wed Dec 17 06:45:12 2025 -0800 Merge branch 'main' into feat/windows-redirect commit 0815003 Merge: 40f35b3 ef6b7ba Author: Anju <anjupathak9810@gmail.com> Date: Wed Dec 17 19:13:16 2025 +0530 Merge branch 'feat/windows-redirect' of github.com:keploy/keploy into feat/windows-redirect commit ef6b7ba Merge: 50ccd8e 86ad97c Author: Anju <168076172+anjupathak03@users.noreply.github.com> Date: Wed Dec 17 19:11:55 2025 +0530 Merge branch 'main' into feat/windows-redirect commit 40f35b3 Merge: 4ae2768 50ccd8e Author: Anju <anjupathak9810@gmail.com> Date: Wed Dec 17 19:08:41 2025 +0530 Merge branch 'feat/windows-redirect' of github.com:keploy/keploy into feat/windows-redirect commit 50ccd8e Author: Anju Pathak <anjupathak9810@gmail.com> Date: Wed Dec 17 13:18:14 2025 +0000 fix: resolve SYS driver file not found by extracting WinDivert to exe directory Signed-off-by: Anju Pathak <anjupathak9810@gmail.com> commit 4ae2768 Author: Anju <anjupathak9810@gmail.com> Date: Wed Dec 17 18:26:21 2025 +0530 fix: resolve SYS driver file not found by extracting WinDivert to exe directory Signed-off-by: Anju <anjupathak9810@gmail.com> commit b987eac Merge: 3fd5857 c1d437a Author: Anju <168076172+anjupathak03@users.noreply.github.com> Date: Tue Dec 16 13:08:46 2025 +0530 Merge branch 'main' into feat/windows-redirect commit 3fd5857 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Fri Dec 12 12:27:23 2025 +0530 fix: build issues Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 66a0851 Merge: b667c94 d157c5d Author: Ayush Sharma <kshitij3160@gmail.com> Date: Fri Dec 12 06:49:35 2025 +0000 Merge branch 'main' into feat/windows-redirect Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit b667c94 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Thu Nov 20 12:14:41 2025 +0000 feat: add dns support for windows Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 4683cb4 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Thu Nov 20 14:15:33 2025 +0530 fix: lint issues and enabled cgo for windows build Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit a4785da Author: Ayush Sharma <kshitij3160@gmail.com> Date: Thu Nov 20 08:37:41 2025 +0000 feat: do not run native on windows arm64 as windivert does not support Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit f511fdd Author: Ayush Sharma <kshitij3160@gmail.com> Date: Wed Nov 19 16:27:39 2025 +0000 feat: send dns port to the redirector Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 7c2fc3a Merge: 18bc102 c161070 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Wed Nov 19 16:06:57 2025 +0000 Merge remote-tracking branch 'origin/main' into feat/windows-redirect Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 18bc102 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Wed Nov 19 16:03:32 2025 +0000 feat: remove the pipelines code Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 54152c9 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Wed Nov 19 15:55:26 2025 +0000 feat: do cleanup of the previous approach Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit e719b38 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Wed Nov 19 15:45:57 2025 +0000 feat: use ffi functions for the go code Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 41af2c3 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 11 14:19:51 2025 +0530 feat: increase the size of body of event Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 60caf5a Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 17:24:47 2025 +0530 fix: keploy agent logs Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit b8a04d9 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 17:00:37 2025 +0530 fix: keploy agent logs Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 967da71 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 16:37:50 2025 +0530 fix: kill keploy in windows pipeline Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 022718f Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 15:10:21 2025 +0530 fix: kill keploy in windows pipeline Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 79b4630 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 13:38:40 2025 +0530 chore: debug agent logs Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit a7034d8 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 13:28:38 2025 +0530 fix: windows pipeline Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 7a0b5d9 Merge: a3ec7e1 6c77cd0 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 13:00:35 2025 +0530 Merge branch 'feat/windows-support-native' of https://github.com/keploy/keploy into feat/windows-support-native Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit a3ec7e1 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 12:59:50 2025 +0530 fix: run native pipeline Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 6c77cd0 Merge: 26758b1 abba501 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 12:45:21 2025 +0530 Merge branch 'main' into feat/windows-support-native commit 26758b1 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 12:43:35 2025 +0530 feat: add windows native pipeline Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 5b01b06 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Tue Nov 4 12:24:54 2025 +0530 fix: race condition Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 5d6289b Author: Ayush Sharma <kshitij3160@gmail.com> Date: Mon Nov 3 17:38:51 2025 +0530 feat: add windows redirector binary Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit b5f9958 Merge: adafe0e f5c619c Author: Ayush Sharma <kshitij3160@gmail.com> Date: Mon Nov 3 14:15:56 2025 +0530 Merge branch 'feat/windows-support-native' of https://github.com/keploy/keploy into feat/windows-support-native Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit f5c619c Merge: a1b5981 7b31c4e Author: Ayush Sharma <kshitij3160@gmail.com> Date: Mon Nov 3 14:14:54 2025 +0530 Merge branch 'main' into feat/windows-support-native commit adafe0e Author: Ayush Sharma <kshitij3160@gmail.com> Date: Mon Nov 3 14:14:29 2025 +0530 fix: app closing in windows Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit a1b5981 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Fri Oct 31 14:56:09 2025 +0530 fix: linux build Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit 7d04af6 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Fri Oct 31 14:51:23 2025 +0530 feat: make record work on windows Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit c3a87d8 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Fri Oct 31 13:52:04 2025 +0530 feat: make replay work for windows Signed-off-by: Ayush Sharma <kshitij3160@gmail.com> commit b123f92 Author: Ayush Sharma <kshitij3160@gmail.com> Date: Thu Oct 30 17:39:30 2025 +0530 feat: initialized windows support for windows Signed-off-by: Ayush Sharma <kshitij3160@gmail.com>

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

Copilot

Pull request overview

This PR adds comprehensive MCP (Model Context Protocol) server support to Keploy, enabling AI assistants to record and replay mocks through a JSON-RPC interface. The implementation includes new services for mock recording and replay, CLI commands, proper stdio/stderr handling for protocol compliance, and extensive documentation.

Changes:

Added MCP server implementation with three tools: list mocks, record mocks, and test/replay mocks
Created mockrecord and mockreplay services with configurable options
Fixed SQL delay conversion bug (multiplying by time.Second)
Enhanced MySQL query logging for better debugging
Added context management for long-lived MCP processes

Reviewed changes

Copilot reviewed 42 out of 44 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
pkg/mcp/*	MCP server implementation with tool handlers, naming logic, and protocol management
pkg/service/mockrecord/*	Mock recording service with metadata extraction
pkg/service/mockreplay/*	Mock replay service with project detection
cli/mcp.go	MCP CLI commands with stdio redirection
utils/ctx.go	New MCP context for signal-only cancellation
pkg/service/replay/replay.go	Fixed SQL delay conversion to seconds
pkg/agent/proxy/integrations/mysql/*	Enhanced MySQL logging and handshake protocol
docs/mcp/*	Comprehensive user guide, architecture, and API documentation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-03T17:18:17Z

cli/mcp.go

+			devNull, err := os.OpenFile(os.DevNull, os.O_WRONLY, 0)
+			if err != nil {
+				return err
+			}
+			defer devNull.Close()
+			os.Stdout = devNull


The os.Stdout redirection to /dev/null happens after capturing originalStdout, but if any code runs between lines 210 and 256 that writes to stdout, it will be lost. The MCP server initialization at line 256 depends on mcpStdoutWriter (line 260) which uses originalStdout. However, between lines 210-256, if serviceFactory.GetService or any initialization code writes to os.Stdout (which is now /dev/null), those writes are lost but don't corrupt the MCP stream. This is actually correct behavior, but consider adding a comment explaining this intentional redirection timing.

Copilot · 2026-02-03T17:18:17Z

pkg/platform/http/agent.go

 			utils.LogError(a.logger, appErr.Err, "error while running the app")
-			appErrCh <- appErr
 		}
+		appErrCh <- appErr


The error at line 591 is now always sent to appErrCh (line 592), even when appErr.Err is nil. Previously, it only sent to the channel when err was not nil (line 590 in old code). This means a successful app execution (appErr.Err == nil) will now send an error to the channel, which could cause incorrect error handling. The logic should check if appErr.Err != nil before sending to appErrCh.

Suggested change

}

appErrCh <- appErr

appErrCh <- appErr

}

Copilot · 2026-02-03T17:18:18Z

utils/ctx.go

+func NewMCPCtx() context.Context {
+	ctx, cancel := context.WithCancel(context.Background())
+
+	// Listen for termination signals and cancel MCP context only on those.
+	sigs := make(chan os.Signal, 1)
+	signal.Notify(sigs, os.Interrupt, syscall.SIGTERM)
+
+	go func() {
+		sig := <-sigs
+		fmt.Printf("MCP server: signal received: %s, shutting down...\n", sig)
+		cancel()
+	}()
+
+	return ctx
+}


The NewMCPCtx creates a context that only listens to OS signals (Ctrl+C, SIGTERM) but doesn't respect cancellation from the parent context passed to server.Run(). This means if the parent context is cancelled for any reason other than OS signals, the MCP server will continue running. While this might be intentional for keeping the server alive across operations, it could lead to resource leaks if the parent context is cancelled due to application shutdown. Consider whether the MCP context should also respect parent context cancellation in addition to OS signals.

Copilot · 2026-02-03T17:18:18Z

pkg/agent/proxy/util/util.go

+func IP4StrToUint32(ipStr string) (uint32, error) {
+	// Split the IP address into its four octets.
+	octets := strings.Split(ipStr, ".")
+	if len(octets) != 4 {
+		return 0, fmt.Errorf("invalid IPv4 address format")
+	}
+
+	// Parse each octet as an integer and shift it to its correct position in the uint32.
+	var ip uint32
+	for i, octet := range octets {
+		octetInt, err := strconv.ParseUint(octet, 10, 8)
+		if err != nil {
+			return 0, fmt.Errorf("invalid octet %q in IP address: %v", octet, err)
+		}
+		ip |= uint32(octetInt) << (8 * uint(3-i))
+	}
+
+	return ip, nil
+}
+
+func IP6StrToUint32(ipStr string) ([4]uint32, error) {
+	// Parse the IPv6 address string
+	ip := net.ParseIP(ipStr)
+	if ip == nil || ip.To16() == nil {
+		return [4]uint32{}, fmt.Errorf("invalid IPv6 address format")
+	}
+
+	// Convert to a 16-byte representation
+	ip = ip.To16()
+
+	// Prepare the array to hold four uint32 values
+	var result [4]uint32
+
+	// Convert each 4-byte segment to a uint32
+	for i := 0; i < 4; i++ {
+		result[i] = binary.BigEndian.Uint32(ip[i*4 : (i+1)*4])
+	}
+
+	return result, nil
+}


The IP4StrToUint32 and IP6StrToUint32 functions are added but there's no import for "binary" package which is used in IP6StrToUint32 at line 701. This will cause a compilation error. Add the missing import: "encoding/binary".

Copilot · 2026-02-03T17:18:18Z

pkg/service/replay/replay.go

 			Rules:          r.config.BypassRules,
 			MongoPassword:  r.config.Test.MongoPassword,
-			SQLDelay:       time.Duration(r.config.Test.Delay),
+			SQLDelay:       time.Duration(r.config.Test.Delay) * time.Second,


The SQLDelay is being multiplied by time.Second here, but the config field might already be in seconds. This could result in double conversion if r.config.Test.Delay is already a duration type. Verify that r.config.Test.Delay is an integer representing seconds, not already a time.Duration.

Copilot · 2026-02-03T17:18:19Z

cli/mcp.go

+		stdoutFile.Close()
+		return nil, nil, err


File handle may be writable as a result of data flow from a call to OpenFile and closing it may result in data loss upon failure, which is not handled explicitly.

Suggested change

stdoutFile.Close()

return nil, nil, err

if cerr := stdoutFile.Close(); cerr != nil {

return nil, nil, fmt.Errorf("open stderr log file %q: %w; additionally failed to close stdout log file %q: %v", stderrPath, err, stdoutPath, cerr)

}

return nil, nil, fmt.Errorf("open stderr log file %q: %w", stderrPath, err)

Copilot · 2026-02-03T17:18:19Z

cli/mcp.go

+				defer stdoutFile.Close()
+			}
+			if stderrFile != nil {
+				defer stderrFile.Close()


File handle may be writable as a result of data flow from a call to OpenFile and closing it may result in data loss upon failure, which is not handled explicitly.

Suggested change

defer stdoutFile.Close()

}

if stderrFile != nil {

defer stderrFile.Close()

defer func(f *os.File) {

// Attempt to flush any buffered data before closing.

if err := f.Sync(); err != nil {

fmt.Fprintf(os.Stderr, `{"level":"warn","msg":"failed to sync MCP stdout log file: %v"}`+"\n", err)

}

if err := f.Close(); err != nil {

fmt.Fprintf(os.Stderr, `{"level":"error","msg":"failed to close MCP stdout log file: %v"}`+"\n", err)

}

}(stdoutFile)

}

if stderrFile != nil {

defer func(f *os.File) {

if err := f.Sync(); err != nil {

fmt.Fprintf(os.Stderr, `{"level":"warn","msg":"failed to sync MCP stderr log file: %v"}`+"\n", err)

}

if err := f.Close(); err != nil {

fmt.Fprintf(os.Stderr, `{"level":"error","msg":"failed to close MCP stderr log file: %v"}`+"\n", err)

}

}(stderrFile)

Copilot · 2026-02-03T17:18:19Z

cli/mcp.go

+				defer stdoutFile.Close()
+			}
+			if stderrFile != nil {
+				defer stderrFile.Close()


File handle may be writable as a result of data flow from a call to OpenFile and closing it may result in data loss upon failure, which is not handled explicitly.

Suggested change

defer stdoutFile.Close()

}

if stderrFile != nil {

defer stderrFile.Close()

defer func() {

if err := stdoutFile.Close(); err != nil {

fmt.Fprintf(os.Stderr, `{"level":"error","msg":"failed to close MCP stdout log file: %v"}`+"\n", err)

}

}()

}

if stderrFile != nil {

defer func() {

if err := stderrFile.Close(); err != nil {

fmt.Fprintf(os.Stderr, `{"level":"error","msg":"failed to close MCP stderr log file: %v"}`+"\n", err)

}

}()

Copilot · 2026-02-03T17:18:20Z

cli/mcp.go

+			if err != nil {
+				return err
+			}
+			defer devNull.Close()


File handle may be writable as a result of data flow from a call to OpenFile and closing it may result in data loss upon failure, which is not handled explicitly.

Suggested change

defer devNull.Close()

defer func() {

if cerr := devNull.Close(); cerr != nil {

fmt.Fprintf(os.Stderr, `{"level":"warn","msg":"failed to close devNull: %v"}`+"\n", cerr)

}

}()

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

The previous calculation caused an integer overflow when was large (e.g., 5s), resulting in a negative read deadline. This caused the proxy to immediately timeout connections during replay. This commit fixes the issue by simplifying the calculation to , ensuring the duration remains positive and accurate. Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

…gument Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

…er to replay Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

…ting without persisted session metadata The problem: Mock routing during recording depended on `session_name` stored inside mock metadata. That created coupling between persisted mock content and runtime session state. As we moved to session-driven replay/loading, we needed session segregation in recording without relying on metadata embedded in saved mocks. What this introduces: A lightweight stream envelope (`MockFrame`) that carries: - the recorded mock payload - the active session name at stream time Why this helps: - Session context stays runtime-only, not persisted in mock schema. - Recorder can route writes correctly: - run root when no session is active - session-scoped path when `/start-session` is active - Replay/session behavior stays consistent with session lifecycle. - Removes need for parser-level session tagging solely for routing. Key implementation idea: Session name is attached at the outgoing stream boundary and consumed by recorder routing logic, instead of being derived from persisted mock metadata. Result: - Cleaner separation of concerns: - mock data = protocol content - routing context = runtime session envelope - Session segregation still works - Root fallback still works - No `session_name` persistence requirement for this flow Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

…y pipeline internals The MCP workflow is simplified to an orchestration-first model: `keploy_manager` now acts as a coordinator that returns the execution sequence, while specialized tools handle prompt generation, mock record, and mock test. This change removes the previous in-server “smart pipeline generator” and other manual orchestration logic that had become heavy, hard to maintain, and tightly coupled to environment detection. The new model keeps business logic minimal on the server and pushes repository-aware edits to the client LLM via prompt tools. What changed: - Converted manager flow to orchestration-only guidance. - Removed legacy action-based routing and related manager schema fields. - Added prompt tooling backed by TOML templates for: - test integration (`start-session` instrumentation prompt), - CI pipeline creation prompt. - Removed old pipeline/template/list-mocks implementation and related types. - Tightened tool-selection guidance so manager is default entrypoint. - Kept replay path behavior optional so downstream replay can resolve latest run. - Simplified agent env export to only `KEPLOY_AGENT_URI`. Why: - Reduce MCP server complexity and duplicated decision logic. - Make behavior deterministic and easier to evolve through prompt templates. - Avoid stale platform/language heuristics inside server code. - Keep orchestration explicit and easier for clients to reason about. Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

kiloconnect · 2026-02-10T16:24:32Z

cli/mcp.go

+			runner, ok := recordSvc.(mockrecord.RecordRunner)
+			if !ok {
+				utils.LogError(mcpLogger, nil, "service doesn't satisfy record runner interface")
+				return nil


WARNING: Silent failure - returns nil instead of an error when type assertion fails.

When recordSvc doesn't satisfy the mockrecord.RecordRunner interface, this logs an error but returns nil, causing the MCP server to silently fail without informing the caller. Consider returning an actual error:

Suggested change

return nil

return fmt.Errorf("service doesn't satisfy record runner interface")

kiloconnect · 2026-02-10T16:24:32Z

cli/mcp.go

+			replayRuntime, ok := replaySvc.(mockreplay.Runtime)
+			if !ok {
+				utils.LogError(mcpLogger, nil, "service doesn't satisfy replay runtime interface")
+				return nil


WARNING: Silent failure - returns nil instead of an error when type assertion fails.

Same issue as line 235. When replaySvc doesn't satisfy the mockreplay.Runtime interface, this silently fails. Consider returning an actual error:

Suggested change

return nil

return fmt.Errorf("service doesn't satisfy replay runtime interface")

kiloconnect · 2026-02-10T16:25:08Z

Code Review Summary

Status: 11 Issues Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	0
WARNING	11
SUGGESTION	0

Issue Details (click to expand)

WARNING

File	Line	Issue
`cli/mcp.go`	92	File handle may be writable as a result of data flow from a user-controlled source
`cli/mcp.go`	194	File handle may be writable as a result of data flow from a user-controlled source
`cli/mcp.go`	209	File handle may be writable as a result of data flow from a user-controlled source
`cli/mcp.go`	210	The os.Stdout redirection to /dev/null happens after capturing originalStdout, but any code that runs before this point could still write to stdout
`cli/mcp.go`	235	Silent failure - returns `nil` instead of an error when service doesn't satisfy interface
`cli/mcp.go`	247	Silent failure - returns `nil` instead of an error when service doesn't satisfy interface
`pkg/platform/http/agent.go`	612	The error at line 591 is now always sent to appErrCh regardless of error type
`utils/ctx.go`	56	The NewMCPCtx creates a context that only listens to OS signals but the goroutine leaks if context is never cancelled
`pkg/service/replay/replay.go`	775	The SQLDelay is being multiplied by time.Second here, but the config value may already be in seconds

Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File	Line	Issue
`pkg/agent/proxy/util/util.go`	N/A	The IP4StrToUint32 and IP6StrToUint32 functions are added but not visible in diff

Files Reviewed (17 files)

Mocker.README.md - 0 issues (documentation)
cli/mcp.go - 6 issues
docs/mcp/API_REFERENCE.md - 0 issues (documentation)
docs/mcp/ARCHITECTURE.md - 0 issues (documentation)
docs/mcp/USER_GUIDE.md - 0 issues (documentation)
pkg/mcp/server.go - 0 issues
pkg/mcp/tools.go - 0 issues
pkg/models/mock.go - 0 issues
pkg/platform/http/agent.go - 1 issue
pkg/service/mockrecord/record.go - 0 issues
pkg/service/mockrecord/service.go - 0 issues
pkg/service/mockreplay/replay.go - 0 issues
pkg/service/mockreplay/service.go - 0 issues
pkg/service/record/record.go - 0 issues
pkg/service/replay/replay.go - 1 issue
utils/ctx.go - 1 issue
utils/mcp.go - 0 issues
utils/log/logger.go - 0 issues
utils/signal_others.go - 0 issues
utils/signal_windows.go - 0 issues
utils/utils.go - 0 issues

Fix these issues in Kilo Cloud

… protocols only Replace timestamp-based mock run directories with sequential mock-set naming, and align replay resolution/docs with the new convention. What changed: - mock record now writes to `mock-set-<n>` under the provided base path. - explicit `mock-set-*` paths are respected as-is; otherwise next set id is computed. - mock replay latest-path resolution now scans only `mock-set-*` directories. - removed legacy `run-*` auto-resolution fallback. - updated CLI/MCP examples and schema descriptions to reference `mock-set-*`. - simplified mockrecord metadata collector to protocol aggregation only. - removed unused endpoint/service-name/timestamp extraction logic and dead helpers. Why: - deterministic, readable mock set naming (`mock-set-0/1/2/...`). - cleaner replay semantics (single naming scheme, no dual-mode resolution). - reduced metadata complexity to what is actively consumed by record summaries/API. Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

Added comprehensive documentation for Keploy's mock recording and replay functionality, including quick start, local setup, cloud integration, CI/CD pipeline integration, and important notes. Signed-off-by: Gourav kumar <44055698+gouravkrosx@users.noreply.github.com>

Switch mock session routing from set-name semantics to explicit file-path semantics. - Treat `/agent/hooks/start-session` `name` as full mock file path (including filename). - Record mode: create/truncate the target file on session start and write outgoing mocks directly to that path. - Replay mode: remove startup/root preloading and load mocks only from the file path provided via start-session. - Add mockdb path-based helpers for direct file operations: - `InsertMockToPath(...)` - `LoadMocksFromPath(...)` - Keep fallback behavior when no start-session is received: existing default mockDB write path flow remains. Signed-off-by: Anju <anjupathak9810@gmail.com%>

… tool - add `keploy_prompt_test_command` tool to return raw prompt text for deriving app test commands - migrate prompt templates from single `prompts.toml` to `pkg/mcp/prompts/*.toml` - `app_test_command.toml` - `test_integration.toml` - `pipeline_creation.toml` - update prompt loading to read embedded prompt files directly - wire `test_command` variable through integration and pipeline prompt builders - remove legacy `pkg/mcp/prompts.toml` - update manager orchestration text to include command-prompt-first flow - clarify tool/server descriptions for unresolved command handling: pass empty command through; server-side elicitation resolves it in execution tools - refresh MCP tool registration/types for the new prompt tool Signed-off-by: Anju <anjupathak9810@gmail.com%>

…sion is absent Replay now preloads default mocks from the resolved replay path (`mocks.yaml`) at startup, so unchanged test suites continue to work without requiring `/agent/hooks/start-session`. Behavior after this change: - Startup: load latest/default mock-set root mocks into memory. - If `/agent/hooks/start-session` is received: load the explicit file path and replace active in-memory mocks for matching. Signed-off-by: Anju <anjupathak9810@gmail.com>

…rchestration flow This commit addresses several critical issues in the MCP agent workflow regarding file permissions, path resolution, and execution continuity. **1. Fix Permission Denied (`EACCES`) on Mock Files:** - Introduced `utils.SetOwnershipToSudoUser` helper to immediately transfer ownership of newly created files/directories to the `SUDO_USER` (via `SUDO_UID`/`SUDO_GID`). - Applied this fix in: - `pkg/service/agent/agent.go`: For `start-session` mock file creation. - `pkg/platform/yaml/mockdb/db.go`: For recording new mocks. - `pkg/platform/yaml/yaml.go`: For general YAML file creation. - **Rationale**: Keploy runs as root (required for eBPF). Unlike standard CLI commands which perform a bulk permission fix on exit, the Agent is long-running and creates files dynamically These files were being created as `root`, preventing users/IDEs from deleting or modifying them while the agent was running. This change ensures files are immediately owned by the user upon creation. **2. Fix Path Resolution and Extensions:** - **Agent**: Removed logic in `StartMockSession` that forced paths to be relative to `config.Path`. Now respects the caller's path relative to the project root, fixing the issue where mocks were incorrectly nested inside `keploy/`. - **Prompts (`test_integration.toml`)**: - Updated instructions to explicitly add the `.yml` extension to `StartKeploySession` calls (e.g., `kMocks/TestName.yml`), ensuring consistent file naming. - Corrected the `start-session` endpoint URL in generated pseudocode (removed incorrect `/agent` prefix; now `/hooks/start-session`). - Clarified that `<SCOPE_PATH>` must be relative to the project root. **3. Enforce Continuous Orchestration:** - Updated `pkg/mcp/server.go` and `pkg/mcp/manager.go` to strictly enforce the full 6-step workflow. - Directives now explicitly instruct the Client LLM to "EXECUTE ALL STEPS IN ONE GO" and "IMMEDIATELY" proceed to Replay and Pipeline creation after Recording, preventing the assistant from pausing or asking for confirmation mid-workflow. ◇ Gemini 3 Pro (Previe Signed-off-by: Anju <anjupathak9810@gmail.com>

Updated README to reflect changes in commands and mock storage structure. Signed-off-by: Gourav kumar <44055698+gouravkrosx@users.noreply.github.com>

Signed-off-by: Gourav kumar <44055698+gouravkrosx@users.noreply.github.com>

anjupathak03 requested review from Sarthak160 and gouravkrosx as code owners February 3, 2026 17:14

Copilot AI review requested due to automatic review settings February 3, 2026 17:14

Copilot started reviewing on behalf of anjupathak03 February 3, 2026 17:14 View session

revert mysql changes

33cc3ba

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

Copilot AI reviewed Feb 3, 2026

View reviewed changes

anjupathak03 and others added 5 commits February 4, 2026 02:42

fix: multi printing of logo

9ea21b2

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

fix: remove integrations from go.mod and go.sum

f3b717c

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

Merge branch 'main' into mcp-support

7d98482

feat: implement api to segregate mocks based on request from a client

31ecb1d

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

anjupathak03 force-pushed the mcp-support branch from a99f631 to 31ecb1d Compare February 5, 2026 21:33

anjupathak03 added 5 commits February 6, 2026 12:07

Merge branch 'main' into mcp-support

3d435a1

fix: build failure

7f94620

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

fix: in mock-replay, load mocks based on the start-session request ar…

b4c605a

…gument Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

fix: make path flag as required for mock replay which takes mock fold…

b420b9a

…er to replay Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

chore: add comments

7d0c0bc

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

anjupathak03 force-pushed the mcp-support branch from fab7f11 to 7d0c0bc Compare February 6, 2026 09:31

anjupathak03 and others added 5 commits February 6, 2026 15:53

fix: make path non mandatory and use latest mock folder for path

13d8f0e

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com>

Merge branch 'main' into mcp-support

6fc2c5c

fix: make path non mandatory

1318a85

Signed-off-by: Anju Pathak <anjupathak9810@gmail.com%>

kiloconnect bot reviewed Feb 10, 2026

View reviewed changes

anjupathak03 and others added 4 commits February 11, 2026 13:54

anjupathak03 and others added 4 commits February 12, 2026 23:12

Revise Mocker README for sandbox commands and structure

77240a5

Updated README to reflect changes in commands and mock storage structure. Signed-off-by: Gourav kumar <44055698+gouravkrosx@users.noreply.github.com>

Update Mocker.README.md

1a15a21

Signed-off-by: Gourav kumar <44055698+gouravkrosx@users.noreply.github.com>

-		stdoutFile.Close()
-		return nil, nil, err
+		if cerr := stdoutFile.Close(); cerr != nil {
+			return nil, nil, fmt.Errorf("open stderr log file %q: %w; additionally failed to close stdout log file %q: %v", stderrPath, err, stdoutPath, cerr)
+		}
+		return nil, nil, fmt.Errorf("open stderr log file %q: %w", stderrPath, err)

-				defer stdoutFile.Close()
-			}
-			if stderrFile != nil {
-				defer stderrFile.Close()
+				defer func(f *os.File) {
+					// Attempt to flush any buffered data before closing.
+					if err := f.Sync(); err != nil {
+						fmt.Fprintf(os.Stderr, `{"level":"warn","msg":"failed to sync MCP stdout log file: %v"}`+"\n", err)
+					}
+					if err := f.Close(); err != nil {
+						fmt.Fprintf(os.Stderr, `{"level":"error","msg":"failed to close MCP stdout log file: %v"}`+"\n", err)
+					}
+				}(stdoutFile)
+			}
+			if stderrFile != nil {
+				defer func(f *os.File) {
+					if err := f.Sync(); err != nil {
+						fmt.Fprintf(os.Stderr, `{"level":"warn","msg":"failed to sync MCP stderr log file: %v"}`+"\n", err)
+					}
+					if err := f.Close(); err != nil {
+						fmt.Fprintf(os.Stderr, `{"level":"error","msg":"failed to close MCP stderr log file: %v"}`+"\n", err)
+					}
+				}(stderrFile)

-			defer devNull.Close()
+			defer func() {
+				if cerr := devNull.Close(); cerr != nil {
+					fmt.Fprintf(os.Stderr, `{"level":"warn","msg":"failed to close devNull: %v"}`+"\n", cerr)
+				}
+			}()

	return nil
	return fmt.Errorf("service doesn't satisfy record runner interface")

Uh oh!

Conversation

anjupathak03 commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Describe the changes that are made

MCP Server Integration

Mock Command Enhancements

Command Flag and Validation Improvements

Logging and Output Handling

General Codebase Improvements

Links & References

🔗 Related PRs

🐞 Related Issues

📄 Related Documents

What type of PR is this? (check all applicable)

Added e2e test pipeline?

Added comments for hard-to-understand areas?

Added to documentation?

Are there any sample code or steps to test the changes?

Self Review done?

Any relevant screenshots, recordings or logs?

🧠 Semantics for PR Title & Branch Name

Additional checklist:

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

kiloconnect bot Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

kiloconnect bot Feb 10, 2026

Choose a reason for hiding this comment

Uh oh!

kiloconnect bot commented Feb 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Review Summary

Overview

WARNING

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

anjupathak03 commented Feb 3, 2026 •

edited

Loading

kiloconnect bot commented Feb 10, 2026 •

edited

Loading