Skip to content

Conversation

@robcohen
Copy link
Member

@robcohen robcohen commented Feb 7, 2026

Summary

Fixes three user-reported issues:

#68 - wasmtime missing

  • Added note in README about wasmtime requirement for deb/rpm installs
  • Users need to install wasmtime manually: curl https://wasmtime.dev/install.sh -sSf | bash

#69 - --version outputs an error

  • Added fallback version string for PyInstaller builds where importlib.metadata is unavailable
  • Changed from @click.version_option(package_name="rustfava") to @click.version_option(version=__version__)

#70 - failed to read file with .. paths

  • Changed WASI sandbox to preopen "/" instead of just the file's parent directory
  • This allows include statements with .. (parent directory references) to work
  • Matches native beancount behavior where includes can reference any path

Test plan

  • rustfava-server --version works in PyInstaller build
  • Include paths with .. work (e.g., include "../other/file.beancount")
  • README shows wasmtime requirement

Closes #69
Closes #70

🤖 Generated with Claude Code

robcohen and others added 3 commits February 7, 2026 12:50
- #68: Document wasmtime requirement for deb/rpm users in README
- #69: Add fallback version for --version in PyInstaller builds
- #70: Allow WASI access to entire filesystem for include paths with ".."

Closes #69
Closes #70

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@robcohen robcohen merged commit 1717157 into main Feb 7, 2026
36 of 37 checks passed
@robcohen robcohen deleted the fix/github-issues-68-69-70 branch February 7, 2026 19:20
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.

failed to read file, Operation not permitted (os error 63) --version outputs an error

1 participant