-
Notifications
You must be signed in to change notification settings - Fork 139
Description
π§ Advanced Contributors
This issue is intended for contributors who are already very familiar with the
Hiero Python SDK codebase and its architectural patterns.
You should feel comfortable:
- navigating multiple modules across
src/ - understanding and modifying core SDK abstractions
- reasoning about API design and backwards compatibility
- updating or extending tests, examples, and documentation as needed
- making changes that may affect public-facing behavior
New developers should start with
Good First Issues or Intermediate Issues first.
π Problem Description
Coderabbit can have custom settings and prompts, telling it exactly how to review certain paths in the code.
We can trial this feature out with /docs.
π‘ Proposed / Expected Solution
We can create an /docs configuration for code rabbit that makes code rabbit commiter or maintainer-level preliminary reviewer, for /docs.
Please note we have:
/docs (general)
/docs/sdk_users (user facing docs)
/docs/sdk_developers (developer facing docs, like tutorials)
Organisation is not yet super great, and the docs are still a WIP
π§ Implementation & Design Notes
To do this, extend:
.coderabbit.yaml
Read examples and config to research what we might want (you'll need to be an original thinker here with full creative ability)
https://docs.coderabbit.ai/reference/configuration
https://github.com/coderabbitai/awesome-coderabbit/blob/main/configs/python/coderabbit-python-portal.yaml
https://github.com/coderabbitai/awesome-coderabbit/tree/main/configs/python
Here are SOME ideas I have, but this is just a rough guide and only a starting point:
path_instructions:
-
path: "docs/**"
-
represent general documentation for both sdk users and sdk developers
-
Focus on correctness, clarity, and usefulness, not detail.
-
Consider whether missing explanations in a pull request should be additions to that PR or a new issue raised
-
Avoid suggesting large-scale restructures unless a problem blocks understanding.
-
Check URls if possible
- path: "docs/sdk_users/**"
instructions: >-
-These documents are user facing guidelines intended for
developers consuming the SDK.
- path: "docs/sdk_users/**"
-
Assume readers have some development experience but may be totally new new to this SDK.
-
Do not assume any knowledge of how to use the sdk
-
Do not assume any knowledge of how the sdk was constructed, nor desire
-
The user just wants to learn how to use the sdk
-
Explanations must be concrete, actionable, and unambiguous.
-
Steps should be explicit and ordered.
-
Ensure snippets are correct, complete, and up to date.
- path: "docs/sdk_developers/**"
instructions: >-
- path: "docs/sdk_developers/**"
-
These documents are developer-facing and may include tutorials,
architectural explanations, or contribution guidance. -
Assume some readers may be total beginners yet some experts
-
Assume intent to extend, debug, or contribute to the SDK.
β Acceptance Criteria
To merge this issue, the pull request must:
- Fully address the problem and design goals described above
- Maintain backwards compatibility unless explicitly approved otherwise
- Follow existing architectural and coding conventions
- Include comprehensive tests covering new and existing behavior
- Update relevant examples and documentation
- Pass all CI checks
- Include a valid changelog entry
- Use DCO and GPG-signed commits
π Additional Context, Links, or Prior Art
Optional.