refactor(runtime_search): 删掉两个只有测试在用的导出函数

paths_of 与 parse_origin 在 src/ 下零消费者。导出面只有测试在撑,就是没有理由存在的
导出面。origin 名字的往返测试改成直接钉 to_string 的字面值 —— 那才是真正要紧的:
它们是 resolution.json 里 runtime.search.closure 的 origin 字段,改名等于改 wire 格式。

Author: speak-agent

src/platform/runtime_search.cppm

@@ -117,14 +117,6 @@ std::string_view to_string(Origin origin) {
     return "unknown";
 }
 
-std::optional<Origin> parse_origin(std::string_view value) {
-    if (value == "payload")      return Origin::Payload;
-    if (value == "package")      return Origin::Package;
-    if (value == "subos_farm")   return Origin::SubosFarm;
-    if (value == "host_default") return Origin::HostDefault;
-    return std::nullopt;
-}
-
 struct Dir {
     std::filesystem::path path;
     Origin origin = Origin::Payload;
@@ -169,15 +161,6 @@ std::vector<Dir> ordered(std::vector<Dir> dirs) {
     return merged;
 }
 
-// Just the paths, in contract order — for the callers that render a flag or
-// walk a search path and have no use for the provenance.
-std::vector<std::filesystem::path> paths_of(std::span<const Dir> dirs) {
-    std::vector<std::filesystem::path> out;
-    out.reserve(dirs.size());
-    for (auto const& dir : dirs) out.push_back(dir.path);
-    return out;
-}
-
 // The declarative exit from xlings' linker wrapper (openxlings/xlings#540).
 //
 // That wrapper appends `-rpath "$XLINGS_SUBOS_LIB" --disable-new-dtags` to

tests/unit/test_runtime_search.cpp

@@ -37,14 +37,15 @@ TEST(RuntimeSearch, MachineLocalIsEverythingButTheHostDefaults) {
     EXPECT_FALSE(search::is_machine_local(Origin::HostDefault));
 }
 
-TEST(RuntimeSearch, OriginNamesRoundTrip) {
-    for (auto origin : {Origin::Payload, Origin::Package,
-                        Origin::SubosFarm, Origin::HostDefault}) {
-        auto parsed = search::parse_origin(search::to_string(origin));
-        ASSERT_TRUE(parsed.has_value()) << search::to_string(origin);
-        EXPECT_EQ(*parsed, origin);
-    }
-    EXPECT_FALSE(search::parse_origin("something_else").has_value());
+// These strings are PUBLISHED — they are the `origin` field of every entry in
+// `resolution.json`'s `runtime.search.closure`, which CI, `mcpp why runtime` and
+// e2e 219 all read. Renaming one is a wire-format change, so it is pinned here
+// rather than left to whatever `to_string` happens to say.
+TEST(RuntimeSearch, OriginNamesArePublishedAndStable) {
+    EXPECT_EQ(search::to_string(Origin::Payload),     "payload");
+    EXPECT_EQ(search::to_string(Origin::Package),     "package");
+    EXPECT_EQ(search::to_string(Origin::SubosFarm),   "subos_farm");
+    EXPECT_EQ(search::to_string(Origin::HostDefault), "host_default");
 }
 
 TEST(RuntimeSearch, OrderedSortsByRankNotByInsertion) {
@@ -104,17 +105,6 @@ TEST(RuntimeSearch, OrderedNormalizesAndDropsEmpty) {
     EXPECT_EQ(out[0].path, std::filesystem::path("/a/b"));
 }
 
-TEST(RuntimeSearch, PathsOfPreservesContractOrder) {
-    auto out = search::ordered({
-        {"/farm", Origin::SubosFarm},
-        {"/pay",  Origin::Payload},
-    });
-    auto paths = search::paths_of(out);
-    ASSERT_EQ(paths.size(), 2u);
-    EXPECT_EQ(paths[0], std::filesystem::path("/pay"));
-    EXPECT_EQ(paths[1], std::filesystem::path("/farm"));
-}
-
 // The opt-out is a CROSS-REPO name (openxlings/xlings#540). Pinning the
 // spelling here means a rename shows up as a failed test rather than as a
 // silently ineffective declaration — the failure mode of an env var nobody