summaryrefslogtreecommitdiffstats
path: root/gitlint-core/gitlint/git.py
blob: 6612a7d0ed59bf154970247ff7fc67811db3fabd (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
import logging
import os
from pathlib import Path

import arrow

from gitlint import shell as sh
from gitlint.cache import PropertyCache, cache
from gitlint.exception import GitlintError

# import exceptions separately, this makes it a little easier to mock them out in the unit tests
from gitlint.shell import CommandNotFound, ErrorReturnCode

# For now, the git date format we use is fixed, but technically this format is determined by `git config log.date`
# We should fix this at some point :-)
GIT_TIMEFORMAT = "YYYY-MM-DD HH:mm:ss Z"

LOG = logging.getLogger(__name__)


class GitContextError(GitlintError):
    """Exception indicating there is an issue with the git context"""


class GitNotInstalledError(GitContextError):
    def __init__(self):
        super().__init__(
            "'git' command not found. You need to install git to use gitlint on a local repository. "
            "See https://git-scm.com/book/en/v2/Getting-Started-Installing-Git on how to install git."
        )


class GitExitCodeError(GitContextError):
    def __init__(self, command, stderr):
        self.command = command
        self.stderr = stderr
        super().__init__(f"An error occurred while executing '{command}': {stderr}")


def _git(*command_parts, **kwargs):
    """Convenience function for running git commands. Automatically deals with exceptions and unicode."""
    git_kwargs = {"_tty_out": False}
    git_kwargs.update(kwargs)
    try:
        LOG.debug(command_parts)
        result = sh.git(*command_parts, **git_kwargs)
        # If we reach this point and the result has an exit_code that is larger than 0, this means that we didn't
        # get an exception (which is the default sh behavior for non-zero exit codes) and so the user is expecting
        # a non-zero exit code -> just return the entire result
        if hasattr(result, "exit_code") and result.exit_code > 0:
            return result
        return str(result)
    except CommandNotFound as e:
        raise GitNotInstalledError from e
    except ErrorReturnCode as e:  # Something went wrong while executing the git command
        error_msg = e.stderr.strip()
        error_msg_lower = error_msg.lower()
        if "_cwd" in git_kwargs and b"not a git repository" in error_msg_lower:
            raise GitContextError(f"{git_kwargs['_cwd']} is not a git repository.") from e

        if (
            b"does not have any commits yet" in error_msg_lower
            or b"ambiguous argument 'head': unknown revision" in error_msg_lower
        ):
            msg = "Current branch has no commits. Gitlint requires at least one commit to function."
            raise GitContextError(msg) from e

        raise GitExitCodeError(e.full_cmd, error_msg) from e


def git_version():
    """Determine the git version installed on this host by calling git --version"""
    return _git("--version").replace("\n", "")


def git_commentchar(repository_path=None):
    """Shortcut for retrieving comment char from git config"""
    commentchar = _git("config", "--get", "core.commentchar", _cwd=repository_path, _ok_code=[0, 1])
    # git will return an exit code of 1 if it can't find a config value, in this case we fall-back to # as commentchar
    if hasattr(commentchar, "exit_code") and commentchar.exit_code == 1:
        commentchar = "#"
    return commentchar.replace("\n", "")


def git_hooks_dir(repository_path):
    """Determine hooks directory for a given target dir"""
    hooks_dir = _git("rev-parse", "--git-path", "hooks", _cwd=repository_path)
    hooks_dir = hooks_dir.replace("\n", "")
    return os.path.realpath(os.path.join(repository_path, hooks_dir))


def _parse_git_changed_file_stats(changed_files_stats_raw):
    """Parse the output of git diff --numstat and return a dict of:
    dict[filename: GitChangedFileStats(filename, additions, deletions)]"""
    changed_files_stats_lines = changed_files_stats_raw.split("\n")
    changed_files_stats = {}
    for line in changed_files_stats_lines[:-1]:  # drop last empty line
        line_stats = line.split()

        # If the file is binary, numstat will show "-"
        # See https://git-scm.com/docs/git-diff#Documentation/git-diff.txt---numstat
        additions = int(line_stats[0]) if line_stats[0] != "-" else None
        deletions = int(line_stats[1]) if line_stats[1] != "-" else None

        changed_file_stat = GitChangedFileStats(line_stats[2], additions, deletions)
        changed_files_stats[line_stats[2]] = changed_file_stat

    return changed_files_stats


class GitCommitMessage:
    """Class representing a git commit message. A commit message consists of the following:
    - context: The `GitContext` this commit message is part of
    - original: The actual commit message as returned by `git log`
    - full: original, but stripped of any comments
    - title: the first line of full
    - body: all lines following the title
    """

    def __init__(self, context, original=None, full=None, title=None, body=None):
        self.context = context
        self.original = original
        self.full = full
        self.title = title
        self.body = body

    @staticmethod
    def from_full_message(context, commit_msg_str):
        """Parses a full git commit message by parsing a given string into the different parts of a commit message"""
        all_lines = commit_msg_str.splitlines()
        cutline = f"{context.commentchar} ------------------------ >8 ------------------------"
        try:
            cutline_index = all_lines.index(cutline)
        except ValueError:
            cutline_index = None
        lines = [line for line in all_lines[:cutline_index] if not line.startswith(context.commentchar)]
        full = "\n".join(lines)
        title = lines[0] if lines else ""
        body = lines[1:] if len(lines) > 1 else []
        return GitCommitMessage(context=context, original=commit_msg_str, full=full, title=title, body=body)

    def __str__(self):
        return self.full

    def __eq__(self, other):
        return (
            isinstance(other, GitCommitMessage)
            and self.original == other.original
            and self.full == other.full
            and self.title == other.title
            and self.body == other.body
        )


class GitChangedFileStats:
    """Class representing the stats for a changed file in git"""

    def __init__(self, filepath, additions, deletions):
        self.filepath = Path(filepath)
        self.additions = additions
        self.deletions = deletions

    def __eq__(self, other):
        return (
            isinstance(other, GitChangedFileStats)
            and self.filepath == other.filepath
            and self.additions == other.additions
            and self.deletions == other.deletions
        )

    def __str__(self) -> str:
        return f"{self.filepath}: {self.additions} additions, {self.deletions} deletions"


class GitCommit:
    """Class representing a git commit.
    A commit consists of: context, message, author name, author email, date, list of parent commit shas,
    list of changed files, list of branch names.
    In the context of gitlint, only the git context and commit message are required.
    """

    def __init__(
        self,
        context,
        message,
        sha=None,
        date=None,
        author_name=None,
        author_email=None,
        parents=None,
        changed_files_stats=None,
        branches=None,
    ):
        self.context = context
        self.message = message
        self.sha = sha
        self.date = date
        self.author_name = author_name
        self.author_email = author_email
        self.parents = parents or []  # parent commit hashes
        self.changed_files_stats = changed_files_stats or {}
        self.branches = branches or []

    @property
    def is_merge_commit(self):
        return self.message.title.startswith("Merge")

    @property
    def is_fixup_commit(self):
        return self.message.title.startswith("fixup!")

    @property
    def is_squash_commit(self):
        return self.message.title.startswith("squash!")

    @property
    def is_fixup_amend_commit(self):
        return self.message.title.startswith("amend!")

    @property
    def is_revert_commit(self):
        return self.message.title.startswith("Revert")

    @property
    def changed_files(self):
        return list(self.changed_files_stats.keys())

    def __str__(self):
        date_str = arrow.get(self.date).format(GIT_TIMEFORMAT) if self.date else None

        if len(self.changed_files_stats) > 0:
            changed_files_stats_str = "\n  " + "\n  ".join([str(stats) for stats in self.changed_files_stats.values()])
        else:
            changed_files_stats_str = " {}"

        return (
            f"--- Commit Message ----\n{self.message}\n"
            "--- Meta info ---------\n"
            f"Author: {self.author_name} <{self.author_email}>\n"
            f"Date:   {date_str}\n"
            f"is-merge-commit:  {self.is_merge_commit}\n"
            f"is-fixup-commit:  {self.is_fixup_commit}\n"
            f"is-fixup-amend-commit: {self.is_fixup_amend_commit}\n"
            f"is-squash-commit: {self.is_squash_commit}\n"
            f"is-revert-commit: {self.is_revert_commit}\n"
            f"Parents: {self.parents}\n"
            f"Branches: {self.branches}\n"
            f"Changed Files: {self.changed_files}\n"
            f"Changed Files Stats:{changed_files_stats_str}\n"
            "-----------------------"
        )

    def __eq__(self, other):
        # skip checking the context as context refers back to this obj, this will trigger a cyclic dependency
        return (
            isinstance(other, GitCommit)
            and self.message == other.message
            and self.sha == other.sha
            and self.author_name == other.author_name
            and self.author_email == other.author_email
            and self.date == other.date
            and self.parents == other.parents
            and self.is_merge_commit == other.is_merge_commit
            and self.is_fixup_commit == other.is_fixup_commit
            and self.is_fixup_amend_commit == other.is_fixup_amend_commit
            and self.is_squash_commit == other.is_squash_commit
            and self.is_revert_commit == other.is_revert_commit
            and self.changed_files == other.changed_files
            and self.changed_files_stats == other.changed_files_stats
            and self.branches == other.branches
        )


class LocalGitCommit(GitCommit, PropertyCache):
    """Class representing a git commit that exists in the local git repository.
    This class uses lazy loading: it defers reading information from the local git repository until the associated
    property is accessed for the first time. Properties are then cached for subsequent access.

    This approach ensures that we don't do 'expensive' git calls when certain properties are not actually used.
    In addition, reading the required info when it's needed rather than up front avoids adding delay during gitlint
    startup time and reduces gitlint's memory footprint.
    """

    def __init__(self, context, sha):
        PropertyCache.__init__(self)
        self.context = context
        self.sha = sha

    def _log(self):
        """Does a call to `git log` to determine a bunch of information about the commit."""
        long_format = "--pretty=%aN%x00%aE%x00%ai%x00%P%n%B"
        raw_commit = _git("log", self.sha, "-1", long_format, _cwd=self.context.repository_path).split("\n")

        (name, email, date, parents), commit_msg = raw_commit[0].split("\x00"), "\n".join(raw_commit[1:])

        commit_parents = [] if parents == "" else parents.split(" ")
        commit_is_merge_commit = len(commit_parents) > 1

        # "YYYY-MM-DD HH:mm:ss Z" -> ISO 8601-like format
        # Use arrow for datetime parsing, because apparently python is quirky around ISO-8601 dates:
        # http://stackoverflow.com/a/30696682/381010
        commit_date = arrow.get(date, GIT_TIMEFORMAT).datetime

        # Create Git commit object with the retrieved info
        commit_msg_obj = GitCommitMessage.from_full_message(self.context, commit_msg)

        self._cache.update(
            {
                "message": commit_msg_obj,
                "author_name": name,
                "author_email": email,
                "date": commit_date,
                "parents": commit_parents,
                "is_merge_commit": commit_is_merge_commit,
            }
        )

    @property
    def message(self):
        return self._try_cache("message", self._log)

    @property
    def author_name(self):
        return self._try_cache("author_name", self._log)

    @property
    def author_email(self):
        return self._try_cache("author_email", self._log)

    @property
    def date(self):
        return self._try_cache("date", self._log)

    @property
    def parents(self):
        return self._try_cache("parents", self._log)

    @property
    def branches(self):
        def cache_branches():
            # We have to parse 'git branch --contains <sha>' instead of 'git for-each-ref' to be compatible with
            # git versions < 2.7.0
            # https://stackoverflow.com/questions/45173979/can-i-force-git-branch-contains-tag-to-not-print-the-asterisk
            branches = _git("branch", "--contains", self.sha, _cwd=self.context.repository_path).split("\n")

            # This means that we need to remove any leading * that indicates the current branch. Note that we can
            # safely do this since git branches cannot contain '*' anywhere, so if we find an '*' we know it's output
            # from the git CLI and not part of the branch name. See https://git-scm.com/docs/git-check-ref-format
            # We also drop the last empty line from the output.
            self._cache["branches"] = [branch.replace("*", "").strip() for branch in branches[:-1]]

        return self._try_cache("branches", cache_branches)

    @property
    def is_merge_commit(self):
        return self._try_cache("is_merge_commit", self._log)

    @property
    def changed_files_stats(self):
        def cache_changed_files_stats():
            changed_files_stats_raw = _git(
                "diff-tree", "--no-commit-id", "--numstat", "-r", "--root", self.sha, _cwd=self.context.repository_path
            )
            self._cache["changed_files_stats"] = _parse_git_changed_file_stats(changed_files_stats_raw)

        return self._try_cache("changed_files_stats", cache_changed_files_stats)


class StagedLocalGitCommit(GitCommit, PropertyCache):
    """Class representing a git commit that has been staged, but not committed.

    Other than the commit message itself (and changed files), a lot of information is actually not known at staging
    time, since the commit hasn't happened yet. However, we can make educated guesses based on existing repository
    information.
    """

    def __init__(self, context, commit_message):
        PropertyCache.__init__(self)
        self.context = context
        self.message = commit_message
        self.sha = None
        self.parents = []  # Not really possible to determine before a commit

    @property
    @cache
    def author_name(self):
        try:
            return _git("config", "--get", "user.name", _cwd=self.context.repository_path).strip()
        except GitExitCodeError as e:
            raise GitContextError("Missing git configuration: please set user.name") from e

    @property
    @cache
    def author_email(self):
        try:
            return _git("config", "--get", "user.email", _cwd=self.context.repository_path).strip()
        except GitExitCodeError as e:
            raise GitContextError("Missing git configuration: please set user.email") from e

    @property
    @cache
    def date(self):
        # We don't know the actual commit date yet, but we make a pragmatic trade-off here by providing the current date
        # We get current date from arrow, reformat in git date format, then re-interpret it as a date.
        # This ensure we capture the same precision and timezone information that git does.
        return arrow.get(arrow.now().format(GIT_TIMEFORMAT), GIT_TIMEFORMAT).datetime

    @property
    @cache
    def branches(self):
        # We don't know the branch this commit will be part of yet, but we're pragmatic here and just return the
        # current branch, as for all intents and purposes, this will be what the user is looking for.
        return [self.context.current_branch]

    @property
    def changed_files_stats(self):
        def cache_changed_files_stats():
            changed_files_stats_raw = _git("diff", "--staged", "--numstat", "-r", _cwd=self.context.repository_path)
            self._cache["changed_files_stats"] = _parse_git_changed_file_stats(changed_files_stats_raw)

        return self._try_cache("changed_files_stats", cache_changed_files_stats)


class GitContext(PropertyCache):
    """Class representing the git context in which gitlint is operating: a data object storing information about
    the git repository that gitlint is linting.
    """

    def __init__(self, repository_path=None):
        PropertyCache.__init__(self)
        self.commits = []
        self.repository_path = repository_path

    @property
    @cache
    def commentchar(self):
        return git_commentchar(self.repository_path)

    @property
    @cache
    def current_branch(self):
        try:
            current_branch = _git("rev-parse", "--abbrev-ref", "HEAD", _cwd=self.repository_path).strip()
        except GitContextError:
            # Maybe there is no commit.  Try another way to get current branch (need Git 2.22+)
            current_branch = _git("branch", "--show-current", _cwd=self.repository_path).strip()
        return current_branch

    @staticmethod
    def from_commit_msg(commit_msg_str):
        """Determines git context based on a commit message.
        :param commit_msg_str: Full git commit message.
        """
        context = GitContext()
        commit_msg_obj = GitCommitMessage.from_full_message(context, commit_msg_str)
        commit = GitCommit(context, commit_msg_obj)
        context.commits.append(commit)
        return context

    @staticmethod
    def from_staged_commit(commit_msg_str, repository_path):
        """Determines git context based on a commit message that is a staged commit for a local git repository.
        :param commit_msg_str: Full git commit message.
        :param repository_path: Path to the git repository to retrieve the context from
        """
        context = GitContext(repository_path=repository_path)
        commit_msg_obj = GitCommitMessage.from_full_message(context, commit_msg_str)
        commit = StagedLocalGitCommit(context, commit_msg_obj)
        context.commits.append(commit)
        return context

    @staticmethod
    def from_local_repository(repository_path, refspec=None, commit_hashes=None):
        """Retrieves the git context from a local git repository.
        :param repository_path: Path to the git repository to retrieve the context from
        :param refspec: The commit(s) to retrieve (mutually exclusive with `commit_hash`)
        :param commit_hash: Hash of the commit to retrieve (mutually exclusive with `refspec`)
        """

        context = GitContext(repository_path=repository_path)

        if refspec:
            sha_list = _git("rev-list", refspec, _cwd=repository_path).split()
        elif commit_hashes:  # One or more commit hashes, just pass it to `git log -1`
            # Even though we have already been passed the commit hash, we ask git to retrieve this hash and
            # return it to us. This way we verify that the passed hash is a valid hash for the target repo and we
            # also convert it to the full hash format (we might have been passed a short hash).
            sha_list = []
            for commit_hash in commit_hashes:
                sha_list.append(_git("log", "-1", commit_hash, "--pretty=%H", _cwd=repository_path).replace("\n", ""))
        else:  # If no refspec is defined, fallback to the last commit on the current branch
            # We tried many things here e.g.: defaulting to e.g. HEAD or HEAD^... (incl. dealing with
            # repos that only have a single commit - HEAD^... doesn't work there), but then we still get into
            # problems with e.g. merge commits. Easiest solution is just taking the SHA from `git log -1`.
            sha_list = [_git("log", "-1", "--pretty=%H", _cwd=repository_path).replace("\n", "")]

        for sha in sha_list:
            commit = LocalGitCommit(context, sha)
            context.commits.append(commit)

        return context

    def __eq__(self, other):
        return (
            isinstance(other, GitContext)
            and self.commits == other.commits
            and self.repository_path == other.repository_path
            and self.commentchar == other.commentchar
            and self.current_branch == other.current_branch
        )