Skip to content

docs: list edge functions the SPA calls. Closes #4 - #15

Merged
telivity-otaip merged 4 commits into
TelivityAI:mainfrom
Acacia21-code:Acacia21-code-patch-2
Aug 12, 2026
Merged

docs: list edge functions the SPA calls. Closes #4#15
telivity-otaip merged 4 commits into
TelivityAI:mainfrom
Acacia21-code:Acacia21-code-patch-2

Conversation

@Acacia21-code

@Acacia21-code Acacia21-code commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds the Edge functions section to web/README.md per #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 by web/index.html), not in index.html itself worth flagging in case the issue's grep instructions get reused elsewhere.

Type

  • Bug fix
  • Feature
  • Docs / deploy
  • Chore

Checklist

  • No secrets or personal data in the diff (web/config.js not committed)
  • Docs updated if behavior or deploy steps changed
  • I agree this contribution is under Apache-2.0

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 telivity-otaip left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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:

  1. Complete the SPA list — please also document upsert_provider_secret and clear_provider_secret from web/ui/settings.mjs (and update the “7 functions” wording).
  2. Source note — several of these now have reference source under supabase/functions/; only fetch-jd and humanize appear 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.

Comment thread web/README.md Outdated
Comment thread web/README.md
@Acacia21-code

Acacia21-code commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

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.
Adding a source note pointing to supabase/functions/README.md, and calling out that fetch-jd and humanize are the two without source in this repo.
Renamed the PR title, added Closes #4, and a trailing newline.

@Acacia21-code Acacia21-code changed the title Acacia21 code patch 2 docs: list edge functions the SPA calls. Closes #4 Aug 11, 2026
@Acacia21-code

Copy link
Copy Markdown
Contributor Author

Done

@telivity-otaip

Copy link
Copy Markdown
Contributor

Thanks again for the solid draft and for jumping on the review notes so quickly — the table shape and the state.mjs catch were spot on.

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 supabase/functions/, trailing newline). Happy to credit you there; feel free to close this PR in favor of that one whenever you’re ready.

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>
@telivity-otaip

Copy link
Copy Markdown
Contributor

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 telivity-otaip left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again — the remaining review notes are on the branch now. Looks good to merge from my side.

@telivity-otaip
telivity-otaip merged commit 4d1c903 into TelivityAI:main Aug 12, 2026
@Acacia21-code
Acacia21-code deleted the Acacia21-code-patch-2 branch August 12, 2026 09:55
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.

3 participants