A collision is two places, not two spellings #71

Merged
jschoubben merged 1 commits from fix/collisions-compare-placed-paths into main 2026-09-26 16:25:40 +00:00
Owner

checkResources compared paths as written, so ${dir:state}/server.env — the same characters in every module, a different directory in each — refused the first two placed modules that met (found landing catalogue #104). Paths are placed before they are compared, under the default root, which keeps every real collision: distinct modules' places are distinct under any one root, and a module stating another's placed root is caught because a pathless directory now owns its placed path in the comparison too. Test covers both directions.

`checkResources` compared paths as written, so `${dir:state}/server.env` — the same characters in every module, a different directory in each — refused the first two placed modules that met (found landing catalogue #104). Paths are placed before they are compared, under the default root, which keeps every real collision: distinct modules' places are distinct under any one root, and a module stating another's placed root is caught because a pathless directory now owns its placed path in the comparison too. Test covers both directions.
jschoubben added 1 commit 2026-09-26 16:25:34 +00:00
checkResources compared paths as written, so ${dir:state}/server.env —
the same characters in every module, a different directory in each —
refused the first two placed modules that met. Paths are placed before
they are compared, under the default root, which keeps every real
collision: distinct modules' places are distinct under any one root,
and a module stating another's placed root is caught because a pathless
directory now owns its placed path in the comparison too.
jschoubben merged commit 95426e25cf into main 2026-09-26 16:25:40 +00:00
jschoubben deleted branch fix/collisions-compare-placed-paths 2026-09-26 16:25:40 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: novox/mesh-controller#71