Skill to convert extensions codebases into functions codebases#21
Skill to convert extensions codebases into functions codebases#21
Conversation
joehan
left a comment
joehan
left a comment
There was a problem hiding this comment.
Love the idea, this is a perfect use case for skills!
Right now, this seems aimed at just users who are finding an extension for the first time. Should we expand it to cover cases where I have an extension configured in my project (with a firebase.json entry, env files, etc), and want to convert it to a functions codebase so I can make changes?
| deploy as their own functions. The only problem is that the extensions namespace | ||
| doesn't express any of the IaC expected in extensions. This fixes taht. | ||
|
|
||
| ## Triggerrs |
There was a problem hiding this comment.
Remove this section - no need to put this in the skill body, because at this point the skill has already triggered.
There was a problem hiding this comment.
This was advice I've seen online. I think the point is that when indexing the skill it tells the agent when to use this like a tool call. If I'm right, this will help. If I'm wrong this will do no harm, so I'd like to keep it.
inlined
left a comment
inlined
left a comment
There was a problem hiding this comment.
Publishing pushback and will have an agent apply the rest.
| deploy as their own functions. The only problem is that the extensions namespace | ||
| doesn't express any of the IaC expected in extensions. This fixes taht. | ||
|
|
||
| ## Triggerrs |
There was a problem hiding this comment.
This was advice I've seen online. I think the point is that when indexing the skill it tells the agent when to use this like a tool call. If I'm right, this will help. If I'm wrong this will do no harm, so I'd like to keep it.
|
/gemini review |
| to add the functions codebase to `firebase.json` for the user so that it will be included | ||
| in subsequent deploys. If the user agrees, add the functions codebase to `firebase.json`. | ||
|
|
||
| ### Testing |
There was a problem hiding this comment.
WDYT about telling the agents to set maxInstances: 1 (or at least <=10) until correct behavior can be confirmed?
There was a problem hiding this comment.
I'm very nervous about this since I intend this to be something that promotes code review and then you run with it.
Co-authored-by: Jeff <3759507+jhuleatt@users.noreply.github.com>
3 participants
Will help people who want to migrate from extensions to functions to customize functionality.
This is just a first draft for your review. I'm tagging Jeff for skill design in general. Tagging Joe and Pavel for ideas on how well this covers the conversion requirements. I can add examples in the future too.
While executing it on Extensions I noticed some features that actually need to be implemented. New features to add to Functions before this skill is complete:
exprtag that can be used for string interpolation with functions so you can say things like "