mirror of
https://github.com/guillaumemeyer/watermarks-remover.git
synced 2026-08-22 13:11:57 +02:00
The c2patool docs URL (opensource.contentauthenticity.org/docs/c2patool/) returns 404. Update all four references in README and CONTRIBUTING to the current repository location. Fixes #2.
3.2 KiB
3.2 KiB
Contributing to watermarks-remover
Thanks for helping keep the skill accurate and the cleaners reliable. The
project is a small Python skill (skills/remove-ai-marks/) plus tests —
focused PRs land fastest.
Who can do what
| Action | Who |
|---|---|
| Open issues | Anyone |
| Suggest a release | Anyone (use the Release suggestion issue template) |
| Open pull requests | Anyone (fork the repo) |
| Approve and merge pull requests | Maintainer only (@guillaumemeyer) |
main is protected. A change needs a pull request, a passing CI check
(test), and an approving review from the code owner before merge. Only the
maintainer can give that approval. Direct pushes to main are blocked for
non-admins.
To suggest a release without a code change: open a Release suggestion issue.
Prerequisites
- Python 3.10+ (stdlib only for the skill scripts; optional rewrite backends use HTTP to local Ollama / OpenAI-compatible endpoints)
- From the repo root:
python3 -m pytest -qshould pass before you open a PR - Optional for manual file checks:
c2patool,exiftool(PDF)
Layout
| Path | Role |
|---|---|
skills/remove-ai-marks/SKILL.md |
Agent skill entry (workflow, ethics) |
skills/remove-ai-marks/scripts/ |
Layer A/B hooks + image/container cleaners |
skills/remove-ai-marks/references/ |
Vendors, mark classes, matrix, ethics |
tests/ |
Pytest suite and fixtures |
.github/workflows/ci.yml |
CI job test |
Layers (what to change where)
- Layer A (Unicode / format controls) — deterministic scripts under
scripts/(text_unicode.py,clean_text.py,inspect_text.py). Prefer tests with fixtures intests/fixtures/. - Layer B (statistical rewrite) — guidance in
SKILL.mdplus optionalrewrite_text.py(print-prompt default; ollama / openai-compatible). No bundled model. Keep ethics-aware. - Files (C2PA / EXIF / XMP / props) —
image_meta.py(PNG/JPEG),container_meta.py(SVG/PDF/DOCX/ODT/HTML/MD), unifiedinspect_file.py/clean_file.py. Preserve document body / pixels; strip provenance metadata only.
Checklist for a change
- Behaviour matches
SKILL.md/references/removal-matrix.mdwhen relevant - Unit tests updated or added under
tests/ python3 -m pytest -qpasses- Docs updated (README and/or skill references) if user-facing behaviour changes
- No drive-by refactors unrelated to the fix or feature
PR expectations
- Stay focused and match existing style (stdlib-first scripts, clear CLI flags)
- Do not commit secrets, private user files, or large binary fixtures unless needed and redacted
- Respect
references/ethics.md: this tool is for content the user owns
Questions? Open an issue describing the input type (text / image / document) and which layer failed or is missing.
Community
- Code of Conduct — expected behaviour in the project
- Security policy — how to report vulnerabilities privately
- Bug report and feature request templates