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:

  1. Check the README and existing issues first.
  2. If your question hasn't been addressed, open a new issue using the Question label.

๐Ÿ› 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:

  1. Search existing issues to avoid duplicates.
  2. If it's new, open a new issue using the Feature Request template.
  3. Describe your idea, use cases, and possible alternatives.

๐Ÿ”ง How to Contribute Code

Step-by-Step Process

  1. Fork the repository.

  2. Clone your fork:

    git clone https://github.com/your-username/opendataloader-pdf.git
    cd opendataloader-pdf
    
  3. Create a feature branch:

    git checkout -b my-feature
    
  4. Build the project:

    Prerequisites: Java 11+, Maven, Python 3.10+, uv, Node.js 24 (current active LTS), pnpm via corepack enable pnpm

    Node 24 and pnpm 11.21.0 are what CI builds against. Enabling Corepack once picks the pnpm version up from the packageManager field, 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 sync
    

    Important: If you modified any CLI options in Java, you must run npm run sync before committing. This regenerates options.json and all Python/Node.js bindings. Forgetting this silently breaks the wrappers.

  5. Make your changes and commit them.

  6. Push your branch:

    git push origin my-feature
    
  7. Open a Pull Request (PR) against the main branch.

  8. 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.sh before 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.