Secrets

[lmangani]

The extension should leverage Secrets as an option to safely store parameters across sessions.

[aborruso]

Ciao @lmangani , I have a related question: now is there no way to save default api_url, api_token and api_key?
Do I have to set them every time at startup?

Thank you for this great tool

[lmangani]

CIao @aborruso at this time the settings are indeed required, but if there's interest I'll get started on stored SECRETS 👍

[aborruso]

CIao @aborruso at this time the settings are indeed required, but if there's interest I'll get started on stored SECRETS 👍

Grazie @lmangani, my use case is very trivial: I would like to open duckdb cli and use this fantastic work of yours, as available by default (with automatic load in .rc file), and without every time writing these parameters.

And set these parameters only when I don't want to use the default ones. If you think this is helpful, and you can put time and head into it, thank you very much. I would like to help you, but I don't know how to do it.

[lmangani]

@aborruso here's an untested PR with SECRETS support and instructions. Would you able to build and validate it locally?

Instructions

Clone and build the PR branch locally:

cd /usr/src
git clone -b secret-manager --recurse-submodules https://github.com/quackscience/duckdb-extension-openprompt
cd duckdb-extension-openprompt
OVERRIDE_GIT_DESCRIBE="v1.1.3" GEN=ninja make

Run the local DuckDB build:

./build/release/duckdb

Add and test SECRETS

D CREATE SECRET IF NOT EXISTS open_prompt (
      TYPE open_prompt,
      PROVIDER config,
      api_token 'your-api-token',
      api_url 'http://localhost:11434/v1/chat/completions',
      model_name 'qwen2.5:0.5b',
      api_timeout '30'
  );

Testing

At this point the parameters should be derived from SECRETS instead of SET parameters.

[lmangani]

I also added ENV parameters to allow provisioning the process from the system at runtime

    OPEN_PROMPT_API_URL
    OPEN_PROMPT_API_TOKEN
    OPEN_PROMPT_MODEL_NAME
    OPEN_PROMPT_API_TIMEOUT

[aborruso]

Grazie @lmangani , I will try tomorrow.