@danieljvdm/dev-kit 0.6.0 → 0.7.0

package/skills/effect-ts/references/guide-testing.md

@@ -46,8 +46,8 @@ repository explicitly owns that workflow.
 Preferred imports for Effect tests:
 
 ```ts
-import { assert, describe, it, layer } from "@effect/vitest"
-import { Effect } from "effect"
+import { assert, describe, it, layer } from "@effect/vitest";
+import { Effect } from "effect";
 ```
 
 `@effect/vitest` re-exports Vitest, so it is the normal entrypoint for test APIs in an Effect codebase.
@@ -74,15 +74,15 @@ This comes directly from the internal implementation.
 Example:
 
 ```ts
-import { assert, it } from "@effect/vitest"
-import { Effect } from "effect"
+import { assert, it } from "@effect/vitest";
+import { Effect } from "effect";
 
 it.effect("loads a user", () =>
-  Effect.gen(function*() {
-    yield* Effect.void
-    assert.isTrue(true)
-  })
-)
+  Effect.gen(function* () {
+    yield* Effect.void;
+    assert.isTrue(true);
+  }),
+);
 ```
 
 Use `it.effect` when:
@@ -100,10 +100,10 @@ Example:
 
 ```ts
 it.live("uses live services", () =>
-  Effect.gen(function*() {
-    yield* Effect.void
-  })
-)
+  Effect.gen(function* () {
+    yield* Effect.void;
+  }),
+);
 ```
 
 Use `it.live` when:
@@ -123,11 +123,11 @@ Preferred pattern:
 
 ```ts
 it.effect("does something", () =>
-  Effect.gen(function*() {
-    const value = yield* someEffect
-    assert.strictEqual(value, 1)
-  })
-)
+  Effect.gen(function* () {
+    const value = yield* someEffect;
+    assert.strictEqual(value, 1);
+  }),
+);
 ```
 
 Prefer `Effect.gen` inside `it.effect` and `it.live` for readability.
@@ -147,9 +147,9 @@ The canonical source also uses `expect` in some tests, but for skill guidance pr
 Examples:
 
 ```ts
-assert.isTrue(value === 1)
-assert.strictEqual(a + b, b + a)
-assert.include(text, substring)
+assert.isTrue(value === 1);
+assert.strictEqual(a + b, b + a);
+assert.include(text, substring);
 ```
 
 ## Test Context
@@ -160,12 +160,12 @@ Example:
 
 ```ts
 it.effect("uses context", (ctx) =>
-  Effect.gen(function*() {
+  Effect.gen(function* () {
     ctx.onTestFailed(() => {
       // cleanup or diagnostics
-    })
-  })
-)
+    });
+  }),
+);
 ```
 
 Use this when you need:
@@ -201,12 +201,12 @@ Use `it.prop` for non-Effect property tests.
 Example:
 
 ```ts
-import { it } from "@effect/vitest"
-import { FastCheck } from "effect/testing"
+import { it } from "@effect/vitest";
+import { FastCheck } from "effect/testing";
 
-const realNumber = FastCheck.float({ noNaN: true, noDefaultInfinity: true })
+const realNumber = FastCheck.float({ noNaN: true, noDefaultInfinity: true });
 
-it.prop("symmetry", [realNumber, FastCheck.integer()], ([a, b]) => a + b === b + a)
+it.prop("symmetry", [realNumber, FastCheck.integer()], ([a, b]) => a + b === b + a);
 ```
 
 Important limitation from the internal implementation:
@@ -225,17 +225,17 @@ This is the more powerful property-testing mode for Effect code.
 Example:
 
 ```ts
-import { assert, it } from "@effect/vitest"
-import { Effect } from "effect"
-import { FastCheck } from "effect/testing"
+import { assert, it } from "@effect/vitest";
+import { Effect } from "effect";
+import { FastCheck } from "effect/testing";
 
-const realNumber = FastCheck.float({ noNaN: true, noDefaultInfinity: true })
+const realNumber = FastCheck.float({ noNaN: true, noDefaultInfinity: true });
 
 it.effect.prop("symmetry", [realNumber, FastCheck.integer()], ([a, b]) =>
-  Effect.gen(function*() {
-    assert.strictEqual(a + b, b + a)
-  })
-)
+  Effect.gen(function* () {
+    assert.strictEqual(a + b, b + a);
+  }),
+);
 ```
 
 Unlike top-level `it.prop`, `it.effect.prop` does support `Schema` inputs by converting them with `Schema.toArbitrary`.
@@ -260,23 +260,23 @@ This is one of the most important `@effect/vitest` features.
 Example:
 
 ```ts
-import { describe, it, layer } from "@effect/vitest"
-import { Context, Effect, Layer } from "effect"
+import { describe, it, layer } from "@effect/vitest";
+import { Context, Effect, Layer } from "effect";
 
 class Foo extends Context.Service<Foo, string>()("Foo") {
-  static readonly layer = Layer.succeed(Foo)("foo")
+  static readonly layer = Layer.succeed(Foo)("foo");
 }
 
 describe("foo", () => {
   layer(Foo.layer)((it) => {
     it.effect("gets foo", () =>
-      Effect.gen(function*() {
-        const foo = yield* Foo
-        return foo
-      })
-    )
-  })
-})
+      Effect.gen(function* () {
+        const foo = yield* Foo;
+        return foo;
+      }),
+    );
+  });
+});
 ```
 
 What it does internally:
@@ -314,14 +314,12 @@ If multiple tests use the same layer, do not write tests like this:
 
 ```ts
 it.effect("creates and lists todos", () =>
-  Effect.gen(function*() {
-    const service = yield* TodoService
-    yield* service.create("write tests")
-    yield* service.create("ship feature")
-  }).pipe(
-    Effect.provide(TodoService.inMemoryLayer)
-  )
-)
+  Effect.gen(function* () {
+    const service = yield* TodoService;
+    yield* service.create("write tests");
+    yield* service.create("ship feature");
+  }).pipe(Effect.provide(TodoService.inMemoryLayer)),
+);
 ```
 
 Why this is the wrong pattern:
@@ -338,23 +336,23 @@ Prefer:
 describe("TodoService", () => {
   layer(TodoService.inMemoryLayer)((it) => {
     it.effect("creates and lists todos", () =>
-      Effect.gen(function*() {
-        const service = yield* TodoService
-        yield* service.create("write tests")
-        yield* service.create("ship feature")
-      })
-    )
+      Effect.gen(function* () {
+        const service = yield* TodoService;
+        yield* service.create("write tests");
+        yield* service.create("ship feature");
+      }),
+    );
 
     it.effect("updates completion and deletes todos", () =>
-      Effect.gen(function*() {
-        const service = yield* TodoService
-        const todo = yield* service.create("close issue")
-        yield* service.setCompleted(todo.id, true)
-        yield* service.remove(todo.id)
-      })
-    )
-  })
-})
+      Effect.gen(function* () {
+        const service = yield* TodoService;
+        const todo = yield* service.create("close issue");
+        yield* service.setCompleted(todo.id, true);
+        yield* service.remove(todo.id);
+      }),
+    );
+  });
+});
 ```
 
 Rule:
@@ -381,14 +379,14 @@ Example:
 layer(Foo.layer)((it) => {
   it.layer(Bar.layer)("nested", (it) => {
     it.effect("gets both", () =>
-      Effect.gen(function*() {
-        const foo = yield* Foo
-        const bar = yield* Bar
-        return [foo, bar]
-      })
-    )
-  })
-})
+      Effect.gen(function* () {
+        const foo = yield* Foo;
+        const bar = yield* Bar;
+        return [foo, bar];
+      }),
+    );
+  });
+});
 ```
 
 Important behavior from the implementation:
@@ -476,11 +474,11 @@ It is not needed for ordinary test structure.
 
 ```ts
 it.effect("does work", () =>
-  Effect.gen(function*() {
-    const value = yield* Effect.succeed(1)
-    assert.strictEqual(value, 1)
-  })
-)
+  Effect.gen(function* () {
+    const value = yield* Effect.succeed(1);
+    assert.strictEqual(value, 1);
+  }),
+);
 ```
 
 ### Pattern: shared layer for a test group
@@ -488,33 +486,33 @@ it.effect("does work", () =>
 ```ts
 layer(AppLayer)("app", (it) => {
   it.effect("uses app services", () =>
-    Effect.gen(function*() {
-      yield* Effect.void
-    })
-  )
-})
+    Effect.gen(function* () {
+      yield* Effect.void;
+    }),
+  );
+});
 ```
 
 ### Pattern: property test with Effect
 
 ```ts
 it.effect.prop("law", [FastCheck.integer()], ([n]) =>
-  Effect.gen(function*() {
-    assert.strictEqual(n + 0, n)
-  })
-)
+  Effect.gen(function* () {
+    assert.strictEqual(n + 0, n);
+  }),
+);
 ```
 
 ### Pattern: use `TestClock`
 
 ```ts
 it.effect("uses TestClock", () =>
-  Effect.gen(function*() {
-    const fiber = yield* Effect.forkChild(Effect.sleep("1 second"))
-    yield* TestClock.adjust("1 second")
-    yield* Fiber.join(fiber)
-  })
-)
+  Effect.gen(function* () {
+    const fiber = yield* Effect.forkChild(Effect.sleep("1 second"));
+    yield* TestClock.adjust("1 second");
+    yield* Fiber.join(fiber);
+  }),
+);
 ```
 
 ## Anti-Patterns

package/skills/effect-ts/references/guide-type-safety-and-boundaries.md

@@ -49,13 +49,13 @@ domain policy.
 
 Choose the Schema adapter that matches the caller's control flow:
 
-| Intent | Adapter |
-| --- | --- |
-| Boolean type guard | `Schema.is(Model)` |
-| Optional tolerant decode | `Schema.decodeUnknownOption(Model)` |
-| Typed Effect failure | `Schema.decodeUnknownEffect(Model)` |
-| JSON string decode | `Schema.fromJsonString(Model)` |
-| Any JSON-compatible value | `Schema.Json` |
+| Intent                    | Adapter                             |
+| ------------------------- | ----------------------------------- |
+| Boolean type guard        | `Schema.is(Model)`                  |
+| Optional tolerant decode  | `Schema.decodeUnknownOption(Model)` |
+| Typed Effect failure      | `Schema.decodeUnknownEffect(Model)` |
+| JSON string decode        | `Schema.fromJsonString(Model)`      |
+| Any JSON-compatible value | `Schema.Json`                       |
 
 Define small schemas for provider responses, SDK payloads, persisted data, and
 other structured external values. Decode once in the adapter and return the

package/skills/effect-ts/references/version-and-source.md

@@ -84,4 +84,3 @@ Before completing version-sensitive work:
 - every installed Effect v4 package resolves to the same beta
 - no source path is assumed solely because it appears in this skill
 - current-main guidance is not presented as installed-version behavior
-

package/src/bin/dev-kit.ts

@@ -1,23 +1,17 @@
 import { NodeRuntime, NodeServices } from "@effect/platform-node";
-import { Argument, CliError, Command as CliCommand, Flag } from "effect/unstable/cli";
 import { Effect, Result } from "effect";
+import { Argument, CliError, Command as CliCommand, Flag } from "effect/unstable/cli";
 
-import { printError } from "../cli-ui.ts";
-import { patchEffectTsgo } from "../effect-tsgo.ts";
-import { syncEffectSource } from "../effect-source.ts";
-import { patchProjectGitignore } from "../gitignore.ts";
-import {
-  DEFAULT_MANIFEST,
-  runProjectSkillPlan,
-} from "../sync.ts";
-import { DEV_KIT_VERSION } from "../tool-metadata.ts";
-import { refreshSkillCatalog } from "../vendor.ts";
 import {
   addCatalogSource,
   listCatalogSources,
   removeCatalogEntry,
   showCatalogSource,
 } from "../catalog-manager.ts";
+import { printError } from "../cli-ui.ts";
+import { syncEffectSource } from "../effect-source.ts";
+import { patchEffectTsgo } from "../effect-tsgo.ts";
+import { patchProjectGitignore } from "../gitignore.ts";
 import {
   addSkills,
   chooseSkillsToAdd,
@@ -28,6 +22,9 @@ import {
   showDashboard,
   showSkill,
 } from "../skill-manager.ts";
+import { DEFAULT_MANIFEST, runProjectSkillPlan } from "../sync.ts";
+import { DEV_KIT_VERSION } from "../tool-metadata.ts";
+import { refreshSkillCatalog } from "../vendor.ts";
 
 const projectFlags = {
   manifest: Flag.string("manifest").pipe(
@@ -80,8 +77,7 @@ const listCommand = CliCommand.make(
     all: Flag.boolean("all").pipe(Flag.withDescription("Include unselected skills.")),
     ...projectFlags,
   },
-  ({ all, manifest, projectDir }) =>
-    listSkills({ all, manifestPath: manifest, projectDir }),
+  ({ all, manifest, projectDir }) => listSkills({ all, manifestPath: manifest, projectDir }),
 ).pipe(CliCommand.withDescription("List selected skills; use --all to browse the catalog."));
 
 const searchCommand = CliCommand.make(
@@ -113,7 +109,9 @@ const planCommand = CliCommand.make(
       manifestPath: manifest,
       projectDir,
     }),
-).pipe(CliCommand.withDescription("Plan ownership-safe project skill changes without writing files."));
+).pipe(
+  CliCommand.withDescription("Plan ownership-safe project skill changes without writing files."),
+);
 
 const applyCommand = CliCommand.make(
   "apply",
@@ -130,7 +128,9 @@ const applyCommand = CliCommand.make(
       manifestPath: manifest,
       projectDir,
     }),
-).pipe(CliCommand.withDescription("Apply ownership-safe project skill changes and update the lock."));
+).pipe(
+  CliCommand.withDescription("Apply ownership-safe project skill changes and update the lock."),
+);
 
 const syncCommand = CliCommand.make(
   "sync",
@@ -166,9 +166,7 @@ const gitignoreCommand = CliCommand.make(
   },
   ({ dryRun, projectDir }) => patchProjectGitignore({ dryRun, projectDir }),
 ).pipe(
-  CliCommand.withDescription(
-    "Idempotently add .repos/ and .dev-kit/ to the project .gitignore.",
-  ),
+  CliCommand.withDescription("Idempotently add .repos/ and .dev-kit/ to the project .gitignore."),
 );
 
 const tsgoPatchCommand = CliCommand.make(
@@ -177,9 +175,7 @@ const tsgoPatchCommand = CliCommand.make(
     dryRun: Flag.boolean("dry-run"),
     force: Flag.boolean("force"),
     projectDir: Flag.string("project-dir").pipe(Flag.withDefault(".")),
-    typescriptPackage: Flag.string("typescript-package").pipe(
-      Flag.withDefault("typescript"),
-    ),
+    typescriptPackage: Flag.string("typescript-package").pipe(Flag.withDefault("typescript")),
   },
   ({ dryRun, force, projectDir, typescriptPackage }) =>
     patchEffectTsgo({ dryRun, force, projectDir, typescriptPackage }),
@@ -247,9 +243,18 @@ const catalogAddCommand = CliCommand.make(
       Flag.withDescription("Approve every skill discovered in this snapshot."),
     ),
     dryRun: Flag.boolean("dry-run").pipe(Flag.withDescription("Inspect without writing.")),
-    id: Flag.string("id").pipe(Flag.withDefault(""), Flag.withDescription("Override the inferred source id.")),
-    license: Flag.string("license").pipe(Flag.withDefault(""), Flag.withDescription("Repository-relative license path.")),
-    ref: Flag.string("ref").pipe(Flag.withDefault(""), Flag.withDescription("Branch, tag, or commit to track.")),
+    id: Flag.string("id").pipe(
+      Flag.withDefault(""),
+      Flag.withDescription("Override the inferred source id."),
+    ),
+    license: Flag.string("license").pipe(
+      Flag.withDefault(""),
+      Flag.withDescription("Repository-relative license path."),
+    ),
+    ref: Flag.string("ref").pipe(
+      Flag.withDefault(""),
+      Flag.withDescription("Branch, tag, or commit to track."),
+    ),
     skill: Flag.string("skill").pipe(
       Flag.atLeast(0),
       Flag.withDescription("Approve one skill; repeat for several."),
@@ -264,7 +269,20 @@ const catalogAddCommand = CliCommand.make(
     ),
     ...catalogFilesFlags,
   },
-  ({ repository, all, dryRun, id, license, ref, skill, skillsPath, stripFrontmatter, lockfile, repoDir, sources }) =>
+  ({
+    repository,
+    all,
+    dryRun,
+    id,
+    license,
+    ref,
+    skill,
+    skillsPath,
+    stripFrontmatter,
+    lockfile,
+    repoDir,
+    sources,
+  }) =>
     addCatalogSource({
       repository,
       all,
@@ -304,7 +322,13 @@ const catalogRemoveCommand = CliCommand.make(
     ...catalogFilesFlags,
   },
   ({ entry, dryRun, yes, lockfile, repoDir, sources }) =>
-    removeCatalogEntry(entry, { dryRun, yes, lockfilePath: lockfile, repoDir, sourcesPath: sources }),
+    removeCatalogEntry(entry, {
+      dryRun,
+      yes,
+      lockfilePath: lockfile,
+      repoDir,
+      sourcesPath: sources,
+    }),
 ).pipe(CliCommand.withDescription("Revoke an approved source or individual skill."));
 
 const catalogRefreshCommand = CliCommand.make(
@@ -316,7 +340,9 @@ const catalogRefreshCommand = CliCommand.make(
   },
   ({ dryRun, locked, lockfile, repoDir, sources }) =>
     refreshSkillCatalog({ dryRun, locked, lockfilePath: lockfile, repoDir, sourcesPath: sources }),
-).pipe(CliCommand.withDescription("Approve the current upstream refs as an exact catalog snapshot."));
+).pipe(
+  CliCommand.withDescription("Approve the current upstream refs as an exact catalog snapshot."),
+);
 
 const catalogVerifyCommand = CliCommand.make(
   "verify",
@@ -349,7 +375,14 @@ const command = CliCommand.make("dev-kit", projectFlags, ({ manifest, projectDir
     { group: "Project", commands: [statusCommand, syncCommand] },
     {
       group: "Advanced",
-      commands: [planCommand, applyCommand, gitignoreCommand, effectCommand, tsgoCommand, catalogCommand],
+      commands: [
+        planCommand,
+        applyCommand,
+        gitignoreCommand,
+        effectCommand,
+        tsgoCommand,
+        catalogCommand,
+      ],
     },
   ] as const),
 );