@lazyingart/agintiflow 0.20.289 → 0.20.290

package/docs/skills-and-tools.md

@@ -79,6 +79,14 @@ tools:
 
 Selected skills are injected into the plan and execution prompts. The LLM still decides what to do; skills only provide domain playbooks and guardrails.
 
+Automatic selection requires substantive relevance. Exact IDs, focused short
+requests, specific triggers, and several matching description terms score
+strongly. Generic words such as `data`, `search`, `project`, or `commit` are
+discounted when they occur incidentally inside a longer request, and automatic
+mode omits weak matches instead of filling the skill budget. Explicit profiles
+and direct topic wording remain authoritative. This keeps unrelated guidance
+out of the context without hard-coding task-specific exclusions.
+
 ## Adding A Skill
 
 For a reusable project workflow, create `.aginti/skills/<id>/SKILL.md` with valid YAML frontmatter and a short Markdown body. These skills are loaded from the current project, shown as `source=project-local`, and are preferred for task-specific knowledge that should not become core runtime policy.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lazyingart/agintiflow",
-  "version": "0.20.289",
+  "version": "0.20.290",
   "type": "module",
   "description": "AgInTiFlow is a project-aware agent workspace for hybrid wet-dry R&D, hardware-aware intelligence, software automation, and industrial workflows.",
   "license": "Apache-2.0",

package/scripts/smoke-skills.js

@@ -112,6 +112,24 @@ const pythonSkill = skills.find((skill) => skill.id === "python");
 assert(pythonSkill?.body.includes("PEP 701 relaxed f-strings"), "Python skill must mention 3.12 f-string compatibility traps");
 assert(pythonSkill?.body.includes("only proves the active interpreter"), "Python skill must guard syntax-check overclaims");
 assert(selectedIds("write SQL migrations for sqlite schema").includes("database"), "database prompt did not select database");
+const imperfectDatabaseRepairSkills = selectedIds(
+  "The reading archive broke after its recent schema upgrade: an old library came back empty, and search gives surprising results for some punctuation. Please inspect the project, fix the underlying problems without discarding existing data, add useful regression coverage, run the checks, and commit the intentional repair. Keep the implementation small and standard-library only."
+);
+assert(imperfectDatabaseRepairSkills.includes("database"), "normal database repair prompt omitted database guidance");
+assert(imperfectDatabaseRepairSkills.includes("qa-testing"), "normal database repair prompt omitted regression-test guidance");
+for (const unrelated of [
+  "autonomous-artifact-pipeline",
+  "bgpt-paper-search",
+  "data-analysis",
+  "exa-search",
+  "exploratory-data-analysis",
+  "optimize-for-gpu",
+]) {
+  assert(
+    !imperfectDatabaseRepairSkills.includes(unrelated),
+    `normal database repair prompt selected unrelated ${unrelated} guidance`
+  );
+}
 assert(selectedIds("debug Docker deployment logs and port config").includes("devops-deployment"), "devops prompt did not select devops-deployment");
 assert(selectedIds("review auth security and secrets handling").includes("security-review"), "security prompt did not select security-review");
 assert(selectedIds("make a PowerPoint pitch deck").includes("presentation-slides"), "slides prompt did not select presentation-slides");

package/src/skill-library.js

@@ -419,21 +419,31 @@ const DESCRIPTION_STOP_WORDS = new Set([
   "after",
   "also",
   "before",
+  "checks",
   "content",
   "create",
+  "data",
   "exact",
+  "existing",
   "file",
   "files",
   "from",
   "general",
   "into",
+  "keep",
+  "only",
   "output",
   "path",
   "paths",
   "project",
   "read",
+  "reading",
+  "recent",
   "report",
   "request",
+  "results",
+  "search",
+  "standard",
   "task",
   "that",
   "their",
@@ -443,14 +453,19 @@ const DESCRIPTION_STOP_WORDS = new Set([
   "user",
   "when",
   "with",
+  "without",
   "work",
   "write",
 ]);
 
 const SKILL_ID_STOP_WORDS = new Set([
+  "and",
   "agent",
   "aginti",
+  "for",
   "development",
+  "of",
+  "the",
   "production",
   "skill",
   "system",
@@ -459,6 +474,30 @@ const SKILL_ID_STOP_WORDS = new Set([
   "workflow",
 ]);
 
+// These words are useful in a focused request, but occur incidentally across
+// many unrelated tasks. Weakening them in longer goals keeps one generic word
+// from consuming a full skill slot while preserving short requests such as
+// "analyze data" or "commit changes".
+const GENERIC_ROUTING_TERMS = new Set([
+  "analysis",
+  "app",
+  "application",
+  "code",
+  "commit",
+  "data",
+  "file",
+  "files",
+  "project",
+  "report",
+  "result",
+  "results",
+  "search",
+  "test",
+  "tests",
+  "testing",
+  "work",
+]);
+
 function descriptionTerms(value = "") {
   return [
     ...new Set(
@@ -470,7 +509,16 @@ function descriptionTerms(value = "") {
   ];
 }
 
-function scoreSkill(skill, text, taskProfile) {
+function focusedGenericSignal(goalText, needle) {
+  if (!GENERIC_ROUTING_TERMS.has(needle)) return true;
+  const terms = String(goalText || "")
+    .toLowerCase()
+    .split(/[^a-z0-9+#.]+/)
+    .filter(Boolean);
+  return terms.length <= 4 && textHasTrigger(goalText, needle);
+}
+
+function scoreSkill(skill, text, taskProfile, goalText = text) {
   let score = 0;
   if (skill.id === taskProfile) score += 10;
   if (skill.triggers.includes(taskProfile)) score += 6;
@@ -479,13 +527,14 @@ function scoreSkill(skill, text, taskProfile) {
     .split(/[^a-z0-9+#.]+/)
     .filter((term) => term.length >= 3 && !SKILL_ID_STOP_WORDS.has(term));
   for (const term of idTerms) {
-    if (textHasTrigger(text, term)) score += 2;
+    if (textHasTrigger(text, term)) score += focusedGenericSignal(goalText, term) ? 2 : 0.5;
   }
   for (const trigger of skill.triggers) {
     const needle = trigger.toLowerCase();
     if (!needle) continue;
     if (textHasTrigger(text, needle)) {
-      score += Math.max(2, Math.min(6, Math.ceil(needle.length / 6)));
+      const triggerScore = Math.max(2, Math.min(6, Math.ceil(needle.length / 6)));
+      score += focusedGenericSignal(goalText, needle) ? triggerScore : 0.5;
       continue;
     }
     const triggerTerms = descriptionTerms(needle);
@@ -495,7 +544,7 @@ function scoreSkill(skill, text, taskProfile) {
   }
   const descriptionMatches = descriptionTerms(skill.description).filter((token) => textHasTrigger(text, token));
   if (descriptionMatches.length >= 3) {
-    score += Math.min(3, descriptionMatches.length * 0.35);
+    score += Math.min(3, descriptionMatches.length * 0.5);
   }
   return score;
 }
@@ -516,19 +565,19 @@ function textHasTrigger(text, needle) {
 
 export function selectSkillsForGoal(goal = "", { taskProfile = "auto", limit = 6, includeBody = true, projectRoot = process.cwd() } = {}) {
   const goalText = String(goal || "").toLowerCase();
-  const text = `${goalText} ${taskProfile}`.toLowerCase();
+  const text = goalText;
   const skills = listSkills({ includeBody, projectRoot });
   const ranked = skills
     .map((skill) => ({
       skill,
-      score: scoreSkill(skill, text, taskProfile),
-      directGoalScore: scoreSkill(skill, goalText, "auto"),
+      score: scoreSkill(skill, text, taskProfile, goalText),
+      directGoalScore: scoreSkill(skill, goalText, "auto", goalText),
     }))
     .filter((item) => item.score > 0)
     .sort((a, b) => b.score - a.score || a.skill.id.localeCompare(b.skill.id));
   const explicitProfile = Boolean(taskProfile && taskProfile !== "auto");
   const bestScore = ranked[0]?.score || 0;
-  const relevanceFloor = explicitProfile && bestScore >= 6 ? Math.max(2, bestScore * 0.5) : 0;
+  const relevanceFloor = explicitProfile && bestScore >= 6 ? Math.max(2, bestScore * 0.5) : 2;
   const scored = ranked
     .filter((item) => item.score >= relevanceFloor || item.directGoalScore >= 4)
     .map((item) => item.skill);