Add Remote config skills#113
Open
rathovarun1032 wants to merge 8 commits intonextfrom
Open
Conversation
* Add firebase-data-connect-basics skill * Add firebase dataconnect basic skills --------- Co-authored-by: Muhammad Talha <muhammadtalhar@google.com>
* Explanding out contributor guidelines (#3) * update claude plugin marketplace command to use the new repo (#2) * add kiro power (#4) * update license (#8) * Adding a token counting script (#11) * Next -> Main (#16) * Add firebase-data-connect-basics skill (#7) * Add firebase-data-connect-basics skill * Add firebase dataconnect basic skills --------- Co-authored-by: Muhammad Talha <muhammadtalhar@google.com> * [Dataconnect] Restructure SKILL.md into a development workflow guide --------- Co-authored-by: Muhammad Talha <muhammadtalhar@google.com> Co-authored-by: Muhammad Talha <126821605+mtr002@users.noreply.github.com> * Adding a Cursor plugin (#13) * Add Cursor plugin configuration * Add logo to cursor plugin config * Move logo to assets directory * Restructure Cursor plugin to multi-plugin format * Clean up language * Use a symlink instead * Add theme to GCLI extension (#17) * Remove curl|bash (#18) * Update SKILL.md to remove references to 2.5 and instead use `latest`. (#19) * feat: Add sync job for genkit-ai/skills (#23) * Change to use `npx` to invoke Firebase CLI (#26) Change all `firebase` shell command mentioned in the skills to use `npx -y firebase-tools@latest` instead to ensure freshness and reduce frictions to the agents. * use pat for cla reasons (#29) * fix: update workflow title and committer details (#33) * Move the local environment setup to a new skill + MCP setup (#31) - Move all one-time-only local environment setup from `firebase-basic` to `firebase-local-env-setup` skill - Add description about how to install skills and MCP server. * fix: Add reviewers to sync-genkit-skills workflow (#35) Based on https://github.com/peter-evans/create-pull-request?tab=readme-ov-file#action-inputs * chore: sync updated skills (#34) Co-authored-by: ssbushi <66321939+ssbushi@users.noreply.github.com> * Update skill names in security rules documentation (#37) * Support Agent Skills for Firestore Enterprise with Native Mode (#27) * Add agent skills for firestore enterprise with native mode * Amend SKILL.md * Format SKILL * Address comments and rename standard edition * Remove files & update frontmatter * Updated to use "npx firebase-tools@latest" --------- Co-authored-by: Sichen Liu <sichenliu@google.com> Co-authored-by: Joe Hanley <joehanley@google.com> * update the skill verification instruction (#38) * chore: sync updated skills (#39) * gemini-3-pro-preview --> gemini-3.1-pro-preview (#40) * Refine `firebase-local-env-setup` instructions for agents (#41) Updated the installation reference guides for all supported agents (Antigravity, Claude Code, Cursor, Gemini CLI, GitHub Copilot, and others) in the `firebase-local-env-setup` skill. - Added detailed steps to locate, verify, and safely merge MCP configurations (`mcp.json`, `claude_desktop_config.json`, etc.) without overwriting existing entries. - Added explicit instructions to check for existing skill installations using `npx skills list` before installing. - Enforced "stop and wait" instructions for user restarts to ensure the agent correctly verifies the MCP server connection securely. * Update `firebase-basics` skills (#42) * Update `firebase-basics` skills - Optimize description so that it is more likely to be loaded - Clearly state the prerequisites, how to validate them and how to meet them. - Add principles about how to optimize agent ability to help with Firebase related task - Indexing other knowledges for progressive disclosure. Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> * Update `firebase-basics` skill and `firebase-local-env-setup` skill (#43) - Update their description so that firebase-basics can be picked up more consistently - Update instructions about how to refresh skills for Antigravity and other agents. * Split out python sdk content (#45) * Fixing up Cursor plugin format (#46) * docs: Improve local installation instructions (#47) * docs: improve local installation instructions * docs: address reviewer feedback on local installation instructions * adjusting cursor plugin to reflect skills and mcp setup path correctly (#52) * adjusting cursor plugin to reflect skills and mcp setup path correctly * Apply suggestion from @gemini-code-assist[bot] Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> * update mcp path to ensure it can find it * correct the path --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> * Add new skills logo (#51) * re-organize claude plugins (#54) * Moving battlehardened AI studio prompt into skills (#53) * Moving battlehardened AI studio prompt into skills * PR fixes * Address PR feedback: add isRecent, isAdmin, fix regex escaping and numbering * PR fixes * PR fixes * Merge `firebase-local-env-setup` skills into `firebase-basics` (#56) * Change all `firebase` CLI reference to use `npx firebase-tools` (#61) --------- Co-authored-by: Charlotte Liang <chliang@google.com> Co-authored-by: Muhammad Talha <muhammadtalhar@google.com> Co-authored-by: Muhammad Talha <126821605+mtr002@users.noreply.github.com> Co-authored-by: christhompsongoogle <106194718+christhompsongoogle@users.noreply.github.com> Co-authored-by: Samuel Bushi <66321939+ssbushi@users.noreply.github.com> Co-authored-by: chkuang-g <31869252+chkuang-g@users.noreply.github.com> Co-authored-by: Morgan Chen <morganchen12@gmail.com> Co-authored-by: Google Open Source Bot <firebase-oss-bot@google.com> Co-authored-by: cmoiccool <cmoiccool@users.noreply.github.com> Co-authored-by: Sichen Liu <lscmirror@gmail.com> Co-authored-by: Sichen Liu <sichenliu@google.com> Co-authored-by: Rosário P. Fernandes <rosariofernandes51@gmail.com> Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Co-authored-by: Peter Friese <peter@peterfriese.de>
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request adds a new skill for Firebase Remote Config, including setup guides for Android and iOS. The review feedback suggests correcting a grammatical error in the main skill file, updating deprecated Firebase KTX dependencies to the recommended artifacts in the Android guide, and fixing a title typo in the iOS guide that incorrectly referenced Crashlytics.
Add Remote config skills
Comment on lines
+65
to
+77
| ```xml | ||
| <?xml version="1.0" encoding="utf-8"?> | ||
| <defaultsMap> | ||
| <entry> | ||
| <key>welcome_message</key> | ||
| <value>Welcome to the app!</value> | ||
| </entry> | ||
| <entry> | ||
| <key>is_feature_enabled</key> | ||
| <value>false</value> | ||
| </entry> | ||
| </defaultsMap> | ||
| ``` |
There was a problem hiding this comment.
Let's add a comment inside the code block or prefix this by saying that this is an example XML defaults file.
Comment on lines
+33
to
+34
| * **Android**: [Get started with Firebase Remote Config on Android](https://firebase.google.com/docs/remote-config/get-started?platform=android) | ||
| * **iOS**: [Get started with Firebase Remote Config on Apple Platforms](https://firebase.google.com/docs/remote-config/get-started?platform=ios) No newline at end of file |
There was a problem hiding this comment.
This feels repetitive with the previous section on SDK setup?
Comment on lines
+26
to
+29
| * **Set In-App Defaults**: Define baseline values to ensure the app functions offline or before the first fetch. | ||
| * **Fetch and Activate**: Retrieve values from the Firebase backend and apply them to the local UI/Logic. | ||
| * **Real-time Updates**: Listen for server-side configuration changes to update the app instantly without a refresh. | ||
| * **Template Management**: Use the Firebase CLI to version-control, get, and deploy your config JSON files. |
Member
There was a problem hiding this comment.
+1 - I'd also suggest stating specific CLI commands that the agent should use for differrnt CUJS
joehan
reviewed
May 4, 2026
Member
joehan
left a comment
joehan
left a comment
There was a problem hiding this comment.
Seems mostly good, but maybe be more specific about CLI commands to use
Comment on lines
+26
to
+29
| * **Set In-App Defaults**: Define baseline values to ensure the app functions offline or before the first fetch. | ||
| * **Fetch and Activate**: Retrieve values from the Firebase backend and apply them to the local UI/Logic. | ||
| * **Real-time Updates**: Listen for server-side configuration changes to update the app instantly without a refresh. | ||
| * **Template Management**: Use the Firebase CLI to version-control, get, and deploy your config JSON files. |
Member
There was a problem hiding this comment.
+1 - I'd also suggest stating specific CLI commands that the agent should use for differrnt CUJS
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This Change introduces the
firebase-remote-config-basicsskill, providing a comprehensive guide for setting up and using Firebase Remote Config on Android and IOS platformsKey Changes:
SKILL.mdwhich defines the skill's purpose, prerequisites, and high-level SDK usage patterns.android_setup.mdcovering Gradle dependencies using the Firebase BoM, google-services.json requirements, and Kotlin code for SDK initialization.ios_setup.mddetailing bundle ID discovery, Swift Package Manager integration, and Swift code for managing remote configurations.