Skip to content

Pin some docs app components#6501

Merged
Alek99 merged 2 commits into
mainfrom
carlos/pin-some-docs-components
May 13, 2026
Merged

Pin some docs app components#6501
Alek99 merged 2 commits into
mainfrom
carlos/pin-some-docs-components

Conversation

@carlosabadia
Copy link
Copy Markdown
Contributor

No description provided.

@carlosabadia carlosabadia requested review from a team and Alek99 as code owners May 13, 2026 10:35
@carlosabadia carlosabadia added the documentation Improvements or additions to documentation label May 13, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 13, 2026

Greptile Summary

This PR pins specific npm package versions for the docs app to improve build reproducibility. A new bun.lock + package.json snapshot are committed under docs/app/reflex.lock/, and version strings are added to several Python component imports (clsx-for-tailwind@1.0.0, tailwindcss-animated@2.0.0, etc.) and documentation examples.

  • docs/app/reflex.lock/bun.lock and package.json are added to lock all resolved npm dependency versions for the docs build.
  • Python component files (twmerge.py, marketing_button.py) and rxconfig.py are updated to include explicit version suffixes in their npm package references.
  • pyproject.toml adds */bun.lock to the codespell skip list to avoid false positives from the new lock file.

Confidence Score: 4/5

Safe to merge — the changes are dependency version pins and a new lock file with no functional logic changes.

All Python changes are straightforward version-string additions to import declarations. The main concern is four packages in package.json (moment-timezone, react-dnd, react-dnd-html5-backend, reactflow) still use caret ranges, which is inconsistent with the rest of the file and could allow unintended updates if the lock file is ever regenerated. No runtime logic is affected by this PR.

docs/app/reflex.lock/package.json — four packages use caret range specifiers instead of exact versions, which is inconsistent with the rest of the file.

Important Files Changed

Filename Overview
docs/app/reflex.lock/package.json New package.json with mostly pinned exact versions, but four packages (moment-timezone, react-dnd, react-dnd-html5-backend, reactflow) still use caret range specifiers inconsistently.
docs/app/reflex.lock/bun.lock New bun.lock file that pins all npm dependency resolutions for the docs app; provides reproducible builds for CI/CD.
docs/app/rxconfig.py Pins tailwindcss-animated to version 2.0.0 in frontend_packages config.
packages/reflex-components-internal/src/reflex_components_internal/utils/twmerge.py Pins clsx-for-tailwind to 1.0.0 in the VarData import declaration.
packages/reflex-site-shared/src/reflex_site_shared/components/marketing_button.py Pins clsx-for-tailwind to 1.0.0 in the add_imports() return value.
docs/wrapping-react/overview.md Pins @splinetool/react-spline and react-colorful (two occurrences) to specific versions in documentation examples.
pyproject.toml Adds */bun.lock to the codespell skip list to prevent false positives on the newly added lock file.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["docs/app/rxconfig.py (pinned)"] --> B[Reflex Build]
    C["package.json (exact versions)"] --> D[Bun Install]
    E["bun.lock (resolved hashes)"] --> D
    D --> B
    B --> F[Reproducible Frontend Bundle]
    G["twmerge.py (clsx-for-tailwind@1.0.0)"] --> B
    H["marketing_button.py (clsx-for-tailwind@1.0.0)"] --> B
Loading

Reviews (1): Last reviewed commit: "codespell skip" | Re-trigger Greptile

Comment thread docs/app/reflex.lock/package.json
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 13, 2026

Merging this PR will not alter performance

✅ 24 untouched benchmarks
⏩ 2 skipped benchmarks1


Comparing carlos/pin-some-docs-components (8d8189e) with main (606bd55)

Open in CodSpeed

Footnotes

  1. 2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@Alek99 Alek99 merged commit c3c720f into main May 13, 2026
70 checks passed
@Alek99 Alek99 deleted the carlos/pin-some-docs-components branch May 13, 2026 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants