diff --git a/README.md b/README.md index c8fef2e..6b0f0f1 100644 --- a/README.md +++ b/README.md @@ -32,10 +32,37 @@ To run gitHappens script anywhere in filesystem, make sure to create an alias. Add following line to your `.bashrc` or `.zshrc` file -`alias gh='python3 ~//gitHappens.py'` +`alias gh='python3 ~//main.py'` + +Existing aliases that point to `gitHappens.py` continue to work because that file now delegates to `main.py`. Run `source ~/.zshrc` or restart terminal. +## Project structure + +The CLI is split into small modules by responsibility: + +```text +main.py # Entry point and argument parsing +gitlab_api.py # GitLab API and glab command interactions +config.py # Configuration and template file loading +templates.py # Template lookup helpers +git_utils.py # Git command helpers and project detection +interactive.py # User prompts and CLI interactions +gitHappens.py # Backward-compatible launcher +commands/ + create_issue.py # Issue creation and incident report workflow + review.py # Review workflow and time tracking + deploy.py # Production deployment checks + open_mr.py # Merge request browser opening +``` + +Run the unit tests with: + +```bash +python3 -B -m unittest discover -s tests +``` + ## Usage āš” ### Project selection @@ -225,4 +252,3 @@ I suggest checking Gitlab's official API documentation: https://docs.gitlab.com/ ## Donating šŸ’œ Make sure to check this project on [OpenPledge](https://app.openpledge.io/repositories/zigcBenx/gitHappens). - diff --git a/commands/__init__.py b/commands/__init__.py new file mode 100644 index 0000000..112df76 --- /dev/null +++ b/commands/__init__.py @@ -0,0 +1 @@ +"""Command orchestration modules for GitHappens.""" diff --git a/commands/create_issue.py b/commands/create_issue.py new file mode 100644 index 0000000..c503fe1 --- /dev/null +++ b/commands/create_issue.py @@ -0,0 +1,136 @@ +import configparser + +import gitlab_api +import interactive +from config import CONFIG_PARSER, INCIDENT_PROJECT_ID + + +def get_selected_milestone(milestone, milestones): + return next((item for item in milestones if item["title"] == milestone), None) + + +def get_milestone(manual): + if manual: + milestones = gitlab_api.list_milestones() + return get_selected_milestone(interactive.select_milestone(milestones), milestones) + return gitlab_api.list_milestones(True) + + +def get_selected_iteration(iteration, iterations): + return next((item for item in iterations if f"{item['start_date']} - {item['due_date']}" == iteration), None) + + +def get_iteration(manual): + if manual: + iterations = gitlab_api.list_iterations() + return get_selected_iteration(interactive.select_iteration(iterations), iterations) + return gitlab_api.get_active_iteration() + + +def get_selected_epic(epic, epics): + return next((item for item in epics if item["title"] == epic), None) + + +def get_epic(): + epics = gitlab_api.list_epics() + return get_selected_epic(interactive.select_epic(epics), epics) + + +def create_issue(title, project_id, milestone_id, epic, iteration, settings): + if settings: + issue_type = settings.get("type") or "issue" + return gitlab_api.create_issue( + project_id, + title, + settings.get("labels"), + milestone_id, + epic, + iteration, + settings.get("weight"), + settings.get("estimated_time"), + issue_type, + ) + print("No settings in template") + exit(2) + + +def start_issue_creation(project_id, title, milestone, epic, iteration, selected_settings, only_issue, main_branch): + estimated_time = interactive.prompt_estimated_time() + + if isinstance(project_id, list): + estimated_time_per_project = int(estimated_time) / len(project_id) if estimated_time else None + else: + estimated_time_per_project = estimated_time + + if estimated_time_per_project: + selected_settings = selected_settings.copy() if selected_settings else {} + selected_settings["estimated_time"] = int(estimated_time_per_project) + + created_issue = create_issue(title, project_id, milestone, epic, iteration, selected_settings) + print(f"Issue #{created_issue['iid']}: {created_issue['title']} created.") + + if only_issue: + return created_issue + + created_branch = gitlab_api.create_branch(project_id, created_issue, main_branch) + created_merge_request = gitlab_api.create_merge_request( + project_id, + created_branch, + created_issue, + selected_settings.get("labels"), + milestone, + main_branch, + ) + print(f"Merge request #{created_merge_request['iid']}: {created_merge_request['title']} created.") + + print("Run:") + print(" git fetch origin") + print(f" git checkout -b '{created_merge_request['source_branch']}' 'origin/{created_merge_request['source_branch']}'") + print("to switch to new branch.") + + return created_issue + + +def select_labels(search, multiple=False): + labels = gitlab_api.get_labels_of_group(search) + return interactive.select_labels(labels, multiple) + + +def process_report(text, minutes): + try: + incident_project_id = INCIDENT_PROJECT_ID or CONFIG_PARSER.get("DEFAULT", "incident_project_id") + except (configparser.NoOptionError, configparser.NoSectionError): + print("Error: incident_project_id not found in config.ini") + print("Please add your incident project ID to configs/config.ini under [DEFAULT] section:") + print("incident_project_id = your_project_id_here") + return + + issue_title = f"Incident Report: {text}" + selected_label = select_labels("Department") + + incident_settings = { + "labels": ["incident", "report"], + "onlyIssue": True, + "type": "incident", + } + + if selected_label: + incident_settings["labels"].append(selected_label) + + try: + iteration = gitlab_api.get_active_iteration() + created_issue = create_issue(issue_title, incident_project_id, False, False, iteration, incident_settings) + issue_iid = created_issue["iid"] + + gitlab_api.close_issue(issue_iid, incident_project_id) + print(f"Incident issue #{issue_iid} created successfully.") + print(f"Title: {issue_title}") + + try: + gitlab_api.add_spent_time(issue_iid, incident_project_id, minutes) + print(f"Added {minutes} minutes to issue time tracking.") + except Exception as e: + print(f"Error adding time tracking: {str(e)}") + + except Exception as e: + print(f"Error creating incident issue: {str(e)}") diff --git a/commands/deploy.py b/commands/deploy.py new file mode 100644 index 0000000..a883e92 --- /dev/null +++ b/commands/deploy.py @@ -0,0 +1,94 @@ +import datetime + +import gitlab_api +import git_utils +from config import MAIN_BRANCH, PRODUCTION_MAPPINGS + + +def get_last_production_deploy(): + try: + project_id = git_utils.resolve_project_id() + params = { + "per_page": 50, + "order_by": "updated_at", + "sort": "desc", + } + + if MAIN_BRANCH: + params["ref"] = MAIN_BRANCH + else: + try: + params["ref"] = git_utils.get_main_branch() + except Exception: + params["ref"] = "main" + + pipelines = gitlab_api.list_pipelines(project_id, params) + if pipelines is None: + return + + production_pipeline = None + for pipeline in pipelines: + jobs = gitlab_api.list_pipeline_jobs(project_id, pipeline["id"]) + if jobs is None: + continue + + for job in jobs: + job_name = job.get("name", "") + stage = job.get("stage", "") + job_status = job.get("status", "").lower() + + if job_status != "success": + continue + + project_mapping = PRODUCTION_MAPPINGS.get(str(project_id)) + if project_mapping: + expected_stage = project_mapping.get("stage", "").lower() + expected_job = project_mapping.get("job", "").lower() + + if stage.lower() == expected_stage or (expected_job and job_name.lower() == expected_job): + production_pipeline = { + "pipeline": pipeline, + "production_job": job, + } + break + else: + print("Didn't find deployment pipeline") + + if production_pipeline: + break + + if not production_pipeline: + print("No production deployment found matching pattern") + return + + pipeline = production_pipeline["pipeline"] + job = production_pipeline["production_job"] + + print("Last Production Deployment:") + print(f" Pipeline: #{pipeline['id']} - {pipeline['status']}") + print(f" Job: {job['name']} ({job['status']})") + print(f" Branch/Tag: {pipeline['ref']}") + print(f" Started: {job.get('started_at', 'N/A')}") + print(f" Finished: {job.get('finished_at', 'N/A')}") + print(f" Duration: {job.get('duration', 'N/A')} seconds" if job.get("duration") else " Duration: N/A") + print(f" Commit: {pipeline['sha'][:8]}") + print(f" URL: {pipeline['web_url']}") + + if job.get("finished_at"): + try: + finished_time = datetime.datetime.fromisoformat(job["finished_at"].replace("Z", "+00:00")) + time_diff = datetime.datetime.now(datetime.timezone.utc) - finished_time + + if time_diff.days > 0: + print(f" {time_diff.days} days ago") + elif time_diff.seconds > 3600: + hours = time_diff.seconds // 3600 + print(f" {hours} hours ago") + else: + minutes = time_diff.seconds // 60 + print(f" {minutes} minutes ago") + except Exception: + pass + + except Exception as e: + print(f"Error fetching last production deploy: {str(e)}") diff --git a/commands/open_mr.py b/commands/open_mr.py new file mode 100644 index 0000000..5d6f1db --- /dev/null +++ b/commands/open_mr.py @@ -0,0 +1,15 @@ +import subprocess +import webbrowser + +from config import BASE_URL +from commands.review import get_active_merge_request_id + + +def open_merge_request_in_browser(): + try: + merge_request_id = get_active_merge_request_id() + remote_url = subprocess.check_output(["git", "config", "--get", "remote.origin.url"], text=True).strip() + url = BASE_URL + "/" + remote_url.split(":")[1][:-4] + webbrowser.open(f"{url}/-/merge_requests/{merge_request_id}") + except subprocess.CalledProcessError: + return None diff --git a/commands/review.py b/commands/review.py new file mode 100644 index 0000000..b6106f7 --- /dev/null +++ b/commands/review.py @@ -0,0 +1,75 @@ +import gitlab_api +import git_utils +import interactive + + +def get_active_merge_request_id(): + branch_to_find = git_utils.get_current_branch() + return find_merge_request_id_by_branch(branch_to_find) + + +def find_merge_request_id_by_branch(branch_name): + return get_merge_request_for_branch(branch_name)["iid"] + + +def get_merge_request_for_branch(branch_name): + project_id = git_utils.resolve_project_id() + return gitlab_api.get_merge_request_for_branch(project_id, branch_name) + + +def get_current_issue_id(): + merge_request = get_merge_request_for_branch(git_utils.get_current_branch()) + return merge_request["description"].replace('"', "").replace("#", "").split()[1] + + +def track_issue_time(): + try: + project_id = git_utils.resolve_project_id() + issue_id = get_current_issue_id() + except Exception as e: + print(f"Error getting issue details: {str(e)}") + return + + spent_time = interactive.prompt_spent_time() + + try: + gitlab_api.add_issue_time_note(project_id, issue_id, spent_time) + print(f"Added {spent_time} minutes to issue {issue_id} time tracking.") + except Exception as e: + print(f"Error tracking issue time: {str(e)}") + + +def choose_reviewers_manually(): + return interactive.choose_reviewers_manually(get_user=gitlab_api.get_user) + + +def add_reviewers_to_merge_request(reviewers=None): + project_id = git_utils.resolve_project_id() + mr_id = get_active_merge_request_id() + gitlab_api.add_reviewers_to_merge_request(project_id, mr_id, reviewers) + + +def set_merge_request_to_auto_merge(): + project_id = git_utils.resolve_project_id() + mr_id = get_active_merge_request_id() + gitlab_api.set_merge_request_to_auto_merge(project_id, mr_id) + + +def run_review_command(select=False, auto_merge=False): + track_issue_time() + reviewers = choose_reviewers_manually() if select else None + add_reviewers_to_merge_request(reviewers=reviewers) + + try: + from ai_code_review import run_review_for_mr + + project_id = git_utils.resolve_project_id() + mr_id = get_active_merge_request_id() + from config import API_URL, GITLAB_TOKEN + + run_review_for_mr(project_id, mr_id, GITLAB_TOKEN, API_URL) + except Exception as e: + print(f"AI review skipped: {e}") + + if auto_merge: + set_merge_request_to_auto_merge() diff --git a/config.py b/config.py new file mode 100644 index 0000000..206ff37 --- /dev/null +++ b/config.py @@ -0,0 +1,113 @@ +from __future__ import annotations + +import configparser +import json +from dataclasses import dataclass +from pathlib import Path + + +ROOT_DIR = Path(__file__).resolve().parent +CONFIG_DIR = ROOT_DIR / "configs" +CONFIG_PATH = CONFIG_DIR / "config.ini" +TEMPLATES_PATH = CONFIG_DIR / "templates.json" + + +@dataclass(frozen=True) +class AppConfig: + base_url: str = "https://gitlab.com" + group_id: str = "" + custom_template: str = "Custom" + gitlab_token: str = "" + delete_branch_after_merge: bool = True + developer_email: str | None = None + squash_commits: bool = True + production_pipeline_name: str = "deploy" + production_job_name: str | None = None + production_ref: str | None = None + incident_project_id: str | None = None + openai_api_key: str | None = None + + @property + def api_url(self) -> str: + return f"{self.base_url}/api/v4" + + +@dataclass(frozen=True) +class TemplateConfig: + templates: list + reviewers: list + production_mappings: dict + + +def _clean_optional(value: str | None) -> str | None: + if value is None: + return None + value = value.strip().strip("\"'") + return value or None + + +def _get_bool(parser: configparser.ConfigParser, option: str, fallback: bool) -> bool: + return parser.get("DEFAULT", option, fallback=str(fallback)).lower() == "true" + + +def load_config_parser(config_path: Path | str = CONFIG_PATH) -> configparser.ConfigParser: + parser = configparser.ConfigParser() + parser.read(config_path) + return parser + + +def load_app_config(config_path: Path | str = CONFIG_PATH) -> AppConfig: + parser = load_config_parser(config_path) + return AppConfig( + base_url=parser.get("DEFAULT", "base_url", fallback="https://gitlab.com"), + group_id=parser.get("DEFAULT", "group_id", fallback=""), + custom_template=parser.get("DEFAULT", "custom_template", fallback="Custom"), + gitlab_token=_clean_optional(parser.get("DEFAULT", "GITLAB_TOKEN", fallback="")) or "", + delete_branch_after_merge=_get_bool(parser, "delete_branch_after_merge", True), + developer_email=_clean_optional(parser.get("DEFAULT", "developer_email", fallback=None)), + squash_commits=_get_bool(parser, "squash_commits", True), + production_pipeline_name=parser.get("DEFAULT", "production_pipeline_name", fallback="deploy"), + production_job_name=_clean_optional(parser.get("DEFAULT", "production_job_name", fallback=None)), + production_ref=_clean_optional(parser.get("DEFAULT", "production_ref", fallback=None)), + incident_project_id=_clean_optional(parser.get("DEFAULT", "incident_project_id", fallback=None)), + openai_api_key=_clean_optional(parser.get("DEFAULT", "OPENAI_API_KEY", fallback=None)), + ) + + +def load_template_config(templates_path: Path | str = TEMPLATES_PATH) -> TemplateConfig: + path = Path(templates_path) + if not path.exists(): + return TemplateConfig(templates=[], reviewers=[], production_mappings={}) + + with path.open("r", encoding="utf-8") as f: + json_config = json.load(f) + + return TemplateConfig( + templates=json_config.get("templates", []), + reviewers=json_config.get("reviewers", []), + production_mappings=json_config.get("productionMappings", {}), + ) + + +APP_CONFIG = load_app_config() +TEMPLATE_CONFIG = load_template_config() +CONFIG_PARSER = load_config_parser() + +BASE_URL = APP_CONFIG.base_url +API_URL = APP_CONFIG.api_url +GROUP_ID = APP_CONFIG.group_id +CUSTOM_TEMPLATE = APP_CONFIG.custom_template +GITLAB_TOKEN = APP_CONFIG.gitlab_token +DELETE_BRANCH = APP_CONFIG.delete_branch_after_merge +DEVELOPER_EMAIL = APP_CONFIG.developer_email +SQUASH_COMMITS = APP_CONFIG.squash_commits +PRODUCTION_PIPELINE_NAME = APP_CONFIG.production_pipeline_name +PRODUCTION_JOB_NAME = APP_CONFIG.production_job_name +PRODUCTION_REF = APP_CONFIG.production_ref +INCIDENT_PROJECT_ID = APP_CONFIG.incident_project_id +OPENAI_API_KEY = APP_CONFIG.openai_api_key +MAIN_BRANCH = "master" + +TEMPLATES = TEMPLATE_CONFIG.templates +REVIEWERS = TEMPLATE_CONFIG.reviewers +PRODUCTION_MAPPINGS = TEMPLATE_CONFIG.production_mappings diff --git a/gitHappens.py b/gitHappens.py index 27d47f3..32bd001 100755 --- a/gitHappens.py +++ b/gitHappens.py @@ -1,845 +1,6 @@ #!/usr/bin/env python3 -import subprocess -import json -import argparse -import configparser -import inquirer -import datetime -import re -import os -import requests -import sys -import webbrowser +from main import main -# Setup config parser and read settings -config = configparser.ConfigParser() -absolute_config_path = os.path.dirname(os.path.abspath(__file__)) -config_path = os.path.join(absolute_config_path, 'configs/config.ini') -config.read(config_path) -BASE_URL = config.get('DEFAULT', 'base_url') -API_URL = BASE_URL + '/api/v4' -GROUP_ID = config.get('DEFAULT', 'group_id') -CUSTOM_TEMPLATE = config.get('DEFAULT', 'custom_template') -GITLAB_TOKEN = config.get('DEFAULT', 'GITLAB_TOKEN').strip('\"\'') -DELETE_BRANCH = config.get('DEFAULT', 'delete_branch_after_merge').lower() == 'true' -DEVELOPER_EMAIL = config.get('DEFAULT', 'developer_email', fallback=None) -SQUASH_COMMITS = config.get('DEFAULT', 'squash_commits').lower() == 'true' -PRODUCTION_PIPELINE_NAME = config.get('DEFAULT', 'production_pipeline_name', fallback='deploy') -PRODUCTION_JOB_NAME = config.get('DEFAULT', 'production_job_name', fallback=None) -PRODUCTION_REF = config.get('DEFAULT', 'production_ref', fallback=None) -MAIN_BRANCH = 'master' - -# Read templates from json config -with open(os.path.join(absolute_config_path,'configs/templates.json'), 'r') as f: - jsonConfig = json.load(f) -TEMPLATES = jsonConfig['templates'] -REVIEWERS = jsonConfig['reviewers'] -PRODUCTION_MAPPINGS = jsonConfig.get('productionMappings', {}) - -def get_project_id(): - project_link = getProjectLinkFromCurrentDir() - if (project_link == -1): - return enterProjectId() - - allProjects = get_all_projects(project_link) - # Find projects id by project ssh link gathered from repo - matching_id = None - for project in allProjects: - if project.get("ssh_url_to_repo") == project_link: - matching_id = project.get("id") - break - return matching_id - -def get_all_projects(project_link): - url = API_URL + "/projects?membership=true&search=" + project_link.split('/')[-1].split('.')[0] - - headers = { - "PRIVATE-TOKEN": GITLAB_TOKEN - } - - response = requests.get(url, headers=headers) - - if response.status_code == 200: - return response.json() - elif response.status_code == 401: - print("Error: Unauthorized (401). Your GitLab token is probably expired, invalid, or missing required permissions.") - print("Please generate a new token and update your configs/config.ini.") - exit(1) - else: - print(f"Request failed with status code {response.status_code}") - return None - -def getProjectLinkFromCurrentDir(): - try: - cmd = 'git remote get-url origin' - result = subprocess.run(cmd.split(), stdout=subprocess.PIPE, stderr=subprocess.PIPE) - if result.returncode == 0: - output = result.stdout.decode('utf-8').strip() - return output - else: - return -1 - except FileNotFoundError: - return -1 - -def enterProjectId(): - while True: - project_id = input('Please enter the ID of your GitLab project: ') - if project_id: - return project_id - exit('Invalid project ID.') - -def list_milestones(current=False): - cmd = f'glab api /groups/{GROUP_ID}/milestones?state=active' - result = subprocess.run(cmd.split(), stdout=subprocess.PIPE) - milestones = json.loads(result.stdout) - if current: - today = datetime.date.today().strftime('%Y-%m-%d') - active_milestones = [] - for milestone in milestones: - start_date = milestone['start_date'] - due_date = milestone['due_date'] - if start_date and due_date and start_date <= today and due_date >= today: - active_milestones.append(milestone) - active_milestones.sort(key=lambda x: x['due_date']) - return active_milestones[0] - return milestones - -def select_template(): - template_names = [t['name'] for t in TEMPLATES] - template_names.append(CUSTOM_TEMPLATE) - questions = [ - inquirer.List('template', - message="Select template:", - choices=template_names, - ), - ] - answer = inquirer.prompt(questions) - return answer['template'] - -def getIssueSettings(template_name): - if template_name == CUSTOM_TEMPLATE: - return {} - return next((t for t in TEMPLATES if t['name'] == template_name), None) - -def createIssue(title, project_id, milestoneId, epic, iteration, settings): - if settings: - issueType = settings.get('type') or 'issue' - return executeIssueCreate(project_id, title, settings.get('labels'), milestoneId, epic, iteration, settings.get('weight'), settings.get('estimated_time'), issueType) - print("No settings in template") - exit(2) - pass - -def executeIssueCreate(project_id, title, labels, milestoneId, epic, iteration, weight, estimated_time, issue_type='issue'): - labels = ",".join(labels) if type(labels) == list else labels - assignee_id = getAuthorizedUser()['id'] - issue_command = [ - "glab", "api", - f"/projects/{str(project_id)}/issues", - "-f", f'title={title}', - "-f", f'assignee_ids={assignee_id}', - "-f", f'issue_type={issue_type}' - ] - if labels: - issue_command.append("-f") - issue_command.append(f'labels={labels}') - - if weight: - issue_command.append("-f") - issue_command.append(f'weight={str(weight)}') - - if milestoneId: - issue_command.append("-f") - issue_command.append(f'milestone_id={str(milestoneId)}') - - if epic: - epicId = epic['id'] - issue_command.append("-f") - issue_command.append(f'epic_id={str(epicId)}') - - # Set the description, including iteration, estimated time, and other info - description = "" - if iteration: - iterationId = iteration['id'] - description += f"/iteration *iteration:{str(iterationId)} " - - if estimated_time: - description += f"\n/estimate {estimated_time}m " - - issue_command.extend(["-f", f'description={description}']) - - issue_output = subprocess.check_output(issue_command) - return json.loads(issue_output.decode()) - -def select_milestone(milestones): - milestones = [t['title'] for t in milestones] - questions = [ - inquirer.List('milestones', - message="Select milestone:", - choices=milestones, - ), - ] - answer = inquirer.prompt(questions) - return answer['milestones'] - -def getSelectedMilestone(milestone, milestones): - return next((t for t in milestones if t['title'] == milestone), None) - -def get_milestone(manual): - if manual: - milestones = list_milestones() - return getSelectedMilestone(select_milestone(milestones), milestones) - milestone = list_milestones(True) # select active for today - return milestone - -def get_iteration(manual): - if manual: - iterations = list_iterations() - return getSelectedIteration(select_iteration(iterations), iterations) - return getActiveIteration() - -def getSelectedIteration(iteration, iterations): - return next((t for t in iterations if t['start_date'] + ' - ' + t['due_date'] == iteration), None) - -def select_iteration(iterations): - iterations = [t['start_date'] + ' - ' + t['due_date'] for t in iterations] - questions = [ - inquirer.List('iterations', - message="Select iteration:", - choices=iterations, - ), - ] - answer = inquirer.prompt(questions) - return answer['iterations'] - -def list_iterations(): - cmd = f'glab api /groups/{GROUP_ID}/iterations?state=opened' - result = subprocess.run(cmd.split(), stdout=subprocess.PIPE) - iterations = json.loads(result.stdout) - return iterations - -def getActiveIteration(): - iterations = list_iterations() - today = datetime.date.today().strftime('%Y-%m-%d') - active_iterations = [] - for iteration in iterations: - start_date = iteration['start_date'] - due_date = iteration['due_date'] - if start_date and due_date and start_date <= today and due_date >= today: - active_iterations.append(iteration) - active_iterations.sort(key=lambda x: x['due_date']) - return active_iterations[0] - -def getAuthorizedUser(): - output = subprocess.check_output(["glab", "api", "/user"]) - return json.loads(output) - -def list_epics(): - cmd = f'glab api /groups/{GROUP_ID}/epics?per_page=1000&state=opened' - result = subprocess.run(cmd.split(), stdout=subprocess.PIPE) - return json.loads(result.stdout) - -def select_epic(epics): - epics = [t['title'] for t in epics] - search_query = inquirer.prompt([ - inquirer.Text('search_query', message='Search epic:'), - ])['search_query'] - - # Filter choices based on search query - filtered_epics = [c for c in epics if search_query.lower() in c.lower()] - questions = [ - inquirer.List('epics', - message="Select epic:", - choices=filtered_epics, - ), - ] - answer = inquirer.prompt(questions) - return answer['epics'] - -def getSelectedEpic(epic, epics): - return next((t for t in epics if t['title'] == epic), None) - -def get_epic(): - epics = list_epics() - return getSelectedEpic(select_epic(epics), epics) - -def create_branch(project_id, issue): - issueId = str(issue['iid']) - title = re.sub('\\s+', '-', issue['title']).lower() - title = issueId + '-' + title.replace(':','').replace('(',' ').replace(')', '').replace(' ','-') - branch_output = subprocess.check_output(["glab", "api", f"/projects/{str(project_id)}/repository/branches", "-f", f'branch={title}', "-f", f'ref={MAIN_BRANCH}', "-f", f'issue_iid={issueId}']) - return json.loads(branch_output.decode()) - -def create_merge_request(project_id, branch, issue, labels, milestoneId): - issueId = str(issue['iid']) - branch = branch['name'] - title = issue['title'] - assignee_id = getAuthorizedUser()['id'] - labels = ",".join(labels) if type(labels) == list else labels - merge_request_command = [ - "glab", "api", - f"/projects/{str(project_id)}/merge_requests", - "-f", f'title={title}', - "-f", f'description="Closes #{issueId}"', - "-f", f'source_branch={branch}', - "-f", f'target_branch={MAIN_BRANCH}', - "-f", f'issue_iid={issueId}', - "-f", f'assignee_ids={assignee_id}' - ] - - if SQUASH_COMMITS: - merge_request_command.append("-f") - merge_request_command.append("squash=true") - - if DELETE_BRANCH: - merge_request_command.append("-f") - merge_request_command.append("remove_source_branch=true") - - if labels: - merge_request_command.append("-f") - merge_request_command.append(f'labels={labels}') - - if milestoneId: - merge_request_command.append("-f") - merge_request_command.append(f'milestone_id={str(milestoneId)}') - - mr_output = subprocess.check_output(merge_request_command) - return json.loads(mr_output.decode()) - -def startIssueCreation(project_id, title, milestone, epic, iteration, selectedSettings, onlyIssue): - # Prompt for estimated time - estimated_time = inquirer.prompt([ - inquirer.Text('estimated_time', - message='Estimated time to complete this issue (in minutes, optional)', - validate=lambda _, x: x == '' or x.isdigit()) - ])['estimated_time'] - - # If multiple project IDs, split the estimated time - if isinstance(project_id, list): - estimated_time_per_project = int(estimated_time) / len(project_id) if estimated_time else None - else: - estimated_time_per_project = estimated_time - - # Modify settings to include estimated time - if estimated_time_per_project: - selectedSettings = selectedSettings.copy() if selectedSettings else {} - selectedSettings['estimated_time'] = int(estimated_time_per_project) - - createdIssue = createIssue(title, project_id, milestone, epic, iteration, selectedSettings) - print(f"Issue #{createdIssue['iid']}: {createdIssue['title']} created.") - - if onlyIssue: - return createdIssue - - createdBranch = create_branch(project_id, createdIssue) - - createdMergeRequest = create_merge_request(project_id, createdBranch, createdIssue, selectedSettings.get('labels'), milestone) - print(f"Merge request #{createdMergeRequest['iid']}: {createdMergeRequest['title']} created.") - - print("Run:") - print(" git fetch origin") - print(f" git checkout -b '{createdMergeRequest['source_branch']}' 'origin/{createdMergeRequest['source_branch']}'") - print("to switch to new branch.") - - return createdIssue - -def getCurrentBranch(): - return subprocess.check_output(['git', 'rev-parse', '--abbrev-ref', 'HEAD'], text=True).strip() - -def openMergeRequestInBrowser(): - try: - merge_request_id = getActiveMergeRequestId() - remote_url = subprocess.check_output(["git", "config", "--get", "remote.origin.url"], text=True).strip() - url = BASE_URL + '/' + remote_url.split(':')[1][:-4] - webbrowser.open(f"{url}/-/merge_requests/{merge_request_id}") - except subprocess.CalledProcessError: - return None - -def getActiveMergeRequestId(): - branch_to_find = getCurrentBranch() - return find_merge_request_id_by_branch(branch_to_find) - -def find_merge_request_id_by_branch(branch_name): - return getMergeRequestForBranch(branch_name)['iid'] - -def getMergeRequestForBranch(branchName): - project_id = get_project_id() - api_url = f"{API_URL}/projects/{project_id}/merge_requests" - headers = {"Private-Token": GITLAB_TOKEN} - - params = { - "source_branch": branchName, - } - - response = requests.get(api_url, headers=headers, params=params) - if response.status_code == 200: - merge_requests = response.json() - for mr in merge_requests: - if mr["source_branch"] == branchName: - return mr - else: - print(f"Failed to fetch Merge Requests: {response.status_code} - {response.text}") - return None - -def chooseReviewersManually(): - """Prompt the user to select reviewers manually from the available list, showing names.""" - # Fetch user details for each reviewer ID - reviewer_choices = [] - for reviewer_id in REVIEWERS: - api_url = f"{API_URL}/users/{reviewer_id}" - headers = {"Private-Token": GITLAB_TOKEN} - try: - response = requests.get(api_url, headers=headers) - if response.status_code == 200: - user = response.json() - display_name = f"{user.get('name')} ({user.get('username')})" - reviewer_choices.append((display_name, reviewer_id)) - else: - reviewer_choices.append((str(reviewer_id), reviewer_id)) - except Exception: - reviewer_choices.append((str(reviewer_id), reviewer_id)) - - questions = [ - inquirer.Checkbox( - "selected_reviewers", - message="Select reviewers", - choices=[(name, str(rid)) for name, rid in reviewer_choices], - ) - ] - answers = inquirer.prompt(questions) - if answers and "selected_reviewers" in answers: - return [int(r) for r in answers["selected_reviewers"]] - else: - return [] - -def addReviewersToMergeRequest(reviewers=None): - project_id = get_project_id() - mr_id = getActiveMergeRequestId() - api_url = f"{API_URL}/projects/{project_id}/merge_requests/{mr_id}" - headers = {"Private-Token": GITLAB_TOKEN} - - data = { - "reviewer_ids": reviewers if reviewers is not None else REVIEWERS - } - - requests.put(api_url, headers=headers, json=data) - -def setMergeRequestToAutoMerge(): - project_id = get_project_id() - mr_id = getActiveMergeRequestId() - api_url = f"{API_URL}/projects/{project_id}/merge_requests/{mr_id}/merge" - headers = {"Private-Token": GITLAB_TOKEN} - - data = { - "id": project_id, - "merge_request_iid": mr_id, - "should_remove_source_branch": True, - "merge_when_pipeline_succeeds": True, - "auto_merge_strategy": "merge_when_pipeline_succeeds", - } - - requests.put(api_url, headers=headers, json=data) - -def getMainBranch(): - command = "git symbolic-ref refs/remotes/origin/HEAD | sed 's@^refs/remotes/origin/@@'" - output = subprocess.check_output(command, shell=True, stderr=subprocess.STDOUT, universal_newlines=True) - return output.strip() - - -def get_two_weeks_commits(return_output=False): - two_weeks_ago = (datetime.datetime.now() - datetime.timedelta(weeks=2)).strftime('%Y-%m-%d') - - cmd = f'git log --since={two_weeks_ago} --format="%ad - %ae - %s" --date=short | grep -v "Merge branch"' - if (DEVELOPER_EMAIL): - cmd = f'{cmd} | grep {DEVELOPER_EMAIL}' - try: - output = subprocess.check_output(cmd, shell=True, text=True, stderr=subprocess.DEVNULL, universal_newlines=True).strip() - if output: - if return_output: - return output - print(output) - else: - print("No commits found.") - return "" if return_output else None - except subprocess.CalledProcessError as e: - print(f"No commits were found or an error occurred. (exit status {e.returncode})") - return "" if return_output else None - except FileNotFoundError: - print("Git is not installed or not found in PATH.") - return "" if return_output else None - -def generate_smart_summary(): - commits = get_two_weeks_commits(return_output=True) - if not commits: - return - - # Check if OpenAI API key is set - openai_api_key = config.get('DEFAULT', 'OPENAI_API_KEY', fallback=None) - if not openai_api_key: - print("OpenAI API key not set. Skipping AI summary generation.") - return - - # Dynamically import openai only if API key is present - try: - import openai - except ImportError: - print("OpenAI package not installed. Please install it using: pip install openai") - return - - openai.api_key = openai_api_key - - try: - response = openai.chat.completions.create( - model="gpt-3.5-turbo", - messages=[ - {"role": "system", "content": "You are a helpful assistant that summarizes git commits. Provide a concise, well-organized summary of the main changes and themes."}, - {"role": "user", "content": f"Please summarize these git commits in a clear, bulleted format:\n\n{commits}"} - ] - ) - - print("\nšŸ“‹ AI-Generated Summary of Recent Changes:\n") - print(response.choices[0].message.content) - except Exception as e: - print(f"Error generating AI summary: {e}") - -def process_report(text, minutes): - # Get the incident project ID from config - try: - incident_project_id = config.get('DEFAULT', 'incident_project_id') - except (configparser.NoOptionError, configparser.NoSectionError): - print("Error: incident_project_id not found in config.ini") - print("Please add your incident project ID to configs/config.ini under [DEFAULT] section:") - print("incident_project_id = your_project_id_here") - return - - issue_title = f"Incident Report: {text}" - - selected_label = selectLabels('Department') - - incident_settings = { - 'labels': ['incident', 'report'], - 'onlyIssue': True, - 'type': 'incident' - } - - if selected_label: - incident_settings['labels'].append(selected_label) - - try: - # Create the incident issue - iteration = getActiveIteration() - created_issue = createIssue(issue_title, incident_project_id, False, False, iteration, incident_settings) - issue_iid = created_issue['iid'] - - closeOpenedIssue(issue_iid, incident_project_id) - print(f"Incident issue #{issue_iid} created successfully.") - print(f"Title: {issue_title}") - - # Add time tracking to the issue - time_tracking_command = [ - "glab", "api", - f"/projects/{incident_project_id}/issues/{issue_iid}/add_spent_time", - "-f", f"duration={minutes}m" - ] - - try: - subprocess.run(time_tracking_command, stdout=subprocess.PIPE, stderr=subprocess.PIPE) - print(f"Added {minutes} minutes to issue time tracking.") - except subprocess.CalledProcessError as e: - print(f"Error adding time tracking: {str(e)}") - - except Exception as e: - print(f"Error creating incident issue: {str(e)}") - -def closeOpenedIssue(issue_iid, project_id): - issue_command = [ - "glab", "api", - f"/projects/{project_id}/issues/{issue_iid}", - '-X', 'PUT', - '-f', 'state_event=close' - ] - try: - subprocess.run(issue_command, stdout=subprocess.PIPE, stderr=subprocess.PIPE) - except subprocess.CalledProcessError as e: - print(f"Error closing issue: {str(e)}") - -def selectLabels(search, multiple = False): - labels = getLabelsOfGroup(search) - labels = sorted([t['name'] for t in labels]) - - question_type = inquirer.Checkbox if multiple else inquirer.List - questions = [ - question_type( - 'labels', - message="Select one or more department labels:", - choices=labels, - ), - ] - answer = inquirer.prompt(questions) - return answer['labels'] - -def getLabelsOfGroup(search=''): - cmd = f'glab api /groups/{GROUP_ID}/labels?search={search}' - try: - result = subprocess.run(cmd.split(), stdout=subprocess.PIPE, check=True) - return json.loads(result.stdout) - except subprocess.CalledProcessError as e: - print(f"Error getting labels: {str(e)}") - return [] - -def getCurrentIssueId(): - mr = getMergeRequestForBranch(getCurrentBranch()) - return mr['description'].replace('"','').replace('#','').split()[1] - -def track_issue_time(): - # Get the current merge request - try: - project_id = get_project_id() - issue_id = getCurrentIssueId() - except Exception as e: - print(f"Error getting issue details: {str(e)}") - return - - # Prompt for actual time spent - spent_time = inquirer.prompt([ - inquirer.Text('spent_time', - message='How many minutes did you actually spend on this issue?', - validate=lambda _, x: x.isdigit()) - ])['spent_time'] - - # Add spent time to the issue description - time_tracking_command = [ - "glab", "api", - f"/projects/{project_id}/issues/{issue_id}/notes", - "-f", f"body=/spend {spent_time}m" - ] - - try: - subprocess.run(time_tracking_command, stdout=subprocess.PIPE, stderr=subprocess.PIPE, check=True) - print(f"Added {spent_time} minutes to issue {issue_id} time tracking.") - except subprocess.CalledProcessError as e: - print(f"Error adding time tracking: {str(e)}") - except Exception as e: - print(f"Error tracking issue time: {str(e)}") - -def get_last_production_deploy(): - try: - project_id = get_project_id() - api_url = f"{API_URL}/projects/{project_id}/pipelines" - headers = {"Private-Token": GITLAB_TOKEN} - - # Set up parameters for the pipeline search - params = { - "per_page": 50, - "order_by": "updated_at", - "sort": "desc" - } - - # Add ref filter if specified in config - if MAIN_BRANCH: - params["ref"] = MAIN_BRANCH - else: - # Use main branch if no specific ref is configured - try: - main_branch = getMainBranch() - params["ref"] = main_branch - except: - # Fallback to common main branch names - params["ref"] = "main" - - response = requests.get(api_url, headers=headers, params=params) - - if response.status_code != 200: - print(f"Failed to fetch pipelines: {response.status_code} - {response.text}") - return - - pipelines = response.json() - production_pipeline = None - - # Look for production pipeline by name pattern - for pipeline in pipelines: - # Get pipeline details to check jobs - pipeline_detail_url = f"{API_URL}/projects/{project_id}/pipelines/{pipeline['id']}/jobs" - detail_response = requests.get(pipeline_detail_url, headers=headers) - - if detail_response.status_code == 200: - jobs = detail_response.json() - - # Check if this pipeline contains production deployment - for job in jobs: - job_name = job.get('name', '') - stage = job.get('stage', '') - job_status = job.get('status', '').lower() - - # Only consider successful jobs - if job_status != 'success': - continue - - # Check project-specific mapping first - project_mapping = PRODUCTION_MAPPINGS.get(str(project_id)) - if project_mapping: - expected_stage = project_mapping.get('stage', '').lower() - expected_job = project_mapping.get('job', '').lower() - - if (stage.lower() == expected_stage or - (expected_job and job_name.lower() == expected_job)): - production_pipeline = { - 'pipeline': pipeline, - 'production_job': job - } - break - else: - print('Didn\'t find deployment pipeline') - - if production_pipeline: - break - - if not production_pipeline: - print(f"No production deployment found matching pattern") - return - - # Display the results - pipeline = production_pipeline['pipeline'] - job = production_pipeline['production_job'] - - print(f"šŸš€ Last Production Deployment:") - print(f" Pipeline: #{pipeline['id']} - {pipeline['status']}") - print(f" Job: {job['name']} ({job['status']})") - print(f" Branch/Tag: {pipeline['ref']}") - print(f" Started: {job.get('started_at', 'N/A')}") - print(f" Finished: {job.get('finished_at', 'N/A')}") - print(f" Duration: {job.get('duration', 'N/A')} seconds" if job.get('duration') else " Duration: N/A") - print(f" Commit: {pipeline['sha'][:8]}") - print(f" URL: {pipeline['web_url']}") - - # Show time since deployment - if job.get('finished_at'): - try: - finished_time = datetime.datetime.fromisoformat(job['finished_at'].replace('Z', '+00:00')) - time_diff = datetime.datetime.now(datetime.timezone.utc) - finished_time - - if time_diff.days > 0: - print(f" ā° {time_diff.days} days ago") - elif time_diff.seconds > 3600: - hours = time_diff.seconds // 3600 - print(f" ā° {hours} hours ago") - else: - minutes = time_diff.seconds // 60 - print(f" ā° {minutes} minutes ago") - except: - pass - - except Exception as e: - print(f"Error fetching last production deploy: {str(e)}") - -def main(): - global MAIN_BRANCH - - parser = argparse.ArgumentParser("Argument description of Git happens") - parser.add_argument("title", nargs="+", help="Title of issue") - parser.add_argument(f"--project_id", type=str, help="Id or URL-encoded path of project") - parser.add_argument("-m", "--milestone", action='store_true', help="Add this flag, if you want to manually select milestone") - parser.add_argument("--no_epic", action="store_true", help="Add this flag if you don't want to pick epic") - parser.add_argument("--no_milestone", action="store_true", help="Add this flag if you don't want to pick milestone") - parser.add_argument("--no_iteration", action="store_true", help="Add this flag if you don't want to pick iteration") - parser.add_argument("--only_issue", action="store_true", help="Add this flag if you don't want to create merge request and branch alongside issue") - parser.add_argument("-am", "--auto_merge", action="store_true", help="Add this flag to review if you want to set merge request to auto merge when pipeline succeeds") - parser.add_argument("--select", action="store_true", help="Manually select reviewers for merge request (interactive)") - - # If no arguments passed, show help - if len(sys.argv) <= 1: - parser.print_help() - exit(1) - - args = parser.parse_args() - if args.title[0] == 'report': - parts = args.title - if len(parts) != 3: - print("Invalid report format. Use: gh report \"text\" minutes") - return - - text = parts[1] - try: - minutes = int(parts[2].strip()) - process_report(text, minutes) - except ValueError: - print("Invalid minutes. Please provide a valid number.") - return - - # So it takes all text until first known argument - title = " ".join(args.title) - - if title == 'open': - openMergeRequestInBrowser() - return - elif title == 'review': - track_issue_time() - reviewers = None - if getattr(args, "select", False): - reviewers = chooseReviewersManually() - addReviewersToMergeRequest(reviewers=reviewers) - - # Run AI code review and post to MR - try: - from ai_code_review import run_review_for_mr - project_id = get_project_id() - mr_id = getActiveMergeRequestId() - run_review_for_mr(project_id, mr_id, GITLAB_TOKEN, API_URL) - except Exception as e: - print(f"AI review skipped: {e}") - - if(args.auto_merge): - setMergeRequestToAutoMerge() - return - elif title == 'summary': - get_two_weeks_commits() - return - elif title == 'summaryAI': - generate_smart_summary() - return - elif title == 'last deploy': - get_last_production_deploy() - return - elif title == 'ai review': - from ai_code_review import run_review - run_review() - return - - # Get settings for issue from template - selectedSettings = getIssueSettings(select_template()) - - # If template is False, ask for each settings - if not len(selectedSettings): - print('Custom selection of issue settings is not supported yet') - pass - - if args.project_id and selectedSettings.get('projectIds'): - print('NOTE: Overwriting project id from argument...') - - project_id = selectedSettings.get('projectIds') or args.project_id or get_project_id() - - milestone = False - if not args.no_milestone: - milestone = get_milestone(args.milestone)['id'] - - iteration = False - if not args.no_iteration: - # manual pick iteration - iteration = get_iteration(True) - - epic = False - if not args.no_epic: - epic = get_epic() - - MAIN_BRANCH = getMainBranch() - - onlyIssue = selectedSettings.get('onlyIssue') or args.only_issue - - if type(project_id) == list: - for id in project_id: - startIssueCreation(id, title, milestone, epic, iteration, selectedSettings, onlyIssue) - else: - startIssueCreation(project_id, title, milestone, epic, iteration, selectedSettings, onlyIssue) - -if __name__ == '__main__': - main() \ No newline at end of file +if __name__ == "__main__": + main() diff --git a/git_utils.py b/git_utils.py new file mode 100644 index 0000000..85a2d5b --- /dev/null +++ b/git_utils.py @@ -0,0 +1,74 @@ +import datetime +import subprocess + +from config import DEVELOPER_EMAIL + + +def get_project_link_from_current_dir(): + try: + result = subprocess.run( + ["git", "remote", "get-url", "origin"], + stdout=subprocess.PIPE, + stderr=subprocess.PIPE, + ) + if result.returncode == 0: + return result.stdout.decode("utf-8").strip() + return -1 + except FileNotFoundError: + return -1 + + +def resolve_project_id(): + from gitlab_api import get_all_projects + from interactive import enter_project_id + + project_link = get_project_link_from_current_dir() + if project_link == -1: + return enter_project_id() + + all_projects = get_all_projects(project_link) + matching_id = None + for project in all_projects: + if project.get("ssh_url_to_repo") == project_link: + matching_id = project.get("id") + break + return matching_id + + +def get_current_branch(): + return subprocess.check_output(["git", "rev-parse", "--abbrev-ref", "HEAD"], text=True).strip() + + +def get_main_branch(): + command = "git symbolic-ref refs/remotes/origin/HEAD | sed 's@^refs/remotes/origin/@@'" + output = subprocess.check_output(command, shell=True, stderr=subprocess.STDOUT, universal_newlines=True) + return output.strip() + + +def get_two_weeks_commits(return_output=False, developer_email=DEVELOPER_EMAIL): + two_weeks_ago = (datetime.datetime.now() - datetime.timedelta(weeks=2)).strftime("%Y-%m-%d") + + cmd = f'git log --since={two_weeks_ago} --format="%ad - %ae - %s" --date=short | grep -v "Merge branch"' + if developer_email: + cmd = f"{cmd} | grep {developer_email}" + try: + output = subprocess.check_output( + cmd, + shell=True, + text=True, + stderr=subprocess.DEVNULL, + universal_newlines=True, + ).strip() + if output: + if return_output: + return output + print(output) + else: + print("No commits found.") + return "" if return_output else None + except subprocess.CalledProcessError as e: + print(f"No commits were found or an error occurred. (exit status {e.returncode})") + return "" if return_output else None + except FileNotFoundError: + print("Git is not installed or not found in PATH.") + return "" if return_output else None diff --git a/gitlab_api.py b/gitlab_api.py new file mode 100644 index 0000000..b64edf4 --- /dev/null +++ b/gitlab_api.py @@ -0,0 +1,265 @@ +import datetime +import json +import re +import subprocess + +import requests + +from config import ( + API_URL, + DELETE_BRANCH, + GITLAB_TOKEN, + GROUP_ID, + MAIN_BRANCH, + REVIEWERS, + SQUASH_COMMITS, +) + + +def private_token_headers(): + return {"Private-Token": GITLAB_TOKEN} + + +def get_all_projects(project_link): + url = f"{API_URL}/projects?membership=true&search={project_link.split('/')[-1].split('.')[0]}" + response = requests.get(url, headers={"PRIVATE-TOKEN": GITLAB_TOKEN}) + + if response.status_code == 200: + return response.json() + if response.status_code == 401: + print("Error: Unauthorized (401). Your GitLab token is probably expired, invalid, or missing required permissions.") + print("Please generate a new token and update your configs/config.ini.") + exit(1) + + print(f"Request failed with status code {response.status_code}") + return None + + +def list_milestones(current=False): + result = subprocess.run( + ["glab", "api", f"/groups/{GROUP_ID}/milestones?state=active"], + stdout=subprocess.PIPE, + ) + milestones = json.loads(result.stdout) + if current: + today = datetime.date.today().strftime("%Y-%m-%d") + active_milestones = [] + for milestone in milestones: + start_date = milestone["start_date"] + due_date = milestone["due_date"] + if start_date and due_date and start_date <= today and due_date >= today: + active_milestones.append(milestone) + active_milestones.sort(key=lambda milestone: milestone["due_date"]) + return active_milestones[0] + return milestones + + +def list_iterations(): + result = subprocess.run( + ["glab", "api", f"/groups/{GROUP_ID}/iterations?state=opened"], + stdout=subprocess.PIPE, + ) + return json.loads(result.stdout) + + +def get_active_iteration(): + iterations = list_iterations() + today = datetime.date.today().strftime("%Y-%m-%d") + active_iterations = [] + for iteration in iterations: + start_date = iteration["start_date"] + due_date = iteration["due_date"] + if start_date and due_date and start_date <= today and due_date >= today: + active_iterations.append(iteration) + active_iterations.sort(key=lambda iteration: iteration["due_date"]) + return active_iterations[0] + + +def get_authorized_user(): + output = subprocess.check_output(["glab", "api", "/user"]) + return json.loads(output) + + +def list_epics(): + result = subprocess.run( + ["glab", "api", f"/groups/{GROUP_ID}/epics?per_page=1000&state=opened"], + stdout=subprocess.PIPE, + ) + return json.loads(result.stdout) + + +def create_issue(project_id, title, labels, milestone_id, epic, iteration, weight, estimated_time, issue_type="issue"): + labels = ",".join(labels) if type(labels) == list else labels + assignee_id = get_authorized_user()["id"] + issue_command = [ + "glab", "api", + f"/projects/{str(project_id)}/issues", + "-f", f"title={title}", + "-f", f"assignee_ids={assignee_id}", + "-f", f"issue_type={issue_type}", + ] + if labels: + issue_command.extend(["-f", f"labels={labels}"]) + + if weight: + issue_command.extend(["-f", f"weight={str(weight)}"]) + + if milestone_id: + issue_command.extend(["-f", f"milestone_id={str(milestone_id)}"]) + + if epic: + issue_command.extend(["-f", f"epic_id={str(epic['id'])}"]) + + description = "" + if iteration: + description += f"/iteration *iteration:{str(iteration['id'])} " + + if estimated_time: + description += f"\n/estimate {estimated_time}m " + + issue_command.extend(["-f", f"description={description}"]) + issue_output = subprocess.check_output(issue_command) + return json.loads(issue_output.decode()) + + +def create_branch(project_id, issue, main_branch=MAIN_BRANCH): + issue_id = str(issue["iid"]) + title = re.sub("\\s+", "-", issue["title"]).lower() + title = issue_id + "-" + title.replace(":", "").replace("(", " ").replace(")", "").replace(" ", "-") + branch_output = subprocess.check_output([ + "glab", "api", + f"/projects/{str(project_id)}/repository/branches", + "-f", f"branch={title}", + "-f", f"ref={main_branch}", + "-f", f"issue_iid={issue_id}", + ]) + return json.loads(branch_output.decode()) + + +def create_merge_request(project_id, branch, issue, labels, milestone_id, main_branch=MAIN_BRANCH): + issue_id = str(issue["iid"]) + branch_name = branch["name"] + assignee_id = get_authorized_user()["id"] + labels = ",".join(labels) if type(labels) == list else labels + merge_request_command = [ + "glab", "api", + f"/projects/{str(project_id)}/merge_requests", + "-f", f"title={issue['title']}", + "-f", f"description=\"Closes #{issue_id}\"", + "-f", f"source_branch={branch_name}", + "-f", f"target_branch={main_branch}", + "-f", f"issue_iid={issue_id}", + "-f", f"assignee_ids={assignee_id}", + ] + + if SQUASH_COMMITS: + merge_request_command.extend(["-f", "squash=true"]) + + if DELETE_BRANCH: + merge_request_command.extend(["-f", "remove_source_branch=true"]) + + if labels: + merge_request_command.extend(["-f", f"labels={labels}"]) + + if milestone_id: + merge_request_command.extend(["-f", f"milestone_id={str(milestone_id)}"]) + + mr_output = subprocess.check_output(merge_request_command) + return json.loads(mr_output.decode()) + + +def get_merge_request_for_branch(project_id, branch_name): + api_url = f"{API_URL}/projects/{project_id}/merge_requests" + response = requests.get(api_url, headers=private_token_headers(), params={"source_branch": branch_name}) + if response.status_code == 200: + merge_requests = response.json() + for merge_request in merge_requests: + if merge_request["source_branch"] == branch_name: + return merge_request + else: + print(f"Failed to fetch Merge Requests: {response.status_code} - {response.text}") + return None + + +def get_user(reviewer_id): + response = requests.get(f"{API_URL}/users/{reviewer_id}", headers=private_token_headers()) + if response.status_code == 200: + return response.json() + return None + + +def add_reviewers_to_merge_request(project_id, mr_id, reviewers=None): + api_url = f"{API_URL}/projects/{project_id}/merge_requests/{mr_id}" + data = {"reviewer_ids": reviewers if reviewers is not None else REVIEWERS} + requests.put(api_url, headers=private_token_headers(), json=data) + + +def set_merge_request_to_auto_merge(project_id, mr_id): + api_url = f"{API_URL}/projects/{project_id}/merge_requests/{mr_id}/merge" + data = { + "id": project_id, + "merge_request_iid": mr_id, + "should_remove_source_branch": True, + "merge_when_pipeline_succeeds": True, + "auto_merge_strategy": "merge_when_pipeline_succeeds", + } + requests.put(api_url, headers=private_token_headers(), json=data) + + +def close_issue(issue_iid, project_id): + issue_command = [ + "glab", "api", + f"/projects/{project_id}/issues/{issue_iid}", + "-X", "PUT", + "-f", "state_event=close", + ] + try: + subprocess.run(issue_command, stdout=subprocess.PIPE, stderr=subprocess.PIPE) + except subprocess.CalledProcessError as e: + print(f"Error closing issue: {str(e)}") + + +def add_spent_time(issue_iid, project_id, minutes): + time_tracking_command = [ + "glab", "api", + f"/projects/{project_id}/issues/{issue_iid}/add_spent_time", + "-f", f"duration={minutes}m", + ] + subprocess.run(time_tracking_command, stdout=subprocess.PIPE, stderr=subprocess.PIPE) + + +def add_issue_time_note(project_id, issue_id, spent_time): + time_tracking_command = [ + "glab", "api", + f"/projects/{project_id}/issues/{issue_id}/notes", + "-f", f"body=/spend {spent_time}m", + ] + subprocess.run(time_tracking_command, stdout=subprocess.PIPE, stderr=subprocess.PIPE, check=True) + + +def get_labels_of_group(search=""): + try: + result = subprocess.run( + ["glab", "api", f"/groups/{GROUP_ID}/labels?search={search}"], + stdout=subprocess.PIPE, + check=True, + ) + return json.loads(result.stdout) + except subprocess.CalledProcessError as e: + print(f"Error getting labels: {str(e)}") + return [] + + +def list_pipelines(project_id, params): + response = requests.get(f"{API_URL}/projects/{project_id}/pipelines", headers=private_token_headers(), params=params) + if response.status_code != 200: + print(f"Failed to fetch pipelines: {response.status_code} - {response.text}") + return None + return response.json() + + +def list_pipeline_jobs(project_id, pipeline_id): + response = requests.get(f"{API_URL}/projects/{project_id}/pipelines/{pipeline_id}/jobs", headers=private_token_headers()) + if response.status_code == 200: + return response.json() + return None diff --git a/interactive.py b/interactive.py new file mode 100644 index 0000000..d2293d1 --- /dev/null +++ b/interactive.py @@ -0,0 +1,150 @@ +try: + import inquirer +except ImportError: + inquirer = None + +from config import CUSTOM_TEMPLATE, REVIEWERS +from templates import template_names + + +def _inquirer(): + if inquirer is None: + raise RuntimeError("The inquirer package is required for interactive prompts. Install it with: pip install inquirer") + return inquirer + + +def enter_project_id(): + while True: + project_id = input("Please enter the ID of your GitLab project: ") + if project_id: + return project_id + exit("Invalid project ID.") + + +def select_template(): + prompt = _inquirer() + questions = [ + prompt.List( + "template", + message="Select template:", + choices=template_names(custom_template=CUSTOM_TEMPLATE), + ), + ] + answer = prompt.prompt(questions) + return answer["template"] + + +def select_milestone(milestones): + prompt = _inquirer() + choices = [milestone["title"] for milestone in milestones] + questions = [ + prompt.List( + "milestones", + message="Select milestone:", + choices=choices, + ), + ] + answer = prompt.prompt(questions) + return answer["milestones"] + + +def select_iteration(iterations): + prompt = _inquirer() + choices = [f"{iteration['start_date']} - {iteration['due_date']}" for iteration in iterations] + questions = [ + prompt.List( + "iterations", + message="Select iteration:", + choices=choices, + ), + ] + answer = prompt.prompt(questions) + return answer["iterations"] + + +def select_epic(epics): + prompt = _inquirer() + choices = [epic["title"] for epic in epics] + search_query = prompt.prompt([ + prompt.Text("search_query", message="Search epic:"), + ])["search_query"] + + filtered_epics = [choice for choice in choices if search_query.lower() in choice.lower()] + questions = [ + prompt.List( + "epics", + message="Select epic:", + choices=filtered_epics, + ), + ] + answer = prompt.prompt(questions) + return answer["epics"] + + +def prompt_estimated_time(): + prompt = _inquirer() + return prompt.prompt([ + prompt.Text( + "estimated_time", + message="Estimated time to complete this issue (in minutes, optional)", + validate=lambda _, value: value == "" or value.isdigit(), + ) + ])["estimated_time"] + + +def prompt_spent_time(): + prompt = _inquirer() + return prompt.prompt([ + prompt.Text( + "spent_time", + message="How many minutes did you actually spend on this issue?", + validate=lambda _, value: value.isdigit(), + ) + ])["spent_time"] + + +def select_labels(labels, multiple=False): + prompt = _inquirer() + labels = sorted([label["name"] for label in labels]) + question_type = prompt.Checkbox if multiple else prompt.List + questions = [ + question_type( + "labels", + message="Select one or more department labels:", + choices=labels, + ), + ] + answer = prompt.prompt(questions) + return answer["labels"] + + +def choose_reviewers_manually(reviewers=None, get_user=None): + prompt = _inquirer() + reviewers = REVIEWERS if reviewers is None else reviewers + reviewer_choices = [] + for reviewer_id in reviewers: + if get_user is None: + reviewer_choices.append((str(reviewer_id), reviewer_id)) + continue + + try: + user = get_user(reviewer_id) + if user: + display_name = f"{user.get('name')} ({user.get('username')})" + reviewer_choices.append((display_name, reviewer_id)) + else: + reviewer_choices.append((str(reviewer_id), reviewer_id)) + except Exception: + reviewer_choices.append((str(reviewer_id), reviewer_id)) + + questions = [ + prompt.Checkbox( + "selected_reviewers", + message="Select reviewers", + choices=[(name, str(reviewer_id)) for name, reviewer_id in reviewer_choices], + ) + ] + answers = prompt.prompt(questions) + if answers and "selected_reviewers" in answers: + return [int(reviewer_id) for reviewer_id in answers["selected_reviewers"]] + return [] diff --git a/main.py b/main.py new file mode 100755 index 0000000..0eccee5 --- /dev/null +++ b/main.py @@ -0,0 +1,143 @@ +#!/usr/bin/env python3 +import argparse +import sys + +import git_utils +import interactive +from commands.create_issue import get_epic, get_iteration, get_milestone, process_report, start_issue_creation +from commands.deploy import get_last_production_deploy +from commands.open_mr import open_merge_request_in_browser +from commands.review import run_review_command +from config import OPENAI_API_KEY +from templates import get_issue_settings + + +def generate_smart_summary(): + commits = git_utils.get_two_weeks_commits(return_output=True) + if not commits: + return + + if not OPENAI_API_KEY: + print("OpenAI API key not set. Skipping AI summary generation.") + return + + try: + import openai + except ImportError: + print("OpenAI package not installed. Please install it using: pip install openai") + return + + openai.api_key = OPENAI_API_KEY + + try: + response = openai.chat.completions.create( + model="gpt-3.5-turbo", + messages=[ + { + "role": "system", + "content": "You are a helpful assistant that summarizes git commits. Provide a concise, well-organized summary of the main changes and themes.", + }, + {"role": "user", "content": f"Please summarize these git commits in a clear, bulleted format:\n\n{commits}"}, + ], + ) + + print("\nAI-Generated Summary of Recent Changes:\n") + print(response.choices[0].message.content) + except Exception as e: + print(f"Error generating AI summary: {e}") + + +def build_parser(): + parser = argparse.ArgumentParser("Argument description of Git happens") + parser.add_argument("title", nargs="+", help="Title of issue") + parser.add_argument("--project_id", type=str, help="Id or URL-encoded path of project") + parser.add_argument("-m", "--milestone", action="store_true", help="Add this flag, if you want to manually select milestone") + parser.add_argument("--no_epic", action="store_true", help="Add this flag if you don't want to pick epic") + parser.add_argument("--no_milestone", action="store_true", help="Add this flag if you don't want to pick milestone") + parser.add_argument("--no_iteration", action="store_true", help="Add this flag if you don't want to pick iteration") + parser.add_argument("--only_issue", action="store_true", help="Add this flag if you don't want to create merge request and branch alongside issue") + parser.add_argument("-am", "--auto_merge", action="store_true", help="Add this flag to review if you want to set merge request to auto merge when pipeline succeeds") + parser.add_argument("--select", action="store_true", help="Manually select reviewers for merge request (interactive)") + return parser + + +def main(argv=None): + argv = sys.argv[1:] if argv is None else argv + parser = build_parser() + + if len(argv) <= 0: + parser.print_help() + exit(1) + + args = parser.parse_args(argv) + if args.title[0] == "report": + parts = args.title + if len(parts) != 3: + print('Invalid report format. Use: gh report "text" minutes') + return + + text = parts[1] + try: + minutes = int(parts[2].strip()) + process_report(text, minutes) + except ValueError: + print("Invalid minutes. Please provide a valid number.") + return + + title = " ".join(args.title) + + if title == "open": + open_merge_request_in_browser() + return + if title == "review": + run_review_command(select=getattr(args, "select", False), auto_merge=args.auto_merge) + return + if title == "summary": + git_utils.get_two_weeks_commits() + return + if title == "summaryAI": + generate_smart_summary() + return + if title == "last deploy": + get_last_production_deploy() + return + if title == "ai review": + from ai_code_review import run_review + + run_review() + return + + selected_settings = get_issue_settings(interactive.select_template()) + + if not len(selected_settings): + print("Custom selection of issue settings is not supported yet") + + if args.project_id and selected_settings.get("projectIds"): + print("NOTE: Overwriting project id from argument...") + + project_id = selected_settings.get("projectIds") or args.project_id or git_utils.resolve_project_id() + + milestone = False + if not args.no_milestone: + milestone = get_milestone(args.milestone)["id"] + + iteration = False + if not args.no_iteration: + iteration = get_iteration(True) + + epic = False + if not args.no_epic: + epic = get_epic() + + main_branch = git_utils.get_main_branch() + only_issue = selected_settings.get("onlyIssue") or args.only_issue + + if type(project_id) == list: + for project in project_id: + start_issue_creation(project, title, milestone, epic, iteration, selected_settings, only_issue, main_branch) + else: + start_issue_creation(project_id, title, milestone, epic, iteration, selected_settings, only_issue, main_branch) + + +if __name__ == "__main__": + main() diff --git a/templates.py b/templates.py new file mode 100644 index 0000000..02457a4 --- /dev/null +++ b/templates.py @@ -0,0 +1,17 @@ +from config import CUSTOM_TEMPLATE, TEMPLATES + + +def template_names(templates=None, custom_template=None): + templates = TEMPLATES if templates is None else templates + custom_template = CUSTOM_TEMPLATE if custom_template is None else custom_template + names = [template["name"] for template in templates] + names.append(custom_template) + return names + + +def get_issue_settings(template_name, templates=None, custom_template=None): + templates = TEMPLATES if templates is None else templates + custom_template = CUSTOM_TEMPLATE if custom_template is None else custom_template + if template_name == custom_template: + return {} + return next((template for template in templates if template["name"] == template_name), None) diff --git a/tests/__init__.py b/tests/__init__.py new file mode 100644 index 0000000..2d05ae9 --- /dev/null +++ b/tests/__init__.py @@ -0,0 +1 @@ +"""Unit tests for the modular GitHappens CLI.""" diff --git a/tests/test_commands.py b/tests/test_commands.py new file mode 100644 index 0000000..2a6c4af --- /dev/null +++ b/tests/test_commands.py @@ -0,0 +1,72 @@ +import unittest +from unittest.mock import patch + +from commands import create_issue, deploy, open_mr, review + + +class CommandTests(unittest.TestCase): + def test_start_issue_creation_splits_estimated_time_for_multiple_projects(self): + settings = {"labels": ["Bug"]} + created_issue = {"iid": 1, "title": "Fix bug"} + + with patch("commands.create_issue.interactive.prompt_estimated_time", return_value="60"), \ + patch("commands.create_issue.create_issue", return_value=created_issue) as create_issue_call, \ + patch("builtins.print"): + result = create_issue.start_issue_creation( + project_id=[123, 456], + title="Fix bug", + milestone=False, + epic=False, + iteration=False, + selected_settings=settings, + only_issue=True, + main_branch="main", + ) + + self.assertEqual(result, created_issue) + self.assertEqual(create_issue_call.call_args.args[5]["estimated_time"], 30) + + def test_get_current_issue_id_reads_closes_description(self): + with patch("commands.review.git_utils.get_current_branch", return_value="1-fix"), \ + patch("commands.review.get_merge_request_for_branch", return_value={"description": '"Closes #42"'}): + self.assertEqual(review.get_current_issue_id(), "42") + + def test_deploy_finds_project_mapping_job(self): + pipeline = { + "id": 1, + "status": "success", + "ref": "master", + "sha": "abcdef123", + "web_url": "https://gitlab.example/pipelines/1", + } + job = { + "name": "production:deploy", + "stage": "deploy", + "status": "success", + "started_at": "2026-05-12T09:00:00Z", + "finished_at": "2026-05-12T09:05:00Z", + "duration": 300, + } + + with patch("commands.deploy.git_utils.resolve_project_id", return_value="9418557"), \ + patch.dict("commands.deploy.PRODUCTION_MAPPINGS", {"9418557": {"stage": "deploy", "job": "production:deploy"}}, clear=True), \ + patch("commands.deploy.gitlab_api.list_pipelines", return_value=[pipeline]), \ + patch("commands.deploy.gitlab_api.list_pipeline_jobs", return_value=[job]), \ + patch("builtins.print") as print_call: + deploy.get_last_production_deploy() + + printed = "\n".join(str(call.args[0]) for call in print_call.call_args_list if call.args) + self.assertIn("Last Production Deployment:", printed) + self.assertIn("production:deploy", printed) + + def test_open_merge_request_in_browser_builds_gitlab_url(self): + with patch("commands.open_mr.get_active_merge_request_id", return_value=12), \ + patch("commands.open_mr.subprocess.check_output", return_value="git@gitlab.com:group/project.git\n"), \ + patch("commands.open_mr.webbrowser.open") as open_browser: + open_mr.open_merge_request_in_browser() + + self.assertTrue(open_browser.call_args.args[0].endswith("/group/project/-/merge_requests/12")) + + +if __name__ == "__main__": + unittest.main() diff --git a/tests/test_config.py b/tests/test_config.py new file mode 100644 index 0000000..57589b9 --- /dev/null +++ b/tests/test_config.py @@ -0,0 +1,68 @@ +import json +import tempfile +import unittest +from pathlib import Path + +from config import load_app_config, load_template_config + + +class ConfigTests(unittest.TestCase): + def test_load_app_config_reads_and_cleans_values(self): + with tempfile.TemporaryDirectory() as temp_dir: + config_path = Path(temp_dir) / "config.ini" + config_path.write_text( + "\n".join([ + "[DEFAULT]", + "base_url=https://example.gitlab.com", + "group_id=42", + "custom_template=Custom template", + "GITLAB_TOKEN='secret'", + "delete_branch_after_merge=false", + "developer_email=\"dev@example.com\"", + "squash_commits=false", + "production_pipeline_name=deploy-prod", + "production_job_name=ship", + "production_ref=main", + "incident_project_id=123", + "OPENAI_API_KEY=key", + ]), + encoding="utf-8", + ) + + app_config = load_app_config(config_path) + + self.assertEqual(app_config.base_url, "https://example.gitlab.com") + self.assertEqual(app_config.api_url, "https://example.gitlab.com/api/v4") + self.assertEqual(app_config.group_id, "42") + self.assertEqual(app_config.gitlab_token, "secret") + self.assertFalse(app_config.delete_branch_after_merge) + self.assertEqual(app_config.developer_email, "dev@example.com") + + def test_load_template_config_defaults_when_missing(self): + template_config = load_template_config("/tmp/does-not-exist-githappens.json") + + self.assertEqual(template_config.templates, []) + self.assertEqual(template_config.reviewers, []) + self.assertEqual(template_config.production_mappings, {}) + + def test_load_template_config_reads_json(self): + with tempfile.TemporaryDirectory() as temp_dir: + templates_path = Path(temp_dir) / "templates.json" + templates_path.write_text( + json.dumps({ + "templates": [{"name": "Bug"}], + "reviewers": [1, 2], + "productionMappings": {"123": {"stage": "deploy"}}, + }), + encoding="utf-8", + ) + + template_config = load_template_config(templates_path) + + self.assertEqual(template_config.templates, [{"name": "Bug"}]) + self.assertEqual(template_config.reviewers, [1, 2]) + self.assertEqual(template_config.production_mappings["123"]["stage"], "deploy") + + +if __name__ == "__main__": + unittest.main() diff --git a/tests/test_git_utils.py b/tests/test_git_utils.py new file mode 100644 index 0000000..338d635 --- /dev/null +++ b/tests/test_git_utils.py @@ -0,0 +1,36 @@ +import subprocess +import unittest +from unittest.mock import Mock, patch + +import git_utils + + +class GitUtilsTests(unittest.TestCase): + def test_get_project_link_from_current_dir_returns_remote(self): + result = Mock(returncode=0, stdout=b"git@gitlab.com:group/project.git\n") + + with patch("git_utils.subprocess.run", return_value=result): + self.assertEqual(git_utils.get_project_link_from_current_dir(), "git@gitlab.com:group/project.git") + + def test_get_project_link_from_current_dir_returns_minus_one_without_git_remote(self): + result = Mock(returncode=1, stdout=b"") + + with patch("git_utils.subprocess.run", return_value=result): + self.assertEqual(git_utils.get_project_link_from_current_dir(), -1) + + def test_get_two_weeks_commits_returns_output(self): + with patch("git_utils.subprocess.check_output", return_value="2026-05-12 - dev@example.com - Add tests\n"): + output = git_utils.get_two_weeks_commits(return_output=True, developer_email=None) + + self.assertIn("Add tests", output) + + def test_get_two_weeks_commits_returns_empty_on_git_error(self): + with patch("git_utils.subprocess.check_output", side_effect=subprocess.CalledProcessError(1, "git")), \ + patch("builtins.print"): + output = git_utils.get_two_weeks_commits(return_output=True, developer_email=None) + + self.assertEqual(output, "") + + +if __name__ == "__main__": + unittest.main() diff --git a/tests/test_gitlab_api.py b/tests/test_gitlab_api.py new file mode 100644 index 0000000..7877dbf --- /dev/null +++ b/tests/test_gitlab_api.py @@ -0,0 +1,64 @@ +import json +import unittest +from unittest.mock import Mock, patch + +import gitlab_api + + +class GitLabApiTests(unittest.TestCase): + def test_get_all_projects_returns_json_for_success(self): + response = Mock(status_code=200) + response.json.return_value = [{"id": 1}] + + with patch("gitlab_api.requests.get", return_value=response) as get: + projects = gitlab_api.get_all_projects("git@gitlab.com:group/project.git") + + self.assertEqual(projects, [{"id": 1}]) + self.assertIn("search=project", get.call_args.args[0]) + + def test_create_issue_builds_glab_command(self): + issue_payload = {"iid": 7, "title": "Fix bug"} + + with patch("gitlab_api.get_authorized_user", return_value={"id": 99}), \ + patch("gitlab_api.subprocess.check_output", return_value=json.dumps(issue_payload).encode()) as check_output: + created = gitlab_api.create_issue( + project_id=123, + title="Fix bug", + labels=["Bug", "P::1"], + milestone_id=5, + epic={"id": 8}, + iteration={"id": 13}, + weight=3, + estimated_time=45, + ) + + command = check_output.call_args.args[0] + self.assertEqual(created, issue_payload) + self.assertIn("/projects/123/issues", command) + self.assertIn("labels=Bug,P::1", command) + self.assertIn("milestone_id=5", command) + self.assertIn("description=/iteration *iteration:13 \n/estimate 45m ", command) + + def test_create_merge_request_includes_branch_options(self): + mr_payload = {"iid": 11, "title": "Fix bug", "source_branch": "7-fix-bug"} + + with patch("gitlab_api.get_authorized_user", return_value={"id": 99}), \ + patch("gitlab_api.subprocess.check_output", return_value=json.dumps(mr_payload).encode()) as check_output: + created = gitlab_api.create_merge_request( + project_id=123, + branch={"name": "7-fix-bug"}, + issue={"iid": 7, "title": "Fix bug"}, + labels=["Bug"], + milestone_id=5, + main_branch="main", + ) + + command = check_output.call_args.args[0] + self.assertEqual(created, mr_payload) + self.assertIn("target_branch=main", command) + self.assertIn("remove_source_branch=true", command) + self.assertIn("squash=true", command) + + +if __name__ == "__main__": + unittest.main() diff --git a/tests/test_interactive.py b/tests/test_interactive.py new file mode 100644 index 0000000..a25d65f --- /dev/null +++ b/tests/test_interactive.py @@ -0,0 +1,45 @@ +import unittest +from unittest.mock import patch + +import interactive + + +class FakePrompt: + class List: + def __init__(self, *args, **kwargs): + self.args = args + self.kwargs = kwargs + + class Text: + def __init__(self, *args, **kwargs): + self.args = args + self.kwargs = kwargs + + class Checkbox: + def __init__(self, *args, **kwargs): + self.args = args + self.kwargs = kwargs + + @staticmethod + def prompt(_questions): + return {"selected_reviewers": ["1", "2"]} + + +class InteractiveTests(unittest.TestCase): + def test_choose_reviewers_manually_uses_user_display_names(self): + users = { + 1: {"name": "Ada Lovelace", "username": "ada"}, + 2: {"name": "Grace Hopper", "username": "grace"}, + } + + with patch.object(interactive, "inquirer", FakePrompt): + selected = interactive.choose_reviewers_manually( + reviewers=[1, 2], + get_user=lambda reviewer_id: users[reviewer_id], + ) + + self.assertEqual(selected, [1, 2]) + + +if __name__ == "__main__": + unittest.main() diff --git a/tests/test_main.py b/tests/test_main.py new file mode 100644 index 0000000..30bd249 --- /dev/null +++ b/tests/test_main.py @@ -0,0 +1,22 @@ +import unittest +from unittest.mock import patch + +import main + + +class MainTests(unittest.TestCase): + def test_parser_accepts_review_select(self): + args = main.build_parser().parse_args(["review", "--select"]) + + self.assertEqual(args.title, ["review"]) + self.assertTrue(args.select) + + def test_main_routes_summary_without_prompts(self): + with patch("main.git_utils.get_two_weeks_commits") as commits: + main.main(["summary"]) + + commits.assert_called_once_with() + + +if __name__ == "__main__": + unittest.main() diff --git a/tests/test_templates.py b/tests/test_templates.py new file mode 100644 index 0000000..5f4caba --- /dev/null +++ b/tests/test_templates.py @@ -0,0 +1,21 @@ +import unittest + +from templates import get_issue_settings, template_names + + +class TemplateTests(unittest.TestCase): + def test_template_names_appends_custom_template(self): + names = template_names([{"name": "Bug"}, {"name": "Feature"}], "Custom") + + self.assertEqual(names, ["Bug", "Feature", "Custom"]) + + def test_get_issue_settings_returns_template_or_custom_settings(self): + templates = [{"name": "Bug", "labels": ["Bug"]}] + + self.assertEqual(get_issue_settings("Bug", templates, "Custom"), templates[0]) + self.assertEqual(get_issue_settings("Custom", templates, "Custom"), {}) + self.assertIsNone(get_issue_settings("Missing", templates, "Custom")) + + +if __name__ == "__main__": + unittest.main()