claude-dev-env 8.96.0 → 8.96.2

package/.agents/skills/pr-visual/SKILL.md

@@ -11,7 +11,7 @@ Turn a PR into a visual explanation of what its code changes. Return the display
 
 Accept a PR URL. Resolve a bare PR number only when the repository is known.
 
-Accept an issue URL for a status catch-up on an epic or tracker. The collector captures the issue, its sub-issues two levels deep, and the pull requests their bodies link. Draw that scene in this same viewer, batched with the scenes of the pull requests it names. Every pr-visual request renders through `assets/viewer.html`; a status report never moves to another page template. A tracker scene always draws every card inside its state chart. Set `board` in the scene: `lanes` run top to bottom in flow order, and each lane names one scene node. A lane with `progress` (`no_pull`, `open_pull`, `merged_pull`) takes every open tracker card in that state, grouped under its parent epic. A lane with `groups` takes cards that each cite a captured record and link inside the repository, such as done, skipped, or follow-up work listed in an epic body. Every node sits in exactly one lane. `strip` drops title prefixes, `titles` gives plain wording by issue number, and `exclude` leaves out issues that are not work. `steps` names the issue body sections each card shows as its own state diagram, such as `Today` and `Fix`, with a label and colour; the step marked `inline` also prints on the card. `wording` swaps words or whole sentences in that text for plain ones. A cited card carries its `steps` itself. Clicking a card opens its diagram in the side panel, with a link to the card on GitHub. `verify.py` checks the card count, that each title fits its card, that the first card in each lane opens a diagram with one box per step, and that board and diagram text is 16px or larger at 7:1 contrast.
+Accept an issue URL for a status catch-up on an epic or tracker. The collector captures the issue, its sub-issues two levels deep, and the pull requests their bodies link. Draw that scene in this same viewer, batched with the scenes of the pull requests it names. Every pr-visual request renders through `assets/viewer.html`; a status report never moves to another page template. A tracker scene always draws every card inside its state chart. Set `board` in the scene: `lanes` run top to bottom in flow order, and each lane names one scene node. A lane with `progress` (`no_pull`, `open_pull`, `merged_pull`) takes every open tracker card in that state, grouped under its parent epic. A lane with `groups` takes cards that each cite a captured record and link inside the repository, such as done, skipped, or follow-up work listed in an epic body. Every node sits in exactly one lane. `strip` drops title prefixes, `titles` gives plain wording by issue number, and `exclude` leaves out issues that are not work. `steps` names the issue body sections each card shows as its own state diagram, such as `Today` and `Fix`, with a label and colour; the step marked `inline` also prints on the card. `wording` swaps words or whole sentences in that text for plain ones. A cited card carries its `steps` itself. `render.py` stops when a card has no step of its own: each card needs one step whose text is not its title or note and appears on no other card, so a board of title and link chips never renders. Clicking a card opens its diagram in the side panel, with a link to the card on GitHub. `verify.py` checks the card count, that each title fits its card, that the first card in each lane opens a diagram with one box per step that shows each step's text, and that board and diagram text is 16px or larger at 7:1 contrast.
 
 Read the description, changed code, relevant callers, and tests. Build an interactive diagram with short labels, visual drilldowns, source details, and a summary-to-code comparison.
 

package/.agents/skills/pr-visual/assets/viewer.html

@@ -132,7 +132,7 @@ function cardFlow(steps){const width=396,pad=14,step=22,flow=el('svg',{class:'ca
   flow.append(el('rect',{x:2,y,width:width-4,height,rx:12,class:`card-step ${v.kind||''}`}));text(flow,pad+2,y+pad+15,v.label,'card-step-label');lines(flow,pad+2,y+pad+15+step,body,'card-step-text','start',step);y+=height+34});
  flow.setAttribute('viewBox',`0 0 ${width} ${y-30}`);return flow}
 function showCard(card,lane,trigger){const s=scenes[active],state=s.nodes.find(n=>n.id===lane.node);lastFocus=trigger;resetSelection();content.replaceChildren();$('[data-detail-title]').textContent=card.title;
- content.append(dom('p',[state.label.join(' '),card.note].filter(Boolean).join(' \u00b7 '),'card-state'),cardFlow(card.steps?.length?card.steps:[{label:'Card',text:card.title}]));
+ content.append(dom('p',[state.label.join(' '),card.note].filter(Boolean).join(' \u00b7 '),'card-state'),cardFlow(card.steps));
  const a=dom('a',card.number?`Open issue ${card.number} on GitHub`:'Open it on GitHub','card-link');a.href=card.url;a.target='_blank';a.rel='noopener noreferrer';content.append(a);
  content.scrollTop=0;drawer.hidden=false;workspace.classList.add('open');$('.close').focus({preventScroll:true})}
 function chartNode(svg,n,x,y,bar){

package/.agents/skills/pr-visual/fixtures/tracker/head/issues/2.md

@@ -1 +1,10 @@
 Synthetic task with a linked fix: https://github.com/example/project/pull/3
+
+## Today
+The synthetic fix merged, and its task stays open.
+
+## Fix
+Close the synthetic task.
+
+## Proof
+The synthetic task reads closed.

package/.agents/skills/pr-visual/fixtures/tracker.scene.json

@@ -80,7 +80,7 @@
       {"heading": "Proof", "label": "Proof", "kind": "green"}
     ],
     "lanes": [
-      {"node": "epic", "groups": [{"title": "Follow-up branches", "cards": [{"title": "Task one follow-up", "source": {"path": "issues/1.md", "line": 5}}]}]},
+      {"node": "epic", "groups": [{"title": "Follow-up branches", "cards": [{"title": "Task one follow-up", "source": {"path": "issues/1.md", "line": 5}, "steps": [{"label": "Today", "kind": "orange", "text": "Task one waits for a follow-up branch."}]}]}]},
       {"node": "left", "progress": "no_pull"},
       {"node": "fixed", "progress": "merged_pull"}
     ]

package/.agents/skills/pr-visual/scripts/render.py

@@ -4,6 +4,7 @@ import copy
 import itertools
 import json
 import re
+from collections import Counter
 from pathlib import Path
 from urllib.parse import quote
 
@@ -166,6 +167,38 @@ def _prepare_board(scene: dict, snapshot: dict, snapshot_path: Path) -> None:
         raise RenderRunFatal("Every board node needs exactly one lane.")
     all_lanes = [_board_lane(each_lane, tracker, settings, snapshot, snapshot_path) for each_lane in settings["lanes"]]
     scene["board"] = {"lanes": all_lanes, "count": sum(each_lane["count"] for each_lane in all_lanes)}
+    require_own_card_content(scene["board"])
+
+
+def require_own_card_content(board: dict) -> None:
+    """Refuse a board whose cards say nothing of their own.
+
+    A card passes when one of its steps carries text that is not its title
+    or note and that no other card on the board repeats, and every step
+    carries text. A card holding only a title, a link, text another card
+    repeats, or a step with no text fails.
+
+    Args:
+        board: Prepared board with ``lanes`` of ``groups`` of ``cards``.
+
+    Raises:
+        RenderRunFatal: Names each card with no content of its own.
+    """
+    all_cards = [each_card for each_lane in board["lanes"] for each_group in each_lane["groups"] for each_card in each_group["cards"]]
+    text_counts = Counter(each_text for each_card in all_cards for each_text in _step_texts(each_card))
+    all_empty_titles = [each_card["title"] for each_card in all_cards if not _has_own_step(each_card, text_counts)]
+    if all_empty_titles:
+        raise RenderRunFatal(c.EMPTY_CARD_MESSAGE.format(count=len(all_empty_titles), total=len(all_cards), titles=c.CARD_TITLE_SEPARATOR.join(all_empty_titles[:c.EMPTY_CARD_TITLE_LIMIT])))
+
+
+def _step_texts(card: dict) -> set[str]:
+    return {each_step.get("text", "").strip() for each_step in card.get("steps", [])}
+
+
+def _has_own_step(card: dict, text_counts: Counter) -> bool:
+    all_borrowed_text = {card["title"].strip(), card.get("note", "").strip(), ""}
+    is_every_step_written = all(each_step.get("text") for each_step in card.get("steps", []))
+    return is_every_step_written and any(text_counts[each_text] == 1 and each_text not in all_borrowed_text for each_text in _step_texts(card))
 
 
 def _prepare_sources(scene: dict, snapshot: dict, snapshot_path: Path) -> None:

package/.agents/skills/pr-visual/scripts/test_render.py

@@ -10,7 +10,7 @@ import pytest
 SCRIPTS_DIRECTORY = Path(__file__).resolve().parent
 sys.path.insert(0, str(SCRIPTS_DIRECTORY))
 from metrics import MetricRunFatal, prepare_chart
-from render import RenderRunFatal, _source_evidence, prepare, render
+from render import RenderRunFatal, _source_evidence, prepare, render, require_own_card_content
 
 FIXTURES = Path(__file__).resolve().parents[1] / "fixtures"
 FORMATS = ("flow", "bar", "line", "scatter")
@@ -115,7 +115,7 @@ def test_tracker_scene_places_every_card_in_its_state_lane(tmp_path: Path) -> No
         {"label": "Build", "kind": "lime", "inline": True, "text": "Build the sample fix and link it."},
         {"label": "Proof", "kind": "green", "text": "A sample test shows the fix."},
     ]
-    assert fixed["groups"][0]["cards"][0]["steps"] == []
+    assert fixed["groups"][0]["cards"][0]["steps"][0] == {"label": "Today", "kind": "orange", "text": "The sample fix merged, and its task stays open."}
     assert "Task one ships" not in (tmp_path / "visual.html").read_text()
 
 
@@ -137,6 +137,66 @@ def test_tracker_cited_card_rejects_a_link_outside_the_repository(tmp_path: Path
         _prepare_copy(tmp_path, scene)
 
 
+def test_tracker_board_rejects_cards_holding_only_a_title_and_link(tmp_path: Path) -> None:
+    """A board of title and link chips stops the render."""
+    scene = _board_scene(tmp_path)
+    scene["board"]["steps"] = []
+    scene["board"]["lanes"][0]["groups"][0]["cards"][0].pop("steps")
+    with pytest.raises(RenderRunFatal, match="3 of 3 board cards carry no content of their own"):
+        _prepare_copy(tmp_path, scene)
+
+
+def test_tracker_board_rejects_a_card_whose_text_another_card_repeats(tmp_path: Path) -> None:
+    """Text another card repeats is not a card's own content."""
+    scene = _board_scene(tmp_path)
+    scene["board"]["lanes"][0]["groups"][0]["cards"][0]["steps"] = [{"label": "Today", "text": "The sample task has no fix yet."}]
+    with pytest.raises(RenderRunFatal, match="1 of 3 board cards .*Task one follow-up"):
+        _prepare_copy(tmp_path, scene)
+
+
+def test_tracker_board_rejects_a_step_that_repeats_the_title(tmp_path: Path) -> None:
+    """A step that only restates the card title is not content."""
+    scene = _board_scene(tmp_path)
+    scene["board"]["lanes"][0]["groups"][0]["cards"][0]["steps"] = [{"label": "Card", "text": "Task one follow-up"}]
+    with pytest.raises(RenderRunFatal, match="Task one follow-up"):
+        _prepare_copy(tmp_path, scene)
+
+
+def test_own_card_content_accepts_cards_that_share_only_a_templated_step() -> None:
+    """A shared step is allowed when each card also carries a step of its own."""
+    shared_step = {"label": "Your pick", "text": "Skip. Nothing gets built."}
+    all_cards = [{"title": f"Card {each_index}", "note": "", "steps": [{"label": "Today", "text": f"Card {each_index} has its own gap."}, shared_step]} for each_index in range(2)]
+    assert require_own_card_content({"lanes": [{"groups": [{"cards": all_cards}]}]}) is None
+
+
+def test_own_card_content_names_the_card_with_only_shared_steps() -> None:
+    """The refusal names the card whose every step another card repeats."""
+    shared_step = {"label": "Your pick", "text": "Skip. Nothing gets built."}
+    all_cards = [{"title": "Own card", "steps": [{"label": "Today", "text": "Own gap."}, shared_step]}, {"title": "Templated card", "steps": [shared_step]}]
+    with pytest.raises(RenderRunFatal, match="1 of 2 board cards .*: Templated card$"):
+        require_own_card_content({"lanes": [{"groups": [{"cards": all_cards}]}]})
+
+
+def test_own_card_content_accepts_text_a_card_repeats_only_within_itself() -> None:
+    """Two steps of one card that share text still count as that card's own."""
+    all_cards = [{"title": "Repeating card", "steps": [{"label": "Today", "text": "See the gap."}, {"label": "Fix", "text": "See the gap."}]}]
+    assert require_own_card_content({"lanes": [{"groups": [{"cards": all_cards}]}]}) is None
+
+
+def test_own_card_content_refuses_steps_without_text() -> None:
+    """A step holding only a label is no content."""
+    all_cards = [{"title": "Label card", "steps": [{"label": "Today"}]}]
+    with pytest.raises(RenderRunFatal, match="Label card"):
+        require_own_card_content({"lanes": [{"groups": [{"cards": all_cards}]}]})
+
+
+def test_own_card_content_refuses_a_card_with_one_textless_step() -> None:
+    """A step the viewer would draw with no text stops the render, even beside an own step."""
+    all_cards = [{"title": "Half card", "steps": [{"label": "Today", "text": "Own gap."}, {"label": "Fix"}]}]
+    with pytest.raises(RenderRunFatal, match="Half card"):
+        require_own_card_content({"lanes": [{"groups": [{"cards": all_cards}]}]})
+
+
 def test_tracker_board_rejects_a_node_with_no_lane(tmp_path: Path) -> None:
     """Every state node on a board sits in exactly one lane."""
     scene = _board_scene(tmp_path)

package/.agents/skills/pr-visual/scripts/verify.py

@@ -66,7 +66,8 @@ def _check_board(root: Locator, scene: dict) -> None:
         card = each_lane["groups"][0]["cards"][0]
         all_cards.nth(first_card).click()
         assert root.locator(c.DETAIL_TITLE_SELECTOR).inner_text() == card["title"]
-        assert root.locator(c.CARD_STEP_SELECTOR).count() == max(1, len(card.get("steps", [])))
+        assert root.locator(c.CARD_STEP_SELECTOR).count() == len(card["steps"])
+        assert all(each_step["text"] in c.SPACE_SEPARATOR.join(root.locator(c.CARD_FLOW_TEXT_SELECTOR).all_text_contents()) for each_step in card["steps"])
         _check_board_text(root, c.CARD_FLOW_TEXT_SELECTOR)
         root.get_by_role("button", name=c.CLOSE_BUTTON_LABEL).click()
         assert root.locator(c.DRAWER_SELECTOR).is_hidden()

package/.agents/skills/pr-visual/scripts/visual_config/config/constants.py

@@ -35,6 +35,9 @@ ALL_PULL_PROGRESS = ("merged_pull", "open_pull", "no_pull")
 ALL_CARD_NOTES = {"no_pull": "", "open_pull": "Open in pull request {pulls}", "merged_pull": "Merged in pull request {pulls}"}
 PULL_REFERENCE = "{number}"
 PULL_REFERENCE_SEPARATOR = ", "
+EMPTY_CARD_MESSAGE = "{count} of {total} board cards carry no content of their own. Give each card a step whose text is not its title or note and no other card repeats: {titles}"
+EMPTY_CARD_TITLE_LIMIT = 5
+CARD_TITLE_SEPARATOR = ", "
 ALL_PULL_FIELDS =("number", "title", "state", "draft", "merged_at", "closed_at", "updated_at", "html_url")
 COMPARE_ENDPOINT = "repos/{repo}/compare/{base}...{head}"
 FILES_ENDPOINT = "repos/{repo}/pulls/{number}/files?per_page={size}&page={page}"

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-dev-env",
-    "version": "8.96.0",
+    "version": "8.96.2",
     "description": "Claude Code development standards — rules, hooks, agents, commands, and skills",
     "type": "module",
     "bin": {