webdiff 1.3.0b1__tar.gz → 1.4.0b1__tar.gz

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/PKG-INFO

@@ -1,28 +1,27 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.3
 Name: webdiff
-Version: 1.3.0b1
+Version: 1.4.0b1
 Summary: Two-column web-based git difftool
-Home-page: https://github.com/danvk/webdiff/
 License: Apache-2.0
 Keywords: git,difftool,web
 Author: Dan Vanderkam
 Author-email: danvdk@gmail.com
-Requires-Python: >=3.10,<4.0
+Requires-Python: >=3.12,<4.0
 Classifier: Development Status :: 4 - Beta
 Classifier: Environment :: Console
 Classifier: Environment :: Web Environment
 Classifier: Intended Audience :: Developers
 Classifier: License :: OSI Approved :: Apache Software License
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.10
-Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
 Classifier: Topic :: Software Development :: Version Control
 Requires-Dist: PyGithub (>=2.3.0,<3.0.0)
 Requires-Dist: aiohttp (>=3.9.5,<4.0.0)
 Requires-Dist: binaryornot
 Requires-Dist: pillow
 Requires-Dist: unidiff (>=0.7.4,<0.8.0)
+Project-URL: Homepage, https://github.com/danvk/webdiff/
 Project-URL: Repository, https://github.com/danvk/webdiff/
 Description-Content-Type: text/markdown
 
@@ -107,6 +106,14 @@ You can also use `git webshow` in place of `git show` to show a single commit:
 
 `git webshow REF` is shorthand for `git webdiff REF^..REF`.
 
+Recommended aliases:
+
+```bash
+alias gw='git webdiff'
+alias gwh='git webshow head'
+alias gwdm='git webdiff $(git merge-base HEAD main)'
+```
+
 ## Configuration
 
 webdiff can be configured via [`git config`][git config]. To change the syntax highlighting theme, for example:
@@ -172,7 +179,7 @@ To iterate on the PyPI package, run:
 
     pip3 uninstall webdiff
     poetry build
-    pip3 install dist/webdiff-?.?.?.tar.gz
+    pip3 install dist/webdiff-(latest).tar.gz
 
 To publish to pypitest:
 
@@ -183,6 +190,8 @@ And to the real pypi:
 
     poetry publish
 
+You can publish pre-release versions to pypi by adding "bN" to the version number.
+
 See [pypirc][] and [poetry][] docs for details on setting up tokens for pypi.
 
 Publication checklist. Do these from _outside_ the webdiff directory:
@@ -212,13 +221,16 @@ When you run `git webdiff (args)`, it runs:
 
 This tells `git` to set up two directories and invoke `webdiff leftdir rightdir`.
 
-There's one complication involving symlinks. `git difftool -d` may fill one of the sides (typically the right) with symlinks. This is faster than copying files, but unfortunately `git diff --no-index` does not resolve these symlinks. To make this work, if a directory contains symlinks, webdiff makes a copy of it before diffing. For file diffs, it resolves the symlink before passing it to `git diff --no-index`. The upshot is that you can run `git webdiff`, edit a file, reload the browser window and see the changes.
+There are two wrinkles here:
+
+- `git difftool -d` may fill one of the sides (typically the right) with symlinks. This is faster than copying files, but unfortunately `git diff --no-index` does not resolve these symlinks. To make this work, if a directory contains symlinks, webdiff makes a copy of it before diffing. For file diffs, it resolves the symlink before passing it to `git diff --no-index`. The upshot is that you can run `git webdiff`, edit a file, reload the browser window and see the changes.
+- `git difftool` cleans up its temporary directories when the main webdiff process terminates. Since webdiff detaches to give you back your terminal, it has to make another copy of the directories (this time without resolving symlinks) to make sure they're still there for the child process.
 
 [pypirc]: https://packaging.python.org/specifications/pypirc/
 [Homebrew]: https://brew.sh/
 [ImageMagick]: https://imagemagick.org/index.php
 [git config]: https://git-scm.com/docs/git-config
-[themes]: http://example.com
+[themes]: https://github.com/danvk/webdiff/tree/main/webdiff/static/css/themes
 [poetry]: https://python-poetry.org/docs/repositories/#publishable-repositories
 [pipx]: https://pipx.pypa.io/stable/
 

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/README.md

@@ -79,6 +79,14 @@ You can also use `git webshow` in place of `git show` to show a single commit:
 
 `git webshow REF` is shorthand for `git webdiff REF^..REF`.
 
+Recommended aliases:
+
+```bash
+alias gw='git webdiff'
+alias gwh='git webshow head'
+alias gwdm='git webdiff $(git merge-base HEAD main)'
+```
+
 ## Configuration
 
 webdiff can be configured via [`git config`][git config]. To change the syntax highlighting theme, for example:
@@ -144,7 +152,7 @@ To iterate on the PyPI package, run:
 
     pip3 uninstall webdiff
     poetry build
-    pip3 install dist/webdiff-?.?.?.tar.gz
+    pip3 install dist/webdiff-(latest).tar.gz
 
 To publish to pypitest:
 
@@ -155,6 +163,8 @@ And to the real pypi:
 
     poetry publish
 
+You can publish pre-release versions to pypi by adding "bN" to the version number.
+
 See [pypirc][] and [poetry][] docs for details on setting up tokens for pypi.
 
 Publication checklist. Do these from _outside_ the webdiff directory:
@@ -184,12 +194,15 @@ When you run `git webdiff (args)`, it runs:
 
 This tells `git` to set up two directories and invoke `webdiff leftdir rightdir`.
 
-There's one complication involving symlinks. `git difftool -d` may fill one of the sides (typically the right) with symlinks. This is faster than copying files, but unfortunately `git diff --no-index` does not resolve these symlinks. To make this work, if a directory contains symlinks, webdiff makes a copy of it before diffing. For file diffs, it resolves the symlink before passing it to `git diff --no-index`. The upshot is that you can run `git webdiff`, edit a file, reload the browser window and see the changes.
+There are two wrinkles here:
+
+- `git difftool -d` may fill one of the sides (typically the right) with symlinks. This is faster than copying files, but unfortunately `git diff --no-index` does not resolve these symlinks. To make this work, if a directory contains symlinks, webdiff makes a copy of it before diffing. For file diffs, it resolves the symlink before passing it to `git diff --no-index`. The upshot is that you can run `git webdiff`, edit a file, reload the browser window and see the changes.
+- `git difftool` cleans up its temporary directories when the main webdiff process terminates. Since webdiff detaches to give you back your terminal, it has to make another copy of the directories (this time without resolving symlinks) to make sure they're still there for the child process.
 
 [pypirc]: https://packaging.python.org/specifications/pypirc/
 [Homebrew]: https://brew.sh/
 [ImageMagick]: https://imagemagick.org/index.php
 [git config]: https://git-scm.com/docs/git-config
-[themes]: http://example.com
+[themes]: https://github.com/danvk/webdiff/tree/main/webdiff/static/css/themes
 [poetry]: https://python-poetry.org/docs/repositories/#publishable-repositories
 [pipx]: https://pipx.pypa.io/stable/

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "webdiff"
-version = "1.3.0b1"
+version = "1.4.0b1"
 description = "Two-column web-based git difftool"
 license = "Apache-2.0"
 readme = "README.md"
@@ -25,7 +25,7 @@ classifiers = [
 include = ["webdiff/static/js/file_diff.js"]
 
 [tool.poetry.dependencies]
-python = "^3.10"
+python = "^3.12"
 binaryornot = "*"
 pillow = "*"
 PyGithub = "^2.3.0"
@@ -33,8 +33,8 @@ unidiff = "^0.7.4"
 aiohttp = "^3.9.5"
 
 [tool.poetry.group.dev.dependencies]
-pytest = "^7.1.3"
-ruff = "0.7"
+pytest = "^9.0"
+ruff = "^0.15"
 
 [tool.poetry.scripts]
 webdiff = "webdiff.app:run"

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/webdiff/app.py

@@ -13,6 +13,7 @@ import os
 import platform
 import signal
 import socket
+import subprocess
 import sys
 import threading
 import time
@@ -24,6 +25,7 @@ from aiohttp import web
 from binaryornot.check import is_binary
 
 from webdiff import argparser, diff, options, util
+from webdiff.dirdiff import make_resolved_dir
 
 VERSION = importlib.metadata.version('webdiff')
 
@@ -46,6 +48,7 @@ DIFF = None
 PORT = None
 HOSTNAME = 'localhost'
 DEBUG = os.environ.get('DEBUG')
+DEBUG_DETACH = os.environ.get('DEBUG_DETACH')
 WEBDIFF_DIR = determine_path()
 
 if DEBUG:
@@ -273,15 +276,8 @@ def pick_a_port(args, webdiff_config):
 
 
 def run_http():
-    sys.stderr.write(
-        """Serving diffs on http://%s:%s
-Close the browser tab or hit Ctrl-C when you're done.
-"""
-        % (HOSTNAME, PORT)
-    )
     threading.Timer(0.1, open_browser).start()
-
-    web.run_app(app, host=HOSTNAME, port=PORT)
+    web.run_app(app, host=HOSTNAME, port=PORT, print=print if DEBUG else None)
     logging.debug('http server shut down')
 
 
@@ -291,7 +287,7 @@ def maybe_shutdown():
 
     def shutdown():
         if LAST_REQUEST_MS <= last_ms:  # subsequent requests abort shutdown
-            sys.stderr.write('Shutting down...\n')
+            logging.debug('Shutting down...')
             signal.raise_signal(signal.SIGINT)
         else:
             logging.debug('Received subsequent request; shutdown aborted.')
@@ -334,7 +330,36 @@ def run():
         else:
             HOSTNAME = _hostname
 
-    run_http()
+    run_in_process = os.environ.get('WEBDIFF_RUN_IN_PROCESS') or (
+        DEBUG and not DEBUG_DETACH
+    )
+
+    if not os.environ.get('WEBDIFF_LOGGED_MESSAGE'):
+        # Printing this in the main process gives you your prompt back more cleanly.
+        print(
+            """Serving diffs on http://%s:%s
+Close the browser tab when you're done to terminate the process."""
+            % (HOSTNAME, PORT)
+        )
+        os.environ['WEBDIFF_LOGGED_MESSAGE'] = '1'
+
+    if run_in_process:
+        run_http()
+    else:
+        os.environ['WEBDIFF_RUN_IN_PROCESS'] = '1'
+        os.environ['WEBDIFF_PORT'] = str(PORT)
+        if os.environ.get('WEBDIFF_FROM_GIT_DIFFTOOL'):
+            # git difftool will clean up these directories when we detach.
+            # To make them accessible to the child process, we make a (shallow) copy.
+            assert 'dirs' in parsed_args
+            dir_a, dir_b = parsed_args['dirs']
+            copied_dir_a = make_resolved_dir(dir_a)
+            copied_dir_b = make_resolved_dir(dir_b)
+            os.environ['WEBDIFF_DIR_A'] = copied_dir_a
+            os.environ['WEBDIFF_DIR_B'] = copied_dir_b
+            logging.debug(f'Copied {dir_a} -> {copied_dir_a} before detaching')
+            logging.debug(f'Copied {dir_b} -> {copied_dir_b} before detaching')
+        subprocess.Popen((sys.executable, *sys.argv))
 
 
 if __name__ == '__main__':

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/webdiff/argparser.py

@@ -4,9 +4,7 @@ import argparse
 import os
 import re
 
-from webdiff import dirdiff
-from webdiff import githubdiff
-from webdiff import github_fetcher
+from webdiff import dirdiff, github_fetcher, githubdiff
 from webdiff.localfilediff import LocalFileDiff
 
 
@@ -81,6 +79,12 @@ def parse(args, version=None):
 
     else:
         a, b = args.dirs
+        if os.environ.get('WEBDIFF_DIR_A') and os.environ.get('WEBDIFF_DIR_B'):
+            # This happens when you run "git webdiff" and we have to make a copy of
+            # the temp directories before we detach and git difftool cleans them up.
+            a = os.environ.get('WEBDIFF_DIR_A')
+            b = os.environ.get('WEBDIFF_DIR_B')
+
         for x in (a, b):
             if not os.path.exists(x):
                 raise UsageError('"%s" does not exist' % x)

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/webdiff/diff.py

@@ -42,7 +42,12 @@ def fast_num_lines(path: str) -> int:
     # See https://stackoverflow.com/q/9629179/388951 for the idea to use a Unix command.
     # Unfortunately `wc -l` ignores the last line if there is no trailing newline. So
     # instead, see https://stackoverflow.com/a/38870057/388951
-    return int(subprocess.check_output(['grep', '-c', '', path]))
+    try:
+        return int(subprocess.check_output(['grep', '-c', '', path]))
+    except subprocess.CalledProcessError as e:
+        if e.returncode == 1:
+            return 0  # grep -c returns an error code if there are no matches
+        raise
 
 
 def get_diff_ops(

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/webdiff/dirdiff.py

@@ -1,7 +1,7 @@
 """Compute the diff between two directories on local disk."""
 
-import os
 import logging
+import os
 import shutil
 import subprocess
 import tempfile
@@ -27,7 +27,7 @@ def contains_symlinks(dir: str):
     return False
 
 
-def make_resolved_dir(dir: str) -> str:
+def make_resolved_dir(dir: str, follow_symlinks=False) -> str:
     # TODO: clean up this directory
     temp_dir = tempfile.mkdtemp(prefix='webdiff')
     for root, dirs, files in os.walk(dir):
@@ -38,7 +38,7 @@ def make_resolved_dir(dir: str) -> str:
             src_file = os.path.join(root, file_name)
             rel = os.path.relpath(src_file, dir)
             dst_file = os.path.join(temp_dir, rel)
-            shutil.copy(src_file, dst_file, follow_symlinks=True)
+            shutil.copy(src_file, dst_file, follow_symlinks=follow_symlinks)
     return temp_dir
 
 
@@ -49,11 +49,11 @@ def gitdiff(a_dir: str, b_dir: str, webdiff_config):
         cmd += ' ' + extra_args
     a_dir_nosym = a_dir
     if contains_symlinks(a_dir):
-        a_dir_nosym = make_resolved_dir(a_dir)
+        a_dir_nosym = make_resolved_dir(a_dir, follow_symlinks=True)
         logging.debug(f'Inlined symlinks in left directory {a_dir} -> {a_dir_nosym}')
     b_dir_nosym = b_dir
     if contains_symlinks(b_dir):
-        b_dir_nosym = make_resolved_dir(b_dir)
+        b_dir_nosym = make_resolved_dir(b_dir, follow_symlinks=True)
         logging.debug(f'Inlined symlinks in right directory {b_dir} -> {b_dir_nosym}')
     args = cmd.split(' ') + [a_dir_nosym, b_dir_nosym]
     logging.debug('Running git command: %s', args)

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/webdiff/github_fetcher.py

@@ -6,11 +6,11 @@ This will create symlinks or clone git repos as needed.
 # Use this PR for testing to see all four types of change at once:
 # https://github.com/danvk/test-repo/pull/2/
 
-from collections import OrderedDict
 import os
 import re
 import subprocess
 import sys
+from collections import OrderedDict
 
 from github import Github, UnknownObjectException
 
@@ -138,7 +138,7 @@ def _get_github_remotes():
 
 # e.g. 'origin	git@github.com:danvk/expandable-image-grid.git (push)'
 ssh_push_re = re.compile(
-    '(?P<name>[^\s]+)\s+((?P<user>[^@]+)@)?(?P<host>[^:]+)(?::(?P<path>[^\s]+))?\s\(push\)'
+    r'(?P<name>[^\s]+)\s+((?P<user>[^@]+)@)?(?P<host>[^:]+)(?::(?P<path>[^\s]+))?\s\(push\)'
 )
 
 # e.g. 'origin	https://github.com/danvk/git-helpers.git (push)'

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/webdiff/gitwebdiff.py

@@ -21,6 +21,7 @@ def run(argv=sys.argv):
             if not os.environ.get('DEBUG')
             else os.path.join(os.path.curdir, 'test.sh')
         )
+        os.environ['WEBDIFF_FROM_GIT_DIFFTOOL'] = '1'
         subprocess.call(f'git difftool -d -x {cmd}'.split(' ') + argv[1:])
     except KeyboardInterrupt:
         # Don't raise an exception to the user when sigint is received

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/webdiff/static/codediff.js/codediff.css

@@ -66,6 +66,12 @@ table.diff tr.skip-row td {
   padding: 0.5em;
 }
 
+/* Partially collapse empty sides of diffs */
+.diff.diff-add td.before,
+.diff.diff-delete td.after {
+  width: 51ch;
+}
+
 .line-no, .code {
   padding: 2px;
   height: 1.11em;
@@ -154,3 +160,53 @@ span.skip {
   -ms-user-select: text;
   user-select: text;
 }
+
+a, a:visited {
+  text-decoration: none;
+  color: #00a;
+}
+
+tr.skip-row {
+  position: relative;
+  line-height: 1.5em;
+}
+.skip.right {
+  position: absolute;
+}
+.skip.right {
+  right: 32px;
+}
+.skip.expand-up, .skip.expand-down {
+  font-size: 12px;
+  font-family: Inconsolata, Consolas, "Liberation Mono", Menlo, Courier, monospace;
+  color: #aaa;
+  background: #fff;
+  border: 1px solid #e8e8e8;
+  padding: 0px 5px;
+  cursor: pointer;
+  white-space: nowrap;
+  display: inline-block;
+  line-height: 1.3;
+  letter-spacing: 0.04em;
+}
+.skip.expand-up:hover, .skip.expand-down:hover {
+  background: #f5f5f5;
+  color: #888;
+  border-color: #ccc;
+}
+/* tab hangs down from the row above: no top border, rounded bottom corners */
+.skip.expand-up {
+  top: 0;
+  border-top: none;
+  border-radius: 0 0 4px 4px;
+}
+/* tab sticks up from the row below: no bottom border, rounded top corners */
+.skip.expand-down {
+  bottom: 0;
+  border-bottom: none;
+  border-radius: 4px 4px 0 0;
+}
+
+a.show-more {
+  color: #888;
+}

{webdiff-1.3.0b1 → webdiff-1.4.0b1}/webdiff/static/css/style.css

@@ -44,6 +44,10 @@ td.code {
   width: 101ch;  /* per-side max line length */
 }
 
+.code.equal.expanded {
+  background-color: #f7f7f7;
+}
+
 .file-selector {
   padding-bottom: 5px;
   border-bottom: 1px solid #ccc;
@@ -106,29 +110,30 @@ ul.file-list {
 .side-a { border: 1px solid red; }
 .side-b { border: 1px solid green; }
 
-.image-diff-controls {
+/* Per-file type diff controls: image diff mode, JSON normalization */
+.diff-mode-controls {
   text-align: center;
   background-color: #f7f7f7;
   padding: 5px;
 }
-.image-diff-controls > input {
+.diff-mode-controls > input {
   margin-left: 20px;
 }
-.image-diff-controls a {
+.diff-mode-controls a {
   text-decoration: none;
 }
-.image-diff-controls a, .image-diff-controls a:visited {
+.diff-mode-controls a, .diff-mode-controls a:visited {
   color: #666;
 }
-.image-diff-controls b {
+.diff-mode-controls b {
   color: black;
 }
-.image-diff-controls .mode {
+.diff-mode-controls .mode {
   padding-left: 5px;
   padding-right: 5px;
   border-right: 1px solid #ccc;
 }
-.image-diff-controls .mode:last-child {
+.diff-mode-controls .mode:last-child {
   border-right: none;
 }
 
@@ -245,3 +250,8 @@ kbd {
   list-style-type: none;
   margin-bottom: 2px;
 }
+
+.suppressed-large-diff {
+  text-align: center;
+  background-color: lightyellow;
+}