Skip to content

feat: improve thought segments syntax extraction - #636

Open
giladgd wants to merge 8 commits into
masterfrom
gilad/adaptToBreakingChanges
Open

feat: improve thought segments syntax extraction#636
giladgd wants to merge 8 commits into
masterfrom
gilad/adaptToBreakingChanges

Conversation

@giladgd

@giladgd giladgd commented Aug 4, 2026

Copy link
Copy Markdown
Member

Description of change

  • feat: improve thought segment syntax extraction
  • feat: expose the model's architecture directly on the model instance
  • fix: adapt to breaking llama.cpp changes
  • fix: map GgmlType configs directly from native code

Fixes #634

Pull-Request Checklist

  • Code is up-to-date with the master branch
  • npm run format to apply eslint formatting
  • npm run test passes with this change
  • This pull request links relevant issues as Fixes #0000
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change
  • The new commits and pull request title follow conventions explained in pull request guidelines (PRs that do not follow this convention will not be merged)

@giladgd
giladgd requested a review from ido-pluto August 4, 2026 05:17
@giladgd giladgd self-assigned this Aug 4, 2026
… thinking segment opening in a generation prompt
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.

bug: JinjaTemplateChatWrapper drops generation-prompt text after the assistant header

1 participant