Merge/pr 158 local #180

73junito · 2026-01-16T01:37:17Z

Summary

Describe the change and why it is needed.

Checklist

I have added tests that prove my change is correct (unit/integration).
I have updated documentation where necessary (docs/...).
CI: All required checks pass (Elixir tests, redaction-tests, trivy-scan, docker-image build).
This PR follows the adaptive debugging policy: no PII in logs or artifacts unless approved.

Testing

Describe how the change was tested locally and CI steps.

Pull request overview

This pull request adds content validation infrastructure for AI-generated learning materials, implementing a "stub" vs "humanized" content workflow. The PR introduces validation scripts, test coverage, CI workflows, and updates all course manifests to include a contentStatus field.

Changes:

Adds content validation scripts and CI workflows to block stub content from being merged
Updates 20+ course manifests to include contentStatus field (mostly "stub", one "humanized")
Adds comprehensive test coverage for the orchestration pipeline and removes duplicate code from Storybook files

Reviewed changes

Copilot reviewed 36 out of 36 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
scripts/orchestrate_content.py	New orchestration script for AI content generation (contains critical structural bugs)
tests/test_orchestrate_content.py	Test suite for orchestration functionality
scripts/validate_no_stubs.py	Validator script to detect stub content in manifests and markdown
scripts/bulk_set_content_status.py	Bulk update utility for setting contentStatus across manifests
.github/workflows/block-stubs.yml	CI workflow to prevent merging PRs with stub content
.github/workflows/orchestrate-test.yml	CI workflow for testing orchestration (references non-existent fixture)
docs/ai/ai-toolkit-guidance.md	Advisory guidance for AI-assisted content authoring (contains duplicate content)
docs/benchmarks/ollama-client-benchmark.md	Benchmark comparison of Python/Node/Rust Ollama clients
content/courses/*/modules.json	Updates to 20+ course manifests adding contentStatus field
README.md	Updates with content standards and benchmark references (broken markdown)
CONTRIBUTING.md	New contributor guidance file
.vscode/settings.json	New VS Code workspace settings (hardcoded Windows paths)
scripts/generate_modules_manifest.ps1	Updated to add contentStatus field to generated manifests
scripts/regenerate_pnpm_lock.ps1	New PowerShell utility for pnpm lockfile regeneration
packages/ui/src/components/*/stories.tsx	Removes duplicate code from Storybook files

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

scripts/orchestrate_content.py

docs/ai/ai-toolkit-guidance.md

.github/workflows/orchestrate-test.yml

CHANGELOG.md

scripts/regenerate_pnpm_lock.ps1

.github/workflows/orchestrate-test.yml

.vscode/settings.json

scripts/orchestrate_content.py

Copilot · 2026-01-16T01:43:40Z

@73junito I've opened a new pull request, #181, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-16T01:44:02Z

@73junito I've opened a new pull request, #182, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-16T01:45:09Z

@73junito I've opened a new pull request, #183, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-16T01:45:31Z

@73junito I've opened a new pull request, #184, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-16T01:46:02Z

@73junito I've opened a new pull request, #185, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-16T01:46:38Z

@73junito I've opened a new pull request, #186, to work on those changes. Once the pull request is ready, I'll request review from you.

73junito

viewed

Copilot

Pull request overview

Copilot reviewed 36 out of 36 changed files in this pull request and generated 10 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

scripts/orchestrate_content.py

docs/ai/ai-toolkit-guidance.md

README.md

.github/workflows/orchestrate-test.yml

docs/ai/ai-toolkit-guidance.md

Copilot

Pull request overview

Copilot reviewed 36 out of 36 changed files in this pull request and generated 7 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

docs/ai/ai-toolkit-guidance.md

.github/workflows/orchestrate-test.yml

docs/ai/ai-toolkit-guidance.md

scripts/validate_no_stubs.py

scripts/bulk_set_content_status.py

Copilot · 2026-01-16T09:36:39Z

@73junito I've opened a new pull request, #189, to work on those changes. Once the pull request is ready, I'll request review from you.

73junito · 2026-01-17T17:19:27Z

@copilot open a new pull request to apply changes based on the comments in this thread

Copilot · 2026-01-17T17:19:37Z

@73junito I've opened a new pull request, #198, to work on those changes. Once the pull request is ready, I'll request review from you.

73junito

viewed

Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 36 out of 36 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

docs/ai/ai-toolkit-guidance.md

scripts/orchestrate_content.py

.github/workflows/orchestrate-test.yml

tests/test_orchestrate_content.py

…8-4409-b0cb-c2de0225020e Fix documentation duplication, CLI argument parsing, and workflow artifacts

Copilot · 2026-01-17T18:47:53Z

@73junito I've opened a new pull request, #208, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-17T18:48:10Z

@73junito I've opened a new pull request, #209, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-17T18:48:24Z

@73junito I've opened a new pull request, #210, to work on those changes. Once the pull request is ready, I'll request review from you.

73junito

viewed

…ture Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>

Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>

…8-4190-957d-deb987d34a81 Strengthen test validation for generated field structure in orchestrate_content tests

…e-4b0d-9f8e-27244d8db1ea Fix Ollama streaming response parser to handle newline-delimited JSON

…c-4a5c-833e-3f3275588d9e Fix Ollama API response parsing to use correct "response" field

73junito self-assigned this Jan 16, 2026

73junito added this to @73junito's autolearnpro Jan 16, 2026

Copilot AI review requested due to automatic review settings January 16, 2026 01:37

github-project-automation bot moved this to Todo in @73junito's autolearnpro Jan 16, 2026

Copilot started reviewing on behalf of 73junito January 16, 2026 01:37 View session

73junito commented Jan 16, 2026

View reviewed changes

github-actions bot added the contains-stubs label Jan 16, 2026

Copilot AI reviewed Jan 16, 2026

View reviewed changes

This was referenced Jan 16, 2026

Fix broken file structure in scripts/orchestrate_content.py #181

Merged

Fix broken file structure in scripts/orchestrate_content.py #182

Merged

This was referenced Jan 16, 2026

Fix orchestrator failure detection in GitHub Actions workflow #183

Merged

Fix workflow failure on push events due to missing pull_request context #184

Merged

Copilot AI mentioned this pull request Jan 16, 2026

Fix PowerShell script: use Write-Output and remove non-existent script reference #185

Merged

4 tasks

Copilot AI mentioned this pull request Jan 16, 2026

Fix Python 3 syntax error in orchestrate_content.py #186

Merged

4 tasks

73junito requested a review from Copilot January 16, 2026 01:47

73junito commented Jan 16, 2026

View reviewed changes

Copilot started reviewing on behalf of 73junito January 16, 2026 01:48 View session

Copilot AI reviewed Jan 16, 2026

View reviewed changes

73junito assigned Copilot Jan 16, 2026

73junito requested a review from Copilot January 16, 2026 09:13

Copilot started reviewing on behalf of 73junito January 16, 2026 09:14 View session

Copilot AI reviewed Jan 16, 2026

View reviewed changes

Copilot AI mentioned this pull request Jan 16, 2026

[WIP] WIP address feedback from PR #180 review comments #189

Merged

Copilot AI added 2 commits January 16, 2026 16:00

Initial plan

6eaa4cd

Initial plan

75073b7

73junito force-pushed the merge/pr-158-local branch from 5e78bf2 to 75073b7 Compare January 16, 2026 22:00

Initial plan

3392d2b

Copilot AI mentioned this pull request Jan 17, 2026

Fix documentation duplication, CLI argument parsing, and workflow artifacts #198

Merged

5 tasks

73junito commented Jan 17, 2026

View reviewed changes

73junito marked this pull request as ready for review January 17, 2026 17:20

73junito requested a review from Copilot January 17, 2026 17:20

Copilot started reviewing on behalf of 73junito January 17, 2026 17:21 View session

Fix review feedback: remove duplication, add CLI args, fix workflow

1e60f74

Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>

Copilot AI reviewed Jan 17, 2026

View reviewed changes

73junito and others added 2 commits January 17, 2026 12:45

Merge pull request #198 from 73junito/copilot/sub-pr-180-2cfe6ae7-313…

cacbebe

…8-4409-b0cb-c2de0225020e Fix documentation duplication, CLI argument parsing, and workflow artifacts

Initial plan

af5aa23

Copilot AI mentioned this pull request Jan 17, 2026

Strengthen test validation for generated field structure in orchestrate_content tests #208

Merged

3 tasks

Initial plan

c89eba9

Copilot AI mentioned this pull request Jan 17, 2026

Fix Ollama streaming response parser to handle newline-delimited JSON #209

Merged

4 tasks

Initial plan

d5ccfbd

Copilot AI mentioned this pull request Jan 17, 2026

Fix Ollama API response parsing to use correct "response" field #210

Merged

2 tasks

73junito commented Jan 17, 2026

View reviewed changes

Copilot AI and others added 6 commits January 17, 2026 18:50

Strengthen test_orchestrate_persist to validate generated field struc…

254bbad

…ture Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>

Fix streaming response parsing to properly parse newline-delimited JSON

db931b2

Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>

Refactor streaming parser to eliminate code duplication

bff05f7

Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>

Merge pull request #208 from 73junito/copilot/sub-pr-180-fade5a8c-670…

2cce672

…8-4190-957d-deb987d34a81 Strengthen test validation for generated field structure in orchestrate_content tests

Merge pull request #209 from 73junito/copilot/sub-pr-180-e91ec357-49f…

f0fd0a8

…e-4b0d-9f8e-27244d8db1ea Fix Ollama streaming response parser to handle newline-delimited JSON

Merge pull request #210 from 73junito/copilot/sub-pr-180-1e6009bd-c71…

aa2f939

…c-4a5c-833e-3f3275588d9e Fix Ollama API response parsing to use correct "response" field

Merge/pr 158 local #180

Are you sure you want to change the base?

Merge/pr 158 local #180

Conversation

73junito commented Jan 16, 2026

Summary

Checklist

Testing

Related

Uh oh!

73junito left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Jan 16, 2026

Uh oh!

Copilot AI commented Jan 16, 2026

Uh oh!

Copilot AI commented Jan 16, 2026

Uh oh!

Copilot AI commented Jan 16, 2026

Uh oh!

Copilot AI commented Jan 16, 2026

Uh oh!

Copilot AI commented Jan 16, 2026

Uh oh!

73junito left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Jan 16, 2026

Uh oh!

73junito commented Jan 17, 2026

Uh oh!

Copilot AI commented Jan 17, 2026

Uh oh!

73junito left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!