Skip to content

feat: add summarization_provider config for compact component and con…#49

Merged
iohub merged 1 commit into
mainfrom
feat-openai-go
May 6, 2026
Merged

feat: add summarization_provider config for compact component and con…#49
iohub merged 1 commit into
mainfrom
feat-openai-go

Conversation

@iohub
Copy link
Copy Markdown
Owner

@iohub iohub commented May 6, 2026

…vert prompts to English

  • Convert defaultSummarizationPrompt from Chinese to English with structured Markdown format (Role → Task → Rules → Output Format) matching agent prompt style
  • Add summarization_provider field to [context] config section, allowing compact summarization to use an independent LLM provider from global.llm.providers
  • Extend ContextCompactConfig, compact.Config, and ConfigFrom with SummarizationProvider field
  • Create independent summaryEngine in app.go when summarization_provider is set, falling back to main engine for backward compatibility
  • Convert getSummarizationPrompt() in conductor.go from Chinese to English
  • Export GetProvider in config package and add NewLoggingEngine factory in llm package
  • Update all NewConductorAgent call sites in tests to pass nil summaryEngine

…vert prompts to English

- Convert defaultSummarizationPrompt from Chinese to English with structured
  Markdown format (Role → Task → Rules → Output Format) matching agent prompt style
- Add summarization_provider field to [context] config section, allowing compact
  summarization to use an independent LLM provider from global.llm.providers
- Extend ContextCompactConfig, compact.Config, and ConfigFrom with
  SummarizationProvider field
- Create independent summaryEngine in app.go when summarization_provider is set,
  falling back to main engine for backward compatibility
- Convert getSummarizationPrompt() in conductor.go from Chinese to English
- Export GetProvider in config package and add NewLoggingEngine factory in llm package
- Update all NewConductorAgent call sites in tests to pass nil summaryEngine
Copy link
Copy Markdown

@sourcery-ai sourcery-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.

Sorry @iohub, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@iohub iohub merged commit a2bc10c into main May 6, 2026
1 check passed
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