Pr/150 #211
Open
Pr/150 #211
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ystems-ase-a5 humanized
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…in permissions Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Add test for skipping modules with contentStatus="humanized" - Add test for error handling when call_ollama raises exceptions - Add test for streaming mode with TTFT capture - Add test for modules.json dict format with 'modules' key - Add test for modules.json plain array format - Fix existing test to meet quality guard requirements Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>
Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>
Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>
…ate() functions Co-authored-by: 73junito <86015877+73junito@users.noreply.github.com>
…ules.json to satisfy stub validator
Exclude generated outputs and rubric scaffolding from stub checks. Skipped paths are logged for transparency; authored content remains enforced.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR introduces an AI-assisted content generation pipeline with guardrails to prevent stub/placeholder content from reaching production. The implementation includes orchestration scripts for Ollama-based content humanization, validation tools, comprehensive tests, CI workflows, and updates to course manifests marking them as humanized.
Changes:
- Content orchestration and validation infrastructure (Python scripts, tests, workflows)
- Multiple course
modules.jsonfiles updated withcontentStatus: "humanized" - Documentation for AI toolkit guidance and Ollama client benchmarks
Reviewed changes
Copilot reviewed 33 out of 33 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/orchestrate_content.py | Ollama content generation orchestration with quality checks and retry logic |
| scripts/validate_no_stubs.py | Validates content for stub markers and contentStatus compliance |
| scripts/mark_modules_humanized.py | Helper to batch-mark modules as humanized |
| scripts/bulk_set_content_status.py | Bulk updates for contentStatus across manifests |
| scripts/regenerate_pnpm_lock.ps1 | PowerShell script for PNPM lock regeneration |
| scripts/generate_modules_manifest.ps1 | Updated to add contentStatus field with default "stub" |
| tests/test_orchestrate_content.py | Comprehensive test suite for orchestration logic |
| docs/benchmarks/ollama-client-benchmark.md | Benchmark documentation for client language selection |
| docs/ai/ai-toolkit-guidance.md | Advisory guidance for AI-assisted development (contains duplicate content) |
| .github/workflows/orchestrate-test.yml | CI workflow for orchestrator testing |
| .github/workflows/block-stubs.yml | CI workflow to block PRs with stub content |
| content/courses/*/modules.json | Multiple course manifests updated with contentStatus and formatting |
| README.md | Updated with content standards and benchmark references |
| CONTRIBUTING.md | New file with minimal contribution guidelines |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 33 out of 33 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…plify generate_thumbnails for CI
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Describe the change and why it is needed.
Checklist
Testing
Describe how the change was tested locally and CI steps.
Related