🧹 [code health improvement] avoid unnecessary vec clone when converting database params

[Tcode-Motion]

🎯 What: Combined get_params_list and params_list_to_sqlite_params into a single extract_sqlite_params function in stdlib/src/database.rs to avoid cloning an entire Vec<RuntimeValue>.

💡 Why: The previous code fetched the runtime values into an intermediate Vec<RuntimeValue> clone just to iterate over it again and map it to a Vec<rusqlite::types::Value>. By directly mapping over the borrowed list items, we avoid the unnecessary clone and memory allocation, improving code health and performance.

✅ Verification: Verified by checking format (cargo fmt --all), running clippy (cargo clippy --workspace --all-targets), and running the full test suite (cargo test --workspace), which all pass successfully.

✨ Result: The codebase is cleaner and slightly more memory-efficient when querying or executing statements in the database module.

---

PR created automatically by Jules for task 4768248487551497203 started by @Tcode-Motion

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.