[curfew-marathon]

Summary

• PR Add type hints, pin Docker/Python version, raise coverage floor #67 accidentally dropped the parentheses around except (KeyboardInterrupt, SystemExit):, leaving except KeyboardInterrupt, SystemExit:.
• That line only parses because PEP 758 (Python 3.14+) allows unparenthesized exception tuples. It works on this project's pinned Python 3.15, but is a SyntaxError on any Python 3 version before 3.14, and reads identically to invalid Python 2 syntax.
• Restores the explicit tuple form so the behavior is unchanged but the code is portable and unambiguous.

Test plan

• Confirmed the broken line still parses and behaves correctly under local Python 3.14.7 (relying on PEP 758), explaining why CI didn't catch it.
• Did not run the full pytest --cov suite or ruff/mypy locally (tools not available in this environment); this is a one-line revert to the prior, already-tested syntax.

Summary by CodeRabbit

• No user-facing changes
  • This update does not change application features or behavior. It includes a small internal code adjustment to how shutdown-related interruptions are handled.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

The ImportrrScheduler.start handler now catches KeyboardInterrupt and SystemExit together with a parenthesized exception tuple.

Changes: Scheduler exception handling

Layer / File(s)	Summary
Group shutdown exceptions src/importrr/launch.py	The start handler groups KeyboardInterrupt and SystemExit in a valid exception tuple. Both use the existing shutdown log path.

Priority: Not assessed

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

Change: Bug fix

Possibly related PRs

• curfew-marathon/importrr#67: Changed the exception handler syntax that this PR updates.

Merge Risk: Moderate · up to 8d3c0

The PR preserves older-Python compatibility but currently cannot pass the required Ruff formatting check; configure Ruff or add a narrow exemption before merging.

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

[curfew-marathon]

Closing without merging: ruff format --check actively enforces the unparenthesized except (KeyboardInterrupt, SystemExit) form as canonical for target-version = py315 (PEP 758), so this change fights the project's own formatter. Confirmed CI (ruff format --check) rejects the parenthesized form. Main's existing syntax is correct and intentional given the pinned Python 3.15 target; no fix needed.