Skip to content

fix(validation): preserve pytest retry semantics across modes - #465

Merged
8Dionysus merged 12 commits into
mainfrom
codex/pytest-lastfailed-review-fixes-20260907
Sep 7, 2026
Merged

8Dionysus merged 12 commits into
mainfrom
codex/pytest-lastfailed-review-fixes-20260907

Conversation

@8Dionysus

Copy link
Copy Markdown
Owner

Summary

  • Record the final pytest partition baseline after last-failed filtering has run.
  • Keep stateful stepwise and non-executing pytest modes on native serial execution, including options from argv, PYTEST_ADDOPTS, and config addopts.
  • Derive retry hints from completed teardown and reported failures so session-policy failures do not cache passing selections.
  • Refresh the canonical repo-local KAG generated family and its budget receipt.

Verification

  • python -m pytest -q tests/test_validation_command_authority.py — 52 passed.
  • Explicit process scheduler over the same command-authority file — 52 passed in 4 shards.
  • Ruff, targeted mypy, and git diff --check passed.
  • Three paired whole-command samples over the unchanged 19-test product-validator selection; all six runs passed with exact 4-shard union. Samples are reported as measurements, not a speedup claim.
  • Pinned aoa-kag owner-family preparation and post-apply sentinel/full-parity/family-contract/compatibility-assembly gate passed.

The generated family is projection-only; no runtime, deployment, host, or release-authority behavior is changed.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3ff428adea

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread scripts/run_pytest_lane.py

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8ad72aacc4

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread scripts/run_pytest_lane.py Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4b0fbfc112

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread scripts/run_pytest_lane.py

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0d7e3f07bb

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread scripts/run_pytest_lane.py
@8Dionysus
8Dionysus merged commit ee60836 into main Sep 7, 2026
2 checks passed
@8Dionysus
8Dionysus deleted the codex/pytest-lastfailed-review-fixes-20260907 branch September 7, 2026 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant