odb: Add swapMaster sanity checker and fix net internality logic by openroad-ci · Pull Request #9505 · The-OpenROAD-Project/OpenROAD

openroad-ci · 2026-02-20T12:55:42Z

Summary

Add dbSwapMasterSanityChecker class that validates hierarchical netlist integrity after swapMaster operations, running 8 comprehensive checks: structural integrity, port/pin matching, hierarchical net connectivity, flat net connectivity, instance hierarchy, hash table integrity, dangling object detection, and combinational loop detection.
Refine dbNet::isInternalTo logic in copyModuleInsts to correctly skip external nets that cross module boundaries (nets with a parent ModNet), preventing duplicate net creation during hierarchical module replacement.
Add dbModNet::getFirstParentModNet() API to traverse the parent module hierarchy for connected ModNets.
The sanity checker is gated behind set_debug_level ODB replace_design_check_sanity 1 and enabled in all replace_hier_module / replace_arith_module regression tests.

Test plan

All existing replace_hier_mod{1..6}, replace_hier_mod_report_checks, replace_hier_mod_undo2 regressions updated with sanity checker output
All replace_arith_modules{1..3} regressions updated with sanity checker output
TestSwapMasterUnusedPort C++ unit test enables sanity checker
Golden .ok files updated to include [INFO ODB-0496] swapMaster sanity check passed (0 failures) messages

🤖 Generated with Claude Code

… integrity after `swapMaster` operations and refine `dbNet::isInternalTo` logic Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

…ressions using replace_hier_module or replace_arith_module commands Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

…lement iterative DFS for combinational loop detection, and initialize member pointers. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

Occurrences of `sta_->cmdMode()` should be read as TODOs for proper multi-mode handling later. Signed-off-by: Martin Povišer <povik@cutebit.org>

Now that the command has different behavior (drops constraints) it was breaking the tests. Signed-off-by: Martin Povišer <povik@cutebit.org>

Signed-off-by: Martin Povišer <povik@cutebit.org>

…egrity failures to prevent invalid pointer dereferences and remove redundant dangling `dbModNet` checks. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

Signed-off-by: Martin Povišer <povik@cutebit.org>

…ate/OpenROAD into secure-fix-swap-master

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

Replace std::string concatenation with fmt::format for improved readability, reduced temporary allocations, and elimination of std::to_string calls. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

- Convert warn()/error() to variadic templates that accept fmt-style format strings directly, eliminating explicit fmt::format() at all call sites. - Replace magic int constants (0/1/2) for DFS state with a local DfsState enum for clarity. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

Add NOLINT suppression for spdlog/fmt/fmt.h include which is used by the variadic warn/error templates but not recognized as direct usage by misc-include-cleaner. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

github-actions · 2026-02-20T13:00:44Z

clang-tidy review says "All clean, LGTM! 👍"

gemini-code-assist

Code Review

The pull request introduces a dbSwapMasterSanityChecker class to validate hierarchical netlist integrity after swapMaster operations. It also refines dbNet::isInternalTo logic and adds dbModNet::getFirstParentModNet() for improved hierarchical net handling. The sanity checker is integrated into dbModInst::swapMaster and enabled in various regression tests. The changes enhance code quality by adding robust integrity checks and improving hierarchical netlist management.

gemini-code-assist · 2026-02-20T13:01:59Z

src/odb/src/db/dbSwapMasterSanityChecker.cpp

+    dbModITerm* parent_iterm = bterm->getParentModITerm();
+    if (parent_iterm == nullptr) {
+      error(
+          "{}: ModBTerm '{}' has null parent ModITerm", ctx, bterm->getName());
+      continue;


A null parent_iterm for a ModBTerm means the master's port is not correctly connected to the instance's port, which is a critical error in hierarchical connectivity.

Yes. So the sanity checker will report it as an error.

gemini-code-assist · 2026-02-20T13:02:00Z

src/odb/src/db/dbSwapMasterSanityChecker.cpp

+        if (state[v] == kInPath) {
+          // Found a cycle — collect instances in the loop
+          std::string loop_path;
+          bool in_loop = false;
+          for (int idx : path) {
+            if (idx == v) {
+              in_loop = true;
+            }
+            if (in_loop) {
+              if (loop_path.empty() == false) {
+                loop_path += " -> ";
+              }
+              loop_path += insts[idx]->getName();
+            }
+          }
+          loop_path += " -> ";
+          loop_path += insts[v]->getName();
+          error("{}: combinational loop detected: {}", ctx, loop_path);
+          return;


The DFS-based cycle detection correctly identifies combinational loops and constructs a detailed loop_path for error reporting. This is a critical check for functional correctness, as combinational loops can lead to unpredictable behavior.

gemini-code-assist · 2026-02-20T13:02:00Z

src/odb/src/db/dbSwapMasterSanityChecker.cpp

+      if (net->getBlock() != block) {
+        error("{}: ITerm '{}' of inst '{}' connected to net in wrong block",
+              ctx,
+              iterm->getName(),
+              inst->getName());
+      }


An ITerm connected to a net in the wrong block is a fundamental data integrity issue, indicating a severe problem in netlist construction or manipulation.

gemini-code-assist · 2026-02-20T13:02:00Z

src/odb/src/db/dbSwapMasterSanityChecker.cpp

+  if (iterm_count != bterm_count) {
+    error("{}: ModITerm count ({}) != ModBTerm count ({})",
+          ctx,
+          iterm_count,
+          bterm_count);
+  }


A mismatch in ModITerm and ModBTerm counts is a strong indicator of a structural problem in the module's port definition or instance mapping.

gemini-code-assist · 2026-02-20T13:02:00Z

src/odb/src/db/dbSwapMasterSanityChecker.cpp

+    if (child->getMaster() == nullptr) {
+      error("{}: child ModInst '{}' has null master", ctx, child->getName());
+    }
+  }


A null master for a child ModInst is a critical structural error, indicating that the instance is not properly defined.

gemini-code-assist · 2026-02-20T13:02:03Z

src/rsz/test/replace_arith_modules1.tcl

 puts "number instances in verilog is $num_instances"

 #set_debug_level RSZ replace_arith 1
+set_debug_level ODB replace_design_check_sanity 1


Enabling the replace_design_check_sanity debug level here ensures that the new sanity checker is active during the replace_arith_modules command, which is good for testing the new feature.

gemini-code-assist · 2026-02-20T13:02:03Z

src/rsz/test/replace_arith_modules2.ok

+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)


The repeated swapMaster sanity check passed (0 failures) messages confirm that the sanity checker is being run for each arithmetic module replacement and is consistently passing. This is good validation for the new feature.

gemini-code-assist · 2026-02-20T13:02:03Z

src/rsz/test/replace_arith_modules2.tcl


 #set_debug_level RSZ replace_arith 1
 #set_debug_level ODB replace_design 10
+set_debug_level ODB replace_design_check_sanity 1


Enabling the replace_design_check_sanity debug level here ensures that the new sanity checker is active during the replace_arith_modules command, which is good for testing the new feature.

gemini-code-assist · 2026-02-20T13:02:03Z

src/rsz/test/replace_arith_modules3.ok

+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)
+[INFO ODB-0496] swapMaster sanity check passed (0 failures)


The repeated swapMaster sanity check passed (0 failures) messages confirm that the sanity checker is being run for each arithmetic module replacement and is consistently passing. This is good validation for the new feature.

gemini-code-assist · 2026-02-20T13:02:03Z

src/rsz/test/replace_arith_modules3.tcl

 set num_instances [llength [get_cells -hier *]]
 puts "number instances in verilog is $num_instances"

+set_debug_level ODB replace_design_check_sanity 1


Enabling the replace_design_check_sanity debug level here ensures that the new sanity checker is active during the replace_arith_modules command, which is good for testing the new feature.

gemini-code-assist · 2026-02-20T13:02:09Z

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

Related net lookup for modnets existed in two variants: slow which performed consistency checking and fast without checking. Transition isPower/isGround to using the fast variant, and rename the slow variant to emphasize its checking purpose. Signed-off-by: Martin Povišer <povik@cutebit.org>

Provide an alias for a method renamed in OpenSTA 3.0 to fix some bazel-orfs tests. Signed-off-by: Martin Povišer <povik@cutebit.org>

Signed-off-by: Martin Povišer <povik@cutebit.org>

…ate/OpenROAD into secure-fix-swap-master

github-actions · 2026-02-22T08:09:23Z

clang-tidy review says "All clean, LGTM! 👍"

jhkim-pii · 2026-02-22T08:41:30Z

Gemini added a bunch of useless comments in this PR.

maliberty · 2026-02-22T15:55:23Z

src/odb/src/db/dbSwapMasterSanityChecker.cpp

+  for ([[maybe_unused]] dbModInst* mi : new_master_->getModInsts()) {
+    ++set_modinst_count;
+  }


Why not use new_master_->getModInsts().size() ? Likewise elsewhere.

maliberty · 2026-02-22T15:59:59Z

src/odb/test/replace_hier_mod2.ok

-          -2.52   slack (VIOLATED)
+          -0.76   slack (VIOLATED)


Why does this PR change the timing? If it is expected then a secure CI is needed.

This is the related ORFS PR.
The-OpenROAD-Project/OpenROAD-flow-scripts#3906

Fortunately, there was no failure in both public & private PDKs.

I'll refresh the OR & ORFS PRs since the MCMM update is merged.

Replace manual counter increments with direct .size() calls for iterm_count and hash table set counts. Remove now-unnecessary loops that only existed for counting. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

Merge latest master which includes STA MCMM update and various module changes. Regoldenized odb replace_hier_mod2.ok test output to reflect updated timing results. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com> # Conflicts: # src/odb/test/replace_hier_mod2.ok

github-actions · 2026-02-23T03:10:05Z

clang-tidy review says "All clean, LGTM! 👍"

github-actions · 2026-02-23T05:37:50Z

clang-tidy review says "All clean, LGTM! 👍"

jhkim-pii and others added 15 commits February 19, 2026 23:08

odb: Add dbSwapMasterSanityChecker to validate hierarchical netlist…

261a625

… integrity after `swapMaster` operations and refine `dbNet::isInternalTo` logic Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

test: Add set_debug_level ODB replace_design_check_sanity 1 for reg…

4542c51

…ressions using replace_hier_module or replace_arith_module commands Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

odb: Fix lint issues

d6b5f06

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

odb: Refactor dbSwapMasterSanityChecker methods to return void, imp…

15b1c01

…lement iterative DFS for combinational loop detection, and initialize member pointers. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

Update to build with multi-mode STA

1a601fa

Occurrences of `sta_->cmdMode()` should be read as TODOs for proper multi-mode handling later. Signed-off-by: Martin Povišer <povik@cutebit.org>

Drop sta::network_changed from tests

0b2e66e

Now that the command has different behavior (drops constraints) it was breaking the tests. Signed-off-by: Martin Povišer <povik@cutebit.org>

Regolden tests after STA update

c332706

Signed-off-by: Martin Povišer <povik@cutebit.org>

Update flow metrics after STA update

962c5e0

Signed-off-by: Martin Povišer <povik@cutebit.org>

odb: Ensure dbSwapMasterSanityChecker exits early on structural int…

64a82a8

…egrity failures to prevent invalid pointer dereferences and remove redundant dangling `dbModNet` checks. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

rsz: Resolve compile warnings

a998009

Signed-off-by: Martin Povišer <povik@cutebit.org>

Merge branch 'master' of https://github.com/The-OpenROAD-Project-priv…

c8adaaa

…ate/OpenROAD into secure-fix-swap-master

Fix clang-tidy warning: add missing <utility> header

8f86e2c

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

openroad-ci assigned jhkim-pii Feb 20, 2026

gemini-code-assist bot reviewed Feb 20, 2026

View reviewed changes

povik and others added 5 commits February 20, 2026 16:45

dbSta: Provide sta::endpoint_path_count alias

21d54bd

Provide an alias for a method renamed in OpenSTA 3.0 to fix some bazel-orfs tests. Signed-off-by: Martin Povišer <povik@cutebit.org>

Pull fix for write_timing_model crash

805a1d9

Signed-off-by: Martin Povišer <povik@cutebit.org>

Update mock-array script for new STA

1b3a3b6

Signed-off-by: Martin Povišer <povik@cutebit.org>

Merge branch 'master' of https://github.com/The-OpenROAD-Project-priv…

66f9499

…ate/OpenROAD into secure-fix-swap-master

openroad-ci mentioned this pull request Feb 22, 2026

Enabled replace_design_check_sanity for replace_arith_module post sanity check The-OpenROAD-Project/OpenROAD-flow-scripts#3906

Open

jhkim-pii requested a review from maliberty February 22, 2026 10:53

maliberty reviewed Feb 22, 2026

View reviewed changes

jhkim-pii added 2 commits February 23, 2026 11:09

Merge origin/master into secure-fix-swap-master

9a75c4e

Merge latest master which includes STA MCMM update and various module changes. Regoldenized odb replace_hier_mod2.ok test output to reflect updated timing results. Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

Merge commit '1b3a3b6a76' into secure-fix-swap-master

9bb980a

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com> # Conflicts: # src/odb/test/replace_hier_mod2.ok

Comments

Conversation

openroad-ci commented Feb 20, 2026

Summary

Test plan

Uh oh!

github-actions bot commented Feb 20, 2026

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

jhkim-pii Feb 22, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot commented Feb 20, 2026

Uh oh!

github-actions bot commented Feb 22, 2026

Uh oh!

jhkim-pii commented Feb 22, 2026

Uh oh!

maliberty Feb 22, 2026

Choose a reason for hiding this comment

Uh oh!

jhkim-pii Feb 23, 2026

Choose a reason for hiding this comment

Uh oh!

maliberty Feb 22, 2026

Choose a reason for hiding this comment

Uh oh!

jhkim-pii Feb 23, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Feb 23, 2026

Uh oh!

github-actions bot commented Feb 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants