Skip to content

Fix series 6 blog accuracy: OllamaSharp narrative and AiService hrInsight wrapper#27

Merged
workcontrolgit merged 3 commits into
masterfrom
develop
Apr 22, 2026
Merged

Fix series 6 blog accuracy: OllamaSharp narrative and AiService hrInsight wrapper#27
workcontrolgit merged 3 commits into
masterfrom
develop

Conversation

@workcontrolgit

Copy link
Copy Markdown
Owner

Summary

  • 6.1 — Correct subtitle, intro, What You'll Learn, Solution, and Why This Matters sections to accurately describe OllamaSharp (not Microsoft.Extensions.AI); remove references to IChatClient, AddOllamaChatClient(), and AddAzureOpenAIChatClient() which are not used in the codebase
  • 6.3 — Fix AiService.hrInsight() code sample to include the ApiResult<T> wrapper and .pipe(map(r => r.value)) unwrap that the actual implementation uses; add explanation of why only hrInsight needs the envelope unwrap

Test plan

  • Verify 6.1 Solution section no longer references Microsoft.Extensions.AI as the abstraction in use
  • Verify 6.3 AiService code block matches src/app/services/api/ai.service.ts exactly
  • Confirm no other series-6 blogs have inaccurate code samples

@workcontrolgit workcontrolgit merged commit 44bc4cf into master Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant