Skip to content

ci: deploy Firestore rules and indexes alongside functions - #69

Merged
nick-transition merged 1 commit into
mainfrom
ci-deploy-firestore-rules
Jul 9, 2026
Merged

ci: deploy Firestore rules and indexes alongside functions#69
nick-transition merged 1 commit into
mainfrom
ci-deploy-firestore-rules

Conversation

@nick-transition

Copy link
Copy Markdown
Owner

Summary

The Deploy to Firebase Hosting workflow deployed hosting + functions but never Firestore rules, so the OAuth 2.1 rules lockdown from #68 (client-write ban on oauthClients, oauthGrants read access for the Connected Apps screen) is not yet live. This extends the deploy step to --only functions,firestore, which covers rules and indexes as configured in firebase.json.

Test plan

  • Merge triggers the deploy workflow; verify the firestore: released rules step succeeds in the run log
  • Prod check: Connected Apps screen can read grants; client-side oauthClients writes are denied

🤖 Generated with Claude Code

The deploy workflow only shipped hosting and functions, so rules
changes (like the OAuth 2.1 lockdown in #68) never reached prod.
firebase deploy --only firestore covers rules + indexes per
firebase.json.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@nick-transition
nick-transition merged commit e7d0d26 into main Jul 9, 2026
2 checks passed
@nick-transition
nick-transition deleted the ci-deploy-firestore-rules branch July 9, 2026 02:22
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