Skip to content

Conversation

bhouston
Copy link
Member

@bhouston bhouston commented Mar 6, 2025

Add maxTokens and temperature config options to CLI

Description

This PR adds two new configuration options to the CLI:

  • maxTokens - with a default value of 4096
  • temperature - with a default value of 0.7

These options can be configured either through command-line arguments or through the config file, similar to how modelProvider and modelName are currently specified.

Changes

  • Added maxTokens and temperature to the default config in config.ts
  • Added these options to the SharedOptions type in options.ts
  • Added them to the sharedOptions object in options.ts
  • Updated $default.ts to pass these options to the agent
  • Updated tests to include the new configuration options

Testing

All tests are passing.

Closes #118

bhouston added 3 commits March 6, 2025 10:49
- Add maxTokens config option with default value of 4096
- Add temperature config option with default value of 0.7
- Update the CLI to pass these options to the generateText function
- Update tests to include the new configuration options

Closes #118
Ensure that the maxTokens config option is actually used in the text generation
by adding it to the generateTextProps object in toolAgentCore.ts
@bhouston bhouston merged commit 6201f39 into main Mar 6, 2025
1 check passed
Copy link

sentry-io bot commented Mar 7, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ Error: Error calling OpenAI API: OpenAI API error: 404 { OpenAIProvider.generateText(openai.ts) View Issue

Did you find this useful? React with a 👍 or 👎

Copy link

🎉 This PR is included in version mycoder-agent-v1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version mycoder-v1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@bhouston bhouston deleted the feature/add-max-tokens-temperature-config branch March 12, 2025 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add maxTokens and temperature config options to CLI

1 participant