fix(workspace): anchor inherited indices lexically

Author: speak-agent

src/project.cppm

@@ -117,7 +117,13 @@ export void inherit_workspace_indices(mcpp::manifest::Manifest& member,
     member.indices = workspace.indices;
     for (auto& [_, idx] : member.indices) {
         if (idx.is_local() && idx.path.is_relative()) {
-            idx.path = std::filesystem::weakly_canonical(wsRoot / idx.path);
+            // This is an ownership/anchoring operation, not a request to
+            // resolve filesystem aliases.  weakly_canonical can rewrite a
+            // Windows short/case-preserving workspace path into a different
+            // spelling before the inherited index is opened.  Keep the path
+            // rooted exactly where the workspace manifest declared it; the
+            // normal reader remains responsible for existence/readability.
+            idx.path = (wsRoot / idx.path).lexically_normal();
         }
     }
 }

tests/e2e/12_add_command.sh

@@ -212,6 +212,10 @@ grep -qE '^widget = "9\.9\.9"$' mcpp.toml || { cat mcpp.toml; echo "widget@9.9.9
 mkdir -p "$TMP/ws"
 cd "$TMP/ws"
 cp -r "$TMP/myapp/index" index
+[[ -f index/pkgs/a/acme.util.lua ]] || {
+    echo "workspace index fixture copy is incomplete"
+    exit 1
+}
 cat > mcpp.toml <<'EOF'
 [workspace]
 members = ["m1"]
@@ -221,7 +225,11 @@ acme = { path = "index" }
 EOF
 "$MCPP" new m1 > /dev/null
 cd m1
-"$MCPP" add acme.util@2.0.0 > /dev/null
+workspace_add=$("$MCPP" add acme.util@2.0.0 2>&1) || {
+    echo "$workspace_add"
+    echo "workspace member could not read its root-owned local index"
+    exit 1
+}
 grep -qE '^util = "2\.0\.0"$' mcpp.toml || { cat mcpp.toml; echo "member should inherit workspace [indices]"; exit 1; }
 err=$("$MCPP" add acme.nope@1.0.0 2>&1) && { echo "expected error inside workspace member"; exit 1; }
 [[ "$err" == *"(acme, nope)"* ]] || { echo "wrong error: $err"; exit 1; }

tests/unit/test_pm_index_route.cpp

@@ -160,3 +160,51 @@ TEST(PmIndexRoute, MissBehindAnUnreadableIndexIsNotConclusive) {
     EXPECT_FALSE(found.hit.has_value());
     EXPECT_FALSE(found.conclusive);
 }
+
+TEST(PmIndexRoute, WorkspaceMemberReadsRootAnchoredRelativeIndex) {
+    auto root = std::filesystem::temp_directory_path()
+              / "mcpp-index-route-workspace-relative";
+    std::filesystem::remove_all(root);
+    struct Cleanup {
+        std::filesystem::path root;
+        ~Cleanup() {
+            std::error_code ec;
+            std::filesystem::remove_all(root, ec);
+        }
+    } cleanup{root};
+
+    std::filesystem::create_directories(root / "index" / "pkgs" / "a");
+    std::filesystem::create_directories(root / "member");
+    std::ofstream(root / "index" / "pkgs" / "a" / "acme.util.lua") << R"(
+package = {
+    spec = "1",
+    namespace = "acme",
+    name = "util",
+    type = "package",
+}
+)";
+    std::ofstream(root / "mcpp.toml") << R"(
+[workspace]
+members = ["member"]
+
+[indices]
+acme = { path = "index" }
+)";
+    std::ofstream(root / "member" / "mcpp.toml") << R"(
+[package]
+name = "member"
+version = "0.1.0"
+)";
+
+    auto indices = mcpp::pm::effective_indices(root / "member");
+    ASSERT_TRUE(indices.contains("acme"));
+    EXPECT_EQ(indices.at("acme").path,
+              (root / "index").lexically_normal());
+
+    mcpp::pm::IndexRoute route{ &indices, root / "member", nullptr };
+    auto selector = mcpp::pm::resolve_dependency_selector("acme.util");
+    auto found = mcpp::pm::lookup_descriptor(route, selector.candidates);
+    ASSERT_TRUE(found.hit.has_value());
+    EXPECT_EQ(found.hit->coord.namespace_, "acme");
+    EXPECT_EQ(found.hit->coord.shortName, "util");
+}