Skip to content

fix: address code quality issues and repo hygiene gaps, bump 0.7.1#12

Merged
1 commit merged intomainfrom
fix/code-quality-and-repo-hygiene
Mar 28, 2026
Merged

fix: address code quality issues and repo hygiene gaps, bump 0.7.1#12
1 commit merged intomainfrom
fix/code-quality-and-repo-hygiene

Conversation

@noexecstack
Copy link
Copy Markdown
Owner

Summary

  • Use LOG.debug instead of logging.debug in _detect_hubble_cmd (bypassed the module logger)
  • Replace assert proc.stdout/stderr is not None with early-return guards in reader threads (assertions vanish under python -O)
  • Guard cursor_flow_idx when ordered_keys is empty in watch select mode
  • Close capture_fh on early exit paths (RuntimeError, FileNotFoundError) to prevent file handle leak
  • Add missing py.typed marker file (referenced in CHANGELOG since v0.4.0 but never created)
  • Add types-PyYAML to dev dependencies so local mypy works out of the box
  • Add .mypy_cache/, .pytest_cache/, .ruff_cache/ to .gitignore
  • Bump version to 0.7.1, update changelog

Test plan

  • ruff check passes
  • ruff format passes
  • mypy passes
  • All 81 tests pass

- Use LOG.debug instead of logging.debug in _detect_hubble_cmd
- Replace assert with early-return guards in reader threads
- Guard cursor_flow_idx when ordered_keys is empty in watch mode
- Close capture file handle on early exit paths to prevent leak
- Add missing py.typed marker (referenced in CHANGELOG since v0.4.0)
- Add types-PyYAML to dev dependencies for local mypy
- Add .mypy_cache, .pytest_cache, .ruff_cache to .gitignore
- Bump version to 0.7.1, update changelog
@noexecstack noexecstack closed this pull request by merging all changes into main in d0a8941 Mar 28, 2026
@noexecstack noexecstack deleted the fix/code-quality-and-repo-hygiene branch March 28, 2026 18:00
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