Skip to content

Conversation

@atishaytripathi
Copy link

@atishaytripathi atishaytripathi commented Feb 11, 2026

🚀 Comprehensive PR Description: Documentation Refinement & Version Consolidation

This PR represents a systematic overhaul of the Keploy documentation to improve maintainability, resolve build-breaking routing conflicts, and enhance the overall reading experience. Below is a detailed section-by-section breakdown of every change made since the main branch.


🛠️ Core Configuration & Infrastructure Changes

File Path Change Summary Technical Reason
docusaurus.config.js Converted to CommonJS; updated versioning logic Stability: Converted import to require to fix module resolution errors in the editor ($ts-check). Set lastVersion: "3.0.0" and restored stable analytics logic.
package.json Added unist-util-visit Dependency Fix: Explicitly added a missing sub-dependency required by custom Markdown processing plugins in our config.
versions.json Removed "4.0.0" Consolidation: Streamlined the version list to ["3.0.0", "2.0.0", "1.0.0"], making the latest features the default "maintained" version.
sidebars.js Restored base structure Cleanup: Reverted to a clean Docusaurus sidebar foundation to ensure logical navigation rendering across deployments.

🎨 UI & User Experience Refinements

File Path Change Summary Design Rationale
src/css/custom.css Implemented "Consistent Spacing System" Readability: Standardized vertical rhythm (margins/padding) for H1-H3, paragraphs, and images. Fixed "clipped descenders" on headings where letters like 'g' or 'y' were being cut off at the bottom.
src/css/custom.css Table Header Alignment Visual Clarity: Forced table headers (th) to vertical-align: top, ensuring information alignment in multi-line comparison tables.

📂 Documentation Versioning & Structural Promotion

1. Version 3.0.0 Promotion (The New Maintained Version)

Originally located in version 4.0.0, these files have been promoted to Version 3.0.0 to represent the current state-of-the-art.

File Path Specific Change Critical Reason
.../server/linux/installation.md id: installationid: linux-installation Resolution: Docusaurus cannot have duplicate IDs. This change resolves build-breaking routing conflicts between OS platforms.
.../server/windows/installation.md id: installationid: windows-installation Resolution: Same as above; ensures stable navigation links for Windows users.
.../server/macos/installation.md id: installationid: macos-installation Resolution: Same as above; ensures stable navigation links for macOS users.
.../quickstart/sample-apps.md Title: "Quickstart Guide" SEO: Replaced "Quickstart filter" with a human-readable title to improve search engine indexing and user onboarding.
.../keploy-cloud/kubernetes-local-setup.md New integration docs Expansion: Added specialized documentation for localized Kubernetes testing environments (via promotion).
.../running-keploy/api-testing-auth-setup.md New auth guide Expansion: Detailed technical steps for configuring authentication in the promoted v3 tier.

2. Version 3.0.0 Archive (Data Preservation)

The original Version 3.0.0 content has been preserved to avoid any data loss while keeping the active site clean.

  • Action: Moved all previous version-3.0.0 content to versioned_docs/version-3.0.0-archive/.
  • Reason: This ensures that historical documentation remains accessible in the codebase for reference without confusing new users who expect "Version 3.0.0" to represent the latest stable interface.

📸 Visual Evidence

1. Sidebar Reference Fix

The promotion of content required updating the id tags to prevent circular or duplicate references.

  • Changes made due to build failure because of duplicate IDs in previous version.

2. Typography & Spacing

Applied global CSS to ensure headings have enough line-height so that descenders (j, p, q, y) are not visually clipped by the container overflow.

  • Spacing issues in between the paragraphs
image
  • Header alignment from the top of the corresponding box (in table)
image
  • Applied global CSS to ensure headings have enough line-height so that descenders (j, p, q, y) are not visually clipped by the container overflow.

    • Before
    image
    • After
    image

✅ Checklist

  • Build Status: Verified via npm run build with zero routing or duplicate ID errors.
  • Editor Support: TypeScript/Vale errors previously cluttering the IDE have been cleared.
  • Versioning Verification: The site now defaults to Version 3.0.0 (promoted content) as the maintained version.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you and congratulations 🎉 for opening your very first pull request in keploy

@kiloconnect
Copy link

kiloconnect bot commented Feb 11, 2026

Code Review Summary

Status: 3 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 3
WARNING 0
SUGGESTION 0
Issue Details (click to expand)

CRITICAL

File Line Issue
src/css/custom.css 2754 Invalid CSS selector - \u003e should be > (child combinator)
src/css/custom.css 2793 Invalid CSS selector - \u003e should be > (child combinator)
src/css/custom.css 2794 Invalid CSS selector - \u003e should be > (child combinator)

Summary of Issues

The CSS file contains literal \u003e Unicode escape sequences instead of the > character. This will cause the CSS child combinator selectors to not work, as browsers expect the actual > character, not the escaped string.

Impact: The spacing rules for article > div, .markdown > *:first-child, and article > .markdown > *:first-child will not be applied.

Files Reviewed (6 files)
  • docusaurus.config.js - Configuration changes (ES modules → CommonJS, version changes)
  • package.json - Added unist-util-visit dependency
  • sidebars.js - Added boilerplate sidebar config
  • src/css/custom.css - 3 issues (invalid CSS selectors)
  • versioned_sidebars/*.json - Version sidebar updates
  • versions.json - Removed v4.0.0

Fix these issues in Kilo Cloud

Signed-off-by: Atishay Tripathi <atishay1922@gmail.com>
margin-bottom: 1.5rem !important; /* Consistent space after images */
}

article \u003e div[style*='backgroundColor'] {
Copy link

Choose a reason for hiding this comment

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

CRITICAL: Invalid CSS selector - \u003e should be >

The literal string \u003e is being used instead of the > (child combinator) character. This CSS selector will not work as intended.

Suggested change
article \u003e div[style*='backgroundColor'] {
article > div[style*='backgroundColor'] {

}

/* Fix the large top margin on first element after header */
.markdown \u003e *:first-child,
Copy link

Choose a reason for hiding this comment

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

CRITICAL: Invalid CSS selector - \u003e should be >

Same issue as above - the literal string \u003e should be replaced with the > character.

Suggested change
.markdown \u003e *:first-child,
.markdown > *:first-child,


/* Fix the large top margin on first element after header */
.markdown \u003e *:first-child,
article \u003e .markdown \u003e *:first-child {
Copy link

Choose a reason for hiding this comment

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

CRITICAL: Invalid CSS selector - \u003e should be >

Multiple occurrences of \u003e that should be > characters.

Suggested change
article \u003e .markdown \u003e *:first-child {
article > .markdown > *:first-child {

@atishaytripathi
Copy link
Author

atishaytripathi commented Feb 11, 2026

🚀 PR Changelog: Version Consolidation & Technical Refinement

This commit consolidates the documentation versioning structure, promoting the latest features to Version 3.0.0 and archiving obsolete data. It also preserves critical technical fixes for build stability and editor performance.


🔄 What was Undone/Restored

  • Version Baseline: Reverted versioned_docs, versioned_sidebars, and versions.json to their original origin/main state before performing the final, hyper-specific migration. This ensured no "ghost files" from previous merge attempts remained.

✅ What was Preserved (Not Undone)

  • Editor Sync (CommonJS Fix): Maintained the conversion of import to require statements in docusaurus.config.js. This resolves the mixed-module syntax errors flagged by the TypeScript checker (// @ts-check).
  • Core Dependencies: Kept unist-util-visit in package.json which is required for the configuration logic to function.
  • UI Polish: Preserved all recent styling refinements in src/css/custom.css.

🚀 Version Migration (v4.0.0 → v3.0.0)

  • Archival (Zero Data Loss):
    • versioned_docs/version-3.0.0 → Renamed to version-3.0.0-archive.
    • versioned_sidebars/version-3.0.0-sidebars.json → Renamed to version-3.0.0-archive-sidebars.json.
  • Promotion:
    • versioned_docs/version-4.0.0 → Renamed to version-3.0.0.
    • versioned_sidebars/version-4.0.0-sidebars.json → Renamed to version-3.0.0-sidebars.json.
  • Maintained State: Set v3.0.0 as the lastVersion (current maintained version).

📁 File Changes Summary

File Action Description
versions.json Updated Reduced versions to ["3.0.0", "2.0.0", "1.0.0"].
docusaurus.config.js Updated Updated version metadata; added archive reference comments.
versioned_docs/v3.0.0/... Renamed Now contains high-quality content originally from v4.0.0.
server/installation.md (v3) Fixed Resolved id conflicts (Windows/Linux/Mac) to fix build crashes.
*.log Deleted Cleaned up temporary build residue.

📚 Related Issues

Closes https://github.com/keploy/product/issues/9


🛠️ Build Status: PASSING

The project has been verified with npm run build. Static files were generated successfully in the build directory without any duplicate-id or sidebar-reference errors.

Copy link
Member

@Achanandhi-M Achanandhi-M left a comment

Choose a reason for hiding this comment

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

Please update the PR description

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