Skip to content

Add ability to configure tabAutocompleteOptions on a per-model basis in config.yaml #5970

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

jpoly1219
Copy link
Collaborator

@jpoly1219 jpoly1219 commented Jun 3, 2025

Description

Implement #4020.

Updated the config.yaml reading process so that we can now read in autocompleteOptions on a per-model basis. The config.yaml using this would look like so:

name: Jacob
version: 1.0.8
schema: v1
models:
#...
  - uses: # your model here...
    with:
      # your key here...
    override:
      autocompleteOptions:
        debounceDelay: 5000
        maxPromptTokens: 1500
        disableInFiles:
          - "*.md"
      # other overrides here...
#...

A good next step would be to:

  • Update our docs to reflect this change.
  • Slowly transition the internal naming scheme from TabAutocomplete to Autocomplete.

Checklist

  • [*] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [*] The relevant tests, if any, have been updated or created

Screenshots

[ For visual changes, include screenshots. Screen recordings are particularly helpful, and appreciated! ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]

@jpoly1219 jpoly1219 requested a review from a team as a code owner June 3, 2025 21:59
@jpoly1219 jpoly1219 requested review from RomneyDa and removed request for a team June 3, 2025 21:59
Copy link

cubic-dev-ai bot commented Jun 3, 2025

Your cubic subscription is currently inactive. Please reactivate your subscription to receive AI reviews and use cubic.

Copy link

netlify bot commented Jun 3, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 55e1dc9
🔍 Latest deploy log https://app.netlify.com/projects/continuedev/deploys/68420a90132ea70008dfc3a2

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 3, 2025
@jpoly1219 jpoly1219 requested review from sestinj and removed request for RomneyDa June 3, 2025 22:00
Copy link
Contributor

@sestinj sestinj left a comment

Choose a reason for hiding this comment

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

few quick comments, but this looks basically perfect!

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Jun 5, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 6, 2025
@sestinj sestinj merged commit a73cbd1 into main Jun 6, 2025
65 of 67 checks passed
@sestinj sestinj deleted the jacob/feature/config-yaml-per-model-tabAutoCompleteOptions branch June 6, 2025 15:42
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Jun 6, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jun 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants