Skip to content

docs: update skill#488

Merged
jpudysz merged 1 commit intomainfrom
feature/update-skill
Apr 7, 2026
Merged

docs: update skill#488
jpudysz merged 1 commit intomainfrom
feature/update-skill

Conversation

@jpudysz
Copy link
Copy Markdown
Collaborator

@jpudysz jpudysz commented Apr 7, 2026

Summary by CodeRabbit

  • New Features

    • Display P3 color support with theme variables
    • Built-in utilities: border-continuous and border-circular classes
    • Reanimated entering/exiting animations with class presets and modifiers
    • Layout transitions support
    • Suspense Support
    • Custom animate-* classes (Pro)
    • Blur transition presets (Pro)
  • Documentation

    • Updated version requirement to 1.6.0+
    • Expanded Pro feature and animation API documentation
    • Theme Transitions API updated with new syntax
  • Pro Updates

    • Pricing model changed to graduated annual per-seat tiers
    • Installation instructions updated

@jpudysz jpudysz requested a review from Brentlok April 7, 2026 11:12
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 7, 2026

📝 Walkthrough

Walkthrough

This PR updates the Uniwind skill documentation to reflect version 1.6.0+, new animation features (Reanimated entering/exiting and layout transitions), Display P3 color support, graduated Pro pricing tiers, updated API usage patterns for theme transitions, and new built-in utility classes.

Changes

Cohort / File(s) Summary
Documentation Updates
skills/uniwind/SKILL.md
Version bump to 1.6.0+; new Display P3 color support, built-in border utilities, custom animate classes, and Reanimated animation presets; Pro pricing changed to graduated per-seat annual tiers; Theme Transitions API refactored from direct preset parameter to options object; Suspense Support added; component animation behavior clarifications.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • docs: add uniwind skill #433: Introduces the initial Uniwind skill documentation that this PR extends with new features, version updates, and API changes.

Suggested reviewers

  • Brentlok

Poem

🐰 A skill doc takes flight, with colors so true,
From v1.5 to 1.6—fresh features and new!
Reanimated dances with entering grace,
While themes transition at a beautiful pace.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'docs: update skill' is too vague and generic. While it correctly identifies that this is a documentation change, it does not convey what specific aspects of the skill documentation were updated (version changes, new color support, pricing updates, animations, etc.). Use a more specific title that highlights the main changes, such as 'docs: update Uniwind skill documentation with v1.6.0 features and Pro pricing updates' or 'docs: add Display P3, animations, and update Uniwind Pro guidance'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/update-skill

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 6

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@skills/uniwind/SKILL.md`:
- Line 1824: The docs show Uniwind.setTheme being called with a second
transition options argument (e.g., Uniwind.setTheme('dark', { preset:
ThemeTransitionPreset.Fade })) but the real API (setTheme in config.common.ts)
only accepts one parameter, causing TS/runtime errors; fix by either updating
the documentation to remove the second argument and explicitly state that theme
transitions are not accepted by Uniwind.setTheme, or implement an
overloaded/extended API in the core by adding a second parameter (e.g.,
ThemeTransitionOptions) to setTheme (and its type/enum like
ThemeTransitionPreset) and wire the transition logic in setTheme so both docs
and implementation match. Ensure references to Uniwind.setTheme, setTheme,
ThemeTransitionPreset (and ThemeTransitionOptions if you add it) are updated
consistently.
- Around line 1689-1695: The docs claim auto-swapping for Text, TextInput, and
Pressable but the code shows Animated exports in
packages/uniwind/src/components/web/Animated.tsx only expose View, Text, Image,
ScrollView, FlatList, SectionList and native Text in
packages/uniwind/src/components/native/Text.tsx wraps RNText (not
Animated.Text), so fix by either (A) making implementation match docs: add
wrappers/exports for Animated.TextInput and Animated.Pressable in Animated.tsx
and change Text.tsx to wrap/forward to Animated.Text (ensure TextInput/Pressable
use Animated equivalents and update any related prop forwarding), or (B) update
SKILL.md to remove/adjust the auto-swap claims and list only the actually
supported components (match the export list from Animated.tsx and note that
native Text does not auto-wrap). Choose one approach and apply it consistently
across SKILL.md and the component files (reference Animated.tsx, Text.tsx,
TextInput, Pressable, and SKILL.md).
- Around line 999-1016: The "Display P3 Colors support" documentation block
incorrectly claims Uniwind parses and converts color(display-p3 ...) values;
either remove this block entirely or change it to clearly state the feature is
not implemented (e.g., mark as "planned" or "experimental") and/or link to the
tracking issue. Locate the "Display P3 Colors support" header and the CSS
example containing the literal text "color(display-p3" in SKILL.md and update
the text accordingly so it does not assert existing functionality.
- Around line 27-29: The SKILL.md header claiming "Uniwind 1.6.0+" conflicts
with the package version "1.4.1" in package.json; fix by aligning the docs and
release state: either change the SKILL.md header and any references to "1.6.0+"
and described features (e.g., "Display P3 colors", "new animations", "updated
APIs") to "1.4.1" and remove unreleased feature claims, or explicitly mark the
document as "unreleased / upcoming" and add a clear note that features are for a
future 1.6.0 release (and ensure a corresponding git tag/release is created if
you intend to ship 1.6.0). Also update any install instructions referencing
version 1.6.0 so they point to the actual "1.4.1" package or to a prerelease
tag.
- Around line 1854-1856: The docs list
ThemeTransitionPreset.Blur/BlurRightToLeft/BlurLeftToRight which aren’t
implemented; either remove those rows from SKILL.md or implement the presets and
wiring: add a ThemeTransitionPreset enum (with Blur, BlurRightToLeft,
BlurLeftToRight), export it, extend the setTheme function signature to accept an
optional preset parameter and options, implement handling in the theme
application code to map those presets to actual transition behavior, and update
any exports so ThemeTransitionPreset is available to consumers; choose one
approach and make the corresponding change and tests/docs consistent.
- Around line 1697-1738: The "Entering & Exiting Animations" documentation
falsely advertises uw-entering-*, uw-exiting-* class handlers, animation
modifiers, and an auto-upgrade mechanism that don't exist; remove this entire
section from SKILL.md and any references to these presets and modifiers, and
instead add a short note directing users to implemented animation features (or
mark as planned). Search for the documented symbols (uw-entering-*,
uw-exiting-*, uw-layout-*, and mentions of withUniwind HOC/auto-upgrade) in the
markdown and delete the whole block describing presets, modifiers, and examples
so docs match the current codebase.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6aa3a9ff-c2d8-40be-9afb-0d99c0eaa473

📥 Commits

Reviewing files that changed from the base of the PR and between d3b7f5e and 55846f8.

📒 Files selected for processing (1)
  • skills/uniwind/SKILL.md

Comment thread skills/uniwind/SKILL.md
Comment thread skills/uniwind/SKILL.md
Comment thread skills/uniwind/SKILL.md
Comment thread skills/uniwind/SKILL.md
Comment thread skills/uniwind/SKILL.md
Comment thread skills/uniwind/SKILL.md
@jpudysz jpudysz merged commit dfaa63d into main Apr 7, 2026
2 checks passed
@jpudysz jpudysz deleted the feature/update-skill branch April 7, 2026 11:22
@github-actions
Copy link
Copy Markdown
Contributor

🚀 This pull request is included in v1.6.3. See Release v1.6.3 for release notes.

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.

2 participants