Skip to content

Conversation

@markijbema
Copy link
Contributor

  • Add supportsFim(), completeFim(), and streamFim() methods to MistralHandler
  • FIM support is only enabled for codestral models (codestral-latest, codestral-2405, etc.)
  • Uses Mistral's v1/fim/completions endpoint with proper base URL routing
  • Supports both streaming and non-streaming FIM completions
  • Add comprehensive test coverage with 10 test cases
  • Update continuedev Mistral LLM to accurately check model support
  • Implementation follows the same pattern as KiloCode OpenRouter handler

Closes: FIM functionality now works for Mistral's codestral models

Context

Implementation

Screenshots

before after

How to Test

Get in Touch

@changeset-bot
Copy link

changeset-bot bot commented Nov 18, 2025

⚠️ No Changeset found

Latest commit: 50cdf29

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@markijbema markijbema force-pushed the mark/fim-for-mistral-provider branch from 1871d6f to 1bcbaca Compare November 21, 2025 10:32
@markijbema
Copy link
Contributor Author

this works, but you have to change the base url in the provider, and ask specific access with a different key

CleanShot 2025-11-21 at 11 44 29@2x

@markijbema markijbema force-pushed the mark/fim-for-mistral-provider branch from 1bcbaca to 00c6fda Compare November 21, 2025 16:23
- Add supportsFim(), completeFim(), and streamFim() methods to MistralHandler
- FIM support is only enabled for codestral models (codestral-latest, codestral-2405, etc.)
- Uses Mistral's v1/fim/completions endpoint with proper base URL routing
- Supports both streaming and non-streaming FIM completions
- Add comprehensive test coverage with 10 test cases
- Update continuedev Mistral LLM to accurately check model support
- Implementation follows the same pattern as KiloCode OpenRouter handler

Closes: FIM functionality now works for Mistral's codestral models
@markijbema markijbema force-pushed the mark/fim-for-mistral-provider branch from 00c6fda to 75b04e9 Compare November 21, 2025 16:40
})),
}
})
// kilocode_change - new file
Copy link
Contributor Author

Choose a reason for hiding this comment

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

no it isnt smartass llm

return finalOptions
}

// kilocode_change start
Copy link
Contributor Author

Choose a reason for hiding this comment

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

unnecessary

Copy link
Contributor Author

Choose a reason for hiding this comment

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

(but we dont care about this file)

- Removed incorrect 'new file' marker from mistral.spec.ts
- Restored original test structure to minimize changes
- Created new mistral-fim.spec.ts for FIM-specific tests
- All tests passing
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.

2 participants