fix: correct windows script linker require paths

The pathdiff crate doesn't behave properly when
one path begins with a long path prefix
and the other doesn't, so we always put the
prefix on paths.

CHANGELOG.md

@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+### Fixed
+- Correct script linker require paths on Windows by @daimond113
+
 ## [0.6.0-rc.4] - 2025-02-08
 ### Fixed
 - Refresh sources before reading package data to ensure the index is even cloned (remote changes to lockfile) by @daimond113

src/linking/generator.rs

@@ -130,7 +130,7 @@ pub fn get_lib_require_path(
 	project_manifest: &Manifest,
 ) -> Result<String, errors::GetLibRequirePath> {
 	let path = pathdiff::diff_paths(destination_dir, base_dir).unwrap();
-	tracing::debug!("diffed path: {}", path.display());
+	tracing::debug!("diffed lib path: {}", path.display());
 	let path = if use_new_structure {
 		lib_file.to_path(path)
 	} else {
@@ -208,7 +208,7 @@ pub fn get_bin_require_path(
 	destination_dir: &Path,
 ) -> String {
 	let path = pathdiff::diff_paths(destination_dir, base_dir).unwrap();
-	tracing::debug!("diffed path: {}", path.display());
+	tracing::debug!("diffed bin path: {}", path.display());
 	let path = bin_file.to_path(path);
 
 	luau_style_path(&path)
@@ -227,7 +227,7 @@ pub fn get_script_require_path(
 	destination_dir: &Path,
 ) -> String {
 	let path = pathdiff::diff_paths(destination_dir, base_dir).unwrap();
-	tracing::debug!("diffed path: {}", path.display());
+	tracing::debug!("diffed script path: {}", path.display());
 	let path = script_file.to_path(path);
 
 	luau_style_path(&path)

src/main.rs

@@ -135,8 +135,23 @@ impl<'a> MakeWriter<'a> for IndicatifWriter {
 	}
 }
 
+#[cfg(windows)]
+fn long_path_prefix(path: PathBuf) -> PathBuf {
+	if path.starts_with(r"\\?\") {
+		path
+	} else {
+		let mut str = std::ffi::OsString::from(r"\\?\");
+		str.push(path.into_os_string());
+		str.into()
+	}
+}
+
 async fn run() -> anyhow::Result<()> {
 	let cwd = std::env::current_dir().expect("failed to get current working directory");
+	// pathdiff doesn't work properly when diffing paths with and without long path prefixes
+	// so, we'll try to use the long path prefix where ever possible
+	#[cfg(windows)]
+	let cwd = long_path_prefix(cwd);
 	// Unix doesn't return the symlinked path, so we need to get it from the 0 argument
 	#[cfg(unix)]
 	let current_exe = PathBuf::from(std::env::args_os().next().expect("argument 0 not set"));