Simplify Repository.cmd_runner

pre_commit/commands.py

@@ -195,11 +195,7 @@ def _run_single_hook(runner, repository, hook_id, args, write, skips=set()):
     write(get_hook_message(_hook_msg_start(hook, args.verbose), end_len=6))
     sys.stdout.flush()
 
-    retcode, stdout, stderr = repository.run_hook(
-        runner.cmd_runner,
-        hook_id,
-        filenames,
-    )
+    retcode, stdout, stderr = repository.run_hook(hook_id, filenames)
 
     if retcode != repository.hooks[hook_id]['expected_return_value']:
         retcode = 1

pre_commit/prefixed_command_runner.py

@@ -40,7 +40,7 @@ class PrefixedCommandRunner(object):
     For instance:
         PrefixedCommandRunner('/tmp/foo').run(['{prefix}foo.sh', 'bar', 'baz'])
 
-    will run ['/tmpl/foo/foo.sh', 'bar', 'baz']
+    will run ['/tmp/foo/foo.sh', 'bar', 'baz']
     """
     def __init__(self, prefix_dir, popen=subprocess.Popen, makedirs=os.makedirs):
         self.prefix_dir = prefix_dir.rstrip(os.sep) + os.sep

pre_commit/repository.py

@@ -43,47 +43,38 @@ class Repository(object):
     def manifest(self):
         return Manifest(self.repo_path_getter)
 
-    def get_cmd_runner(self, hooks_cmd_runner):
-        # TODO: this effectively throws away the original cmd runner
-        return PrefixedCommandRunner.from_command_runner(
-            hooks_cmd_runner, self.repo_path_getter.repo_path,
-        )
+    @cached_property
+    def cmd_runner(self):
+        return PrefixedCommandRunner(self.repo_path_getter.repo_path)
 
-    def require_installed(self, cmd_runner):
+    def require_installed(self):
         if self.__installed:
             return
 
-        self.install(cmd_runner)
+        self.install()
         self.__installed = True
 
-    def install(self, cmd_runner):
-        """Install the hook repository.
-
-        Args:
-            cmd_runner - A `PrefixedCommandRunner` bound to the hooks workspace
-        """
-        repo_cmd_runner = self.get_cmd_runner(cmd_runner)
+    def install(self):
+        """Install the hook repository."""
         for language_name in self.languages:
             language = languages[language_name]
             if (
                 language.ENVIRONMENT_DIR is None or
-                repo_cmd_runner.exists(language.ENVIRONMENT_DIR)
+                self.cmd_runner.exists(language.ENVIRONMENT_DIR)
             ):
                 # The language is already installed
                 continue
-            language.install_environment(repo_cmd_runner)
+            language.install_environment(self.cmd_runner)
 
-    def run_hook(self, cmd_runner, hook_id, file_args):
+    def run_hook(self, hook_id, file_args):
         """Run a hook.
 
         Args:
-            cmd_runner - A `PrefixedCommandRunner` bound to the hooks workspace
             hook_id - Id of the hook
             file_args - List of files to run
         """
-        self.require_installed(cmd_runner)
-        repo_cmd_runner = self.get_cmd_runner(cmd_runner)
+        self.require_installed()
         hook = self.hooks[hook_id]
         return languages[hook['language']].run_hook(
-            repo_cmd_runner, hook, file_args,
+            self.cmd_runner, hook, file_args,
         )