-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
enhancementNew feature or requestNew feature or request
Description
Feature Request: Teaching Config Consolidation
Date: 2026-01-27
Updated: 2026-02-03 (narrowed scope — lesson plan extraction shipped in v6.1.0, config rename dropped)
Priority: Medium
Effort: Small (~1-2 hours)
What's Already Done (v6.1.0) ✅
- ✅ Lesson plan extraction —
teach migrate-configseparates weeks into.flow/lesson-plans.yml - ✅ Single-file lesson plan loading —
_teach_load_lesson_plan()reads from.flow/lesson-plans.ymlwith fallback to embedded weeks - ✅ Backward compatibility — Old embedded format still works with deprecation warning
- ✅ teach plan CRUD — Full create/list/show/edit/delete on
.flow/lesson-plans.yml
Dropped ❌
- ❌ Config file rename — Both flow-cli (~14 refs) and Scholar (~30+ refs in source, tests, docs) use
.flow/teach-config.yml. Renaming toconfig-teach.ymlwould be a breaking change across both repos with no functional benefit. Keeping current name.
Remaining Work
1. Merge teaching style into teach-config.yml
Currently teaching style lives in .claude/teaching-style.local.md (markdown frontmatter), read by Scholar's style-loader.js (line 153). Add support for a teaching_style: section directly in .flow/teach-config.yml:
# .flow/teach-config.yml
course:
name: "STAT 545"
teaching_style:
pedagogical_approach:
primary: "problem-based"
notation_conventions:
expectation: "\\E{}"
command_overrides:
slides:
style: "rigorous"flow-cli side:
New helpers needed:
_teach_get_style() # Read teaching_style.<key> from teach-config.yml
_teach_get_command_override() # Read command_overrides.<cmd>.<key>teach doctor should report teaching style source location.
Scholar side (see #299):
Update loadCourseStyle() in style-loader.js to check .flow/teach-config.yml first, fall back to .claude/teaching-style.local.md.
2. Schema update
Add teaching_style and command_overrides to lib/templates/teaching/teach-config.schema.json.
Acceptance Criteria
-
teaching_style:section readable from.flow/teach-config.yml -
.claude/teaching-style.local.mdstill works as fallback -
_teach_get_style()and_teach_get_command_override()helpers exist - Schema validates new sections
-
teach doctorreports teaching style source
Related
- feat: Scholar plugin config consolidation - read from .flow/config-teach.yml #299 — Scholar plugin side (blocked on this issue)
- feat(teach): Add
teach dashboardsubcommand for dynamic website content #275 — teach dashboard (independent)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request