Skip to content

(experiment) llm tools: position before user prompt and after#232

Closed
mialso wants to merge 11 commits into
mainfrom
experiment/llm-tool-call-before-and-after
Closed

(experiment) llm tools: position before user prompt and after#232
mialso wants to merge 11 commits into
mainfrom
experiment/llm-tool-call-before-and-after

Conversation

@mialso

@mialso mialso commented Feb 11, 2026

Copy link
Copy Markdown
Contributor

Description:

  • LLM: experiment tools different positions: before user prompt and after

tool call correctness test (no cache) ✔️ :

bare examples/toolCalling.js

Results: pass ✔️

  • 'Qwen3-1.7B-Q4_0.gguf' model - the results are the same, regardless tools go before or after the user prompt

dynamic tool call cache test ❌ :

cd ./packages/qvac-sdk
bun run build:pack
bun run examples/llamacpp-dynamic-tools.ts

Results: fail ❌

  • 'Qwen3-1.7B-Q4_0.gguf' model - dynamic tool calls invalidates the cache, model doesn't remember it's previous answer

@elchiapp elchiapp closed this Mar 4, 2026
@elchiapp elchiapp reopened this Mar 4, 2026
@mialso

mialso commented Mar 20, 2026

Copy link
Copy Markdown
Contributor Author

implemented #706 and #745

@mialso mialso closed this Mar 20, 2026
@mialso mialso deleted the experiment/llm-tool-call-before-and-after branch March 20, 2026 08:25
@github-actions

Copy link
Copy Markdown
Contributor

Tier-based Approval Status

**PR Tier:** TIER1

**Current Status:** ❌ PENDING

**Requirements:**
- 1 Team Member approval ❌ (0/1)
- 1 Team Lead OR Management approval ❌ (0/1)



---
*This comment is automatically updated when reviews change.*

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