[Discussion] Should ^ create sub-directories when wrapWithDirectory is true?

[rafaelramalho19]

As discussed in web3-storage/web3.storage#375, filenames with ^ are creating sub-directories for each ^ present.
Do you believe this is something that we should address here or should this package continue with that functionality?

[yusefnapora]

It doesn't look like this behavior is intentional...

AFAICT, it's happening due to this regex:

const toPathComponents = (path = '') => {
  // split on / unless escaped with \
  return (path
    .trim()
    .match(/([^\\^/]|\\\/)+/g) || [])
    .filter(Boolean)
}

The comment suggests it's supposed to split on / characters, but the regex is also matching ^ as a literal char, not a regex control character.

[github-actions]

🎉 This issue has been resolved in version ipfs-unixfs-importer-v9.0.7 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀