From f4badcbce8dc6477b5c955bae694b5514095684c Mon Sep 17 00:00:00 2001
From: Matt Liberty <mliberty@precisioninno.com>
Date: Thu, 14 May 2026 07:06:19 +0000
Subject: [PATCH] mpl: factor shared internal headers into private hdrs target

Move the two headers that were duplicated across :mpl and :ui srcs in
src/mpl/BUILD (MplObserver.h, clusterEngine.h) into a new
:mpl_private_hdrs cc_library with //visibility:private, so they're
owned in one place and unreachable from outside //src/mpl. The new
target declares its own deps (:pusher, //src/odb/src/db) to be
self-contained. Also drop the redundant include/mpl/rtl_mp.h entry
from :ui srcs (already exposed via :mpl's hdrs).

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
---
 src/mpl/BUILD | 22 +++++++++++++++++-----
 1 file changed, 17 insertions(+), 5 deletions(-)

diff --git a/src/mpl/BUILD b/src/mpl/BUILD
index dc9b6eeecf..0432f194fb 100644
--- a/src/mpl/BUILD
+++ b/src/mpl/BUILD
@@ -48,10 +48,24 @@ cc_library(
     ],
 )
 
+# Headers owned by :mpl but also consumed by :ui. Kept package-private so
+# nothing outside //src/mpl can depend on them.
+cc_library(
+    name = "mpl_private_hdrs",
+    hdrs = [
+        "src/MplObserver.h",
+        "src/clusterEngine.h",
+    ],
+    visibility = ["//visibility:private"],
+    deps = [
+        ":pusher",
+        "//src/odb/src/db",
+    ],
+)
+
 cc_library(
     name = "mpl",
     srcs = [
-        "src/MplObserver.h",
         "src/SACoreHardMacro.cpp",
         "src/SACoreHardMacro.h",
         "src/SACoreSoftMacro.cpp",
@@ -59,7 +73,6 @@ cc_library(
         "src/SimulatedAnnealingCore.cpp",
         "src/SimulatedAnnealingCore.h",
         "src/clusterEngine.cpp",
-        "src/clusterEngine.h",
         "src/hier_rtlmp.cpp",
         "src/hier_rtlmp.h",
         "src/rtl_mp.cpp",
@@ -71,6 +84,7 @@ cc_library(
         "include",
     ],
     deps = [
+        ":mpl_private_hdrs",
         ":pusher",
         ":snapper",
         "//src/dbSta",
@@ -87,10 +101,7 @@ cc_library(
 cc_library(
     name = "ui",
     srcs = [
-        "include/mpl/rtl_mp.h",
         "src/MakeMacroPlacer.cpp",
-        "src/MplObserver.h",
-        "src/clusterEngine.h",
         "src/graphics.cpp",
         "src/graphics.h",
         ":swig",
@@ -105,6 +116,7 @@ cc_library(
     ],
     deps = [
         ":mpl",
+        ":mpl_private_hdrs",
         ":pusher",
         "//:ord",
         "//src/gui",