This repository was archived by the owner on Mar 27, 2026. It is now read-only.
Merged
Conversation
Also refactored and simplified config and main file. From now is not needed to specify summarizer type because, program uses only Ollama BREAKING CHANGE: OpenAI and Claude are no avaialbale for summarizing anymore. the type field in config is removed
From now user needs to read emails and prepare data for summarizing outside this program. nitrodigest from now do two things: 1. summmarize (one file or files in a directory) 2. save summarized text to a file User can pass file or folder via argument. Besides I simplified config and from now it's a flat objects BREAAKING CHANGE: program doesn't process emails, it expects that text to summarize will be provided by user
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors NitroDigest by removing email processing and remote LLM support in favor of a simplified, local-only text/directory summarization workflow. Key changes include:
- Removal of Claude and OpenAI providers and associated email processing components.
- Updates to the CLI to accept a file or directory as input.
- Config and documentation changes to reflect the focus on local summarization via Ollama.
Reviewed Changes
Copilot reviewed 16 out of 19 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| nitrodigest/summary_writer.py | Updated file writing logic and logging message improvements. |
| nitrodigest/summarizer/providers/openai.py | Removed OpenAI summarizer implementation. |
| nitrodigest/summarizer/providers/claude.py | Removed Claude summarizer implementation. |
| nitrodigest/summarizer/providers/init.py | Removed unused provider imports. |
| nitrodigest/summarizer/ollama.py | Updated variable names and API endpoints for clarity. |
| nitrodigest/main.py | Reworked CLI and processing logic to accept file/directory input. |
| nitrodigest/config.py | Simplified configuration data structure. |
| nitrodigest/README.md | Updated documentation and configuration snippet. |
Files not reviewed (3)
- nitrodigest/config.json.example: Language not supported
- nitrodigest/prompt_template2.txt: Language not supported
- nitrodigest/requirements.txt: Language not supported
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
BREAKING CHANGES:
After this refactoring, the usage of this tool looks like this:
Config structure:
{ "model": "mistral", "ollama_api_url": "http://localhost:11434", "timeout": 300, "prompt_file": "prompt_template.txt", "summaries_path": "summaries" }I updated README plus added some sample newsletters for testing purposes.
nitrodigest in action now:
Closes #119