Skip to content

refactor(apollo-react): rename BYO edit-link field to byoConfigurationId#920

Open
denispetre wants to merge 1 commit into
mainfrom
refactor/model-picker-byo-configuration-id
Open

refactor(apollo-react): rename BYO edit-link field to byoConfigurationId#920
denispetre wants to merge 1 commit into
mainfrom
refactor/model-picker-byo-configuration-id

Conversation

@denispetre

Copy link
Copy Markdown
Contributor

Follow-up to #879, matching the Discovery contract shipped in UiPath/Arima#2659: the ModelPicker's BYO edit deep-link field is byomDetails.byoConfigurationId (was productLlmConfigurationId, which Discovery never served). Rename only — types, edit-action read, README, and the URL-builder doc comment. Behavior is unchanged: with the id the edit action deep-links to the configuration's edit form, without it it falls back to the configurations list.

Verified: tsc -b clean, 70/70 picker tests pass.

🤖 Generated with Claude Code

Match the Discovery contract (UiPath/Arima#2659):
byomDetails.byoConfigurationId instead of productLlmConfigurationId.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 16, 2026 12:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (PT)
apollo-design 🟢 Ready Preview, Logs Jul 16, 2026, 05:49:37 AM
apollo-docs 🟢 Ready Preview, Logs Jul 16, 2026, 05:49:37 AM
apollo-landing 🟢 Ready Preview, Logs Jul 16, 2026, 05:49:37 AM
apollo-vertex 🟢 Ready Preview, Logs Jul 16, 2026, 05:49:37 AM

@github-actions github-actions Bot added the size:S 10-29 changed lines. label Jul 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Dependency License Review

  • 1950 package(s) scanned
  • ✅ No license issues found
  • ⚠️ 2 package(s) excluded (see details below)
License distribution
License Packages
MIT 1720
ISC 89
Apache-2.0 55
BSD-3-Clause 27
BSD-2-Clause 23
BlueOak-1.0.0 8
MPL-2.0 4
MIT-0 3
CC0-1.0 3
MIT OR Apache-2.0 2
(MIT OR Apache-2.0) 2
Unlicense 2
LGPL-3.0-or-later 1
Python-2.0 1
CC-BY-4.0 1
(MPL-2.0 OR Apache-2.0) 1
Unknown 1
Artistic-2.0 1
(WTFPL OR MIT) 1
(BSD-2-Clause OR MIT OR Apache-2.0) 1
CC-BY-3.0 1
0BSD 1
(MIT OR CC0-1.0) 1
MIT AND ISC 1
Excluded packages
Package Version License Reason
@img/sharp-libvips-linux-x64 1.2.4 LGPL-3.0-or-later LGPL pre-built binary, not linked
khroma 2.1.0 Unknown MIT per GitHub repo, missing license field in package.json

@github-actions

Copy link
Copy Markdown
Contributor

📊 Coverage + size by package

Per-package coverage and bundle size on this PR. New-line coverage = of the source lines this PR adds or changes, the % hit by tests.

Package Coverage New-line coverage Packed (gzip) Unpacked vs main
@uipath/apollo-core 9.0% 43.82 MB 57.31 MB ±0
@uipath/apollo-react 35.7% — (7 untracked) 7.35 MB 27.99 MB −13 B
@uipath/apollo-wind 40.3% 395.2 KB 2.57 MB −1 B
@uipath/ap-chat 85.8% 43.43 MB 55.91 MB −3 B

"Coverage" is each package's own coverage.include scope (e.g. apollo-core instruments only scripts/). "Packed"/"Unpacked" come from npm pack --dry-run and only cover built packages — "—" means not measured this run (package not affected / not built). "vs main" is the packed (gzipped) delta against the last successful main build (the package-sizes artifact from the Release workflow); "—" there means no main baseline was available this run. The baseline is main's latest build, not this PR's exact merge-base, so it includes any drift since the branch diverged. Packages with no vitest config are omitted.

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

Labels

pkg:apollo-react size:S 10-29 changed lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants