Skip to content

Add browser login and personal todo ownership - #74

Merged
walkerhughes merged 2 commits into
supabase-rbac-mcpfrom
rbac-todo-browser-auth
Sep 14, 2026
Merged

walkerhughes merged 2 commits into
supabase-rbac-mcpfrom
rbac-todo-browser-auth

Conversation

@walkerhughes

Copy link
Copy Markdown
Owner

Adds browser sign-in and signup to rbac-todo, with personal lists instead of a shared list. New accounts get admin access to their own todos; members can read only their own rows. Supabase Auth handles passwords, and Codex uses OAuth with PKCE and refresh tokens instead of exported bearer tokens.

Stacked on #73. Merge that PR first, then retarget this one to main.

  • Minimal React/shadcn sign-in, signup, email-confirmation notice, and consent page: https://rbac-todo-login.banachspace.chatgpt.site
  • Ownership RLS and (user_id, created_at DESC, id DESC) index. Migration preserved all five existing demo todos under the sole existing admin and refuses ambiguous ownership on other deployments.
  • OAuth discovery in the Edge Function, plugin 0.2.0 configurations, installation docs, and CI checks.

Validation: Deno formatting/lint/type checks, Python configuration test and Ruff, web lint/build, zero npm audit findings, desktop/mobile browser checks, live PKCE consent/token refresh/revocation, native codex mcp login, and deployed MCP/REST tests for CRUD, two-user isolation, forged ownership, member write denial, and immediate role demotion. Temporary test users were removed. Schema and Edge Function are deployed to supabase-demo; the sign-in site is public.

Remaining deployment setup: configure custom SMTP in Supabase for new-user confirmation emails. No SMTP provider is configured yet. Returning confirmed users can log in; arbitrary new-user email delivery and the full email-confirmation journey remain unverified. Email confirmation stays enabled.

@walkerhughes
walkerhughes added this pull request to stack #75 September 14, 2026 04:35
@walkerhughes
walkerhughes marked this pull request as ready for review September 14, 2026 04:50
@walkerhughes
walkerhughes merged commit 07164a7 into main Sep 14, 2026
4 checks passed
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.

1 participant