Skip to content

docs: update LF Desktop logging path#10089

Merged
mendonk merged 2 commits into
mainfrom
docs-update-desktop-logs-path
Oct 2, 2025
Merged

docs: update LF Desktop logging path#10089
mendonk merged 2 commits into
mainfrom
docs-update-desktop-logs-path

Conversation

@mendonk
Copy link
Copy Markdown
Collaborator

@mendonk mendonk commented Oct 2, 2025

LF Desktop log storage paths:

- **macOS**: `/Users/<username>/Library/Logs/com.LangflowDesktop`
- **Windows**: `C:\Users\<username>\AppData\Roaming\com.LangflowDesktop\cache`

Summary by CodeRabbit

  • Documentation
    • Updated logging docs to reflect Langflow Desktop naming across macOS and Windows.
    • Revised log directory and cache paths; refreshed example commands in “Log storage” and “Access Langflow Desktop logs.”
    • Corrected path notation to match OS conventions for each platform.
    • Improves clarity and consistency when locating and accessing desktop logs.
    • No functional changes for users.

@mendonk mendonk self-assigned this Oct 2, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 2, 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

Updates logging documentation to replace references from com.Langflow to com.LangflowDesktop across macOS and Windows paths, including desktop cache paths and command examples. No code or control-flow changes.

Changes

Cohort / File(s) Summary
Docs: Logging path updates
docs/docs/Develop/logging.mdx
Replaced com.Langflow with com.LangflowDesktop in log directories and example commands for macOS/Windows; updated desktop cache path references; documentation-only string updates with no logic changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

documentation, lgtm

Suggested reviewers

  • italojohnny
  • Cristhianzl
  • aimurphy

Pre-merge checks and finishing touches

✅ Passed checks (7 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly and concisely summarizes the main change, namely updating the Langflow Desktop logging path in the documentation, and uses a standard prefix.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Test Coverage For New Implementations ✅ Passed The PR only updates the docs/docs/Develop/logging.mdx documentation file and does not introduce any new or modified implementation code, so there is no new functionality requiring accompanying tests; consequently, the absence of added or updated test files is acceptable for this change.
Test Quality And Coverage ✅ Passed The PR only updates documentation strings and command examples in docs/docs/Develop/logging.mdx, introducing no new functionality or code paths and therefore no testing surface that requires coverage, so the absence of tests is acceptable and consistent with the existing testing strategy.
Test File Naming And Structure ✅ Passed The pull request only modifies the documentation file docs/docs/Develop/logging.mdx and introduces no new or updated test files, so there are no changes requiring verification against the prescribed test naming or structural guidelines.
Excessive Mock Usage Warning ✅ Passed The pull request only updates documentation and does not touch any test files, so it introduces no new mocks and cannot contribute to excessive mock usage; with no changes to tests, there is nothing to warn about for this check.

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 documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Oct 2, 2025
Comment thread docs/docs/Develop/logging.mdx Outdated

```cmd
cd %LOCALAPPDATA%\com.langflow\logs
cd %APPDATA%\com.LangflowDesktop\cache
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

%LOCALAPPDATA% this part doesn't change, just the com.LangflowDesktop

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oh right, because it's roaming. Thanks @Cristhianzl that change is reverted

@github-actions github-actions Bot added the lgtm This PR has been approved by a maintainer label Oct 2, 2025
@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 150dfa8 and 1407603.

📒 Files selected for processing (1)
  • docs/docs/Develop/logging.mdx (3 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (.cursor/rules/docs_development.mdc)

docs/**/*.{md,mdx}: All Markdown/MDX pages must start with front matter including at least title and description; include sidebar_position for docs pages when applicable
Code blocks must specify a language and may include a title (```lang title="…")
Use sentence case for headings and keep paragraphs short and scannable
Write in second person, present tense, with a professional but approachable tone
Use inline code with backticks for code terms; use bold for UI elements and italics for emphasis; keep lists in parallel structure
Ensure internal links are functional and navigation works (update cross-references as needed)
Verify all code examples in docs and blog actually run as shown
Use correct terminology capitalization: Langflow, Component, Flow, API, JSON
Reference images with absolute paths under /img/... and provide descriptive alt text

Files:

  • docs/docs/Develop/logging.mdx
docs/docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (.cursor/rules/docs_development.mdc)

Use Docusaurus admonitions (:::+tip|warning|danger) instead of custom callouts in docs pages

Files:

  • docs/docs/Develop/logging.mdx
⏰ 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

Comment on lines 162 to 164
```cmd
cd %LOCALAPPDATA%\com.langflow\logs
cd %APPDATA%\com.LangflowDesktop\cache
```
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Quote Windows path to handle spaces

When %APPDATA% expands to a profile path containing spaces (common on Windows), cd %APPDATA%\com.LangflowDesktop\cache fails with “Too many parameters.” Please wrap the expansion in quotes so the command succeeds for all users.

-   cd %APPDATA%\com.LangflowDesktop\cache
+   cd "%APPDATA%\com.LangflowDesktop\cache"
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
```cmd
cd %LOCALAPPDATA%\com.langflow\logs
cd %APPDATA%\com.LangflowDesktop\cache
```
🤖 Prompt for AI Agents
In docs/docs/Develop/logging.mdx around lines 162 to 164, the Windows cd example
uses cd %APPDATA%\com.LangflowDesktop\cache which breaks when %APPDATA% expands
to a path with spaces; update the example to wrap the expanded path in double
quotes so it works for all users (i.e., use cd "
%APPDATA%\com.LangflowDesktop\cache " — remove any spaces inside quotes as
appropriate) and optionally note that PowerShell users can use the same quoted
path.

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

sonarqubecloud Bot commented Oct 2, 2025

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 2, 2025

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

@Cristhianzl Cristhianzl added this pull request to the merge queue Oct 2, 2025
@Cristhianzl Cristhianzl removed this pull request from the merge queue due to a manual request Oct 2, 2025
@mendonk mendonk added this pull request to the merge queue Oct 2, 2025
Merged via the queue into main with commit 4a06423 Oct 2, 2025
17 checks passed
@mendonk mendonk deleted the docs-update-desktop-logs-path branch October 2, 2025 14:20
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