[robdobsn]

Summary

getFilePath() and fsMutex() took local/ or sd/ anywhere in a filename as naming the file system:

• Wrong file. A file in a folder called sd on the local file system was looked up at the wrong path. For example, filehash/local/test~sd~x.txt looked for /test/sd/x.txt, not /local/test/sd/x.txt.
• Wrong lock. fsMutex() checked sd/ first, so a local file in a folder called sd was written holding the SD card's lock rather than the local one.

The new fileSystemNamedIn() recognises local or sd only at the start of the name, after an optional /, followed by / or nothing. getFilePath() and fsMutex() both use it, so the path and the lock always agree. Clients that name the file system send /<fs>/<path> (RaftCLI, raftjs), which is unchanged.

Testing

• New test_file_system_paths (Linux), covering:

  • names that do name a file system: local/a, /sd/a, the bare sd;
  • names that don't: logs/sd/x.csv, /data/local/x, backup/nonlocal/x, sdcard/x.

• make check passes.

• On an Axiom:

  	Uploading test/sd/x.txt with RaftCLI	fileread/local/test~sd~x.txt
  Before	Reported a failure: the file was written, but RaftCLI's filehash check got nofile	nofile
  After	Uploaded and verified	Returns the file

🤖 Generated with Claude Code