Conversation
…y broadcasting integration tests - Remove duplicate ❌ Missing rows in Daydreamer section of PLAN.md - Update ExperienceReplay status to ✅ Complete - Update Daydreamer completion to 6/6 (100%) - Mark Blocker 3 (sharing pipeline) as RESOLVED - Simplify ClusterStability.collectAllShelves/collectAllVolumes to use MetadataStore.getAllShelves()/getAllVolumes() instead of complex workaround - Add integration tests for curiosity broadcasting (PII gating) and graph fragment import (identity stripping, discoverability) Co-authored-by: devlux76 <86517969+devlux76@users.noreply.github.com>
…dary Co-authored-by: devlux76 <86517969+devlux76@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] [P2-X] Ensure P2 background consolidation and sharing pipeline
P2-X: Fix PLAN.md drift, simplify ClusterStability, add curiosity broadcasting integration tests
Mar 14, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR cleans up P2 documentation drift, simplifies Daydreamer’s ClusterStability volume/shelf collection by using the MetadataStore aggregate APIs, and adds integration coverage around the sharing pipeline (curiosity probe broadcasting, eligibility gating, and fragment import discoverability).
Changes:
- Remove stale/duplicate “missing” module entries in
PLAN.md, update Daydreamer completion status, and mark Sharing Blocker 3 as resolved. - Simplify
ClusterStabilityshelf/volume collection to usemetadataStore.getAllShelves()/getAllVolumes()directly. - Add integration tests for curiosity probe broadcasting + eligibility filtering and for importing a peer graph fragment into
MetadataStore.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
tests/integration/Daydreamer.test.ts |
Adds integration tests for curiosity probe broadcasting/PII gating and fragment import discoverability. |
daydreamer/ClusterStability.ts |
Replaces expensive traversal helpers with direct MetadataStore aggregate methods. |
PLAN.md |
Removes duplicate “missing” rows, updates Daydreamer status to 6/6, and marks the sharing blocker as resolved. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.
P2 background consolidation + sharing pipeline implementations all exist and pass tests, but PLAN.md had stale duplicate entries marking completed modules as missing,
ClusterStabilityused an unnecessarily complex traversal based on an incorrect assumption about theMetadataStoreAPI, and integration test coverage was missing for the curiosity broadcasting pipeline.PLAN.md cleanup
ClusterStability simplification
collectAllShelves()andcollectAllVolumes()were doing O(pages × books × volumes × shelves) traversal because a comment incorrectly claimedMetadataStorelackedgetAllShelves()—it's been incore/types.tssince the interface was defined.Integration test coverage
SubgraphImporterpersists pages, strips sender identity, and makes imported content discoverable viagetPage()Original prompt
🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.