Skip to content

feat(DocumentQueries): Diagram.show names the optional notation elements a document diagram draws - #6

Merged
HuiJun merged 1 commit into
mainfrom
feature/document-queries-diagram-show
Oct 12, 2026
Merged

HuiJun merged 1 commit into
mainfrom
feature/document-queries-diagram-show

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Adds one optional attribute to DocumentQueries::Diagram, after ports:

/* show: the optional notation elements the diagram draws besides those
 * its view's rendering shows, comma-separated: "parameter-symbols",
 * "port-symbols", "compartments", "edge-labels", "keywords",
 * "multiplicities", "typings", "qualifiers", or "all". */
attribute show : String[0..1];

A document diagram's view rendering decides which notation elements are drawn by default; show lets a document ask for more of them (or "all") without changing the view. Like ports and overlay, it is read by the document plan compiler that consumes DocumentQueries::Diagram; the engine-contract.json entry for Diagram lists no attributes, so the manifest is unchanged.

Semver impact: MINOR (additive; contract stays at 1).

Checks: ./scripts/validate-libraries.sh (0 errors, 0 warnings), ./scripts/check-engine-contract.sh --base origin/main and ./scripts/check-readme-table.sh pass.

Independent of #5; OpenSysML re-pins its vendored copy to main once this merges.

Link to Devin session: https://nasa-jpl-demo.devinenterprise.com/sessions/1927cf08c2b9400b8a4c522dde258549
Open in Devin Desktop: https://nasa-jpl-demo.devinenterprise.com/desktop/session/1927cf08c2b9400b8a4c522dde258549?variant=devin
Requested by: @HuiJun

…nts a document diagram draws

show lists, comma-separated, the notation elements a document diagram
draws besides those its view's rendering shows: parameter-symbols,
port-symbols, compartments, edge-labels, keywords, multiplicities,
typings, qualifiers, or all.

Co-Authored-By: jason.han <hanhuijun@gmail.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration
devin-ai-integration Bot marked this pull request as ready for review October 12, 2026 03:14
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

The required job (Validate the extension libraries) is green. The advisory OpenSysML against these libraries job fails only internal/semantic/semantics TestKerMLFeatureMetaclassClassification, which reads examples/pilot-corpora/... that the job does not download; every other OpenSysML package in that run passes against these libraries, including tests/hygiene (the engine contract is unchanged).

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@HuiJun
HuiJun merged commit 73bed35 into main Oct 12, 2026
1 of 2 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.

1 participant