Skip to content

Commit f59a405

Browse files
committed
Detect package name mismatch
1 parent ae01437 commit f59a405

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

rewatch/src/build/compile.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -560,6 +560,7 @@ fn get_dependency_paths(
560560
let package_name = dependent_package.name();
561561
let dependency_path = if let Some(packages) = packages {
562562
packages
563+
// package name (from rescript.json) is used here.
563564
.get(package_name)
564565
.as_ref()
565566
.map(|package| package.path.clone())

rewatch/src/build/packages.rs

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -422,8 +422,20 @@ fn make_package(config: config::Config, package_path: &Path, is_pinned_dep: bool
422422
};
423423

424424
let package_name = read_package_name(package_path).expect("Could not read package name");
425+
if package_name != config.name {
426+
log::warn!(
427+
"\nPackage name mismatch:\n\
428+
- package.json name: {}\n\
429+
- rescript.json name: {}\n\
430+
This inconsistency might cause issues with package resolution.\n",
431+
package_name,
432+
config.name,
433+
);
434+
}
435+
425436
Package {
426-
name: package_name,
437+
// Reuse name from rescript.json instead.
438+
name: config.name.clone(),
427439
config: config.to_owned(),
428440
source_folders,
429441
source_files: None,

0 commit comments

Comments
 (0)