@sweny-ai/core 0.1.29 → 0.1.30

package/dist/__tests__/skills-index.test.js

@@ -65,11 +65,11 @@ describe("skills registry", () => {
             expect(validCategories).toContain(skill.category);
         }
     });
-    it("isSkillConfigured checks env vars", () => {
+    it("isSkillConfigured checks required env vars", () => {
         expect(isSkillConfigured(github, { GITHUB_TOKEN: "ghp_xxx" })).toBe(true);
-        expect(isSkillConfigured(github, {})).toBe(false);
-        // Slack has all-optional config — needs at least one to be usable
-        expect(isSkillConfigured(slack, {})).toBe(false);
+        expect(isSkillConfigured(github, {})).toBe(false); // GITHUB_TOKEN is required
+        // Slack has all-optional config — always configured, tools validate at call time
+        expect(isSkillConfigured(slack, {})).toBe(true);
         expect(isSkillConfigured(slack, { SLACK_WEBHOOK_URL: "https://hooks.slack.com/..." })).toBe(true);
         expect(isSkillConfigured(slack, { SLACK_BOT_TOKEN: "xoxb-..." })).toBe(true);
     });

package/dist/__tests__/skills.test.js

@@ -391,16 +391,16 @@ describe("configuredSkills with custom skills", () => {
     }
     it("includes custom skills alongside configured built-ins", () => {
         writeSkill("my-custom", `---\nname: my-custom\ndescription: Custom\n---\nBody`);
-        // Provide a notification webhook so at least one built-in is configured
-        const skills = configuredSkills({ NOTIFICATION_WEBHOOK_URL: "https://example.com" }, tmpDir);
+        const skills = configuredSkills({}, tmpDir);
         const ids = skills.map((s) => s.id);
         expect(ids).toContain("my-custom");
+        // Skills with only optional config are always configured
         expect(ids).toContain("notification");
+        expect(ids).toContain("slack");
     });
     it("built-in skills take precedence over custom skills with same ID", () => {
-        // notification is configured when a webhook URL is set
         writeSkill("notification", `---\nname: notification\ndescription: Custom override attempt\n---\nBody`);
-        const skills = configuredSkills({ NOTIFICATION_WEBHOOK_URL: "https://example.com" }, tmpDir);
+        const skills = configuredSkills({}, tmpDir);
         const notif = skills.filter((s) => s.id === "notification");
         expect(notif).toHaveLength(1);
         // Should be the built-in (has tools), not the custom one (tools: [])

package/dist/skills/index.d.ts

@@ -41,10 +41,9 @@ export declare function allSkills(): Map<string, Skill>;
 /**
  * Check if a skill is usable given the available environment.
  *
- * - All required config fields must have values.
- * - If a skill has env-backed config fields (even optional ones),
- *   at least one must be set — otherwise the skill has zero credentials.
- * - Skills with no env-backed config are always considered configured.
+ * A skill is configured when all its *required* config fields are set.
+ * Optional fields are validated by individual tool handlers at call time,
+ * so missing optional config never disqualifies the skill.
  */
 export declare function isSkillConfigured(skill: Skill, env?: Record<string, string | undefined>): boolean;
 /**

package/dist/skills/index.js

@@ -106,22 +106,16 @@ export function allSkills() {
 /**
  * Check if a skill is usable given the available environment.
  *
- * - All required config fields must have values.
- * - If a skill has env-backed config fields (even optional ones),
- *   at least one must be set — otherwise the skill has zero credentials.
- * - Skills with no env-backed config are always considered configured.
+ * A skill is configured when all its *required* config fields are set.
+ * Optional fields are validated by individual tool handlers at call time,
+ * so missing optional config never disqualifies the skill.
  */
 export function isSkillConfigured(skill, env = process.env) {
-    const envFields = Object.values(skill.config).filter((f) => f.env);
-    if (envFields.length === 0)
-        return true; // no env needed
-    // All required fields must be set
-    for (const field of envFields) {
-        if (field.required && !env[field.env])
+    for (const field of Object.values(skill.config)) {
+        if (field.required && field.env && !env[field.env])
             return false;
     }
-    // At least one env-backed field must be set (handles all-optional skills like slack)
-    return envFields.some((f) => env[f.env]);
+    return true;
 }
 /**
  * From all builtins + custom repo skills, return only the skills that are usable.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sweny-ai/core",
-  "version": "0.1.29",
+  "version": "0.1.30",
   "type": "module",
   "bin": {
     "sweny": "./dist/cli/main.js"