Sitelet https://github.com/microsoft/TypeAgent/pull/3129
Skip to content

[Git Story] Scaffold Session Watcher responsibilities - #3129

Merged
George Ng (GeorgeNgMsft) merged 9 commits into
mainfrom
georgeng/git-story-session-watcher-stubs
Oct 3, 2026
Merged

George Ng (GeorgeNgMsft) merged 9 commits into
mainfrom
georgeng/git-story-session-watcher-stubs

Conversation

@GeorgeNgMsft

@GeorgeNgMsft George Ng (GeorgeNgMsft) commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Define the Session Watcher's responsibilities and data types, with pseudocode describing each step. The watcher gathers context from coding sessions and prepares it for memory processing. This PR adds the outline, not the working implementation.

  • Capture new conversations, tool activity, and any edit diffs recorded in the transcript.
  • Convert events into a common format while keeping their original IDs and the session they belong to.
  • Collect session details such as the client, models, timestamps, and parent session.
  • Connect a separate privacy filter before sending events and session details to memory processing. If filtering fails, do not send unfiltered data.
  • Save the session ID, transcript file, last complete event ID, and read position so capture can resume. This tracks reading progress, not whether memory processing has finished.
  • Leave memory creation, story writing, and deciding what belongs in each commit to other components.
  • Add tests for the stubs and a short README explanation.
SessionWatchRequest + saved read checkpoint
    ↓
Read new transcript records
    ├── Save the next read checkpoint
    ↓
Convert events to a common format
    ↓
Package events with session details
    ↓
Apply privacy filtering
    ↓
Send approved content to memory processing

Validation: the package builds, all 10 tests pass, and formatting checks pass.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Simplify event identity and naming, define transcript read checkpoints, and clarify watcher responsibilities.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@GeorgeNgMsft
George Ng (GeorgeNgMsft) added this pull request to the merge queue Oct 3, 2026
Comment thread ts/packages/git-story/src/sessionWatcher.ts Outdated
@datduyng
Dominic Nguyen (datduyng) removed this pull request from the merge queue due to a manual request Oct 3, 2026
Outline one-time GUID assignment for ID-less events and clarify byte-offset capture checkpoints.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Separate byte-offset resumption from event correlation and document persisted GUID assignments for replay.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@GeorgeNgMsft
George Ng (GeorgeNgMsft) added this pull request to the merge queue Oct 3, 2026
Merged via the queue into main with commit 33256bd Oct 3, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants