Prepared release 0.1.9 with some minor improvements

Author: sandreas

tone/Commands/DumpCommand.cs

@@ -46,13 +46,22 @@ private async Task<ReturnCode> SpectreConsoleDelegate(IFileSystem fs, string out
         return await Task.FromResult(ReturnCode.Success);
     }
 
+    private async Task<ReturnCode> NoBreakDelegate(IFileSystem fs, string outputFile, IEnumerable<string> lines,  CancellationToken ct) 
+    {
+        foreach (var line in lines)
+        {
+            _console.WriteNoBreakLine(line);
+        }
+        return await Task.FromResult(ReturnCode.Success);
+    }
+    
     private async Task<ReturnCode> WriteFileDelegate(IFileSystem fs, string outputFile, IEnumerable<string> lines,  CancellationToken ct) 
     {
         try
         {
             await fs.File.WriteAllLinesAsync(outputFile, lines, ct);
         }
-        catch (Exception e)
+        catch (Exception)
         {
             return ReturnCode.GeneralError;
         }
@@ -90,6 +99,11 @@ public override async Task<int> ExecuteAsync(CommandContext context, DumpCommand
                 _ => NoSpecSuffix
             };
         }
+        else if(settings.Format != SerializerFormat.Default && Console.IsOutputRedirected)
+        {
+            // Output width needs to be ajusted when output is redirected and format is json
+            outputDelegate = NoBreakDelegate;
+        }
 
         foreach (var file in inputFiles)
         {

tone/Program.cs

@@ -4,6 +4,7 @@
 using System.Linq;
 using System.Net.Http;
 using System.Threading;
+using ATL;
 using GrokNet;
 using Jint;
 using Microsoft.Extensions.Configuration;
@@ -38,6 +39,13 @@
 {
     var debugMode = args.Contains("--debug");
 
+    // atldotnet basic settings to change the common behaviour
+    Settings.UseFileNameWhenNoTitle = false; // don't fallback to filename when no title is set
+    // Settings.NullAbsentValues = true; // use null instead of default when tags are considered empty
+    Settings.OutputStacktracesToConsole = false; // don't output something to console
+    // Settings.ID3v2_tagSubVersion = 3; // use id3v2.4 by default (4 for 2.4)
+    // Settings.MP3_parseExactDuration = true; // more exact duration, takes longer
+    
     var settingsProvider = new CustomCommandSettingsProvider();
 
     var services = new ServiceCollection();
@@ -223,7 +231,7 @@
         config.UseStrictParsing();
         config.CaseSensitivity(CaseSensitivity.None);
         config.SetApplicationName("tone");
-        config.SetApplicationVersion("0.1.8");
+        config.SetApplicationVersion("0.1.9");
         config.ValidateExamples();
         config.AddCommand<DumpCommand>("dump")
             .WithDescription("dump metadata for files and directories (directories are traversed recursively)")

tone/Services/SpectreConsoleService.cs

@@ -16,7 +16,19 @@ public SpectreConsoleService(IAnsiConsole? stdout = null, IAnsiConsole? stderr =
         {
             Out = new AnsiConsoleOutput(Console.Error)
         });
-        
+    }
+
+    public void WriteNoBreakLine(string text)
+    {
+        var oldWidth = Output.Profile.Width;
+        Output.Profile.Width = text.Length;
+        Output.WriteLine(text, Style.Plain);
+        Output.Profile.Width = oldWidth;
+    } 
+    
+    /*
+    public void AdjustOutputWidth()
+    {
         // fix unwanted wrapping if output is redirected
         if (ShouldAdjustOutputWidth())
         {
@@ -28,7 +40,7 @@ public SpectreConsoleService(IAnsiConsole? stdout = null, IAnsiConsole? stderr =
             Error.Profile.Width = int.MaxValue;
         }
     }
-
+    
     private static bool ShouldAdjustOutputWidth() => Environment.GetEnvironmentVariable("TONE_OUTPUT_REDIRECT") switch
     {
         "0" => false,
@@ -42,6 +54,7 @@ public SpectreConsoleService(IAnsiConsole? stdout = null, IAnsiConsole? stderr =
         "1" => true,
         _ => Console.IsErrorRedirected
     };
+    */
 
     public Profile Profile => Output.Profile;
     public IAnsiConsoleCursor Cursor => Output.Cursor;

tone/doc/release/release-notes-v0.1.9.md

@@ -0,0 +1,22 @@
+# Release Notes
+
+## Fixed
+- #44 - Could not remove `movement` tag with `--meta-remove-property` - should now work as expected
+- #46 - Files without any metadata could not be modified in some cases (e.g. using only `--meta-subtitle`) - should now work as expected
+
+## Changed
+- Improved fix for #68 - no environment variable needed any more
+
+
+
+## Setup instructions
+
+`tone` is released as single monolithic binary, so you don't need a setup file or any dependencies (not even a `.NET` runtime). Download the `tone` 
+release for your platform, extract it and run it via command line. If you need help choosing your download, here are some hints:
+
+- For Windows, only the x64 platform is available... choose `-win-x64.zip`
+- For `musl` (an alternative C library) choose your arch prefixed by `musl` (usually this is used in alpine `docker` images and other lightweight distributions)
+- For standard Linux (like *Fedora*, *Ubuntu*, etc.), it is recommended choose your arch without `musl` prefix
+- For *macOS* you might need to run `xattr -rd com.apple.quarantine tone` after extracting to remove `quarantine` flag
+
+

tone/tone.csproj

@@ -47,7 +47,7 @@
       <PackageReference Include="Microsoft.Extensions.Http" Version="8.0.1" />
       <PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="8.0.0" />
       <PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
-      <PackageReference Include="Sandreas.AudioMetadata" Version="0.1.8" />
+      <PackageReference Include="Sandreas.AudioMetadata" Version="0.2.1" />
       <PackageReference Include="Sandreas.SpectreConsoleHelpers" Version="0.0.2" />
       <PackageReference Include="Serilog" Version="4.0.2" />
       <PackageReference Include="Serilog.Extensions.Logging" Version="8.0.0" />
@@ -63,6 +63,7 @@
       <Folder Include="Common\Extensions\IEnumerable" />
       <Folder Include="var\data\output" />
       <Folder Include="var\issues\15" />
+      <Folder Include="var\issues\55\input\" />
       <Folder Include="var\issues\59\" />
     </ItemGroup>