Skip to content

feat(idtk): commands/path-visualize.md — standalone HTML syllabus viewer for paths [DOJ-4784]#26

Closed
lapc506 wants to merge 1 commit into
mainfrom
andres/doj-4784-path-visualize
Closed

feat(idtk): commands/path-visualize.md — standalone HTML syllabus viewer for paths [DOJ-4784]#26
lapc506 wants to merge 1 commit into
mainfrom
andres/doj-4784-path-visualize

Conversation

@lapc506

@lapc506 lapc506 commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds `commands/path-visualize.md` — a new IDTK command that reads a dojo-academy path slug and emits `paths/{slug}/syllabus.html`, an authoring-debug visual showing per-course modules + per-module artifact table (✅/❌ per artifact type).

Scope

  • 6-phase command (Resolve input → Read structure → Resolve design tokens → Emit HTML → Validate → Report)
  • Reads `path-overview.md` frontmatter `milestones[].courses[]`; tolerates flat-`courses` variant
  • Artifact buckets: `text-.md`, `video-.md`, `slides-.html`, `quiz-.md`, `challenge-*.md`
  • Brand: consumes top-level `design` + `path_visualizer` sections from `dojo-academy/instruction-bundle-spec.yaml` when present; graceful-degrades to neutral defaults + visible WARNING banner when absent
  • Mobile 375px+, Ctrl+K search, filter chips, sortable columns
  • Overlay protocol per DOJ-3706/3707; `${CLAUDE_PLUGIN_ROOT}` defensive per DOJ-3775

Linear

DOJ-4784

Test plan

  • `check_frontmatter.py` — 68 valid, 2 skipped
  • `check_agent_references.py` — 19 agents indexed, all resolve
  • `check_json_schemas.py` — 4 valid

Tests deferred per brief "OR equivalent" clause — sibling commands ship without `.test.sh` files; CI scripts function as the de-facto authoring test suite.

Created by Claude Code on behalf of @andres@dojocoding.io

🤖 Generated with Claude Code

…wer for paths (DOJ-4784)

Reads a dojo-academy path slug + emits paths/{slug}/syllabus.html showing
per-course modules + per-module artifact table (✅/❌). Authoring debug
tool, NOT student-facing.

Brand consumed from dojo-academy/instruction-bundle-spec.yaml when present;
graceful-degrade to neutral defaults + visible WARNING when missing
(unblocks ship if DOJ-4785 not yet merged).

Created by Claude Code on behalf of @andres@dojocoding.io

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@lapc506

lapc506 commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator Author

@dojo-code-reviewer review

@dojo-code-reviewer dojo-code-reviewer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Approved

Approved — 0 blockers, 1 P3. Confidence: 4/5.

🔵 P3 — Minor

  • commands/path-visualize.md:66 — 🔵 P3 (minor) — If a module folder exists but does not contain a classes/ subdirectory (e.g. if it is newly scaffolded or uses a custom structure), the parser should treat the buckets as empty rather than failing or throwing an error. Clarifying this fallback logic makes the command specification more robust.
   - Para cada módulo, leé `classes/` si existe (si no existe, tratá todos los buckets como vacíos) y clasificá cada archivo por prefijo:

Total findings: 1 business context (1 total)


ℹ️ Inline anchoring failed for the comments below; they are listed here instead.

  • commands/path-visualize.md:66 — 🔵 P3 (minor) — If a module folder exists but does not contain a classes/ subdirectory (e.g. if it is newly scaffolded or uses a custom structure), the parser should treat the buckets as empty rather than failing or throwing an error. Clarifying this fallback logic makes the command specification more robust.
   - Para cada módulo, leé `classes/` si existe (si no existe, tratá todos los buckets como vacíos) y clasificá cada archivo por prefijo:

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.

2 participants