docs/test: add TSDoc and unit tests for LLM processors, summarizer, security plugin, a2a utils, and compacted events#413
Open
nuthalapativarun wants to merge 7 commits into
Conversation
…sor, and RequestConfirmationLlmRequestProcessor Add @param and @yields TSDoc to the runAsync overrides in the three processors that were missing inline documentation.
…cyEngine, InMemoryPolicyEngine, and SecurityPlugin
…ionResponseProcessor
This was referenced Jun 6, 2026
kalenkevich
approved these changes
Jun 8, 2026
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
Consolidates six separate docs/test PRs (#397, #398, #399, #400, #401, #402) into one as requested by @kalenkevich.
TSDoc additions:
BasicLlmRequestProcessor,IdentityLlmRequestProcessor,RequestConfirmationLlmRequestProcessor(closes docs: add TSDoc to BasicLlmRequestProcessor, IdentityLlmRequestProcessor, and RequestConfirmationLlmRequestProcessor #397)LlmSummarizerOptions,LlmSummarizer(closes docs: add TSDoc to LlmSummarizerOptions and LlmSummarizer #398)PolicyCheckResult,ToolCallPolicyContext,BasePolicyEngine,InMemoryPolicyEngine,SecurityPlugin(closes docs: add TSDoc to SecurityPlugin, policy engine interfaces, and InMemoryPolicyEngine #399)a2autility functions —@paramand@returnsfor all public exports (closes docs: add @param and @returns TSDoc to a2a utility functions #401)Unit tests:
isCompactedEventandcreateCompactedEvent(closes test: add unit tests for isCompactedEvent and createCompactedEvent #400)CodeExecutionRequestProcessorandCodeExecutionResponseProcessor(closes test: add unit tests for CodeExecutionRequestProcessor and CodeExecutionResponseProcessor #402)Test plan
npm testpasses (all new test files included)🤖 Generated with Claude Code