Skip to content

fix(ci): rename bundles to -nightly during nightly build#13194

Merged
erichare merged 1 commit into
feat/bundle-mass-extractionfrom
fix/nightly-build-bundle-lfx-dep-mass
May 19, 2026
Merged

fix(ci): rename bundles to -nightly during nightly build#13194
erichare merged 1 commit into
feat/bundle-mass-extractionfrom
fix/nightly-build-bundle-lfx-dep-mass

Conversation

@erichare
Copy link
Copy Markdown
Collaborator

Summary

Companion to #13193 targeting feat/bundle-mass-extraction.

The nightly build script was missing a step to rename lfx-* bundle packages to their -nightly counterparts during nightly tag creation. This caused uv lock to fail with:

Because only lfx<=0.4.3 is available and lfx-arxiv depends on lfx>=0.5.0,<0.6.0, we can conclude that lfx-arxiv's requirements are unsatisfiable.

After PR #13193, the new scripts/ci/update_bundle_versions.py is glob-based (walks src/bundles/*/pyproject.toml), so it Just Works on this branch's expanded set of 86 bundles with no further changes. This PR is the same commit cherry-picked onto feat/bundle-mass-extraction.

Test plan

Verified locally on feat/bundle-mass-extraction by running the exact nightly script chain from the failing CI log:

  • update_sdk_version.py v0.2.0.dev38 — passes
  • update_lfx_version.py v0.5.0.dev38 v0.2.0.dev38 — passes
  • update_bundle_versions.py v0.5.0.dev38renames all 86 bundles (lfx-agentics, lfx-agentql, ..., lfx-zep) to their -nightly counterparts, version 0.1.0.dev38
  • update_pyproject_combined.py main v1.10.0.dev38 v1.10.0.dev38 v0.5.0.dev38 — passes
  • uv lock — resolves 864 packages (previously failed with lfx-arxiv requirements unsatisfiable)
  • cd src/backend/base && uv lock and cd src/lfx && uv lock — both pass
  • Re-running update_bundle_versions.py is idempotent (no diff)

🤖 Generated with Claude Code

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 19, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)
  • release-.*

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5bd4ce97-69d7-4ad1-bee6-8500b1661d59

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/nightly-build-bundle-lfx-dep-mass

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.

@github-actions github-actions Bot added the bug Something isn't working label May 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

✅ Test Coverage Advisor

No source changes detected without accompanying tests. Thanks for keeping coverage up! 🎉

Advisory check only — never blocks merge.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.19%. Comparing base (14058a4) to head (1314bb3).

Additional details and impacted files

Impacted file tree graph

@@                       Coverage Diff                       @@
##           feat/bundle-mass-extraction   #13194      +/-   ##
===============================================================
- Coverage                        55.68%   55.19%   -0.50%     
===============================================================
  Files                             2168     2168              
  Lines                           201764   201764              
  Branches                         30381    30380       -1     
===============================================================
- Hits                            112361   111368     -993     
- Misses                           88117    89112     +995     
+ Partials                          1286     1284       -2     
Flag Coverage Δ
backend 60.29% <ø> (-0.02%) ⬇️
frontend 54.35% <ø> (-0.73%) ⬇️
lfx 53.68% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 169 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Copy Markdown
Contributor

Frontend Unit Test Coverage Report

Coverage Summary

Lines Statements Branches Functions
Coverage: 38%
38.83% (48728/125475) 67.96% (6631/9756) 38.61% (1116/2890)

Unit Test Results

Tests Skipped Failures Errors Time
4332 0 💤 0 ❌ 0 🔥 9m 1s ⏱️

@erichare erichare merged commit d8868e7 into feat/bundle-mass-extraction May 19, 2026
117 of 119 checks passed
@erichare erichare deleted the fix/nightly-build-bundle-lfx-dep-mass branch May 19, 2026 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant