Skip to content

Rename lifecycle hooks to match functions vocab#1920

Merged
wandamora merged 2 commits into
masterfrom
morawand-rename-lifecycle-events
Jul 7, 2026
Merged

Rename lifecycle hooks to match functions vocab#1920
wandamora merged 2 commits into
masterfrom
morawand-rename-lifecycle-events

Conversation

@wandamora

Copy link
Copy Markdown
Contributor

Description

This renames afterInstall to afterFirstDeploy and afterUpdate to afterRedeploy to better mirror functions vocabulary and move away from Extensions terms.

Code sample

export const runInitialSetup = onTaskDispatched(async (request) => {
  await performDatabaseSeeding(request.data);
});

// The CLI automatically discovers this call and queues the task after first-time deploy
afterFirstDeploy({
  task: {
    function: "runInitialSetup",
    body: { data: "default_catalog" }
  }
});

This renames afterInstall to afterFirstDeploy and afterUpdate to afterRedeploy
to better mirror functions vocabulary and move away from Extensions
terms.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request renames the lifecycle hooks afterInstall and afterUpdate to afterFirstDeploy and afterRedeploy across the codebase and test suites to align with Cloud Functions terminology. The feedback suggests updating the JSDoc comments for both functions to use 'deployed' and 'redeployed' instead of 'installed' and 'updated' to maintain consistency with the new naming convention.

Comment thread src/lifecycle/index.ts Outdated
Comment thread src/lifecycle/index.ts
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@wandamora wandamora merged commit 9d9230a into master Jul 7, 2026
31 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.

3 participants