[stefankoegl]

Result of reading the whole project for things that are broken, inconsistent, inefficient or no longer needed. This PR contains only the fixes that are safe; everything that needs a decision is listed in the session, not changed here.

Four commits, one per area.

Library (jsonpatch.py, stubs)

• Remove Python 2 leftovers: coding line, (object) base classes, two-argument super(), the __ne__ methods (Python 3 derives the same result from __eq__) and JsonPatch.__nonzero__.
• DiffBuilder._insert_test no longer deep-copies the values of generated test operations. from_diff has copied every operation of the patch since 2e37755, so these were copied twice.

Visible change: JsonPatch.__nonzero__ is gone. Nothing calls it on Python 3; bool(patch) is unaffected.

Command line (jsonpatch_cli.py)

• jsonpatch never closed PATCH, and closed ORIGINAL only with --in-place, while jsondiff closed both inputs. Both now share one helper, which leaves stdin open.
• Fix the tab, trailing whitespace and blank lines flake8 reported.

Tests

• python tests.py ran the doctests of jsonpatch twice and only the classes in a hand-maintained list (nothing was missing from it). It now uses unittest.main() with load_tests. The test count goes from 360 to 353: the 7 doctests that were run twice.
• tests.js is found independent of the working directory and read as UTF-8.
• ext_tests.py: remove the coverage report. It started after jsonpatch was imported, so it reported about 43% whatever was run, and was skipped when tests failed. Also remove the doctests, which tests.py runs.
• property_tests.py: no property is an expected failure any more, but the module still worked around them. Drop the "safe" documents without - keys and the test that only differed in using them, and let the state machine replace - members and test values that are equal in Python but not in JSON.
• requirements-dev.txt: drop wheel, which nothing uses.

Packaging and docs

• setup.py: close the files it reads; drop the package_data entry for requirements.txt, which matched nothing.
• MANIFEST.in: the sdist contained the .rst files and tests, but not what is needed to use them. Add doc/conf.py, doc/Makefile, doc/requirements.txt, requirements-dev.txt and makefile.
• README: fix the doubled slash in the badge links, link the tutorial on Read the Docs instead of "See source code for examples" (the relative link to doc/tutorial.rst was broken on PyPI), say how to run the tests, drop the note on expected failures.
• Use readthedocs.io and datatracker.ietf.org instead of the hosts that redirect to them.
• Docs: .. mod-jsonpatch: was a comment, not a label; typos in the tutorial; remove the empty _static directory, the reference to a missing _templates directory, u'' prefixes and the "One line description of project." placeholder from conf.py.

Verification

Run locally on Python 3.13:

• coverage run tests.py (353 tests) and property_tests.py (25 tests) pass; coverage of jsonpatch.py stays at 100%.
• The property tests whose inputs were widened pass with HYPOTHESIS_PROFILE=thorough (5000 examples each).
• pytest, ext_tests.py tests.js, the flake8 gate, mypy.stubtest, mypy on typetests/usage.py, sphinx-build -W and python -m build + twine check --strict pass.
• The wheel has the same files as before; the only metadata change is the documentation URL. Tests and the docs build also pass from the unpacked sdist.

Not checked: the two new Read the Docs links in the README (/en/latest/tutorial.html), as the sandbox could not reach the site. They follow the standard URL layout.

---

Generated by Claude Code

[coveralls]

coverage: 100.0%. remained the same — claude/inspiring-turing-zftuu5 into master