Skip to content

Conversation

@igordayen
Copy link
Contributor

@igordayen igordayen commented Jan 7, 2026

Overview

Diagrams additions:
Sections: Planners, Agent Platform, Agent Process

Please refer to issue:
#40

Implementation

Diagrams got generated using Graphiz in ".dot"-format.

Syntax on inclusion into Asciidoc:

[graphviz, embabel_planning_system.dot, png]
----
include::../diagrams/architecture/embabel_planning_system.dot[]
----

Maven does include proper diagram plugin:


                            <groupId>org.asciidoctor</groupId>
                            <artifactId>asciidoctorj-diagram</artifactId>
                            <version>${asciidoctorj.diagram.version}</version>

We need to test whether this works properly in Github build.

In order to render diagrams in Intellij Kroki should be enabled:

image

Note on Rendering

if auto-rendering from .dot file is not working properly, will switch to 2-step process on generating svg / png from .dot file

section Planners, Agent Platform, Agent Process
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 7, 2026

@alexheifetz
Copy link
Contributor

Let me see if it's buildable on GitHub.

@alexheifetz
Copy link
Contributor

Looks like it's buildable on GitHub with our current doc build setup.

image

I will merge PR to get it moving forward.

@johnsonr, when get a chance please review arch diagrams for correctness
@jasperblues please review for look and feel compatibility

@alexheifetz alexheifetz merged commit beb9c4c into main Jan 8, 2026
14 checks passed
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.

3 participants