feat(web): list archived sessions - #391
Conversation
|
CI context: both Linux Node jobs reach the pre-existing packed standalone Web CLI smoke, install the tarball, and then the server exits before announcing its origin ( This PR does not change package metadata, CLI startup, or peer-dependency resolution. Its local check, focused tests, full Node suite, and Vitest suite pass as recorded in the PR body. |
ce94c25 to
83e76ac
Compare
tt-a1i
left a comment
There was a problem hiding this comment.
The archived-history API has useful value, but two P2 pagination failures were reproduced using real Pi Session files on this exact head. Please address the inline findings and add regression tests before merge. No implementation changes were made during review. Existing focused tests pass 41/41; they do not cover these cases.
|
Fixed and pushed as
|
tt-a1i
left a comment
There was a problem hiding this comment.
Approved exact head ee70b28.
This is a valuable read-only archived-Session projection with no model-context, mutation, execution-authority, or Session-lifecycle change. Exact-head focused tests pass 44/44, required CI is green, and the current-main merge-tree is clean.
Remaining P2/P3 follow-ups are non-blocking under the merge policy: generated cursors can be unreplayable for very long paths; activity-order changes can silently skip an item; the projection omits source/origin/controller/readOnly; the 5,000 cap is after Pi listAll I/O; and the route is not GET-only. Please track these as follow-up correctness/acceptance work.
Problem
The Web archive lifecycle tracked by #347 can hide Session entries, and #361 adds an unarchive mutation, but there is no bounded read API that can discover archived Sessions for an archive view.
Value
Archived conversations become recoverable through a stable, searchable projection without copying or rewriting canonical Pi Session data.
Approach
This is distinct from #361 unarchive mutation work. It does not add deletion, fork, historical editing, or UI.
Validation
Impact
Related to #347.