learn_bash_from_session_data 1.0.5 → 1.0.7

package/bin/learn-bash.js

@@ -240,26 +240,29 @@ function checkPython() {
  */
 function openInBrowser(filePath) {
   const platform = os.platform();
+  const isWindows = platform === 'win32' ||
+    process.env.MSYSTEM != null ||
+    (process.env.OSTYPE && process.env.OSTYPE.includes('msys')) ||
+    isWSL();
   let cmd;
   let args;
 
-  switch (platform) {
-    case 'darwin':
-      cmd = 'open';
-      args = [filePath];
-      break;
-    case 'win32':
-      cmd = 'cmd';
-      args = ['/c', 'start', '', filePath];
-      break;
-    default:
-      // Linux and others
-      cmd = 'xdg-open';
-      args = [filePath];
+  if (platform === 'darwin') {
+    cmd = 'open';
+    args = [filePath];
+  } else if (isWindows) {
+    // Works from native Windows, MSYS, Git Bash, and WSL
+    cmd = process.env.COMSPEC || 'cmd.exe';
+    args = ['/c', 'start', '', filePath.replace(/\//g, '\\')];
+  } else {
+    cmd = 'xdg-open';
+    args = [filePath];
   }
 
   try {
-    spawn(cmd, args, { detached: true, stdio: 'ignore' }).unref();
+    const child = spawn(cmd, args, { detached: true, stdio: 'ignore' });
+    child.on('error', () => {});  // Suppress async spawn errors
+    child.unref();
     return true;
   } catch (e) {
     return false;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "learn_bash_from_session_data",
-  "version": "1.0.5",
-  "description": "Learn bash from your Claude Code sessions - extracts commands and generates interactive HTML lessons",
+  "version": "1.0.7",
+  "description": "Learn bash from your Claude Code sessions - extracts commands and generates interactive HTML lessons with 400+ commands, quizzes, and comprehensive coverage",
   "main": "bin/learn-bash.js",
   "bin": {
     "learn-bash": "bin/learn-bash.js",
@@ -16,7 +16,11 @@
     "claude",
     "cli",
     "terminal",
-    "shell"
+    "shell",
+    "quiz",
+    "commands",
+    "interactive",
+    "education"
   ],
   "author": "",
   "license": "MIT",

package/scripts/html_generator.py

@@ -12,6 +12,16 @@ from pathlib import Path
 import html
 import json
 
+try:
+    from scripts.knowledge_base import COMMAND_DB, get_flags_for_command, get_command_info
+except ImportError:
+    try:
+        from knowledge_base import COMMAND_DB, get_flags_for_command, get_command_info
+    except ImportError:
+        COMMAND_DB = {}
+        def get_flags_for_command(cmd): return {}
+        def get_command_info(cmd): return None
+
 
 def _generate_html_impl(analysis_result: dict[str, Any], quizzes: list[dict[str, Any]]) -> str:
     """
@@ -376,7 +386,7 @@ def render_commands_tab(commands: list[dict]) -> str:
         # Syntax highlighted command
         highlighted = _syntax_highlight(cmd.get("full_command", ""))
 
-        # Flags breakdown
+        # Flags breakdown with descriptions
         flags = cmd.get("flags", [])
         flags_html = ""
         if flags:
@@ -384,9 +394,27 @@ def render_commands_tab(commands: list[dict]) -> str:
             for flag in flags:
                 flag_name = html.escape(flag.get("flag", ""))
                 flag_desc = html.escape(flag.get("description", ""))
-                flags_html += f'<li><code class="flag">{flag_name}</code> - {flag_desc}</li>'
+                if flag_desc:
+                    flags_html += f'<li><code class="flag">{flag_name}</code> <span class="flag-desc">{flag_desc}</span></li>'
+                else:
+                    flags_html += f'<li><code class="flag">{flag_name}</code></li>'
             flags_html += '</ul></div>'
 
+        # Subcommand description
+        subcommand_desc = cmd.get("subcommand_desc", "")
+        subcmd_html = ""
+        if subcommand_desc:
+            subcmd_html = f'<div class="subcmd-section"><span class="subcmd-label">Subcommand:</span> {html.escape(subcommand_desc)}</div>'
+
+        # Common patterns / examples from knowledge base
+        common_patterns = cmd.get("common_patterns", [])
+        patterns_html = ""
+        if common_patterns:
+            patterns_html = '<div class="patterns-section"><h5>Common Patterns:</h5><ul class="patterns-list">'
+            for pattern in common_patterns[:5]:
+                patterns_html += f'<li><code>{html.escape(pattern)}</code></li>'
+            patterns_html += '</ul></div>'
+
         # Output preview
         output_preview = cmd.get("output_preview", "")
         output_html = ""
@@ -418,7 +446,9 @@ def render_commands_tab(commands: list[dict]) -> str:
                                     <h5>Description:</h5>
                                     <p>{description}</p>
                                 </div>
+                                {subcmd_html}
                                 {flags_html}
+                                {patterns_html}
                                 {output_html}
                             </div>
                         </div>'''
@@ -512,13 +542,37 @@ def render_lessons_tab(categories: dict, commands: list[dict]) -> str:
             complexity = cmd.get("complexity", "simple")
             highlighted = _syntax_highlight(cmd.get("full_command", ""))
 
+            # Get flags and patterns for this command from COMMAND_DB
+            cmd_flags = cmd.get("flags", [])
+            lesson_flags_html = ""
+            if cmd_flags:
+                lesson_flags_html = '<div class="lesson-flags"><strong>Flags used:</strong> '
+                flag_parts = []
+                for flag in cmd_flags:
+                    fname = html.escape(flag.get("flag", "") if isinstance(flag, dict) else str(flag))
+                    fdesc = html.escape(flag.get("description", "") if isinstance(flag, dict) else "")
+                    if fdesc:
+                        flag_parts.append(f'<code class="flag">{fname}</code> ({fdesc})')
+                    else:
+                        flag_parts.append(f'<code class="flag">{fname}</code>')
+                lesson_flags_html += ', '.join(flag_parts) + '</div>'
+
+            # Subcommand info
+            subcmd_desc = cmd.get("subcommand_desc", "")
+            lesson_subcmd = ""
+            if subcmd_desc:
+                lesson_subcmd = f'<div class="lesson-subcmd"><em>{html.escape(subcmd_desc)}</em></div>'
+
             cat_commands_html += f'''
                             <div class="lesson-command">
                                 <div class="lesson-command-header">
                                     <code class="cmd">{base_cmd}</code>
+                                    <span class="lesson-complexity complexity-{complexity}">{complexity}</span>
                                 </div>
                                 <pre class="syntax-highlighted">{highlighted}</pre>
                                 <p class="lesson-description">{description}</p>
+                                {lesson_subcmd}
+                                {lesson_flags_html}
                             </div>'''
 
         # Patterns observed
@@ -559,20 +613,17 @@ def render_lessons_tab(categories: dict, commands: list[dict]) -> str:
 def _get_category_concept(category: str) -> str:
     """Get concept overview for a category."""
     concepts = {
-        "File Management": "Commands for creating, copying, moving, deleting, and organizing files and directories in the filesystem.",
-        "Text Processing": "Tools for searching, filtering, transforming, and manipulating text content in files and streams.",
-        "System Administration": "Commands for managing system resources, processes, users, and system configuration.",
-        "Network": "Utilities for network communication, file transfer, and connectivity diagnostics.",
-        "Package Management": "Tools for installing, updating, and managing software packages on the system.",
-        "Version Control": "Git and other version control commands for tracking changes and collaborating on code.",
-        "Process Management": "Commands for viewing, controlling, and managing running processes.",
-        "User Management": "Tools for managing user accounts, permissions, and access control.",
-        "Disk Management": "Utilities for managing disk space, partitions, and storage devices.",
-        "Shell Scripting": "Built-in shell commands and constructs for scripting and automation.",
-        "Development": "Programming tools, compilers, interpreters, and development utilities.",
-        "Compression": "Tools for compressing, archiving, and extracting files.",
-        "Search": "Commands for finding files, searching content, and locating resources.",
-        "Permissions": "Tools for managing file permissions, ownership, and access control.",
+        "File System": "Commands for navigating, viewing, creating, and managing files and directories in the filesystem.",
+        "Text Processing": "Tools for viewing, searching, filtering, and transforming text content in files and streams.",
+        "Git": "Version control system commands for tracking changes, managing branches, and collaborating on code.",
+        "Package Management": "Package managers for installing, updating, and managing software dependencies across languages and platforms.",
+        "Process & System": "Commands for monitoring, managing, and controlling running processes and system resources.",
+        "Networking": "Commands for network operations, file transfers, remote access, and connectivity diagnostics.",
+        "Permissions": "Commands for managing file ownership, access permissions, and user/group administration.",
+        "Compression": "Commands for compressing, archiving, and extracting files using various algorithms.",
+        "Search & Navigation": "Commands for finding files, searching content, and navigating the filesystem efficiently.",
+        "Development": "Development tools for building, testing, compiling, and running code across languages.",
+        "Shell Builtins": "Built-in shell commands for scripting, variable management, and interactive shell use.",
     }
     return concepts.get(category, f"Commands related to {category.lower()} operations and utilities.")
 
@@ -580,20 +631,17 @@ def _get_category_concept(category: str) -> str:
 def _get_category_icon(category: str) -> str:
     """Get icon for a category."""
     icons = {
-        "File Management": "&#128193;",
+        "File System": "&#128193;",
         "Text Processing": "&#128196;",
-        "System Administration": "&#9881;",
-        "Network": "&#127760;",
+        "Git": "&#128202;",
         "Package Management": "&#128230;",
-        "Version Control": "&#128202;",
-        "Process Management": "&#9654;",
-        "User Management": "&#128100;",
-        "Disk Management": "&#128191;",
-        "Shell Scripting": "&#10095;",
-        "Development": "&#128187;",
-        "Compression": "&#128230;",
-        "Search": "&#128269;",
+        "Process & System": "&#9881;",
+        "Networking": "&#127760;",
         "Permissions": "&#128274;",
+        "Compression": "&#128230;",
+        "Search & Navigation": "&#128269;",
+        "Development": "&#128187;",
+        "Shell Builtins": "&#10095;",
     }
     return icons.get(category, "&#128204;")
 
@@ -1390,6 +1438,22 @@ def get_inline_css() -> str:
             color: #34a853;
         }
 
+        .flag-desc { color: #6c757d; margin-left: 4px; }
+        .flags-list li { margin: 4px 0; line-height: 1.5; }
+        .subcmd-section { background: #f0f7ff; padding: 8px 12px; border-radius: 6px; margin: 8px 0; border-left: 3px solid #4a9eff; }
+        .subcmd-label { font-weight: 600; color: #4a9eff; }
+        .patterns-section { margin: 8px 0; }
+        .patterns-section h5 { margin: 4px 0; color: #666; font-size: 0.85em; }
+        .patterns-list { list-style: none; padding: 0; margin: 4px 0; }
+        .patterns-list li { padding: 3px 0; }
+        .patterns-list code { background: #f5f5f5; padding: 2px 6px; border-radius: 3px; font-size: 0.85em; }
+        .lesson-flags { margin: 6px 0; font-size: 0.9em; color: #555; }
+        .lesson-subcmd { font-size: 0.9em; color: #4a9eff; margin: 4px 0; }
+        .lesson-complexity { font-size: 0.75em; padding: 2px 8px; border-radius: 10px; margin-left: 8px; }
+        .complexity-simple { background: #e8f5e9; color: #2e7d32; }
+        .complexity-intermediate { background: #fff3e0; color: #e65100; }
+        .complexity-advanced { background: #fce4ec; color: #c62828; }
+
         .syntax-highlighted .string {
             color: #ff6d01;
         }
@@ -2113,27 +2177,70 @@ def generate_html_files(
     # Transform commands to expected format
     formatted_commands = []
     for cmd in analyzed_commands:
-        # Convert flags to expected format (list of dicts with 'flag' and 'description')
+        cmd_str = cmd.get('command', '')
+        base_cmd = cmd.get('base_command', cmd_str.split()[0] if cmd_str else '')
+        complexity_score = cmd.get('complexity', 1)
+
+        # Look up COMMAND_DB info for this command
+        cmd_info = COMMAND_DB.get(base_cmd, {})
+        kb_flags = get_flags_for_command(base_cmd)
+
+        # Convert flags to expected format WITH descriptions from knowledge base
         raw_flags = cmd.get('flags', [])
         formatted_flags = []
         for f in raw_flags:
-            if isinstance(f, dict):
-                formatted_flags.append(f)
+            if isinstance(f, dict) and 'flag' in f:
+                # Already formatted - but enrich description if empty
+                flag_name = f.get('flag', '')
+                flag_desc = f.get('description', '')
+                if not flag_desc and flag_name in kb_flags:
+                    flag_desc = kb_flags[flag_name]
+                formatted_flags.append({'flag': flag_name, 'description': flag_desc})
             elif isinstance(f, str):
-                formatted_flags.append({'flag': f, 'description': ''})
-
-        cmd_str = cmd.get('command', '')
-        complexity_score = cmd.get('complexity', 1)
+                # Raw flag string - look up description from knowledge base
+                flag_desc = kb_flags.get(f, '')
+                # For combined flags like -la, try individual characters
+                if not flag_desc and len(f) > 2 and f.startswith('-') and not f.startswith('--'):
+                    char_descs = []
+                    for char in f[1:]:
+                        single = f'-{char}'
+                        if single in kb_flags:
+                            char_descs.append(f'{single}: {kb_flags[single]}')
+                    if char_descs:
+                        flag_desc = '; '.join(char_descs)
+                formatted_flags.append({'flag': f, 'description': flag_desc})
+
+        # Get educational description from COMMAND_DB if session description is empty/generic
+        session_desc = cmd.get('description', '')
+        kb_desc = cmd_info.get('description', '')
+        description = session_desc if session_desc else kb_desc
+
+        # Get subcommand info (for commands like git, docker, npm)
+        subcommands = cmd_info.get('subcommands', {})
+        # Try to identify the subcommand from the full command
+        cmd_tokens = cmd_str.split() if cmd_str else []
+        subcommand_desc = ''
+        if subcommands and len(cmd_tokens) > 1:
+            for token in cmd_tokens[1:]:
+                if not token.startswith('-') and token in subcommands:
+                    subcommand_desc = subcommands[token]
+                    break
+
+        # Get common patterns from COMMAND_DB
+        common_patterns = cmd_info.get('common_patterns', [])
 
         formatted_commands.append({
-            'base_command': cmd.get('base_command', cmd_str.split()[0] if cmd_str else ''),
+            'base_command': base_cmd,
             'full_command': cmd_str,
             'category': cmd.get('category', 'Other'),
             'complexity': complexity_to_label(complexity_score),
             'complexity_score': complexity_score,
             'frequency': frequency_map.get(cmd_str, 1),
-            'description': cmd.get('description', ''),
+            'description': description,
             'flags': formatted_flags,
+            'subcommand_desc': subcommand_desc,
+            'common_patterns': common_patterns[:6],
+            'args': cmd.get('args', []),
             'is_new': False,
         })