Skip to content

Convert existing Script Runner dialogs to Composition API#2828

Draft
EmilyRagan wants to merge 11 commits intomainfrom
begin-composition-api-transition
Draft

Convert existing Script Runner dialogs to Composition API#2828
EmilyRagan wants to merge 11 commits intomainfrom
begin-composition-api-transition

Conversation

@EmilyRagan
Copy link
Contributor

The is the beginning of the transition to Composition API, starting with Script Runner. Existing dialog components were very easy to convert using Claude. The diffs are unfortunately too great to sync up the lines, but the changes are straightforward:

  • Props have the same type information as before
  • Prop values are now accessed via props.<propName> within the script, instead of this.<propName>
  • Computed values are now accessed via <computedName>.value within the script, instead of this.<computedValue>
  • Use defineModel over explicit computed with emit where appropriate
  • Data values that are mutable are defined with ref, immutable values are just const
  • Data values are accessed and updated via <dataName>.value within the script, instead of this.<dataName>
  • Methods are now accessed just by their name, without this.
  • created lifecycle hook is replaced by onMounted for some cases, immediate initialization in other cases
  • Promise chaining was updated to use async/await

@EmilyRagan EmilyRagan self-assigned this Feb 12, 2026
@codecov
Copy link

codecov bot commented Feb 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.23%. Comparing base (02086f8) to head (1c673bb).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2828      +/-   ##
==========================================
- Coverage   78.25%   78.23%   -0.02%     
==========================================
  Files         673      673              
  Lines       55191    55191              
  Branches      728      728              
==========================================
- Hits        43190    43181       -9     
- Misses      11923    11932       +9     
  Partials       78       78              
Flag Coverage Δ
python 79.33% <ø> (-0.01%) ⬇️
ruby-api 79.52% <ø> (-0.17%) ⬇️
ruby-backend 82.17% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@EmilyRagan EmilyRagan force-pushed the begin-composition-api-transition branch from d1687ef to 8976a4c Compare February 12, 2026 23:26
@OpenC3 OpenC3 locked and limited conversation to collaborators Feb 13, 2026
@OpenC3 OpenC3 unlocked this conversation Feb 13, 2026
@EmilyRagan EmilyRagan marked this pull request as draft February 13, 2026 15:42
@EmilyRagan EmilyRagan force-pushed the begin-composition-api-transition branch from 8976a4c to bdd1bc7 Compare February 13, 2026 18:50
@EmilyRagan EmilyRagan force-pushed the begin-composition-api-transition branch from bdd1bc7 to f980aab Compare February 24, 2026 20:34
@EmilyRagan EmilyRagan force-pushed the begin-composition-api-transition branch from f980aab to 1c673bb Compare March 3, 2026 16:41
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
3 New issues

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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.

1 participant