第 11 章: イベントとメール
ここまでの処理はすべてリクエストの中で完結していました。検証し、行を 1 つ書き、リダイレクトする。この章の主題は、そうであってはならない仕事です。Bob が Ada の投稿にコメントすると Ada にメールが届きますが、Bob のブラウザは、ページを目にする前にメールサーバーの応答を待たされてはいけません。
第 11 章: イベントとメール
ここまでの処理はすべてリクエストの中で完結していました。検証し、行を 1 つ書き、リダイレクトする。この章の主題は、そうであってはならない仕事です。Bob が Ada の投稿にコメントすると Ada にメールが届きますが、Bob のブラウザは、ページを目にする前にメールサーバーの応答を待たされてはいけません。
その一文に名前が 4 つ付きます。そしてこの章の大半は、なぜ 4 つなのかという話です。
| 部品 | 答えるもの |
|---|---|
| イベント | 何かが起きた。コントローラーはそれを告知し、あとは気にしません。 |
| listener | 気にする誰か。告知に対して何をするかを決めます。 |
| ジョブ | リクエストより長生きする仕事。キューに載るペイロードで、拾った者が実行します。 |
| メール | メッセージそのもの。件名、本文、宛先。 |
リクエストは最初の箱で終わります。その先は、読者が待たされない仕事です。
flowchart LR
Controller["CommentController<br/>emit(new CommentPosted)"]
Listener["SendCommentMailListener<br/>ジョブを dispatch する"]
Job["SendCommentMailJob<br/>payload: { commentId }"]
Mail["NewCommentMail<br/>投稿の著者へ"]
Controller --> Listener --> Job --> Mail
この章で学ぶこと:
- 4 つのそれぞれがどこで登録されるのか、そして誰も代わりに検査してくれない唯一の登録
- ジョブのペイロードがレコードではなく id である理由
QUEUE_CONNECTION=syncが実際に何をしているのか、そして sync でなくなったときに何が変わるのか- 3 つの異なる継ぎ目に対する 3 つの fake と、そのうちひとつがコンテナのバインディングになり得ない理由
開発サーバーが動いていなければ起動します。
bun run dev
1. 3 つのレイヤー、3 つのコマンド
bunx guren add events
bunx guren add queue
bunx guren add mail
それぞれが、その種類のサンプルとプロバイダーを 1 つずつ書き、フレームワークのサービスプロバイダーとあなたのものの両方を src/app.ts に登録しました。開いてみてください。providers の配列は 1 行に書き直され、末尾に 6 つの要素が増えています。コマンドごとに、フレームワークのプロバイダーとアプリのプロバイダーが 1 つずつです。この 1 行化はパッチを当てたコマンドの仕業であってあなたのものではなく、どの add コマンドも同じ形を残していきます。
この 3 つのプロバイダーは読む価値があります。うち 2 つは、これからあなたが編集するファイルだからです。
app/Providers/EventProvider.tsはイベントクラスを listener オブジェクトに結び付けます。その結び付きは規約ではなくコードの 1 行です。app/Listeners/を走査して仕事を探すものは何もありません。app/Providers/QueueProvider.tsはキューマネージャーを構築し、ジョブクラスごとにregisterJob()を呼びます。ドライバーの行に注目してください。QUEUE_CONNECTION=syncは dispatch されたジョブをインラインで、dispatch したプロセスの中で実行します。memoryはワーカーが処理するキューに載せます。あなたの.envはすでにsyncと言っています。app/Providers/MailProvider.tsはメールマネージャーを構築します。こちらもすでに.envにあるMAIL_MAILER=logは、送信する代わりに送信予定のメールをサーバーの出力に印字します。申し込むものは何も無く、うっかり配送してしまうものも何もありません。
サンプル(OrderPlaced、SendOrderReceiptListener、ProcessWelcomeSequenceJob、WelcomeEmailMail)は、それぞれのファイルの形を目にできるように置かれています。第 3 節でこの 4 つすべてを置き換えます。
2. メールを仕様化する
テストは 3 つ、そのそれぞれが意図的に違う形の fake を使っています。アサーションより先にセットアップを読んでください。
tests/CommentMail.test.tsimport { beforeAll, beforeEach, describe, it } from 'bun:test'
import { MailManager, getQueueDriver, setQueueDriver } from '@guren/core'
import { TestApp, fakeMail, fakeQueue } from '@guren/testing'
import app from '../src/app.js'
import { resetDatabase } from '../config/database.js'
import { Post, type PostRecord } from '../app/Models/Post.js'
import { User, type UserRecord } from '../app/Models/User.js'
import { SendCommentMailJob, type SendCommentMailPayload } from '../app/Jobs/SendCommentMailJob.js'
const mail = fakeMail()
describe('comment mail', () => {
let http: TestApp
let ada: UserRecord
let bob: UserRecord
let post: PostRecord
let asBob: TestApp
beforeAll(async () => {
http = await TestApp.fromApp(app)
// Mail.send() asks the manager for a transport, so the fake goes inside a
// real manager rather than in place of one.
const manager = new MailManager({ default: 'fake', from: { email: 'blog@example.com', name: 'Blog' } })
manager.registerTransport('fake', () => mail.getTransport())
app.container.fake('mail', manager)
})
beforeEach(async () => {
await resetDatabase()
mail.clear()
ada = await User.create({ name: 'Ada', email: 'ada@example.com', password: 'correct horse battery' })
bob = await User.create({ name: 'Bob', email: 'bob@example.com', password: 'correct horse battery' })
post = await Post.forceCreate({ title: 'Relativity', body: 'A body', authorId: ada.id })
asBob = await http.actingAs(bob).withCsrf()
})
it('mails the post author when someone else comments', async () => {
await asBob.post(`/posts/${post.id}/comments`, { body: 'Nice post' }).assertRedirect(`/posts/${post.id}`)
mail.assertSentTo('ada@example.com')
mail.assertSentWithSubject('New comment on Relativity')
mail.assertSentWithBodyContaining('Bob')
})
it('does not mail you about your own comment', async () => {
const asAda = await http.actingAs(ada).withCsrf()
await asAda.post(`/posts/${post.id}/comments`, { body: 'A note to myself' }).assertRedirect(`/posts/${post.id}`)
mail.assertNothingSent()
})
it('hands the mail to the queue instead of sending it in the request', async () => {
const queue = fakeQueue()
// Job.dispatch() reads a module-level driver, not the container, so this
// seam is a setter and not a container fake. Put the real one back after.
const real = getQueueDriver()
setQueueDriver(queue.getDriver())
try {
await asBob.post(`/posts/${post.id}/comments`, { body: 'Nice post' }).assertRedirect(`/posts/${post.id}`)
queue.assertPushed<SendCommentMailPayload>(SendCommentMailJob, (payload) => payload.commentId > 0)
mail.assertNothingSent()
} finally {
if (real) setQueueDriver(real)
}
})
})
assertPushed にはペイロードの型を明示的に渡しています。Job.dispatch はジェネリックな static なので、ジョブクラスだけでは自分のペイロードが何かを TypeScript に伝えられず、推論された unknown は述語をコンパイルできなくします。
3 つ目のテストは、機能ではなく設計を記述しているものです。fake のキュードライバーを差し込むとジョブは記録されるだけで実行されないので、メールは 1 通も出ません。もしこのテストが通り、かつメールが送られているなら、コントローラーが自分で仕事をしているということです。
bun test
import で赤です。SendCommentMailJob がまだありません。
3. 4 つの部品を手で書く
イベントは、何が起きたかを特定できる最小のものを運びます。
app/Events/CommentPosted.tsimport { Event } from '@guren/core'
export class CommentPosted extends Event {
static override eventName = 'CommentPosted'
constructor(public readonly commentId: number) {
super()
}
}
listener は、それが起きたことの意味を決めます。この listener は自分では何もせず、仕事をキューに渡して戻ります。
app/Listeners/SendCommentMailListener.tsimport { Listener } from '@guren/core'
import { CommentPosted } from '../Events/CommentPosted.js'
import { SendCommentMailJob } from '../Jobs/SendCommentMailJob.js'
export class SendCommentMailListener extends Listener<CommentPosted> {
static override event = CommentPosted
async handle(event: CommentPosted): Promise<void> {
await SendCommentMailJob.dispatch({ commentId: event.commentId })
}
}
ジョブは、別のプロセスで、数分後に実行されるかもしれない部品です。
app/Jobs/SendCommentMailJob.tsimport { Job } from '@guren/core'
import { Comment } from '../Models/Comment.js'
import { User } from '../Models/User.js'
import { NewCommentMail } from '../Mail/NewCommentMail.js'
/** A queued payload is JSON on its way to another process: ids, never records. */
export interface SendCommentMailPayload {
commentId: number
}
export class SendCommentMailJob extends Job<SendCommentMailPayload> {
static override queue = 'default'
static override maxAttempts = 3
async handle(payload: SendCommentMailPayload): Promise<void> {
const comment = await Comment.findWith(payload.commentId, ['post', 'author'])
if (!comment?.post || !comment.author) return
const postAuthor = await User.find(comment.post.authorId)
if (!postAuthor || postAuthor.id === comment.authorId) return
await new NewCommentMail(this.make('mail'), {
postTitle: comment.post.title,
commenter: comment.author.name,
body: comment.body,
url: `/posts/${comment.post.id}`,
})
.to(postAuthor.email)
.send()
}
}
このファイルの中の 2 つの決定が、この章の本当の中身です。ペイロードはコメントではなく commentId です。これが走るころには行が変わっているかもしれませんし、そもそもレコードはキューにシリアライズできません。そして「自分のコメントについて自分にメールを送らない」は、コントローラーではなくここ、送信の隣にあります。コントローラーは何が起きたかを告知するのであって、誰がそれについてのメールを受け取るに値するかを決めるのではありません。
メールはメッセージであって、それ以外の何物でもありません。
app/Mail/NewCommentMail.tsimport { Mail, type MailManager } from '@guren/core'
export interface NewCommentMailData {
postTitle: string
commenter: string
body: string
url: string
}
export class NewCommentMail extends Mail {
constructor(
manager: MailManager,
private readonly data: NewCommentMailData,
) {
super(manager)
}
build(): this {
return this.subject(`New comment on ${this.data.postTitle}`).text(
`${this.data.commenter} wrote:\n\n${this.data.body}\n\nRead it: ${this.data.url}`,
)
}
}
build() を自分で呼ぶことはありません。send() が一度だけ呼び、それからメッセージに宛先と件名と本文があることを検査して、トランスポートに渡します。
では 2 つの登録です。イベントプロバイダーは、クラスが購読になる場所です。
app/Providers/EventProvider.tsimport { ServiceProvider, type EventManager } from '@guren/core'
import { CommentPosted } from '../Events/CommentPosted.js'
import { SendCommentMailListener } from '../Listeners/SendCommentMailListener.js'
export default class EventProvider extends ServiceProvider {
register(): void {}
boot(): void {
const events = this.container.make<EventManager>('events')
const listener = new SendCommentMailListener()
events.on(CommentPosted, (event) => listener.handle(event), {
priority: SendCommentMailListener.priority,
})
}
}
この配線はよく読んでください。そうしないとあとで出会うことになるバグの一群を説明しているからです。プロバイダーは priority を渡し、handle を呼びます。読んでいるのはそれだけです。Listener 基底クラスは shouldQueue、queue、そして省略可能な shouldHandle() も宣言していますが、この配線はそのどれも尊重しません。shouldQueue = true を設定してフレームワークがキューに載せてくれると期待した listener は、黙ってインラインで実行されます。クラスが何を宣言していようと、真実はこのファイルの中の行です。
キュープロバイダーは、ジョブクラスが dispatch 可能になる場所です。
app/Providers/QueueProvider.tsimport { ServiceProvider, MemoryDriver, SyncDriver, createQueueManager, registerJob, type QueueManager } from '@guren/core'
import { SendCommentMailJob } from '../Jobs/SendCommentMailJob.js'
export default class QueueProvider extends ServiceProvider {
register(): void {
const queue = createQueueManager({
// QUEUE_CONNECTION=sync executes jobs inline on dispatch (default,
// no worker process needed); 'memory' queues them for a Worker.
default: process.env.QUEUE_CONNECTION === 'memory' ? 'memory' : 'sync',
drivers: {
sync: () => new SyncDriver(),
memory: () => new MemoryDriver(),
},
})
this.container.instance('queue', queue)
}
boot(): void {
// A queued message carries the job's name, so the driver can only run a job
// the registry knows. Nothing in `guren check` looks for a missing one.
registerJob(SendCommentMailJob)
const queue = this.container.make<QueueManager>('queue')
queue.driver()
}
}
最後に、コントローラーが告知します。
app/Http/Controllers/CommentController.tsimport { Controller } from '@guren/core'
import { Post } from '../../Models/Post.js'
import { Comment } from '../../Models/Comment.js'
import type { UserRecord } from '../../Models/User.js'
import { CommentPosted } from '../../Events/CommentPosted.js'
import { CommentPayloadSchema } from '../Validators/CommentValidator.js'
export default class CommentController extends Controller {
async store(): Promise<Response> {
const post = this.model(Post)
await this.authorize('create', Comment)
const author = await this.auth.userOrFail<UserRecord>()
const data = await this.validateBody(CommentPayloadSchema)
const comment = await Comment.forceCreate({ ...data, postId: post.id, authorId: author.id })
await this.make('events').emit(new CommentPosted(comment.id))
return this.redirect(`/posts/${post.id}`)
}
async destroy(): Promise<Response> {
const comment = this.model(Comment)
await this.authorize('delete', [Comment, comment])
await Comment.delete({ id: comment.id })
return this.redirect(`/posts/${comment.postId}`)
}
}
emit は await されており、それが優先度順にすべての listener を await します。sync のもとでは、ジョブも含めた連鎖全体がリダイレクトの返却より前に終わるということです。ここは目を曇らせずに見ておく価値があります。sync は仕事を非同期にするのではなく、コードを非同期の形にするのです。ワーカーへ移行しても、コントローラーは変わりません。
ブループリントが導入した 4 つのサンプルは、もう持ち主がいません。
rm app/Events/OrderPlaced.ts app/Listeners/SendOrderReceiptListener.ts app/Jobs/ProcessWelcomeSequenceJob.ts app/Mail/WelcomeEmailMail.ts
bun test
緑です。
チェックポイント: ブラウザで他人の投稿にコメントし、bun run dev が動いているターミナルを見てください。
[mail] ------------------------------------------------------------
[mail] To: ada@example.com
[mail] From: noreply@example.com
[mail] Subject: New comment on Relativity
[mail] Bob wrote:
[mail]
[mail] Nice post
[mail]
[mail] Read it: /posts/1
[mail] ------------------------------------------------------------
これが log トランスポートです。MAIL_MAILER を本物のトランスポートに向ければ、同じメッセージが建物の外へ出ていきます。
bunx guren gate
git add -A
git commit -m "feat: mail the post author when someone comments"
4. 誰も検査しない登録
整合性チェックを実行し、そこに無いものを読み取ってください。
bunx guren check
これはあなたのルートについて、ページについて、スキーマについて、attachments について意見を持っています。app/Jobs/ については何も持っていません。registerJob() に届かないジョブクラスは完璧に見えます。コンパイルは通り、lint も通り、キューを fake するテストなら通ります。それが失敗するのは、何かが本当にそれを dispatch した最初のときで、少なくとも問題の名前は告げてくれるメッセージを伴います。
SyncDriver: job class "SendCommentMailJob" is not registered. Call registerJob() with the class whose jobName (or class name) is "SendCommentMailJob".
これはまさに第 8 章が扱っていた状況です。フレームワークには見えないプロジェクトの不変条件。ならば、エージェントが読む場所に書き留めましょう。
.claude/rules/background-work.md---
description: Events, listeners and jobs — every job is registered, every listener is wired, and the payload is ids
globs:
- "app/Events/**"
- "app/Listeners/**"
- "app/Jobs/**"
- "app/Mail/**"
- "app/Providers/EventProvider.ts"
- "app/Providers/QueueProvider.ts"
---
# Background work
1. **Every `Job` subclass is registered.** Add `registerJob(TheJob)` to `boot()` in `app/Providers/QueueProvider.ts` in the same change that adds the class. A queued message carries the job's name and the driver resolves it through that registry; an unregistered job throws at dispatch time and `guren check` says nothing about it.
2. **Every listener is wired.** A class in `app/Listeners/` runs only because `app/Providers/EventProvider.ts` calls `events.on(TheEvent, (event) => listener.handle(event), …)`. `shouldQueue`, `queue` and `shouldHandle()` on the class are inert unless that wiring reads them, so do not rely on them: to queue work, dispatch a job from `handle`.
3. **A job payload is JSON: ids, never records.** The job may run in another process, after the row has changed. Load what you need inside `handle`, and return early when the record is gone.
4. **Controllers announce, listeners decide.** A controller emits an event and returns. Rules about who gets mail (skip the actor, skip duplicates) live in the job or the listener, not in the action.
5. **Test the seam, not the plumbing.** Mail is faked by registering a `fakeMail()` transport on a real `MailManager` and binding that with `app.container.fake('mail', manager)`. The queue is faked with `setQueueDriver(fakeQueue().getDriver())`, never through the container, because `Job.dispatch()` reads a module-level driver.
PostToolUse hook は編集のたびに guren check --arch を実行しますが、check はこの 5 つのどれについても黙ったままです。rule こそがチェックなのです。
git add -A
git commit -m "docs: add a background-work rule for the agent"
5. 告知を仕様化する
投稿の公開は、わざわざコメントしてくれた全員に知らせるべきです。同じ 4 つの部品で、形はひとつ難しくなります。重複排除のルールを伴う一斉送信です。
tests/PostPublishedMail.test.tsimport { beforeAll, beforeEach, describe, it } from 'bun:test'
import { MailManager } from '@guren/core'
import { TestApp, fakeMail } from '@guren/testing'
import app from '../src/app.js'
import { resetDatabase } from '../config/database.js'
import { Post, type PostRecord } from '../app/Models/Post.js'
import { Comment } from '../app/Models/Comment.js'
import { User, type UserRecord } from '../app/Models/User.js'
const mail = fakeMail()
describe('publishing a post', () => {
let http: TestApp
let ada: UserRecord
let post: PostRecord
let asAda: TestApp
beforeAll(async () => {
http = await TestApp.fromApp(app)
const manager = new MailManager({ default: 'fake', from: { email: 'blog@example.com', name: 'Blog' } })
manager.registerTransport('fake', () => mail.getTransport())
app.container.fake('mail', manager)
})
beforeEach(async () => {
await resetDatabase()
mail.clear()
ada = await User.create({ name: 'Ada', email: 'ada@example.com', password: 'correct horse battery' })
post = await Post.forceCreate({ title: 'Relativity', body: 'A body', authorId: ada.id })
asAda = await http.actingAs(ada).withCsrf()
})
it('mails everyone who commented, once each', async () => {
const bob = await User.create({ name: 'Bob', email: 'bob@example.com', password: 'correct horse battery' })
const cleo = await User.create({ name: 'Cleo', email: 'cleo@example.com', password: 'correct horse battery' })
await Comment.forceCreate({ body: 'First', postId: post.id, authorId: bob.id })
await Comment.forceCreate({ body: 'Second', postId: post.id, authorId: bob.id })
await Comment.forceCreate({ body: 'Third', postId: post.id, authorId: cleo.id })
await asAda.post(`/posts/${post.id}/publish`).assertRedirect(`/posts/${post.id}`)
mail.assertSentTo('bob@example.com')
mail.assertSentTo('cleo@example.com')
mail.assertSentWithSubject('Relativity is published')
mail.assertSentTimes(2)
})
it('does not mail the author their own post', async () => {
await Comment.forceCreate({ body: 'A note to myself', postId: post.id, authorId: ada.id })
await asAda.post(`/posts/${post.id}/publish`).assertRedirect(`/posts/${post.id}`)
mail.assertNothingSent()
})
})
assertSentTimes(2) が最初のテストの要点のすべてです。Bob は 2 回コメントしましたが、Bob が受け取るメールは 1 通です。
bun test
赤が 2 件。
6. 委ねる
When a post is published, mail everyone who commented on it. Emit a
PostPublishedevent frompublishinPostController, wire a listener inEventProviderthat dispatches aNotifyCommentersJob, and send aPostPublishedMailto each distinct commenter, skipping the post's author.tests/PostPublishedMail.test.tsdescribes it; make it pass.
このプロンプトは registerJob に触れていませんし、その必要もありません。第 4 節であなたが書いた rule は app/Jobs/** と app/Providers/QueueProvider.ts にスコープされているので、エージェントはそのどちらかを書く前にそれを読みます。それがこの実験のすべてです。ほかの何より先に、diff の中の登録の行を確かめてください。
手元にエージェントが無い場合は、 イベントが投稿を運びます。
app/Events/PostPublished.tsimport { Event } from '@guren/core'
export class PostPublished extends Event {
static override eventName = 'PostPublished'
constructor(public readonly postId: number) {
super()
}
}
app/Listeners/NotifyCommentersListener.tsimport { Listener } from '@guren/core'
import { PostPublished } from '../Events/PostPublished.js'
import { NotifyCommentersJob } from '../Jobs/NotifyCommentersJob.js'
export class NotifyCommentersListener extends Listener<PostPublished> {
static override event = PostPublished
async handle(event: PostPublished): Promise<void> {
await NotifyCommentersJob.dispatch({ postId: event.postId })
}
}
app/Mail/PostPublishedMail.tsimport { Mail, type MailManager } from '@guren/core'
export interface PostPublishedMailData {
postTitle: string
url: string
}
export class PostPublishedMail extends Mail {
constructor(
manager: MailManager,
private readonly data: PostPublishedMailData,
) {
super(manager)
}
build(): this {
return this.subject(`${this.data.postTitle} is published`).text(
`A post you commented on is now published.\n\nRead it: ${this.data.url}`,
)
}
}
app/Jobs/NotifyCommentersJob.tsimport { Job } from '@guren/core'
import { Post } from '../Models/Post.js'
import { Comment } from '../Models/Comment.js'
import { User } from '../Models/User.js'
import { PostPublishedMail } from '../Mail/PostPublishedMail.js'
export interface NotifyCommentersPayload {
postId: number
}
export class NotifyCommentersJob extends Job<NotifyCommentersPayload> {
static override queue = 'default'
static override maxAttempts = 3
async handle(payload: NotifyCommentersPayload): Promise<void> {
const post = await Post.find(payload.postId)
if (!post) return
const comments = await Comment.where('postId', post.id).get()
const recipientIds = [...new Set(comments.map((comment) => comment.authorId))].filter(
(id) => id !== post.authorId,
)
if (recipientIds.length === 0) return
const recipients = await User.where({ id: recipientIds }).get()
const manager = this.make('mail')
for (const recipient of recipients) {
await new PostPublishedMail(manager, {
postTitle: post.title,
url: `/posts/${post.id}`,
})
.to(recipient.email)
.send()
}
}
}
app/Providers/EventProvider.tsimport { ServiceProvider, type EventManager } from '@guren/core'
import { CommentPosted } from '../Events/CommentPosted.js'
import { PostPublished } from '../Events/PostPublished.js'
import { SendCommentMailListener } from '../Listeners/SendCommentMailListener.js'
import { NotifyCommentersListener } from '../Listeners/NotifyCommentersListener.js'
export default class EventProvider extends ServiceProvider {
register(): void {}
boot(): void {
const events = this.container.make<EventManager>('events')
const commentListener = new SendCommentMailListener()
const publishListener = new NotifyCommentersListener()
events.on(CommentPosted, (event) => commentListener.handle(event), {
priority: SendCommentMailListener.priority,
})
events.on(PostPublished, (event) => publishListener.handle(event), {
priority: NotifyCommentersListener.priority,
})
}
}
app/Providers/QueueProvider.tsimport { ServiceProvider, MemoryDriver, SyncDriver, createQueueManager, registerJob, type QueueManager } from '@guren/core'
import { SendCommentMailJob } from '../Jobs/SendCommentMailJob.js'
import { NotifyCommentersJob } from '../Jobs/NotifyCommentersJob.js'
export default class QueueProvider extends ServiceProvider {
register(): void {
const queue = createQueueManager({
// QUEUE_CONNECTION=sync executes jobs inline on dispatch (default,
// no worker process needed); 'memory' queues them for a Worker.
default: process.env.QUEUE_CONNECTION === 'memory' ? 'memory' : 'sync',
drivers: {
sync: () => new SyncDriver(),
memory: () => new MemoryDriver(),
},
})
this.container.instance('queue', queue)
}
boot(): void {
// A queued message carries the job's name, so the driver can only run a job
// the registry knows. Nothing in `guren check` looks for a missing one.
registerJob(SendCommentMailJob)
registerJob(NotifyCommentersJob)
const queue = this.container.make<QueueManager>('queue')
queue.driver()
}
}
そして publish アクションが告知します。
app/Http/Controllers/PostController.tsimport { Controller, ValidationException, paginate, type PaginatedPageProps } from '@guren/core'
import { pages } from '@/.guren/pages.gen'
import { Post } from '../../Models/Post.js'
import { Comment } from '../../Models/Comment.js'
import { Tag } from '../../Models/Tag.js'
import { PostTag } from '../../Models/PostTag.js'
import type { UserRecord } from '../../Models/User.js'
import { PostPublished } from '../../Events/PostPublished.js'
import { PostResource, type PostResourceData } from '../Resources/PostResource.js'
import { CommentResource } from '../Resources/CommentResource.js'
import { ListPostsQuerySchema, PostIdParamSchema, PostImageParamSchema, PostPayloadSchema } from '../Validators/PostValidator.js'
type PostsIndexProps = PaginatedPageProps<PostResourceData>
async function syncTags(postId: number, names: string[]): Promise<void> {
await PostTag.delete({ postId })
for (const name of names) {
const tag = (await Tag.first({ name })) ?? (await Tag.create({ name }))
await PostTag.forceCreate({ postId, tagId: tag.id })
}
}
export default class PostController extends Controller {
async index(): Promise<Response> {
const { page } = this.validateQuery(ListPostsQuerySchema)
const result = await Post.withPaginate('author', { page, perPage: 10, orderBy: ['id', 'desc'] })
const paginator = paginate(result, { path: this.request.path ?? '/posts' })
return this.inertia(pages.posts.Index, {
data: result.data.map((post) => new PostResource(post).toJSON()),
pagination: {
meta: paginator.meta(),
links: paginator.links(),
},
} satisfies PostsIndexProps)
}
async show(): Promise<Response> {
const { id } = this.validateParams(PostIdParamSchema)
const post = await Post.findWithOrFail(id, ['author', 'tags'])
const [withFiles] = await Post.withAttachments([post], ['cover', 'images'])
const comments = await Comment.where('postId', post.id).with('author').orderBy('id', 'asc').get()
return this.inertia(pages.posts.Show, {
post: new PostResource(withFiles!).toJSON(),
canManage: await this.can('update', [Post, post]),
comments: await Promise.all(
comments.map(async (comment) => ({
...new CommentResource(comment).toJSON(),
canDelete: await this.can('delete', [Comment, comment]),
})),
),
})
}
async create(): Promise<Response> {
return this.inertia(pages.posts.New, {})
}
async store(): Promise<Response> {
const author = await this.auth.userOrFail<UserRecord>()
const { tags, ...data } = await this.validateBody(PostPayloadSchema)
const post = await Post.forceCreate({ ...data, authorId: author.id })
await syncTags(post.id, tags)
const cover = await this.file('cover')
if (cover) {
await Post.attach(post.id, 'cover', cover)
}
for (const file of await this.files('images')) {
await Post.attach(post.id, 'images', file)
}
return this.redirect(`/posts/${post.id}`)
}
async edit(): Promise<Response> {
const post = this.model(Post)
await this.authorize('update', [Post, post])
const withTags = await Post.findWithOrFail(post.id, 'tags')
return this.inertia(pages.posts.Edit, {
post: new PostResource(withTags).toJSON(),
})
}
async update(): Promise<Response> {
const post = this.model(Post)
await this.authorize('update', [Post, post])
const { tags, ...data } = await this.validateBody(PostPayloadSchema)
await Post.update({ id: post.id }, data)
await syncTags(post.id, tags)
return this.redirect(`/posts/${post.id}`)
}
async cover(): Promise<Response> {
const post = this.model(Post)
await this.authorize('update', [Post, post])
const cover = await this.file('cover')
if (!cover) {
throw new ValidationException({ cover: ['Choose an image.'] })
}
await Post.attach(post.id, 'cover', cover)
return this.redirect(`/posts/${post.id}`)
}
async destroyImage(): Promise<Response> {
const post = this.model(Post)
await this.authorize('update', [Post, post])
const { attachment } = this.validateParams(PostImageParamSchema)
await Post.detach(post.id, 'images', attachment)
return this.redirect(`/posts/${post.id}`)
}
async destroy(): Promise<Response> {
const post = this.model(Post)
await this.authorize('delete', [Post, post])
await Post.purgeAttachments(post.id)
await Post.delete({ id: post.id })
return this.redirect('/posts')
}
async publish(): Promise<Response> {
const post = this.model(Post)
await this.authorize('publish', [Post, post])
await Post.forceUpdate({ id: post.id }, { publishedAt: new Date().toISOString() })
await this.make('events').emit(new PostPublished(post.id))
return this.redirect(`/posts/${post.id}`)
}
async unpublish(): Promise<Response> {
const post = this.model(Post)
await this.authorize('publish', [Post, post])
await Post.forceUpdate({ id: post.id }, { publishedAt: null })
return this.redirect(`/posts/${post.id}`)
}
}
bun test
rubric は次のとおりです。
registerJob(NotifyCommentersJob)がQueueProvider.boot()にあり、events.on(PostPublished, …)がEventProvider.boot()にある。両方が無ければ、この機能はコンパイルの通る死んだコードです。- ペイロードは
{ postId }。宛先は渡されるのではなくhandleの中で解決される。 - コメントした人が著者 id で重複排除され、投稿の著者がリストから除かれる。しかもジョブの中で。Bob からのコメント 2 件は、Bob へのメール 1 通です。
publishは emit して戻る。コメントを問い合わせもしないし、メールの存在も知らない。- 新しいテスト 2 件と、第 2 節の 3 件がどちらも緑。
チェックポイント: 下書きに 2 つのアカウントからコメントし、それを公開して、[mail] のブロックが 2 つ出ることと、自分宛ての 3 つ目が出ないことを確かめてください。
bunx guren gate
git add -A
git commit -m "feat: mail commenters when a post is published"
いまいる場所
- イベント、listener、ジョブ、メール。そのそれぞれが、指し示せる場所で登録されている。
- 告知して戻るコントローラーと、送信の隣に置かれた宛先に関する業務ルール。
- テストの継ぎ目 3 つ: 本物のマネージャーの中のメールトランスポート、グローバルに設定されたキュードライバー、そしてその 2 つがつながっていることを示すリクエスト。
guren checkが意見を持たない唯一の不変条件を運ぶプロジェクトの rule と、それに従ったエージェント。
よくあるつまずき
SyncDriver: job class "X" is not registered.QueueProvider.boot()にregisterJob(X)がありません。第 4 節の rule が防ぐために存在する、まさにこのエラーです。Email must have at least one recipient(あるいは subject、body)。send()は組み立てられたメッセージを検証します。undefinedを受け取ったto()も、件名を設定する前に return するbuild()も、どちらもここに行き着きます。- 何も届かないのにエラーも出ない。 listener が
EventProvider.boot()で配線されているか確かめてください。listener がひとつも無いイベントは、成功したemitです。 - テストで
container.fake('queue', …)をしても何も変わらない。Job.dispatch()はコンテナではなく、モジュールレベルの setter からドライバーを解決します。setQueueDriver()を使い、前のドライバーを戻してください。 fakeMail()でmailを直接 fake したテストが throw する。Mail.send()はmanager.transport(name)を呼びますが、fake はマネージャーではなくトランスポートです。本物のMailManagerに登録し、それをバインドしてください。- キューがあるのにメールがリクエストの中で送られる。 それは
QUEUE_CONNECTION=syncが設計どおりに動いているということです。memoryに設定してbunx guren queue:workを実行すれば、代わりにワーカーがキューを処理するのを見られます。
演習
.envのQUEUE_CONNECTIONをmemoryにしてサーバーを再起動し、コメントを投稿してください。メールは出ません。次に別のターミナルでbunx guren queue:work --onceを走らせてください。それでも何も起きません。理由を説明してから値を戻してください。その答えが、memoryが開発用のドライバーであってデプロイ用でない理由です。CommentPostedに、ログを出すだけでpriorityの高い listener をもう 1 つ登録してください。先に走るのはどちらですか。次に先に走るほうで例外を投げて、もう一方とリクエストに何が起きるかを答えてください。
次へ
第 12 章: アプリをエージェントのツールにする では、すでにあるルートをエージェントが呼び出せるツールに変え、第 7 章と同じ認可のギャップが、通る audit ではなく明確な失敗になるところを示します。