@coldtea/pr-lens-agent-skill 0.1.3 → 0.1.5
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/SKILL.md +72 -24
- package/package.json +1 -1
package/SKILL.md
CHANGED
|
@@ -1,41 +1,89 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: pr-lens
|
|
3
|
-
description:
|
|
3
|
+
description: "WHAT: Draws a code change or part of a codebase as an animated architecture or data-flow diagram, on its own or in a pull request. WHEN: asked to diagram, visualise or explain a change or a system, or when a pull request should carry a diagram. KEYWORDS: PR Lens, diagram, architecture, data flow, visualise, visualize, pull request"
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# PR Lens
|
|
7
7
|
|
|
8
|
-
PR Lens
|
|
8
|
+
PR Lens draws code as visually rich animated diagrams. It can represent diffs, architecture, data flows, and more.
|
|
9
9
|
|
|
10
|
-
|
|
10
|
+
The diff or code is represented as one JSON document (lanes, nodes, edges, ordered flows) and it renders the JSON as an animated SVG
|
|
11
11
|
|
|
12
|
-
##
|
|
12
|
+
## Operating manual
|
|
13
|
+
|
|
14
|
+
1. **Read the diff.** When asked to represent a code change: `git diff --find-renames <base>...<head>`. The base is the merge base, not the tip of the base branch.
|
|
15
|
+
|
|
16
|
+
If not expressing a code diff, read the code to be visually represented
|
|
17
|
+
|
|
18
|
+
2. **Write the document** to `.pr-lens/graph.json`, following `references/graph-document.md`. `references/example.graph.json` is valid reference with three lanes, all four delta states, a hero edge, a seven-step flow, a nested drill-down tree. Read it before you write your first one. It is quicker than reading the reference.
|
|
19
|
+
|
|
20
|
+
3. **Validate, and fix**
|
|
13
21
|
|
|
14
|
-
1. **Read the diff.** `git diff --find-renames <base>...<head>`. The base is the merge base, not the tip of the base branch.
|
|
15
|
-
2. **Write the document** to `.pr-lens/graph.json`, following `references/graph-document.md`. That page is the whole shape: every field, every enum, every limit, and the four rules a JSON Schema cannot express. `references/example.graph.json` is a document that validates — three lanes, all four delta states, a hero edge, a seven-step flow, a nested drill-down tree. Read it before you write your first one. It is quicker than reading the reference.
|
|
16
|
-
3. **Validate, and fix what it names.**
|
|
17
22
|
```bash
|
|
18
23
|
npx @coldtea/pr-lens-cli validate .pr-lens/graph.json
|
|
19
24
|
```
|
|
20
|
-
|
|
25
|
+
|
|
26
|
+
Fix every failure and run it again. Do not render an invalid document; do not "work around" a failure by deleting the element it names.
|
|
27
|
+
|
|
21
28
|
4. **Render.**
|
|
29
|
+
|
|
22
30
|
```bash
|
|
23
|
-
npx @coldtea/pr-lens-cli render .pr-lens/graph.json
|
|
31
|
+
npx @coldtea/pr-lens-cli render .pr-lens/graph.json --theme dark
|
|
24
32
|
```
|
|
25
|
-
|
|
26
|
-
|
|
33
|
+
|
|
34
|
+
Render dark as the default theme unless explicitly requested. The SVGs, the manifest and `drawn.graph.json` land in `.pr-lens/`, which the CLI adds to the repository's .gitignore. Do not commit any of it. These files are rebuilt from the diff whenever anyone wants them again. Each SVG is named after its view, the theme and a content hash; `manifest.json` lists them by lens and view, so read the names from there or from the directory.
|
|
35
|
+
|
|
36
|
+
If the user asked for a diagram, an explanation or a picture of the architecture and nothing more, put it on a canvas and hand back the link:
|
|
37
|
+
|
|
38
|
+
```bash
|
|
39
|
+
npx @coldtea/pr-lens-cli canvas push
|
|
40
|
+
```
|
|
41
|
+
|
|
42
|
+
This pushes `.pr-lens/drawn.graph.json` and prints three links. Give the user the view link, `https://prlens.dev/c/{id}`: that is the diagram, full screen, every view on one page, and it opens without a login. The edit link, the one ending in `#w=…`, lets its holder push over the canvas, so leave it out of the reply unless they ask, and never paste it anywhere public. The embed link serves the top view as an SVG for a README.
|
|
43
|
+
|
|
44
|
+
Pushing the same file again updates the same canvas, so a follow-up such as "rename that node" or "add the queue" is: edit the document, validate, render, push. The link stays the same. If the push fails, say so and tell them where the SVGs are and which one is the top view.
|
|
45
|
+
|
|
46
|
+
5. **Attach, when there is a pull request to attach to.** That means the user asked you to open a PR, asked for a diagram on one that exists, or you are opening a PR as part of changes made. Otherwise skip this step.
|
|
47
|
+
|
|
48
|
+
GitHub CLI uploads the diagram with the pull request. Write the body with a Markdown image pointing at the local file, then pass the same path to `--attach`. `gh` rewrites the reference to the uploaded asset and keeps the alt text you wrote:
|
|
49
|
+
|
|
50
|
+
```markdown
|
|
51
|
+
Moves bulk sending off the per-recipient trigger and onto a batch endpoint.
|
|
52
|
+
|
|
53
|
+
|
|
54
|
+
```
|
|
55
|
+
|
|
56
|
+
```bash
|
|
57
|
+
gh pr create --title "Batch broadcast sends" --body-file .pr-lens/body.md \
|
|
58
|
+
--attach .pr-lens/overview-dark-4f9bd6c1.svg
|
|
59
|
+
```
|
|
60
|
+
|
|
61
|
+
On a pull request that already exists, `gh pr edit <number>` with the same two flags puts the diagram in the description, and `gh pr comment <number>` puts it in a comment. Repeat `--attach` for each diagram the body references.
|
|
62
|
+
|
|
63
|
+
gh has three rules:
|
|
64
|
+
- The reference has to be a Markdown image, ``. An HTML `<img>` or `<picture>` is left as written, and the file is appended at the bottom of the body instead.
|
|
65
|
+
- The alt text is the caption a reader without images gets. Say what the diagram shows, in one line.
|
|
66
|
+
- `--attach` arrived in GitHub CLI 2.99. Check with `gh --version` before you write a body around it.
|
|
67
|
+
|
|
68
|
+
Attach the views a reviewer needs and leave the rest in `.pr-lens/`: the top architecture view first, then a data flow if the change has a sequence worth following. A body with four diagrams reads worse than one with two, except the four are really needed to understand the change e.g., in the case of a complex feature or refactor.
|
|
69
|
+
|
|
70
|
+
When `--attach` is not an option, publish the SVGs somewhere durable and let the CLI compose the comment instead:
|
|
71
|
+
|
|
27
72
|
```bash
|
|
28
73
|
npx @coldtea/pr-lens-cli comment \
|
|
29
74
|
--graph .pr-lens/drawn.graph.json \
|
|
30
75
|
--manifest .pr-lens/manifest.json \
|
|
31
76
|
--asset-base-url https://raw.githubusercontent.com/<owner>/<repo>/<branch>/<dir>
|
|
32
77
|
```
|
|
33
|
-
The markdown goes to stdout, with each diagram as a `<picture>` pair so it reads in both GitHub themes. Posting it is your business.
|
|
34
78
|
|
|
35
|
-
|
|
79
|
+
`--graph` takes `drawn.graph.json`, not the document you wrote, because corrections change what the diagrams show and the CLI refuses a document its manifest does not describe. `--asset-base-url` is where you published the SVGs; leave it out and the markdown points at local paths no reader can fetch. The markdown goes to stdout, with each diagram as a `<picture>` pair; posting it is your business.
|
|
36
80
|
|
|
37
81
|
If you would rather not author the document yourself, `npx @coldtea/pr-lens-cli analyze --base <ref>` does steps 1 and 2 by asking a provider — Gemini, OpenAI, or any endpoint speaking `/chat/completions` — with a key of your own. That is the only path here that needs one.
|
|
38
82
|
|
|
83
|
+
## The pull request body, when there is one
|
|
84
|
+
|
|
85
|
+
A reviewer should understand the change before reading the diff, so the diagram goes where they look first: the description, not a trailing comment. Open with one sentence on why the change exists, then the architecture diagram, then whatever proves the change works, such as a screenshot of the result or a recording of the interaction. Use one visual per idea. A diagram that needs a paragraph of explanation has a document problem; go back to step 2.
|
|
86
|
+
|
|
39
87
|
## What makes a document worth reading
|
|
40
88
|
|
|
41
89
|
- **Include what did not change.** A diagram of only the changed nodes says nothing about blast radius. The unchanged neighbours a change touches are the context; mark them `delta: "unchanged"`.
|
|
@@ -57,12 +105,12 @@ Keep data-flow views as separate roots rather than nesting them in the architect
|
|
|
57
105
|
|
|
58
106
|
Read `references/graph-document.md` before writing. The four failures that account for nearly everything:
|
|
59
107
|
|
|
60
|
-
| Code
|
|
61
|
-
|
|
|
62
|
-
| `BROKEN_REFERENCE`
|
|
63
|
-
| `INVALID_DOCUMENT`
|
|
64
|
-
| `DUPLICATE_ID`
|
|
65
|
-
| `UNSUPPORTED_SCHEMA_VERSION` | `schemaVersion` is not the contract version installed
|
|
108
|
+
| Code | What you did |
|
|
109
|
+
| ---------------------------- | -------------------------------------------------------------------- |
|
|
110
|
+
| `BROKEN_REFERENCE` | an edge, a flow step or a view names an id you never declared |
|
|
111
|
+
| `INVALID_DOCUMENT` | an invented field; the schemas are strict, unknown keys are rejected |
|
|
112
|
+
| `DUPLICATE_ID` | two nodes, edges or views sharing an id |
|
|
113
|
+
| `UNSUPPORTED_SCHEMA_VERSION` | `schemaVersion` is not the contract version installed |
|
|
66
114
|
|
|
67
115
|
Four rules cannot be expressed in JSON Schema and are checked only by the parser, so structured output alone does not make a document valid: referential integrity, a line range that ends before it starts, a `self` message whose endpoints disagree, and a patch whose two commits are the same. Always validate.
|
|
68
116
|
|
|
@@ -93,10 +141,10 @@ A `match` beginning with `id:` addresses one node exactly; anything else is a pa
|
|
|
93
141
|
|
|
94
142
|
Everything you need is beside this page. Nothing here asks you to install a package first.
|
|
95
143
|
|
|
96
|
-
|
|
|
97
|
-
|
|
|
98
|
-
| `references/graph-document.md`
|
|
99
|
-
| `references/config.md`
|
|
100
|
-
| `references/example.graph.json` | one complete document that validates, to read and to copy the shape of
|
|
144
|
+
| | |
|
|
145
|
+
| ------------------------------- | ---------------------------------------------------------------------------------- |
|
|
146
|
+
| `references/graph-document.md` | the document, field by field: enums, limits, and where documents actually go wrong |
|
|
147
|
+
| `references/config.md` | `.github/pr-lens.yml`, the correction overlay, in full |
|
|
148
|
+
| `references/example.graph.json` | one complete document that validates, to read and to copy the shape of |
|
|
101
149
|
|
|
102
150
|
The same document ships as `postmark-refactor.graph.json` in `@coldtea/pr-lens-schema`, and the JSON Schema the validator enforces is published at `https://unpkg.com/@coldtea/pr-lens-schema/json-schema/graph-doc.schema.json`. Neither is something you need to fetch to write a document.
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@coldtea/pr-lens-agent-skill",
|
|
3
|
-
"version": "0.1.
|
|
3
|
+
"version": "0.1.5",
|
|
4
4
|
"description": "The PR Lens skill for coding agents: author a graph document from a diff, validate it, render it, and correct a repository's map.",
|
|
5
5
|
"license": "MIT",
|
|
6
6
|
"author": "Coldtea AI",
|