Skip to content

clean up subpackage pyproject.toml dependencies#6377

Merged
adhami3310 merged 1 commit intomainfrom
masenf/subpackage-lower-bounds
Apr 23, 2026
Merged

clean up subpackage pyproject.toml dependencies#6377
adhami3310 merged 1 commit intomainfrom
masenf/subpackage-lower-bounds

Conversation

@masenf
Copy link
Copy Markdown
Collaborator

@masenf masenf commented Apr 23, 2026

set lower bounds for all internal packages that appear in pyproject.toml dependencies

set lower bounds for all internal packages that appear in pyproject.toml
dependencies
@masenf masenf requested a review from a team as a code owner April 23, 2026 22:28
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 23, 2026

Greptile Summary

This PR adds >= 0.9.0 lower bounds to all internal reflex-* package dependencies across 15 subpackage pyproject.toml files and the root pyproject.toml. It also makes reflex-base an explicit direct dependency for packages that previously declared empty or implicit dependency lists.

Confidence Score: 5/5

This PR is safe to merge — it only adds version lower bounds and makes implicit dependencies explicit, with no functional code changes.

All changes are purely in pyproject.toml dependency declarations. The version constraint >= 0.9.0 is applied uniformly across all internal packages. No logic, APIs, or runtime behavior is modified. No P0 or P1 findings were identified.

No files require special attention.

Important Files Changed

Filename Overview
pyproject.toml Adds >= 0.9.0 lower bounds to all 13 internal reflex-* component dependencies; no other changes.
packages/reflex-components-code/pyproject.toml Adds explicit reflex-base >= 0.9.0 dependency and lower bounds to existing component dependencies.
packages/reflex-components-core/pyproject.toml Adds explicit reflex-base >= 0.9.0 and lower bounds to existing component dependencies alongside third-party deps.
packages/reflex-components-lucide/pyproject.toml Previously had empty dependencies; now explicitly declares reflex-base >= 0.9.0.
packages/reflex-components-gridjs/pyproject.toml Previously had empty dependencies; now explicitly declares reflex-base >= 0.9.0.
packages/reflex-components-moment/pyproject.toml Previously had empty dependencies; now explicitly declares reflex-base >= 0.9.0.
packages/reflex-components-recharts/pyproject.toml Previously had empty dependencies; now explicitly declares reflex-base >= 0.9.0.
packages/reflex-components-internal/pyproject.toml Adds >= 0.9.0 lower bound to reflex; retains reflex (full package) rather than reflex-base, consistent with its role.
packages/reflex-docgen/pyproject.toml Adds >= 0.9.0 lower bound to reflex; other third-party deps already had lower bounds.
packages/reflex-site-shared/pyproject.toml Adds >= 0.9.0 lower bounds to reflex and reflex-components-internal.
packages/reflex-components-markdown/pyproject.toml Adds explicit reflex-base >= 0.9.0 and lower bounds to existing component dependencies.
packages/reflex-components-radix/pyproject.toml Adds explicit reflex-base >= 0.9.0 and lower bounds to existing component dependencies.
packages/reflex-components-dataeditor/pyproject.toml Adds explicit reflex-base >= 0.9.0 and lower bound to existing reflex-components-core dependency.
packages/reflex-components-plotly/pyproject.toml Adds explicit reflex-base >= 0.9.0 and lower bound to existing reflex-components-core dependency.
packages/reflex-components-react-player/pyproject.toml Adds explicit reflex-base >= 0.9.0 and lower bound to existing reflex-components-core dependency.
packages/reflex-components-sonner/pyproject.toml Adds explicit reflex-base >= 0.9.0 and lower bound to existing reflex-components-lucide dependency.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    reflex["reflex\n(root pyproject.toml)"]
    reflex-base["reflex-base >= 0.9.0"]
    core["reflex-components-core >= 0.9.0"]
    lucide["reflex-components-lucide >= 0.9.0"]
    sonner["reflex-components-sonner >= 0.9.0"]
    radix["reflex-components-radix >= 0.9.0"]
    code["reflex-components-code >= 0.9.0"]
    markdown["reflex-components-markdown >= 0.9.0"]
    dataeditor["reflex-components-dataeditor >= 0.9.0"]
    plotly["reflex-components-plotly >= 0.9.0"]
    recharts["reflex-components-recharts >= 0.9.0"]
    gridjs["reflex-components-gridjs >= 0.9.0"]
    moment["reflex-components-moment >= 0.9.0"]
    react-player["reflex-components-react-player >= 0.9.0"]

    reflex --> reflex-base
    reflex --> core
    reflex --> lucide
    reflex --> sonner
    reflex --> radix
    reflex --> code
    reflex --> markdown
    reflex --> dataeditor
    reflex --> plotly
    reflex --> recharts
    reflex --> gridjs
    reflex --> moment
    reflex --> react-player

    core --> reflex-base
    core --> lucide
    core --> sonner

    lucide --> reflex-base
    sonner --> reflex-base
    sonner --> lucide

    radix --> reflex-base
    radix --> core
    radix --> lucide

    code --> reflex-base
    code --> core
    code --> radix
    code --> lucide

    markdown --> reflex-base
    markdown --> code
    markdown --> core
    markdown --> radix

    dataeditor --> reflex-base
    dataeditor --> core

    plotly --> reflex-base
    plotly --> core

    react-player --> reflex-base
    react-player --> core

    recharts --> reflex-base
    gridjs --> reflex-base
    moment --> reflex-base
Loading

Reviews (1): Last reviewed commit: "clean up subpackage pyproject.toml depen..." | Re-trigger Greptile

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 23, 2026

Merging this PR will not alter performance

✅ 9 untouched benchmarks


Comparing masenf/subpackage-lower-bounds (bb0624d) with main (e9a7b86)

Open in CodSpeed

@adhami3310 adhami3310 merged commit ab3550b into main Apr 23, 2026
68 of 69 checks passed
@adhami3310 adhami3310 deleted the masenf/subpackage-lower-bounds branch April 23, 2026 23: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.

2 participants