The statusMatrix throw a TypeError when use in the browser with memfs

[fengyuanchen]

1. Start this demo of the memfs.
2. Append the following code after line 35 of the main.ts:

const statusMatrix = await git.statusMatrix({
  fs,
  dir: '/repo',
});
console.log('statusMatrix', statusMatrix);

3. Open https://localhost:9876/.
4. You will see the error in the console:

TypeError: Failed to execute 'getFileHandle' on 'FileSystemDirectoryHandle': Name is not allowed.

[fengyuanchen]

This error comes from line 202 of the src/api/statusMatrix.js file.

I figured out a quick solution:

const statusMatrix = await git.statusMatrix({
  fs,
  dir: '/repo',
  filter: (filepath) => filepath !== '.', // Exclude the current directory
});
console.log('statusMatrix', statusMatrix);

[jcubic]

This is most likely an error from FsaNodeFs module you're using. If this is you're project then memfs don't follow the spec of FS for isomorphic-git. It doesn't work the same as a real NodeJS fs module or lightning-fs, they work just fine.

[jcubic]

But also I'm wondering if the '.' is read for those FS modules that do work, which means that it will do extra work for nothing. Do you want to contribute to the code, investigate and maybe fix it (ignore '.' directory if it's read on different fs modules)?

[jcubic]

I was checking the source code and it seems that _walk function always returns '.' as the first entry, it do that only once. I'm not sure if I can change this behavior, it will be a breaking change. Some users may rely on this behavior.