Skip to content

fix(desktop): normalize Windows inbox filing paths - #201

Merged
h4yfans merged 2 commits into
mainfrom
memry/fix-windows-folder-filing
Apr 13, 2026
Merged

fix(desktop): normalize Windows inbox filing paths#201
h4yfans merged 2 commits into
mainfrom
memry/fix-windows-folder-filing

Conversation

@h4yfans

@h4yfans h4yfans commented Apr 11, 2026

Copy link
Copy Markdown
Collaborator

What

Fix Windows inbox folder filing by normalizing vault-relative paths and the inline folder-create flow.

Why

A Windows user reported in #136 that creating a folder while filing a note/link could leave the app unresponsive, and after restart the folder existed but the item still remained in Inbox. The filing flow was mixing Windows \\-delimited relative paths with /-delimited folder ids.

How

  • add normalizeRelativePath and use it anywhere the desktop app emits vault-relative note, folder, watcher, inbox, or attachment paths
  • normalize folder search and AI suggestion paths in the inbox filing UI
  • update inline folder creation to insert the new folder into the React Query cache immediately instead of waiting for a refetch
  • add and update cross-platform tests so normalized relative paths are asserted explicitly

Type

  • fix — bug fix
  • feat — new feature
  • refactor — restructure without behavior change
  • style — visual/UI only
  • perf — performance improvement
  • test — adding or updating tests
  • chore — tooling, deps, config
  • docs — documentation only
  • ci — CI/CD changes

Test plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing (describe below)
  • pnpm lint
  • pnpm typecheck
  • pnpm test
  • pnpm ipc:check

Screenshots

  • N/A (behavioral fix, no visible UI change)

Checklist

  • Self-reviewed the diff
  • No hardcoded secrets or credentials
  • Files stay under ~500 LOC
  • Follows immutable data patterns

@h4yfans
h4yfans merged commit db56ae4 into main Apr 13, 2026
2 checks passed
@h4yfans
h4yfans deleted the memry/fix-windows-folder-filing branch April 13, 2026 09:41
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.

1 participant