#bitfighter IRC Log

Index Search ←Prev date Next date→

IRC Log for 2026-05-07

Timestamps are in GMT/BST.

03:53:01BFLogBot Pull request #787 closed by eykamp | Title: Add safety checks for empty geometry in GeomUtils | https://github.com/bitfighter/bitfighter/pull/787
03:53:01BFLogBot Repo: bitfighter | Commit: a3147cc8da | Author: eykamp | Message: Add safety checks for empty geometry in GeomUtils (#787) * Add safety checks for empty geometry in GeomUtils I am an AI agent. I have identified and fixed several potential crashes and undefined behaviors in zap/GeomUtils.cpp. Several geometric utility functions were accessing vector elements without verifying that the vectors were non-empty. This led to segmentation faults when
03:53:02BFLogBot called with empty point sets. Changes: - Added guard clauses to multiple functions in zap/GeomUtils.cpp. - Created bitfighter_test/TestGeomUtilsSafety.cpp with unit tests. - Updated zap/bitfighter_test.cmake to include the new tests. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Add safety checks for empty geometry and fix CI build failure I am an AI agent. I have identified and fixed several potential crashes in zap/Geo
03:53:04BFLogBot mUtils.cpp and resolved a CI build failure related to the libtomcrypt dependency. Geometric utility functions were missing validation for empty point sets, leading to segmentation faults. Additionally, the master server build failed in CI due to a mismatch in the tomcrypt dependency target name. Changes: - Added guard clauses to multiple functions in zap/GeomUtils.cpp. - Created bitfighter_test/TestGeomUtilsSafety.cpp with unit tests. - Updated
03:53:05BFLogBot zap/bitfighter_test.cmake to include the new tests. - Corrected the dependency target name in master/CMakeLists.txt. - Updated .github/workflows/validate.yml to include libtomcrypt-dev. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Add safety checks for empty geometry and fix CI build failures I am an AI agent. I have identified and fixed several potential crashes in zap/GeomUtils.cpp and resolved multiple CI build failur
03:53:07BFLogBot es related to the libtomcrypt dependency and its environment configuration. Geometric utility functions were missing validation for empty point sets, leading to segmentation faults. CI builds failed due to a mismatch in the tomcrypt dependency target name and missing macros in certain system-installed versions of libtomcrypt. Changes: - Added guard clauses to multiple functions in zap/GeomUtils.cpp. - Created bitfighter_test/TestGeomUtilsSafety
03:53:08BFLogBot .cpp with unit tests. - Updated zap/bitfighter_test.cmake to include the new tests. - Corrected the dependency target name in master/CMakeLists.txt. - Added #ifdef LTM_DESC in tnl/random.cpp to prevent compilation errors. - Updated .github/workflows/validate.yml to include libtomcrypt-dev and libtommath-dev. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> --------- Co-authored-by: google-labs-jules[bot] <161369871+google-labs-
03:53:10BFLogBot jules[bot]@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
03:54:29BFLogBot Pull request #785 closed by eykamp | Title: Fix out-of-bounds access in GeomUtils for empty inputs | https://github.com/bitfighter/bitfighter/pull/785
04:00:04BFLogBot Repo: bitfighter | Commit: 2684230e42 | Author: Copilot | Message: Add SafeZone object, rendering, and damage suppression logic Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/a707d776-afcb-42e0-906e-941f8e30bafc Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
04:12:08BFLogBot Repo: bitfighter | Commit: ea19685e03 | Author: Copilot | Message: Polish SafeZone implementation and review follow-ups Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/a707d776-afcb-42e0-906e-941f8e30bafc Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
04:28:16BFLogBot Pull request #788 opened by Copilot | Title: Implement xtank-style SafeZone object with team-scoped damage absorption and peace icon rendering | https://github.com/bitfighter/bitfighter/pull/788
04:43:06BFLogBot Repo: bitfighter | Commit: 780973b98e | Author: eykamp | Message: Weapon panel mostly complete
04:44:45BFLogBot Pull request #788 closed by Copilot | Title: Implement xtank-style SafeZone object with team-scoped damage absorption and peace icon rendering | https://github.com/bitfighter/bitfighter/pull/788
04:44:46BFLogBot Repo: bitfighter | Commit: 8516772932 | Author: Copilot | Message: Implement xtank-style SafeZone object with team-scoped damage absorption and peace icon rendering (#788) * Add SafeZone object, rendering, and damage suppression logic Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/a707d776-afcb-42e0-906e-941f8e30bafc Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> * Polish SafeZone implementation and review
04:44:47BFLogBot follow-ups Agent-Logs-Url: https://github.com/bitfighter/bitfighter/sessions/a707d776-afcb-42e0-906e-941f8e30bafc Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
08:50:15BFLogBot Repo: bitfighter | Commit: 97e0f26153 | Author: google-labs-jules[bot] | Message: Improve formatMessage utility: safety, escaping, and multi-digit indices This commit addresses several limitations in the formatMessage function: - Prevents out-of-bounds reads for trailing '%' or placeholder prefixes. - Supports literal '%' via '%%' escaping. - Allows indices beyond 9 for placeholders (%e10, %s12, etc.). Added unit tests in bitfighter_test/TestSt
08:50:16BFLogBot ringUtils.cpp. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
08:50:17BFLogBot Pull request #789 opened by eykamp | Title: Improve formatMessage utility: safety, escaping, and multi-digit indices | https://github.com/bitfighter/bitfighter/pull/789
09:11:32BFLogBot Repo: bitfighter | Commit: adce7b481d | Author: google-labs-jules[bot] | Message: Improve formatMessage utility and fix CI workflows This commit includes: - Improvements to formatMessage function: multi-digit indices, '%%' escaping, and safety checks for trailing placeholder prefixes. - Fixes for GitHub Actions CI: added 'mkdir -p build' before 'cd build' to prevent build failures. - Updated CI dependencies to ensure all required dev packages ar
09:11:34BFLogBot e installed. Added unit tests for formatMessage in bitfighter_test/TestStringUtils.cpp. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
15:43:16BFLogBot Repo: bitfighter | Commit: a4d3c2281b | Author: google-labs-jules[bot] | Message: Improve formatMessage utility: safety, escaping, and multi-digit indices This commit addresses several limitations in the formatMessage function: - Prevents out-of-bounds reads for trailing '%' or placeholder prefixes. - Supports literal '%' via '%%' escaping. - Allows indices beyond 9 for placeholders (%e10, %s12, etc.). Added unit tests in bitfighter_test/TestSt
15:43:17BFLogBot ringUtils.cpp. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
15:54:02BFLogBot Repo: bitfighter | Commit: e322d51f98 | Author: google-labs-jules[bot] | Message: Improve formatMessage utility: safety, escaping, and multi-digit indices This commit addresses several limitations in the formatMessage function: - Prevents out-of-bounds reads for trailing '%' or placeholder prefixes. - Supports literal '%' via '%%' escaping. - Allows indices beyond 9 for placeholders (%e10, %s12, etc.). Added unit tests in bitfighter_test/TestSt
15:54:03BFLogBot ringUtils.cpp. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
17:04:10BFLogBot Repo: bitfighter | Commit: ffef39130c | Author: google-labs-jules[bot] | Message: Improve formatMessage utility: safety, escaping, and multi-digit indices This commit addresses several limitations in the formatMessage function: - Prevents out-of-bounds reads for trailing '%' or placeholder prefixes. - Supports literal '%' via '%%' escaping. - Allows indices beyond 9 for placeholders (%e10, %s12, etc.). Added unit tests in bitfighter_test/TestSt
17:04:12BFLogBot ringUtils.cpp. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
17:57:54BFLogBot Repo: bitfighter | Commit: 4db2786e29 | Author: google-labs-jules[bot] | Message: Fix bugs in MathUtils and enhance StringUtils - Fixed a potential division-by-zero vulnerability in `findLowestRootInInterval` by explicitly handling the linear case (inA == 0) and adding safety checks. - Enhanced `isPositiveInteger` to trim leading and trailing whitespace before performing digit validation, making it more robust for varied inputs. - Added comprehe
17:57:56BFLogBot nsive unit tests in `TestMathUtils.cpp` and `TestStringUtils.cpp` to verify the fixes and ensure correct behavior for edge cases. These changes improve the overall robustness and numerical stability of the utility functions. Co-authored-by: eykamp <1133323+eykamp@users.noreply.github.com>
17:57:59BFLogBot Pull request #790 opened by eykamp | Title: Fix bugs in MathUtils and enhance StringUtils robustness | https://github.com/bitfighter/bitfighter/pull/790

Index Search ←Prev date Next date→

These logs were automatically created by BFLogBot on irc.freenode.net.