fix: improve skill detection (#2079)

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

Author: amitksingh1490

crates/forge_services/src/tool_services/skill.rs

@@ -5,11 +5,11 @@ use forge_app::SkillFetchService;
 use forge_domain::Skill;
 use tokio::sync::OnceCell;
 
-/// Fetches detailed information about a specific skill. Use this tool to load
-/// skill content and instructions when you need to understand how to perform a
-/// specialized task. Skills provide domain-specific knowledge, workflows, and
-/// best practices. Only invoke skills that are listed in the available skills
-/// section. Do not invoke a skill that is already active.
+/// Loads specialized skills for specific task types. ALWAYS check the
+/// available_skills list when a user request matches a skill's description or
+/// trigger conditions. Skills provide domain-specific workflows and must be
+/// invoked BEFORE attempting the task directly. Only invoke skills listed in
+/// available_skills. Do not invoke a skill that is already active.
 pub struct ForgeSkillFetch<R> {
     repository: Arc<R>,
     cache: OnceCell<Vec<Skill>>,

templates/forge-partial-skill-instructions.md

@@ -1,6 +1,6 @@
 ## Skill Instructions:
 
-When users ask you to perform tasks, check if any of the available skills below can help complete the task more effectively. Skills provide specialized capabilities and domain knowledge.
+**CRITICAL**: Before attempting any task, ALWAYS check if a skill exists for it in the available_skills list below. Skills are specialized workflows that must be invoked when their trigger conditions match the user's request.
 
 How skills work: