claude-dev-env 8.56.2 → 8.58.0

package/.agents/skills/correction/SKILL.md

@@ -0,0 +1,25 @@
+---
+name: correction
+description: Turn a correction the user just made into a short handoff brief they paste into a corrections intake. Use when the user types /correction followed by what the agent got wrong.
+argument-hint: "<what the agent got wrong, in the user's words>"
+---
+
+# /correction
+
+The user corrected an agent and wants the fix to stick. Write one brief they can paste into the place that turns corrections into lasting fixes.
+
+`$ARGUMENTS` is the user's statement of the correction. Fill the other fields from this session: the transcript, the files and pull requests it touched, and the messages it sent. When the session does not settle a field, write `unknown` there.
+
+Print the brief as one fenced `text` block, and nothing after it:
+
+```text
+Correction: <$ARGUMENTS, word for word>
+Asked: <what the user asked for, quoted where a message carries it>
+Agent did: <what the agent did, with the reply, file, or command that shows it>
+Corrected to: <what the user wanted instead, or the fix already made>
+Where: <repository, branch or pull request, files, tool, or thread>
+Evidence: <file:line, log line, or quoted message that shows the miss>
+Repeat: <yes, with the earlier time it happened | no | unknown>
+```
+
+Keep each field to one or two lines. Leave out the layer and the fix design; the intake decides those.

package/bin/ever-shipped-skills.mjs

@@ -21,6 +21,7 @@ export const EVER_SHIPPED_SKILL_NAMES = new Set([
     'context7-mcp',
     'copilot-finding-triage',
     'copilot-review',
+    'correction',
     'deep-research',
     'descriptions',
     'e-code-review',

package/bin/ever-shipped-skills.test.mjs

@@ -41,6 +41,10 @@ test('EVER_SHIPPED_SKILL_NAMES includes the pull request lifecycle skill', () =>
     assert.equal(EVER_SHIPPED_SKILL_NAMES.has('pr-lifecycle'), true);
 });
 
+test('EVER_SHIPPED_SKILL_NAMES includes the correction skill', () => {
+    assert.equal(EVER_SHIPPED_SKILL_NAMES.has('correction'), true);
+});
+
 test('EVER_SHIPPED_SKILL_NAMES includes the windows scheduled task skill', () => {
     assert.equal(EVER_SHIPPED_SKILL_NAMES.has('windows-scheduled-task'), true);
 });

package/docs/templates/artifact-page/README.md

@@ -16,6 +16,7 @@
 | Readable text | Text is 16px or larger and uses the full text color. Gray stays on lines and borders only. Text holds a 7:1 contrast ratio against its background. |
 | Touch targets | Buttons and chips are 40px or taller, with a visible focus ring. |
 | Copy fallback | The copy button uses the clipboard API and falls back to a hidden text area, then says when the copy is blocked. |
+| Review rounds | Only the parts changed since the viewer's last look carry a thin bar and a "Changed" label. [review-rounds.md](review-rounds.md) holds the data fields and behavior. |
 
 ## House rules for a page built from it
 
@@ -23,3 +24,13 @@
 - A page that needs extra files publishes them beside the page through the Artifact `files` field, and links them by relative path.
 - A page that needs live data or saved answers declares the matching runtime capability. `localStorage` holds only per-viewer conveniences.
 - Render the page at 400px in light and dark, and exercise every control, before publishing.
+
+## Convert an existing page
+
+Ask an agent: "convert <artifact link> to our format."
+
+1. Read the published page with the Artifact tool's `read` action, and keep its data and every working control.
+2. Map each part to its piece in `template.html`: status groups to board lanes, records to picture cards, filters to tabs and chips, and the data to the JSON data block.
+3. Apply the house rules above and the template's color tokens.
+4. Check that text is 16px or larger, text contrast is 7:1 or better, and the page has no horizontal scroll at 390px, in light and dark.
+5. Publish to the same link as a new version, so the reviewer keeps one URL.

package/docs/templates/artifact-page/review-rounds.md

@@ -0,0 +1,26 @@
+# Marking what changed between review rounds
+
+A page that a reviewer reads more than once shows what changed since their last look. The template marks only the parts that changed: each one gets a thin bar on its left edge and a small "Changed" label at its start. The card around them stays as it is, and its board chip gets a dot in the same color. There is no diff and no list of changes.
+
+## Data
+
+| Field | Where | Meaning |
+|---|---|---|
+| `round` | top of the data block | The review round this revision answers. Add one each time the page is republished after feedback. |
+| `changed_round` | an item | The round in which this item last changed. |
+| `changed_parts` | an item | The parts that changed: `title`, `next`, `summary`, `steps`, `links`. An item with no list gets no mark. |
+
+When a revision answers feedback, set `round` to the new number and stamp each item it changed with that number and the parts it touched. Leave older stamps in place; they stop showing once the reviewer has seen their round.
+
+## Behavior
+
+- The page keeps the last round each viewer saw in `localStorage`. Parts stamped after that round are marked.
+- A first visit, or a browser that blocks storage, marks the parts stamped in the current round.
+- The marks hold through a reload in the same tab, through `sessionStorage`, and clear on the next visit.
+- A changed summary opens by itself.
+
+## Other pages
+
+Stamp the smallest element that changed with `data-changed-round="N"`; it takes the bar and the "Changed" label, and a `<details>` element takes the label in its summary. Never stamp a whole card or section. An element with `data-change-mark="dot"` takes the dot and a screen reader label in place of the bar and label.
+
+The `--change` and `--change-ink` tokens hold the mark colors in light and dark. Keep the label text at 7:1 or better against `--change`, and the bar at 3:1 or better against `--surface` and `--bg`.

package/docs/templates/artifact-page/template.html

@@ -7,6 +7,7 @@
   --accent: #2245a2; --accent-soft: #e6ecfb; --accent-ink: #ffffff;
   --ok: #165737; --ok-soft: #e3f3ea; --bad: #8b2621; --bad-soft: #fbe5e3; --warn: #6f4100; --warn-soft: #fbf0dc;
   --violet: #553b95; --violet-soft: #efe9fa;
+  --change: #8a1f6b; --change-ink: #ffffff;
   --font-body: "Manrope", system-ui, -apple-system, "Segoe UI", sans-serif;
 }
 @media (prefers-color-scheme: dark) { :root:not([data-theme="light"]) {
@@ -14,12 +15,14 @@
   --accent: #a8bfff; --accent-soft: #1f2a45; --accent-ink: #0d1424;
   --ok: #6fd3a0; --ok-soft: #173327; --bad: #ffa8a0; --bad-soft: #3a1c1a; --warn: #f0b860; --warn-soft: #362a14;
   --violet: #c5b1f2; --violet-soft: #251f38;
+  --change: #f5a3dc; --change-ink: #1f0a19;
   color-scheme: dark } }
 :root[data-theme="dark"] {
   --bg: #12151b; --surface: #1a1f28; --fg: #e6e9ef; --edge: #6d7891; --line: #2b3240;
   --accent: #a8bfff; --accent-soft: #1f2a45; --accent-ink: #0d1424;
   --ok: #6fd3a0; --ok-soft: #173327; --bad: #ffa8a0; --bad-soft: #3a1c1a; --warn: #f0b860; --warn-soft: #362a14;
   --violet: #c5b1f2; --violet-soft: #251f38;
+  --change: #f5a3dc; --change-ink: #1f0a19;
   color-scheme: dark }
 * { box-sizing: border-box }
 body { background: var(--bg); color: var(--fg); font-family: var(--font-body); font-size: 17px; line-height: 1.4; padding-inline: 16px; padding-block: 24px 48px; margin: 0 }
@@ -95,6 +98,12 @@ details.what[open] summary::before { content: "▾" }
 .steps .sh { font-weight: 800 }
 .step { display: flex; gap: 10px; align-items: flex-start }
 .step .num { flex: none; width: 30px; height: 30px; border-radius: 50%; background: var(--warn); color: var(--surface); display: grid; place-items: center; font-weight: 800 }
+.lchip.changed::before { content: ""; width: 10px; height: 10px; border-radius: 50%; background: var(--change); flex: none }
+.changed { box-shadow: inset 4px 0 0 var(--change); padding-left: 12px; border-radius: 4px }
+.next.changed { border-radius: 12px }
+.lchip.changed { box-shadow: none; border-radius: 999px; padding-left: 10px; border-color: var(--change); display: inline-flex; align-items: center; gap: 6px }
+.pill-changed { display: inline-flex; align-items: center; min-height: 28px; padding: 0 10px; border-radius: 999px; background: var(--change); color: var(--change-ink); font-size: 16px; font-weight: 800; margin-right: 8px; vertical-align: middle; flex: none; white-space: nowrap }
+.sr { position: absolute; width: 1px; height: 1px; overflow: hidden; clip-path: inset(50%); white-space: nowrap }
 .empty { background: var(--surface); border: 2px dashed var(--edge); border-radius: 14px; padding: 20px; font-weight: 600 }
 @media (hover: hover) { .lchip:hover, .fchip:hover, .tab:hover, .copy:hover { border-color: var(--accent) } }
 </style>
@@ -117,6 +126,7 @@ details.what[open] summary::before { content: "▾" }
 <script type="application/json" id="tracker-data">
 {
  "updated": "2026-01-15T17:30:00Z",
+ "round": 3,
  "owners": [
   {
    "key": "web",
@@ -147,6 +157,8 @@ details.what[open] summary::before { content: "▾" }
    "needs_you": true,
    "card": true,
    "next": "choose between the two dates in the linked plan.",
+   "changed_round": 3,
+   "changed_parts": ["next"],
    "summary": [
     "One release date everyone plans around",
     "The launch checklist dated backward from it",
@@ -167,6 +179,8 @@ details.what[open] summary::before { content: "▾" }
    "status": "blocked",
    "card": true,
    "next": "wait for the staging database to come back online.",
+   "changed_round": 2,
+   "changed_parts": ["next"],
    "summary": [
     "The login test passes ten runs in a row",
     "The root cause written in the pull request"
@@ -186,6 +200,8 @@ details.what[open] summary::before { content: "▾" }
    "status": "doing",
    "card": true,
    "next": "finish the index change and rerun the timing check.",
+   "changed_round": 3,
+   "changed_parts": ["summary"],
    "summary": [
     "Search answers in under one second",
     "A timing check that guards it",
@@ -288,6 +304,8 @@ const STAGE_BY_KEY = Object.fromEntries(STAGES.map(s => [s.key, s]));
 const TRACK = ["Card", "Builder", "Pull request", "Merged"];
 const state = { tab: "open", owner: "all", query: "" };
 try { const saved = JSON.parse(localStorage.getItem("work-tracker-view") || "{}"); if (saved.tab) state.tab = saved.tab; if (saved.owner) state.owner = saved.owner; } catch (e) {}
+const REVIEW_ROUND = Number(TRACKER.round) || 0;
+const CHANGE_BASELINE = changeBaseline();
 const ownerByKey = Object.fromEntries(TRACKER.owners.map(o => [o.key, o]));
 const isDone = it => it.status === "done";
 function stageOf(it) {
@@ -300,6 +318,33 @@ function stageOf(it) {
 const slug = it => "item-" + it.title.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-|-$/g, "");
 const fmt = iso => { if (!iso) return ""; try { return new Date(iso).toLocaleString(undefined, { month: "long", day: "numeric", hour: "numeric", minute: "2-digit" }); } catch (e) { return iso; } };
 const el = (tag, cls, text) => { const n = document.createElement(tag); if (cls) n.className = cls; if (text != null) n.textContent = text; return n; };
+function changeBaseline() {
+  try {
+    const held = JSON.parse(sessionStorage.getItem("work-tracker-baseline") || "null");
+    if (held && held.round === REVIEW_ROUND) return held.baseline;
+  } catch (e) {}
+  let baseline = REVIEW_ROUND - 1;
+  try {
+    const seen = localStorage.getItem("work-tracker-seen-round");
+    if (seen !== null && Number.isFinite(Number(seen))) baseline = Math.min(Number(seen), REVIEW_ROUND);
+  } catch (e) {}
+  try { sessionStorage.setItem("work-tracker-baseline", JSON.stringify({ round: REVIEW_ROUND, baseline })); } catch (e) {}
+  try { localStorage.setItem("work-tracker-seen-round", String(REVIEW_ROUND)); } catch (e) {}
+  return baseline;
+}
+function stampChange(node, it, part) {
+  const parts = [].concat(it.changed_parts || []);
+  if (it.changed_round && (part ? parts.includes(part) : parts.length)) node.dataset.changedRound = it.changed_round;
+  return node;
+}
+function markChanges() {
+  for (const node of document.querySelectorAll("[data-changed-round]")) {
+    if (Number(node.dataset.changedRound) <= CHANGE_BASELINE || node.classList.contains("changed")) continue;
+    node.classList.add("changed");
+    const label = node.querySelector(":scope > summary") || node;
+    label.prepend(node.dataset.changeMark === "dot" ? el("span", "sr", "Changed. ") : el("span", "pill-changed", "Changed"));
+  }
+}
 function save() { try { localStorage.setItem("work-tracker-view", JSON.stringify({ tab: state.tab, owner: state.owner })); } catch (e) {} }
 function inView(it) {
   if ((state.tab === "done") !== isDone(it)) return false;
@@ -329,7 +374,8 @@ function renderBoard(items) {
     if (mine.length) {
       const chips = el("div", "lchips");
       for (const it of mine) {
-        const b = el("button", "lchip", it.title);
+        const b = stampChange(el("button", "lchip", it.title), it);
+        b.dataset.changeMark = "dot";
         b.setAttribute("aria-label", "Go to " + it.title);
         b.onclick = () => jumpTo(it);
         chips.append(b);
@@ -398,16 +444,17 @@ function itemCard(it) {
   band.append(proj);
   card.append(band);
   const body = el("div", "body");
-  body.append(el("h3", "title", it.title));
+  body.append(stampChange(el("h3", "title", it.title), it, "title"));
   body.append(trackFor(it));
   if (it.next) {
-    const next = el("div", "next");
+    const next = stampChange(el("div", "next"), it, "next");
     next.append(el("span", "ic", isDone(it) ? "✅" : "👉"), el("span", null, it.next.charAt(0).toUpperCase() + it.next.slice(1)));
     body.append(next);
   }
   const points = [].concat(it.summary || []);
   if (points.length) {
-    const d = el("details", "what");
+    const d = stampChange(el("details", "what"), it, "summary");
+    d.open = d.dataset.changedRound > CHANGE_BASELINE;
     d.append(el("summary", null, "What it delivers"));
     const ul = el("ul", "points");
     for (const p of points) ul.append(el("li", null, p));
@@ -415,12 +462,12 @@ function itemCard(it) {
     body.append(d);
   }
   if (it.proposal && !isDone(it) && it.owner_steps && it.owner_steps.length) {
-    const st = el("div", "steps");
+    const st = stampChange(el("div", "steps"), it, "steps");
     st.append(el("div", "sh", "Your steps to create " + it.proposal.name));
     it.owner_steps.forEach((text, i) => { const row = el("div", "step"); row.append(el("span", "num", String(i + 1)), el("span", null, text)); st.append(row); });
     body.append(st);
   }
-  const actions = el("div", "actions");
+  const actions = stampChange(el("div", "actions"), it, "links");
   for (const l of it.links || []) {
     const a = el("a", "link");
     a.append(el("span", null, "🔗"), el("span", null, l.label));
@@ -458,6 +505,7 @@ function render() {
     list.append(h);
     mine.forEach(i => list.append(itemCard(i)));
   }
+  markChanges();
 }
 document.querySelectorAll(".tab").forEach(t => t.onclick = () => { state.tab = t.dataset.tab; save(); render(); });
 document.getElementById("search").oninput = e => { state.query = e.target.value; render(); };

package/package.json

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