Merge remote-tracking branch 'private/master' into rsz-fix-tie-placement

Author: gudeh

.clang-tidy

@@ -48,6 +48,7 @@ Checks: >
   -modernize-raw-string-literal,
   -modernize-return-braced-init-list,
   -modernize-use-auto,
+  -modernize-use-constraints,
   -modernize-use-nodiscard,
   -modernize-use-trailing-return-type,
   -modernize-use-transparent-functors,

.github/PULL_REQUEST_TEMPLATE.md

@@ -16,6 +16,8 @@
 - [ ] I have verified that the local build succeeds (`./etc/Build.sh`).
 - [ ] I have run the relevant tests and they pass.
 - [ ] My code follows the repository's formatting guidelines.
+<!-- Delete next item if this PR is not a bug fix or new feature -->
+- [ ] I have included tests to prevent regressions.
 - [ ] **I have signed my commits (DCO).**
 
 ## Related Issues

.github/workflows/github-actions-format-on-push.yml

@@ -23,7 +23,10 @@ jobs:
         run: |
           clang-format --version
           for file in ${{ steps.changed-files.outputs.all_changed_files }}; do
-            if [[ "${file##*.}" =~ ^(h|C|cc|cp|cpp|c++|CPP|cxx|hh)$ && "${file}" != "src/sta/"* && "${file}" != "src/odb/src/codeGenerator/"* ]]; then
+            if [[ "${file##*.}" =~ ^(h|C|cc|cp|cpp|c++|CPP|cxx|hh)$ \
+                  && "${file}" != "src/sta/"* \
+                  && "${file}" != "src/odb/src/codeGenerator/"* \
+                  && "${file}" != "third-party/"* ]]; then
               clang-format --dry-run --Werror $file
             fi
           done

.gitignore

@@ -66,3 +66,5 @@ projectview.bazelproject
 .bazelbsp/
 /buildifier
 tmp/
+
+.playwright-mcp

AGENTS.md

@@ -13,7 +13,7 @@ Detailed guides are in `docs/agents/` subdirectory:
 
 ## Critical Rules
 
-1. **Never modify `src/sta/` files** -- OpenSTA is managed upstream (`Sdc.cc`, `Power.cc`, `Sdc.tcl`, etc.). **Only test files may be modified.** All fixes must be in OpenROAD code (e.g., `src/dbSta/`, `src/rsz/`). If OpenSTA code change is inevitable, notify user. Exception: you *may* modify `src/sta/` files temporarily for **debugging purposes** only.
+1. **Ask before modifying `src/sta/` files** -- OpenSTA is managed upstream (`Sdc.cc`, `Power.cc`, `Sdc.tcl`, etc.). Prefer fixes in OpenROAD code (e.g., src/dbSta/, src/rsz/) when possible. 
 2. Run `clang-format -i <files>` for C++ files before commit. **NEVER** for `src/sta/*` and `*.i` files.
 3. **Always use `git commit -s`** for DCO compliance.
 4. When amending submodule commits, parent repo submodule reference must also be updated via `git submodule update --init --recursive`. It is needed after any merge/pull.

BUILD.bazel

@@ -163,6 +163,7 @@ cc_library(
     name = "tcl_readline_setup",
     srcs = ["src/tcl_readline_setup.cc"],
     hdrs = ["src/tcl_readline_setup.h"],
+    includes = ["src"],
     visibility = ["//src/sta:__pkg__"],
     deps = [
         "@tcl_lang//:tcl",
@@ -197,6 +198,7 @@ cc_binary(
         "//src/gui",
         "//src/sta:opensta_lib",
         "//src/utl",
+        "//src/web",
         "@boost.stacktrace",
         "@rules_cc//cc/runfiles",  # sets BAZEL_CURRENT_REPOSITORY
         "@tcl_lang//:tcl",
@@ -512,6 +514,7 @@ sh_test(
     srcs = ["//bazel:bzl_lint_test.sh"],
     args = ["$(rootpath @buildifier_prebuilt//:buildifier)"],
     data = [
+        "MODULE.bazel",
         "@buildifier_prebuilt//:buildifier",
     ],
     tags = ["local"],
@@ -525,6 +528,7 @@ sh_test(
     srcs = ["//bazel:bzl_fmt_test.sh"],
     args = ["$(rootpath @buildifier_prebuilt//:buildifier)"],
     data = [
+        "MODULE.bazel",
         "@buildifier_prebuilt//:buildifier",
     ],
     tags = ["local"],
@@ -571,6 +575,7 @@ sh_binary(
         "$(rootpath @buildifier_prebuilt//:buildifier)",
     ],
     data = [
+        "MODULE.bazel",
         "tclint.toml",
         "//bazel:bzl_lint_test.sh",
         "//bazel:bzl_tidy.sh",

MODULE.bazel

@@ -7,13 +7,13 @@ module(
 
 # --- Regular dependencies (propagated to downstream consumers via MVS) ---
 
-bazel_dep(name = "abc", version = "0.62-yosyshq")
-bazel_dep(name = "abseil-cpp", version = "20260107.0")
+bazel_dep(name = "abc", version = "0.64-yosyshq.bcr.1")
+bazel_dep(name = "abseil-cpp", version = "20260107.1")
 bazel_dep(name = "bazel_skylib", version = "1.7.1")
 bazel_dep(name = "coin-or-lemon", version = "1.3.1")
-bazel_dep(name = "platforms", version = "0.0.11")
+bazel_dep(name = "platforms", version = "1.0.0")
 bazel_dep(name = "rules_bison", version = "0.3.1")
-bazel_dep(name = "rules_cc", version = "0.2.17")
+bazel_dep(name = "rules_cc", version = "0.2.18")
 bazel_dep(name = "rules_flex", version = "0.3.1")
 bazel_dep(name = "rules_pkg", version = "1.2.0")
 bazel_dep(name = "rules_python", version = "1.8.5")
@@ -59,17 +59,16 @@ bazel_dep(name = "boost.thread", version = BOOST_VERSION)
 bazel_dep(name = "boost.tokenizer", version = BOOST_VERSION)
 bazel_dep(name = "boost.unordered", version = BOOST_VERSION)
 bazel_dep(name = "boost.utility", version = BOOST_VERSION)
-bazel_dep(name = "cudd", version = "3.0.0")
+bazel_dep(name = "cudd", version = "3.0.0.bcr.2")
 bazel_dep(name = "eigen", version = "3.4.0.bcr.3")
-bazel_dep(name = "openmp", version = "21.1.5")
+bazel_dep(name = "openmp", version = "21.1.5.bcr.1")
 bazel_dep(name = "or-tools", version = "9.15")
-bazel_dep(name = "readline", version = "8.2.bcr.3")
 bazel_dep(name = "spdlog", version = "1.15.1")
-bazel_dep(name = "tcl_lang", version = "8.6.16.bcr.1")
+bazel_dep(name = "tcl_lang", version = "9.0.2.bcr.1")
 bazel_dep(name = "tclreadline", version = "2.4.1")
 bazel_dep(name = "tcmalloc", version = "0.0.0-20250927-12f2552")
 bazel_dep(name = "yaml-cpp", version = "0.9.0")
-bazel_dep(name = "zlib", version = "1.3.1.bcr.5")
+bazel_dep(name = "zlib", version = "1.3.1.bcr.8")
 
 # JavaScript / web UI — only used for JS unit tests in src/web/test
 bazel_dep(name = "aspect_rules_js", version = "3.0.2", dev_dependency = True)
@@ -83,7 +82,7 @@ bazel_dep(name = "rules_nodejs", version = "6.7.3", dev_dependency = True)
 bazel_dep(name = "qt-bazel")
 git_override(
     module_name = "qt-bazel",
-    commit = "df022f4ebaa4130713692fffd2f519d49e9d0b97",
+    commit = "886104974c2fd72439f2c33b5deebf0fe4649df7",
     remote = "https://github.com/The-OpenROAD-Project/qt_bazel_prebuilts",
 )
 
@@ -98,17 +97,6 @@ bazel_dep(name = "buildifier_prebuilt", version = "8.5.1.2", dev_dependency = Tr
 bazel_dep(name = "googletest", version = "1.17.0.bcr.2", dev_dependency = True)
 bazel_dep(name = "rules_verilator", version = "0.1.0", dev_dependency = True)
 bazel_dep(name = "verilator", version = "5.036.bcr.3", dev_dependency = True)
-
-# Used by chisel_binary from @bazel-orfs in test/orfs/mock-array
-bazel_dep(name = "rules_jvm_external", version = "6.4", dev_dependency = True)
-bazel_dep(name = "rules_chisel", version = "0.2.0", dev_dependency = True)
-single_version_override(
-    module_name = "rules_chisel",
-    patch_strip = 1,
-    patches = ["//:rules-chisel-dev-dep.patch"],
-)
-
-bazel_dep(name = "rules_scala", version = "7.1.5", dev_dependency = True)
 bazel_dep(name = "bazel-orfs", dev_dependency = True)
 bazel_dep(name = "bazel-orfs-verilog", dev_dependency = True)
 
@@ -195,69 +183,6 @@ orfs.default(
 )
 use_repo(orfs, "docker_orfs")
 
-SCALA_VERSION = "2.13.17"
-
-CHISEL_VERSION = "7.2.0"
-
-FIRTOOL_RESOLVER_VERSION = "2.0.1"
-
-scala_config = use_extension(
-    "@rules_scala//scala/extensions:config.bzl",
-    "scala_config",
-    dev_dependency = True,
-)
-scala_config.settings(
-    scala_version = SCALA_VERSION,
-)
-use_repo(scala_config, "rules_scala_config")
-
-scala_deps = use_extension(
-    "@rules_scala//scala/extensions:deps.bzl",
-    "scala_deps",
-    dev_dependency = True,
-)
-scala_deps.settings(
-    fetch_sources = True,
-)
-scala_deps.scala()
-
-chisel = use_extension("@rules_chisel//chisel:extensions.bzl", "chisel", dev_dependency = True)
-chisel.toolchain(
-    chisel_version = CHISEL_VERSION,
-    firtool_resolver_version = FIRTOOL_RESOLVER_VERSION,
-    scala_version = SCALA_VERSION,
-)
-use_repo(chisel, "chisel_maven")
-
-maven = use_extension("@rules_jvm_external//:extensions.bzl", "maven", dev_dependency = True)
-maven.install(
-    name = "maven",
-    artifacts = [
-    ],
-    # Update by running: bazelisk run @maven//:pin
-    lock_file = "//bazel:maven_install.json",
-    repositories = [
-        "https://repo1.maven.org/maven2",
-        "https://s01.oss.sonatype.org/content/repositories/releases",
-        "https://s01.oss.sonatype.org/content/repositories/snapshots",
-    ],
-)
-use_repo(maven, "maven")
-
-# --- External archives ---
-
-http_archive = use_repo_rule("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
-
-http_archive(
-    name = "circt",
-    build_file_content = """
-exports_files(glob(["bin/*"]), visibility = ["//visibility:public"])
-""",
-    sha256 = "b1b57d1b76ceba2bb37229dab53839d5db6d61394149708c71f6265565991ce6",
-    strip_prefix = "firtool-1.108.0",
-    url = "https://github.com/llvm/circt/releases/download/firtool-1.108.0/firrtl-bin-linux-x64.tar.gz",
-)
-
 # --- Overrides ---
 
 single_version_override(