Skip to content

Fix config validation and animation lifecycle - #57

Merged
RockerFlower merged 5 commits into
masterfrom
codex/fix-config-validation-lifecycle
Jul 8, 2026
Merged

RockerFlower merged 5 commits into
masterfrom
codex/fix-config-validation-lifecycle

Conversation

@RockerFlower

@RockerFlower RockerFlower commented Jul 8, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Adds runtime validation for layer lengths, durations, height percentages, and gradients.
  • Makes SingleConfig and RandomConfig render with sensible defaults while keeping existing CustomConfig usage compatible.
  • Improves WaveWidget animation lifecycle handling, including updates and non-looping animations.
  • Adds Flutter 3.7 CI, FVM version metadata, tests, and README/CHANGELOG notes.
  • Excludes deployment and IDE metadata from the pub.dev package with .pubignore.

Why

The package exposed SingleConfig and RandomConfig, but only CustomConfig rendered. CustomConfig also missed gradient length validation, which could fail later during paint/build with a range error. The widget lifecycle kept animation state in a way that could become stale across widget updates.

Compatibility

This is intended as a non-breaking bugfix. The existing Dart SDK constraint remains unchanged, and CustomConfig keeps its existing constructor shape and nullable public fields.

Validation

  • flutter pub get
  • flutter analyze
  • flutter test
  • flutter pub publish --dry-run
  • GitHub Actions: CI passed
  • GitHub Actions: Deploy Workers Static Assets PR build passed
  • git diff --check

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
wave Error Error Jul 8, 2026 4:41am

@RockerFlower
RockerFlower marked this pull request as ready for review July 8, 2026 06:04
@RockerFlower
RockerFlower merged commit 44a9c1c into master Jul 8, 2026
2 checks passed
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.

1 participant