feat: complete TS/Python parity — session store + full research package (53 tests)#620
Merged
jayscambler merged 1 commit intomainfrom Apr 4, 2026
Merged
feat: complete TS/Python parity — session store + full research package (53 tests)#620jayscambler merged 1 commit intomainfrom
jayscambler merged 1 commit intomainfrom
Conversation
678dde2 to
a972c8c
Compare
Closes all parity gaps between Python and TypeScript: session/store.ts (6 tests) SQLite-backed SessionStore matching Python's session/store.py. Added toJSON/fromJSON to Session and Turn for round-trip serde. research/types.ts (7 tests) ResearchQuery, Citation, ResearchResult, ResearchConfig, Urgency. Full serde support via toJSON/fromJSON. research/runtime.ts (7 tests) ResearchEnabledSession: adapter attachment, budget, events, history. research/consultation.ts (9 tests) ResearchBrief value object + ResearchConsultant domain service. Confidence filtering, citation dedup, markdown rendering. research/persistence.ts (7 tests) ResearchStore: JSON-file-backed brief persistence with manifest. research/prompt-wiring.ts (8 tests) ResearchPromptInjector: format briefs for LLM context injection. research/evaluation.ts (9 tests) EvalResult, BatchSummary, ResearchEvaluator: A/B comparison. Total: 7 new modules, 53 new tests. Combined with existing session modules: 17 TS modules with 136 tests matching 17 Python modules. Full cross-language parity achieved.
a972c8c to
cc471f8
Compare
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.
Summary
Closes all remaining TS/Python parity gaps. 7 new modules, 53 new tests, 1,131 lines.
Parity Matrix — Complete
Session Package (11 modules each)
Research Package (6 modules each)
17 Python modules ↔ 17 TS modules. Full parity.
Verification
tsc --noEmit— zero errors