style: apply dprint formatting

simonheimlicher · simonheimlicher · commit 4e22bd34338e · 2026-03-26T14:50:54.000+01:00
Trailing commas, operator-first line breaks, import sorting,
and whitespace normalization across src, tests, specs, and spx.
diff --git a/eslint-rules/no-bdd-try-catch-anti-pattern.ts b/eslint-rules/no-bdd-try-catch-anti-pattern.ts
@@ -31,8 +31,7 @@ const rule: Rule.RuleModule = {
         "BDD violation: Conditional assertions after catch blocks hide test failures. Make assertions unconditional.",
       bddViolationGenericSwallowing:
         "BDD violation: Generic error swallowing in tests prevents proper failure reporting. Let tests fail loudly.",
-      suggestProperBdd:
-        "Consider: Remove try-catch and let test fail, or use expect().rejects for error testing.",
+      suggestProperBdd: "Consider: Remove try-catch and let test fail, or use expect().rejects for error testing.",
     },
   },
   create(context: Rule.RuleContext) {
@@ -42,10 +41,10 @@ const rule: Rule.RuleModule = {
     function isTestFile() {
       const filename = context.getFilename?.() || context.filename;
       return (
-        filename.includes(".test.") ||
-        filename.includes(".spec.") ||
-        filename.includes("/tests/") ||
-        filename.includes("/__tests__/")
+        filename.includes(".test.")
+        || filename.includes(".spec.")
+        || filename.includes("/tests/")
+        || filename.includes("/__tests__/")
       );
     }
     /**
@@ -59,9 +58,9 @@ const rule: Rule.RuleModule = {
       }
       // Chained expect().method() calls
       if (
-        node.callee?.type === "MemberExpression" &&
-        node.callee.object?.type === "CallExpression" &&
-        node.callee.object.callee?.name === "expect"
+        node.callee?.type === "MemberExpression"
+        && node.callee.object?.type === "CallExpression"
+        && node.callee.object.callee?.name === "expect"
       ) {
         return true;
       }
@@ -112,18 +111,18 @@ const rule: Rule.RuleModule = {
         if (!node) return false;
         // Check for "error instanceof SomeError" pattern
         if (
-          node.type === "BinaryExpression" &&
-          node.operator === "instanceof" &&
-          node.right?.name
+          node.type === "BinaryExpression"
+          && node.operator === "instanceof"
+          && node.right?.name
         ) {
           return notImplementedErrorPatterns.includes(node.right.name);
         }
         // Check for error.constructor === SomeError
         if (
-          node.type === "BinaryExpression" &&
-          node.operator === "===" &&
-          node.left?.type === "MemberExpression" &&
-          node.left.property?.name === "constructor"
+          node.type === "BinaryExpression"
+          && node.operator === "==="
+          && node.left?.type === "MemberExpression"
+          && node.left.property?.name === "constructor"
         ) {
           return notImplementedErrorPatterns.includes(node.right?.name);
         }
diff --git a/specs/work/doing/capability-15_infrastructure/feature-21_testable-validation/story-48_validation-output-enhancements/validation-output-enhancements.story.md b/specs/work/doing/capability-15_infrastructure/feature-21_testable-validation/story-48_validation-output-enhancements/validation-output-enhancements.story.md
@@ -86,6 +86,7 @@ AND durations >= 1000ms show as "X.Xs"
 ## Testing Strategy
 
 > Use `/testing-typescript` skill to understand testing strategy.
+
 ### Level Assignment
 
 | Component                  | Level | Justification                    |
diff --git a/specs/work/doing/capability-27_spec-domain/spec-domain.capability.md b/specs/work/doing/capability-27_spec-domain/spec-domain.capability.md
@@ -10,7 +10,6 @@
 
 ## Testing Strategy
 
-
 ### Level Assignment
 
 | Component             | Level | Justification                                       |
diff --git a/specs/work/doing/capability-33_claude-settings/claude-settings.capability.md b/specs/work/doing/capability-33_claude-settings/claude-settings.capability.md
@@ -61,7 +61,7 @@ describe("Capability: Claude Settings - Marketplace Install", () => {
     // Verify marketplace files exist
     const marketplacePath = path.join(
       tempHome,
-      ".claude/plugins/cache/spx-claude",
+      ".claude/plugins/cache/outcomeeng",
     );
     expect(await exists(marketplacePath)).toBe(true);
     expect(
@@ -175,7 +175,7 @@ describe("Capability: Claude Settings - Marketplace Update", () => {
     // Verify new plugin exists in installed marketplace
     const installedPath = path.join(
       tempHome,
-      ".claude/plugins/cache/spx-claude",
+      ".claude/plugins/cache/outcomeeng",
     );
     expect(
       await exists(path.join(installedPath, "plugins/new-plugin")),
@@ -252,7 +252,7 @@ This capability provides the `spx claude` command namespace for user-facing Clau
 
 | Command                           | Purpose                                                  |
 | --------------------------------- | -------------------------------------------------------- |
-| `spx claude init`                 | Install spx-claude marketplace into Claude Code          |
+| `spx claude init`                 | Install outcomeeng marketplace into Claude Code          |
 | `spx claude update`               | Update marketplace from configured source                |
 | `spx claude status`               | Show marketplace installation status                     |
 | `spx claude settings consolidate` | Consolidate permissions from projects into global config |
diff --git a/specs/work/doing/capability-33_claude-settings/claude-settings.prd.md b/specs/work/doing/capability-33_claude-settings/claude-settings.prd.md
@@ -22,7 +22,7 @@
 ```
 As a Claude Code user, I face two interconnected problems:
 
-1. I must manually install and update the spx-claude marketplace,
+1. I must manually install and update the outcomeeng marketplace,
    which is error-prone and requires knowing the source URL.
 
 2. I constantly re-approve the same permissions across multiple projects,
@@ -56,7 +56,7 @@ and wastes time on repetitive permission approvals.
 ```
 Implement spx claude subcommands that:
 
-1. Install and update the spx-claude marketplace from a configured source,
+1. Install and update the outcomeeng marketplace from a configured source,
    reading Claude Code's configuration to enable one-command management.
 
 2. Consolidate all project-local permissions into the global settings file,
@@ -73,20 +73,20 @@ This results in streamlined marketplace updates and centralized permission manag
 
 #### `spx claude init [--source <url|path>]`
 
-Install spx-claude marketplace into user's Claude Code setup.
+Install outcomeeng marketplace into user's Claude Code setup.
 
 ```bash
 # Install from GitHub (default)
 spx claude init
 
 # Install from local path (development)
-spx claude init --source ~/Code/spx/spx-claude
+spx claude init --source ~/Code/spx/outcomeeng/claude
 ```
 
 **Behavior:**
 
 1. Detect Claude Code plugins directory (`~/.claude/plugins/` or platform equivalent)
-2. If `--source` not provided, use default: `https://github.com/shz/spx-claude`
+2. If `--source` not provided, use default: `https://github.com/shz/outcomeeng/claude`
 3. Clone/copy marketplace to plugins cache
 4. Register marketplace in Claude Code settings
 
@@ -404,7 +404,7 @@ describe("Feature: spx claude init", () => {
     expect(exitCode).toBe(0);
     expect(stdout).toContain("Marketplace installed");
     expect(
-      await exists(path.join(tempHome, ".claude/plugins/cache/spx-claude")),
+      await exists(path.join(tempHome, ".claude/plugins/cache/outcomeeng/claude")),
     ).toBe(true);
   });
 
@@ -719,7 +719,7 @@ describe("Feature: spx claude settings consolidate", () => {
 
 ```bash
 # Marketplace management
-spx claude init                              # Install spx-claude marketplace
+spx claude init                              # Install outcomeeng marketplace
 spx claude init --source ~/local             # Install from local path
 spx claude update                            # Update from configured source
 spx claude status                            # Show installation status
diff --git a/specs/work/doing/capability-33_claude-settings/decisions/adr-002_security-first-conflict-resolution.md b/specs/work/doing/capability-33_claude-settings/decisions/adr-002_security-first-conflict-resolution.md
@@ -111,9 +111,9 @@ fc.assert(
     const result2 = resolveConflicts(permissions);
 
     return (
-      JSON.stringify(result1.resolved.allow.sort()) ===
-        JSON.stringify(result2.resolved.allow.sort()) &&
-      JSON.stringify(result1.resolved.deny.sort()) === JSON.stringify(result2.resolved.deny.sort())
+      JSON.stringify(result1.resolved.allow.sort())
+        === JSON.stringify(result2.resolved.allow.sort())
+      && JSON.stringify(result1.resolved.deny.sort()) === JSON.stringify(result2.resolved.deny.sort())
     );
   }),
 );
@@ -164,9 +164,9 @@ fc.assert(
     const twice = resolveConflicts(once.resolved);
 
     return (
-      JSON.stringify(once.resolved.allow.sort()) === JSON.stringify(twice.resolved.allow.sort()) &&
-      JSON.stringify(once.resolved.deny.sort()) === JSON.stringify(twice.resolved.deny.sort()) &&
-      twice.conflictCount === 0 // No conflicts after first resolution
+      JSON.stringify(once.resolved.allow.sort()) === JSON.stringify(twice.resolved.allow.sort())
+      && JSON.stringify(once.resolved.deny.sort()) === JSON.stringify(twice.resolved.deny.sort())
+      && twice.conflictCount === 0 // No conflicts after first resolution
     );
   }),
 );
diff --git a/specs/work/doing/capability-33_claude-settings/decisions/adr-003_atomic-writes-with-temp-files.md b/specs/work/doing/capability-33_claude-settings/decisions/adr-003_atomic-writes-with-temp-files.md
@@ -214,7 +214,7 @@ describe("writeSettings - Error Handling", () => {
     await writeSettings("/target/settings.json", mockSettings, { fs: testFs });
 
     // Verify 2-space indent and trailing newline
-    expect(writtenContent).toContain('  "permissions"');
+    expect(writtenContent).toContain("  \"permissions\"");
     expect(writtenContent).toEndWith("\n");
     expect(JSON.parse(writtenContent)).toEqual(mockSettings);
   });
diff --git a/specs/work/doing/capability-33_claude-settings/feature-54_settings-consolidation/story-21_discovery-parsing/tests/unit/parser.test.ts b/specs/work/doing/capability-33_claude-settings/feature-54_settings-consolidation/story-21_discovery-parsing/tests/unit/parser.test.ts
@@ -88,7 +88,7 @@ describe("parsePermission", () => {
 
   test("throws error for malformed permission - no parentheses", () => {
     expect(() => parsePermission("Bash git:*", "allow")).toThrow(
-      'Malformed permission string: "Bash git:*"',
+      "Malformed permission string: \"Bash git:*\"",
     );
   });
 
diff --git a/specs/work/doing/capability-33_claude-settings/feature-54_settings-consolidation/story-32_subsumption-merging/tests/unit/merger.properties.test.ts b/specs/work/doing/capability-33_claude-settings/feature-54_settings-consolidation/story-32_subsumption-merging/tests/unit/merger.properties.test.ts
@@ -80,11 +80,11 @@ describe("resolveConflicts - Property: Determinism", () => {
         const subsumed2Sorted = JSON.stringify([...result2.subsumed].sort());
 
         return (
-          allow1Sorted === allow2Sorted &&
-          deny1Sorted === deny2Sorted &&
-          ask1Sorted === ask2Sorted &&
-          countsMatch &&
-          subsumed1Sorted === subsumed2Sorted
+          allow1Sorted === allow2Sorted
+          && deny1Sorted === deny2Sorted
+          && ask1Sorted === ask2Sorted
+          && countsMatch
+          && subsumed1Sorted === subsumed2Sorted
         );
       }),
       { numRuns: 100 },
diff --git a/spx/21-core-cli.capability/21-pattern-matching.feature/54-validate-bsp-numbers.story/tests/validation.unit.test.ts b/spx/21-core-cli.capability/21-pattern-matching.feature/54-validate-bsp-numbers.story/tests/validation.unit.test.ts
@@ -2,8 +2,8 @@
  * Level 1: Pure function tests for BSP number validation
  * Story: story-54_validate-bsp-numbers
  */
-import { describe, it, expect } from "vitest";
 import { isValidBSPNumber, validateBSPNumber } from "@/scanner/validation";
+import { describe, expect, it } from "vitest";
 
 describe("isValidBSPNumber", () => {
   it("GIVEN number 10 WHEN validating THEN returns true", () => {
diff --git a/spx/21-core-cli.capability/21-pattern-matching.feature/54-validate-bsp-numbers.story/validate-bsp-numbers.story.md b/spx/21-core-cli.capability/21-pattern-matching.feature/54-validate-bsp-numbers.story/validate-bsp-numbers.story.md
@@ -67,10 +67,7 @@ This story stays at Level 1 because:
 ```typescript
 // test/unit/scanner/validation.test.ts
 import { describe, expect, it } from "vitest";
-import {
-  isValidBSPNumber,
-  validateBSPNumber,
-} from "../../../src/scanner/validation";
+import { isValidBSPNumber, validateBSPNumber } from "../../../src/scanner/validation";
 
 describe("isValidBSPNumber", () => {
   /**
diff --git a/spx/21-core-cli.capability/21-pattern-matching.feature/65-test-factories.story/test-factories.story.md b/spx/21-core-cli.capability/21-pattern-matching.feature/65-test-factories.story/test-factories.story.md
@@ -64,11 +64,7 @@ This story stays at Level 1 because:
 ```typescript
 // test/unit/fixtures/factories.test.ts
 import { describe, expect, it } from "vitest";
-import {
-  createRandomWorkItem,
-  createWorkItem,
-  createWorkItemName,
-} from "../../fixtures/factories";
+import { createRandomWorkItem, createWorkItem, createWorkItemName } from "../../fixtures/factories";
 
 describe("createWorkItemName", () => {
   /**
@@ -245,7 +241,6 @@ function randomBSPNumber(): number {
 
 ## Architectural Requirements
 
-
 ## Quality Requirements
 
 ### QR1: Type Safety
diff --git a/spx/21-core-cli.capability/32-directory-walking.feature/32-pattern-filter.story/tests/walk.unit.test.ts b/spx/21-core-cli.capability/32-directory-walking.feature/32-pattern-filter.story/tests/walk.unit.test.ts
@@ -2,8 +2,8 @@
  * Level 1: Unit tests for pattern filtering
  * Story: story-32_pattern-filter
  */
-import { describe, it, expect } from "vitest";
 import { filterWorkItemDirectories } from "@/scanner/walk";
+import { describe, expect, it } from "vitest";
 
 describe("filterWorkItemDirectories", () => {
   it("GIVEN directories with work item names WHEN filtering THEN includes them", () => {
diff --git a/spx/21-core-cli.capability/32-directory-walking.feature/54-edge-cases.story/tests/walk.unit.test.ts b/spx/21-core-cli.capability/32-directory-walking.feature/54-edge-cases.story/tests/walk.unit.test.ts
@@ -2,8 +2,8 @@
  * Level 1: Unit tests for path normalization
  * Story: story-54_edge-cases
  */
-import { describe, it, expect } from "vitest";
 import { normalizePath } from "@/scanner/walk";
+import { describe, expect, it } from "vitest";
 
 describe("normalizePath", () => {
   /**
diff --git a/spx/21-core-cli.capability/87-e2e-workflow.feature/21-fixture-generator.story/fixture-generator.story.md b/spx/21-core-cli.capability/87-e2e-workflow.feature/21-fixture-generator.story/fixture-generator.story.md
@@ -98,10 +98,7 @@ This story uses Level 1 only because:
 
 ```typescript
 // tests/unit/helpers/fixture-generator.test.ts
-import {
-  generateFixtureTree,
-  PRESETS,
-} from "@/tests/helpers/fixture-generator";
+import { generateFixtureTree, PRESETS } from "@/tests/helpers/fixture-generator";
 import { describe, expect, it } from "vitest";
 
 describe("generateFixtureTree", () => {
@@ -175,8 +172,7 @@ describe("PRESETS", () => {
   });
 
   it("SHALLOW_50 produces ~50 items (wide, shallow)", () => {
-    const { capabilities: c, featuresPerCapability: f, storiesPerFeature: s } =
-      PRESETS.SHALLOW_50;
+    const { capabilities: c, featuresPerCapability: f, storiesPerFeature: s } = PRESETS.SHALLOW_50;
     const total = c + c * f + c * f * s;
     expect(total).toBeGreaterThanOrEqual(50);
   });
diff --git a/spx/21-core-cli.capability/87-e2e-workflow.feature/32-fixture-writer.story/fixture-writer.story.md b/spx/21-core-cli.capability/87-e2e-workflow.feature/32-fixture-writer.story/fixture-writer.story.md
@@ -107,10 +107,7 @@ This story uses Level 2 because:
 
 ```typescript
 // tests/integration/helpers/fixture-writer.test.ts
-import {
-  generateFixtureTree,
-  PRESETS,
-} from "@/tests/helpers/fixture-generator";
+import { generateFixtureTree, PRESETS } from "@/tests/helpers/fixture-generator";
 import { materializeFixture } from "@/tests/helpers/fixture-writer";
 import { existsSync, readdirSync } from "node:fs";
 import { afterEach, describe, expect, it } from "vitest";
diff --git a/spx/21-core-cli.capability/87-e2e-workflow.feature/43-e2e-validation.story/e2e-validation.story.md b/spx/21-core-cli.capability/87-e2e-workflow.feature/43-e2e-validation.story/e2e-validation.story.md
@@ -237,9 +237,7 @@ describe("E2E: Output Formats", () => {
       ["bin/spx.js", "status"],
       { cwd: fixture.path },
     );
-    const textLines = textOut.split("\n").filter(l =>
-      l.match(/capability-|feature-|story-/)
-    );
+    const textLines = textOut.split("\n").filter(l => l.match(/capability-|feature-|story-/));
     expect(textLines.length).toBe(expectedTotal);
   });
 });
diff --git a/spx/26-scoped-cli.capability/21-domain-router.feature/21-domain-router-infrastructure.story/domain-router-infrastructure.story.md b/spx/26-scoped-cli.capability/21-domain-router.feature/21-domain-router-infrastructure.story/domain-router-infrastructure.story.md
@@ -165,12 +165,7 @@ Level 1 is sufficient because:
 
 ```typescript
 // specs/doing/capability-26_scoped-cli/feature-21_domain-router/story-21_domain-router-infrastructure/tests/domain-registry.test.ts
-import {
-  clearDomains,
-  getAllDomains,
-  getDomain,
-  registerDomain,
-} from "@/domains/registry";
+import { clearDomains, getAllDomains, getDomain, registerDomain } from "@/domains/registry";
 import type { Domain } from "@/domains/types";
 import { beforeEach, describe, expect, it } from "vitest";
 
diff --git a/spx/26-scoped-cli.capability/21-domain-router.feature/32-spec-domain-implementation.story/spec-domain-implementation.story.md b/spx/26-scoped-cli.capability/21-domain-router.feature/32-spec-domain-implementation.story/spec-domain-implementation.story.md
@@ -49,10 +49,7 @@ THEN register status and next commands under spec namespace
 // src/domains/spec/index.ts
 import type { Command } from "commander";
 import { nextCommand } from "../../commands/spec/next.js";
-import {
-  type OutputFormat,
-  statusCommand,
-} from "../../commands/spec/status.js";
+import { type OutputFormat, statusCommand } from "../../commands/spec/status.js";
 import type { Domain } from "../types.js";
 
 /**
@@ -79,9 +76,7 @@ function registerSpecCommands(specCmd: Command): void {
             format = options.format as OutputFormat;
           } else {
             console.error(
-              `Error: Invalid format "${options.format}". Must be one of: ${
-                validFormats.join(", ")
-              }`,
+              `Error: Invalid format "${options.format}". Must be one of: ${validFormats.join(", ")}`,
             );
             process.exit(1);
           }
diff --git a/spx/26-scoped-cli.capability/scoped-cli.capability.md b/spx/26-scoped-cli.capability/scoped-cli.capability.md
@@ -219,7 +219,7 @@ proven by zero breaking changes during migration and successful addition of clau
 #### Claude Domain (Infrastructure Only - Capability 32)
 
 - **`spx claude init [--source <url>]`**
-  - Install spx-claude marketplace (implemented in capability-32)
+  - Install outcomeeng marketplace (implemented in capability-32)
 
 - **`spx claude update`**
   - Update marketplace from source (implemented in capability-32)
diff --git a/src/index.ts b/src/index.ts
@@ -2,6 +2,6 @@
  * Main library entry point for spx
  */
 
-export * from "./types.js";
 export * from "./scanner/patterns.js";
 export * from "./scanner/validation.js";
+export * from "./types.js";
diff --git a/src/reporter/markdown.ts b/src/reporter/markdown.ts
@@ -48,7 +48,7 @@ export function formatMarkdown(tree: WorkItemTree): string {
 function formatNode(
   node: TreeNode,
   level: number,
-  sections: string[]
+  sections: string[],
 ): void {
   const displayNumber = getDisplayNumber(node);
   const name = `${node.kind}-${displayNumber}_${node.slug}`;
diff --git a/src/reporter/table.ts b/src/reporter/table.ts
diff --git a/src/reporter/text.ts b/src/reporter/text.ts
diff --git a/src/scanner/patterns.ts b/src/scanner/patterns.ts
diff --git a/src/scanner/validation.ts b/src/scanner/validation.ts
diff --git a/src/scanner/walk.ts b/src/scanner/walk.ts
diff --git a/src/status/state.ts b/src/status/state.ts
diff --git a/src/tree/validate.ts b/src/tree/validate.ts
diff --git a/tests/unit/reporter/table.test.ts b/tests/unit/reporter/table.test.ts
diff --git a/tests/unit/validation/argument-builders.test.ts b/tests/unit/validation/argument-builders.test.ts
