Skip to content

Add configurable temperature, language, and folder-level custom context#167

Open
shenxianpeng wants to merge 6 commits into
mainfrom
fix-159
Open

Add configurable temperature, language, and folder-level custom context#167
shenxianpeng wants to merge 6 commits into
mainfrom
fix-159

Conversation

@shenxianpeng
Copy link
Copy Markdown
Member

@shenxianpeng shenxianpeng commented May 10, 2026

Description

  • Add temperature setting (null = omit, letting provider defaults apply) to GlobalConfigurationImpl, ExplainErrorStep, ExplainErrorFolderProperty
  • Thread temperature through BaseAIProvider.explainError() into each provider's createAssistant(), replacing hardcoded 0.3
  • Add language setting to GlobalConfigurationImpl and ExplainErrorFolderProperty with cascade: step > folder > global
  • Add customContext setting to ExplainErrorFolderProperty with cascade: step > folder > global
  • Update manual button (ConsoleExplainErrorAction) path to also resolve language/customContext/temperature from folder/global

Related Issues

Closes #160

Testing done

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

Was generative AI tooling used to co-author this PR?
  • Yes (please specify below)

Generated-by: Pi + DeepSeek v4

@shenxianpeng shenxianpeng added the major-rfe For changelog: Major enhancement. Will be highlighted on the top label May 10, 2026
Comment thread src/main/java/io/jenkins/plugins/explain_error/ErrorExplainer.java Fixed
… context

- Add temperature setting (null = omit, letting provider defaults apply)
  to GlobalConfigurationImpl, ExplainErrorStep, ExplainErrorFolderProperty
- Thread temperature through BaseAIProvider.explainError() into each
  provider's createAssistant(), replacing hardcoded 0.3
- Add language setting to GlobalConfigurationImpl and
  ExplainErrorFolderProperty with cascade: step > folder > global
- Add customContext setting to ExplainErrorFolderProperty with
  cascade: step > folder > global
- Update manual button (ConsoleExplainErrorAction) path to also
  resolve language/customContext/temperature from folder/global

Closes #160
@shenxianpeng shenxianpeng marked this pull request as ready for review May 24, 2026 12:19
@shenxianpeng shenxianpeng requested a review from a team as a code owner May 24, 2026 12:19
…MicrosoftFoundry fix path

- Restore hardcoded temperature 0.3 in CustomOktaAIProvider.buildFixRequestBody() that was accidentally removed
- Change MicrosoftFoundryProvider buildChatModel() to omit temperature (null) for fix path, consistent with PR philosophy
Keep both descriptorListsQuotaWindowOptions (from #183) and
the new prompt settings cascade tests in the same file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major-rfe For changelog: Major enhancement. Will be highlighted on the top

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support for detailed configuration at root and folder level

1 participant