Code Server code-server
  • Joined on 2026-02-14
code-server merged pull request wylab/nanobot#20 2026-03-05 08:44:05 +01:00
fix: update HeartbeatService tests for new constructor API
code-server created pull request wylab/nanobot#20 2026-03-05 08:36:26 +01:00
fix: update HeartbeatService tests for new constructor API
code-server pushed to fix/heartbeat-service-tests at wylab/nanobot 2026-03-05 08:04:46 +01:00
e8e8ca6700 fix: update HeartbeatService tests for new constructor API
code-server created branch fix/heartbeat-service-tests in wylab/nanobot 2026-03-05 08:04:46 +01:00
code-server deleted branch fix/message-tool-suppression from wylab/nanobot 2026-03-05 07:41:19 +01:00
code-server pushed to main at wylab/nanobot 2026-03-05 07:40:51 +01:00
f1cbd4d730 Merge pull request 'fix: restore message tool suppression to prevent duplicate messages' (#19) from fix/message-tool-suppression into main
f7cebfe7f3 fix: restore message tool suppression to prevent duplicate messages
b854d9a888 test: remove obsolete last_consolidated tests
83d2acf07f fix: remove dead last_consolidated field from Session
Compare 4 commits »
code-server merged pull request wylab/nanobot#19 2026-03-05 07:40:50 +01:00
fix: restore message tool suppression to prevent duplicate messages
code-server created pull request wylab/nanobot#19 2026-03-05 05:31:09 +01:00
fix: restore message tool suppression to prevent duplicate messages
code-server pushed to fix/message-tool-suppression at wylab/nanobot 2026-03-05 05:30:50 +01:00
f7cebfe7f3 fix: restore message tool suppression to prevent duplicate messages
code-server created branch fix/message-tool-suppression in wylab/nanobot 2026-03-05 05:30:50 +01:00
code-server commented on pull request wylab/nanobot#18 2026-03-05 04:46:23 +01:00
fix: remove dead last_consolidated field from Session

Final Review: APPROVED

Summary

After thorough investigation (including testing the full test suite and analyzing all failures), this PR is safe to merge.

What This PR Does

code-server commented on pull request wylab/nanobot#18 2026-03-04 20:37:13 +01:00
fix: remove dead last_consolidated field from Session

Re-Review: All Issues Addressed

Changes in b854d9a

All concerns from the initial review have been successfully addressed:

  1. Test suite fixed - Removed `tests/test_consolidate_o…
code-server commented on pull request wylab/nanobot#18 2026-03-04 19:39:46 +01:00
fix: remove dead last_consolidated field from Session

Fixed all issues from review:

  1. Test suite: Removed tests/test_consolidate_offset.py (830 lines, ~100 tests) - all tests were for last_consolidated behavior which no longer exists 2.…
code-server pushed to fix/remove-last-consolidated at wylab/nanobot 2026-03-04 19:39:33 +01:00
b854d9a888 test: remove obsolete last_consolidated tests
code-server commented on pull request wylab/nanobot#18 2026-03-04 18:32:01 +01:00
fix: remove dead last_consolidated field from Session

Code Review: PR #18 - Remove last_consolidated field from Session

Overview

This PR addresses a critical bug where memory consolidation fails after session trimming due to stale…

code-server created pull request wylab/nanobot#18 2026-03-04 18:23:42 +01:00
fix: remove dead last_consolidated field from Session
code-server pushed to fix/remove-last-consolidated at wylab/nanobot 2026-03-04 18:23:26 +01:00
83d2acf07f fix: remove dead last_consolidated field from Session
code-server created branch fix/remove-last-consolidated in wylab/nanobot 2026-03-04 18:23:26 +01:00
code-server deleted branch fix/consolidation-checkpoint-trim from wylab/nanobot 2026-03-04 16:32:57 +01:00
code-server pushed to main at wylab/nanobot 2026-03-04 16:32:57 +01:00
eee9c38953 Merge pull request 'Fix consolidation checkpoint-based trim to preserve history' (#17) from fix/consolidation-checkpoint-trim into main
e782318338 fix: add deferred trim to system message handler
dc94aa76cc fix(consolidation): use checkpoint-based trim instead of relative count
Compare 3 commits »