Skip to content

refactor: converge electrochem runner helpers#17

Merged
Pluze merged 1 commit into
mainfrom
codex/electrochem-runner-convergence
Jun 6, 2026
Merged

refactor: converge electrochem runner helpers#17
Pluze merged 1 commit into
mainfrom
codex/electrochem-runner-convergence

Conversation

@Pluze

@Pluze Pluze commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary:

  • Route EIS GUI plot/summary paths through app-owned eis.view helpers and remove duplicate local runner helpers.
  • Remove the dead CSC local compute helper copy so the runner has one csc.ops.computeCSC path.
  • Add guardrails against +ui/runApp helper shadow copies and direct wearable family helper packages.
  • Document UI tool private helper contracts and shrink private contract debt from 73 to 62 files.
  • Record the future ECG Print app-owned package target without moving the runner.

Tests:

  • git diff --check
  • scripts/run_matlab_tests.sh testProject
  • scripts/run_matlab_tests.sh checkStyle
  • scripts/run_matlab_tests.sh testAppsElectrochem
  • scripts/run_matlab_tests.sh testAppsElectrochemGui
  • scripts/run_matlab_tests.sh test

Not changed:

  • No public app launch commands changed.
  • No +labkit public facade changes.
  • No buildtool task changes.
  • No DTA legacy bridge fields removed.
  • DIC and wearable private runners were not migrated.

Manual GUI:

  • Interactive GUI workflow feel and file-dialog workflows were not run; only automated noninteractive GUI structural checks were run.

@Pluze Pluze merged commit e3de76b into main Jun 6, 2026
7 checks passed
@Pluze Pluze deleted the codex/electrochem-runner-convergence branch June 6, 2026 00:56
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