Uh oh!
There was an error while loading. Please reload this page.
docs(design): cmk:visualize, draft plus repo-map specification - #20
Open
harrymove-ctrl wants to merge 2 commits into
Open
docs(design): cmk:visualize, draft plus repo-map specification#20harrymove-ctrl wants to merge 2 commits into
harrymove-ctrl wants to merge 2 commits into
Conversation
Specifies a skill that renders content someone already has (a design doc, a tracker query, an inline description) into a reviewable visual, through a Source Resolver / Form Selector / Renderer pipeline. Status is draft: trigger phrases, the tracker-security confirmation flow, and an upstream requirements doc are still open points.
Adds the repository's own code as a fourth source kind and pins the scene-graph contract that everything else hangs off: the model emits one validated JSON document, fixed renderers draw it. Citations are a schema rule, so an uncited node or edge cannot reach a picture. Covers altitude by node budget with a named-subsystem override and eager drill-down, four independent form axes, secret redaction at sample time, and the twelve-asset website playground. System Architecture is the only diagram type in v1. Process Flow and AI / Data Flow are designed for and deferred; the broader diagram-generator surface is recorded in a later checklist rather than designed here.
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 freeto 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.
Two commits. The first adds
docs/design/visualize.md, the feature-level draft for a newcmk:visualizeskill. The second addsdocs/design/visualize-repo-map.md, which specifies one material type in depth: a map of a repository's own architecture, traced from its real code.The skill
Renders content someone already has, a design doc, a tracker query, or a description given inline, into something reviewable. It renders, it does not invent. Deciding what to build stays with
cmk:design,cmk:requirements, andcmk:adr.Pipeline:
Source Resolver→Form Selector→Renderer.What the repo-map doc adds
docs/ai/only as a routing hint and gathers a freshfile:linefor every node and edge in the run.Scope calls
System Architecture is the only diagram type in v1. Process Flow and AI / Data Flow share the analysis pass and slot in later without rework. The broader diagram-generator surface (IoT scenes, travel routes, teaching diagrams, product structure) is recorded in a later checklist with a note that it cannot reuse the analysis pass and would likely be a separate skill.
Consequences worth flagging to a reviewer
skills/visualize/assets/and must be vanilla, offline, and build-free, becausecmk:agent-vendorsforbids a package referencing anything outside itself by relative traversal. Compatible withcmk:syncas-is, sincecontent_hashalready covers all files in a package.Status: draft
Open points recorded in the docs: trigger phrases, the drill-down depth cap, node-budget calibration against real repos, and the absence of any upstream requirements doc.