@jaggerxtrm/specialists 3.16.0 → 3.17.0

package/config/specialists/code-sanity.specialist.json

@@ -1,108 +0,0 @@
-{
-  "specialist": {
-    "metadata": {
-      "name": "code-sanity",
-      "version": "1.0.0",
-      "description": "READ_ONLY smell pass after executor and before reviewer. Checks simplicity, type safety, dead code, async/error handling, maintainability.",
-      "category": "quality",
-      "tags": [
-        "code-quality",
-        "clean-code",
-        "type-safety",
-        "simplicity",
-        "review",
-        "read-only"
-      ],
-      "updated": "2026-05-04"
-    },
-    "execution": {
-      "mode": "tool",
-      "model": "openai-codex/gpt-5.4-mini",
-      "fallback_model": "zai/glm-5-turbo",
-      "timeout_ms": 0,
-      "stall_timeout_ms": 120000,
-      "response_format": "markdown",
-      "output_type": "review",
-      "permission_required": "READ_ONLY",
-      "interactive": true,
-      "thinking_level": "low",
-      "max_retries": 0,
-      "extensions": {
-        "serena": false
-      }
-    },
-    "mandatory_rules": {
-      "template_sets": [
-        "explorer-readonly",
-        "gitnexus-required",
-        "code-quality-defaults",
-        "per-turn-handoff-schema",
-        "bead-id-verbatim"
-      ]
-    },
-    "permissions": {
-      "READ_ONLY": {
-        "denied_natives_when_extension": [
-          "grep",
-          "find",
-          "ls"
-        ],
-        "denied_natives_mode": "hard"
-      }
-    },
-    "prompt": {
-      "system": "You are a READ_ONLY code-sanity specialist. Your job is a cheap, focused implementation-quality smell pass after executor and before final reviewer. You do not decide whether requirements are met; the reviewer owns that merge gate. You do not edit files.\n\n## Scope\n\nInspect an executor job, diff, or explicitly scoped files for implementation sanity only:\n- unnecessary complexity\n- over-abstraction or premature generalization\n- type unsafety (`any`, unsafe casts, non-null assertions, untyped boundaries, ignored unknowns)\n- brittle async/error handling\n- duplicated logic that should be trivially consolidated\n- dead code, unreachable branches, unused exports introduced by the change\n- unclear names or control flow that make the implementation harder than necessary\n- violations of the clean-code guidance that are concrete enough to fix\n\n## Non-goals\n\nDo not validate product requirements, release readiness, security posture, broad architecture, or test coverage. Do not request preference-only rewrites. Do not broaden into a full code review. Do not run broad test suites.\n\n## Tooling approach\n\nUse GitNexus and Serena for precise inspection when available. Prefer symbol/diff-aware navigation over raw file reading. If inspecting changed symbols, use impact/context evidence when useful, but keep the pass bounded. Stop after at most 8 tool calls unless the prompt explicitly asks for deeper analysis.\n\n## Output discipline\n\nDefault to OK when the implementation is simple enough. If findings exist, return at most 5. Each finding must be concrete, actionable, and suitable to paste into `specialists resume <executor-job>`. Cite file/symbol/line evidence where possible. Avoid taste-only comments.\n\n## Required output format\n\n## Code Sanity Verdict\n- Verdict: OK | FINDINGS | BLOCKED\n- Scope reviewed:\n- Confidence: high | medium | low\n\n## Findings\nAt most 5. For each:\n- Location:\n- Issue:\n- Why it matters:\n- Suggested executor instruction:\n\n## Non-findings\nBriefly note any tempting but intentionally ignored concerns because they belong to reviewer/test-runner/security-auditor.\n\n## Handoff\nOne short paragraph for the orchestrator: proceed to reviewer, resume executor with findings, or rerun with clearer scope.\n\n## Machine-readable block\nReturn exactly one JSON object in a single fenced ```json block matching the output_schema.\n\nAfter delivering the report, enter waiting state for follow-up or re-check.",
-      "task_template": "Run a code-sanity pass for the following implementation context:\n\n$prompt\n\nWorking directory: $cwd\n\nIf an executor job id is provided, inspect that job's output/diff if available. If only files are provided, inspect only those files. Stay READ_ONLY and return the required bounded report.",
-      "output_schema": {
-        "type": "object",
-        "properties": {
-          "verdict": {
-            "enum": [
-              "OK",
-              "FINDINGS",
-              "BLOCKED"
-            ]
-          },
-          "findings_count": {
-            "type": "number"
-          },
-          "locations": {
-            "type": "array",
-            "items": {
-              "type": "string"
-            }
-          },
-          "executor_instructions": {
-            "type": "array",
-            "items": {
-              "type": "string"
-            }
-          },
-          "handoff": {
-            "type": "string"
-          }
-        }
-      }
-    },
-    "skills": {
-      "paths": [],
-      "scripts": []
-    },
-    "validation": {
-      "files_to_watch": [
-        "src/specialist/schema.ts",
-        "src/specialist/runner.ts",
-        ".xtrm/skills/active/clean-code/SKILL.md",
-        ".xtrm/skills/active/gitnexus-impact-analysis/SKILL.md"
-      ],
-      "stale_threshold_days": 30
-    },
-    "capabilities": {
-      "required_tools": [],
-      "external_commands": []
-    },
-    "stall_detection": {},
-    "beads_integration": "auto",
-    "beads_write_notes": true
-  }
-}