Skip to content

Commit 65bddcb

Browse files
refactor: delete BookToolHelpers, inline extension normalization at call sites
The helper class only had one method (NormalizeExtension) used in 3 places. The VB.NET/F# cases were dead code — all 1,396 real book listings are .cs. Simplified to a ternary inline at each call site; non-cs extensions pass through unchanged. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 27ca727 commit 65bddcb

3 files changed

Lines changed: 3 additions & 17 deletions

File tree

EssentialCSharp.Web/Tools/BookContentTool.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ public async Task<string> GetListingWithContext(
158158
return $"Listing {chapter}.{listing} not found. Verify the chapter and listing numbers.";
159159
}
160160

161-
string langHint = BookToolHelpers.NormalizeExtension(response.FileExtension);
161+
string langHint = response.FileExtension == "cs" ? "csharp" : response.FileExtension;
162162
var sb = new StringBuilder();
163163
sb.AppendLine(CultureInfo.InvariantCulture, $"## Listing {response.ChapterNumber}.{response.ListingNumber}");
164164
sb.AppendLine(CultureInfo.InvariantCulture, $"```{langHint}");

EssentialCSharp.Web/Tools/BookListingTool.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ public async Task<string> GetListingSourceCode(
3131
return $"Listing {chapter}.{listing} not found. Verify that both the chapter and listing numbers are correct.";
3232
}
3333

34-
string langHint = BookToolHelpers.NormalizeExtension(response.FileExtension);
34+
string langHint = response.FileExtension == "cs" ? "csharp" : response.FileExtension;
3535
return $"## Listing {response.ChapterNumber}.{response.ListingNumber}\n\n```{langHint}\n{response.Content}\n```";
3636
}
3737

@@ -68,7 +68,7 @@ public async Task<string> SearchListingsByCode(
6868
if (found >= maxResults) break;
6969
if (listing.Content.Contains(pattern, StringComparison.OrdinalIgnoreCase))
7070
{
71-
string langHint = BookToolHelpers.NormalizeExtension(listing.FileExtension);
71+
string langHint = listing.FileExtension == "cs" ? "csharp" : listing.FileExtension;
7272
sb.AppendLine(CultureInfo.InvariantCulture, $"### Listing {listing.ChapterNumber}.{listing.ListingNumber}");
7373
sb.AppendLine(CultureInfo.InvariantCulture, $"```{langHint}");
7474
sb.AppendLine(listing.Content);

EssentialCSharp.Web/Tools/BookToolHelpers.cs

Lines changed: 0 additions & 14 deletions
This file was deleted.

0 commit comments

Comments
 (0)