feat(skill): add plan-first mode for manual edge routing - #338
Open
sanpatricky wants to merge 2 commits into
Open
feat(skill): add plan-first mode for manual edge routing#338sanpatricky wants to merge 2 commits into
sanpatricky wants to merge 2 commits into
Conversation
When validation reports edge overlaps requiring manual routing, the skill now follows a structured workflow: check port spread first, read route-mode docs, calculate spatial budget, then batch-implement coupled controls. Also adds a batch-planning exception to authoring-contract.md for edges sharing a spatial channel. Closes tt-a1i#336
…tial channels When multiple edges share a spatial channel, the 'one control at a time' rule causes cascading fixes. Add exception: plan all coupled controls (via, labelAt, channelY) as a batch using --layout-json evidence. Part of tt-a1i#336
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem and value
When the skill encounters edge overlaps requiring manual routing, the current instructions lead to iterative trial-and-error: guess coordinates → validate → fail → adjust → repeat. In a real-world workflow diagram with 15 edges, fixing a single overlap took 4+ iterations because the skill added manual
viarouting when simply removingroute: "drop"would have let port spread resolve it automatically.Fixes #336
Scope
archify/SKILL.mdstep 3: added plan-first mode workflow (steps a–e) triggered when validation reports geometry errors or visual inspection reveals edge overlaps. Key addition: prefer automatic over manual — check whether removing a spread-disabling control resolves the overlap before resorting to manualvia.archify/SKILL.mdline 46: added exception to docs-reading ban — when manual routing is needed, readingauthoring-contract.mdroute-preset and port-spread sections is allowed before attempting fixes.archify/references/authoring-contract.md§ Spacing and labels: added batch-planning exception to the "one control at a time" rule — when multiple edges share a spatial channel, plan all coupled controls as a batch using--layout-jsonevidence.Stability impact
Tests run
Not applicable — documentation-only change. No renderer, validator, or package code modified.
Visual evidence
Not applicable — this changes skill authoring instructions only, not rendered output.
Generated artifacts
None. No renderer source, schema, or example input changed.
Checklist
npm testinarchify/. — N/A: documentation-only change