test: add edge-case coverage for windowing and rules

[stacknil]

Summary

This PR implements issue #4 by adding focused edge-case tests for the telemetry MVP.

Closes #4.

What changed

• added tests/test_edge_cases.py

Added coverage

• empty input produces no windows, features, or alerts
• single-event input creates one window with one counted event
• duplicate timestamps are counted in the same window
• window boundaries follow left-closed / right-open behavior
• small window and step sizes keep overlapping counts explicit
• out-of-order timestamps are normalized before windowing
• threshold equality is explicit for strict vs inclusive rules

Verification

Commands run:

pytest
pytest tests/test_edge_cases.py

Observed results:

• pytest passed: 13 passed
• README unchanged and still accurate
• no runtime behavior changed

Scope

• tests only
• no production features
• no runtime code changes
• no unrelated refactors

Review focus

Please focus on:

• whether the boundary assumptions are explicit and reasonable
• whether any important edge case is still missing
• whether this PR stayed within issue v0.2: add edge-case tests for timestamp parsing, window boundaries, and empty input #4

[stacknil]

@codex review
Focus on:

• missing edge-case tests
• correctness of window-boundary assumptions
• whether the PR stays strictly within issue v0.2: add edge-case tests for timestamp parsing, window boundaries, and empty input #4

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 031281e407

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[stacknil]

@codex review
Focus on:

• whether the updated assertion now protects against duplicate-alert regressions
• whether any other edge-case test still loses multiplicity information

[stacknil]

@codex review
Focus on:

• whether the invalid-timestamp test now covers the normalization failure path
• whether any remaining time-parsing edge case is still missing

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 561c7fdf4c

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[stacknil]

@codex review
Focus on:

• mixed timezone-offset normalization coverage
• whether any remaining timestamp-parsing edge case is still missing
• whether PR test: add edge-case coverage for windowing and rules #5 still stays strictly within issue v0.2: add edge-case tests for timestamp parsing, window boundaries, and empty input #4

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5020001983

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[stacknil]

@codex review
Focus on:

• mixed naive-plus-aware timestamp parsing coverage
• whether any remaining timestamp-parsing edge case is still missing
• whether PR test: add edge-case coverage for windowing and rules #5 still stays strictly within issue v0.2: add edge-case tests for timestamp parsing, window boundaries, and empty input #4

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0fc7a6fe77

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".