Skip to content

[dev] [tofikwest] tofik/hotfix-cloud-autofix-temperature#3302

Merged
tofikwest merged 2 commits into
mainfrom
tofik/hotfix-cloud-autofix-temperature
Jun 29, 2026
Merged

[dev] [tofikwest] tofik/hotfix-cloud-autofix-temperature#3302
tofikwest merged 2 commits into
mainfrom
tofik/hotfix-cloud-autofix-temperature

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

This is an automated pull request to merge tofik/hotfix-cloud-autofix-temperature into dev.
It was created by the [Auto Pull Request] action.


Summary by cubic

Fixes cloud auto-fix failures by removing the unsupported temperature param from remediation model calls to claude-opus-4-8. Auto-fix now generates plans again for AWS, GCP, and Azure; empty-plan retries re-sample using model defaults.

  • Bug Fixes
    • Removed temperature from all MODEL generateObject calls (generate/refine/permissions/step-repair) to avoid 400s from claude-opus-4-8; fallback paths that still accept it keep temperature.
    • Changed empty-plan retry to re-sample once without a temperature bump.
    • Expanded regression tests to assert temperature is never sent and verified AWS/GCP/Azure plan generation paths.

Written for commit 1cf2269. Summary will update on new commits.

Review in cubic

… model calls

The remediation model (claude-opus-4-8) no longer accepts the `temperature`
parameter and returns a 400 ("temperature is deprecated for this model").
Every generateObject call in the AI remediation service passed `temperature`,
so all fix-plan generation threw, was caught, and silently fell back to manual
"Remediation Steps" — making auto-fix appear broken for all AWS/GCP/Azure
findings.

Remove `temperature` from all 9 MODEL (opus-4-8) calls (AWS + GCP + Azure
generate/refine/permission/step-repair). The empty-plan retry now re-samples
at the model default instead of bumping temperature. The Sonnet fallback
(generateManualSteps) keeps temperature — sonnet-4-6 and haiku-4-5 both still
accept it.

Updated specs to assert temperature is never sent to MODEL, plus regression
tests for the AWS/GCP/Azure generate paths. 26/26 tests pass.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AWJZw7nETZu7JHaEdGuGhu
@vercel

vercel Bot commented Jun 29, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
comp-framework-editor Ready Ready Preview, Comment Jun 29, 2026 7:46pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
app Skipped Skipped Jun 29, 2026 7:46pm
portal Skipped Skipped Jun 29, 2026 7:46pm

Request Review

@cubic-dev-ai cubic-dev-ai 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.

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@vercel vercel Bot temporarily deployed to Preview – portal June 29, 2026 19:44 Inactive
@vercel vercel Bot temporarily deployed to Preview – app June 29, 2026 19:44 Inactive
@tofikwest tofikwest merged commit 8f40b30 into main Jun 29, 2026
11 checks passed
@claudfuen

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.93.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants