Repository navigation
Conversation
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
JasonW404
force-pushed
the
fix/dreaming-summary-memory-scope
branch
from
October 29, 2026 06:21
e2010ab to
c1c8e56
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Target corrected to
develop(aa713dab0cc42ae388dc25bc815010e3d411f270). The task commits were rebased so this PR contains only its 21 intended files, without unrelated main backports. Current head:c1c8e56b3e6e5a080e15cf98c1f54f541f4845da. The VM still runs the previously accepted main-based image; develop-based CI and container acceptance must be repeated before treating that head as verified.Dreaming used the observer class as its default, so semantic summaries raised a missing-argument error and fell back mechanically. Create an isolated observer instance while preserving explicitly supplied observers.
Require automatic memories to declare durable scope and quote exact source evidence. Reject local answer constraints, indirect references to existing preferences, invented paraphrases and assistant-derived preferences. Route extraction through the monitored model wrapper so the actual call is visible in Phoenix.
Previous main-based validation (before the develop rebase):
e2010ab1404d323ba39becc95b9bdbc082fde5beand deployed through repository scripts on the existing Nexent-Test VM after a clean snapshot restore. Live model/extraction source hashes match HEAD; no dev source mounts.AGT-DREAMING-SEMANTIC-01passed again on the final containers: actual worker, semantic active version, no fallback.nexent-pr4249-verified: final V2 summary trace02dcfc3fb724297b810a591416086516; fresh-conversation tracee9f158fd1994ae2cdba9dd0c10b3129a.The full formal-asset validator has an existing unrelated missing test reference in
CHAT-AGENT-DETAIL-20261026; this change's scoped schemas/execution mapping passed. One task-owned synthetic failure record from the earlier indirect-reference test was soft-deleted after retaining evidence, then the request was replayed against the corrected build. Original user data and real recall statistics were preserved.