Don't style text when output terminal is not interactive

[pavelzw]

When writing the report to a file, the styling doesn't get removed and results in ascii escape sequences in the file:

tach report my_module > report.txt

opening this in my IDE:

I would expect something like this instead (also without the warning, xref #676):

[emdoyle]

@pavelzw I'll fix this! We have functions to detect this but they are scattered across the Python/Rust boundary. This output is entirely rendered in Rust I believe, so should be a straightforward fix.

It'll take some time to work through the rest of the formatting/stdout issues you've raised but they seem reasonable. Could I ask about your use case? I'd love to learn more about how you're using Tach's output so that we can build in that direction :)

[pavelzw]

Basically i'm trying to build a tool that checks within my project that i only use certain external dependencies in certain submodules

allow:
  # only allow importing the `logging` module in the following submodules
  logging:
    - my_project.log
    # The below directories should be migrated in the near future.
    # We should remove each directory as they are migrated.
    - my_project.legacy
   # ...

deny:
  # Packages that exist under this tag will be denied usage
  # in precisely these modules.
  pandas:
    - my_project.new

The idea was to leverage tach report --output json (#678) and then just ensure all of these requirements are met.
But since #678 is not yet done, I planned to parse the output of tach report myself for now.

[emdoyle]

Ok I see! I just made the changes to fix styling in the report, this will merge and release later today. Note that the paths in the non-interactive report will be absolute.

Your actual use-case is something that Tach will support first-class very soon. Do you have a strong preference to define these rules on the external dependencies vs. opt-in on each module? The most natural extensions for Tach config would be external_depends_on/external_cannot_depend_on.