@fro.bot/systematic 2.31.0 → 2.32.0

package/dist/cli.js

@@ -6,7 +6,7 @@ import {
   findCommandsInDir,
   findSkillsInDir,
   getConfigPaths
-} from "./index-ybqr13k7.js";
+} from "./index-wjkgb2gb.js";
 
 // src/cli.ts
 import fs from "fs";

package/dist/{index-ybqr13k7.js → index-wjkgb2gb.js}

@@ -1414,6 +1414,164 @@ function printParseErrorCode(code) {
   return "<unknown ParseErrorCode>";
 }
 
+// src/lib/bundled-names.ts
+var BUNDLED_AGENT_NAMES = [
+  "adversarial-document-reviewer",
+  "adversarial-reviewer",
+  "agent-native-reviewer",
+  "ankane-readme-writer",
+  "api-contract-reviewer",
+  "architecture-strategist",
+  "best-practices-researcher",
+  "bug-reproduction-validator",
+  "cli-agent-readiness-reviewer",
+  "cli-readiness-reviewer",
+  "code-simplicity-reviewer",
+  "coherence-reviewer",
+  "correctness-reviewer",
+  "data-integrity-guardian",
+  "data-migration-expert",
+  "data-migrations-reviewer",
+  "deployment-verification-agent",
+  "design-implementation-reviewer",
+  "design-iterator",
+  "design-lens-reviewer",
+  "dhh-rails-reviewer",
+  "feasibility-reviewer",
+  "figma-design-sync",
+  "framework-docs-researcher",
+  "git-history-analyzer",
+  "issue-intelligence-analyst",
+  "julik-frontend-races-reviewer",
+  "kieran-python-reviewer",
+  "kieran-rails-reviewer",
+  "kieran-typescript-reviewer",
+  "learnings-researcher",
+  "lint",
+  "maintainability-reviewer",
+  "pattern-recognition-specialist",
+  "performance-oracle",
+  "performance-reviewer",
+  "pr-comment-resolver",
+  "previous-comments-reviewer",
+  "product-lens-reviewer",
+  "project-standards-reviewer",
+  "reliability-reviewer",
+  "repo-research-analyst",
+  "schema-drift-detector",
+  "scope-guardian-reviewer",
+  "security-lens-reviewer",
+  "security-reviewer",
+  "security-sentinel",
+  "slack-researcher",
+  "spec-flow-analyzer",
+  "systematic-implementer",
+  "testing-reviewer"
+];
+var BUNDLED_AGENT_QUALIFIED_IDS = [
+  "design/design-implementation-reviewer",
+  "design/design-iterator",
+  "design/figma-design-sync",
+  "docs/ankane-readme-writer",
+  "document-review/adversarial-document-reviewer",
+  "document-review/coherence-reviewer",
+  "document-review/design-lens-reviewer",
+  "document-review/feasibility-reviewer",
+  "document-review/product-lens-reviewer",
+  "document-review/scope-guardian-reviewer",
+  "document-review/security-lens-reviewer",
+  "research/best-practices-researcher",
+  "research/framework-docs-researcher",
+  "research/git-history-analyzer",
+  "research/issue-intelligence-analyst",
+  "research/learnings-researcher",
+  "research/repo-research-analyst",
+  "research/slack-researcher",
+  "review/adversarial-reviewer",
+  "review/agent-native-reviewer",
+  "review/api-contract-reviewer",
+  "review/architecture-strategist",
+  "review/cli-agent-readiness-reviewer",
+  "review/cli-readiness-reviewer",
+  "review/code-simplicity-reviewer",
+  "review/correctness-reviewer",
+  "review/data-integrity-guardian",
+  "review/data-migration-expert",
+  "review/data-migrations-reviewer",
+  "review/deployment-verification-agent",
+  "review/dhh-rails-reviewer",
+  "review/julik-frontend-races-reviewer",
+  "review/kieran-python-reviewer",
+  "review/kieran-rails-reviewer",
+  "review/kieran-typescript-reviewer",
+  "review/maintainability-reviewer",
+  "review/pattern-recognition-specialist",
+  "review/performance-oracle",
+  "review/performance-reviewer",
+  "review/previous-comments-reviewer",
+  "review/project-standards-reviewer",
+  "review/reliability-reviewer",
+  "review/schema-drift-detector",
+  "review/security-reviewer",
+  "review/security-sentinel",
+  "review/testing-reviewer",
+  "workflow/bug-reproduction-validator",
+  "workflow/lint",
+  "workflow/pr-comment-resolver",
+  "workflow/spec-flow-analyzer",
+  "workflow/systematic-implementer"
+];
+var BUNDLED_SKILL_NAMES = [
+  "agent-browser",
+  "agent-native-architecture",
+  "agent-native-audit",
+  "andrew-kane-gem-writer",
+  "ce:brainstorm",
+  "ce:compound",
+  "ce:compound-refresh",
+  "ce:ideate",
+  "ce:plan",
+  "ce:review",
+  "ce:work",
+  "changelog",
+  "claude-permissions-optimizer",
+  "compound-docs",
+  "deepen-plan",
+  "deploy-docs",
+  "dhh-rails-style",
+  "document-review",
+  "dspy-ruby",
+  "every-style-editor",
+  "feature-video",
+  "frontend-design",
+  "gemini-imagegen",
+  "generate_command",
+  "git-clean-gone-branches",
+  "git-commit",
+  "git-commit-push-pr",
+  "git-worktree",
+  "lfg",
+  "onboarding",
+  "orchestrating-subagents",
+  "orchestrating-swarms",
+  "proof",
+  "rclone",
+  "report-bug-ce",
+  "reproduce-bug",
+  "resolve-pr-feedback",
+  "setup",
+  "slfg",
+  "test-browser",
+  "test-driven-development",
+  "test-xcode",
+  "todo-create",
+  "todo-resolve",
+  "todo-triage",
+  "using-systematic",
+  "writing-skills",
+  "writing-systematic-skills"
+];
+
 // node_modules/.bun/zod@4.4.3/node_modules/zod/v4/classic/external.js
 var exports_external = {};
 __export(exports_external, {
@@ -15701,163 +15859,9 @@ var OPENCODE_AGENT_COLOR_TOKENS = [
   "info"
 ];
 
-// src/lib/bundled-names.ts
-var BUNDLED_AGENT_NAMES = [
-  "adversarial-document-reviewer",
-  "adversarial-reviewer",
-  "agent-native-reviewer",
-  "ankane-readme-writer",
-  "api-contract-reviewer",
-  "architecture-strategist",
-  "best-practices-researcher",
-  "bug-reproduction-validator",
-  "cli-agent-readiness-reviewer",
-  "cli-readiness-reviewer",
-  "code-simplicity-reviewer",
-  "coherence-reviewer",
-  "correctness-reviewer",
-  "data-integrity-guardian",
-  "data-migration-expert",
-  "data-migrations-reviewer",
-  "deployment-verification-agent",
-  "design-implementation-reviewer",
-  "design-iterator",
-  "design-lens-reviewer",
-  "dhh-rails-reviewer",
-  "feasibility-reviewer",
-  "figma-design-sync",
-  "framework-docs-researcher",
-  "git-history-analyzer",
-  "issue-intelligence-analyst",
-  "julik-frontend-races-reviewer",
-  "kieran-python-reviewer",
-  "kieran-rails-reviewer",
-  "kieran-typescript-reviewer",
-  "learnings-researcher",
-  "lint",
-  "maintainability-reviewer",
-  "pattern-recognition-specialist",
-  "performance-oracle",
-  "performance-reviewer",
-  "pr-comment-resolver",
-  "previous-comments-reviewer",
-  "product-lens-reviewer",
-  "project-standards-reviewer",
-  "reliability-reviewer",
-  "repo-research-analyst",
-  "schema-drift-detector",
-  "scope-guardian-reviewer",
-  "security-lens-reviewer",
-  "security-reviewer",
-  "security-sentinel",
-  "slack-researcher",
-  "spec-flow-analyzer",
-  "systematic-implementer",
-  "testing-reviewer"
-];
-var BUNDLED_AGENT_QUALIFIED_IDS = [
-  "design/design-implementation-reviewer",
-  "design/design-iterator",
-  "design/figma-design-sync",
-  "docs/ankane-readme-writer",
-  "document-review/adversarial-document-reviewer",
-  "document-review/coherence-reviewer",
-  "document-review/design-lens-reviewer",
-  "document-review/feasibility-reviewer",
-  "document-review/product-lens-reviewer",
-  "document-review/scope-guardian-reviewer",
-  "document-review/security-lens-reviewer",
-  "research/best-practices-researcher",
-  "research/framework-docs-researcher",
-  "research/git-history-analyzer",
-  "research/issue-intelligence-analyst",
-  "research/learnings-researcher",
-  "research/repo-research-analyst",
-  "research/slack-researcher",
-  "review/adversarial-reviewer",
-  "review/agent-native-reviewer",
-  "review/api-contract-reviewer",
-  "review/architecture-strategist",
-  "review/cli-agent-readiness-reviewer",
-  "review/cli-readiness-reviewer",
-  "review/code-simplicity-reviewer",
-  "review/correctness-reviewer",
-  "review/data-integrity-guardian",
-  "review/data-migration-expert",
-  "review/data-migrations-reviewer",
-  "review/deployment-verification-agent",
-  "review/dhh-rails-reviewer",
-  "review/julik-frontend-races-reviewer",
-  "review/kieran-python-reviewer",
-  "review/kieran-rails-reviewer",
-  "review/kieran-typescript-reviewer",
-  "review/maintainability-reviewer",
-  "review/pattern-recognition-specialist",
-  "review/performance-oracle",
-  "review/performance-reviewer",
-  "review/previous-comments-reviewer",
-  "review/project-standards-reviewer",
-  "review/reliability-reviewer",
-  "review/schema-drift-detector",
-  "review/security-reviewer",
-  "review/security-sentinel",
-  "review/testing-reviewer",
-  "workflow/bug-reproduction-validator",
-  "workflow/lint",
-  "workflow/pr-comment-resolver",
-  "workflow/spec-flow-analyzer",
-  "workflow/systematic-implementer"
-];
-var BUNDLED_SKILL_NAMES = [
-  "agent-browser",
-  "agent-native-architecture",
-  "agent-native-audit",
-  "andrew-kane-gem-writer",
-  "ce:brainstorm",
-  "ce:compound",
-  "ce:compound-refresh",
-  "ce:ideate",
-  "ce:plan",
-  "ce:review",
-  "ce:work",
-  "changelog",
-  "claude-permissions-optimizer",
-  "compound-docs",
-  "deepen-plan",
-  "deploy-docs",
-  "dhh-rails-style",
-  "document-review",
-  "dspy-ruby",
-  "every-style-editor",
-  "feature-video",
-  "frontend-design",
-  "gemini-imagegen",
-  "generate_command",
-  "git-clean-gone-branches",
-  "git-commit",
-  "git-commit-push-pr",
-  "git-worktree",
-  "lfg",
-  "onboarding",
-  "orchestrating-subagents",
-  "orchestrating-swarms",
-  "proof",
-  "rclone",
-  "report-bug-ce",
-  "reproduce-bug",
-  "resolve-pr-feedback",
-  "setup",
-  "slfg",
-  "test-browser",
-  "test-driven-development",
-  "test-xcode",
-  "todo-create",
-  "todo-resolve",
-  "todo-triage",
-  "using-systematic",
-  "writing-skills",
-  "writing-systematic-skills"
-];
+// src/lib/removed-names.ts
+var REMOVED_BUNDLED_SKILL_NAMES = [];
+var REMOVED_BUNDLED_AGENT_NAMES = [];
 
 // src/lib/config-schema.ts
 var permissionSettingSchema = exports_external.enum(["ask", "allow", "deny"]);
@@ -15982,7 +15986,13 @@ var BootstrapSchema = exports_external.object({
   examples: [{ enabled: true }, { enabled: false }]
 });
 function createSystematicConfigSchema(opts) {
-  const { agentNames, qualifiedAgentIds, skillNames } = opts;
+  const {
+    agentNames,
+    qualifiedAgentIds,
+    skillNames,
+    removedSkillNames = [],
+    removedAgentNames = []
+  } = opts;
   return exports_external.object({
     $schema: exports_external.string().url().optional().meta({
       description: "JSON Schema URL for IDE autocomplete. The value is informational only \u2014 the loader does not fetch or validate against it. Add this to enable IDE schema activation and field-level autocomplete in editors that support JSON Schema (VSCode, Zed, IntelliJ).",
@@ -16005,11 +16015,15 @@ function createSystematicConfigSchema(opts) {
       description: "Per-category configuration overlays keyed by category name",
       examples: [{ review: { model: "anthropic/claude-opus-4-7" } }, {}]
     }),
-    disabled_skills: exports_external.array(exports_external.enum(skillNames)).default([]).meta({
+    disabled_skills: exports_external.array(exports_external.enum([...skillNames, ...removedSkillNames])).default([]).meta({
       description: "Array of bundled skill names to disable globally. Unknown skill names are rejected at parse time.",
       examples: [["ce:plan", "ce:review"]]
     }),
-    disabled_agents: exports_external.array(exports_external.enum([...agentNames, ...qualifiedAgentIds])).default([]).meta({
+    disabled_agents: exports_external.array(exports_external.enum([
+      ...agentNames,
+      ...qualifiedAgentIds,
+      ...removedAgentNames
+    ])).default([]).meta({
       description: "Array of bundled agent names (bare or qualified category/name) to disable globally. Unknown agent names are rejected at parse time.",
       examples: [
         ["previous-comments-reviewer", "cli-readiness-reviewer"],
@@ -16037,7 +16051,9 @@ function createSystematicConfigSchema(opts) {
 var SystematicConfigSchema = createSystematicConfigSchema({
   agentNames: BUNDLED_AGENT_NAMES,
   qualifiedAgentIds: BUNDLED_AGENT_QUALIFIED_IDS,
-  skillNames: BUNDLED_SKILL_NAMES
+  skillNames: BUNDLED_SKILL_NAMES,
+  removedSkillNames: REMOVED_BUNDLED_SKILL_NAMES,
+  removedAgentNames: REMOVED_BUNDLED_AGENT_NAMES
 });
 var SECURITY_OVERLAY_FIELDS = [
   "model",
@@ -16058,6 +16074,22 @@ var DEFAULT_CONFIG = {
   categories: {}
 };
 var SECURITY_OVERLAY_FIELDS2 = new Set(SECURITY_OVERLAY_FIELDS);
+var CURRENT_SKILL_NAMES_SET = new Set(BUNDLED_SKILL_NAMES);
+var CURRENT_AGENT_NAMES_SET = new Set([
+  ...BUNDLED_AGENT_NAMES,
+  ...BUNDLED_AGENT_QUALIFIED_IDS
+]);
+function computeDroppedNames(names, allowedSet) {
+  return names.filter((n) => !allowedSet.has(n));
+}
+function warnDroppedNames(dropped, field, warned) {
+  for (const name of dropped) {
+    if (warned.has(name))
+      continue;
+    warned.add(name);
+    console.warn(`[systematic] "${name}" in \`${field}\` is no longer a bundled name and will be ignored. Remove it from your config to silence this warning.`);
+  }
+}
 function resolveConfigPath(dir, basename) {
   const jsoncPath = path3.join(dir, `${basename}.jsonc`);
   if (fs4.existsSync(jsoncPath))
@@ -16210,7 +16242,19 @@ function loadConfigWithSources(projectDir) {
     agents: overlayValues(overlays.agents),
     categories: overlayValues(overlays.categories)
   };
-  return { config: result, overlays };
+  const warned = new Set;
+  const droppedSkills = computeDroppedNames(result.disabled_skills, CURRENT_SKILL_NAMES_SET);
+  warnDroppedNames(droppedSkills, "disabled_skills", warned);
+  const droppedAgents = computeDroppedNames(result.disabled_agents, CURRENT_AGENT_NAMES_SET);
+  warnDroppedNames(droppedAgents, "disabled_agents", warned);
+  const droppedSkillSet = new Set(droppedSkills);
+  const droppedAgentSet = new Set(droppedAgents);
+  const effectiveConfig = droppedSkillSet.size === 0 && droppedAgentSet.size === 0 ? result : {
+    ...result,
+    disabled_skills: result.disabled_skills.filter((n) => !droppedSkillSet.has(n)),
+    disabled_agents: result.disabled_agents.filter((n) => !droppedAgentSet.has(n))
+  };
+  return { config: effectiveConfig, overlays };
 }
 function mergeOverlaySources(sources) {
   const result = {

package/dist/index.js

@@ -13,7 +13,7 @@ import {
   loadConfig,
   loadConfigWithSources,
   parseFrontmatter
-} from "./index-ybqr13k7.js";
+} from "./index-wjkgb2gb.js";
 
 // src/index.ts
 import fs5 from "fs";

package/dist/lib/config-schema.d.ts

@@ -87,6 +87,8 @@ export interface SystematicConfigSchemaOptions {
     readonly agentNames: readonly string[];
     readonly qualifiedAgentIds: readonly string[];
     readonly skillNames: readonly string[];
+    readonly removedSkillNames?: readonly string[];
+    readonly removedAgentNames?: readonly string[];
 }
 /**
  * Construct the top-level Systematic config schema from a discovered set of

package/dist/lib/config.d.ts

@@ -26,6 +26,20 @@ export interface SystematicConfig {
     categories?: OverlayConfigMap;
 }
 export declare const DEFAULT_CONFIG: SystematicConfig;
+/**
+ * Return the subset of `names` that are absent from `allowedSet`. These are
+ * names that parsed successfully (they are in the removed-names list so the
+ * schema accepted them) but are no longer active bundled names and must be
+ * dropped from the effective config.
+ */
+export declare function computeDroppedNames(names: readonly string[], allowedSet: ReadonlySet<string>): string[];
+/**
+ * Emit a `[systematic]` warning for each dropped name that has not already
+ * been warned about in this load invocation. The `warned` set is local to a
+ * single load call -- callers must NOT share it across independent loads.
+ * Passing a fresh set per load ensures no cross-load suppression.
+ */
+export declare function warnDroppedNames(dropped: string[], field: string, warned: Set<string>): void;
 export declare function loadConfig(projectDir: string): SystematicConfig;
 export declare function loadConfigWithSources(projectDir: string): SourceAwareConfigResult;
 export declare function getConfigPaths(projectDir: string): {

package/dist/lib/removed-names.d.ts

@@ -0,0 +1,15 @@
+/**
+ * Known-removed bundled skill and agent names.
+ *
+ * Names listed here were present in a past release but have since been removed
+ * from the bundled catalog. They are accepted (warn-and-ignore) in
+ * `disabled_skills` and `disabled_agents` so that upgrading does not brick
+ * configs that disabled them before removal. Genuinely-unknown names (typos,
+ * names that never existed) are still rejected by strict validation.
+ *
+ * This list is empty until v3 populates it with the first removed names.
+ * Add a name here only when the corresponding skill or agent directory is
+ * actually deleted from the repo.
+ */
+export declare const REMOVED_BUNDLED_SKILL_NAMES: readonly string[];
+export declare const REMOVED_BUNDLED_AGENT_NAMES: readonly string[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fro.bot/systematic",
-  "version": "2.31.0",
+  "version": "2.32.0",
   "description": "Structured engineering workflows for OpenCode",
   "type": "module",
   "homepage": "https://fro.bot/systematic",
@@ -66,18 +66,18 @@
   },
   "devDependencies": {
     "@biomejs/biome": "2.4.16",
-    "@opencode-ai/plugin": "1.15.13",
-    "@opencode-ai/sdk": "1.15.13",
+    "@opencode-ai/plugin": "1.17.3",
+    "@opencode-ai/sdk": "1.17.3",
     "@semantic-release/exec": "7.1.0",
     "@types/bun": "latest",
     "@types/js-yaml": "4.0.9",
-    "@types/node": "24.13.0",
+    "@types/node": "24.13.2",
     "ajv": "8.20.0",
     "ajv-formats": "3.0.1",
     "conventional-changelog-conventionalcommits": "9.3.1",
     "markdownlint-cli": "0.48.0",
     "rimraf": "6.1.3",
-    "semantic-release": "25.0.3",
+    "semantic-release": "25.0.5",
     "semantic-release-export-data": "1.2.0",
     "typescript": "6.0.3"
   },

package/skills/orchestrating-subagents/SKILL.md

@@ -44,11 +44,11 @@ task({
 })
 ```
 
-**When background is available:** OpenCode exposes `task_status` in the tool list and the `task()` tool description includes background mode instructions. This requires `OPENCODE_EXPERIMENTAL_BACKGROUND_SUBAGENTS=true` or the umbrella `OPENCODE_EXPERIMENTAL=true` flag.
+**When background is available:** `OPENCODE_EXPERIMENTAL_BACKGROUND_SUBAGENTS=true` (or the umbrella `OPENCODE_EXPERIMENTAL=true`) must be set. When enabled, `background: true` runs the subagent asynchronously. OpenCode automatically injects the result into the parent session as a synthetic message when the subagent completes. You are notified; you do not poll.
 
-**When background is unavailable:** `task_status` is not registered. Passing `background: true` returns an error. Fall back to foreground dispatch — dispatch subagents serially or in small foreground batches instead.
+**When background is unavailable:** Passing `background: true` returns an error. Fall back to foreground dispatch — dispatch subagents serially or in small foreground batches instead.
 
-**Check before assuming:** If you see `task_status` in your available tools, background dispatch is enabled. If you do not see it, use foreground dispatch only.
+**Check before assuming:** Rely on whether `background: true` is accepted (i.e., the env flag is set). If it returns an error, use foreground dispatch only.
 
 ## Serial vs Parallel Dispatch
 
@@ -119,7 +119,7 @@ After subagents complete, the orchestrator synthesizes results:
 - If a subagent returns an error or its output is incomplete, diagnose before dispatching dependent units.
 - Do not dispatch dependent units on a broken tree.
 - Retry a failed unit by dispatching a new `task()` call with a corrected prompt, or resume the prior session with `task_id`.
-- For background tasks (when available): use `task_status` to poll or wait for terminal state before retrying.
+- For background tasks (when available): wait for the automatic completion notification (the result is injected into the parent session). Do not poll or sleep. Retry by dispatching a new `task()` with a corrected prompt, or resume with `task_id`.
 
 ## Quick Reference
 
@@ -128,7 +128,7 @@ After subagents complete, the orchestrator synthesizes results:
 | Units have dependencies | Serial foreground dispatch |
 | Units share files | Serial foreground dispatch |
 | Units are independent, no file overlap | Parallel foreground dispatch |
-| Background available + long-running work | Parallel background dispatch with `task_status` |
+| Background available + long-running work | Parallel background dispatch; results are pushed back to the parent on completion (no polling) |
 | Background unavailable | Foreground only — serial or batched |
 | Subagent fails | Diagnose, fix prompt, retry with new `task()` or resume with `task_id` |
 | File collision detected post-parallel | Stage non-colliding files (if workflow owns git ops), re-run colliding units serially |
@@ -137,8 +137,8 @@ After subagents complete, the orchestrator synthesizes results:
 
 | Mistake | Fix |
 |---|---|
-| Assuming `task_status` is always available | Check tool list first; fall back to foreground if absent |
+| Polling or sleeping to wait for a background subagent | Background results are pushed into the parent session automatically; never poll or sleep |
 | Parallel subagents staging or committing | Instruct subagents not to stage/commit; the current workflow owner handles git ops when applicable, otherwise synthesize file inventory and results for the caller or user |
 | Dispatching dependent units without waiting | Always wait for prerequisites to complete and verify their output |
 | Ignoring file overlap in parallel batches | Run the Parallel Safety Check before every parallel dispatch |
-| Using `background: true` without checking the flag | Only use when `task_status` appears in available tools |
+| Using `background: true` without the experimental flag enabled | Only use background when `OPENCODE_EXPERIMENTAL_BACKGROUND_SUBAGENTS=true`; otherwise dispatch foreground |