Skip to content

docs(skills): testing-sysml-repl covers v1 migration audits and inherited-state rendering - #1026

Merged
HuiJun merged 2 commits into
developfrom
devin/1791546128-skill-migration-rendering
Oct 9, 2026
Merged

HuiJun merged 2 commits into
developfrom
devin/1791546128-skill-migration-rendering

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

What and why

Adds a "Migration and state-rendering comparisons" section to .agents/skills/testing-sysml-repl/SKILL.md, distilled from testing #1025 against a real Cameo archive:

  • how to audit migrated flow direction and endpoints against the source XMI (informationSource/informationTarget, realizingConnector, end partWithPort/role, port types), and which negative controls to include;
  • that a lowering fix repairs old migrated notation too, so a before/after needs a separately built pre-change binary, not the same binary on two files;
  • -render '#state:M' -render-form text / -graphs M to establish a non-empty inherited state graph, since exit status and strict validation do not;
  • positioned migrated views: default DOT omits unpositioned nodes, -render-unplaced strip includes them — do not mistake this for failed lowering.

Skill-only change; no code, docs site, or changelog impact.

How it was verified

Each item was exercised during the #1025 test run (recording and evidence linked from that PR). Markdown renders; file otherwise unchanged.

Checklist

  • make test and make lint pass locally (no code touched)
  • Tests added or updated for the change — n/a
  • Documentation extended where it already covers the surface (see CONTRIBUTING.md)
  • Changelog entry added — n/a, agent skill only
  • baselines regenerated — n/a
  • No internal work-item labels (waves, slices, F4, K5) in the body, docs, or changelog

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

…tate rendering

Direction audits of migrated item flows against the source XMI, the
pre-change-binary negative control for lowering fixes, and the
positioned-view pitfalls (-render-unplaced strip, -graphs) that make an
empty state graph look like a placement issue or vice versa.

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

Co-Authored-By: jason.han <hanhuijun@gmail.com>
@HuiJun
HuiJun marked this pull request as ready for review October 9, 2026 12:06

@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 714c459 into develop Oct 9, 2026
21 checks passed
@HuiJun
HuiJun deleted the devin/1791546128-skill-migration-rendering branch October 9, 2026 12:33
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