fix: address PR review feedback

- Add skills list refresh after deleteSkill to prevent race condition
- Fix frontend validation regex to match backend spec (no consecutive/trailing hyphens)

Author: Sannidhya

src/core/webview/webviewMessageHandler.ts

@@ -3062,7 +3062,9 @@ export const webviewMessageHandler = async (
 
 				await skillsManager.deleteSkill(skillName, source, skillMode)
 
-				// UI will handle refresh via setTimeout
+				// Send updated skills list
+				const skills = skillsManager.getSkillsMetadata()
+				await provider.postMessageToWebview({ type: "skills", skills })
 			} catch (error) {
 				const errorMessage = error instanceof Error ? error.message : String(error)
 				provider.log(`Error deleting skill: ${errorMessage}`)

webview-ui/src/components/settings/CreateSkillDialog.tsx

@@ -27,14 +27,16 @@ interface CreateSkillDialogProps {
 
 /**
  * Validate skill name according to agentskills.io spec:
- * - Must start with a letter
  * - Only lowercase letters, numbers, and hyphens allowed
+ * - No leading/trailing hyphens
+ * - No consecutive hyphens
  * - 1-64 characters
  */
 const validateSkillName = (name: string): string | null => {
 	if (!name) return "settings:skills.validation.nameRequired"
 	if (name.length > 64) return "settings:skills.validation.nameTooLong"
-	if (!/^[a-z][a-z0-9-]*$/.test(name)) {
+	// Match backend validation: /^[a-z0-9]+(?:-[a-z0-9]+)*$/
+	if (!/^[a-z0-9]+(?:-[a-z0-9]+)*$/.test(name)) {
 		return "settings:skills.validation.nameInvalid"
 	}
 	return null