Add support for syncing all file types for both pcwatch and pcsync

[tatelax]

• Adds binary file support for pcwatch
• Adds -a & --all argument to sync changes of all file types to and from PlayCanvas for both pcwatch and pcsync

[willeastcott]

LGTM! Thanks for the contribution. I think someone else should also cast an eye. @kpal81xd, can you also do a review pass before we merge please?

[tatelax]

Sometimes this error is thrown:

/Users/tatemccormick/Documents/Projects/playcanvas-sync/src/utils/common-utils.js:179
            return md5 === remote.file.hash;
                                       ^

TypeError: Cannot read properties of null (reading 'hash')
    at Object.sameHashAsRemote (/Users/tatemccormick/Documents/Projects/playcanvas-sync/src/utils/common-utils.js:179:40)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ComputeDiffAll.handleFileOnBoth (/Users/tatemccormick/Documents/Projects/playcanvas-sync/src/sync-commands/compute-diff-all.js:97:28)
    at async Promise.all (index 49)
    at async ComputeDiffAll.handleAllFiles (/Users/tatemccormick/Documents/Projects/playcanvas-sync/src/sync-commands/compute-diff-all.js:83:9)
    at async ComputeDiffAll.run (/Users/tatemccormick/Documents/Projects/playcanvas-sync/src/sync-commands/compute-diff-all.js:40:9)
    at async reportDiffAll (/Users/tatemccormick/Documents/Projects/playcanvas-sync/src/sync-commands/sync-utils.js:10:19)
    at async Object.wrapUserErrors (/Users/tatemccormick/Documents/Projects/playcanvas-sync/src/utils/common-utils.js:112:20)
    at async Object.compareAndPrompt (/Users/tatemccormick/Documents/Projects/playcanvas-sync/src/sync-commands/sync-utils.js:34:19)

[tatelax]

Just pushed an issue that fixes this. One of the util files assumed that a download was a file.

Also, I fixed an issue where pulling changes showed many missing files/directories on local that exist on remote. What actually is happening is that PlayCanvas handles some files as directories (fbx, glb, etc.) but when you download them via API, they download as files.

[tatelax]

TODO: Deleting local files that have already been deleted on remote causes crash

[kpal81xd]

Left a couple comments for layout but otherwise looks good nice 🙂

[tatelax]

TODO: Deleting local files that have already been deleted on remote causes crash

Pushed a fix for this: fabc1a8

[tatelax]

#73

I agree with this issue. This PR could include this change. It's a bit confusing to run pcsync pullAll --all.