chore(ci): track ai-workflows @v3 floating tag#36296
Conversation
🤖 Bedrock Review —
|
Pin the orchestrator to the floating major tag @V3 (currently v3.1.6) instead of an exact patch tag, so consumers auto-pick up v3.x releases. @V3 is a tag (refs/tags/v3), so it satisfies the Bedrock OIDC trust that requires job_workflow_ref to match refs/tags/* (a branch/SHA would be denied at configure-aws-credentials). v3.1.6 adds the configurable max_output_tokens input and raises the bedrock-generic default 2048 -> 32000 so reasoning-model (DeepSeek R1) reviews stop truncating. Validated e2e against the real model (steve-quarterly-planning #109). Refs #36285 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
81ddbe4 to
db56963
Compare
🤖 Bedrock Review —
|
sfreudenthaler
left a comment
There was a problem hiding this comment.
like the new floating tag. will also keep minting specific version tags / releases in ai-workflows repo.
good to go
|
Queued — the merge queue status continues in this comment ↓. |
Merge Queue Status
This pull request spent 18 seconds in the queue, including 3 seconds running CI. Required conditions to merge
ReasonPull request #36296 has been dequeued GitHub refused to merge the pull request. Pull Request is in the merge queue. This is usually enforced by a branch protection or ruleset rule. HintYou should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it. Tick the box to put this pull request back in the merge queue (same as
|
This PR fixes #36285
What
Pin the
ai-workflowsorchestrator to the floating major tag@v3(currently v3.1.6) instead of an exact patch tag, across all 4 references in 3 workflows. Consumers now auto-pick upv3.xreleases without a per-release bump PR.Why
@v3and not@main/SHAThe Bedrock OIDC trust (
GitHubActions-BedrockCodeReview, IaC #7833) requiresjob_workflow_refto matchrefs/tags/*. A branch or commit SHA is denied atconfigure-aws-credentials(AssumeRoleWithWebIdentityfails).@v3is a tag (refs/tags/v3), so it satisfies the trust while still floating to the latest v3.x.What v3.1.6 brings
Configurable
max_output_tokensorchestrator input and the bedrock-generic default raised2048->32000. Core routesus.deepseek.r1-v1:0(a reasoning model) through that path, where reasoning tokens count against the same budget — the old 2048 cap truncated reviews mid-thought. Completes the budget side of the R1 reviewer upgrade in #36285.Files
.github/workflows/ai_claude-rollback-safety.yml.github/workflows/ai_claude-orchestrator.yml(2 refs).github/workflows/ai_claude-backend-reviewer.ymlValidation
v3.1.6 was validated end-to-end against the real DeepSeek R1 model in
dotCMS/steve-quarterly-planning#109 (now closed): correct routing to the bedrock-generic path, sticky comment posted, all planted bugs caught,maxTokens=32000accepted by R1 with no truncation.Release: https://github.com/dotCMS/ai-workflows/releases/tag/v3.1.6
🤖 Generated with Claude Code