Updated Visualizer to use GraphViz #144
Conversation
aidanm3341
requested review from
lbulanti-ms,
willosborne and
a team
as code owners
| @@ -25,7 +25,9 @@ | |||
| "ajv": "^8.12.0", | |||
| "commander": "^12.0.0", | |||
There was a problem hiding this comment.
Are we removing Mermaid now that it's no longer needed? That should remove the dependency on Playwright/puppeteer
There was a problem hiding this comment.
Ah yeah, I'll do a follow up PR to remove all the old visualizer code once this is all merged in and done
There was a problem hiding this comment.
This should definitely go into a shared module eventually. Other projects should be re-using this
There was a problem hiding this comment.
agreed. I'll just move it now so it's done
| } | ||
|
|
||
| function addRelationship(g: Digraph, relationship: CALMRelationship): void { | ||
| if ('connects' in relationship['relationship-type']) { |
There was a problem hiding this comment.
Could we potentially add methods to work on relationship types such as .getRelationshipType now that we have a model for it?
There was a problem hiding this comment.
Just a thought, this works fine
In order to get visualisation working on the CLI without the use of a headless browser, I have switched to using GraphViz.
I've also moved the visualisation code into the CLI folder directly to facilitate publishing it to NPM.