Missing copyright, typos, code style (microsoft#847)

Add missing copyright Fix incorrect namespaces Remove dead code Fix typos Fix indentation Fix spacing
lemillermicrosoft · May 8, 2023 · 1b01049 · 1b01049
1 parent a1bdafb
commit 1b01049
Show file tree

Hide file tree

Showing 25 changed files with 69 additions and 61 deletions.
diff --git a/dotnet/SK-dotnet.sln.DotSettings b/dotnet/SK-dotnet.sln.DotSettings
@@ -107,6 +107,7 @@
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=OK/@EntryIndexedValue">OK</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=OS/@EntryIndexedValue">OS</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=PR/@EntryIndexedValue">PR</s:String>
+	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=QA/@EntryIndexedValue">QA</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=SK/@EntryIndexedValue">SK</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=SKHTTP/@EntryIndexedValue">SKHTTP</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=SSL/@EntryIndexedValue">SSL</s:String>
@@ -188,6 +189,7 @@ public void It$SOMENAME$()
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=Chunker/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=Ctors/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=davinci/@EntryIndexedValue">True</s:Boolean>
+	<s:Boolean x:Key="/Default/UserDictionary/Words/=ENDPART/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=greaterthan/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=Joinable/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=keyvault/@EntryIndexedValue">True</s:Boolean>

diff --git a/dotnet/src/Connectors/Connectors.AI.OpenAI/AzureSdk/ClientBase.cs b/dotnet/src/Connectors/Connectors.AI.OpenAI/AzureSdk/ClientBase.cs
@@ -58,7 +58,7 @@ public abstract class ClientBase
     }
 
     /// <summary>
-    /// Creates a completion stream for the pormpt and settings.
+    /// Creates a completion stream for the prompt and settings.
     /// </summary>
     /// <param name="text">The prompt to complete.</param>
     /// <param name="requestSettings">Request settings for the completion API</param>

diff --git a/dotnet/src/Connectors/Connectors.AI.OpenAI/CustomClient/OpenAIClientBase.cs b/dotnet/src/Connectors/Connectors.AI.OpenAI/CustomClient/OpenAIClientBase.cs
@@ -220,9 +220,7 @@ private async Task<T> ExecutePostRequestAsync<T>(string url, string requestBody,
             var result = Json.Deserialize<T>(responseJson);
             if (result is null)
             {
-                throw new AIException(
-                AIException.ErrorCodes.InvalidResponseContent,
-                "Response JSON parse error");
+                throw new AIException(AIException.ErrorCodes.InvalidResponseContent, "Response JSON parse error");
             }
 
             return result;

diff --git a/dotnet/src/Connectors/Connectors.Memory.Qdrant/QdrantVectorRecord.cs b/dotnet/src/Connectors/Connectors.Memory.Qdrant/QdrantVectorRecord.cs
@@ -1,7 +1,6 @@
 // Copyright (c) Microsoft. All rights reserved.
 
 using System.Collections.Generic;
-using System.Diagnostics.CodeAnalysis;
 using System.Text.Json;
 using System.Text.Json.Serialization;
 using Microsoft.SemanticKernel.Connectors.Memory.Qdrant.Diagnostics;

diff --git a/dotnet/src/Connectors/Connectors.Memory.Sqlite/Connectors.Memory.Sqlite.csproj b/dotnet/src/Connectors/Connectors.Memory.Sqlite/Connectors.Memory.Sqlite.csproj
@@ -25,7 +25,7 @@
   </ItemGroup>
 
   <ItemGroup>
-    <InternalsVisibleTo Include="SemanticKernelConnectorsSqliteTests" PublicKey="$(StongNamePublicKey)"/>
+    <InternalsVisibleTo Include="SemanticKernelConnectorsSqliteTests" PublicKey="$(StrongNamePublicKey)"/>
   </ItemGroup>
 
 </Project>
diff --git a/dotnet/src/Connectors/Connectors.Memory.Sqlite/SqliteMemoryStore.cs b/dotnet/src/Connectors/Connectors.Memory.Sqlite/SqliteMemoryStore.cs
@@ -2,7 +2,6 @@
 
 using System;
 using System.Collections.Generic;
-using System.Diagnostics.CodeAnalysis;
 using System.Globalization;
 using System.Linq;
 using System.Runtime.CompilerServices;

diff --git a/dotnet/src/Extensions/Planning.ActionPlanner/Planning.ActionPlanner.csproj b/dotnet/src/Extensions/Planning.ActionPlanner/Planning.ActionPlanner.csproj
@@ -21,7 +21,7 @@
   </ItemGroup>
 
   <ItemGroup>
-    <InternalsVisibleTo Include="IntegrationTests" PublicKey="$(StongNamePublicKey)" />
+    <InternalsVisibleTo Include="IntegrationTests" PublicKey="$(StrongNamePublicKey)" />
   </ItemGroup>
 
   <ItemGroup>

diff --git a/dotnet/src/Extensions/Planning.SequentialPlanner/Planning.SequentialPlanner.csproj b/dotnet/src/Extensions/Planning.SequentialPlanner/Planning.SequentialPlanner.csproj
@@ -21,8 +21,8 @@
   </ItemGroup>
 
   <ItemGroup>
-    <InternalsVisibleTo Include="IntegrationTests" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="SemanticKernel.Extensions.UnitTests" PublicKey="$(StongNamePublicKey)"/>
+    <InternalsVisibleTo Include="IntegrationTests" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="SemanticKernel.Extensions.UnitTests" PublicKey="$(StrongNamePublicKey)"/>
   </ItemGroup>
 
   <ItemGroup>

diff --git a/dotnet/src/SemanticKernel.Abstractions/Connectors/WebApi/Rest/Model/RestApiOperation.cs b/dotnet/src/SemanticKernel.Abstractions/Connectors/WebApi/Rest/Model/RestApiOperation.cs
@@ -156,7 +156,7 @@ public Uri BuildOperationUrl(IDictionary<string, string> arguments)
 
             //Getting metadata for the header
             var headerMetadata = this.Parameters.FirstOrDefault(p => p.Location == RestApiOperationParameterLocation.Header && p.Name == headerName)
-                ?? throw new RestApiOperationException($"No value for the '{headerName} header is found.'");
+                                 ?? throw new RestApiOperationException($"No value for the '{headerName} header is found.'");
 
             //If parameter is required it's value should always be provided.
             if (headerMetadata.IsRequired)

diff --git a/dotnet/src/SemanticKernel.Abstractions/SemanticKernel.Abstractions.csproj b/dotnet/src/SemanticKernel.Abstractions/SemanticKernel.Abstractions.csproj
@@ -23,15 +23,15 @@
   </ItemGroup>
 
   <ItemGroup>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Core" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Connectors.AI.OpenAI" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Planning.ActionPlanner" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Planning.SequentialPlanner" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Skills.OpenAPI" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Connectors.Memory.CosmosDB" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="SemanticKernel.Skills.UnitTests" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="SemanticKernel.UnitTests" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="IntegrationTests" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="DynamicProxyGenAssembly2" PublicKey="$(StongNamePublicKey)"/> <!-- Moq -->
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Core" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Connectors.AI.OpenAI" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Planning.ActionPlanner" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Planning.SequentialPlanner" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Skills.OpenAPI" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Connectors.Memory.CosmosDB" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="SemanticKernel.Skills.UnitTests" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="SemanticKernel.UnitTests" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="IntegrationTests" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="DynamicProxyGenAssembly2" PublicKey="$(StrongNamePublicKey)"/> <!-- Moq -->
   </ItemGroup>
 </Project>
diff --git a/dotnet/src/SemanticKernel/Connectors/HuggingFace/TextCompletion/HuggingFaceTextCompletion.cs b/dotnet/src/SemanticKernel/Connectors/HuggingFace/TextCompletion/HuggingFaceTextCompletion.cs
@@ -4,7 +4,6 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Net.Http;
-using System.Runtime.CompilerServices;
 using System.Text.Json;
 using System.Threading;
 using System.Threading.Tasks;

diff --git a/dotnet/src/SemanticKernel/Orchestration/ContextVariablesExtensions.cs b/dotnet/src/SemanticKernel/Orchestration/ContextVariablesExtensions.cs
@@ -1,6 +1,7 @@
 // Copyright (c) Microsoft. All rights reserved.
 
 // ReSharper disable once CheckNamespace - Using NS of SKContext
+
 namespace Microsoft.SemanticKernel.Orchestration;
 
 /// <summary>

diff --git a/dotnet/src/SemanticKernel/SemanticKernel.csproj b/dotnet/src/SemanticKernel/SemanticKernel.csproj
@@ -20,14 +20,14 @@ Install this package manually only if you are selecting individual Semantic Kern
   </PropertyGroup>
 
   <ItemGroup>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Planning.ActionPlanner" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Planning.SequentialPlanner" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Connectors.AI.OpenAI" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Skills.OpenAPI" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="SemanticKernel.UnitTests" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="SemanticKernel.Skills.UnitTests" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="SemanticKernel.IntegrationTests" PublicKey="$(StongNamePublicKey)"/>
-    <InternalsVisibleTo Include="DynamicProxyGenAssembly2" PublicKey="$(StongNamePublicKey)"/> <!-- Moq -->
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Planning.ActionPlanner" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Planning.SequentialPlanner" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Connectors.AI.OpenAI" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="Microsoft.SemanticKernel.Skills.OpenAPI" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="SemanticKernel.UnitTests" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="SemanticKernel.Skills.UnitTests" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="SemanticKernel.IntegrationTests" PublicKey="$(StrongNamePublicKey)"/>
+    <InternalsVisibleTo Include="DynamicProxyGenAssembly2" PublicKey="$(StrongNamePublicKey)"/> <!-- Moq -->
   </ItemGroup>
 
   <ItemGroup>

diff --git a/dotnet/src/Skills/Skills.MsGraph/Connectors/MicrosoftGraphModelExtensions.cs b/dotnet/src/Skills/Skills.MsGraph/Connectors/MicrosoftGraphModelExtensions.cs
@@ -45,15 +45,19 @@ public static Models.EmailAddress ToEmailAddress(this Microsoft.Graph.EmailAddre
     /// Convert a calendar event to a Microsoft Graph event.
     /// </summary>
     public static Graph.Event ToGraphEvent(this CalendarEvent calendarEvent)
-       => new()
-       {
-           Subject = calendarEvent.Subject,
-           Body = new ItemBody { Content = calendarEvent.Content, ContentType = BodyType.Html },
-           Start = calendarEvent.Start.HasValue ? DateTimeTimeZone.FromDateTimeOffset(calendarEvent.Start.Value) : DateTimeTimeZone.FromDateTime(System.DateTime.Now),
-           End = calendarEvent.End.HasValue ? DateTimeTimeZone.FromDateTimeOffset(calendarEvent.End.Value) : DateTimeTimeZone.FromDateTime(System.DateTime.Now + TimeSpan.FromHours(1)),
-           Location = new Location { DisplayName = calendarEvent.Location },
-           Attendees = calendarEvent.Attendees?.Select(a => new Attendee { EmailAddress = new Microsoft.Graph.EmailAddress { Address = a } })
-       };
+        => new()
+        {
+            Subject = calendarEvent.Subject,
+            Body = new ItemBody { Content = calendarEvent.Content, ContentType = BodyType.Html },
+            Start = calendarEvent.Start.HasValue
+                ? DateTimeTimeZone.FromDateTimeOffset(calendarEvent.Start.Value)
+                : DateTimeTimeZone.FromDateTime(System.DateTime.Now),
+            End = calendarEvent.End.HasValue
+                ? DateTimeTimeZone.FromDateTimeOffset(calendarEvent.End.Value)
+                : DateTimeTimeZone.FromDateTime(System.DateTime.Now + TimeSpan.FromHours(1)),
+            Location = new Location { DisplayName = calendarEvent.Location },
+            Attendees = calendarEvent.Attendees?.Select(a => new Attendee { EmailAddress = new Microsoft.Graph.EmailAddress { Address = a } })
+        };
 
     /// <summary>
     /// Convert a Microsoft Graph event to a calendar event.
@@ -69,4 +73,3 @@ public static Models.CalendarEvent ToCalendarEvent(this Event msGraphEvent)
             Attendees = msGraphEvent.Attendees?.Select(a => a.EmailAddress.Address)
         };
 }
-
diff --git a/dotnet/src/Skills/Skills.MsGraph/EmailSkill.cs b/dotnet/src/Skills/Skills.MsGraph/EmailSkill.cs
@@ -7,10 +7,10 @@
 using System.Threading.Tasks;
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Logging.Abstractions;
-using Microsoft.Graph;
 using Microsoft.SemanticKernel.Orchestration;
 using Microsoft.SemanticKernel.SkillDefinition;
 using Microsoft.SemanticKernel.Skills.MsGraph.Diagnostics;
+using Microsoft.SemanticKernel.Skills.MsGraph.Models;
 
 namespace Microsoft.SemanticKernel.Skills.MsGraph;
 
@@ -128,11 +128,11 @@ public async Task<string> GetEmailMessagesAsync(SKContext context)
             }
         }
 
-        IEnumerable<Models.EmailMessage> messages = await this._connector.GetMessagesAsync(
-            top: top,
-            skip: skip,
-            select: selectString,
-            context.CancellationToken)
+        IEnumerable<EmailMessage> messages = await this._connector.GetMessagesAsync(
+                top: top,
+                skip: skip,
+                select: selectString,
+                context.CancellationToken)
             .ConfigureAwait(false);
 
         return JsonSerializer.Serialize(

diff --git a/dotnet/src/Skills/Skills.OpenAPI/Extensions/RestApiOperationExtensions.cs b/dotnet/src/Skills/Skills.OpenAPI/Extensions/RestApiOperationExtensions.cs
@@ -5,7 +5,6 @@
 using System.Net.Http;
 using System.Text.RegularExpressions;
 using Microsoft.SemanticKernel.Connectors.WebApi.Rest.Model;
-using Microsoft.SemanticKernel.Text;
 
 #pragma warning disable IDE0130
 // ReSharper disable once CheckNamespace
@@ -37,7 +36,6 @@ public static IReadOnlyList<RestApiOperationParameter> GetParameters(this RestAp
                 defaultValue: serverUrlOverride?.AbsoluteUri ?? operation.ServerUrl?.AbsoluteUri)
         };
 
-
         // Register the "payload" parameter to be advertised for Put and Post operations.
         if (operation.Method == HttpMethod.Put || operation.Method == HttpMethod.Post)
         {

diff --git a/dotnet/src/Skills/Skills.OpenAPI/Skills.OpenAPI.csproj b/dotnet/src/Skills/Skills.OpenAPI/Skills.OpenAPI.csproj
@@ -21,7 +21,7 @@
   </ItemGroup>
 
   <ItemGroup>
-    <InternalsVisibleTo Include="SemanticKernel.Skills.UnitTests" PublicKey="$(StongNamePublicKey)"/>
+    <InternalsVisibleTo Include="SemanticKernel.Skills.UnitTests" PublicKey="$(StrongNamePublicKey)"/>
   </ItemGroup>
 
   <ItemGroup>

diff --git a/dotnet/src/Skills/Skills.Web/Bing/BingConnector.cs b/dotnet/src/Skills/Skills.Web/Bing/BingConnector.cs
@@ -35,7 +35,9 @@ public BingConnector(string apiKey, ILogger<BingConnector>? logger = null)
     public async Task<IEnumerable<string>> SearchAsync(string query, int count = 1, int offset = 0, CancellationToken cancellationToken = default)
     {
         if (count <= 0) { throw new ArgumentOutOfRangeException(nameof(count)); }
+
         if (count >= 50) { throw new ArgumentOutOfRangeException(nameof(count), "{nameof(count)} value must be less than 50."); }
+
         if (offset < 0) { throw new ArgumentOutOfRangeException(nameof(offset)); }
 
         Uri uri = new($"https://api.bing.microsoft.com/v7.0/search?q={Uri.EscapeDataString(query)}&count={count}&offset={offset}");

diff --git a/dotnet/src/Skills/Skills.Web/Google/GoogleConnector.cs b/dotnet/src/Skills/Skills.Web/Google/GoogleConnector.cs
@@ -45,7 +45,9 @@ public sealed class GoogleConnector : IWebSearchEngineConnector, IDisposable
         CancellationToken cancellationToken)
     {
         if (count <= 0) { throw new ArgumentOutOfRangeException(nameof(count)); }
+
         if (count > 10) { throw new ArgumentOutOfRangeException(nameof(count), "{nameof(count)} value must be between 0 and 10, inclusive."); }
+
         if (offset < 0) { throw new ArgumentOutOfRangeException(nameof(offset)); }
 
         var search = this._search.Cse.List();

diff --git a/samples/apps/copilot-chat-app/webapi/Skills/ChatSkill.cs b/samples/apps/copilot-chat-app/webapi/Skills/ChatSkill.cs
@@ -12,7 +12,7 @@
 using Microsoft.SemanticKernel.SkillDefinition;
 using SemanticKernel.Service.Config;
 using SemanticKernel.Service.Model;
-using SemanticKernel.Service.Skills.OpenApiSkills;
+using SemanticKernel.Service.Skills.OpenApiSkills.GitHubSkill.Model;
 using SemanticKernel.Service.Storage;
 
 namespace SemanticKernel.Service.Skills;

diff --git a/samples/apps/copilot-chat-app/webapi/Skills/OpenApiSkills/GitHubSkill/Model/Label.cs b/samples/apps/copilot-chat-app/webapi/Skills/OpenApiSkills/GitHubSkill/Model/Label.cs
@@ -1,6 +1,8 @@
-using System.Text.Json.Serialization;
+// Copyright (c) Microsoft. All rights reserved.
 
-namespace SemanticKernel.Service.Skills.OpenApiSkills;
+using System.Text.Json.Serialization;
+
+namespace SemanticKernel.Service.Skills.OpenApiSkills.GitHubSkill.Model;
 
 /// <summary>
 /// Represents a pull request label.

diff --git a/samples/apps/copilot-chat-app/webapi/Skills/OpenApiSkills/GitHubSkill/Model/PullRequest.cs b/samples/apps/copilot-chat-app/webapi/Skills/OpenApiSkills/GitHubSkill/Model/PullRequest.cs
@@ -1,6 +1,8 @@
-using System.Text.Json.Serialization;
+// Copyright (c) Microsoft. All rights reserved.
 
-namespace SemanticKernel.Service.Skills.OpenApiSkills;
+using System.Text.Json.Serialization;
+
+namespace SemanticKernel.Service.Skills.OpenApiSkills.GitHubSkill.Model;
 
 /// <summary>
 /// Represents a GitHub Pull Request.

diff --git a/samples/apps/copilot-chat-app/webapi/Skills/OpenApiSkills/GitHubSkill/Model/Repo.cs b/samples/apps/copilot-chat-app/webapi/Skills/OpenApiSkills/GitHubSkill/Model/Repo.cs
@@ -1,6 +1,8 @@
-using System.Text.Json.Serialization;
+// Copyright (c) Microsoft. All rights reserved.
 
-namespace SemanticKernel.Service.Skills.OpenApiSkills;
+using System.Text.Json.Serialization;
+
+namespace SemanticKernel.Service.Skills.OpenApiSkills.GitHubSkill.Model;
 
 /// <summary>
 /// Represents a GitHub Repo.
@@ -19,7 +21,6 @@ public class Repo
     [JsonPropertyName("full_name")]
     public string FullName { get; set; }
 
-
     /// <summary>
     /// Initializes a new instance of the <see cref="Repo"/>.
     /// </summary>

diff --git a/samples/apps/copilot-chat-app/webapi/Skills/OpenApiSkills/GitHubSkill/Model/User.cs b/samples/apps/copilot-chat-app/webapi/Skills/OpenApiSkills/GitHubSkill/Model/User.cs
@@ -1,6 +1,6 @@
 using System.Text.Json.Serialization;
 
-namespace SemanticKernel.Service.Skills.OpenApiSkills;
+namespace SemanticKernel.Service.Skills.OpenApiSkills.GitHubSkill.Model;
 
 /// <summary>
 /// Represents a user on GitHub.

diff --git a/samples/dotnet/kernel-syntax-examples/Example31_CustomPlanner.cs b/samples/dotnet/kernel-syntax-examples/Example31_CustomPlanner.cs
@@ -199,9 +199,9 @@ private static Plan NodeListToPlan(XmlNodeList nodes, SKContext context, string
             }
             else
             {
-                if (string.IsNullOrEmpty(skillName) ?
-                    !context.Skills!.TryGetFunction(functionName, out var _) :
-                    !context.Skills!.TryGetFunction(skillName, functionName, out var _))
+                if (string.IsNullOrEmpty(skillName)
+                        ? !context.Skills!.TryGetFunction(functionName, out var _)
+                        : !context.Skills!.TryGetFunction(skillName, functionName, out var _))
                 {
                     var planStep = new Plan(node.InnerText);
                     planStep.Parameters.Update(node.InnerText);