Skip to content

docs: update database location for windows desktop#10320

Merged
Cristhianzl merged 2 commits into
mainfrom
docs-langflow-desktop-db-dir
Oct 17, 2025
Merged

docs: update database location for windows desktop#10320
Cristhianzl merged 2 commits into
mainfrom
docs-langflow-desktop-db-dir

Conversation

@mendonk
Copy link
Copy Markdown
Collaborator

@mendonk mendonk commented Oct 17, 2025

Langflow for Desktop Windows db location moved.
Closes #10318

Summary by CodeRabbit

  • Documentation
    • Updated Windows desktop database storage path and filename documentation to reflect current configuration specifications.

@mendonk mendonk requested a review from Cristhianzl October 17, 2025 13:54
@mendonk mendonk self-assigned this Oct 17, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 17, 2025

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Documentation updates for Langflow's memory and storage configuration, including corrected Windows Desktop database path to com.LangflowDesktop\data\database.db, improved markdown formatting for environment variables and configuration sections, and enhanced clarity for memory components.

Changes

Cohort / File(s) Summary
Documentation - Memory & Storage Configuration
docs/docs/Develop/memory.mdx
Updated Windows Desktop storage path from com.Langflow\data\langflow.db to com.LangflowDesktop\data\database.db; improved formatting of environment variable descriptions with consistent markdown lists and tables; clarified explanations for Agent, Message History, and third-party chat memory components

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • langflow-ai/langflow#10041 — Updates Windows Desktop documentation for default Langflow database path with com.LangflowDesktop\data\database.db naming
  • langflow-ai/langflow#10029 — Modifies Windows Desktop documentation concerning default database paths for upgrades
  • langflow-ai/langflow#10089 — Updates documentation paths and filenames to use com.LangflowDesktop naming convention for memory DB and logging paths

Suggested labels

documentation, lgtm

Suggested reviewers

  • Cristhianzl
  • italojohnny

Pre-merge checks and finishing touches

❌ Failed checks (4 inconclusive)
Check name Status Explanation Resolution
Linked Issues Check ❓ Inconclusive The pull request addresses the primary objective from linked issue #10318 by updating the Windows storage path for Langflow Desktop in the memory.mdx documentation file, matching the requirement to document the path as com.LangflowDesktop\data\database.db. However, the raw summary provided is limited to path and filename updates and does not explicitly confirm whether the PR also addresses the secondary objectives from #10318, such as formatting improvements, environment variable clarifications, component documentation clarifications, and navigation improvements mentioned in the issue description. Review the actual changes in memory.mdx to verify whether all objectives from #10318 have been addressed, including formatting improvements for environment variable descriptions, component documentation clarifications for Agent and Message History components, and improvements to the "See also" navigation section, not just the database path update.
Test Quality And Coverage ❓ Inconclusive This pull request is a documentation-only change that updates the Windows Desktop database file location path in the memory configuration guide (docs/docs/Develop/memory.mdx), changing the path from com.Langflow\\data\\langflow.db to com.LangflowDesktop\\data\\database.db. The custom check "Test Quality and Coverage" is designed to evaluate tests for new code implementations, including validation of async functions, API endpoints, and behavior testing. Since this PR contains no new code implementations, functional changes, or backend/frontend features, the test evaluation criteria specified in the check instructions cannot be meaningfully applied.
Test File Naming And Structure ❓ Inconclusive This pull request is a documentation-only update that modifies exclusively the docs/docs/Develop/memory.mdx file to update the Windows desktop database location for Langflow. The verification confirms no test files of any kind are present in this PR—no backend test_*.py files, no frontend *.test.ts or *.test.tsx files, and no integration test files. The custom check "Test File Naming and Structure" is designed to verify pytest structure, Playwright test patterns, test organization, descriptive function names, and coverage of edge cases, but all these criteria require the presence of actual test files to evaluate. The custom check for test file naming and structure cannot be meaningfully evaluated against this documentation PR since it contains no test files whatsoever. The check is not applicable to documentation-only changes. If test files were expected to be included in this PR based on the objectives or nature of the changes being documented, they should be added. Otherwise, this custom check should not be applied to documentation updates, and a more appropriate check for documentation quality (such as content accuracy, formatting consistency, or link validation) would be more suitable.
Excessive Mock Usage Warning ❓ Inconclusive This PR modifies only docs/docs/Develop/memory.mdx, which is a documentation file that updates the Windows desktop database path for Langflow. The custom check "Excessive Mock Usage Warning" is designed to review test files for poor testing practices involving excessive mocks. Since this PR contains no test files, modified code, or test-related changes, the check criteria cannot be meaningfully applied or evaluated. The "Excessive Mock Usage Warning" check is intended for evaluating test design patterns in test files and is not applicable to documentation-only PRs. To properly assess this PR, consider using a documentation-focused check that verifies accuracy of path information, consistency of formatting, or completeness of the documentation updates. Alternatively, this check should only be applied to PRs that include modifications to test files.
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "docs: update database location for windows desktop" directly corresponds to the main change documented in the raw summary, which indicates that the Windows desktop default database path has been updated from com.Langflow\data\langflow.db to com.LangflowDesktop\data\database.db. The title is concise, specific, and clearly identifies the primary change as a documentation update focused on Windows Desktop database location.
Out of Scope Changes Check ✅ Passed The pull request modifies only one file, docs/docs/Develop/memory.mdx, with changes limited to updating the Windows desktop database path from com.Langflow to com.LangflowDesktop and updating the filename from langflow.db to database.db. All changes are directly related to the primary objective of linked issue #10318, which explicitly requires updating the default Windows storage path for Langflow Desktop documentation.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Test Coverage For New Implementations ✅ Passed This PR is a documentation-only update that modifies the file docs/docs/Develop/memory.mdx to correct Windows Desktop database paths and improve formatting. The custom check instructions are specifically designed to verify test coverage for code implementations, bug fixes, and new features—requiring tests to follow naming conventions like test_*.py or *.test.ts. Since this PR contains no code changes, no new components, no bug fixes, and no new features in the codebase, the test coverage check criteria are not applicable. Documentation updates do not require corresponding test files under standard software development practices.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Oct 17, 2025
@github-actions

This comment has been minimized.

@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Oct 17, 2025
@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Oct 17, 2025
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 17, 2025

Build successful! ✅
Deploying docs draft.
Deploy successful! View draft

Copy link
Copy Markdown
Member

@Cristhianzl Cristhianzl left a comment

Choose a reason for hiding this comment

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

lgtm

@github-actions github-actions Bot added the lgtm This PR has been approved by a maintainer label Oct 17, 2025
@Cristhianzl Cristhianzl added this pull request to the merge queue Oct 17, 2025
Merged via the queue into main with commit 08287c2 Oct 17, 2025
22 checks passed
@Cristhianzl Cristhianzl deleted the docs-langflow-desktop-db-dir branch October 17, 2025 15:08
mokeyish pushed a commit to mokeyish/langflow that referenced this pull request Oct 21, 2025
* update-database-location-for-windows-desktop

* duplicate article
korenLazar pushed a commit to kiran-kate/langflow that referenced this pull request Nov 13, 2025
* update-database-location-for-windows-desktop

* duplicate article
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants