Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parse Error when rendering #47

Open
guicane opened this issue Aug 29, 2024 · 3 comments
Open

Parse Error when rendering #47

guicane opened this issue Aug 29, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@guicane
Copy link

guicane commented Aug 29, 2024

Describe the bug
Trying Terramaid for the first time and when seeing the preview in IntelliJ with the mermaid plugin installed I get a parse error. Relatively simple terraform configuration. Seems to fail when loading the google cloud terraform module.

Parse error on line 5:
...		subgraph Provider[\Registry.terraform
----------------------^
Expecting 'SEMI', 'NEWLINE', 'SPACE', 'EOF', 'GRAPH', 'DIR', 'subgraph', 'SQS', 'end', 'AMP', 'COLON', 'START_LINK', 'STYLE', 'LINKSTYLE', 'CLASSDEF', 'CLASS', 'CLICK', 'DOWN', 'UP', 'NUM', 'NODE_STRING', 'BRKT', 'MINUS', 'MULT', 'UNICODE_TEXT', got 'INVTRAPSTART'

To Reproduce
Preview the output file Terramaid.md in IntelliJ (with mermaid extension installed) throws the error.

Expected behavior
I should be able to see the graph.

Screenshots
Screenshot 2024-08-29 at 12 26 42

@RoseSecurity RoseSecurity self-assigned this Sep 4, 2024
@RoseSecurity RoseSecurity added bug Something isn't working enhancement New feature or request and removed enhancement New feature or request labels Sep 4, 2024
@RoseSecurity
Copy link
Owner

Thanks for reporting this @guicane. Is there any way you would be able to provide the markdown in this thread so I can copy it out and work on a fix?

@RoseSecurity
Copy link
Owner

@RoseSecurity
Copy link
Owner

I initially attempted a quick fix for this without success, but it is my next priority to get this working in a better way. I think it may require a major rewrite of how the logic converts DOT into Mermaid, but it is best for the longevity of the project to accomplish this in a cleaner and more sustainable way.

Thank you for your patience!

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
Status: Ready
Development

No branches or pull requests

2 participants