Skip to content

feat(integrations): load bundle capability manifests - #14920

Open
erichare wants to merge 1 commit into
fix/le-2460-connection-contractfrom
fix/int-3-extension-integrations
Open

feat(integrations): load bundle capability manifests#14920
erichare wants to merge 1 commit into
fix/le-2460-connection-contractfrom
fix/int-3-extension-integrations

Conversation

@erichare

@erichare erichare commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

  • Add optional, bundle-owned integration capability manifest references to the extension schema.
  • Validate versioned provider metadata covering actions, authentication, required scopes, policy keys, execution substrate, maturity, and deployment contexts.
  • Expose validated capabilities through both extension load results and the process-wide bundle registry.
  • Document the authoring contract while preserving compatibility with extension manifests that omit integrations.

Stacked on #14919.

Compatibility and safety

  • Existing extension manifests continue to load with an empty integrations collection.
  • Capability paths must be relative JSON paths contained by the owning bundle.
  • Provider IDs must match between the extension reference and capability document.
  • Capability validation runs before model-provider registration side effects.

Validation

  • 699 passed — lfx integration and extension test suites
  • 36 passed — capability-matrix CI test suite
  • Ruff checks passed
  • Extension schema export passed
  • Capability-matrix completeness check passed
  • Bundle API changelog gate and commit hooks passed
  • scripts/ci/sync_bundle_lfx_pin.py 1.13.0 completed without a diff because existing bundle floors already use the canonical constraint

@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

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: Team

Run ID: b1297129-8620-40a2-a15f-e8fe9cfd2d76

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

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.

@github-actions github-actions Bot added the enhancement New feature or request label Sep 3, 2026
@github-actions

github-actions Bot commented Sep 3, 2026

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.

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Build successful! ✅
Deploying docs draft.
Deploy successful! View draft

@github-actions github-actions Bot added enhancement New feature or request and removed enhancement New feature or request labels Sep 3, 2026
@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 74.10072% with 36 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.50%. Comparing base (47c482e) to head (70f7e0c).

Files with missing lines Patch % Lines
src/lfx/src/lfx/extension/manifest.py 58.62% 8 Missing and 4 partials ⚠️
src/lfx/src/lfx/integrations/capabilities.py 64.70% 8 Missing and 4 partials ⚠️
src/lfx/src/lfx/extension/integration_manifest.py 87.50% 3 Missing and 1 partial ⚠️
src/lfx/src/lfx/extension/loader/_orchestrator.py 77.77% 1 Missing and 3 partials ⚠️
src/lfx/src/lfx/extension/validate.py 57.14% 1 Missing and 2 partials ⚠️
src/lfx/src/lfx/extension/loader/_startup.py 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@                         Coverage Diff                         @@
##           fix/le-2460-connection-contract   #14920      +/-   ##
===================================================================
- Coverage                            66.53%   66.50%   -0.03%     
===================================================================
  Files                                 2515     2516       +1     
  Lines                               262282   262413     +131     
  Branches                             36857    36879      +22     
===================================================================
+ Hits                                174497   174516      +19     
- Misses                               85575    85673      +98     
- Partials                              2210     2224      +14     
Flag Coverage Δ
backend 73.98% <ø> (-0.15%) ⬇️
lfx 65.11% <74.10%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/lfx/src/lfx/extension/bundle_registry.py 87.03% <100.00%> (+0.89%) ⬆️
src/lfx/src/lfx/extension/loader/_types.py 94.54% <100.00%> (+1.21%) ⬆️
src/lfx/src/lfx/extension/reload.py 86.17% <ø> (ø)
src/lfx/src/lfx/interface/components.py 59.56% <100.00%> (+0.05%) ⬆️
src/lfx/src/lfx/extension/loader/_startup.py 89.74% <0.00%> (-2.37%) ⬇️
src/lfx/src/lfx/extension/validate.py 79.84% <57.14%> (-0.63%) ⬇️
src/lfx/src/lfx/extension/integration_manifest.py 87.50% <87.50%> (ø)
src/lfx/src/lfx/extension/loader/_orchestrator.py 83.77% <77.77%> (-0.57%) ⬇️
src/lfx/src/lfx/extension/manifest.py 83.03% <58.62%> (-2.96%) ⬇️
src/lfx/src/lfx/integrations/capabilities.py 76.98% <64.70%> (-5.13%) ⬇️

... and 35 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

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Frontend Unit Test Coverage Report

Coverage Summary

Lines Statements Branches Functions
Coverage: 55%
55.65% (84754/152292) 72.47% (12552/17320) 50.95% (2000/3925)

Unit Test Results

Tests Skipped Failures Errors Time
6641 0 💤 0 ❌ 0 🔥 25m 2s ⏱️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant