docs: Clean unused assets, reorganize the directory structure to match the nav for easier 3rd party contribution#10025
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughDocumentation reorganization: multiple “Integrations” pages removed or migrated to “Bundles.” Added/expanded bundle docs (Notion, NVIDIA, Google BigQuery guide), updated links, slugs, and asset paths. Updated site redirects and sidebar structure to align with the new docs layout. No runtime or public API changes. Changes
Sequence Diagram(s)Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Pre-merge checks and finishing touches✅ Passed checks (7 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🧪 Early access (Sonnet 4.5): enabledWe are currently testing the Sonnet 4.5 model, which is expected to improve code review quality. However, this model may lead to increased noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience. Note:
Comment |
aimurphy
changed the title
github-actions
Bot
added
the
documentation
This comment has been minimized.
This comment has been minimized.
github-actions
Bot
added
documentation
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (3)
docs/docs/Components/bundles-vertexai.mdx (1)
1-4: Add description field to frontmatter.Per coding guidelines, all documentation Markdown and MDX files must begin with a frontmatter block including at least
titleanddescriptionfields. This file is missing thedescriptionfield.Apply this diff to add the description:
--- title: Vertex AI slug: /bundles-vertexai +description: Learn how to use Google Vertex AI components in Langflow for text generation and embeddings with authentication setup guide. ---.cursor/rules/docs_development.mdc (1)
46-68: Critical: Documentation structure uses incorrect directory naming convention.The guidelines document specifies lowercase-hyphenated directory names (
agents/,get-started/,tutorials/, etc.), but the actual implementation uses PascalCase directory names (Agents/,Get-Started/,Tutorials/, etc.). This is confirmed by:
- Filesystem: All directories under
docs/docs/use PascalCasesidebars.js: All document IDs reference PascalCase paths (e.g.,Get-Started/about-langflow,Agents/agents,Flows/concepts-overview)Required action: Update lines 46-58 to reflect the actual PascalCase directory structure used in the codebase.
docs/docusaurus.config.js (1)
299-357: Critical: Redirect targets are missing the/Components/prefix.All bundle redirect targets are incorrect and will result in 404 errors. Based on the codebase structure:
- Bundle files are located at
docs/docs/Components/bundles-*.mdx- Sidebar configuration references them as
"Components/bundles-*"- Docusaurus serves docs at root (
routeBasePath: "/")- Actual URLs:
/Components/bundles-google,/Components/bundles-apify, etc.- Current redirects target:
/bundles-google,/bundles-apify, etc. (missing/Components/)Required fixes (lines 299-357):
Update all redirect targets to include the
/Components/prefix:
/bundles-google→/Components/bundles-google/bundles-apify→/Components/bundles-apify/bundles-assemblyai→/Components/bundles-assemblyai/bundles-cleanlab→/Components/bundles-cleanlab/bundles-composio→/Components/bundles-composio/bundles-docling→/Components/bundles-docling/bundles-notion→/Components/bundles-notion/bundles-nvidia→/Components/bundles-nvidia
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (96)
docs/docs/Integrations/505849097.pngis excluded by!**/*.pngdocs/docs/Integrations/965098683.pngis excluded by!**/*.pngdocs/static/files/customer_orders.csvis excluded by!**/*.csvdocs/static/img/apify_agent_flow.pngis excluded by!**/*.pngdocs/static/img/apify_agent_flow_simple.pngis excluded by!**/*.pngdocs/static/img/apify_flow_wcc.pngis excluded by!**/*.pngdocs/static/img/assemblyai-components.pngis excluded by!**/*.pngdocs/static/img/cleanlab_remediator_example.pngis excluded by!**/*.pngdocs/static/img/cleanlab_remediator_example_fallback.pngis excluded by!**/*.pngdocs/static/img/component-combine-text.pngis excluded by!**/*.pngdocs/static/img/component-data-to-dataframe.pngis excluded by!**/*.pngdocs/static/img/component-embedding-models.pngis excluded by!**/*.pngdocs/static/img/component-local-db.pngis excluded by!**/*.pngdocs/static/img/component-message-history-message-store.pngis excluded by!**/*.pngdocs/static/img/component-run-flow.pngis excluded by!**/*.pngdocs/static/img/component-save-to-file.pngis excluded by!**/*.pngdocs/static/img/component-webhook-in-basic-prompting.pngis excluded by!**/*.pngdocs/static/img/composio-create-flow.pngis excluded by!**/*.pngdocs/static/img/configure-vertex-ai-credentials-in-langflow.gifis excluded by!**/*.gifdocs/static/img/connect-handles.pngis excluded by!**/*.pngdocs/static/img/eval_rag.pngis excluded by!**/*.pngdocs/static/img/eval_response.pngis excluded by!**/*.pngdocs/static/img/eval_summary_rag.pngis excluded by!**/*.pngdocs/static/img/google/create-a-google-cloud-project.gifis excluded by!**/*.gifdocs/static/img/google/create-oauth-client-id.gifis excluded by!**/*.gifdocs/static/img/integrations-bigquery.pngis excluded by!**/*.pngdocs/static/img/integrations-docling-split-text.pngis excluded by!**/*.pngdocs/static/img/langflow-astradb-component.pngis excluded by!**/*.pngdocs/static/img/langflow-logo-black.svgis excluded by!**/*.svgdocs/static/img/langflow-logo-white.svgis excluded by!**/*.svgdocs/static/img/lf-docs-mini.svgis excluded by!**/*.svgdocs/static/img/logo.svgis excluded by!**/*.svgdocs/static/img/mcp-server-api-key.pngis excluded by!**/*.pngdocs/static/img/notion/AddContentToPage_flow_example.pngis excluded by!**/*.pngdocs/static/img/notion/AddContentToPage_flow_example_dark.pngis excluded by!**/*.pngdocs/static/img/notion/NotionDatabaseProperties_flow_example.pngis excluded by!**/*.pngdocs/static/img/notion/NotionDatabaseProperties_flow_example_dark.pngis excluded by!**/*.pngdocs/static/img/notion/NotionListPages_flow_example.pngis excluded by!**/*.pngdocs/static/img/notion/NotionListPages_flow_example_dark.pngis excluded by!**/*.pngdocs/static/img/notion/NotionPageContent_flow_example.pngis excluded by!**/*.pngdocs/static/img/notion/NotionPageContent_flow_example_dark.pngis excluded by!**/*.pngdocs/static/img/notion/NotionPageCreator_flow_example.pngis excluded by!**/*.pngdocs/static/img/notion/NotionPageCreator_flow_example_dark.pngis excluded by!**/*.pngdocs/static/img/notion/NotionPageUpdate_flow_example.pngis excluded by!**/*.pngdocs/static/img/notion/NotionPageUpdate_flow_example_dark.pngis excluded by!**/*.pngdocs/static/img/notion/NotionSearch_flow_example.pngis excluded by!**/*.pngdocs/static/img/notion/NotionSearch_flow_example_dark.pngis excluded by!**/*.pngdocs/static/img/notion/NotionUserList_flow_example.pngis excluded by!**/*.pngdocs/static/img/notion/NotionUserList_flow_example_dark.pngis excluded by!**/*.pngdocs/static/img/notion/notion_bundle.jpgis excluded by!**/*.jpgdocs/static/img/notion/notion_components_bundle.pngis excluded by!**/*.pngdocs/static/img/notion/notion_components_bundle_dark.pngis excluded by!**/*.pngdocs/static/img/notion_conversational_agent_tools.pngis excluded by!**/*.pngdocs/static/img/notion_meeting_agent_part_1.pngis excluded by!**/*.pngdocs/static/img/notion_meeting_agent_part_2.pngis excluded by!**/*.pngdocs/static/img/nvidia-component-ingest-astra.pngis excluded by!**/*.pngdocs/static/img/open-logs-pane.pngis excluded by!**/*.pngdocs/static/img/openai-model-component.pngis excluded by!**/*.pngdocs/static/img/quickstart-add-document-ingestion.pngis excluded by!**/*.pngdocs/static/img/quickstart-basic-prompt-no-connections.pngis excluded by!**/*.pngdocs/static/img/settings-messages.pngis excluded by!**/*.pngdocs/static/img/starter-flow-basic-prompting.pngis excluded by!**/*.pngdocs/static/img/starter-flow-blog-writer.pngis excluded by!**/*.pngdocs/static/img/starter-flow-document-qa.pngis excluded by!**/*.pngdocs/static/img/starter-flow-financial-report-parser.pngis excluded by!**/*.pngdocs/static/img/starter-flow-memory-chatbot.pngis excluded by!**/*.pngdocs/static/img/starter-flow-sequential-agent.pngis excluded by!**/*.pngdocs/static/img/starter-flow-simple-agent-repl.pngis excluded by!**/*.pngdocs/static/img/starter-flow-simple-agent.pngis excluded by!**/*.pngdocs/static/img/starter-flow-travel-agent.pngis excluded by!**/*.pngdocs/static/img/starter-flow-vector-rag.pngis excluded by!**/*.pngdocs/static/img/tool-calling-agent-add-tools.pngis excluded by!**/*.pngdocs/static/img/url-component.pngis excluded by!**/*.pngdocs/static/img/vector-store-agent-retrieval-tool.pngis excluded by!**/*.pngdocs/static/logos/cloud_deploy.svgis excluded by!**/*.svgdocs/static/logos/greencheck.svgis excluded by!**/*.svgdocs/static/logos/monitor-langflow.svgis excluded by!**/*.svgdocs/static/logos/playbutton.svgis excluded by!**/*.svgdocs/static/notion_imgs/1911112500.pngis excluded by!**/*.pngdocs/static/notion_imgs/2091252224.pngis excluded by!**/*.pngdocs/static/videos/chat_memory.mp4is excluded by!**/*.mp4docs/static/videos/combine_text.mp4is excluded by!**/*.mp4docs/static/videos/create_record.mp4is excluded by!**/*.mp4docs/static/videos/langflow_api.mp4is excluded by!**/*.mp4docs/static/videos/langflow_build.mp4is excluded by!**/*.mp4docs/static/videos/langflow_collection.mp4is excluded by!**/*.mp4docs/static/videos/langflow_collection_example.mp4is excluded by!**/*.mp4docs/static/videos/langflow_fork.mp4is excluded by!**/*.mp4docs/static/videos/langflow_global_variables.mp4is excluded by!**/*.mp4docs/static/videos/langflow_parameters.mp4is excluded by!**/*.mp4docs/static/videos/langflow_playground.mp4is excluded by!**/*.mp4docs/static/videos/langflow_widget.mp4is excluded by!**/*.mp4docs/static/videos/pass.mp4is excluded by!**/*.mp4docs/static/videos/store_message.mp4is excluded by!**/*.mp4docs/static/videos/sub_flow.mp4is excluded by!**/*.mp4docs/static/videos/text_operator.mp4is excluded by!**/*.mp4
📒 Files selected for processing (23)
.cursor/rules/docs_development.mdc(1 hunks)docs/docs/Components/bundles-apify.mdx(2 hunks)docs/docs/Components/bundles-assemblyai.mdx(3 hunks)docs/docs/Components/bundles-cleanlab.mdx(2 hunks)docs/docs/Components/bundles-composio.mdx(2 hunks)docs/docs/Components/bundles-docling.mdx(2 hunks)docs/docs/Components/bundles-google.mdx(5 hunks)docs/docs/Components/bundles-notion.mdx(1 hunks)docs/docs/Components/bundles-nvidia.mdx(3 hunks)docs/docs/Components/bundles-vertexai.mdx(1 hunks)docs/docs/Components/components-bundles.mdx(0 hunks)docs/docs/Components/components-data.mdx(3 hunks)docs/docs/Integrations/Google/integrations-google-big-query.mdx(0 hunks)docs/docs/Integrations/Notion/integrations-notion.mdx(0 hunks)docs/docs/Integrations/Notion/notion-agent-conversational.mdx(0 hunks)docs/docs/Integrations/Notion/notion-agent-meeting-notes.mdx(0 hunks)docs/docs/Integrations/Nvidia/integrations-nvidia-g-assist.mdx(0 hunks)docs/docs/Integrations/Nvidia/integrations-nvidia-ingest.mdx(0 hunks)docs/docs/Integrations/Nvidia/integrations-nvidia-nim-wsl2.mdx(0 hunks)docs/docs/Support/release-notes.mdx(3 hunks)docs/docs/Tutorials/agent.mdx(1 hunks)docs/docusaurus.config.js(3 hunks)docs/sidebars.js(11 hunks)
💤 Files with no reviewable changes (8)
- docs/docs/Integrations/Google/integrations-google-big-query.mdx
- docs/docs/Integrations/Notion/notion-agent-conversational.mdx
- docs/docs/Integrations/Nvidia/integrations-nvidia-ingest.mdx
- docs/docs/Integrations/Notion/notion-agent-meeting-notes.mdx
- docs/docs/Integrations/Notion/integrations-notion.mdx
- docs/docs/Integrations/Nvidia/integrations-nvidia-nim-wsl2.mdx
- docs/docs/Integrations/Nvidia/integrations-nvidia-g-assist.mdx
- docs/docs/Components/components-bundles.mdx
🧰 Additional context used
📓 Path-based instructions (3)
docs/docs/**/*.{md,mdx}
📄 CodeRabbit inference engine (.cursor/rules/docs_development.mdc)
docs/docs/**/*.{md,mdx}: All documentation content must be written in Markdown or MDX files located under docs/docs/, following the prescribed directory structure for guides, reference, how-to, concepts, and API documentation.
All documentation Markdown and MDX files must begin with a frontmatter block including at least title and description fields.
Use admonitions (:::tip, :::warning, :::danger) in Markdown/MDX files to highlight important information, warnings, or critical issues.
All images referenced in documentation must include descriptive alt text for accessibility.
All code examples included in documentation must be tested and verified to work as shown.
Internal links in documentation must be functional and not broken.
Content must follow the style guide: professional but approachable tone, second person voice, present tense, short paragraphs, sentence case headers, inline code with backticks, bold for UI elements, italic for emphasis, and parallel structure in lists.
Use consistent terminology: always capitalize Langflow, Component, Flow, and uppercase API and JSON.
Files:
docs/docs/Tutorials/agent.mdxdocs/docs/Components/bundles-docling.mdxdocs/docs/Components/bundles-notion.mdxdocs/docs/Components/bundles-vertexai.mdxdocs/docs/Components/bundles-apify.mdxdocs/docs/Components/components-data.mdxdocs/docs/Components/bundles-google.mdxdocs/docs/Support/release-notes.mdxdocs/docs/Components/bundles-cleanlab.mdxdocs/docs/Components/bundles-assemblyai.mdxdocs/docs/Components/bundles-composio.mdxdocs/docs/Components/bundles-nvidia.mdx
docs/docusaurus.config.js
📄 CodeRabbit inference engine (.cursor/rules/docs_development.mdc)
Site configuration such as title, tagline, URL, and navbar items must be managed in docs/docusaurus.config.js.
Files:
docs/docusaurus.config.js
docs/sidebars.js
📄 CodeRabbit inference engine (.cursor/rules/docs_development.mdc)
Sidebar navigation must be updated in docs/sidebars.js to reflect new or reorganized documentation content.
Files:
docs/sidebars.js
🧠 Learnings (5)
📚 Learning: 2025-09-07T05:44:46.715Z
Learnt from: TensorNull
PR: langflow-ai/langflow#9735
File: docs/docs/Components/bundles-cometapi.mdx:9-9
Timestamp: 2025-09-07T05:44:46.715Z
Learning: In Langflow bundle documentation files (docs/docs/Components/bundles-*.mdx), the standard link pattern for referencing the main Bundles page is [Bundles](/components-bundle-components), not /components-bundles. This pattern is used consistently across all 37+ bundle documentation files.
Applied to files:
docs/docs/Components/bundles-docling.mdxdocs/docs/Components/bundles-apify.mdxdocs/docs/Components/components-data.mdxdocs/docs/Components/bundles-cleanlab.mdxdocs/docs/Components/bundles-assemblyai.mdxdocs/docs/Components/bundles-composio.mdx
📚 Learning: 2025-07-18T18:26:42.027Z
Learnt from: CR
PR: langflow-ai/langflow#0
File: .cursor/rules/docs_development.mdc:0-0
Timestamp: 2025-07-18T18:26:42.027Z
Learning: Applies to docs/docs/**/*.{md,mdx} : Internal links in documentation must be functional and not broken.
Applied to files:
docs/docs/Components/components-data.mdx
📚 Learning: 2025-06-23T12:46:29.953Z
Learnt from: CR
PR: langflow-ai/langflow#0
File: .cursor/rules/docs_development.mdc:0-0
Timestamp: 2025-06-23T12:46:29.953Z
Learning: All terminology such as 'Langflow', 'Component', 'Flow', 'API', and 'JSON' must be capitalized or uppercased as specified in the terminology section.
Applied to files:
docs/docs/Components/bundles-assemblyai.mdx
📚 Learning: 2025-07-18T18:26:42.027Z
Learnt from: CR
PR: langflow-ai/langflow#0
File: .cursor/rules/docs_development.mdc:0-0
Timestamp: 2025-07-18T18:26:42.027Z
Learning: Applies to docs/sidebars.js : Sidebar navigation must be updated in docs/sidebars.js to reflect new or reorganized documentation content.
Applied to files:
docs/sidebars.js
📚 Learning: 2025-07-18T18:26:42.027Z
Learnt from: CR
PR: langflow-ai/langflow#0
File: .cursor/rules/docs_development.mdc:0-0
Timestamp: 2025-07-18T18:26:42.027Z
Learning: Applies to docs/docs/**/*.{md,mdx} : Use consistent terminology: always capitalize Langflow, Component, Flow, and uppercase API and JSON.
Applied to files:
docs/sidebars.js
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: Test Docs Build / Test Docs Build
- GitHub Check: build-and-deploy
🔇 Additional comments (51)
docs/docs/Tutorials/agent.mdx (1)
33-33: Verified: File path is correct.The file exists at the new location (
docs/static/files/customer_orders.csv). The simplified path/files/customer_orders.csvis valid and will not result in a broken link.docs/docs/Components/bundles-composio.mdx (3)
3-3: LGTM: Slug migration aligns with documentation reorganization.The slug change from
/integrations-composioto/bundles-composiocorrectly reflects the documentation restructuring from integrations to bundles terminology.
74-74: LGTM: Image path standardized to root-relative format.The image path update to
/img/composio-create-flow.pngfollows the root-relative pattern used across the documentation reorganization.
3-3: LGTM!The slug update from
/integrations-composioto/bundles-composiocorrectly aligns with the documentation reorganization described in the PR objectives.docs/docs/Components/bundles-vertexai.mdx (2)
105-105: LGTM: Image path simplified and standardized.The image path has been updated to remove the
/google/subdirectory, aligning with the standardized root-relative path structure used throughout the documentation reorganization.
105-105: LGTM!The image path update standardizes the asset location by moving it from a subdirectory to the root
/imgdirectory, consistent with the documentation reorganization. The alt text is descriptive and meets accessibility requirements.docs/docs/Components/bundles-apify.mdx (3)
3-3: LGTM: Slug migration aligns with bundle reorganization.The slug change to
/bundles-apifycorrectly reflects the transition from integrations to bundles taxonomy.
34-46: LGTM: All image paths standardized to root-relative format.All three image references have been correctly updated to use root-relative paths (
/img/apify_flow_wcc.png,/img/apify_agent_flow_simple.png,/img/apify_agent_flow.png), consistent with the documentation reorganization pattern.
3-3: LGTM!The slug update to
/bundles-apifyand the standardization of image paths to root-relative format (e.g.,/img/apify_flow_wcc.png) correctly align with the documentation reorganization. All images include descriptive alt text as required.Also applies to: 34-34, 38-38, 46-46
docs/docs/Components/bundles-assemblyai.mdx (4)
3-3: LGTM: Slug migration follows bundle structure.The slug update to
/bundles-assemblyaicorrectly aligns with the documentation reorganization from integrations to bundles.
27-27: LGTM: Image path standardized.The image path has been correctly updated to the root-relative format
/img/assemblyai-components.png.
128-128: LGTM: File download link uses root-relative path.The AssemblyAI flow JSON file reference has been correctly updated to
/files/AssemblyAI_Flow.json, consistent with the root-relative path pattern.
3-3: LGTM!The slug update to
/bundles-assemblyaiand the standardization of asset paths to root-relative format (/img/assemblyai-components.pngand/files/AssemblyAI_Flow.json) correctly align with the documentation reorganization. All references include appropriate alt text or link descriptions.Also applies to: 27-27, 128-128
docs/docs/Components/bundles-nvidia.mdx (11)
7-7: LGTM: PartialParams import added for documentation consistency.The import of
PartialParamsfrom the MDX partial is appropriate for documenting hidden parameters, consistent with other bundle documentation files.
30-54: LGTM: Comprehensive NVIDIA NIM on WSL2 section added.The new section provides clear, step-by-step instructions for setting up NVIDIA NIM on WSL2 with appropriate prerequisites, system requirements, and configuration details. The content follows the style guide with numbered steps and proper formatting.
83-194: LGTM: Detailed NVIDIA Retriever Extraction documentation added.The expanded section provides comprehensive coverage of the NVIDIA nv-ingest integration, including:
- Clear usage instructions with prerequisites
- Installation commands for both source and PyPI installations
- Complete parameter documentation
- Output format specifications with document type details
The content is well-structured and follows documentation standards.
136-136: LGTM: Image reference uses root-relative path.The image path
/img/nvidia-component-ingest-astra.pngfollows the standardized root-relative format.
197-225: LGTM: NVIDIA System-Assist integration guide added.The expanded section provides clear integration instructions for NVIDIA G-Assist, including system requirements, flow construction steps, and example usage. The content is well-organized and follows the documentation style guide.
2-2: LGTM!The title "NVIDIA" uses proper all-caps formatting as specified in the coding guidelines for terminology consistency.
7-7: LGTM!The PartialParams import is properly added and utilized on line 127 in the NVIDIA Retriever Extraction section.
106-117: LGTM!The bash code blocks are properly formatted with clear placeholders for user-specific values. The commands demonstrate both installation methods (from source and from PyPI) appropriately.
83-193: Comprehensive expansion of NVIDIA Retriever Extraction documentation.The expanded section includes detailed integration steps, prerequisites, parameter tables, and proper use of admonitions. The image includes descriptive alt text, terminology follows the coding guidelines, and all internal documentation links are verified to be functional.
30-54: Well-structured documentation section with verified external links.The NVIDIA NIM on WSL2 section provides comprehensive setup instructions with proper formatting, terminology, and style. All external NVIDIA documentation links have been verified and are accessible.
197-225: Well-documented NVIDIA System-Assist integration guide.The expanded section provides clear prerequisites, step-by-step instructions, and proper formatting. Terminology is consistent with coding guidelines, and UI elements are properly emphasized. Both the external GitHub link and internal documentation link have been verified as functional.
docs/docs/Components/bundles-google.mdx (3)
60-60: Image alt text is appropriately descriptive.The alt text "BigQuery component connected to chat input and output" clearly describes the image content for accessibility.
66-88: Verify the SQL query example.The SQL query example should be tested against an actual BigQuery dataset to ensure it works as shown. The example queries a table named
oscar_winnersin a datasetthe_oscar_awardwithin a projectbig-query-langflow-project.As per coding guidelines, all code examples in documentation must be tested and verified to work as shown. Please confirm this query has been tested with the referenced dataset structure.
144-148: Internal links follow consistent patterns.The links correctly reference bundle pages with the
/bundles-*pattern, which aligns with the documentation reorganization.Based on learnings.
docs/docs/Components/bundles-docling.mdx (2)
3-3: Slug update aligns with documentation reorganization.The slug change from
/integrations-doclingto/bundles-doclingcorrectly implements the PR objective to reorganize subdirectories to match the new navigation layout.
47-47: Image path and alt text are correct.The absolute path
/img/integrations-docling-split-text.pngfollows the documentation guidelines, and the alt text clearly describes the image content for accessibility..cursor/rules/docs_development.mdc (1)
46-53: Documentation structure reorganization is clearly documented.The updated directory layout accurately reflects the PR's objective to reorganize the
/docssubdirectories. The removal ofintegrations/and addition of new top-level directories likeagents/,get-started/,tutorials/,flows/,deployment/,develop/, andsupport/aligns with the new navigation layout mentioned in the PR description.docs/docs/Components/bundles-cleanlab.mdx (3)
3-3: Slug update aligns with documentation reorganization.The slug change from
/integrations-cleanlabto/bundles-cleanlabcorrectly implements the PR objective.
115-138: Image references use absolute paths with descriptive alt text.All image paths correctly use absolute paths (
/img/), and the alt text for each image is appropriately descriptive for accessibility compliance.As per coding guidelines.
107-107: Download file verified and is valid JSON.The file
eval_and_remediate_cleanlab.jsonexists atdocs/static/files/eval_and_remediate_cleanlab.jsonand contains valid JSON content. The link/files/eval_and_remediate_cleanlab.jsoncorrectly maps to this location.docs/docs/Support/release-notes.mdx (3)
82-82: Documentation link correctly updated.The link to the Docling bundle correctly references
/bundles-doclinginstead of the old/integrations-doclingpath, aligning with the documentation reorganization.
101-101: Documentation link correctly updated.The link to the Composio bundle correctly references
/bundles-composio, consistent with the new bundle-based documentation structure.
164-164: Documentation link correctly updated.The link to the Cleanlab bundle correctly references
/bundles-cleanlab, completing the migration from the integrations structure to the bundles structure.docs/docs/Components/bundles-notion.mdx (6)
1-4: Frontmatter is complete and follows guidelines.The frontmatter includes the required
titleandslugfields, and the slug follows the bundle naming pattern/bundles-notion.
164-164: Image verified to exist.The image file exists at
docs/static/img/notion_conversational_agent_tools.png, which correctly corresponds to the markdown reference/img/notion_conversational_agent_tools.png.
84-84: Verified: All referenced JSON files exist.The downloadable flow JSON files referenced in the documentation have been confirmed to exist at the correct locations:
/files/Meeting_Notes_Agent.json→ exists atdocs/static/files/Meeting_Notes_Agent.json/files/Conversational_Notion_Agent.json→ exists atdocs/static/files/Conversational_Notion_Agent.jsonNo issues found.
92-92: Add more descriptive alt text for accessibility.The image exists at the correct location. However, the current alt text "Notion Meeting Agent Part 1" should be more descriptive about what the image actually shows (e.g., the components, connections, or workflow depicted). According to accessibility guidelines, alt text should describe the content and purpose of the image for users who cannot see it.
Please verify the image content and update the alt text to be more descriptive about what is shown in the screenshot.
116-116: Image verified; alt text is adequate.The image
/img/notion_meeting_agent_part_2.pngexists atdocs/static/img/notion_meeting_agent_part_2.png. The current alt text "Notion Meeting Agent Part 2" appropriately identifies the subject and indicates it's part of a sequence, which is sufficient for this documentation context.
73-73: Internal link verified and functional.The link
/data-typescorrectly resolves todocs/docs/Develop/data-types.mdx, which definesslug: /data-typesat line 3. This link is used consistently across 73 locations in the documentation without issues.docs/docusaurus.config.js (2)
342-348: Comprehensive Notion integration migration.The redirect configuration properly handles the migration from the old Notion integration structure (with separate setup, meeting-notes, and conversational agent pages) to the unified
/bundles-notionpage. This aligns well with the new bundles-notion.mdx file added in this PR.
181-188: Redirect target verified and exists.The redirect target
/concepts-overviewexists atdocs/docs/Flows/concepts-overview.mdx. The redirect configuration correctly maps legacy paths (/workspace-overview,/365085a8-a90a-43f9-a779-f8769ec7eca1,/My-Collection,/workspace,/settings-project-general-settings) to the new location.docs/sidebars.js (5)
362-362: TypeScript client documentation path verified.The file
docs/docs/API-Reference/typescript-client.mdxexists at the expected location. The sidebar reference is correct.
88-90: MCP documentation files verified.All three MCP documentation files exist in the
Agents/directory:
mcp-client.mdxmcp-server.mdxmcp-component-astra.mdxThe sidebar references are correct.
97-160: All referenced files verified to exist.All 20 documentation files referenced in the reorganized Develop section have been confirmed to exist in
docs/docs/Develop/:
- Storage and memory: concepts-file-management, memory, session-id, configuration-custom-database, enterprise-database-guide
- Observability/Monitoring: logging, integrations-arize, integrations-langfuse, integrations-langsmith, integrations-langwatch, integrations-opik, integrations-instana-traceloop, contributing-telemetry
- Other: api-keys-and-authentication, install-custom-dependencies, configuration-global-variables, environment-variables, data-types, concepts-voice-mode, configuration-cli
40-71: All Flows section file paths verified.All documentation files referenced in lines 40-71 exist at their new
Flows/*locations:
Flows/concepts-overview.mdxFlows/concepts-flows.mdxFlows/concepts-publish.mdxFlows/webhook.mdxFlows/concepts-playground.mdxFlows/concepts-flows-import.mdxThe path reorganization from
Concepts/*toFlows/*is correctly implemented with no broken references.
291-327: All bundle documentation files verified and exist.All referenced bundle documentation files have been confirmed to exist in the correct location (
docs/docs/Components/):
- bundles-apify.mdx
- bundles-assemblyai.mdx
- bundles-cleanlab.mdx
- bundles-composio.mdx
- bundles-docling.mdx
- bundles-google.mdx
- bundles-notion.mdx
- bundles-nvidia.mdx
docs/docs/Components/components-data.mdx (2)
604-604: The Composio bundle link target exists and is valid.The referenced file
docs/docs/Components/bundles-composio.mdxexists in the documentation. The link/bundles-composiowill resolve correctly.
200-200: The Docling bundle link target exists and is valid.The file
docs/docs/Components/bundles-docling.mdxexists in the documentation, and the/bundles-doclinglink is correctly referenced in multiple locations including the sidebar configuration and other documentation files. The link is functional.
github-actions
Bot
added
documentation
This comment has been minimized.
This comment has been minimized.
github-actions
Bot
added
documentation
This comment has been minimized.
This comment has been minimized.
|
@aimurphy You can remove this action too https://github.com/langflow-ai/langflow/blob/main/.github/workflows/fetch_docs_notion.yml |
mendonk
left a comment
mendonk
left a comment
There was a problem hiding this comment.
LGTM
Consider removing the notion-docs workflow file too.
github-actions
Bot
added
documentation
|
This comment has been minimized.
This comment has been minimized.
1 similar comment
|
Build successful! ✅ |
2 participants
Closes LFOSS-1977
Reorganize the
/docssubdirectories to match the new navigation layout.Remove unused assets (images, files, videos) from
/docs.Move all images and files to the
/staticsubdirectoryCombine Google, NVIDA, Notion pages into single pages.
Summary by CodeRabbit
Added Notion bundle guide; expanded NVIDIA docs; improved Google BigQuery guidance.removed legacy Integrations pages; overhauled sidebar navigation.