@mrclrchtr/supi-flow 0.6.1 → 0.10.0

package/README.md

@@ -53,20 +53,15 @@ flowchart TD
     ARCHIVE["/skill:supi-flow-archive [ID]
          Fresh verification (gate function)
          Update living documentation
-         Slop-scan docs
          Quality gate checklist"]
-    ARCHIVE --> SLOP["/skill:supi-flow-slop-detect
-         Tier 1-4 vocabulary
-         11 structural patterns
-         Score target: < 1.5"]
-    SLOP --> QGATE{"Quality gate
+    ARCHIVE --> QGATE{"Quality gate
          passes?"}
-    QGATE -->|"No"| SLOP
+    QGATE -->|"No"| ARCHIVE
     QGATE -->|"Yes"| CLOSE
 
     CLOSE["supi_flow_close
          Sets status=done, flow:done
-         Auto-commits .tndm/"]
+         Writes archive.md"]
 
     classDef phase fill:#e8f5e9,stroke:#4caf50,stroke-width:2
     classDef decision fill:#e3f2fd,stroke:#2196f3
@@ -84,7 +79,7 @@ Non-trivial flows require a TNDM ticket created by `supi_flow_start`. Trivial ch
 
 ## Skills
 
-Six skills ship under `skills/`:
+Five skills ship under `skills/`:
 
 | Skill | Trigger | Purpose |
 |---|---|---|
@@ -93,7 +88,6 @@ Six skills ship under `skills/`:
 | `supi-flow-apply` | `/supi-flow-apply` | Execute plan task by task |
 | `supi-flow-archive` | `/supi-flow-archive` | Verify, update docs, close out |
 | `supi-flow-debug` | Loaded on demand when blocked | Root-cause debugging protocol |
-| `supi-flow-slop-detect` | Loaded on demand during archive | AI-prose detection in docs |
 
 ## Tools
 
@@ -105,7 +99,7 @@ Five custom tools registered by the extension:
 | `supi_flow_start` | Create a ticket with status=todo, tag=flow:brainstorm, and optional design context in `content.md` |
 | `supi_flow_plan` | Store the executable implementation plan in `plan.md` while leaving `content.md` as the approved design summary |
 | `supi_flow_complete_task` | Check off a numbered task (`**Task N**`) in the registered `plan` document |
-| `supi_flow_close` | Mark done, write verification results to `archive.md`, and auto-commit `.tndm/` |
+| `supi_flow_close` | Mark done and write verification results to `archive.md` |
 
 Tools should be used instead of calling `tndm` via bash. The agent invokes them with structured parameters.
 
@@ -119,13 +113,6 @@ Tools should be used instead of calling `tndm` via bash. The agent invokes them
 
 Older tickets may still contain a legacy brainstorm sidecar document, but new flow work should not create or depend on it.
 
-## Commands
-
-| Command | Description |
-|---|---|
-| `/supi-flow` | List available flow commands |
-| `/supi-flow-status` | Query TNDM for active flow tickets and show the next recommended step |
-
 ## Prompt templates
 
 | Prompt | Description |
@@ -145,9 +132,22 @@ Flow phases map to TNDM statuses and tags:
 
 ## Dependencies
 
-- **tndm CLI**: required (all ticket operations shell out to `tndm`)
+- **tndm CLI** (`tandem-cli`): required (all ticket operations shell out to `tndm`)
+
+  ```sh
+  brew install mrclrchtr/tap/tandem-cli
+  ```
+
 - **pi**: discovers bundled skills and prompt templates automatically from the package
 
+## PI package
+
+This extension is published as a [`pi-package`](https://pi.dev/packages) — listed in the PI package gallery. Install directly:
+
+```bash
+pi install npm:@mrclrchtr/supi-flow
+```
+
 ## Installation
 
 The extension is auto-discovered when the plugin directory is in pi's extension search path:
@@ -158,7 +158,7 @@ ln -s "$(pwd)/plugins/supi-flow" ~/.pi/agent/extensions/supi-flow
 
 # Option 2: settings.json
 # Add to ~/.pi/agent/settings.json:
-# { "extensions": ["./plugins/supi-flow/src/index.ts"] }
+# { "extensions": ["./plugins/supi-flow/extensions/index.ts"] }
 ```
 
 ## Development

package/{src → extensions}/cli.ts

@@ -15,7 +15,13 @@ async function run(
     const child = execFile(file, args, options, (error, stdout, stderr) => {
       if (error) {
         const msg = toString(stderr).trim() || error.message;
-        reject(new Error(`"${file} ${args.join(" ")}" failed: ${msg}`));
+        const wrapped = new Error(`"${file} ${args.join(" ")}" failed: ${msg}`);
+        // Preserve ENOENT and similar system error codes for callers
+        const errno = error as NodeJS.ErrnoException;
+        if (errno.code) {
+          (wrapped as NodeJS.ErrnoException).code = errno.code;
+        }
+        reject(wrapped);
         return;
       }
       resolve({
@@ -31,7 +37,37 @@ async function run(
  * Throws on non-zero exit, timeout, or other exec error.
  */
 export async function tndm(args: string[]): Promise<ExecResult> {
-  return run("tndm", args, { timeout: 30_000 });
+  try {
+    return await run("tndm", args, { timeout: 30_000 });
+  } catch (error) {
+    if (
+      error instanceof Error &&
+      (error as NodeJS.ErrnoException).code === "ENOENT"
+    ) {
+      throw new Error(
+        "tndm is not installed or not on your PATH.\n\n" +
+          "Install it with one of:\n" +
+          "  brew install mrclrchtr/tap/tndm\n" +
+          "  cargo install tandem-cli\n" +
+          "  curl -LsSf https://github.com/mrclrchtr/tandem/releases/latest/download/tandem-cli-installer.sh | sh\n",
+      );
+    }
+    throw error;
+  }
+}
+
+/**
+ * Run tndm --version and return the parsed semver string, or null if unavailable.
+ * Never throws — callers handle absence gracefully.
+ */
+export async function tndmVersion(): Promise<string | null> {
+  try {
+    const { stdout } = await run("tndm", ["--version"], { timeout: 5_000 });
+    const match = stdout.match(/tndm\s+(\d+\.\d+\.\d+)/);
+    return match ? match[1] : null;
+  } catch {
+    return null;
+  }
 }
 
 /**
@@ -53,28 +89,3 @@ export async function tndmJson<T = Record<string, unknown>>(
     );
   }
 }
-
-/**
- * Run `git add .tndm/` and `git commit -m <message>`.
- * Uses `git diff --cached --quiet` to check for staged changes via exit code,
- * avoiding locale-dependent string parsing.
- * Throws on non-zero exit from `git commit`.
- */
-export async function gitAddCommit(message: string): Promise<{ commitHash: string }> {
-  await run("git", ["add", ".tndm/"]);
-
-  // Check exit code instead of parsing locale-dependent output strings.
-  // git diff --cached --quiet exits 0 (no staged changes), non-zero (changes exist or error).
-  try {
-    await run("git", ["diff", "--cached", "--quiet"]);
-    // Exit 0: no changes staged — nothing to commit
-    return { commitHash: "" };
-  } catch {
-    // Exit non-zero: changes exist, or a real git error.
-    // Proceed to commit; real errors will surface there.
-  }
-
-  const { stdout } = await run("git", ["commit", "-m", message]);
-  const match = stdout.match(/\[[^\]]+ ([a-f0-9]+)\]/);
-  return { commitHash: match ? match[1] : "" };
-}

package/{src → extensions}/index.ts

@@ -1,8 +1,9 @@
+import { readFileSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
-import { tndmJson } from "./cli.js";
+import { tndmVersion } from "./cli.js";
 import { supi_tndm_cli_params, executeTndmCli } from "./tools/tndm-cli.js";
 import {
   supiFlowStartParams,
@@ -16,13 +17,34 @@ import {
 } from "./tools/flow-tools.js";
 
 const baseDir = dirname(dirname(fileURLToPath(import.meta.url)));
+const pkg = JSON.parse(readFileSync(join(baseDir, "package.json"), "utf-8"));
+export const FLOW_VERSION: string = pkg.version;
+
+/**
+ * Check tndm version against supi-flow version. Notifies on mismatch.
+ * Exported for testing.
+ */
+export async function checkTndmVersion(
+  event: { reason: string },
+  ctx: { ui: { notify: (message: string, type?: "info" | "warning" | "error") => void } },
+): Promise<void> {
+  if (event.reason !== "startup" && event.reason !== "reload") return;
+  const tndmVer = await tndmVersion();
+  if (!tndmVer) return;
+  if (tndmVer !== FLOW_VERSION) {
+    ctx.ui.notify(
+      `tndm v${tndmVer} found, but supi-flow expects v${FLOW_VERSION}. ` +
+        `Install matching version: brew install mrclrchtr/tap/tndm`,
+      "warning",
+    );
+  }
+}
 
 export default function (pi: ExtensionAPI) {
-  // ── Resource discovery ──────────────────────────────────────
-  pi.on("resources_discover", () => ({
-    skillPaths: [join(baseDir, "skills")],
-    promptPaths: [join(baseDir, "prompts")],
-  }));
+  // ── Version check on startup ────────────────────────────────
+  pi.on("session_start", async (event, ctx) => {
+    await checkTndmVersion(event, ctx);
+  });
 
   // ── Tool: supi_tndm_cli ─────────────────────────────────────
   pi.registerTool({
@@ -105,8 +127,7 @@ export default function (pi: ExtensionAPI) {
     label: "Flow Close",
     description:
       "Close a ticket and finalize the flow. " +
-      "Writes verification results to archive.md, sets status=done, tags=flow:done, " +
-      "and auto-commits .tndm/ changes.",
+      "Writes verification results to archive.md, sets status=done, and tags=flow:done.",
     promptSnippet: "Close a TNDM ticket after implementation and verification",
     promptGuidelines: [
       "Use supi_flow_close at the end of the archive phase after all verification is complete",
@@ -118,50 +139,4 @@ export default function (pi: ExtensionAPI) {
     },
   });
 
-  // ── Command: /supi-flow-status ──────────────────────────────
-  pi.registerCommand("supi-flow-status", {
-    description: "Show current flow workflow state",
-    handler: async (_args, ctx) => {
-      const tickets = await tndmJson<Array<{ id: string; status: string; tags?: string[] }>>([
-        "ticket",
-        "list",
-      ]);
-      const activeTickets = tickets.filter((ticket) => {
-        if (ticket.status === "done") return false;
-        const tags = ticket.tags ?? [];
-        return tags.includes("flow:brainstorm") || tags.includes("flow:planned") || tags.includes("flow:applying");
-      });
-
-      if (activeTickets.length === 0) {
-        ctx.ui.notify("No active flow tickets. Start with /skill:supi-flow-brainstorm.", "info");
-        return;
-      }
-
-      const lines = activeTickets.map((ticket) => {
-        const tags = ticket.tags ?? [];
-        const nextStep = tags.includes("flow:applying")
-          ? `/skill:supi-flow-archive ${ticket.id}`
-          : tags.includes("flow:planned")
-            ? `/skill:supi-flow-apply ${ticket.id}`
-            : `/skill:supi-flow-plan ${ticket.id}`;
-        return `${ticket.id} (${ticket.status}) -> ${nextStep}`;
-      });
-
-      ctx.ui.notify(`Active flow tickets:\n${lines.join("\n")}`, "info");
-    },
-  });
-
-  // ── Command: /supi-flow ─────────────────────────────────────
-  pi.registerCommand("supi-flow", {
-    description: "List available flow workflow commands",
-    handler: async (_args, ctx) => {
-      ctx.ui.notify(
-        "Flow: /skill:supi-flow-brainstorm -> /skill:supi-flow-plan -> /skill:supi-flow-apply -> /skill:supi-flow-archive\n" +
-          "  /supi-flow-status -- show current state\n" +
-          "  /supi-flow        -- this help\n" +
-          "Available tools: supi_tndm_cli, supi_flow_start, supi_flow_plan, supi_flow_complete_task, supi_flow_close",
-        "info",
-      );
-    },
-  });
 }

package/{src → extensions}/tools/flow-tools.ts

@@ -2,7 +2,7 @@ import { dirname, join } from "node:path";
 import { readFileSync, writeFileSync } from "node:fs";
 import { type Static, Type } from "typebox";
 import { StringEnum } from "@earendil-works/pi-ai";
-import { gitAddCommit, tndm, tndmJson } from "../cli.js";
+import { tndm, tndmJson } from "../cli.js";
 
 // ─── supi_flow_start ───────────────────────────────────────────
 
@@ -10,12 +10,14 @@ export const supiFlowStartParams = Type.Object({
   title: Type.String({ description: "Ticket title describing the change" }),
   priority: Type.Optional(
     StringEnum(["p0", "p1", "p2", "p3", "p4"] as const, {
-      description: "Priority (default: p2)",
+      description: "Priority",
+      default: "p2",
     }),
   ),
   type: Type.Optional(
     StringEnum(["task", "bug", "feature", "chore", "epic"] as const, {
-      description: "Ticket type (default: task)",
+      description: "Ticket type",
+      default: "task",
     }),
   ),
   context: Type.Optional(
@@ -41,8 +43,12 @@ export async function executeFlowStart(params: FlowStartParams) {
   if (params.priority) args.push("--priority", params.priority);
   if (params.type) args.push("--type", params.type);
 
-  const result = await tndm(args);
-  const ticketId = result.stdout.trim();
+  // Use --json on create to get id + content_path in one call
+  const createResult = await tndmJson<{ id: string; content_path?: string }>(args);
+  const ticketId = createResult.id;
+  const contentPath = createResult.content_path ?? "";
+  const ticketDir = contentPath ? dirname(contentPath) : "";
+  const pathInfo = ticketDir ? ` at ${ticketDir}` : "";
 
   if (params.context) {
     await tndm(["ticket", "update", ticketId, "--content", params.context]);
@@ -52,10 +58,16 @@ export async function executeFlowStart(params: FlowStartParams) {
     content: [
       {
         type: "text" as const,
-        text: `Created ticket ${ticketId} with status=todo and flow:brainstorm tag.`,
+        text: `Created ticket ${ticketId}${pathInfo} with status=todo and flow:brainstorm tag.`,
       },
     ],
-    details: { action: "flow_start", ticketId, status: "todo", tags: "flow:brainstorm" },
+    details: {
+      action: "flow_start",
+      ticketId,
+      ticketPath: ticketDir,
+      status: "todo",
+      tags: "flow:brainstorm",
+    },
   };
 }
 
@@ -79,8 +91,14 @@ export type FlowPlanParams = Static<typeof supiFlowPlanParams>;
 
 export async function executeFlowPlan(params: FlowPlanParams) {
   // Create a "plan" document and get its path
-  const docResult = await tndm(["ticket", "doc", "create", params.ticket_id, "plan"]);
-  const docPath = docResult.stdout.trim();
+  const docResult = await tndmJson<{ path: string }>([
+    "ticket",
+    "doc",
+    "create",
+    params.ticket_id,
+    "plan",
+  ]);
+  const docPath = docResult.path;
 
   let content = params.plan_content;
 
@@ -100,14 +118,23 @@ export async function executeFlowPlan(params: FlowPlanParams) {
 
   // Sync fingerprints and update tags
   await tndm(["ticket", "sync", params.ticket_id]);
+
+  // Replace any flow-state tag with flow:planned — remove all possible flow-state tags
+  // first, then add flow:planned, to work correctly regardless of the ticket's current
+  // flow state (brainstorm, planned, applying, or done).
+  await tndm([
+    "ticket",
+    "update",
+    params.ticket_id,
+    "--remove-tags",
+    "flow:brainstorm,flow:planned,flow:applying,flow:done",
+  ]);
   await tndm([
     "ticket",
     "update",
     params.ticket_id,
     "--add-tags",
     "flow:planned",
-    "--remove-tags",
-    "flow:brainstorm",
   ]);
 
   return {
@@ -287,16 +314,28 @@ export async function executeFlowClose(params: FlowCloseParams) {
 
   if (params.verification_results) {
     // Create/register archive.md via document registry, then write results
-    try {
-      const docResult = await tndm(["ticket", "doc", "create", params.ticket_id, "archive"]);
-      archivePath = docResult.stdout.trim();
-      writeFileSync(archivePath, `# Archive\n\n${params.verification_results}\n`, "utf-8");
-      await tndm(["ticket", "sync", params.ticket_id]);
-    } catch {
-      // Non-fatal if doc create fails
-    }
+    const docResult = await tndmJson<{ path: string }>([
+      "ticket",
+      "doc",
+      "create",
+      params.ticket_id,
+      "archive",
+    ]);
+    archivePath = docResult.path;
+    writeFileSync(archivePath, `# Archive\n\n${params.verification_results}\n`, "utf-8");
+    await tndm(["ticket", "sync", params.ticket_id]);
   }
 
+  // Replace any flow-state tag with flow:done — remove all possible flow-state tags
+  // first, then set status and add flow:done, to work correctly regardless of the
+  // ticket's current flow state.
+  await tndm([
+    "ticket",
+    "update",
+    params.ticket_id,
+    "--remove-tags",
+    "flow:brainstorm,flow:planned,flow:applying,flow:done",
+  ]);
   await tndm([
     "ticket",
     "update",
@@ -305,25 +344,13 @@ export async function executeFlowClose(params: FlowCloseParams) {
     "done",
     "--add-tags",
     "flow:done",
-    "--remove-tags",
-    "flow:applying",
   ]);
 
-  let commitHash = "";
-  try {
-    const commitResult = await gitAddCommit(`chore(tndm): close ${params.ticket_id}`);
-    commitHash = commitResult.commitHash;
-  } catch {
-    // Non-fatal if commit fails
-  }
-
   return {
     content: [
       {
         type: "text" as const,
-        text: `Ticket ${params.ticket_id} closed (status=done, flow:done).${
-          commitHash ? ` Committed as ${commitHash}.` : ""
-        }`,
+        text: `Ticket ${params.ticket_id} closed (status=done, flow:done).`,
       },
     ],
     details: {
@@ -331,7 +358,6 @@ export async function executeFlowClose(params: FlowCloseParams) {
       ticketId: params.ticket_id,
       status: "done",
       tags: "flow:done",
-      commitHash,
     },
   };
 }

package/{src → extensions}/tools/tndm-cli.ts

@@ -8,8 +8,6 @@ export const actionEnum = StringEnum([
   "show",
   "list",
   "awareness",
-  "doc_create",
-  "sync",
 ] as const);
 
 export const supi_tndm_cli_params = Type.Object({
@@ -61,11 +59,6 @@ export const supi_tndm_cli_params = Type.Object({
     Type.String({ description: "Markdown content body for the ticket" }),
   ),
 
-  // Document params
-  name: Type.Optional(
-    Type.String({ description: "Document name for doc_create (e.g. 'plan', 'archive')" }),
-  ),
-
   // List params
   all: Type.Optional(Type.Boolean({ description: "Include done tickets in list" })),
   definition: Type.Optional(
@@ -89,6 +82,7 @@ export const supi_tndm_cli_params = Type.Object({
  *   show       → tndm ticket show <id> --json
  *   list       → tndm ticket list [--all] [--definition <state>] --json
  *   awareness  → tndm awareness --against <ref> --json
+ *   doc_create and sync are internal operations used by flow tools, not exposed here.
  */
 export type TndmCliParams = Static<typeof supi_tndm_cli_params>;
 
@@ -192,43 +186,6 @@ export async function executeTndmCli(params: TndmCliParams) {
         details: { action: "awareness", awareness: result },
       };
     }
-
-    case "doc_create": {
-      if (!params.id) {
-        throw new Error("supi_tndm_cli: id is required for doc_create");
-      }
-      if (!params.name) {
-        throw new Error("supi_tndm_cli: name is required for doc_create");
-      }
-      const result = await tndm(["ticket", "doc", "create", params.id, params.name]);
-      return {
-        content: [
-          {
-            type: "text" as const,
-            text: result.stdout || `Document '${params.name}' created for ${params.id}`,
-          },
-        ],
-        details: {
-          action: "doc_create",
-          ticketId: params.id,
-          name: params.name,
-          path: result.stdout.trim(),
-        },
-      };
-    }
-
-    case "sync": {
-      if (!params.id) {
-        throw new Error("supi_tndm_cli: id is required for sync");
-      }
-      const result = await tndm(["ticket", "sync", params.id]);
-      return {
-        content: [
-          { type: "text" as const, text: result.stdout || `Ticket ${params.id} synced` },
-        ],
-        details: { action: "sync", ticketId: params.id },
-      };
-    }
   }
 }
 

package/package.json

@@ -1,15 +1,21 @@
 {
   "name": "@mrclrchtr/supi-flow",
-  "version": "0.6.1",
+  "version": "0.10.0",
   "private": false,
-  "description": "SuPi flow extension — spec-driven workflow coupled to TNDM ticket coordination",
+  "description": "PI extension for spec-driven workflow (brainstorm → plan → apply → archive) with TNDM ticket coordination, custom tools, and 5 auto-discovered skills",
+  "keywords": [
+    "pi-package",
+    "pi-extension",
+    "tndm",
+    "workflow",
+    "ticket-coordination"
+  ],
   "license": "Apache-2.0",
-  "pi": {
-    "extensions": [
-      "./src/index.ts"
-    ]
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/mrclrchtr/tandem"
   },
-  "main": "src/index.ts",
+  "main": "extensions/index.ts",
   "dependencies": {
     "typebox": "^1.1.38"
   },
@@ -18,15 +24,15 @@
     "@earendil-works/pi-coding-agent": "*"
   },
   "files": [
-    "src/",
+    "extensions/",
     "skills/",
     "prompts/",
     "README.md"
   ],
   "devDependencies": {
-    "@earendil-works/pi-ai": "^0.74.0",
-    "@earendil-works/pi-coding-agent": "*",
-    "@types/node": "^25.6.0",
-    "vitest": "^4.1.4"
+    "@earendil-works/pi-ai": "0.74.0",
+    "@earendil-works/pi-coding-agent": "0.74.0",
+    "@types/node": "25.7.0",
+    "vitest": "4.1.6"
   }
 }

package/prompts/supi-coding-retro.md

@@ -2,17 +2,62 @@
 description: Agent retrospective on project setup, architecture, tooling, and workflows
 ---
 
-Agent Retrospective — Project Setup & Tooling
+# Agent Retrospective — Project Setup & Tooling
 
-While implementing this feature, what aspects of the project setup, architecture, tooling, workflows, or conventions made development harder than necessary?
+Reflect on what made development harder than necessary during this coding session.
 
-Please specifically identify:
+Focus only on friction points actually encountered while implementing, debugging, validating, or navigating the task. Do not infer or invent issues from the repository structure, architecture, codebase shape, or available tooling alone.
+
+Do not do additional research or inspect the repository again. Use only your memory of this session.
+
+If no concrete friction points were encountered, write exactly:
+
+> No concrete friction points encountered during this session.
+
+Consider concrete examples such as:
 - sources of friction or unnecessary complexity
 - confusing or inconsistent patterns
 - tooling or processes that slowed implementation
-- over-engineered abstractions
+- over-engineered abstractions encountered during the work
 - repetitive manual work that could be automated
-- missing documentation, scripts, or conventions
-- anything that should be simplified, standardized, or removed
+- missing documentation, scripts, tests, or conventions
+- setup, validation, CI, or workflow issues
+- anything that should be simplified, standardized, removed, or made more obvious
+
+Be honest and specific. The goal is to identify practical improvements that would reduce unnecessary work, reduce token usage, reduce debugging, or improve outcomes in future sessions. This is not a complaint log.
+
+## Output Format
+
+Group items under the sections below. Skip sections that have no concrete items.
+
+Sections are grouped by where the fix should happen. Each friction point should appear once, under the best-fitting section.
+
+Order items by expected future work avoided, highest first.
+
+Each item should include:
+
+- **Problem**: What concrete friction occurred?
+- **Impact**: What extra work, risk, delay, token usage, or confusion it caused.
+- **Fix**: The smallest concrete change that would prevent or reduce it next time.
+
+## Sections
+
+### Repository Changes
+
+Changes to files, code organization, architecture, abstractions, conventions, tests, or docs.
+
+### Tooling & Process Changes
+
+Changes to scripts, commands, setup flows, CI checks, local validation, automation, or manual workflows.
+
+### Harness / Agent Tool Changes
+
+Changes to the agent harness, available tools, PI extensions, context providers, retrieval tools, or generated prompt context.
+
+### Prompt / Context Changes
+
+Changes to the coding-session prompt, repo instructions, task framing, generated context, or other information shown to the agent.
+
+### Other
 
-Focus on concrete examples and practical improvements that would increase implementation speed, reliability, and developer/agent experience for future work.
+Anything that should be simplified, standardized, removed, or made more obvious that does not fit above.