ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 1 | # Copyright 2013 The Chromium Authors. All rights reserved. |
| 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | """Interactive tool for finding reviewers/owners for a change.""" |
| 6 | |
Raul Tambre | 80ee78e | 2019-05-06 22:41:05 +0000 | [diff] [blame] | 7 | from __future__ import print_function |
| 8 | |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 9 | import os |
| 10 | import copy |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 11 | |
| 12 | |
Gavin Mak | d36dbbd | 2021-01-25 19:34:58 +0000 | [diff] [blame] | 13 | import git_common |
Edward Lesmes | ae3586b | 2020-03-23 21:21:14 +0000 | [diff] [blame] | 14 | import gclient_utils |
| 15 | |
| 16 | |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 17 | def first(iterable): |
| 18 | for element in iterable: |
| 19 | return element |
| 20 | |
| 21 | |
| 22 | class OwnersFinder(object): |
| 23 | COLOR_LINK = '\033[4m' |
| 24 | COLOR_BOLD = '\033[1;32m' |
| 25 | COLOR_GREY = '\033[0;37m' |
| 26 | COLOR_RESET = '\033[0m' |
| 27 | |
| 28 | indentation = 0 |
| 29 | |
Edward Lesmes | 5cd7547 | 2021-02-19 00:34:25 +0000 | [diff] [blame^] | 30 | def __init__(self, files, author, reviewers, owners_client, |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 31 | email_postfix='@chromium.org', |
Jochen Eisinger | d0573ec | 2017-04-13 10:55:06 +0200 | [diff] [blame] | 32 | disable_color=False, |
Sylvain Defresne | b1f865d | 2019-02-12 12:38:22 +0000 | [diff] [blame] | 33 | ignore_author=False): |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 34 | self.email_postfix = email_postfix |
| 35 | |
| 36 | if os.name == 'nt' or disable_color: |
| 37 | self.COLOR_LINK = '' |
| 38 | self.COLOR_BOLD = '' |
| 39 | self.COLOR_GREY = '' |
| 40 | self.COLOR_RESET = '' |
| 41 | |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 42 | self.author = author |
| 43 | |
| 44 | filtered_files = files |
| 45 | |
Edward Lemur | 707d70b | 2018-02-07 00:50:14 +0100 | [diff] [blame] | 46 | reviewers = list(reviewers) |
Sylvain Defresne | b1f865d | 2019-02-12 12:38:22 +0000 | [diff] [blame] | 47 | if author and not ignore_author: |
Edward Lemur | 707d70b | 2018-02-07 00:50:14 +0100 | [diff] [blame] | 48 | reviewers.append(author) |
| 49 | |
| 50 | # Eliminate files that existing reviewers can review. |
Edward Lesmes | 5cd7547 | 2021-02-19 00:34:25 +0000 | [diff] [blame^] | 51 | self.owners_client = owners_client |
| 52 | approval_status = self.owners_client.GetFilesApprovalStatus( |
Gavin Mak | d36dbbd | 2021-01-25 19:34:58 +0000 | [diff] [blame] | 53 | filtered_files, reviewers, []) |
| 54 | filtered_files = [ |
| 55 | f for f in filtered_files |
Edward Lesmes | 5cd7547 | 2021-02-19 00:34:25 +0000 | [diff] [blame^] | 56 | if approval_status[f] != self.owners_client.APPROVED] |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 57 | |
| 58 | # If some files are eliminated. |
| 59 | if len(filtered_files) != len(files): |
| 60 | files = filtered_files |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 61 | |
Edward Lesmes | 5cd7547 | 2021-02-19 00:34:25 +0000 | [diff] [blame^] | 62 | self.files_to_owners = self.owners_client.BatchListOwners(files) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 63 | |
| 64 | self.owners_to_files = {} |
Gavin Mak | d36dbbd | 2021-01-25 19:34:58 +0000 | [diff] [blame] | 65 | self._map_owners_to_files() |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 66 | |
| 67 | self.original_files_to_owners = copy.deepcopy(self.files_to_owners) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 68 | |
| 69 | # This is the queue that will be shown in the interactive questions. |
| 70 | # It is initially sorted by the score in descending order. In the |
| 71 | # interactive questions a user can choose to "defer" its decision, then the |
| 72 | # owner will be put to the end of the queue and shown later. |
| 73 | self.owners_queue = [] |
| 74 | |
| 75 | self.unreviewed_files = set() |
| 76 | self.reviewed_by = {} |
| 77 | self.selected_owners = set() |
| 78 | self.deselected_owners = set() |
| 79 | self.reset() |
| 80 | |
| 81 | def run(self): |
| 82 | self.reset() |
| 83 | while self.owners_queue and self.unreviewed_files: |
| 84 | owner = self.owners_queue[0] |
| 85 | |
| 86 | if (owner in self.selected_owners) or (owner in self.deselected_owners): |
| 87 | continue |
| 88 | |
| 89 | if not any((file_name in self.unreviewed_files) |
| 90 | for file_name in self.owners_to_files[owner]): |
| 91 | self.deselect_owner(owner) |
| 92 | continue |
| 93 | |
| 94 | self.print_info(owner) |
| 95 | |
| 96 | while True: |
| 97 | inp = self.input_command(owner) |
| 98 | if inp == 'y' or inp == 'yes': |
| 99 | self.select_owner(owner) |
| 100 | break |
| 101 | elif inp == 'n' or inp == 'no': |
| 102 | self.deselect_owner(owner) |
| 103 | break |
| 104 | elif inp == '' or inp == 'd' or inp == 'defer': |
| 105 | self.owners_queue.append(self.owners_queue.pop(0)) |
| 106 | break |
| 107 | elif inp == 'f' or inp == 'files': |
| 108 | self.list_files() |
| 109 | break |
| 110 | elif inp == 'o' or inp == 'owners': |
| 111 | self.list_owners(self.owners_queue) |
| 112 | break |
| 113 | elif inp == 'p' or inp == 'pick': |
Edward Lesmes | ae3586b | 2020-03-23 21:21:14 +0000 | [diff] [blame] | 114 | self.pick_owner(gclient_utils.AskForData('Pick an owner: ')) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 115 | break |
| 116 | elif inp.startswith('p ') or inp.startswith('pick '): |
| 117 | self.pick_owner(inp.split(' ', 2)[1].strip()) |
| 118 | break |
| 119 | elif inp == 'r' or inp == 'restart': |
| 120 | self.reset() |
| 121 | break |
| 122 | elif inp == 'q' or inp == 'quit': |
| 123 | # Exit with error |
| 124 | return 1 |
| 125 | |
| 126 | self.print_result() |
| 127 | return 0 |
| 128 | |
Gavin Mak | d36dbbd | 2021-01-25 19:34:58 +0000 | [diff] [blame] | 129 | def _map_owners_to_files(self): |
| 130 | for file_name in self.files_to_owners: |
| 131 | for owner in self.files_to_owners[file_name]: |
| 132 | self.owners_to_files.setdefault(owner, set()) |
| 133 | self.owners_to_files[owner].add(file_name) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 134 | |
| 135 | def reset(self): |
| 136 | self.files_to_owners = copy.deepcopy(self.original_files_to_owners) |
| 137 | self.unreviewed_files = set(self.files_to_owners.keys()) |
| 138 | self.reviewed_by = {} |
| 139 | self.selected_owners = set() |
| 140 | self.deselected_owners = set() |
| 141 | |
Bruce Dawson | 37740e2 | 2019-11-14 00:27:44 +0000 | [diff] [blame] | 142 | # Randomize owners' names so that if many reviewers have identical scores |
| 143 | # they will be randomly ordered to avoid bias. |
Edward Lesmes | 5cd7547 | 2021-02-19 00:34:25 +0000 | [diff] [blame^] | 144 | owners = list(self.owners_client.ScoreOwners(self.files_to_owners.keys())) |
Gavin Mak | d36dbbd | 2021-01-25 19:34:58 +0000 | [diff] [blame] | 145 | if self.author and self.author in owners: |
| 146 | owners.remove(self.author) |
| 147 | self.owners_queue = owners |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 148 | self.find_mandatory_owners() |
| 149 | |
| 150 | def select_owner(self, owner, findMandatoryOwners=True): |
| 151 | if owner in self.selected_owners or owner in self.deselected_owners\ |
| 152 | or not (owner in self.owners_queue): |
| 153 | return |
| 154 | self.writeln('Selected: ' + owner) |
| 155 | self.owners_queue.remove(owner) |
| 156 | self.selected_owners.add(owner) |
| 157 | for file_name in filter( |
| 158 | lambda file_name: file_name in self.unreviewed_files, |
| 159 | self.owners_to_files[owner]): |
| 160 | self.unreviewed_files.remove(file_name) |
| 161 | self.reviewed_by[file_name] = owner |
| 162 | if findMandatoryOwners: |
| 163 | self.find_mandatory_owners() |
| 164 | |
| 165 | def deselect_owner(self, owner, findMandatoryOwners=True): |
| 166 | if owner in self.selected_owners or owner in self.deselected_owners\ |
| 167 | or not (owner in self.owners_queue): |
| 168 | return |
| 169 | self.writeln('Deselected: ' + owner) |
| 170 | self.owners_queue.remove(owner) |
| 171 | self.deselected_owners.add(owner) |
| 172 | for file_name in self.owners_to_files[owner] & self.unreviewed_files: |
| 173 | self.files_to_owners[file_name].remove(owner) |
| 174 | if findMandatoryOwners: |
| 175 | self.find_mandatory_owners() |
| 176 | |
| 177 | def find_mandatory_owners(self): |
| 178 | continues = True |
| 179 | for owner in self.owners_queue: |
| 180 | if owner in self.selected_owners: |
| 181 | continue |
| 182 | if owner in self.deselected_owners: |
| 183 | continue |
| 184 | if len(self.owners_to_files[owner] & self.unreviewed_files) == 0: |
| 185 | self.deselect_owner(owner, False) |
| 186 | |
| 187 | while continues: |
| 188 | continues = False |
| 189 | for file_name in filter( |
| 190 | lambda file_name: len(self.files_to_owners[file_name]) == 1, |
| 191 | self.unreviewed_files): |
| 192 | owner = first(self.files_to_owners[file_name]) |
| 193 | self.select_owner(owner, False) |
| 194 | continues = True |
| 195 | break |
| 196 | |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 197 | def print_file_info(self, file_name, except_owner=''): |
| 198 | if file_name not in self.unreviewed_files: |
| 199 | self.writeln(self.greyed(file_name + |
| 200 | ' (by ' + |
| 201 | self.bold_name(self.reviewed_by[file_name]) + |
| 202 | ')')) |
| 203 | else: |
| 204 | if len(self.files_to_owners[file_name]) <= 3: |
| 205 | other_owners = [] |
| 206 | for ow in self.files_to_owners[file_name]: |
| 207 | if ow != except_owner: |
| 208 | other_owners.append(self.bold_name(ow)) |
| 209 | self.writeln(file_name + |
| 210 | ' [' + (', '.join(other_owners)) + ']') |
| 211 | else: |
| 212 | self.writeln(file_name + ' [' + |
| 213 | self.bold(str(len(self.files_to_owners[file_name]))) + |
| 214 | ']') |
| 215 | |
| 216 | def print_file_info_detailed(self, file_name): |
| 217 | self.writeln(file_name) |
| 218 | self.indent() |
| 219 | for ow in sorted(self.files_to_owners[file_name]): |
| 220 | if ow in self.deselected_owners: |
| 221 | self.writeln(self.bold_name(self.greyed(ow))) |
| 222 | elif ow in self.selected_owners: |
| 223 | self.writeln(self.bold_name(self.greyed(ow))) |
| 224 | else: |
| 225 | self.writeln(self.bold_name(ow)) |
| 226 | self.unindent() |
| 227 | |
| 228 | def print_owned_files_for(self, owner): |
| 229 | # Print owned files |
Edward Lesmes | c3c15a1 | 2021-01-19 20:19:14 +0000 | [diff] [blame] | 230 | self.writeln(self.bold_name(owner)) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 231 | self.writeln(self.bold_name(owner) + ' owns ' + |
| 232 | str(len(self.owners_to_files[owner])) + ' file(s):') |
| 233 | self.indent() |
| 234 | for file_name in sorted(self.owners_to_files[owner]): |
| 235 | self.print_file_info(file_name, owner) |
| 236 | self.unindent() |
| 237 | self.writeln() |
| 238 | |
| 239 | def list_owners(self, owners_queue): |
| 240 | if (len(self.owners_to_files) - len(self.deselected_owners) - |
| 241 | len(self.selected_owners)) > 3: |
| 242 | for ow in owners_queue: |
| 243 | if ow not in self.deselected_owners and ow not in self.selected_owners: |
Edward Lesmes | c3c15a1 | 2021-01-19 20:19:14 +0000 | [diff] [blame] | 244 | self.writeln(self.bold_name(ow)) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 245 | else: |
| 246 | for ow in owners_queue: |
| 247 | if ow not in self.deselected_owners and ow not in self.selected_owners: |
| 248 | self.writeln() |
| 249 | self.print_owned_files_for(ow) |
| 250 | |
| 251 | def list_files(self): |
| 252 | self.indent() |
| 253 | if len(self.unreviewed_files) > 5: |
| 254 | for file_name in sorted(self.unreviewed_files): |
| 255 | self.print_file_info(file_name) |
| 256 | else: |
| 257 | for file_name in self.unreviewed_files: |
| 258 | self.print_file_info_detailed(file_name) |
| 259 | self.unindent() |
| 260 | |
| 261 | def pick_owner(self, ow): |
| 262 | # Allowing to omit domain suffixes |
| 263 | if ow not in self.owners_to_files: |
| 264 | if ow + self.email_postfix in self.owners_to_files: |
| 265 | ow += self.email_postfix |
| 266 | |
| 267 | if ow not in self.owners_to_files: |
| 268 | self.writeln('You cannot pick ' + self.bold_name(ow) + ' manually. ' + |
| 269 | 'It\'s an invalid name or not related to the change list.') |
| 270 | return False |
| 271 | elif ow in self.selected_owners: |
| 272 | self.writeln('You cannot pick ' + self.bold_name(ow) + ' manually. ' + |
| 273 | 'It\'s already selected.') |
| 274 | return False |
| 275 | elif ow in self.deselected_owners: |
| 276 | self.writeln('You cannot pick ' + self.bold_name(ow) + ' manually.' + |
| 277 | 'It\'s already unselected.') |
| 278 | return False |
| 279 | |
| 280 | self.select_owner(ow) |
| 281 | return True |
| 282 | |
| 283 | def print_result(self): |
| 284 | # Print results |
| 285 | self.writeln() |
| 286 | self.writeln() |
Bruce Dawson | 9b4a057 | 2020-05-06 17:05:01 +0000 | [diff] [blame] | 287 | if len(self.selected_owners) == 0: |
| 288 | self.writeln('This change list already has owner-reviewers for all ' |
| 289 | 'files.') |
| 290 | self.writeln('Use --ignore-current if you want to ignore them.') |
| 291 | else: |
| 292 | self.writeln('** You selected these owners **') |
| 293 | self.writeln() |
| 294 | for owner in self.selected_owners: |
| 295 | self.writeln(self.bold_name(owner) + ':') |
| 296 | self.indent() |
| 297 | for file_name in sorted(self.owners_to_files[owner]): |
| 298 | self.writeln(file_name) |
| 299 | self.unindent() |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 300 | |
| 301 | def bold(self, text): |
| 302 | return self.COLOR_BOLD + text + self.COLOR_RESET |
| 303 | |
| 304 | def bold_name(self, name): |
| 305 | return (self.COLOR_BOLD + |
| 306 | name.replace(self.email_postfix, '') + self.COLOR_RESET) |
| 307 | |
| 308 | def greyed(self, text): |
| 309 | return self.COLOR_GREY + text + self.COLOR_RESET |
| 310 | |
| 311 | def indent(self): |
| 312 | self.indentation += 1 |
| 313 | |
| 314 | def unindent(self): |
| 315 | self.indentation -= 1 |
| 316 | |
| 317 | def print_indent(self): |
| 318 | return ' ' * self.indentation |
| 319 | |
| 320 | def writeln(self, text=''): |
Raul Tambre | 80ee78e | 2019-05-06 22:41:05 +0000 | [diff] [blame] | 321 | print(self.print_indent() + text) |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 322 | |
| 323 | def hr(self): |
| 324 | self.writeln('=====================') |
| 325 | |
| 326 | def print_info(self, owner): |
| 327 | self.hr() |
| 328 | self.writeln( |
| 329 | self.bold(str(len(self.unreviewed_files))) + ' file(s) left.') |
| 330 | self.print_owned_files_for(owner) |
| 331 | |
| 332 | def input_command(self, owner): |
| 333 | self.writeln('Add ' + self.bold_name(owner) + ' as your reviewer? ') |
Edward Lesmes | ae3586b | 2020-03-23 21:21:14 +0000 | [diff] [blame] | 334 | return gclient_utils.AskForData( |
ikarienator@chromium.org | faf3fdf | 2013-09-20 02:11:48 +0000 | [diff] [blame] | 335 | '[yes/no/Defer/pick/files/owners/quit/restart]: ').lower() |