DanCardin

I'd have to double check the code, but i wouldn't expect it to. There might be a user introspectable marker for whether it has been populated yet, but i haven't...

`language` and `returns` are both already available, unless i'm missing a particular aspect of your question. `security definer` at least seems straightforward to add. almost certainly `security=FunctionSecurity.defineer` and/or `function.security_definer()`

I'd appreciate it if you could test the PR out ahead of my merging it, to make sure it does what you're anticipating it do.

it feels like that setting would have to be itself an env var (or uv global setting) to make any sense to me... Like if you're explicitly describing your index-url...

I'm not sure exactly what mitigation makes the most sense, but basically my point was that the operator of the environment ought to be the one controlling access (default or...

~ah, you'll also need to rebase onto main, where i fixed the CI problem.~ JK i just rebased the PR version, but just something to be aware of if/when you...

i think this is still fairly untested, particularly the different kinds of input types. but i prototyped pulling the individual argument components into a `FunctionParam` construct, rather than attempting to...

Odds any interested parties want to test out the PR in its current state? It does currently 'regress' relative to the original suggestion allowing `returns='table(a int)'`, requiring a more strict...

indeed. the current support is very oriented around support for triggers. i dont personally have a need for function arguments, but if i can extrapolate from your need for some...

I decided to just merge it since no one responded and as far as my tests show, there's no effect on o.g. functions so 🤷. Feel free to submit follow...