CI: drop daily-CI extra cases; fix dash-incompatible test step

[zhangqi-chen]

Summary

• Delete .github/daily_ci_cases.tsv, which referenced model files (qwen3_14b_decode.py, qwen3_14b_decode_full.py) renamed/split in Refactor Qwen3 decode and split final head #319 and was therefore stale
• Remove the TSV-reading "extras" logic from both jobs in daily_ci.yml; daily CI now watches only the default no-arg invocation of every model file
• Fix the a2a3 job failing on push-to-main: pin shell: bash on the sim and a2a3 run steps. The container's default shell is dash, which cannot parse the failed=() / ${#failed[@]} bash-array logic added in ci: run PR-changed example/model files on sim and a2a3 #334. The bug was masked on ci: run PR-changed example/model files on sim and a2a3 #334's own PR because no example/model .py files changed, so the device jobs were skipped.

Related Issues

N/A

[coderabbitai]

📝 Walkthrough

Walkthrough

The daily CI workflow simplifies model test execution by removing parameterized case handling. Both model-tests-sim and model-tests-a2a3 jobs now run only the default non-draft models under models once each, without per-case arguments, and the logic that read .github/daily_ci_cases.tsv for extra cases is removed.

Changes

Daily CI Model Test Simplification

Layer / File(s)	Summary
Simplify model-tests-sim and model-tests-a2a3 jobs .github/workflows/daily_ci.yml	Both model test jobs are adjusted to execute only the default set of non-draft models under models exactly once with no additional per-case arguments, and the follow-up parameterized case logic is removed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• hw-native-sys/pypto-lib#145: Both PRs modify the GitHub Actions test runner logic in .github/workflows/daily_ci.yml to control which models/*.py files are executed (with *draft* excluded), so this PR's cleanup of parameterized case behavior is directly related.

Poem

🐰 The CI learns to hop more straight,
No winding paths through draft or state,
Each model runs its course just once—
The workflow's simpler by bunches!
One hop per script, no extra dance! 🎯

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Title check	✅ Passed	The title accurately summarizes the main changes: dropping daily-CI extra cases and fixing a shell compatibility issue in the test step.
Description check	✅ Passed	The description is directly related to the changeset, detailing the removal of stale TSV file, deletion of extras logic, and shell compatibility fix.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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 and usage tips.}

[gemini-code-assist]

Code Review

This pull request deletes the configuration file for daily CI parametric cases. A review comment points out that the PR description mentions changes to the CI workflow file which are not present in the diff, warning that deleting this file without updating the workflow will likely cause CI failures.

[gemini-code-assist]

The pull request description states that logic was removed from daily_ci.yml, but that file is not included in the changes. Deleting this TSV file without updating the CI workflow that references it will likely cause CI failures.