Skip to content

refactor: clean labkit ui api surface#19

Merged
Pluze merged 5 commits into
mainfrom
codex/labkit-ui-api-cleanup
Jun 6, 2026
Merged

refactor: clean labkit ui api surface#19
Pluze merged 5 commits into
mainfrom
codex/labkit-ui-api-cleanup

Conversation

@Pluze

@Pluze Pluze commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Scope

Clean the migration-era LabKit UI API surface in one branch with phased commits:

  • remove createShell shellOptions and panel/draw/update aliases
  • remove plotOptions and topBottomPlotControls from labkit.ui.view.panel
  • move app-specific top/bottom plot controls and right-side axes pairs into app-owned UI packages
  • remove createShell dualPlot/rightKind/showPlotControls/title specialization
  • require structured labkit.ui.view.form specs and remove raw args/shorthand passthrough

Validation

  • git diff --check
  • scripts/run_matlab_tests.sh testLabkitUiGui
  • scripts/run_matlab_tests.sh testAppsElectrochemGui
  • scripts/run_matlab_tests.sh testAppsDicGui
  • scripts/run_matlab_tests.sh testAppsImageMeasurementGui
  • scripts/run_matlab_tests.sh testAppsGui
  • scripts/run_matlab_tests.sh testProject

Notes

  • Kept app-specific Plot Options sections/local option structs; only the LabKit panel facade was removed.
  • Kept form kind info as a structured spec because it has row+label layout semantics distinct from readonly.
  • Interactive GUI workflow feel was not manually verified; automated GUI coverage is structural.

@Pluze Pluze merged commit 0f428eb into main Jun 6, 2026
7 checks passed
@Pluze Pluze deleted the codex/labkit-ui-api-cleanup branch June 6, 2026 18:53
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