Merge pull request #535 from dahlia/deps

Implement automatic dependency-aware builds using pnpm workspace filters

Author: dahlia

AGENTS.md

@@ -45,7 +45,8 @@ Development environment
  -  Primary environment: [Deno]
  -  Additional test environments: [Node.js] and [Bun]
  -  Recommended editor: [Visual Studio Code] with [Deno extension]
- -  **CRITICAL**: Run `mise run codegen` before working with the codebase
+ -  **CRITICAL**: Run `mise run install` (or `pnpm install`) after checkout.
+    This automatically runs code generation and builds all packages.
  -  Lockfiles: Both *deno.lock* and *pnpm-lock.yaml* are committed.
     Update them when changing dependencies.
 
@@ -119,6 +120,9 @@ Development workflow
 
  -  **Code Generation**: Run `mise run codegen` whenever vocabulary YAML files
     or code generation scripts change.
+ -  **Building Packages**: After installation, all packages are automatically
+    built. To rebuild a specific package and its dependencies, run `pnpm build`
+    in that package's directory.
  -  **Checking Code**: Run `mise run check` before committing.
  -  **Running Tests**: Use `mise run test:deno` for Deno tests or
     `mise run test` for all environments.

CONTRIBUTING.md

@@ -391,15 +391,16 @@ see the [*Set Up Your Environment* section][1] in the Deno manual.
 
 > [!CAUTION]
 >
-> Fedify heavily depends on code generation, so you need to run
-> `mise run codegen` before coding or testing.
+> Fedify heavily depends on code generation and all packages must be built
+> before coding or testing. Running `mise run install` (or `pnpm install`)
+> automatically handles code generation and builds all packages.
 
 Assuming you have Deno and Visual Studio Code installed, you can open
 the repository in Visual Studio Code and get ready to hack on Fedify by running
 the following commands at the *root* of the repository:
 
 ~~~~ bash
-mise run codegen
+mise run install  # This runs codegen and builds all packages
 code .
 ~~~~
 
@@ -412,9 +413,10 @@ code .
 > mise run hooks:install
 > ~~~~
 
-Note that the `mise run codegen` command is required to run only once at
-the very first time, or when you update the code generation scripts.  Otherwise,
-you can skip the command and just run:
+Note that the `mise run install` command is required to run only once at
+the very first time after checkout. When you update dependencies or code
+generation scripts, run `mise run install` again. Otherwise, you can skip
+the command and just run:
 
 ~~~~ bash
 code .

mise.toml

@@ -24,7 +24,7 @@ run = "deno task codegen"
 description = "Prepare the development environment (codegen, install, build fixtures)"
 env = { CI = "true" }  # Prevent pnpm from prompting for confirmation
 depends = ["install"]
-run = "pnpm --filter '@fedify/*' --recursive --parallel build"
+run = "pnpm -r run build:self"
 
 [tasks.prepare-each]
 description = "Prepare specific package(s)"

package.json

@@ -1,4 +1,7 @@
 {
+  "scripts": {
+    "prepare": "pnpm -r run build:self"
+  },
   "pnpm": {
     "onlyBuiltDependencies": [
       "esbuild"

packages/amqp/package.json

@@ -68,11 +68,12 @@
     "typescript": "catalog:"
   },
   "scripts": {
-    "build": "tsdown",
-    "prepack": "tsdown",
-    "prepublish": "tsdown",
-    "test": "tsdown && node --experimental-transform-types --test",
-    "test:bun": "tsdown && bun test --timeout 15000",
+    "build:self": "tsdown",
+    "build": "pnpm --filter @fedify/amqp... run build:self",
+    "prepack": "pnpm build",
+    "prepublish": "pnpm build",
+    "test": "pnpm build && node --experimental-transform-types --test",
+    "test:bun": "pnpm build && bun test --timeout 15000",
     "test:deno": "deno task test",
     "test-all": "tsdown && node --experimental-transform-types --test && bun test --timeout 15000 && deno task test"
   }

packages/cfworkers/package.json

@@ -62,9 +62,10 @@
     "wrangler": "^4.21.1"
   },
   "scripts": {
-    "build": "tsdown",
-    "prepack": "tsdown",
-    "prepublish": "tsdown",
+    "build:self": "tsdown",
+    "build": "pnpm --filter @fedify/cfworkers... run build:self",
+    "prepack": "pnpm build",
+    "prepublish": "pnpm build",
     "test": "vitest run"
   }
 }

packages/cli/package.json

@@ -86,16 +86,16 @@
     "typescript": "catalog:"
   },
   "scripts": {
-    "codegen": "deno task -f @fedify/fedify codegen",
-    "build": "pnpm run codegen && pnpm run --filter fedify build && pnpm run --filter relay build && pnpm run --filter sqlite build && tsdown",
-    "prepack": "pnpm run build",
-    "prepublish": "pnpm run build",
+    "build:self": "tsdown",
+    "build": "pnpm --filter @fedify/cli... run build:self",
+    "prepack": "pnpm build",
+    "prepublish": "pnpm build",
     "test": "pnpm build && node --test --experimental-transform-types 'src/**/*.test.ts' '!src/init/test/**'",
     "test-init": "deno task test-init",
-    "test:bun": "tsdown && bun test",
-    "run": "tsdown && node dist/mod.js",
+    "test:bun": "pnpm build && bun test",
+    "run": "pnpm build && node dist/mod.js",
     "runi": "tsdown && node dist/mod.js",
-    "run:bun": "tsdown && bun dist/mod.js",
+    "run:bun": "pnpm build && bun dist/mod.js",
     "runi:bun": "tsdown && bun dist/mod.js"
   }
 }

packages/elysia/package.json

@@ -46,9 +46,10 @@
     "package.json"
   ],
   "scripts": {
-    "build": "tsdown",
-    "prepack": "tsdown",
-    "prepublish": "tsdown"
+    "build:self": "tsdown",
+    "build": "pnpm --filter @fedify/elysia... run build:self",
+    "prepack": "pnpm build",
+    "prepublish": "pnpm build"
   },
   "peerDependencies": {
     "elysia": "^1.3.6",

packages/express/package.json

@@ -58,8 +58,9 @@
     "typescript": "catalog:"
   },
   "scripts": {
-    "build": "tsdown",
-    "prepack": "tsdown",
-    "prepublish": "tsdown"
+    "build:self": "tsdown",
+    "build": "pnpm --filter @fedify/express... run build:self",
+    "prepack": "pnpm build",
+    "prepublish": "pnpm build"
   }
 }

packages/fastify/package.json

@@ -56,9 +56,10 @@
     "typescript": "catalog:"
   },
   "scripts": {
-    "build": "tsdown",
-    "prepack": "tsdown",
-    "prepublish": "tsdown",
+    "build:self": "tsdown",
+    "build": "pnpm --filter @fedify/fastify... run build:self",
+    "prepack": "pnpm build",
+    "prepublish": "pnpm build",
     "test": "node --experimental-transform-types --test"
   }
 }