Back to Signal Feed
CodeTracked since May 19, 2026

Fix skill manager import name so skill operations stop failing

A broken import in `manager.py` referenced `get_default_skill_directories`, which does not exist in `discovery.py`; this PR changed it to `get_default_skill_dirs`, restoring normal execution for `create_skill`, `edit_skill`, `delete_skill`, `patch_skill`, and `write_skill_file`.

MervinPraison/PraisonAImanager.pydiscovery.pyget_default_skill_dirs

What Happened

  • A broken import in `manager.py` referenced `get_default_skill_directories`, which does not exist in `discovery.py`; this PR changed it to `get_default_skill_dirs`, restoring normal execution for `create_skill`, `edit_skill`, `delete_skill`, `patch_skill`, and `write_skill_file`.
  • A broken import in `manager.py` referenced `get_default_skill_directories`, which does not exist in `discovery.py`; this PR changed it to `get_default_skill_dirs`, restoring normal execution for `create_skill`, `edit_skill`, `delete_skill`, `patch_skill`, and `write_skill_file`.
  • 1 evidence item attached for review.

What is Different

Before

Scattered source updates, isolated context, and manual follow-up across multiple feeds.

Now

Corrected a concrete runtime bug by aligning the imported helper symbol in `manager.py` with its real definition in `discovery.py`, which removes the ImportError barrier for skill management execution.

Why Track This

Why It Matters

Developers using PraisonAI skill workflows can again run core operations like creating, editing, deleting, patching, and writing skills without the process stopping on import, so automation and manual skill-management flows stay usable instead of failing early. The fix is a correctness fix in module wiring: a function-name mismatch between files had been breaking `SkillManager` entrypoints and making repeated skill operations unreliable. Continue to watch for similarly mismatched imports in adjacent utility modules, especially if discovery-related symbols are renamed again.

Impact

Developers using PraisonAI skill workflows can again run core operations like creating, editing, deleting, patching, and writing skills without the process stopping on import, so automation and manual skill-management flows stay usable instead of failing early. The fix is a correctness fix in module wiring: a function-name mismatch between files had been breaking `SkillManager` entrypoints and making repeated skill operations unreliable. Continue to watch for similarly mismatched imports in adjacent utility modules, especially if discovery-related symbols are renamed again.

What To Watch Next

  • Watch whether MervinPraison/PraisonAI becomes a repeated pattern.
  • Track follow-up changes around AI IDE.
  • Compare future signals against this evidence trail.
  • Re-check risk flags: unchecked_import_name_mismatches, missing_import_smoke_tests_for_skill_manager.
Open Topic TimelineOpen Technical EventOpen Original Sourceunchecked_import_name_mismatches / missing_import_smoke_tests_for_skill_manager / future_discovery_symbol_renames_without_updates

Supporting Evidence

GITHUB PULL REQUESTHigh Trust

MervinPraison/PraisonAI PR #1687: fix: correct import name in skills manager (fixes #1683)

The patch replaces a non-existent import (`get_default_skill_directories`) with the existing function name (`get_default_skill_dirs`), resolving ImportError in skill-management code paths.