Simplify template parameters to single FullTaskName, add analyzer and…

Frends.Template/.template.config/template.json

@@ -14,22 +14,29 @@
   "sourceName": "Frends.Echo.Execute",
   "preferNameDirectory": true,
   "symbols": {
-    "taskName": {
+    "FullTaskName": {
       "type": "parameter",
       "datatype": "string",
-      "defaultValue": "Execute",
-      "replaces": "Execute"
+      "defaultValue": "Frends.Echo.Execute",
+      "replaces": "Frends.Echo.Execute",
+      "description": "Full name in format: Frends.ClassName.MethodName"
     },
     "className": {
-      "type": "parameter",
-      "datatype": "string",
-      "defaultValue": "Echo",
+      "type": "derived",
+      "valueSource": "FullTaskName",
+      "valueTransform": "GetClassName",
       "replaces": "Echo"
     },
+    "taskName": {
+      "type": "derived",
+      "valueSource": "FullTaskName",
+      "valueTransform": "GetMethodName",
+      "replaces": "Execute"
+    },
     "workflows": {
-      "type": "parameter",
-      "datatype": "string",
-      "defaultValue": "Execute",
+      "type": "derived",
+      "valueSource": "FullTaskName",
+      "valueTransform": "GetMethodName",
       "FileRename": "Execute"
     },
     "CurrentDate": {
@@ -49,6 +56,18 @@
       "replaces": "GeneratedYear"
     }
   },
+  "forms": {
+    "GetClassName": {
+      "identifier": "replace",
+      "pattern": "^.*\\.([^.]+)\\.[^.]+$",
+      "replacement": "$1"
+    },
+    "GetMethodName": {
+      "identifier": "replace",
+      "pattern": "^.*\\.([^.]+)$",
+      "replacement": "$1"
+    }
+  },
   "SpecialCustomOperations": {
     "**/*.md": {
       "operations": [
@@ -151,4 +170,4 @@
       "continueOnError": false
     }
   ]
-}
+}

Frends.Template/CHANGELOG.md

@@ -2,6 +2,6 @@
 
 ## [1.0.0] - GeneratedDate
 
-### Changed
+### Added
 
 - Initial implementation

Frends.Template/Frends.Echo.Execute.Tests/ErrorHandlerTest.cs

@@ -0,0 +1,54 @@
+using System;
+using System.Threading;
+using Frends.Echo.Execute.Definitions;
+using NUnit.Framework;
+
+namespace Frends.Echo.Execute.Tests;
+
+// TODO: Adjust the test to use a real invalid Input scenario (e.g., missing or malformed data)
+[TestFixture]
+public class ErrorHandlerTest
+{
+    private const string CustomErrorMessage = "CustomErrorMessage";
+
+    [Test]
+    public void Should_Throw_Error_When_ThrowErrorOnFailure_Is_True()
+    {
+        var ex = Assert.Throws<Exception>(() =>
+           Echo.Execute(DefaultInput(), DefaultConnection(), DefaultOptions(), CancellationToken.None));
+        Assert.That(ex, Is.Not.Null);
+    }
+
+    [Test]
+    public void Should_Return_Failed_Result_When_ThrowErrorOnFailure_Is_False()
+    {
+        var options = DefaultOptions();
+        options.ThrowErrorOnFailure = false;
+        var result = Echo.Execute(DefaultInput(), DefaultConnection(), options, CancellationToken.None);
+        Assert.That(result.Success, Is.False);
+    }
+
+    [Test]
+    public void Should_Use_Custom_ErrorMessageOnFailure()
+    {
+        var options = DefaultOptions();
+        options.ErrorMessageOnFailure = CustomErrorMessage;
+        var ex = Assert.Throws<Exception>(() =>
+            Echo.Execute(DefaultInput(), DefaultConnection(), options, CancellationToken.None));
+        Assert.That(ex, Is.Not.Null);
+        Assert.That(ex.Message, Contains.Substring(CustomErrorMessage));
+    }
+
+    private static Input DefaultInput() => new()
+    {
+        Repeat = -1, // Invalid value to cause an exception
+    };
+
+    private static Connection DefaultConnection() => new();
+
+    private static Options DefaultOptions() => new()
+    {
+        ThrowErrorOnFailure = true,
+        ErrorMessageOnFailure = string.Empty,
+    };
+}

Frends.Template/Frends.Echo.Execute.Tests/GlobalSuppressions.cs

@@ -1,10 +1,11 @@
 using System.Diagnostics.CodeAnalysis;
 
+[assembly: SuppressMessage("StyleCop.CSharp.SpecialRules", "SA0001::XmlCommentAnalysisDisabled", Justification = "Following Frends documentation guidelines")]
 [assembly: SuppressMessage("StyleCop.CSharp.SpacingRules", "SA1000:KeywordsMustBeSpacedCorrectly", Justification = "Incorrect for .NET 8.0")]
 [assembly: SuppressMessage("StyleCop.CSharp.SpacingRules", "SA1010:OpeningSquareBracketsMustBeSpacedCorrectly", Justification = "Incorrect for .NET 8.0")]
 [assembly: SuppressMessage("StyleCop.CSharp.ReadabilityRules", "SA1101:PrefixLocalCallsWithThis", Justification = "Following Frends documentation guidelines")]
 [assembly: SuppressMessage("StyleCop.CSharp.OrderingRules", "SA1200:UsingDirectivesMustBePlacedWithinNamespace", Justification = "Following Frends documentation guidelines")]
 [assembly: SuppressMessage("StyleCop.CSharp.OrderingRules", "SA1206:DeclarationKeywordsMustFollowOrder", Justification = "Incorrect for .NET 8.0")]
 [assembly: SuppressMessage("StyleCop.CSharp.LayoutRules", "SA1503:BracesMustNotBeOmitted", Justification = "Following Frends documentation guidelines")]
 [assembly: SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1600:ElementsMustBeDocumented", Justification = "Following Frends documentation guidelines")]
-[assembly: SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1633:FileMustHaveHeader", Justification = "Following Frends documentation guidelines")]
+[assembly: SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1633:FileMustHaveHeader", Justification = "Following Frends documentation guidelines")]

Frends.Template/Frends.Echo.Execute/Definitions/Connection.cs

@@ -16,5 +16,5 @@ public class Connection
     /// <example>Host=127.0.0.1;Port=5432</example>
     [DisplayFormat(DataFormatString = "Text")]
     [DefaultValue("")]
-    public string ConnectionString { get; set; }
+    public string ConnectionString { get; set; } = string.Empty;
 }

Frends.Template/Frends.Echo.Execute/Definitions/Error.cs

@@ -1,3 +1,5 @@
+using System;
+
 namespace Frends.Echo.Execute.Definitions;
 
 /// <summary>
@@ -14,7 +16,7 @@ public class Error
     /// <summary>
     /// Additional information about the error.
     /// </summary>
-    /// <example>object { Exception Exception }</example>
+    /// <example>object { Exception AdditionalInfo }</example>
     // TODO: Add task specific additional information. Strong typing is recommended when reasonable.
-    public dynamic AdditionalInfo { get; set; }
+    public Exception AdditionalInfo { get; set; }
 }

Frends.Template/Frends.Echo.Execute/Definitions/Options.cs

@@ -21,13 +21,13 @@ public class Options
     /// </summary>
     /// <example>false</example>
     [DefaultValue(true)]
-    public bool ThrowErrorOnFailure { get; set; }
+    public bool ThrowErrorOnFailure { get; set; } = true;
 
     /// <summary>
     /// Overrides the error message on failure.
     /// </summary>
     /// <example>Custom error message</example>
     [DisplayFormat(DataFormatString = "Text")]
     [DefaultValue("")]
-    public string ErrorMessageOnFailure { get; set; }
+    public string ErrorMessageOnFailure { get; set; } = string.Empty;
 }

Frends.Template/Frends.Echo.Execute/Definitions/Result.cs

@@ -20,6 +20,6 @@ public class Result
     /// <summary>
     /// Error that occurred during task execution.
     /// </summary>
-    /// <example>object { string Message, object { Exception Exception } AdditionalInfo }</example>
+    /// <example>object { string Message, Exception AdditionalInfo }</example>
     public Error Error { get; set; }
-}
+}

Frends.Template/Frends.Echo.Execute/Frends.Echo.Execute.cs

@@ -3,11 +3,12 @@
 using System.Linq;
 using System.Threading;
 using Frends.Echo.Execute.Definitions;
+using Frends.Echo.Execute.Helpers;
 
 namespace Frends.Echo.Execute;
 
 /// <summary>
-/// Task class.
+/// Task Class for Echo operations.
 /// </summary>
 public static class Echo
 {
@@ -19,7 +20,7 @@ public static class Echo
     /// <param name="connection">Connection parameters.</param>
     /// <param name="options">Additional parameters.</param>
     /// <param name="cancellationToken">A cancellation token provided by Frends Platform.</param>
-    /// <returns>object { bool Success, string Output, object Error { string Message, dynamic AdditionalInfo } }</returns>
+    /// <returns>object { bool Success, string Output, object Error { string Message, Exception AdditionalInfo } }</returns>
     // TODO: Remove Connection parameter if the task does not make connections
     public static Result Execute(
         [PropertyTab] Input input,
@@ -36,6 +37,9 @@ public static Result Execute(
             // and checked during long-running operations, e.g., loops
             cancellationToken.ThrowIfCancellationRequested();
 
+            if (input.Repeat < 0)
+                throw new Exception("Repeat count cannot be negative.");
+
             var output = string.Join(options.Delimiter, Enumerable.Repeat(input.Content, input.Repeat));
 
             return new Result
@@ -45,33 +49,9 @@ public static Result Execute(
                 Error = null,
             };
         }
-        catch (Exception e) when (e is not OperationCanceledException)
+        catch (Exception ex)
         {
-            if (options.ThrowErrorOnFailure)
-            {
-                if (string.IsNullOrEmpty(options.ErrorMessageOnFailure))
-                    throw new Exception(e.Message, e);
-
-                throw new Exception(options.ErrorMessageOnFailure, e);
-            }
-
-            var errorMessage = !string.IsNullOrEmpty(options.ErrorMessageOnFailure)
-                ? $"{options.ErrorMessageOnFailure}: {e.Message}"
-                : e.Message;
-
-            return new Result
-            {
-                Success = false,
-                Output = null,
-                Error = new Error
-                {
-                    Message = errorMessage,
-                    AdditionalInfo = new
-                    {
-                        Exception = e,
-                    },
-                },
-            };
+            return ErrorHandler.Handle(ex, options.ThrowErrorOnFailure, options.ErrorMessageOnFailure);
         }
     }
 }

Frends.Template/Frends.Echo.Execute/Frends.Echo.Execute.csproj

@@ -31,9 +31,21 @@
     </ItemGroup>
 
     <ItemGroup>
-        <None Include="FrendsTaskMetadata.json" Pack="true" PackagePath="/">
+		<AdditionalFiles Include="FrendsTaskMetadata.json" Pack="true" PackagePath="/">
             <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
-        </None>
+        </AdditionalFiles>
     </ItemGroup>
+
+	<ItemGroup>
+		<Content Include="migration.json" PackagePath="/" Pack="true" />
+		<Content Include="../CHANGELOG.md" PackagePath="/" Pack="true" />
+	</ItemGroup>
+
+	<ItemGroup>
+		<PackageReference Include="FrendsTaskAnalyzers" Version="*">
+			<PrivateAssets>all</PrivateAssets>
+			<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+		</PackageReference>
+	</ItemGroup>
 
 </Project>

Frends.Template/Frends.Echo.Execute/GlobalSuppressions.cs

@@ -6,6 +6,7 @@
 [assembly: SuppressMessage("StyleCop.CSharp.OrderingRules", "SA1200:UsingDirectivesMustBePlacedWithinNamespace", Justification = "Following Frends documentation guidelines")]
 [assembly: SuppressMessage("StyleCop.CSharp.OrderingRules", "SA1206:DeclarationKeywordsMustFollowOrder", Justification = "Incorrect for .NET 8.0")]
 [assembly: SuppressMessage("StyleCop.CSharp.LayoutRules", "SA1503:BracesMustNotBeOmitted", Justification = "Following Frends documentation guidelines")]
+[assembly: SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1600:Elements should be documented", Justification = "Documentation checked by custom analyzers")]
 [assembly: SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1623:PropertySummaryDocumentationMustMatchAccessors", Justification = "Following Frends documentation guidelines")]
 [assembly: SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1629:DocumentationTextMustEndWithAPeriod", Justification = "Following Frends Tasks guidelines")]
-[assembly: SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1633:FileMustHaveHeader", Justification = "Following Frends documentation guidelines")]
+[assembly: SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1633:FileMustHaveHeader", Justification = "Following Frends documentation guidelines")]

Frends.Template/Frends.Echo.Execute/Helpers/ErrorHandler.cs

@@ -0,0 +1,32 @@
+using System;
+using Frends.Echo.Execute.Definitions;
+
+namespace Frends.Echo.Execute.Helpers;
+
+internal static class ErrorHandler
+{
+    internal static Result Handle(Exception exception, bool throwOnFailure, string errorMessageOnFailure)
+    {
+        if (throwOnFailure)
+        {
+            if (!string.IsNullOrEmpty(errorMessageOnFailure))
+            {
+                throw new Exception(errorMessageOnFailure, exception);
+            }
+
+            throw exception;
+        }
+
+        var errorMessage = string.IsNullOrEmpty(errorMessageOnFailure)
+            ? exception.Message
+            : $"{errorMessageOnFailure}: {exception.Message}";
+
+        var error = new Error
+        {
+            Message = errorMessage,
+            AdditionalInfo = exception,
+        };
+
+        return new Result { Success = false, Error = error };
+    }
+}

Frends.Template/Frends.Echo.Execute/migration.json

@@ -0,0 +1,12 @@
+[
+  {
+    "Task": "Frends.Echo.Execute",
+    "Migrations": [
+      {
+        "Version": "1.0.0",
+        "Description": "",
+        "Migration": []
+      }
+    ]
+  }
+]

Frends.Template/README.md

@@ -8,7 +8,7 @@ Description of what the Task will do.
 
 ## Installing
 
-You can install the Task via frends UI Task View.
+You can install the Task via Frends UI Task View.
 
 ## Building
 
@@ -30,7 +30,7 @@ Run the tests
 
 `dotnet pack --configuration Release`
 
-### Third party licenses
+### Third-party licenses
 
 StyleCop.Analyzer version (unmodified version 1.1.118) used to analyze code uses Apache-2.0 license, full text and
 source code can be found at https://github.com/DotNetAnalyzers/StyleCopAnalyzers

Frends.Template/workflows/Execute_build_and_test_on_main.yml

@@ -14,5 +14,6 @@ jobs:
     with:
       workdir: Frends.Echo.Execute
       dotnet_version: 8.0.x
+      strict_analyzers: true
     secrets:
       badge_service_api_key: ${{ secrets.BADGE_SERVICE_API_KEY }}

Frends.Template/workflows/Execute_build_and_test_on_push.yml

@@ -14,6 +14,7 @@ jobs:
     with:
       workdir: Frends.Echo.Execute
       dotnet_version: 8.0.x
+      strict_analyzers: true
     secrets:
       badge_service_api_key: ${{ secrets.BADGE_SERVICE_API_KEY }}
       test_feed_api_key: ${{ secrets.TASKS_TEST_FEED_API_KEY }}

Frends.Template/workflows/Execute_release.yml

@@ -9,5 +9,6 @@ jobs:
     with:
       workdir: Frends.Echo.Execute
       dotnet_version: 8.0.x
+      strict_analyzers: true
     secrets:
       feed_api_key: ${{ secrets.TASKS_FEED_API_KEY }}

FrendsTaskTemplate.csproj

@@ -2,14 +2,14 @@
 
     <PropertyGroup>
         <PackageType>Template</PackageType>
-        <PackageVersion>1.3.0</PackageVersion>
+        <PackageVersion>1.4.0</PackageVersion>
         <PackageId>frendstasktemplate</PackageId>
         <Title>Frends Task template</Title>
         <Authors>Frends</Authors>
         <Description>Template for creating Frends Tasks.</Description>
         <PackageProjectUrl>https://github.com/FrendsPlatform/FrendsTaskTemplate</PackageProjectUrl>
         <PackageTags>dotnet-new;templates;frends</PackageTags>
-        <TargetFramework>net6.0</TargetFramework>
+        <TargetFramework>net8.0</TargetFramework>
 
         <IncludeContentInPack>true</IncludeContentInPack>
         <IncludeBuildOutput>false</IncludeBuildOutput>