Skip to content

Simplify trigger syntax#623

Merged
leo merged 19 commits intomainfrom
triggers
Oct 29, 2025
Merged

Simplify trigger syntax#623
leo merged 19 commits intomainfrom
triggers

Conversation

@leo
Copy link
Copy Markdown
Contributor

@leo leo commented Oct 28, 2025

This change further simplifies the syntax of triggers.


Note

Simplifies trigger handlers to a single options parameter, updates typing and integration across client/server, and adds a utility to compose and chain triggers.

  • Triggers (client):
    • Replace multi-arg handlers with a single options object (TriggerOptions) carrying query, multipleRecords, client, context, parentTrigger, previousRecords, records, etc.
    • Export TRIGGER_TYPES, getTriggerName; rename invokeTriggers to applyTrigger and getMethodName to getTriggerName.
    • Unify trigger type aliases into generic Trigger, Triggers, and TriggersPerModel; adjust QueryHandlerOptions.triggers to TriggersPerModel and parentTrigger typing.
  • Server integration:
    • Update server-side trigger types to mirror client generics and options shape; simplify trigger wrappers to pass merged options; adjust TriggersList typing.
  • Utilities:
    • Export trigger helpers via utils; remove omit helper; add universal triggers composer to merge/chain trigger groups; re-export in public schema API.
  • Tests:
    • Migrate integration tests to new options-based trigger signatures and updated type names; validate sink/database trigger options and following/resolving behavior.

Written by Cursor Bugbot for commit 7c0077c. This will update automatically on new commits. Configure here.

@leo leo requested review from NuroDev, colodenn and juriadams October 28, 2025 18:17
@leo leo added the enhancement New feature or request label Oct 28, 2025
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Oct 28, 2025

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
docs 7c0077c Commit Preview URL

Branch Preview URL
Oct 29 2025, 03:26 PM

@netlify
Copy link
Copy Markdown

netlify bot commented Oct 28, 2025

Deploy Preview for blade-basic canceled.

Name Link
🔨 Latest commit 7c0077c
🔍 Latest deploy log https://app.netlify.com/projects/blade-basic/deploys/690231d90b059800085d9915

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Oct 28, 2025

Open in StackBlitz

blade

npm i https://pkg.pr.new/ronin-co/blade@623

blade-auth

npm i https://pkg.pr.new/ronin-co/blade/blade-auth@623

blade-cli

npm i https://pkg.pr.new/ronin-co/blade/blade-cli@623

blade-client

npm i https://pkg.pr.new/ronin-co/blade/blade-client@623

blade-codegen

npm i https://pkg.pr.new/ronin-co/blade/blade-codegen@623

blade-compiler

npm i https://pkg.pr.new/ronin-co/blade/blade-compiler@623

blade-syntax

npm i https://pkg.pr.new/ronin-co/blade/blade-syntax@623

create-blade

npm i https://pkg.pr.new/ronin-co/blade/create-blade@623

commit: 7c0077c

@leo leo marked this pull request as ready for review October 29, 2025 15:16
@leo leo enabled auto-merge (squash) October 29, 2025 15:25
@leo leo merged commit 2cf5a10 into main Oct 29, 2025
15 of 18 checks passed
@leo leo deleted the triggers branch October 29, 2025 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants