Skip to content

fix: CI build order + feat: shape properties support - #58

Merged
rajnavakoti merged 2 commits into
masterfrom
fix/ci-and-properties-support
Apr 1, 2026
Merged

fix: CI build order + feat: shape properties support#58
rajnavakoti merged 2 commits into
masterfrom
fix/ci-and-properties-support

Conversation

@rajnavakoti

Copy link
Copy Markdown
Contributor

Summary

  • CI fix: moved build step before test — cli.test.ts requires dist/cli.js
  • Properties support: shapes can define custom key-value properties (owner, status, criticality, etc.) that appear in draw.io's Edit Data dialog
    • YAML schema updated with properties array
    • Generator wraps shapes with properties in <object> elements
    • Extract command captures custom attributes from .drawio files
    • Example properties on application-component and business-process

Test plan

  • npm test — 415 tests passing
  • npm run validate — all 60 YAML files valid with new schema
  • Generated XML verified: application-component has owner, status, criticality attributes

🤖 Generated with Claude Code

rajnavakoti and others added 2 commits April 1, 2026 20:51
The CLI integration tests require dist/cli.js to exist.
Moved build step before test step in the CI pipeline.

Co-Authored-By: Claude <noreply@anthropic.com>
Shapes can now define custom properties (key-value metadata) in YAML.
Properties are embedded as <object> attributes in generated library XML,
appearing in draw.io's Edit Data dialog when shapes are placed on canvas.

- YAML schema: properties array with key, label, type, default, options
- Generator: uses <object> wrapper when properties exist
- Extract: captures custom attributes from .drawio files
- Example properties on application-component (owner, status, criticality)
  and business-process (processOwner, automationLevel, maturity)

Co-Authored-By: Claude <noreply@anthropic.com>
@rajnavakoti
rajnavakoti merged commit 5fbbc4f into master Apr 1, 2026
@rajnavakoti
rajnavakoti deleted the fix/ci-and-properties-support branch April 1, 2026 18:54
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