fix(ci): accept current direct-openai Strix fallback

[seonghobae]

The required Strix smoke test is trusted from default-branch source. The current workflow contract uses gpt-5.6-luna, while the trusted smoke accepted only gpt-5.4, causing a same-head required check failure on PR #1328 before scanning.

Accept both the existing gpt-5.4 and the current gpt-5.6-luna direct-OpenAI fallback spellings, preserving compatibility and fail-closed validation.

Local evidence: bash -n and the required smoke test pass. This prerequisite must merge before PR #1328 can prove its current model contract.

---

Summary by CodeRabbit

• 테스트
  • Strix 필수 워크플로 스모크 테스트의 검증 범위를 확장했습니다.
  • 모델 폴백 계약 검사가 기존 모델 변형과 함께 gpt-5.6-luna 변형도 허용하도록 개선되었습니다.
  • 여러 허용 문자열 중 하나라도 일치하면 성공하도록 검증 처리가 강화되었습니다.

[coderabbitai]

Warning

Review limit reached

Next included review available in 56 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6b796934-aec3-431a-bcd0-5fc06fae89cf

📥 Commits

Reviewing files that changed from the base of the PR and between 052d42e and 7e80b0a.

📒 Files selected for processing (1)

• scripts/ci/strix_required_workflow_smoke.sh

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: dd59ba5a-5947-4e7b-bc1d-387074b1f12f

📥 Commits

Reviewing files that changed from the base of the PR and between 8fd471a and 052d42e.

📒 Files selected for processing (1)

• scripts/ci/strix_required_workflow_smoke.sh

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

Strix 필수 워크플로 스모크 테스트에 여러 허용 문자열을 검사하는 헬퍼를 추가했습니다. 모델 폴백 계약 검사는 기존 gpt-5.4 변형과 새 gpt-5.6-luna 변형을 허용합니다.

Changes

Strix 폴백 계약 검사

Layer / File(s)	Summary
다중 허용 문자열 검사 및 모델 폴백 계약 확장 scripts/ci/strix_required_workflow_smoke.sh	assert_file_contains_any 헬퍼가 여러 문자열 중 하나가 일치하면 성공하도록 추가되었습니다. 모델 폴백 검사는 gpt-5.4 변형, gpt-5.6-luna 변형, 설명 메시지를 허용합니다.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to 052d4

This localized CI compatibility update accepts the current and prior direct-OpenAI fallback spellings while preserving fail-closed validation; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 1 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 Strix CI 스모크 테스트가 현재 direct-OpenAI fallback 형식을 허용하도록 수정한 핵심 변경을 정확히 요약합니다.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/strix-trusted-fallback-contract

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[seonghobae]

Addressed Devin review at exact head 7e80b0aa4884257aabf0078afd124b57d06cab23: removed the now-unused two-needle helper and made the variadic helper take its message as the final argument, matching sibling assertion call conventions. bash -n and bash scripts/ci/strix_required_workflow_smoke.sh pass locally.

[seonghobae]

This compatibility change targets the wrong owner boundary and would weaken the trusted smoke.

Protected main@8fd471a31399a914d9cb22a840f4a4c68e010ea6 requires gpt-5.4 / openai-direct/gpt-5.4; #1328 introduced gpt-5.6-luna in .github/workflows/strix.yml as an unrelated branch regression. Accepting both spellings in the default-branch smoke would hide that contradiction instead of repairing it.

The smallest causal repair is to restore #1328's Strix workflow and Strix-specific assertions to the protected-main gpt-5.4 contract. After that, this PR is unnecessary and should remain unmerged/superseded. Keep provider unavailability non-passing; do not broaden the allowed model contract to make a stale branch pass.