🐛 FIX: pre-commit hook for file creation

Author: chrisjsewell

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
 -   repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v2.3.0
+    rev: v3.2.0
     hooks:
     -   id: check-yaml
     -   id: end-of-file-fixer
@@ -12,11 +12,11 @@ repos:
     -   id: black
 
 - repo: https://github.com/mgedmin/check-manifest
-  rev: "0.39"
+  rev: "0.42"
   hooks:
   - id: check-manifest
 
 - repo: https://gitlab.com/pycqa/flake8
-  rev: 3.7.9
+  rev: 3.8.3
   hooks:
   - id: flake8

scss_compile/__init__.py

@@ -4,6 +4,7 @@
 import hashlib
 import os
 from pathlib import Path
+from subprocess import check_call
 import sys
 from typing import Set
 
@@ -237,18 +238,14 @@ def run_compile(
         else:
             css_out_path = out_dir / (out_name + ".css")
         if not test_run:
-            if css_out_path.exists():
-                if css_str != css_out_path.read_text(encoding=encoding):
-                    changed_files = True
-            else:
+
+            if update_file(css_out_path, css_str, encoding):
                 changed_files = True
-            css_out_path.write_text(css_str, encoding=encoding)
-            if sourcemap:
-                if not (out_dir / (scss_path.name + ".map.json")).exists():
-                    changed_files = True
-                (out_dir / (scss_path.name + ".map.json")).write_text(
-                    sourcemap_str, encoding=encoding
-                )
+            if sourcemap and update_file(
+                out_dir / (scss_path.name + ".map.json"), sourcemap_str, encoding
+            ):
+                changed_files = True
+
         if not quiet:
             click.echo(f"Compiled: {str(scss_path)} -> {str(css_out_path)}")
 
@@ -262,5 +259,21 @@ def run_compile(
 
     if changed_files:
         if not quiet:
-            click.secho("File changed", fg="yellow")
+            click.secho("File(s) changed", fg="yellow")
         sys.exit(exit_code)
+
+
+def update_file(path, text, encoding) -> bool:
+
+    if not path.exists():
+        path.write_text(text, encoding=encoding)
+        # this is required, to ensure creations are picked up by pre-commit
+        # TODO ignore for non-git repositories?
+        check_call(["git", "add", "--intent-to-add", str(path)])
+        return True
+
+    if text != path.read_text(encoding=encoding):
+        path.write_text(text, encoding=encoding)
+        return True
+
+    return False