Skip to content

fix: summarize single daily exchanges - #104

Open
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:fix/summarize-single-exchange
Open

fix: summarize single daily exchanges#104
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:fix/summarize-single-exchange

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Generate a Claude daily summary when a channel has one complete user-and-bot exchange.
  • Keep the existing raw-log fallback for empty or single-line logs.
  • Add regression coverage and an Unreleased changelog entry.

Verification

  • npm ci — passed; npm reported 13 existing audit vulnerabilities and a pending node-pty install-script approval warning.
  • npm run build — passed.
  • node --test tests/summaryWorkload.test.mjs — passed (2 tests).
  • npm test — passed (66 tests).
  • Direct node-pty import probe — passed.
  • git diff --check upstream/main...HEAD — passed.
  • TDD RED: the new focused test failed before the fix because two-line logs were copied without invoking the summary runner; GREEN passed after changing the minimum meaningful log to two lines.

Risk

Low. This only changes the daily-summary threshold from three lines to two, allowing one complete exchange to be summarized; empty and one-line logs retain their existing fallback behavior.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

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.

1 participant