Skip to content

docs & dx: fix typo, improve pytest config, add security contact#1130

Open
Atman36 wants to merge 1 commit intoMoonshotAI:mainfrom
Atman36:codex/docs-dx-security-followups
Open

docs & dx: fix typo, improve pytest config, add security contact#1130
Atman36 wants to merge 1 commit intoMoonshotAI:mainfrom
Atman36:codex/docs-dx-security-followups

Conversation

@Atman36
Copy link

@Atman36 Atman36 commented Feb 13, 2026

What

This PR bundles several small, non-breaking improvements:

  1. CONTRIBUTING.md — Fix capitalization typo: Prek hooksprek hooks
  2. pytest.ini — Add testpaths and pythonpath for better test discovery
  3. SECURITY.md — Add email contact for sensitive security issues
  4. exception.py — Remove redundant pass statements and add __all__ declaration

Why

  • Consistency: Fix typo to match the tool's actual name
  • DX: Explicit pytest paths reduce command-line arguments needed
  • Security: Provide direct contact channel for sensitive reports
  • Code quality: Explicit exports improve IDE support and follow Python best practices

How to verify

# Check syntax
python3 -m py_compile src/kimi_cli/exception.py

# Verify __all__ exports
python3 -c "from kimi_cli.exception import __all__; print(__all__)"

Notes

  • Breaking change: No
  • Risk: Low
  • Files changed: 4
  • Total lines: ~30 (net reduction due to removed pass statements)

PR prepared with AI assistance (Claude Code)


Open with Devin

…anup exceptions

- Fix 'Prek' → 'prek' capitalization in CONTRIBUTING.md
- Add testpaths and pythonpath to pytest.ini for better test discovery
- Add security contact email to SECURITY.md
- Remove redundant pass statements from exception classes
- Add __all__ declaration to exception module for explicit exports

All changes are non-breaking improvements to documentation,
developer experience, and code quality.
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

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