* Fix how-to guide builder edge case
* Resolve merge conflict
* fix: Handle None ai_analysis in how-to guide builder
Fixes NoneType AttributeError when ai_analysis is explicitly None.
The issue occurred when workflow.get('ai_analysis') returned None and
code attempted to call .get() without checking if it was None first.
Using 'workflow.get("ai_analysis", {})' only provides default {} when
the key is missing, not when the value is None. Changed to use
'workflow.get("ai_analysis") or {}' pattern which handles both cases.
Also added isinstance() type safety check in _extract_workflow_examples
to gracefully handle malformed data.
Changes:
- _group_by_ai_tutorial_group: ai_analysis = workflow.get() or {}
- _extract_workflow_examples: isinstance(ex, dict) check added
- _create_guide: ai_analysis = primary_workflow.get() or {} (2 locations)
- _generate_overview: ai_analysis = primary_workflow.get() or {}
All 34 how-to guide builder tests passing.
Closes#242
Co-authored-by: yashrastogi019-cell <yashrastogi019@gmail.com>
---------
Co-authored-by: yashrastogi019-cell <yashrastogi019@gmail.com>
Thanks @franklegolasyoung for the excellent work on the core fixes for issues #267, #242, and #260! 🙏
Your comprehensive approach to fixing PDF processing, expanding workflow detection, and improving the Chinese README documentation is much appreciated. I've added code quality fixes and comprehensive tests to ensure everything passes CI.
All 1266+ tests are now passing, and the issues are resolved! 🎉
- Create src/skill_seekers/_version.py as single source of truth
- Read version dynamically from pyproject.toml at runtime
- Update all __init__.py files to import from _version module
- Add tomli dependency for Python <3.11 (built-in tomllib for 3.11+)
- Remove hardcoded version duplicates (2.7.2 in 3 files)
- Fixes version mismatch: pyproject.toml (2.7.4) vs __init__.py (2.7.2)
Benefits:
- Single place to update version (pyproject.toml)
- No more version mismatches across files
- Automatic version consistency
- Works across Python 3.10-3.13
Before:
- pyproject.toml: 2.7.4
- src/skill_seekers/__init__.py: 2.7.2
- src/skill_seekers/cli/__init__.py: 2.7.2
- src/skill_seekers/mcp/__init__.py: 2.7.2
After:
- pyproject.toml: 2.7.4 (single source of truth)
- All other files: import from _version.py
- Remove SPYKE-related client documentation files
- Fix critical ruff linter errors:
- Remove unused 'os' import in test_analyze_e2e.py
- Remove unused 'setups' variable in test_test_example_extractor.py
- Prefix unused output_dir parameter in codebase_scraper.py
- Fix import sorting in test_integration.py
- Update CHANGELOG.md with comprehensive PR #272 feature documentation
These changes were part of PR #272 cleanup but didn't make it into the squash merge.
Complete implementation of C3.9, granular AI enhancement control, performance optimizations, and bug fixes.
Features:
- C3.9 Project Documentation Extraction (markdown files)
- Granular AI enhancement control (--enhance-level 0-3)
- C# test extraction support
- 6-12x faster LOCAL mode with parallel execution
- Auto-enhancement UX improvements
- LOCAL mode fallback for all AI enhancements
Bug Fixes:
- C# language support
- Config type field compatibility
- LocalSkillEnhancer import
Documentation:
- Updated CHANGELOG.md
- Updated CLAUDE.md
- Removed client-specific files
Tests: All 1,257 tests passing
Critical linter errors: Fixed
- Scan ALL .md files in project (README, docs/, etc.)
- Smart categorization by folder/filename (overview, architecture, guides, etc.)
- Processing depth: surface=raw copy, deep=parse+summarize, full=AI-enhanced
- AI enhancement at level 2+ adds topic extraction and cross-references
- New "Project Documentation" section in SKILL.md with summaries
- Output to references/documentation/ organized by category
- Default ON, use --skip-docs to disable
- Add skip_docs parameter to MCP scrape_codebase_tool
- Add 15 new tests for markdown documentation features
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Levels:
- 0 (off): No AI enhancement (default)
- 1 (minimal): SKILL.md enhancement only (fast, high value)
- 2 (standard): SKILL.md + Architecture + Config enhancement
- 3 (full): Everything including patterns and test examples
--comprehensive and --enhance-level are INDEPENDENT:
- --comprehensive: Controls depth and features (full depth + all features)
- --enhance-level: Controls AI enhancement level
Usage examples:
skill-seekers analyze --directory . --enhance-level 1 # SKILL.md AI only
skill-seekers analyze --directory . --enhance # Same as level 1
skill-seekers analyze --directory . --comprehensive # All features, no AI
skill-seekers analyze --directory . --comprehensive --enhance-level 2 # All features + standard AI
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Increase default batch size from 5 to 20 patterns per CLI call
- Add parallel execution with 3 concurrent workers (configurable)
- Add ai_enhancement settings to config_manager:
- local_batch_size: patterns per Claude CLI call (default: 20)
- local_parallel_workers: concurrent CLI calls (default: 3)
- Expected speedup: 6-12x faster for large codebases
Config settings can be changed via:
skill-seekers config (coming soon) or editing ~/.config/skill-seekers/config.json
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
When no ANTHROPIC_API_KEY is set, ai_enhancer.py now falls back to LOCAL
mode (Claude Code CLI) instead of disabling AI enhancement entirely.
Changes to AIEnhancer base class:
- AUTO mode now falls back to LOCAL instead of disabling
- Added _check_claude_cli() to verify Claude CLI is available
- Added _call_claude_local() method using Claude Code CLI
- Refactored _call_claude() to dispatch to API or LOCAL mode
- 2 minute timeout per LOCAL call (reasonable for batch processing)
This affects all AI enhancements:
- Design pattern enhancement (C3.1) → LOCAL fallback ✅
- Test example enhancement (C3.2) → LOCAL fallback ✅
- Architectural analysis (C3.7) → LOCAL fallback ✅
Before (bad UX):
ℹ️ AI enhancement disabled (no API key found)
After (good UX):
ℹ️ No API key found, using LOCAL mode (Claude Code CLI)
✅ AI enhancement enabled (using LOCAL mode - Claude Code CLI)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
UX improvement: When running `skill-seekers analyze --enhance` or
`skill-seekers analyze --comprehensive`, the SKILL.md is now automatically
enhanced as the final step. No need for a separate `skill-seekers enhance`
command.
Changes:
- After analyze_main() completes successfully, check if --enhance or
--comprehensive was used
- If SKILL.md exists, automatically run SkillEnhancer in headless mode
- Use force=True to skip all prompts (consistent with --comprehensive UX)
- 10 minute timeout for large codebases
- Graceful fallback with retry instructions if enhancement fails
Before (bad UX):
skill-seekers analyze --directory /path --enhance
# Then separately:
skill-seekers enhance output/codebase/
After (good UX):
skill-seekers analyze --directory /path --enhance
# Everything done in one command!
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The previous implementation had a design flaw - it ran `claude prompt_file`
and expected Claude to magically create a JSON file in a temp directory.
This never worked because Claude CLI is interactive and doesn't auto-save.
Changes:
- Use `--dangerously-skip-permissions` flag to bypass permission prompts
- Create a dedicated temp directory for each enhancement session
- Embed absolute output file path in the prompt so Claude knows where to write
- Run Claude from the temp directory as working_dir
- Improved prompt with explicit Write tool instruction
- Better error handling and logging (file not found, JSON parse errors)
- Show settings preview in prompt for better AI context
The LOCAL mode now follows the same pattern as enhance_skill_local.py
which works correctly.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Bug fixes:
- Fix KeyError in config_enhancer.py where "config_type" was expected but
config_extractor saves as "type". Now supports both field names for
backward compatibility.
- Fix settings "value_type" vs "type" mismatch in the same file.
New features:
- Add C# support for regex-based test example extraction
- Add language alias mapping (C# -> csharp, C++ -> cpp)
- Enhanced C# patterns for NUnit, xUnit, MSTest test frameworks
- Support for mock patterns (NSubstitute, Moq)
- Support for Zenject dependency injection patterns
- Support for setup/teardown method extraction
Tests:
- Add 2 new C# test extraction tests (NUnit tests, mock patterns)
- All 1257 tests pass (165 skipped)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Implements Phase 1 of the codebase analysis UX improvement plan, making the
command discoverable and adding intuitive preset flags while maintaining 100%
backward compatibility.
New Features:
- Add 'analyze' subcommand to main CLI (skill-seekers analyze)
- Add --quick preset: Fast analysis (1-2 min, basic features only)
- Add --comprehensive preset: Full analysis (20-60 min, all features + AI)
- Add --enhance flag: Simple AI enhancement with auto-detection
- Improve help text with timing estimates and mode descriptions
Files Modified:
- src/skill_seekers/cli/main.py: Add analyze subcommand (lines 15, 273-311, 542-589)
- src/skill_seekers/cli/codebase_scraper.py: Add preset logic and improve help text
- tests/test_analyze_command.py: NEW - 20 comprehensive tests
- tests/test_cli_paths.py: Fix version check (2.7.0 -> 2.7.2)
- tests/test_package_structure.py: Fix 4 version checks (2.7.0 -> 2.7.2)
- README.md: Update examples to use 'analyze' command
- CLAUDE.md: Update examples to use 'analyze' command
Test Results:
- 81 tests related to Phase 1: ALL PASSING ✅
- 20 new tests for analyze command: ALL PASSING ✅
- Zero regressions introduced
- 100% backward compatibility maintained
Backward Compatibility:
- Old 'skill-seekers-codebase' command still works
- All existing flags (--depth, --ai-mode, --skip-*) still functional
- No breaking changes
Usage Examples:
skill-seekers analyze --directory . --quick
skill-seekers analyze --directory . --comprehensive
skill-seekers analyze --directory . --enhance
Fixes#262 (codebase UX issues)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Three critical UX improvements for custom config handling:
1. User config directory support:
- Added ~/.config/skill-seekers/configs/ to search path
- Users can now place custom configs in their home directory
- Path resolution order: exact path → ./configs/ → user config dir → API
2. Better error messages:
- Show all searched absolute paths when config not found
- Added get_last_searched_paths() function to track locations
- Clear guidance on where to place custom configs
3. Auto-create config.json:
- ConfigManager now creates config.json on first initialization
- Creates configs/ subdirectory for user custom configs
- Display shows custom configs directory path
Fixes reported by @melamers in issue #262 where:
- Config path shown by `skill-seekers config` didn't exist
- Unclear where to save custom configs
- Error messages didn't show exact paths searched
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Fixes#264
Users reported that preset configs (react.json, godot.json, etc.) were not
found after installing via pip/uv, causing immediate failure on first use.
Solution: Instead of bundling configs in the package, the CLI now automatically
fetches missing configs from the SkillSeekersWeb.com API.
Changes:
- Created config_fetcher.py with smart config resolution:
1. Check local path (backward compatible)
2. Check with configs/ prefix
3. Auto-fetch from SkillSeekersWeb.com API (new!)
- Updated doc_scraper.py to use ConfigValidator (supports unified configs)
- Added 15 comprehensive tests for auto-fetch functionality
User Experience:
- Zero configuration needed - presets work immediately after install
- Better error messages showing available configs from API
- Downloaded configs are cached locally for future use
- Fully backward compatible with existing local configs
Testing:
- 15 new unit tests (all passing)
- 2 integration tests with real API
- Full test suite: 1387 tests passing
- No breaking changes
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Merging with admin override due to known issues:
✅ **What Works**:
- GLM-4.7 Claude-compatible API support (correctly implemented)
- PDF scraper improvements (content truncation fixed, page traceability added)
- Documentation updates comprehensive
⚠️ **Known Issues (will be fixed in next commit)**:
1. Import bugs in 3 files causing UnboundLocalError (30 tests failing)
2. PDF scraper test expectations need updating for new behavior (5 tests failing)
3. test_godot_config failure (pre-existing, not caused by this PR - 1 test failing)
**Action Plan**:
Fixes for issues #1 and #2 are ready and will be committed immediately after merge.
Issue #3 requires separate investigation as it's a pre-existing problem.
Total: 36 failing tests, 35 will be fixed in next commit.
This hotfix resolves 4 critical bugs reported by users:
Issue #258: install command fails with unified_scraper
- Added --fresh and --dry-run flags to unified_scraper.py
- Updated main.py to pass both flags to unified scraper
- Fixed "unrecognized arguments" error
Issue #259 (Original): scrape command doesn't accept positional URL and --max-pages
- Added positional URL argument to scrape command
- Added --max-pages flag with safety warnings (>1000 pages, <10 pages)
- Updated doc_scraper.py and main.py argument parsers
Issue #259 (Comment A): Version shows 2.7.0 instead of actual version
- Fixed hardcoded version in main.py
- Now reads version dynamically from __init__.py
Issue #259 (Comment B): PDF command shows empty "Error: " message
- Improved exception handler in main.py to show exception type if message is empty
- Added proper error handling in pdf_scraper.py with context-specific messages
- Added traceback support in verbose mode
All fixes tested and verified with exact commands from issue reports.
Resolves: #258, #259
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Start development cycle for v2.8.0.
Version updated in 5 locations:
- pyproject.toml
- src/skill_seekers/__init__.py
- src/skill_seekers/cli/__init__.py
- src/skill_seekers/mcp/__init__.py
- src/skill_seekers/mcp/tools/__init__.py
All version numbers synchronized to prevent Issue #248.
[Unreleased] section in CHANGELOG.md ready for v2.8.0 changes.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Fixed version mismatch bug where hardcoded versions were out of sync
with pyproject.toml.
Updated version from 2.5.2 to 2.7.0 in:
- src/skill_seekers/__init__.py
- src/skill_seekers/cli/__init__.py
- src/skill_seekers/mcp/__init__.py
- src/skill_seekers/mcp/tools/__init__.py
Now skill-seekers --version correctly reports: 2.7.0
Fixes#248
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Fixed incorrect variable names in list comprehensions that were causing
NameError in CI (Python 3.11/3.12):
Critical fixes:
- tests/test_markdown_parsing.py: 'l' → 'link' in list comprehension
- src/skill_seekers/cli/pdf_extractor_poc.py: 'l' → 'line' (2 occurrences)
Additional auto-lint fixes:
- Removed unused imports in llms_txt_downloader.py, llms_txt_parser.py
- Fixed comparison operators in config files
- Fixed list comprehension in other files
All tests now pass in CI.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Fixes 2 failing tests in test_architecture_scenarios.py that were trying to
read from stdin during pytest execution, causing:
OSError: pytest: reading from stdin while output is captured!
Changes:
- Added 'interactive' parameter to UnifiedCodebaseAnalyzer.analyze() (defaults to True)
- Pass interactive flag through to _analyze_github() and GitHubThreeStreamFetcher
- Updated failing tests to pass interactive=False
Tests fixed:
- test_scenario_1_github_three_stream_fetcher
- test_scenario_1_unified_analyzer_github
The interactive parameter controls whether the code prompts the user for
input (e.g., 'Continue without token?'). Setting it to False prevents
input() calls, making the code safe for CI/CD and test environments.
All 1386 tests now pass.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
The code was still referencing `args.build_api_reference` which was
changed to `args.skip_api_reference` in v2.5.2 (opt-in to opt-out flags).
This caused the codebase analysis to fail at the end with:
AttributeError: 'Namespace' object has no attribute 'build_api_reference'
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Added find_configs_directory() to use same logic as API (api/configs_repo/official first, then configs/)
- Added list_all_configs() to display all 24 configs grouped by category with descriptions
- Updated CLI to support --all flag, making config argument optional when --all is used
- Added 2 new tests for --all flag functionality
- All 51 tests passing (51 passed, 1 skipped)
This enables users to discover all available preset configs without checking the API or filesystem directly.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Fixed bug where _get_language_stats expected Path objects but received
dictionaries from results['files'].
Root cause: results['files'] contains dicts with 'language' key, not Path objects
Solution: Changed function to extract language from dict instead of calling detect_language()
Before:
for file_path in files:
lang = detect_language(file_path) # ❌ file_path is dict, not Path
After:
for file_data in files:
lang = file_data.get('language', 'Unknown') # ✅ Extract from dict
Tested: Successfully generated SKILL.md for AstroValley (90 lines, 19 C# files)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
CRITICAL BUG FIX:
- Fixed documentation scraper overwriting list with dict
- Changed self.scraped_data['documentation'] = {...} to .append({...})
- Bug was breaking unified skill builder reference generation
AI ENHANCEMENT UPDATES:
- Added repo_id extraction in utils.py for multi-repo support
- Enhanced grouping by (source, repo_id) tuple in both enhancement files
- Added MULTI-REPOSITORY HANDLING section to AI prompts
- AI now correctly identifies and synthesizes multiple repos
CHANGES:
1. src/skill_seekers/cli/utils.py:
- _determine_source_metadata() now returns (source, confidence, repo_id)
- Extracts repo_id from codebase_analysis/{repo_id}/ paths
- Added repo_id field to reference metadata dict
2. src/skill_seekers/cli/enhance_skill_local.py:
- Group references by (source_type, repo_id) instead of just source_type
- Display repo identity in prompt sections
- Detect multiple repos and add explicit guidance to AI
3. src/skill_seekers/cli/enhance_skill.py:
- Same grouping and display logic as local enhancement
- Multi-repository handling section added
4. src/skill_seekers/cli/unified_scraper.py:
- FIX: Documentation scraper now appends to list instead of overwriting
- Added source_id, base_url, refs_dir to documentation metadata
- Update refs_dir after moving to cache
TESTING:
- All 57 tests passing (unified, C3, utilities)
- Single-source verified: httpx comprehensive (219→749 lines after enhancement)
- Multi-source verified: encode/httpx + encode/httpcore (523 lines)
- AI enhancement working: Professional output with source attribution
QUALITY:
- Enhanced httpx SKILL.md: 749 lines, 19KB, A+ quality
- Source attribution working correctly
- Multi-repo synthesis transparent and accurate
- Reference structure clean and organized
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Resolved conflicts between router quality improvements and multi-source
synthesis architecture:
1. **unified_skill_builder.py**:
- Updated _generate_architecture_overview() signature to accept github_data
- Ensures GitHub metadata is available for enhanced router generation
2. **test_c3_integration.py**:
- Updated test data structure to multi-source list format
- Tests now properly mock github data for architecture generation
- All 8 C3 integration tests passing
**Test Results**:
- ✅ All 8 C3 integration tests pass
- ✅ All 26 unified tests pass
- ✅ All 116 GitHub-related tests pass
- ✅ All 62 multi-source architecture tests pass
The changes maintain backward compatibility while enabling router skills
to leverage GitHub insights (issues, labels, metadata) for better quality.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
PROBLEM:
AI enhancement was running Claude Code but SKILL.md was never updated.
Users saw "Claude finished but SKILL.md was not updated" error.
ROOT CAUSE:
Claude CLI was called with invalid --yes flag (doesn't exist).
Permission checks prevented file modifications from nested Claude sessions.
THE FIX:
1. Removed invalid --yes flag
2. Added --dangerously-skip-permissions flag to bypass ALL permission checks
3. Added explicit save instructions in prompt
4. Added debug output showing before/after file stats
CHANGES IN enhance_skill_local.py:
Line 614: Changed subprocess command
- Before: ['claude', '--yes', '--dangerously-skip-permissions', prompt_file]
- After: ['claude', '--dangerously-skip-permissions', prompt_file]
Lines 363-377: Enhanced prompt with explicit save instructions
- Added "You MUST save" language
- Added "This is NOT a read-only task" clarification
- Added "Even if running from within another Claude Code session" permission
- Added verification requirements
Lines 644-654: Enhanced debug output
- Shows before/after mtime and size
- Displays last 20 lines of Claude output
- Helps identify what went wrong
VERIFICATION:
Tested on output/httpx/:
- Before: 219 lines, 5,582 bytes
- After: 702 lines, 21,377 bytes (+283% size, +221% lines)
- Enhancement time: 152.8 seconds
- Status: ✅ SUCCESS - File updated correctly
IMPACT:
✅ AI enhancement now works automatically
✅ No more "file not updated" errors
✅ SKILL.md properly expands from 200 to 700+ lines
✅ Rich content with real examples from references
✅ Works even when called from within Claude Code session
The --dangerously-skip-permissions flag allows Claude Code to modify
files without permission prompts, essential for automated workflows.
🚨 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
