docs: list edge functions the SPA calls. Closes #4 - #15
Conversation
Create a new branch for this commit and start a pull request.
docs: list edge functions the SPA calls
docs: list edge functions the SPA calls
telivity-otaip
left a comment
There was a problem hiding this comment.
Thank you for taking this on — really appreciate the careful write-up for #4, and especially flagging that the invokes live in web/ui/state.mjs rather than index.html. The table shape is exactly what we needed.
A couple of small accuracy fixes before we merge, if you don’t mind:
- Complete the SPA list — please also document
upsert_provider_secretandclear_provider_secretfromweb/ui/settings.mjs(and update the “7 functions” wording). - Source note — several of these now have reference source under
supabase/functions/; onlyfetch-jdandhumanizeappear missing. A pointer to that README would help self-hosters a lot.
Optional nits (happy either way): rename the PR title to something like docs: list edge functions the SPA calls, add a trailing newline, and Closes #4 in the description.
Thanks again — happy to merge once those land.
|
Thanks for the review! Pushing fixes now: Adding upsert_provider_secret and clear_provider_secret (both in web/ui/settings.mjs) to the table, updating 7 functions → 9 functions. |
|
Done |
|
Thanks again for the solid draft and for jumping on the review notes so quickly — the table shape and the It looks like the README commit didn’t quite make it onto the branch yet, so we landed the remaining accuracy fixes in #16 (all 9 invokes, corrected source note pointing at |
Address review feedback on TelivityAI#15: document upsert/clear_provider_secret, point source note at supabase/functions (fetch-jd and humanize still absent), and clarify BYO OpenAI-compat bypass. Co-authored-by: Mbali Simelane <mbalisimelane633@gmail.com>
|
Pushed a small follow-up commit onto this branch with the remaining review fixes (all 9 invokes, corrected source note, trailing newline) so you keep authorship/credit on the PR — thanks again for the solid draft. Closing #16 in favor of this one. Happy to merge whenever you’re ready. |
telivity-otaip
left a comment
There was a problem hiding this comment.
Thanks again — the remaining review notes are on the branch now. Looks good to merge from my side.
Summary
Adds the Edge functions section to
web/README.mdper #4. Documents the 9 distinct Supabase Edge Functions the SPA calls (run-search-mt,fetch-jd,resume-match,resume-rewrite,chat,ai-free,humanize), when each is triggered in the UI, and whether it's required for a minimal board-only install.Note: the calls actually live in
web/ui/state.mjs(loaded as a module byweb/index.html), not inindex.htmlitself worth flagging in case the issue's grep instructions get reused elsewhere.Type
Checklist
web/config.jsnot committed)