Skip to content

Use AI builder, enterprise and hosting docs from reflex#1808

Merged
Kastier1 merged 8 commits intomainfrom
carlos/replace-some-docs
Apr 6, 2026
Merged

Use AI builder, enterprise and hosting docs from reflex#1808
Kastier1 merged 8 commits intomainfrom
carlos/replace-some-docs

Conversation

@carlosabadia
Copy link
Copy Markdown
Collaborator

@carlosabadia carlosabadia marked this pull request as ready for review April 6, 2026 16:11
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 6, 2026

Greptile Summary

This PR migrates AI builder, enterprise, and hosting documentation from local markdown files in docs/ to the bundled docs shipped with the reflex package itself (resolving via the _reflex_docs_dir docgen pipeline), enabling a single source of truth for these docs across the reflex ecosystem. It also extends the docgen pipeline to expose raw markdown content for the "Copy to Markdown" sidebar feature, reduces the TOC sidebar height from max-h-[80vh] to max-h-[60vh] with proper overflow-y-auto for scrollable long tables of contents, and removes stale debug comments.

  • ~8,200 lines of documentation deleted from the repo — now sourced from the reflex package at 0.9.0a1 (depends on reflex-dev/reflex#6277)
  • get_component_docgen now reads raw doc_content from disk and returns ((toc, doc_content), rendered), matching the existing flexdown pipeline shape so the "Copy to Markdown" button works for docgen-based pages
  • enterprise.py stub page deleted; enterprise routing now auto-derived from the package docs
  • TOC <ul> gets max-h-[60vh] overflow-y-auto — fixes visual overflow on pages with many headings
  • uv.lock updated: reflex-core replaced by reflex-base in the reflex dependency tree; click, narwhals, and python-multipart also bumped

Confidence Score: 5/5

Safe to merge once reflex-dev/reflex#6277 is confirmed included in the pinned reflex commit — all code logic is correct and well-structured

All Python changes are correct: the nested-tuple decoder in docpage.py handles both flexdown and docgen pipelines properly via the isinstance check, the TOC scrolling fix is straightforward CSS, and the enterprise.py deletion is clean. No bugs were introduced. The main dependency is the linked reflex PR being present at the pinned commit hash in uv.lock.

uv.lock pins reflex to rev=main at commit 11102878 — verify reflex-dev/reflex#6277 is included at that specific commit before merging

Important Files Changed

Filename Overview
pcweb/pages/docs/init.py Adds doc_content reading in get_component_docgen returning ((toc, doc_content), rendered); removes stale debug comments
pcweb/pages/docs/enterprise.py Deleted — enterprise docs stub page removed; routing now auto-derived from reflex package docs
pcweb/templates/docpage/docpage.py TOC max-height reduced from 80vh to 60vh with overflow-y-auto added for proper independent scrolling
uv.lock Reflex bumped to 0.9.0a1; reflex-base added, reflex-core removed as part of internal mono-repo restructuring
docs/ai_builder/overview/what_is_reflex_build.md Deleted — content migrated to reflex package bundled docs
docs/hosting/deploy-quick-start.md Deleted — content migrated to reflex package bundled docs
docs/enterprise/overview.md Deleted — content migrated to reflex package bundled docs

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[docs/ directory] -->|flexdown_docs list| B[get_component]
    C[reflex package /docs/] -->|docgen_docs dict| D[get_component_docgen]
    B --> E["get_toc(d, path)\n→ (headings, doc_content)"]
    B --> F["xd.render(d, path)\n→ rx.Component"]
    E --> G["return ((headings, doc_content), rendered)"]
    F --> G
    D --> H["get_docgen_toc(actual)\n→ headings list"]
    D --> I["Path(actual).read_text()\n→ doc_content str"]
    D --> J["render_docgen_document()\n→ rx.Component"]
    H --> K["return ((toc, doc_content), rendered)"]
    I --> K
    J --> K
    G --> L[docpage decoder]
    K --> L
    L -->|"isinstance(first, tuple)"| M["toc, doc_content = first\ncomp = second"]
    M --> N[TOC sidebar]
    M --> O["copy_to_markdown(doc_content)"]
    M --> P[Rendered article]
Loading

Reviews (1): Last reviewed commit: "use reflex_base" | Re-trigger Greptile

@Kastier1 Kastier1 merged commit c44220e into main Apr 6, 2026
10 checks passed
@Kastier1 Kastier1 deleted the carlos/replace-some-docs branch April 6, 2026 19:27
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