Skip to content

feat(github-actions): add daily Gemini model update workflow and local action#3706

Open
josephperrott wants to merge 2 commits into
angular:mainfrom
josephperrott:update-models-action
Open

feat(github-actions): add daily Gemini model update workflow and local action#3706
josephperrott wants to merge 2 commits into
angular:mainfrom
josephperrott:update-models-action

Conversation

@josephperrott
Copy link
Copy Markdown
Member

Automatically updates Gemini model versions based on the official deprecation schedules. Rebuilds any dependent checked-in action bundles and verifies all tests pass daily.

@josephperrott josephperrott added the action: merge The PR is ready for merge by the caretaker label May 27, 2026
@josephperrott josephperrott requested a review from alan-agius4 May 27, 2026 15:20
@angular-robot angular-robot Bot added the detected: feature PR contains a feature commit label May 27, 2026
@josephperrott josephperrott force-pushed the update-models-action branch from ed2b900 to 77868c5 Compare May 27, 2026 15:21
Copy link
Copy Markdown

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

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 introduces a new local GitHub Action, update-models, designed to parse the Gemini deprecations page and automatically update deprecated model references across .ts, .yml, and .yaml files in the repository. Feedback on the implementation highlights a few critical issues: the model replacement logic is fragile and needs stricter regex validation to avoid corrupting files with descriptive text, and fast-glob requires the dot: true option to ensure .github directories are scanned. Additionally, it is recommended to make the parseDeprecations function synchronous since it contains no asynchronous operations.

Comment thread .github/local-actions/update-models/lib/main.ts
Comment thread .github/local-actions/update-models/lib/main.ts
Comment thread .github/local-actions/update-models/lib/main.ts Outdated
Comment thread .github/local-actions/update-models/lib/main.ts Outdated
@josephperrott josephperrott force-pushed the update-models-action branch 3 times, most recently from e4f328c to 498e00b Compare May 27, 2026 15:27
@josephperrott josephperrott force-pushed the update-models-action branch from 498e00b to de3a723 Compare May 27, 2026 15:28
@josephperrott josephperrott force-pushed the update-models-action branch from de3a723 to 2158982 Compare May 27, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker detected: feature PR contains a feature commit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant