Skip to content

fix(cli): initialize DB for workspace run-step#106

Merged
Emin017 merged 7 commits into
mainfrom
emin/fix-run-step-db-init
Jun 4, 2026
Merged

fix(cli): initialize DB for workspace run-step#106
Emin017 merged 7 commits into
mainfrom
emin/fix-run-step-db-init

Conversation

@Emin017

@Emin017 Emin017 commented Jun 3, 2026

Copy link
Copy Markdown
Member

Summary

Fixes direct ecc workspace run-step execution so pending or rerun steps initialize the ECC DB wrapper before invoking the step runner, while preserving existing run-flow, unknown-step, and
already-successful skip behavior.

This also keeps workspace --json stdout clean when runtime code redirects process stdio to step logs, and aligns the Nix-packaged thirdparty tool pins with the current alpha.4 sources.

Changes

  • Initialize the DB engine for real workspace run-step execution without changing EngineFlow.run_step().
  • Avoid DB initialization when a non-rerun step is already Success.
  • Keep runtime log output out of JSON stdout for workspace commands.
  • Keep programmatic cli_main.run() stdio restoration behavior separate from installed CLI process behavior.
  • Update Nix packaging for ecc-dreamplace and ecc-tools alpha.4.

Verification

  • ruff passed
  • focused CLI/formal tests passed
  • full pytest passed: 823 passed, 9 skipped, 1 deselected, 13 xfailed
  • nix build .#cli passed
  • packaged CLI smoke passed for workspace create, run-step Synthesis, and run-step Floorplan
  • codex review --base main reported no introduced bugs

@Emin017 Emin017 added the bug Something isn't working label Jun 4, 2026
@Emin017 Emin017 changed the title fix(cli): add db init for run_step fix(cli): initialize DB for workspace run-step Jun 4, 2026
@Emin017 Emin017 added this to the 0.1.0-Alpha milestone Jun 4, 2026
@Emin017 Emin017 merged commit e2515dc into main Jun 4, 2026
3 checks passed
@Emin017 Emin017 deleted the emin/fix-run-step-db-init branch June 4, 2026 01:29
Emin017 added a commit to openecos-projects/ecos-studio that referenced this pull request Jun 4, 2026
Bump ecc to include fix(cli): initialize DB for workspace run-step
(openecos-projects/ecc#106)

Signed-off-by: Emin <me@emin.chat>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant