Skip to content

FS index#208

Open
LilithSilver wants to merge 14 commits into
fs-fixesfrom
fs-cache
Open

FS index#208
LilithSilver wants to merge 14 commits into
fs-fixesfrom
fs-cache

Conversation

@LilithSilver
Copy link
Copy Markdown
Collaborator

@LilithSilver LilithSilver commented Apr 18, 2026

This PR implements a full file index using blake3 hashing.

Experimentally, this PR uses that file index for local change check-in. If this is performant and OK, we can switch to it. Otherwise, we should go back to only checking-in the watched paths.

Once this PR is complete, then we have the functionality to scan the local filesystem for changes and compare it to Automerge. Only then are we able to properly implement #63.

TODO:

  • Fix text files being read as changed when they aren't
    • This could be due to hashing methodology, or due to file differences (line endings much?)
  • Fix hidden files (prefixed with .) behaving weird and thinking they're getting created during diffs
  • Fix scene file thrashing on Threadbare
  • Performance check: Threadbare seems slow. Figure out where it's coming from. Potential optimizations include:
    • Cache directories; don't crawl unless necessary (if FS crawl is the slow part)
    • Check to make sure fast hashing is actually being done
    • Go back to the old watched-paths-only method (if FS crawl is the slow part)
  • Bug check: Give it a solid runthrough with multiple clients
  • Ensure we don't break older clients

LilithSilver and others added 14 commits April 18, 2026 01:51
Updated README to improve clarity and added Discord link.
* fix chmodding release

* fix
I missed this special-case when I added the moddable-pong bundle.

Rather than adding a third case, generalise it to any artifact ending with
"-with-backstitch".

Resolves #219
I missed one more place where the environment variables have to be set. As a
result the zip file contains the backstitch repo rather than moddable-pong.

I feel like there ought to be a way to simplify this, but can't see it immediately.
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.

3 participants