Skip to content

feat(website): add Framefields hero animation for the landing page - #803

Merged
Ferryx349 merged 1 commit into
mainfrom
feat/website-hero-animation
Oct 9, 2026
Merged

Ferryx349 merged 1 commit into
mainfrom
feat/website-hero-animation

Conversation

@Ferryx349

@Ferryx349 Ferryx349 commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Description

This PR adds website/hero-animation/, Framefields source for the landing herowhich includes :- what nostream is, event flow, features, deploy, NIP count from root package.json, Biome/Docker/changeset tweaks only.

Related issue

closes :- #804

How Has This Been Tested?

Test: cd website/hero-animation && pnpm install && pnpm run setup && pnpm run typecheck

Screenshots (if appropriate):

nostream-hero-low.mp4

@changeset-bot

changeset-bot Bot commented Oct 9, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 2e8b76a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedframefields@​2.0.86910010091100
Added@​types/​node@​24.19.11001008196100
Addedtsx@​4.23.151001008193100
Addedtypescript@​5.9.31001009010090
Addedsharp@​0.35.59710010094100

View full report

@greptile-apps

greptile-apps Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

[Low impact] Adds a standalone animation tool for the website landing page.

The PR appears safe to merge, with a non-blocking follow-up to check the animation package in CI.

Findings

  1. P2 CI skips the animation package ▶

Summary

Adds a standalone Framefields package for a 34-second landing-page animation.

  • The landing-page animation explains how nostream works.
  • Contributors can preview the animation and export video or frames.

Non-blocking follow-up: add CI coverage for the package's type check.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Fonts[Font downloads] --> Setup[Setup assets]
  Logos[Brand images] --> Setup
  Effects[Generated backgrounds] --> Setup
  Setup --> Film[Build animation]
  Package[Root supported NIPs] --> Film
  Film --> Preview[Browser preview]
  Film --> Video[MP4 export]
  Film --> Frames[PNG frames]
Loading

Reviews (1) · Last reviewed commit: "feat(website): add Framefields hero anim..." · Reviewed by Greptile

"render": "tsx src/render.ts video",
"render:small": "tsx src/render.ts video low",
"frames": "tsx src/render.ts frames 130 300 500 620 800 1010",
"typecheck": "tsc -p ."

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 CI skips the animation package

The new typecheck command is not run by CI. Existing checks install only the root package, and later changes under website/** do not trigger the build job. Future animation changes can pass CI even when the package no longer compiles.

As a non-blocking follow-up, add a separate job that installs this package with its lockfile and runs pnpm run typecheck when these files change. This catches broken imports and API changes without requiring a GPU.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 72.58%. remained the same — feat/website-hero-animation into main

@Ferryx349
Ferryx349 merged commit 486f513 into main Oct 9, 2026
18 of 19 checks passed
@Ferryx349
Ferryx349 deleted the feat/website-hero-animation branch October 9, 2026 14:07
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.

3 participants