Skip to content

Harden draw.io CLI harness#86

Open
getmored-create wants to merge 1 commit intoHKUDS:mainfrom
getmored-create:codex/drawio-harness-hardening
Open

Harden draw.io CLI harness#86
getmored-create wants to merge 1 commit intoHKUDS:mainfrom
getmored-create:codex/drawio-harness-hardening

Conversation

@getmored-create
Copy link
Contributor

Summary

  • harden the draw.io harness for Windows-friendly CLI usage
  • add a module entry point and remove non-compliant export fallback behavior
  • add regression coverage and refresh TEST.md with the passing test run

Validation

  • python -m pip install -e .
  • python -m pytest cli_anything/drawio/tests/ -v --tb=no

@sehawq
Copy link
Collaborator

sehawq commented Mar 16, 2026

Thanks for the PR! There’s one thing to note:

render_or_save no longer contains a fallback function; it now only calls the render function, but the documentation still refers to it as a “backward-compatible wrapper.” If someone has relied on this fallback function, this is a breaking change. Please either update the documentation or note this as a breaking change in the notes.

A quick question: Do E2E tests skip smoothly when the draw.io CLI isn’t installed? If they don’t, CI/dev environments could break.

Thanks!

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.

2 participants