What are the reasons to consolidate all the functions for different dbt commands in a single Python file ?

[tahseenadit]

Hi, I was looking into this file https://github.com/dbt-labs/dbt-core/blob/main/core/dbt/cli/main.py and was wondering what was the thinking behind writing all the functions for different dbt commands in a single python file because each of these functions use multiple decorators.

Here is why I am asking this question:

Imagine someone developed a custom decorator that generates an error like below:

def some_func(func):
    raise ImportError("Deliberately raising error")
 

Now someone at some point adds another custom command in main.py file that uses the above decorator like below:

@custom.some_func
def custom_command(ctx, **kwargs):
    pass
 

Then when main.py is executed, it fails.

File "/Users/tahseen/My Work Files/dbt-core-exploration/core/dbt/cli/custom.py", line 2, in func
    raise ImportError("Deliberately raising error")
ImportError: Deliberately raising error

I am not sure if there will ever be a need to execute main.py separately but I am just exploring here. Would like to know the thinking behind the current implementation design. Can you share anything regarding that ?