[CaliLuke]

The Selvedge error-policy tests asserted exact invocation counts while also imposing a one-second attempt deadline. If scheduling delayed the handler past that deadline, native timeout policy correctly admitted another attempt—even though the eventual handler error was non-retryable.

Remove the unrelated deadlines from retry-policy and retry-option tests, retain their exact invocation counts, and require the expected permanent error. Separate timeout tests now verify native defaults and portable overrides using cancellation, typed timeout errors, and recorded options. The overall test context still bounds execution.

Validation: a controlled delayed-handler probe reproduced two invocations with the old deadline and one without it; an opt-in TLA model records the counterexample and corrected assumption. The complete Selvedge package passed 30 race-enabled runs with the failing CI shuffle seed, followed by a final affected-package race run after a mechanical allocation correction. No production runtime or dependency behavior changed, and no retry/skip was added to the tests.

Closes #1053.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.}