Contributing to This Project
August 13, 2026 ยท View on GitHub
Thank you for your interest in contributing!
We welcome contributions from everyone. This document outlines the guidelines for how to contribute effectively and
respectfully.
๐ Types of Contributions We Welcome
We appreciate various kinds of contributions, including but not limited to:
- ๐ ๏ธ Code contributions (bug fixes, performance improvements, new features)
- ๐ Bug reports
- ๐ก Feature suggestions
- โ Questions and discussions
- ๐ Improving documentation
โ How to Ask Questions
If you have questions:
- Check the README and existing issues first.
- If your question hasn't been addressed, open a new issue using the
Questionlabel.
๐ How to Report Bugs
When reporting a bug, please include the following:
- A clear and descriptive title
- Steps to reproduce the issue
- Expected vs actual behavior
- Environment info (OS, version, etc.)
- Logs or screenshots if available
Use the Bug Report issue template when creating the issue.
๐ก How to Suggest a Feature
To suggest a new feature:
- Search existing issues to avoid duplicates.
- If it's new, open a new issue using the Feature Request template.
- Describe your idea, use cases, and possible alternatives.
๐ง How to Contribute Code
Step-by-Step Process
-
Fork the repository.
-
Clone your fork:
git clone https://github.com/your-username/opendataloader-pdf.git cd opendataloader-pdf -
Create a feature branch:
git checkout -b my-feature -
Build the project:
Prerequisites: Java 11+, Maven, Python 3.10+, uv, Node.js 24 (current active LTS), pnpm via
corepack enable pnpmNode 24 and pnpm 11.21.0 are what CI builds against. Enabling Corepack once picks the pnpm version up from the
packageManagerfield, so there is no global install and no version to remember. Node must be >=22.13 โ pnpm 11 refuses to install on anything older. See the Development Workflow guide for OS-specific install instructions.# Build Java packages npm run build-java # If you changed CLI options in Java, sync bindings (regenerates options.json, Python/Node.js wrappers) npm run syncImportant: If you modified any CLI options in Java, you must run
npm run syncbefore committing. This regeneratesoptions.jsonand all Python/Node.js bindings. Forgetting this silently breaks the wrappers. -
Make your changes and commit them.
-
Push your branch:
git push origin my-feature -
Open a Pull Request (PR) against the
mainbranch. -
Respond to review comments and update your PR as needed.
๐งน Coding Style & Guidelines
- Follow existing code conventions.
- Run linters/formatters before committing.
- Write unit tests for any new or changed logic.
- Run
./scripts/bench.shbefore submitting a PR โ CI will fail if benchmark scores drop below thresholds. - Keep your changes minimal and focused.
โ Commit Message Guidelines
Use the following format:
<type> <short summary>
Common types:
- Add: New feature
- Fix: Bug fix
- Update: Code update
๐ค Agent Skill Maintenance
This repo ships an AI-agent skill under skills/odl-pdf/. It is a version-independent procedure โ it reads the installed CLI's own --help at runtime and bakes no option name, value, or default into its prose, so renaming a flag or flipping a default does not require touching the skill.
What still needs manual review when you change the CLI:
- Silent-failure behavior (e.g. an enrichment that is skipped unless the whole document is routed to the backend; structured output that does not stream to stdout). If you add, remove, or change such behavior, update the hazard principles and the release-review checklist in
skills/odl-pdf-maintenance/MAINTAINING.md. - The version-coupling lint (
skills/odl-pdf-maintenance/sync-skill-refs.py, run in CI) fails the build if a version number or an option name is ever baked into the skill. If it fails, fix the skill text โ do not add to its allowlist.
The skills/odl-pdf-maintenance/ directory is developer-only and is not part of the installable skill.
๐ CLA / DCO Requirements
Depending on your contribution, we may ask you to sign:
- CLA โ Contributor License Agreement
- DCO โ Developer Certificate of Origin
To sign the DCO, add Signed-off-by to your commit message:
git commit -s -m "your message"
Make sure your Git config contains your real name and email.
Thank you again for helping us improve this project! ๐ If you have any questions, open an issue or join the discussion.