Skip to content

Conversation

fredzqm
Copy link
Contributor

@fredzqm fredzqm commented May 15, 2025

Mostly refactors to make existing RTDB init flow fit into the MCP flow.

Init from an empty folder

image

firebase init database
Screenshot 2025-05-14 at 5 08 42 PM

Copy link
Contributor

@mbleigh mbleigh left a comment

Choose a reason for hiding this comment

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

It's a little lopsided to have this as our second init when we don't otherwise have RTDB tools, but 🤷 . Let's target Firestore next.

Comment on lines 20 to 27
rulesFilename: z
.string()
.default("database.rules.json")
.describe("The file to use for Realtime Database Security Rules."),
rules: z
.string()
.default(DEFAULT_RULES)
.describe("The security rules to use for Realtime Database Security Rules."),
Copy link
Contributor

Choose a reason for hiding this comment

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

Make both of these .optional() in addition to having defaults defined.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure

@fredzqm
Copy link
Contributor Author

fredzqm commented May 15, 2025

It's a little lopsided to have this as our second init when we don't otherwise have RTDB tools, but 🤷 . Let's target Firestore next.

Only doing it because I already got familiar with its init code~ Firestore and storage will come very soon.

@fredzqm fredzqm enabled auto-merge (squash) May 15, 2025 05:39
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.01%. Comparing base (0b54e4d) to head (922c874).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8584      +/-   ##
==========================================
- Coverage   51.03%   51.01%   -0.03%     
==========================================
  Files         426      426              
  Lines       30637    30664      +27     
  Branches     6286     6292       +6     
==========================================
+ Hits        15636    15642       +6     
- Misses      13607    13628      +21     
  Partials     1394     1394              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fredzqm fredzqm merged commit 12f569e into master May 15, 2025
49 of 50 checks passed
@github-project-automation github-project-automation bot moved this from Approved [PR] to Done in [Cloud] Extensions + Functions May 15, 2025
@fredzqm fredzqm deleted the fz/mcp branch May 15, 2025 05:50
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