Skip to content

fix(run): provider crash on v0.7.0 — hotfix v0.7.1#598

Merged
kokevidaurre merged 1 commit intodevelopfrom
hotfix/v0.7.1
Mar 11, 2026
Merged

fix(run): provider crash on v0.7.0 — hotfix v0.7.1#598
kokevidaurre merged 1 commit intodevelopfrom
hotfix/v0.7.1

Conversation

@kokevidaurre
Copy link
Contributor

Summary

  • Fixes provider crash when non-Anthropic providers are used in squads run
  • Rebased on develop to include all infra cleanup and phase execution changes

Originally #589, rebased to resolve conflicts with develop.

Test plan

  • Build passes
  • All 1731 tests pass
  • No conflict markers remain

🤖 Generated with Claude Code

- Fix ReferenceError: provider is not defined in executeWithClaude
- Add missing outcomes.ts, insights.ts, stats.ts from develop
- Every squads run crashed on v0.7.0, D1 retention effectively 0%

Co-Authored-By: Claude <noreply@anthropic.com>
@kokevidaurre kokevidaurre enabled auto-merge (squash) March 11, 2026 20:36
@github-actions github-actions bot added the core label Mar 11, 2026
@gemini-code-assist
Copy link

Summary of Changes

Hello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request delivers a crucial hotfix, version 0.7.1, to resolve a provider-related crash encountered when executing squads run with non-Anthropic providers. Additionally, the branch has been rebased to align with the latest develop branch, integrating recent infrastructure improvements and ensuring a stable foundation for future development.

Highlights

  • Provider Crash Fix: Addressed a critical crash that occurred when using non-Anthropic providers within the squads run command, ensuring broader compatibility.
  • Version Hotfix: Released a hotfix version 0.7.1 to quickly resolve the identified provider crash.
  • Rebase and Infrastructure Updates: Rebased the branch on develop to incorporate recent infrastructure cleanup and phase execution changes, resolving conflicts from a previous PR (fix(run): provider crash on v0.7.0 — hotfix v0.7.1 #589).
Changelog
  • package-lock.json
    • Updated the project version to 0.7.1.
  • package.json
    • Updated the project version to 0.7.1.
  • src/lib/outcomes.ts
    • Renamed a local variable _unmergedPRs to unmergedPRs.
Activity
  • The pull request was rebased on develop to resolve conflicts from an earlier PR (fix(run): provider crash on v0.7.0 — hotfix v0.7.1 #589).
  • A comprehensive test plan was executed, confirming that the build passes, all 1731 tests pass, and no conflict markers remain.
  • The changes were generated using Claude Code.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@kokevidaurre kokevidaurre merged commit 7b580e2 into develop Mar 11, 2026
11 checks passed
Copy link

@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 primarily bumps the version for a hotfix release. I've noticed a minor code quality issue in src/lib/outcomes.ts where a variable was renamed but remains unused. I've left a comment with a suggestion to remove it for better code clarity.

const totalPRs = records.reduce((sum, r) => sum + r.artifacts.prsCreated.length, 0);
const mergedPRs = records.reduce((sum, r) => sum + r.outcomes.prsMerged, 0);
const _unmergedPRs = records.reduce((sum, r) => sum + r.outcomes.prsClosedUnmerged, 0);
const unmergedPRs = records.reduce((sum, r) => sum + r.outcomes.prsClosedUnmerged, 0);

Choose a reason for hiding this comment

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

medium

The unmergedPRs variable is declared but its value is never used within the computeScorecard function. This is considered dead code and should be removed to improve maintainability.

@kokevidaurre kokevidaurre deleted the hotfix/v0.7.1 branch March 11, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant