Skip to content

Add cross-platform mount support#2

Merged
alexbowe merged 7 commits intomainfrom
codex/cross-platform-mount-support
May 7, 2026
Merged

Add cross-platform mount support#2
alexbowe merged 7 commits intomainfrom
codex/cross-platform-mount-support

Conversation

@alexbowe
Copy link
Copy Markdown
Contributor

@alexbowe alexbowe commented May 7, 2026

Adds Windows WinFsp and macOS macFUSE mount support, updates release packaging and CI validation, and refactors the mount code before merge.

@alexbowe alexbowe merged commit a8048c9 into main May 7, 2026
10 checks passed
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 04d59411cd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +593 to +595
if (!FillDirectoryEntry(buffer, filler, entry.Name, stat))
{
return 0;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Track readdir offsets for large macFUSE directories

When a macFUSE directory has more entries than fit in one FUSE buffer, this returns successfully after filler reports the buffer is full, but every entry was emitted with offset 0 and the callback ignores the incoming offset. In that scenario the kernel has no resume position for the next readdir, so large Steam depot directories can be truncated or repeatedly start from the first entries instead of listing the whole directory.

Useful? React with 👍 / 👎.

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