Skip to content

adding SummarizingAgent#174

Open
ndebard wants to merge 1 commit intolanl:mainfrom
ndebard:feature/summarizing-agent
Open

adding SummarizingAgent#174
ndebard wants to merge 1 commit intolanl:mainfrom
ndebard:feature/summarizing-agent

Conversation

@ndebard
Copy link
Collaborator

@ndebard ndebard commented Feb 3, 2026

This adds a new, fairly simple agent called the SummarizingAgent. It has some interesting features where it can summarize a set of documents (rather than just 1 doc) and does some neat things with chunking up the docs and summarizing in pieces, merging, etc.

  • Prompts are added to the prompt library.
  • An example is provided in single_agents/summarizing_agent/
  • Unit tests are provided

Below is a sample output of running the example, which is itself explained in a README.md file.

image

@ndebard
Copy link
Collaborator Author

ndebard commented Feb 3, 2026

summary.txt

I've attached here a summary example of above, after it summarized those 3 freely available novels. Just in case it's useful to any reviewer. Obviously this stuff is stochastic.

@mikegros
Copy link
Collaborator

mikegros commented Mar 9, 2026

This looks good but you had some thoughts about whether to put it in the repo or not.

Also, there is one system message test it fails for me:

test_summarizing_agent_rewrite_pass_triggers_and_scrubs_forbidden_refs - AssertionError: Unexpected system prompt passed to LLM: "You are a summarization engine.\nHard constraints: do NOT mention filenames, file paths, URLs/links, sources, documents, chunks, excerp...

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.

2 participants