fix(ci): add dead_code annotations to skill-based prompt API

The skill-based prompt functions are public API designed for future
integration but are not yet used outside of tests. Add #[allow(dead_code)]
annotations to suppress warnings while preserving the API surface.

Also applies rustfmt formatting fixes.

Author: echobt

src/cortex-engine/src/session/prompt.rs

@@ -25,6 +25,7 @@ pub(crate) const BASE_PROMPT_WITH_SKILLS: &str =
 ///
 /// When `true`, the system uses a minimal base prompt with skills loaded on-demand.
 /// When `false`, the full monolithic prompt is used.
+#[allow(dead_code)]
 pub const USE_SKILL_BASED_PROMPT: bool = true;
 
 /// Build the system prompt for the agent.
@@ -225,6 +226,7 @@ fn get_system_info() -> String {
 /// let skills = auto_detect_skills_from_message("Fix this bug and create a PR");
 /// let prompt = build_system_prompt_with_skills(&config, &skills);
 /// ```
+#[allow(dead_code)]
 pub fn build_system_prompt_with_skills(config: &Config, skills: &[&str]) -> String {
     // If skills mode is disabled and no skills specified, use monolithic prompt
     if !USE_SKILL_BASED_PROMPT && skills.is_empty() {
@@ -348,6 +350,7 @@ pub fn build_system_prompt_with_skills(config: &Config, skills: &[&str]) -> Stri
 /// assert!(prompt.contains("Git Operations Skill"));
 /// assert!(prompt.contains("Debugging Skill"));
 /// ```
+#[allow(dead_code)]
 pub fn inject_skills(base_prompt: &str, skills: &[&str]) -> String {
     if skills.is_empty() {
         return base_prompt.to_string();
@@ -383,6 +386,7 @@ pub fn inject_skills(base_prompt: &str, skills: &[&str]) -> String {
 ///
 /// Skills include YAML frontmatter for metadata, but we don't need it
 /// in the injected prompt.
+#[allow(dead_code)]
 fn strip_yaml_frontmatter(content: &str) -> &str {
     if !content.starts_with("---\n") {
         return content;
@@ -425,6 +429,7 @@ fn strip_yaml_frontmatter(content: &str) -> &str {
 /// assert!(skills.contains(&"file-operations"));
 /// assert!(skills.contains(&"code-quality"));
 /// ```
+#[allow(dead_code)]
 pub fn auto_detect_skills_from_message(message: &str) -> Vec<&'static str> {
     cortex_prompt_harness::prompts::get_recommended_skills(message)
 }
@@ -434,6 +439,7 @@ pub fn auto_detect_skills_from_message(message: &str) -> Vec<&'static str> {
 /// # Returns
 ///
 /// A slice of all available skill names.
+#[allow(dead_code)]
 pub fn available_skills() -> &'static [&'static str] {
     cortex_prompt_harness::prompts::AVAILABLE_SKILLS
 }
@@ -447,6 +453,7 @@ pub fn available_skills() -> &'static [&'static str] {
 /// # Returns
 ///
 /// `true` if the skill exists, `false` otherwise.
+#[allow(dead_code)]
 pub fn is_valid_skill(skill: &str) -> bool {
     cortex_prompt_harness::prompts::is_builtin_skill(skill)
 }

src/cortex-engine/src/tools/handlers/skill.rs

@@ -230,9 +230,8 @@ impl SkillLoader {
             return Ok(None);
         }
 
-        let content = get_builtin_skill(name).ok_or_else(|| {
-            CortexError::NotFound(format!("Built-in skill not found: {}", name))
-        })?;
+        let content = get_builtin_skill(name)
+            .ok_or_else(|| CortexError::NotFound(format!("Built-in skill not found: {}", name)))?;
 
         let (definition, markdown) = parse_skill_md(content)?;
 

src/cortex-prompt-harness/src/prompts/base_agent.rs

@@ -157,50 +157,122 @@ pub const SKILL_METADATA: &[SkillInfo] = &[
         name: "git",
         description: "Version control operations",
         keywords: &[
-            "git", "commit", "push", "pull", "merge", "branch", "pr", "pull request",
-            "rebase", "cherry-pick", "checkout", "stash", "diff", "log", "blame",
+            "git",
+            "commit",
+            "push",
+            "pull",
+            "merge",
+            "branch",
+            "pr",
+            "pull request",
+            "rebase",
+            "cherry-pick",
+            "checkout",
+            "stash",
+            "diff",
+            "log",
+            "blame",
         ],
     },
     SkillInfo {
         name: "code-quality",
         description: "Code standards and testing",
         keywords: &[
-            "lint", "test", "format", "style", "convention", "review", "refactor",
-            "clean", "quality", "coverage", "eslint", "pylint", "clippy", "prettier",
-            "jest", "pytest", "cargo test",
+            "lint",
+            "test",
+            "format",
+            "style",
+            "convention",
+            "review",
+            "refactor",
+            "clean",
+            "quality",
+            "coverage",
+            "eslint",
+            "pylint",
+            "clippy",
+            "prettier",
+            "jest",
+            "pytest",
+            "cargo test",
         ],
     },
     SkillInfo {
         name: "file-operations",
         description: "File handling best practices",
         keywords: &[
-            "create", "file", "write", "edit", "move", "rename", "delete", "copy",
-            "directory", "folder", "path", "backup",
+            "create",
+            "file",
+            "write",
+            "edit",
+            "move",
+            "rename",
+            "delete",
+            "copy",
+            "directory",
+            "folder",
+            "path",
+            "backup",
         ],
     },
     SkillInfo {
         name: "debugging",
         description: "Failure protocol and error handling",
         keywords: &[
-            "debug", "error", "fix", "bug", "crash", "exception", "trace", "stack",
-            "breakpoint", "investigate", "troubleshoot", "diagnose", "failing", "broken",
+            "debug",
+            "error",
+            "fix",
+            "bug",
+            "crash",
+            "exception",
+            "trace",
+            "stack",
+            "breakpoint",
+            "investigate",
+            "troubleshoot",
+            "diagnose",
+            "failing",
+            "broken",
         ],
     },
     SkillInfo {
         name: "security",
         description: "Security rules and secrets handling",
         keywords: &[
-            "security", "secret", "key", "token", "password", "credential", "auth",
-            "authentication", "authorization", "encrypt", "hash", "vulnerability",
-            "audit", "sensitive", "env", "environment variable",
+            "security",
+            "secret",
+            "key",
+            "token",
+            "password",
+            "credential",
+            "auth",
+            "authentication",
+            "authorization",
+            "encrypt",
+            "hash",
+            "vulnerability",
+            "audit",
+            "sensitive",
+            "env",
+            "environment variable",
         ],
     },
     SkillInfo {
         name: "planning",
         description: "Task decomposition and cognitive phases",
         keywords: &[
-            "plan", "design", "architect", "complex", "multi-step", "breakdown",
-            "decompose", "strategy", "roadmap", "milestone", "phase", "implement feature",
+            "plan",
+            "design",
+            "architect",
+            "complex",
+            "multi-step",
+            "breakdown",
+            "decompose",
+            "strategy",
+            "roadmap",
+            "milestone",
+            "phase",
+            "implement feature",
         ],
     },
 ];

src/cortex-prompt-harness/src/prompts/builtin_skills.rs

@@ -1574,11 +1574,23 @@ mod tests {
         ];
 
         for skill in skills {
-            assert!(skill.starts_with("---\n"), "Skill should start with YAML frontmatter");
-            assert!(skill.contains("\n---\n"), "Skill should have frontmatter end marker");
+            assert!(
+                skill.starts_with("---\n"),
+                "Skill should start with YAML frontmatter"
+            );
+            assert!(
+                skill.contains("\n---\n"),
+                "Skill should have frontmatter end marker"
+            );
             assert!(skill.contains("name:"), "Skill should have name field");
-            assert!(skill.contains("description:"), "Skill should have description field");
-            assert!(skill.contains("version:"), "Skill should have version field");
+            assert!(
+                skill.contains("description:"),
+                "Skill should have description field"
+            );
+            assert!(
+                skill.contains("version:"),
+                "Skill should have version field"
+            );
             assert!(skill.contains("tags:"), "Skill should have tags field");
         }
     }

src/cortex-prompt-harness/src/prompts/mod.rs

@@ -60,17 +60,17 @@ pub use base_agent::{
     AVAILABLE_SKILLS, CORTEX_BASE_PROMPT, CORTEX_BASE_PROMPT_WITH_SKILLS_PRELOADED,
     format_skill_loading_prompt, get_recommended_skills,
 };
+pub use builtin_skills::{
+    BUILTIN_SKILL_NAMES, SKILL_CODE_QUALITY, SKILL_DEBUGGING, SKILL_FILE_OPERATIONS, SKILL_GIT,
+    SKILL_PLANNING, SKILL_SECURITY, builtin_skill_count, get_builtin_skill, is_builtin_skill,
+    list_builtin_skills,
+};
 pub use core::{
     CORTEX_MAIN_PROMPT, CortexPromptBuilder, SECTION_ANTI_PATTERNS, SECTION_CODE_DISCIPLINE,
     SECTION_COGNITIVE_ARCHITECTURE, SECTION_FAILURE_PROTOCOL, SECTION_HEADER, SECTION_NAMES,
     SECTION_OUTPUT_FORMAT, SECTION_PRIME_DIRECTIVES, SECTION_QUALITY_CHECKPOINTS,
     SECTION_RESPONSE_PATTERNS, SECTION_TOOLKIT, TUI_SYSTEM_PROMPT_TEMPLATE,
 };
-pub use builtin_skills::{
-    BUILTIN_SKILL_NAMES, SKILL_CODE_QUALITY, SKILL_DEBUGGING, SKILL_FILE_OPERATIONS, SKILL_GIT,
-    SKILL_PLANNING, SKILL_SECURITY, builtin_skill_count, get_builtin_skill, is_builtin_skill,
-    list_builtin_skills,
-};
 pub use tasks::{COMPACTION_PROMPT, SUMMARIZATION_PROMPT};
 pub use top_agent::{
     TOP_AGENT_SECTION_NAMES, TOP_AGENT_SYSTEM_PROMPT, TopAgentPresets, TopAgentPromptBuilder,