diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 2fcce51..8e7209d 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1,19 +1,82 @@ -# Issues - Rules +# Issues +## Warnung +Before opening Issue, if it is more question than problem or bug - ask about that [in our chat](https://t.me/telemtrs) + ## What it is not - NOT Question and Answer - NOT Helpdesk -# Pull Requests - Rules +***Each of your Issues triggers attempts to reproduce problems and analyze them, which are done manually by people*** + +--- + +# Pull Requests + ## General - ONLY signed and verified commits - ONLY from your name -- DO NOT commit with `codex` or `claude` as author/commiter +- DO NOT commit with `codex`, `claude`, or other AI tools as author/committer - PREFER `flow` branch for development, not `main` -## AI -We are not against modern tools, like AI, where you act as a principal or architect, but we consider it important: +--- -- you really understand what you're doing -- you understand the relationships and dependencies of the components being modified -- you understand the architecture of Telegram MTProto, MTProxy, Middle-End KDF at least generically -- you DO NOT commit for the sake of commits, but to help the community, core-developers and ordinary users +## Definition of Ready (MANDATORY) + +A Pull Request WILL be ignored or closed if: + +- it does NOT build +- it does NOT pass tests +- it does NOT follow formatting rules +- it contains unrelated or excessive changes +- the author cannot clearly explain the change + +--- + +## Blessed Principles +- PR must build +- PR must pass tests +- PR must be understood by author + +--- + +## AI Usage Policy + +AI tools (Claude, ChatGPT, Codex, DeepSeek, etc.) are allowed as **assistants**, NOT as decision-makers. + +By submitting a PR, you confirm that: + +- you fully understand the code you submit +- you verified correctness manually +- you reviewed architecture and dependencies +- you take full responsibility for the change + +AI-generated code is treated as **draft** and must be validated like any other external contribution. + +PRs that look like unverified AI dumps WILL be closed + +--- + +## Maintainer Policy + +Maintainers reserve the right to: + +- close PRs that do not meet basic quality requirements +- request explanations before review +- ignore low-effort contributions + +Respect the reviewers time + +--- + +## Enforcement + +Pull Requests that violate project standards may be closed without review. + +This includes (but is not limited to): + +- non-building code +- failing tests +- unverified or low-effort changes +- inability to explain the change + +These actions follow the Code of Conduct and are intended to preserve signal, quality, and Telemt's integrity \ No newline at end of file