docs: add ssePort, protocol config, and port mapping documentation - #806
officialasishkumar wants to merge 2 commits into
Conversation
Document the new ssePort field, extensible protocol-level port overrides, and replaceWith port mapping. Update the replay target resolution order to reflect the new priority layers.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Incremental Review (since 3db12a5)The incremental changes address both previous review comments:
The documentation is now internally consistent:
Files Reviewed (1 file)
Reviewed by claude-4.5-opus-20251124 · 94,993 tokens |
There was a problem hiding this comment.
Pull request overview
Updates the Keploy v4 configuration-file documentation to better explain replay host/port overriding, including new protocol-specific port overrides, an ssePort option, and replaceWith port-to-port mappings, plus a clarified (and expanded) target resolution order.
Changes:
- Documents new
test.ssePortandtest.protocol.<protocol>.portoptions with YAML examples. - Extends
replaceWithdocs to includeportmappings in addition to URL / gRPC:authorityreplacements. - Revises the “Replay Target Resolution Order” section to reflect the new override precedence.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Remove CLI flag reference from ssePort description - Fix Replay Target Resolution Order: swap steps 6 and 7 so replaceWith matching comes before test.host fallback
chilkotiKartik
left a comment
There was a problem hiding this comment.
LGTM! Thoroughly verified changes in docs: add ssePort, protocol config, and port mapping documentation. Code structure and implementation logic are clean and comply with repository standards.
What has changed?
This pull request updates the documentation for the Keploy configuration file to clarify and extend how port and host overrides are handled during test replay. It introduces new configuration options for protocol-specific port overrides, documents the new
ssePortfield, and revises the target resolution order for improved clarity and flexibility.Configuration options and protocol-specific overrides:
protocolsection in thetestconfiguration, allowing users to specify custom ports forhttp,sse, andgrpcprotocols. This makes it easier to add new protocols in the future without code changes. (versioned_docs/version-4.0.0/running-keploy/configuration-file.md) [1] [2]ssePortfield, which allows users to set a custom port for SSE (Server-Sent Events) test cases. This field overrides the general HTTP port for SSE requests. (versioned_docs/version-4.0.0/running-keploy/configuration-file.md) [1] [2]Replay target resolution and replaceWith enhancements:
versioned_docs/version-4.0.0/running-keploy/configuration-file.md)replaceWithfeature to support port-to-port mappings in addition to URL/gRPC authority replacements, and documented how these mappings are applied last as the highest-priority override. (versioned_docs/version-4.0.0/running-keploy/configuration-file.md) [1] [2]Documentation improvements:
versioned_docs/version-4.0.0/running-keploy/configuration-file.md)This PR Resolves keploy/keploy#3974
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please run npm run build and npm run serve to check if the changes are working as expected. Please include screenshots of the output of both the commands. Add screenshots/gif of the changes if possible.
Checklist: