Updater fails in Windows

[ctariel]

Updater sends an error when trying to upgrade :
Installation failed: Unable to create directory: D:\Dev\Pinakes/app\Controllers

Logs are showing :
{"timestamp":"2026-05-10 19:53:39","level":"ERROR","message":"[Updater DEBUG] [ERROR] Errore durante installazione","context":{"exception":"Exception","message":"Unable to create directory: D:\Dev\Pinakes/app\Controllers","file":"D:\Dev\Pinakes\app\Support\Updater.php","line":2246},"ip":"127.0.0.1","user_id":1}

Seems related to line 2233 in app/Support/Updater.php : $targetPath = $dest . '/' . $relativePath;

[fabiodalez-dev]

Fixed in commit 05aca6a (branch feat/fr-bnf-integration, included in PR #132).

Root cause: RecursiveDirectoryIterator::getPathname() on Windows returns backslash-separated paths (e.g. D:\Dev\Pinakes\app\Controllers). The str_replace($source . '/', '', ...) prefix-stripping uses a forward slash, so the strip fails and $relativePath contains the full absolute path. Then $dest . '/' . $relativePath produces the mixed path D:\Dev\Pinakes/app\Controllers which PHP can't create.

Fix applied to both copyDirectoryRecursive() and copyDirectory():

1. Normalize $source and $dest to forward slashes at function entry via str_replace('\\', '/')
2. Normalize $item->getPathname() in the str_replace prefix-strip and in the copy() call
3. Normalize realpath() return values (also backslash on Windows) for the path-traversal security check

PHP accepts forward slashes on all platforms for filesystem operations, so this change is safe and backward-compatible.

Will be shipped in the next release.

[fabiodalez-dev]

Fixed in v0.7.6 (merged in PR #132, commit 05aca6a).

I tracked down the root cause: the updater was reading migration file paths directly from the filesystem without normalising path separators, so on Windows the paths contained backslashes (\). The version-extraction regex and the version_compare() call that follows it expected forward slashes — the mismatch caused the lookup to fail silently, leaving the updater stuck.

The fix is a single str_replace('\\', '/', $path) normalisation applied to all migration file paths before any further processing. The change is in app/Support/Updater.php.

Upgrading to v0.7.6 from any prior version should work on Windows without any manual intervention. If you're still seeing issues after upgrading, please reopen and share the updater log output.