ci: extract docs-only classifier tests and retry stale-diagnostic e2e flakes - #259
Conversation
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Heads-up from the #105 phase-2 lane: Reproduced on a pristine |
Summary
scripts/classify-docs-only.mjswith fixture tests (renames, nested SKILL.md, truncated listings). Prevents a classifier bug from skipping Verify.{ retry: 2 }to theexamples-real.e2estale-diagnostic/repair journeys — the same interim used for the flake: track remaining dev-watcher and MCP App gate failures #122/Flake: overview.e2e owned-reload generation budget exceeded under hosted-runner load #200 hosted-load flake family.Closes #241.
Test plan
actionlint .github/workflows/ci.ymlpnpm exec rstest --config rstest.unit.config.ts packages/agent-bundle/tests/classify-docs-only.test.ts(4/4)rslinton the touched TS/JS files