Skip to content

Conversation

@danpung2
Copy link

@danpung2 danpung2 commented Feb 11, 2026

Summary

  • Add positive-path tests for session-manager todo and transcript readers.

Changes

  • Add readSessionTodos happy-path test using a session-matching todo JSON file
  • Add readSessionTranscript happy-path test counting non-empty lines in .jsonl

Testing

bun run typecheck
bun test

Summary by cubic

Add happy-path tests for session-manager storage to verify todo parsing and transcript line counting.
Covers readSessionTodos mapping from a matching JSON file and readSessionTranscript returning the count of non-empty lines in .jsonl.

Written for commit 6a050a3. Summary will update on new commits.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 11, 2026

All contributors have signed the CLA. Thank you! ✅
Posted by the CLA Assistant Lite bot.

@danpung2
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Feb 11, 2026
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Requires human review: Cannot be 100% sure no regressions. The PR adds readSessionTranscript to imports but no implementation is shown in the diff - risk of undefined function. Also, new test expects lineCount=4 but logic

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