chore(skills): remove builtin aionui skills (#513)

## Summary
- Remove the auto-injected builtin `aionui-skills` marketplace skill
from the embedded builtin skill corpus.
- Assert `/api/skills/builtin-auto` no longer returns `aionui-skills`.
- Lower embedded auto-inject count expectations from 4 to 3 while
leaving AionUi Butler skills unchanged.

## Test Plan
- `cargo fmt --all -- --check`
- `cargo test -p aionui-app --test skills_builtin_e2e`
- `cargo test -p aionui-extension
skill_service::tests::embedded_lists_auto_inject_from_corpus -- --exact`
- `cargo clippy -p aionui-app -p aionui-extension -- -D warnings`
- `cargo test --workspace`

Note: `just push` was started after the commit, but was interrupted at
user request because the full workspace test had already completed
successfully.

Co-authored-by: zk <>

Author: kaizhou-lab

crates/aionui-app/assets/builtin-skills/auto-inject/aionui-skills/SKILL.md

@@ -120,7 +120,12 @@ async fn builtin_auto_lists_entries_from_embedded_corpus() {
     let json = body_json(resp).await;
     assert_eq!(json["success"], true);
     let arr = json["data"].as_array().unwrap();
-    assert!(arr.len() >= 4, "expected ≥4 auto-inject entries, got {}", arr.len());
+    assert!(arr.len() >= 3, "expected ≥3 auto-inject entries, got {}", arr.len());
+    let names: Vec<&str> = arr.iter().filter_map(|item| item["name"].as_str()).collect();
+    assert!(
+        !names.contains(&"aionui-skills"),
+        "aionui-skills should not be shipped as an auto-inject builtin skill: {names:?}",
+    );
     for item in arr {
         assert!(item["name"].is_string());
         assert!(item["description"].is_string());

crates/aionui-app/tests/skills_builtin_e2e.rs

@@ -2154,7 +2154,7 @@ mod tests {
         let paths = make_embedded_paths(tmp.path()).await;
 
         let autos = list_builtin_auto_skills(&paths).await.unwrap();
-        assert!(autos.len() >= 4, "expected ≥4 auto-inject entries, got {}", autos.len());
+        assert!(autos.len() >= 3, "expected ≥3 auto-inject entries, got {}", autos.len());
         for item in &autos {
             assert!(
                 item.location.starts_with("auto-inject/"),