File commands.py has 1066 lines of code (exceeds 250 allowed). Consider refactoring.

Open

from __future__ import annotations

import copy
import logging
import subprocess

File onyo.py has 574 lines of code (exceeds 250 allowed). Consider refactoring.

Open

from __future__ import annotations

import logging
import shutil
import subprocess

File inventory.py has 540 lines of code (exceeds 250 allowed). Consider refactoring.

Open

from __future__ import annotations

from dataclasses import dataclass
from functools import partial
from pathlib import Path

File main.py has 421 lines of code (exceeds 250 allowed). Consider refactoring.

Open

from __future__ import annotations

import os
import re
import sys

Function is_equal_assets_dict has a Cognitive Complexity of 39 (exceeds 5 allowed). Consider refactoring.

Open

def is_equal_assets_dict(a: Dict | UserDict, b: Dict | UserDict) -> bool:
    r"""Whether two asset dictionaries have the same content.

    This accounts for comments in YAML.
    For this to return `True`, both assets need to

• Read up
  Read up

Function _edit_asset has a Cognitive Complexity of 35 (exceeds 5 allowed). Consider refactoring.

Open

def _edit_asset(inventory: Inventory,
                asset: dict | UserDict,
                operation: Callable,
                editor: str | None) -> dict | UserDict:
    r"""Edit `asset` via configured editor and a temporary asset file.

• Read up
  Read up

File git.py has 328 lines of code (exceeds 250 allowed). Consider refactoring.

Open

from __future__ import annotations

import logging
import subprocess
from pathlib import Path

Function main has a Cognitive Complexity of 26 (exceeds 5 allowed). Consider refactoring.

Open

def main() -> None:
    r"""Execute Onyo's CLI.
    """
    #
    # ARGPARSE Hack #1

• Read up
  Read up

File utils.py has 297 lines of code (exceeds 250 allowed). Consider refactoring.

Open

from __future__ import annotations

import copy
from collections import UserDict
from io import StringIO

Function onyo_get has a Cognitive Complexity of 21 (exceeds 5 allowed). Consider refactoring.

Open

def onyo_get(inventory: Inventory,
             include: list[Path] | None = None,
             exclude: list[Path] | Path | None = None,
             depth: int = 0,
             machine_readable: bool = False,

• Read up
  Read up

Function setup_parser has 69 lines of code (exceeds 25 allowed). Consider refactoring.

Open

def setup_parser() -> OnyoArgumentParser:
    r"""Setup and return a fully populated OnyoArgumentParser for Onyo and all subcommands.
    """
    from onyo.onyo_arguments import args_onyo
    from onyo.cli.cat import args_cat, epilog_cat

Function __getitem__ has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.

Open

    def __getitem__(self, key):
        if isinstance(key, str):
            parts = key.split('.')
            effective_dict = self.data
            if len(parts) > 1:

• Read up
  Read up

OnyoRepo has 21 functions (exceeds 20 allowed). Consider refactoring.

Open

class OnyoRepo(object):
    r"""
    An object representing an Onyo repository.

    Allows identifying and working with asset paths and directories, getting and

Function new has a Cognitive Complexity of 16 (exceeds 5 allowed). Consider refactoring.

Open

def new(args: argparse.Namespace) -> None:
    r"""
    Create new **ASSET**\ s and populate with **KEY-VALUE** pairs. Destination
    directories are created if they are missing.

• Read up
  Read up

Function remove_directory has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    def remove_directory(self, directory: Path, recursive: bool = True) -> list[InventoryOperation]:
        if directory in self._get_pending_removals(mode='dirs'):
            ui.log_debug(f"{directory} already queued for removal")
            # TODO: Consider NoopError when addressing #546.
            return []

• Read up
  Read up

Function __call__ has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    def __call__(self,
                 parser: argparse.ArgumentParser,
                 namespace: argparse.Namespace,
                 key_values: list[str],
                 option_string: str | None = None) -> None:

• Read up
  Read up

Function request_user_response has a Cognitive Complexity of 13 (exceeds 5 allowed). Consider refactoring.

Open

    def request_user_response(self,
                              question: str,
                              default: str = 'yes',
                              answers: list[tuple] | None = None) -> Any:
        r"""Print `question` and read a response from `stdin`.

• Read up
  Read up

Function generate_auto_message has a Cognitive Complexity of 13 (exceeds 5 allowed). Consider refactoring.

Open

    def generate_auto_message(format_string: str,
                              max_length: int = 80,
                              **kwargs) -> str:
        r"""Generate a commit message subject.

• Read up
  Read up

Function modify_asset has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    def modify_asset(self, asset: dict | UserDict | Path, new_asset: dict | UserDict) -> list[InventoryOperation]:
        operations = []
        path = asset if isinstance(asset, Path) else Path(asset.get('path'))
        if not self.repo.is_asset_path(path):
            raise ValueError(f"No such asset: {path}")

• Read up
  Read up

Function add_asset has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    def add_asset(self, asset: dict | UserDict) -> list[InventoryOperation]:
        # TODO: what if I call this with a modified (possibly moved) asset?
        # -> check for conflicts and raise InvalidInventoryOperationError("something about either commit first or rest")
        operations = []
        path = None

• Read up
  Read up