Skip to content

chore: Gandalf rename, autograder fixes, phrase cleanup, and doc currency update#206

Open
accesswatch wants to merge 4 commits into
mainfrom
chore/gandalf-consistency-transcript-sprint
Open

chore: Gandalf rename, autograder fixes, phrase cleanup, and doc currency update#206
accesswatch wants to merge 4 commits into
mainfrom
chore/gandalf-consistency-transcript-sprint

Conversation

@accesswatch
Copy link
Copy Markdown
Collaborator

@accesswatch accesswatch commented May 14, 2026

Summary

This PR completes a major content sprint across the entire curriculum and podcast corpus.

Changes

Gandalf Bot Rename (48 files)

  • Renames Aria to Gandalf across all curriculum docs, challenge content, admin guides, and podcast scripts
  • Preserves @aria-BOT\ backward-compat alias in comment-responder.js for the transition period
  • Updates SVG captions, challenge content review, qa-bundle mirrors, and all workflow descriptions

Issue #204 - Stale Autograder References (6 scripts)

  • Fixes cc-07, cc-10, cc-14, cc-16, cc-bonus-a, ep48
  • Removes point-based scoring language (10/10/0/15 point references)
  • Replaces 'autograded check' with Gandalf validation workflow language
  • Preserves autograder-template.yml (Ch14) and autograder-capstone.yml (Ch16) which still run

Podcast Template Phrase Cleanup (75 scripts, -1,782 lines)

  • Removes ~1,782 injected template phrases across all 75 podcast scripts (chapters, challenges, appendices)
  • Strips standalone filler sentences, prefix scaffolding, and Jamie-only filler blocks
  • Segments.json regenerated from cleaned scripts without touching the full regeneration pipeline

Documentation Currency - Appendix K + Appendix L

Research findings from VS Code v1.97-v1.101 and GitHub Copilot docs, verified June 2026:

  • Fix billing date: usage-based billing (premium requests) launched June 18, 2025
  • Fix Free plan limits: 2,000 completions + 50 premium requests/month (was '50 completions, 2K tokens')
  • Mark @workspace\ as superseded by #codebase; add #githubRepo\ variable (v1.100)
  • Fix Chat modes: unified view is GA since v1.99; Edit mode is NOT deprecated
  • Fix Appendix L authoritative source: Copilot Extensions docs now redirect to MCP
  • Update billing facts to past tense (transition occurred)

Files Changed

  • 48 curriculum docs (Gandalf rename)
  • 18 podcast scripts + docs (autograder + Gandalf fixes)
  • 77 files (phrase cleanup + Appendix K/L updates)
  • 148 transcript JSON files (regenerated from clean scripts)

Closes #204

GitHub Workshop Bot and others added 4 commits May 14, 2026 09:42
docs: rename Aria to Gandalf across all curriculum documentation
9632cca 
Completes the bot rename started in chore: rename workshop agent to
Gandalf with magic messaging (84a5a75). That commit renamed the
bot in scripts and workflows; this commit renames it across all
documentation, QA guides, accuracy-review backlog, and podcast
REGENERATION.md template.

- GO-LIVE-QA-GUIDE.md: 5 Aria references updated to Gandalf
- docs/05-working-with-issues.md: Aria -> Gandalf, fix she -> it pronoun
- docs/06-working-with-pull-requests.md: Aria -> Gandalf
- admin/qa-bundle mirrors updated to match
- admin/accuracy-review/findings-backlog.md: search term updated
- podcasts/REGENERATION.md: episode template updated
- Merge conflict resolved: upstream backward-compat aliases kept in
  comment-responder.js (@aria-BOT and @gandalf-bot both handled)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
fix: complete Gandalf rename and fix stale autograder references (Issue
b25daad 
#204)

Gandalf rename - remaining source files:
- REPOSITORY_SECURITY.md: 4 Aria references updated
- classroom/challenge-content-review.md: ~25 Aria refs + gender pronouns updated
- admin/qa-bundle/classroom/challenge-content-review.md: mirrored
- docs/images/learning-room-tree.svg: caption updated
- podcasts/build-challenge-bundles.js: template prompt updated
- podcasts/generate-draft-transcripts.js: proper noun regex updated
- podcasts/scripts/{cc-01,cc-03,cc-04,cc-05,ep00,ep04}: 43 bot name refs

Issue #204 - stale autograder language in podcast scripts:
- cc-07: autograded check replaced with Gandalf validation language
- cc-10: autograded check + raw YAML metadata removed
- cc-14: autograder check references updated to validation workflow
- cc-16: 8 autograder references replaced (Validation Report, no point values)
- cc-bonus-a: 7 autograder references replaced
- ep48-build-your-agent-capstone: 7 autograder references replaced

Removes: Classroom point-based scoring (10/10, 0/15), autograded section
labels, and listen-for-score instructions. Replaces with: Challenge 16
validation workflow checks, Gandalf PR Validation Report language.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
docs: update Appendix K/L with current Copilot/VS Code research findings
4c9acfb 
- Fix billing date: usage-based billing launched June 18, 2025 (not June 1, 2026)
- Fix Free plan limits: 2,000 completions + 50 premium requests/month (not 50 completions)
- Add #codebase as preferred alternative to superseded @workspace participant
- Add #githubRepo variable (added VS Code v1.100)
- Fix Chat modes description: unified view since v1.99, Edit mode is NOT deprecated
- Update Plan mode: remove '(Public preview)' label - it is now GA
- Fix Appendix L authoritative source: Copilot Extensions docs redirect to MCP
- Strip 1,782 template phrase lines from 75 podcast scripts (chapters, challenges, appendices)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
chore: regenerate transcript segments.json from phrase-cleaned scripts
8b6a5b6 
Updated all 75 segments.json files to reflect the phrase cleanup commit.
Used a parse-only script (not the full regeneration which overwrites .txt source).
Temporary helper podcasts/parse-scripts-to-segments.js removed.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 14, 2026 17:15
@accesswatch accesswatch requested a review from taylorarndt as a code owner May 14, 2026 17:15
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 14, 2026

Hi! I am Gandalf, your workshop agent. I bring a little magic to make this interaction clear, helpful, and fun. Here is my review of your pull request:

Report Status

Validation Needs Attention [ACTION REQUIRED]

Required Checks

  • Issue Reference

  • PR Description

  • File Location
    Changes should be in learning-room/ directory only
    Student contributions should modify files in learning-room/docs/ only. Other directories are workshop infrastructure.

Suggestions for Improvement

These are optional but will make your contribution even better:

  • Tables include proper headers
    Screen readers announce table headers, helping users navigate data.
  • Good use of code blocks for examples
    Code blocks help both sighted and non-sighted users understand technical content.
  • Tables include proper headers
    Screen readers announce table headers, helping users navigate data.

Accessibility Analysis

  • Broken Link
    Link points to non-existent file: ../../facilitator/FACILITATOR_GUIDE.md
    learning-room/.github/IMPLEMENTATION_GUIDE.md
    Fix: Verify the file path is correct or check if the file exists.

  • Heading Hierarchy Skip
    Heading jumps from H1 to H3. This breaks screen reader navigation.
    learning-room/.github/SETUP_AND_MAINTENANCE.md (line 142)
    Fix: Change "H3" to "H2" or add intermediate heading levels.

  • Heading Hierarchy Skip
    Heading jumps from H1 to H3. This breaks screen reader navigation.
    learning-room/.github/SETUP_AND_MAINTENANCE.md (line 167)
    Fix: Change "H3" to "H2" or add intermediate heading levels.

  • Heading Hierarchy Skip
    Heading jumps from H1 to H3. This breaks screen reader navigation.
    learning-room/.github/SETUP_AND_MAINTENANCE.md (line 185)
    Fix: Change "H3" to "H2" or add intermediate heading levels.

  • Heading Hierarchy Skip
    Heading jumps from H1 to H3. This breaks screen reader navigation.
    learning-room/.github/SETUP_AND_MAINTENANCE.md (line 326)
    Fix: Change "H3" to "H2" or add intermediate heading levels.

Learning Resources

Based on your changes, these guides might help:

Automated validation by Learning Room Bot. Gandalf generated this review. Last updated: 2026-05-14T17:15:35.731Z
Questions? Check PR Guidelines or mention @Facilitator

@github-actions github-actions Bot added accessibility documentation Improvements or additions to documentation needs-work labels May 14, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 14, 2026

Learning Room Validation Report

Great work. Your changes look good.

Resources

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 14, 2026

Peer Review Assigned

Hi @accesswatch! Your PR has been automatically paired with @taylorarndt for peer review.

For @taylorarndt:

This is a great opportunity to practice code review skills! Here's what to look for:

Content Quality:

  • Does the change accomplish what the issue describes?
  • Is the writing clear and helpful?
  • Are there any typos or grammar issues?

Accessibility:

  • Proper heading hierarchy (H1 → H2 → H3, no skips)?
  • Descriptive link text (not "click here")?
  • Alt text on images?
  • [TODO] markers removed?

Documentation:

  • Code blocks are properly formatted?
  • Tables have headers?
  • References/links work correctly?

Review Guidelines:

  • Be kind and constructive
  • Suggest improvements, don't just point out problems
  • Ask questions if something is unclear
  • Approve when ready or request changes with explanation

Resources:

Pairing by Learning Room Grouping Engine

Copilot AI reviewed May 14, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@taylorarndt taylorarndt Awaiting requested review from taylorarndt taylorarndt is a code owner

Assignees

No one assigned

Labels

accessibility documentation Improvements or additions to documentation needs-work

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Refresh Challenge Coach podcast transcripts after PR #202 / #203 (autograding + registration migrations)

2 participants

@accesswatch