Skip to content

Fix/windows path separator session list#28165

Closed
shifttttttt wants to merge 4 commits into
anomalyco:devfrom
shifttttttt:fix/windows-path-separator-session-list
Closed

Fix/windows path separator session list#28165
shifttttttt wants to merge 4 commits into
anomalyco:devfrom
shifttttttt:fix/windows-path-separator-session-list

Conversation

@shifttttttt
Copy link
Copy Markdown

Issue for this PR

Closes #23864

Type of change

  • [ √ ] Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

fix the bug of session lost when when open project across the UI from web command.

How did you verify your code works?

open a old project through UI

Screenshots / recordings

If this is a UI change, please include a screenshot or recording.

no

Checklist

  • [ √] I have tested my changes locally
  • [√ ] I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

@github-actions github-actions Bot added needs:compliance This means the issue will auto-close after 2 hours. needs:title labels May 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • No "Type of change" checkbox is checked. Please select at least one.
  • Not all checklist items are checked. Please confirm you have tested locally and have not included unrelated changes.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions
Copy link
Copy Markdown
Contributor

Hey! Your PR title Fix/windows path separator session list doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

@github-actions
Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Potential Duplicate PRs Found

PR #28027 - fix(session): match Windows directory separators

  • Very similar fix addressing Windows path separator issues in sessions

PR #28040 - fix(session): normalize session directory filters across path variants

  • Related fix for normalizing session directory paths across variants

PR #26580 - fix: normalize Windows desktop session paths

  • Related Windows session path normalization

PR #26463 - fix: normalize Windows session directory filters

  • Similar Windows session path normalization fix

PR #23862 - fix: sessions missing from sidebar on Windows due to path separator mismatch

  • Directly related to the same issue (missing sessions on Windows due to path separators)

The most likely candidates for duplicate work are #28027, #28040, and #23862, which all address Windows path separator issues in sessions. You may want to check if these PRs are already merged or if they overlap with the current PR #28165.

@github-actions
Copy link
Copy Markdown
Contributor

This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window.

Feel free to open a new pull request that follows our guidelines.

@github-actions github-actions Bot removed the needs:compliance This means the issue will auto-close after 2 hours. label May 18, 2026
@github-actions github-actions Bot closed this May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sessions missing from sidebar on Windows due to path separator mismatch

1 participant