Remove Manifest, no longer a useful abstraction

asottile · asottile · commit 84b1ba520d42 · 2017-10-26T16:17:52.000-07:00
diff --git a/pre_commit/commands/autoupdate.py b/pre_commit/commands/autoupdate.py
@@ -57,7 +57,7 @@ def _update_repo(repo_config, runner, tags_only):
 
     # See if any of our hooks were deleted with the new commits
     hooks = {hook['id'] for hook in repo.repo_config['hooks']}
-    hooks_missing = hooks - (hooks & set(new_repo.manifest.hooks))
+    hooks_missing = hooks - (hooks & set(new_repo.manifest_hooks))
     if hooks_missing:
         raise RepositoryCannotBeUpdatedError(
             'Cannot update because the tip of master is missing these hooks:\n'
diff --git a/pre_commit/commands/try_repo.py b/pre_commit/commands/try_repo.py
@@ -9,8 +9,8 @@
 import pre_commit.constants as C
 from pre_commit import git
 from pre_commit import output
+from pre_commit.clientlib import load_manifest
 from pre_commit.commands.run import run
-from pre_commit.manifest import Manifest
 from pre_commit.runner import Runner
 from pre_commit.store import Store
 from pre_commit.util import tmpdir
@@ -23,8 +23,10 @@ def try_repo(args):
         if args.hook:
             hooks = [{'id': args.hook}]
         else:
-            manifest = Manifest(Store(tempdir).clone(args.repo, ref))
-            hooks = [{'id': hook_id} for hook_id in sorted(manifest.hooks)]
+            repo_path = Store(tempdir).clone(args.repo, ref)
+            manifest = load_manifest(os.path.join(repo_path, C.MANIFEST_FILE))
+            manifest = sorted(manifest, key=lambda hook: hook['id'])
+            hooks = [{'id': hook['id']} for hook in manifest]
 
         items = (('repo', args.repo), ('sha', ref), ('hooks', hooks))
         config = {'repos': [collections.OrderedDict(items)]}
diff --git a/pre_commit/manifest.py b/pre_commit/manifest.py
diff --git a/pre_commit/repository.py b/pre_commit/repository.py
@@ -14,10 +14,10 @@
 from pre_commit import five
 from pre_commit import git
 from pre_commit.clientlib import is_local_repo
+from pre_commit.clientlib import load_manifest
 from pre_commit.clientlib import MANIFEST_HOOK_DICT
 from pre_commit.languages.all import languages
 from pre_commit.languages.helpers import environment_dir
-from pre_commit.manifest import Manifest
 from pre_commit.prefixed_command_runner import PrefixedCommandRunner
 from pre_commit.schema import apply_defaults
 from pre_commit.schema import validate
@@ -152,13 +152,14 @@ def _cmd_runner_from_deps(self, language_name, deps):
         return self._cmd_runner
 
     @cached_property
-    def manifest(self):
-        return Manifest(self._repo_path)
+    def manifest_hooks(self):
+        manifest_path = os.path.join(self._repo_path, C.MANIFEST_FILE)
+        return {hook['id']: hook for hook in load_manifest(manifest_path)}
 
     @cached_property
     def hooks(self):
         for hook in self.repo_config['hooks']:
-            if hook['id'] not in self.manifest.hooks:
+            if hook['id'] not in self.manifest_hooks:
                 logger.error(
                     '`{}` is not present in repository {}.  '
                     'Typo? Perhaps it is introduced in a newer version?  '
@@ -169,7 +170,7 @@ def hooks(self):
                 exit(1)
 
         return tuple(
-            (hook['id'], _hook(self.manifest.hooks[hook['id']], hook))
+            (hook['id'], _hook(self.manifest_hooks[hook['id']], hook))
             for hook in self.repo_config['hooks']
         )
 
diff --git a/tests/manifest_test.py b/tests/manifest_test.py
diff --git a/tests/repository_test.py b/tests/repository_test.py
@@ -746,3 +746,29 @@ def test_versions_ok(tempdir_factory, store, version):
     config = make_config_from_repo(path)
     # Should succeed
     Repository.create(config, store).require_installed()
+
+
+def test_manifest_hooks(tempdir_factory, store):
+    path = make_repo(tempdir_factory, 'script_hooks_repo')
+    config = make_config_from_repo(path)
+    repo = Repository.create(config, store)
+
+    assert repo.manifest_hooks['bash_hook'] == {
+        'always_run': False,
+        'additional_dependencies': [],
+        'args': [],
+        'description': '',
+        'entry': 'bin/hook.sh',
+        'exclude': '^$',
+        'files': '',
+        'id': 'bash_hook',
+        'language': 'script',
+        'language_version': 'default',
+        'log_file': '',
+        'minimum_pre_commit_version': '0',
+        'name': 'Bash hook',
+        'pass_filenames': True,
+        'stages': [],
+        'types': ['file'],
+        'exclude_types': [],
+    }
