Skip to content

docs: custom model definitions#6910

Merged
dianed-square merged 3 commits intomainfrom
docs/custom-model-definition
Feb 4, 2026
Merged

docs: custom model definitions#6910
dianed-square merged 3 commits intomainfrom
docs/custom-model-definition

Conversation

@dianed-square
Copy link
Contributor

@dianed-square dianed-square commented Feb 3, 2026

Summary

This PR documents GOOSE_PREDEFINED_MODELS support.

Documentation updates:

  • documentation/docs/guides/environment-variables.md:
    • Add "Custom Model Definitions" subsection to "Model Configuration"

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

Manual testing (minimal)


@dianed-square dianed-square requested a review from a team as a code owner February 3, 2026 02:56
Copilot AI review requested due to automatic review settings February 3, 2026 02:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Documents the new GOOSE_PREDEFINED_MODELS environment variable and how to define custom model configurations for goose.

Changes:

  • Added a “Custom Model Definitions” subsection under “Model Configuration”.
  • Documented the GOOSE_PREDEFINED_MODELS variable and the expected JSON fields for custom models.
  • Included examples for Anthropic and OpenAI/Databricks custom models, including context limits and provider-specific request parameters.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-02-04 15:17 UTC

| `request_params` | No | object | Provider-specific parameters included in API requests |

:::info
The `id`, `alias`, and `subtext` fields are currently not used.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Desktop will use these but desktop support still needs some iteration

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah main thing is you can't add context or request_params for models through the ui. It functionally works in desktop with it configured through the environment variables only at this point, no way to visually see/edit them etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

k thanks, but this note is okay for now? I wasn't able to get the env var to apply to a Desktop session launched from the Dock (tried in my .zshrc file)

@dianed-square dianed-square added this pull request to the merge queue Feb 4, 2026
Merged via the queue into main with commit f3d969c Feb 4, 2026
20 checks passed
@dianed-square dianed-square deleted the docs/custom-model-definition branch February 4, 2026 15:12
stebbins pushed a commit to stebbins/goose that referenced this pull request Feb 4, 2026
Signed-off-by: Harrison <hcstebbins@gmail.com>
kuccello pushed a commit to kuccello/goose that referenced this pull request Feb 7, 2026
Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026
Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026
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.

4 participants