Skip to content

Update transitions branch with upstream changes#53

Merged
rauhryan merged 57 commits into
transitionsfrom
rr/pr-27-upstream-merge-fixes
Jun 13, 2026
Merged

Update transitions branch with upstream changes#53
rauhryan merged 57 commits into
transitionsfrom
rr/pr-27-upstream-merge-fixes

Conversation

@rauhryan

@rauhryan rauhryan commented May 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

This is a follow-up PR targeting the transitions branch from #27. It merges the latest main changes into the transitions work and includes small integration fixes found while validating the merged branch.

Changes

  • Merge latest origin/main into transitions
  • Resolve the upstream demo/ -> examples/keyboard/ relocation for transition demos
  • Export animating from the WASM build so the rebuilt native module matches term-native.ts
  • Include transition bytes when sizing snapshot() buffers
  • Allow transition ops in runtime validation
  • Enforce the existing spec rule that transition duration must be nonnegative

This branch is intended for Charles to merge into the PR #27 source branch (transitions).

cowboyd and others added 30 commits April 17, 2026 19:47
Introduces a new `snapshot(ops)` constructor that pre-packs a directive
array into its transfer encoding. The returned opaque `Op` can be
spliced into any directive array, and during packing its bytes are
copied directly into the command buffer without re-encoding.

This enables higher-level frameworks to implement dirty tracking:
unchanged component subtrees can reuse a cached snapshot, skipping
the per-frame packing cost entirely.
Co-Authored-By: ghostdevv <git@willow.sh>
for some reason the schema for the actions wants it to be an array
Without this it means that, for example, if I push a change to a PR then
shortly push again this workflow will be running twice. This change will
cancel the old run before starting the new one, which reduces the
overall actions cost and DX as you don't have extra runs
Without this it means that, for example, if I push a change to a PR then
shortly push again this workflow will be running twice. This change will
cancel the old run before starting the new one, which reduces the
overall actions cost and DX as you don't have extra runs
codspeed-hq Bot and others added 23 commits May 24, 2026 17:34
chore: update preview workflow
chore: update verify workflow
Adds performance benchmarks with tinybench, benchmark CI via CodSpeed
chore: update publish workflow
Add snapshot() for pre-packing directive subtrees
# Conflicts:
#	examples/keyboard/clay-transitions.ts
#	examples/keyboard/transitions.ts
@pkg-pr-new

pkg-pr-new Bot commented May 31, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/clayterm@53

commit: 7eab1e4

@rauhryan rauhryan merged commit 2bd8740 into transitions Jun 13, 2026
2 checks passed
@rauhryan rauhryan deleted the rr/pr-27-upstream-merge-fixes branch June 13, 2026 18:53
cowboyd added a commit that referenced this pull request Jun 26, 2026
* 📝 add transitions design specification

Design spec for clayterm transitions: frame-snapshot-compatible interpolation
of element position, size, and color properties. Defines the deltaTime
convention, the animating signal on RenderResult, declarative enter/exit
semantics that replace Clay's function-pointer callbacks, and cancellation
as a structural consequence of re-describing state. Implementation is
gated on bumping the Clay submodule past the upstream transition commit.

* ⬆️ bump Clay submodule to latest main (transitions API)

* 🔧 adapt clayterm to new Clay signatures (OpenTextElement, EndLayout)

* ⬆️ pin Clay to 938967a (work around upstream CLAY_WASM_EXPORT typo)

* ✨ add deltaTime parameter to reduce()

* 🔧 add deltaTime to Native.reduce signature

* ✨ track deltaTime on Term, accept deltaTime override

* ✨ add animating_count to Clayterm context

* 🔧 expose animating() via Native binding

* ✨ surface animating: boolean on RenderResult

* 📝 rewrite transitions spec for v1 (Clay-supported subset)

Scope v1 to what Clay currently supports without userData on transition
callbacks: one duration and one easing per element, applied to all listed
properties. Drop per-property longhand, enter/exit deltas, cubicBezier,
and corner radius — each with an explicit "Deferred Until Upstream Clay"
entry in §13 referencing nicbarker/clay#603 and the forthcoming exit-flag
work. Easings are plain string literals ("linear" | "easeIn" | "easeOut"
| "easeInOut") since v1 has no parametric easings.

* ✨ add transition property names, bitmask helpers, and Easing

* ✨ add transition field type to OpenElement

* ✨ encode transition block in pack()

* ✨ register Clay handlers, interpolate on property change

Co-Authored-By:

* ✨ reset deltaTime to 0 after idle (preserve transitions across long gaps)

* ✅ verify color transitions work in line mode

* 🎨 apply deno fmt and clang-format

* ✨ add transitions demo (collapsing sidebar)

* 📝 reference transitions-spec from renderer-spec

* ✨ rewrite transitions demo as interactive full-screen menu

* ✨ add clay-transitions demo port (v1-compatible subset)

Ports the spirit of the raylib-transitions demo to clayterm: a 4×4 grid
of colored boxes that animate position, size, and bg color. Shuffle (s)
animates positions via Clay's transition system; recolor (c) toggles
between two palettes with animated bg interpolation; hover tints each
box by blending its bg toward white (overlay-color field is not yet in
the v1 command buffer, so lighten-on-hover substitutes). Full mouse
tracking is wired via mouseTracking() + pointer state from input events.

* 🎨 let clay-transitions demo rows fill available height

* 🎨 remove modeline from clay-transitions demo

* 📝 note ct_active_context is a workaround for Clay userData PR

* 🎨 use border-only boxes in clay-transitions demo

* 🎨 prevent menu text from wrapping during sidebar transition

* 🔥 drop unused grow import in transitions-run test

* Update transitions branch with upstream changes (#53)

* ✨ add snapshot() for pre-packing directive subtrees

Introduces a new `snapshot(ops)` constructor that pre-packs a directive
array into its transfer encoding. The returned opaque `Op` can be
spliced into any directive array, and during packing its bytes are
copied directly into the command buffer without re-encoding.

This enables higher-level frameworks to implement dirty tracking:
unchanged component subtrees can reuse a cached snapshot, skipping
the per-frame packing cost entirely.

* docs: add maintainer build guide

* 💄 format build docs

* 🐛 improve pack string overflow errors

* os matrix test in ci (#36)

* 🧼 optimize build

* 🧼 compress bundled wasm

* 🧼 optimize wcwidth.c size

* ⚙️ update npm settings

* 🐛 install wasm-opt in ci

* ⚡ use brotli-11 + z85 wasm encoding

* 📌 pin @types/node to v22

* 🧼 apply @ghostdevv review suggestions from PR #35

Co-Authored-By: ghostdevv <git@willow.sh>

* 🔨 add type to bundle-wasm

* 💌 signed, sealed, delivered

* chore: use hashes for versions

* chore: don't save git credentials

* chore: use array syntax

for some reason the schema for the actions wants it to be an array

* perf: set concurrency limits to reduce cost and improve dx

Without this it means that, for example, if I push a change to a PR then
shortly push again this workflow will be running twice. This change will
cancel the old run before starting the new one, which reduces the
overall actions cost and DX as you don't have extra runs

* chore: use hashes for versions

* chore: update node version

* perf: set concurrency limits to reduce cost and improve dx

Without this it means that, for example, if I push a change to a PR then
shortly push again this workflow will be running twice. This change will
cancel the old run before starting the new one, which reduces the
overall actions cost and DX as you don't have extra runs

* chore: don't save git credentials

* chore: mitigate potential template injection

See https://docs.zizmor.sh/audits/#template-injection

* chore: update ::set-output command to new syntax

https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

* chore: use hashes for versions

* chore: don't save git credentials

* chore: limit id-token permission to the publishing steps

* chore: explicitly disable npm cache to mitigate cache poisoning attacks

* chore: mitigate potential template injection

See https://docs.zizmor.sh/audits/#template-injection

* chore: use oidc

This should be using OIDC for publishing

* chore: update ::set-output command to new syntax

https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

* 🙅 revert aggressive optimization experiments pending benchmark

Reverts three changes that need benchmarks before landing:
- -Oz / wasm-opt
- brotli+Z85 wasm compression
- wcwidth.c rewrite

* Add CodSpeed performance benchmarks

* 🔨 use deno

* 🧼 deno fmt

* ⚙️ vitest -> tinybench

* 🧼 remove codspeed assets

* 🔨 fix ci

* add type module

* fmt

* chore: update github url (#38)

* downgrade to tinybench@5

* move to examples folder with readme

* 🔧 export animating from wasm build

* 🧪 cover transitions in snapshots and validation

* ✅ enforce nonnegative transition duration

---------

Co-authored-by: Charles Lowell <cowboyd@frontside.com>
Co-authored-by: Jacob Bolda <me@jacobbolda.com>
Co-authored-by: Nate Moore <nate@natemoo.re>
Co-authored-by: ghostdevv <git@willow.sh>
Co-authored-by: Nate Moore <git@natemoo.re>
Co-authored-by: codspeed-hq[bot] <117304815+codspeed-hq[bot]@users.noreply.github.com>
Co-authored-by: Nate Moore <natemoo-re@users.noreply.github.com>

* remove uncecessary note

* clean up and consolidate

* re-organize transitions examples

* 🔥remove redundant cast

---------

Co-authored-by: Ryan Rauh <rauhryan@users.noreply.github.com>
Co-authored-by: Jacob Bolda <me@jacobbolda.com>
Co-authored-by: Nate Moore <nate@natemoo.re>
Co-authored-by: ghostdevv <git@willow.sh>
Co-authored-by: Nate Moore <git@natemoo.re>
Co-authored-by: codspeed-hq[bot] <117304815+codspeed-hq[bot]@users.noreply.github.com>
Co-authored-by: Nate Moore <natemoo-re@users.noreply.github.com>
Co-authored-by: Ryan Rauh <rauh.ryan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants