Skip to content

fix(statechart): route skill instructions through Pi entry - #44

Merged
jiangkoumo merged 1 commit into
mainfrom
fix/statechart-skill-entry
Oct 26, 2026
Merged

jiangkoumo merged 1 commit into
mainfrom
fix/statechart-skill-entry

Conversation

@jiangkoumo

Copy link
Copy Markdown
Owner

The generated statechart skill sent agents to the raw upstream CLI and promised automatic TLC/Java installation before the Pi guidance appeared. Generate the skill with explicit, reviewed replacements for the Pi entry and preinstalled tool policy, and put runnable from-spec/from-chart/render examples near the top. Keep upstream snapshots and all scripts unchanged.

Help keeps its upstream wording; the skill now explicitly maps every node statechart.mjs help example to the Pi entry and gives the Pi installation policy precedence. Record the replacements in port-map and the fidelity allowlist, without relaxing fidelity thresholds.

Validation:

  • Regression failed before the change; adapter suite now 4 pass / 0 fail.
  • npm test: exit 0, 38/38 checks.
  • check:generate: exit 0, 146 resources, no drift.
  • check:fidelity: exit 0, 4467/4752 = 94.0025% (previously 94.0446%).
  • entry.mjs --help: exit 0.
  • git diff --check: exit 0; no diff in upstream/ or skills/statechart/scripts/.

No npm publication.

Copilot AI balanced review requested due to automatic review settings October 26, 2026 06:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@jiangkoumo
jiangkoumo merged commit cfd2c85 into main Oct 26, 2026
9 checks passed
@jiangkoumo
jiangkoumo deleted the fix/statechart-skill-entry branch October 26, 2026 08:17
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.

2 participants