fix: use list.remove() instead of list.pop() in whitelist_skill (#394)

list.pop() takes an integer index; passing a skill ID string raises
TypeError at runtime. list.remove() takes the value to remove.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: JarbasAl

ovos_workshop/permissions.py

@@ -24,7 +24,7 @@ def whitelist_skill(skill, config=None):
     skills_config = config.get("skills", {})
     blacklisted_skills = skills_config.get("blacklisted_skills", [])
     if skill in blacklisted_skills:
-        blacklisted_skills.pop(skill)
+        blacklisted_skills.remove(skill)
         conf = {
             "skills": {
                 "blacklisted_skills": blacklisted_skills