skilldrop-cli 0.16.14 → 0.16.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skilldrop-cli",
-  "version": "0.16.14",
+  "version": "0.16.15",
   "description": "Portable AI-agent skills, measured against the agent without them. ADRs, PRDs, runbooks, decks and reviews, installable into Claude Code, Cursor, Kiro, and any IDE that takes custom instructions. One command per skill or per role-based pack.",
   "author": {
     "name": "Sanjay Ananthanarayan",

package/packs/dev-team/loops/release/LOOP.md

@@ -29,8 +29,8 @@ being folded into either neighbour.
    checks rollback, detection, response, measured targets, data and privacy, comms and
    exposure, each with evidence. `REVISE` returns to `plan`, and the report names which gap to
    close and which skill closes it. A missing rollback is `BLOCKED` on its own.
-3. **Announce only once it is ready.** `release-notes` drafts the customer-facing and
-   internal notes from the git range. Writing them before G3 risks announcing a launch that
+3. **Announce only once it is ready.** `release-notes` drafts the notes for each audience
+   you name from the git range, with nothing the log doesn't say. Writing them before G3 risks announcing a launch that
    then does not happen.
 4. **A human decides at G3.1.** The release owner decides whether to ship *now*, given
    timing, freeze windows and on-call coverage. When that decision is contested, use

package/packs/dev-team/loops/release/loop.json

@@ -1,7 +1,7 @@
 {
   "$comment": "RFC-0031. Covers the step between build (ends at merge, G2.1) and operate (starts at instrument). Its gates are G3 and G3.1: the whole number places the loop in the lifecycle (discover G0 through operate G4), and a decimal is a second gate inside the same loop (RFC-0044).",
   "name": "release",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Take merged code to live users through a planned, reversible rollout — plan the phases and the rollback, judge readiness on evidence, draft the release notes, then a human makes the go/no-go call. Use when the user has merged a change and asks how to roll it out, whether they are ready to launch, or for a go/no-go decision.",
   "entrypoint": "LOOP.md",
   "kind": "loop",

package/packs/dev-team/skills/release-notes/SKILL.md

@@ -1,69 +1,96 @@
 ---
 name: release-notes
-description: Turn git history between two refs into release notes — a customer-facing version written in reader benefits and an internal Keep-a-Changelog version with commit traceability, breaking changes always first. Use when the user wants release notes, a changelog, "what's in this release", a deploy announcement, or to summarize changes since the last tag.
+description: Write release notes from a git log, a list of merged PRs or rough notes about what changed — for the audience and in the format the user names (a customer changelog page, an email, an in-app "what's new", store text, a CHANGELOG.md entry). Says nothing the input doesn't, writes a vague change only as far as it was described, puts breaking changes and anything the reader must do first, and keeps internal noise and working notes out of the text. Use when the user wants release notes, a changelog entry, "what's in this release", a deploy announcement, or to summarise changes since the last tag.
 ---
 
 # release-notes
 
-Converts commit history into two artifacts with different readers: **customer-facing release notes** (benefits, no commit-speak) and an **internal changelog** (Keep a Changelog format, every line traceable to a commit). The rule that drives everything: write for the reader who didn't make the change.
+**The rule: nothing goes in the notes that the log or the user didn't say.** A release note is
+read as a promise. The agent's habit is to make a vague commit sound finished: "tweaked onboarding"
+becomes "onboarding is now simpler", a half-remembered fix gets a symptom, a reverted feature
+gets announced because its commit is in the range. Each of those reads exactly like a real
+change, and a customer will hold you to it. This skill's job is to write only what the input
+supports, as far as it supports it.
 
 ## How to respond
 
-**Four rules come before the steps and outrank them:**
-
-- **Answer what was asked, first.** Open with the answer, the decision or the artifact, in plain words. Scores, matrices, frameworks and tags come after it, and anything that doesn't change the answer is cut.
-- **Use only what you were given.** Don't add facts, names, numbers, incidents, history, steps or sections the input doesn't contain. What you need and don't have is left out of the artifact and listed once at the end under "To confirm".
-- **Deliver from what you have.** When the request gives you something to work on, state your assumptions in a line and produce the result. When it gives you nothing to work on, ask for it in one or two plain sentences and say what you will do once you have it.
-- **Write for someone who has never heard of this skill.** No skill names, no paths or scripts from this folder, no internal terms, and nothing about how the run was set up. A next step is one plain sentence at the end that describes the work.
-- **For this skill:** Write one set of notes for the audience the user named. Add the second, internal version only when they ask for both. Use the product's name only if they gave it, and count commits and pull requests from the list in front of you.
-
-1. **Resolve the range.** Default: last tag to HEAD (`git describe --tags --abbrev=0`). If no tags exist, ask for a start ref or take the user's two refs. State the resolved range at the top of the output: "`v1.4.0..HEAD`, 37 commits, 2026-05-02 → 2026-06-10".
-
-2. **Gather the raw material.** `git log <range> --no-merges --pretty='%h|%s|%an|%ad' --date=short` for the commit list; `git log <range> --stat` only when a subject line is too vague to classify. Where the `gh` CLI is available and the repo uses PRs, prefer PR titles over commit subjects (`gh pr list --state merged --search "merged:>={date}"`) — they're written closer to reader language. Don't require `gh`; commits alone are enough.
-
-3. **Classify every change** into Keep-a-Changelog categories plus one local bucket:
-   - **Added / Changed / Fixed / Deprecated / Removed / Security** — reader-visible
-   - **Internal** — refactors, CI, deps, tests; appears in the internal changelog, never in customer notes
-   - **Needs review** — can't tell what it does from the commit; goes in a list for the user, never guessed into customer notes
-
-4. **Hoist breaking changes and migrations to the top** of both artifacts, each with an **Action required** line. A breaking change buried under "Changed" is the cardinal failure of release notes. Detect via `BREAKING`, `!:` conventional-commit markers, migration files, major dependency bumps, and removed public API — and say which signal triggered the flag.
-
-5. **Rewrite, don't transcribe.** Each customer-facing line states what the reader can now do or no longer suffers. ✅ *"Fixed a crash when exporting reports with more than 10,000 rows"* — ❌ *"fix NPE in ReportExporter.flush()"*. Drop ticket numbers and author names from customer notes; keep both in the internal changelog with the short hash on every line.
-
-6. **Emit both artifacts** with [`templates/release-notes.md`](templates/release-notes.md): customer notes first, internal changelog second, then the **Needs review** list. If the user wants only one artifact, emit that one — but the needs-review list is never optional.
-
-## Useful references in this skill
-
-- [`templates/release-notes.md`](templates/release-notes.md) — both artifact skeletons plus the classification cheat-sheet
+1. **Find out what actually shipped.** Read the whole range or list before writing a line.
+   - A change that was reverted later in the same range did not ship. Leave it out.
+   - A later commit can narrow or contradict an earlier one ("partial fix", a figure revised,
+     a date moved). Write the final state, not the first claim.
+   - Where you have a repo and no list, take the range from the last tag to HEAD
+     (`git describe --tags --abbrev=0`, then `git log <tag>..HEAD --no-merges`). Read a commit's
+     body or diff only when its subject is too vague to tell what it does.
+
+2. **Find what breaks or needs action, wherever it is said.** A breaking change is often not
+   marked: a removed option in a PR description, a renamed setting in a commit body, an old
+   endpoint "no longer accepted". Each one goes first, with the step the reader has to take,
+   written for how this audience gets the release. Customers of a hosted service upgrade
+   nothing; they need to know what stops working and what to do instead. A library's users
+   need the version and the migration.
+
+3. **Write only as far as each change was described.** "Fixed a bug in exports" stays that
+   vague if that's all the input says. Don't add a symptom, a cause, a number, a date, a
+   version, a PR number or a count the input doesn't give. A change you can't tell the meaning
+   of stays out of the notes and goes in the short list in step 6.
+
+4. **Write for this reader, in this format.** Use the channel, length and tone the user asked
+   for: a customer changelog page, an email, a pop-up of a few lines, store text with a
+   character limit, or the repo's own CHANGELOG style if there is one. Say what the reader can
+   now do or no longer suffers ✅ *"Exports with more than 10,000 rows no longer fail"*, not
+   what the code does ❌ *"fix NPE in ReportExporter.flush()"*. Use the product name only if
+   you were given it.
+
+5. **Leave internal work out of anything a customer reads.** CI, refactors, tests, dependency
+   bumps, typo fixes and tooling are not news, not even as "various improvements". In an
+   internal changelog they may appear if the repo's existing style includes them.
+
+6. **Hand back the notes, then at most a few lines for the user.** The notes are clean: no
+   commit range, commit count, hashes, "flagged by" notes, or review lists inside them unless
+   the user's format asks for hashes (an internal CHANGELOG often does). After the notes, list
+   once, briefly, what you left out and why: changes too vague to describe, claims the log
+   contradicts, anything the user should confirm before publishing.
+
+7. **Deliver from what you have.** With a log, a list or notes, write the notes and state any
+   assumption about audience or format in one line. Ask first only when there is nothing to
+   work from.
 
 ## Quality bar
 
-- **Breaking changes are first, with an action.** Every breaking entry has "Action required: …" a reader can execute. No action identifiable → it goes to Needs review, not to the top section unverified.
-- **No commit-speak in customer notes.** Class names, function names, internal service names, and ticket IDs are all signals the line wasn't rewritten.
-- **Every internal-changelog line ends in a short hash** (and PR number when available). A claim nobody can trace is a claim nobody can verify.
-- **Internal noise stays internal.** Dependency bumps, refactors, CI, and test-only changes never appear in customer notes — not even as "various improvements".
-- **Nothing is guessed.** A vague commit ("fix stuff", "wip") lands in Needs review with its hash; the skill never invents a user-facing meaning for it.
-- **The range is stated.** Refs, commit count, and date span at the top — so a reader knows exactly what window the notes cover.
+- **Nothing in the notes goes beyond the input.** No invented feature, behaviour, symptom,
+  figure, date, version, PR number or count.
+- **Reverted and superseded changes don't appear;** the final state does.
+- **Breaking changes and required actions come first,** found from the text as well as the
+  markers, with an action that fits how the audience gets the release.
+- **Internal noise is absent from customer text.**
+- **The format, length and tone are the ones the user asked for,** and the notes carry no
+  working notes.
+- **Vague items are left out or kept vague, and named once to the user.**
 
 ## When to use this skill
 
 - ✅ Cutting a release and writing "what's new" for users or customers
-- ✅ Maintaining `CHANGELOG.md` between versions
-- ✅ A deploy/release announcement for internal stakeholders
-- ✅ "Summarize everything merged since the last tag"
+- ✅ A `CHANGELOG.md` entry between versions
+- ✅ A deploy or release announcement for internal stakeholders
+- ✅ "Summarise everything merged since the last tag"
 
 ## When NOT to use this skill
 
 - ❌ A single PR description — that's one change; use `pr-description-writer`
 - ❌ Reviewing the changes for quality or risk — that's `devils-advocate` / `sonar-review`
-- ❌ Stakeholder narrative about *why* the release matters strategically — feed the notes to `exec-summary`
-- ❌ Generating commit messages going forward — this skill reads history, it doesn't write it
+- ❌ A narrative about *why* the release matters strategically — feed the notes to `exec-summary`
+- ❌ Writing commit messages going forward — this skill reads history, it doesn't write it
 
 ## Anti-patterns to avoid
 
-- ❌ **Transcribing commit subjects with bullets in front.** That's `git log` with extra steps — the rewrite into reader language is the entire job.
-- ❌ **"Various bug fixes and improvements."** Either the fix is worth a line or it's internal; the vague catch-all is neither.
-- ❌ **Burying a breaking change** in the middle of "Changed" because the commit didn't shout about it. The detection signals exist precisely for quiet breaks.
-- ❌ **Padding a thin release.** Two real changes get two lines. Inflating with internal noise to look substantial erodes trust in every future release note.
-- ❌ **Marketing adjectives.** "Blazing-fast", "completely redesigned", "supercharged" — state the measurable change ("export is ~3× faster on large reports") or state nothing.
-- ❌ **Silently skipping unparseable commits.** They go to Needs review; the count of skipped commits is part of the output.
+- ❌ **Finishing a vague change.** "Looked at the search thing" becomes "search is faster".
+  Keep it as vague as it was, or leave it out and say so.
+- ❌ **Announcing reverted work.** A commit in the range is not a change that shipped.
+- ❌ **Burying a quiet break.** No `!:` marker doesn't mean nothing broke; read the bodies.
+- ❌ **"Various bug fixes and improvements."** Either a fix is worth a line or it's internal.
+- ❌ **The working notes in the text.** Ranges, counts, hashes, "flagged by", a "Needs review"
+  table in a customer email.
+- ❌ **A changelog where an email was asked for.** Headings and bullets in a two-paragraph
+  member update.
+- ❌ **Marketing adjectives.** "Blazing-fast", "completely redesigned": state the measured
+  change the input gives, or nothing.

package/packs/dev-team/skills/release-notes/evals/evals.json

@@ -5,12 +5,12 @@
       "id": 1,
       "prompt": "Write release notes for v2.4.0 from this log (range v2.3.0..HEAD, 2026-08-12 to 2026-09-28):\n\na1b2c3d|feat: bulk CSV import for contacts (#412)\ne4f5a6b|fix: NPE in ReportExporter.flush() when >10k rows (#418)\nc7d8e9f|feat!: remove legacy /v1/contacts endpoint (#420)\n0a1b2c3|chore(deps): bump lodash 4.17.20 -> 4.17.21\n4d5e6f7|refactor: extract ExportService\n8a9b0c1|fix stuff\n2d3e4f5|perf: cache tenant settings, report export ~3x faster (#425)",
       "assertions": [
-        "States the range, commit count (7), and date span at the top",
-        "The removal of /v1/contacts is hoisted to the top of both artifacts with an 'Action required' line and names the !: marker that flagged it",
+        "Delivers the notes from the log without stopping to ask who they are for, stating any assumption about the audience in a line",
+        "The removal of /v1/contacts comes first, with the step API users must take",
         "Customer notes contain no commit-speak — no ReportExporter.flush(), class names, ticket numbers, or author names",
-        "The lodash bump and the ExportService refactor appear only in the internal changelog, never in customer notes, not even as 'various improvements'",
-        "Every internal changelog line ends with its short hash, plus PR number where present",
-        "The vague 'fix stuff' commit (8a9b0c1) goes in the Needs review list with its hash, not guessed into customer notes",
+        "The lodash bump and the ExportService refactor do not appear in customer-facing notes, not even as 'various improvements'",
+        "The notes carry no working notes: no commit range, commit count, hashes or review lists in customer-facing text",
+        "The vague 'fix stuff' commit is not given an invented meaning in the notes, and is named to the user outside them",
         "No marketing adjectives; the performance line states the measured change (~3x faster export) rather than 'blazing fast'",
         "Does not invent features or fixes not present in the log"
       ]
@@ -72,6 +72,34 @@
         "It does not invent crash counts, percentages, a release date for Android, or features that are not in the list",
         "The store text itself is short: about 100 words or fewer per store"
       ]
+    },
+    {
+      "id": 6,
+      "held_out": "2026-10-17",
+      "prompt": "Our next Quillmoor release goes out tomorrow. Quillmoor is our invoicing app for freelancers and small studios, and some of our customers use our API. The git log since the last release is in `git-log.txt`. Write the customer-facing release notes for our public changelog page: Markdown, 200 words max. I want them ready to paste.",
+      "assertions": [
+        "The webhook payload change (`amount` now an integer in minor units, `client_name` renamed to `client.display_name`) is in the notes and comes before every other change, worded so a customer with a webhook integration can tell they must update their handler",
+        "The Swiss QR-bill feature is not presented as part of the release, since it was reverted in the same range",
+        "No internal-only work appears in the notes: no CI changes, dependency bumps, test fixtures, the auth/session refactor, feature-flag removal, release checklist, or typo fixes in comments or CONTRIBUTING",
+        "The 'misc fixes from Henrike's QA pass' and 'sync stuff' commits are either left out or mentioned no more specifically than general small fixes; no particular behaviour, such as a sync feature, is attributed to them",
+        "Does not invent anything absent from the log or request: no version number, release date, PR or ticket number, count of changes, or figure such as how much faster the invoice list is",
+        "Covers at least six of these seven customer-visible changes: duplicating an invoice, receipt photos from a phone's browser, the client-portal ZIP download, no more reminders for invoices marked paid offline, the VAT rounding fix, the due date column in CSV export, and the faster invoice list",
+        "The pasteable notes are 200 words or fewer and in Markdown, are delivered without first stopping to ask questions, and contain no working notes about what was left out or why"
+      ]
+    },
+    {
+      "id": 7,
+      "held_out": "2026-10-17",
+      "prompt": "I run the Bramblegate Tool Library, it's a volunteer thing where members borrow tools through our little app. My nephew does the app and he did a bunch of stuff this month. From memory:\n\n- you can reserve tools a week ahead now instead of 3 days? I think a week\n- the login code by text message, he fixed that, people were complaining it didn't arrive\n- something about late fees, he changed how they work, I need to check with him but basically you get a reminder before it's late now\n- everyone has to re-accept the borrowing agreement next time they log in because the committee updated it\n- new photos on the tool pages\n- also a lot of backend stuff I don't understand\n\nCan you write the text for the \"What's new\" pop-up members see when they open the app? It's a small box, so 75 words max.",
+      "assertions": [
+        "Re-accepting the updated borrowing agreement at next log-in comes first in the pop-up text and is stated as something the member will have to do",
+        "Does not invent details: no late-fee amounts, no number of days before the due date that the reminder arrives, no reservation window other than a week, no cause for the login problem, no date, and no change that is not in the notes",
+        "The late-fee item goes only as far as described, a reminder before an item is late, and does not claim fees were raised, lowered or removed",
+        "The 'backend stuff' is not in the pop-up text, and nothing specific is claimed about it",
+        "Mentions reserving further ahead, the fixed text-message login code, and the new tool photos",
+        "The pop-up text is 75 words or fewer and in plain language for library members, with no developer jargon such as 'backend', 'OTP' or 'SMS verification flow'",
+        "Delivers the pop-up text without first stopping to ask for confirmation; any suggestion to check details with the developer sits outside the pop-up text"
+      ]
     }
   ]
 }

package/packs/dev-team/skills/release-notes/evals/files/6/git-log.txt

@@ -0,0 +1,136 @@
+commit 0d1f6b8
+Author: Jun Okafor
+
+    chore(release): scaffold release checklist
+
+commit a4c91e0
+Author: Odile Brandt
+
+    chore(ci): cache node_modules between workflow runs
+
+commit 7d2b33f
+Author: Tomasz Wereszka
+
+    Revert "feat(invoices): Swiss QR-bill block on PDF invoices"
+
+    This reverts commit 3e8f10b. The bank validator rejects our reference
+    field. Pulling it until we can test against real QR scanners.
+
+commit e01f5c9
+Author: Marisol Quintero
+
+    fix(reminders): stop sending payment reminders for invoices marked paid offline
+
+    Invoices marked "paid by bank transfer" by hand were still getting the
+    automatic reminder email sent to the client.
+
+commit 5bb7a20
+Author: Tomasz Wereszka
+
+    misc fixes from Henrike's QA pass
+
+commit c33d9e4
+Author: Jun Okafor
+
+    chore(deps): bump pdfkit from 0.13.0 to 0.15.1
+
+commit 2f6a811
+Author: Marisol Quintero
+
+    feat(expenses): attach receipt photos to expenses from your phone's browser
+
+    Camera upload on mobile web. JPG, PNG and HEIC, up to 10 MB each.
+
+commit 88e0b4c
+Author: Jun Okafor
+
+    refactor(webhooks): normalise payload serializer
+
+    Moves all webhook payloads onto the shared serializer. Side effect:
+    `amount` on invoice.created and invoice.paid is now an integer in minor
+    units (cents) instead of a decimal string, to match the REST API, and
+    `client_name` is renamed to `client.display_name`. Anyone parsing our
+    webhooks has to update their handler. There is no compatibility flag.
+
+commit 41d7f02
+Author: Tomasz Wereszka
+
+    test: add fixtures for multi-currency invoices
+
+commit 3e8f10b
+Author: Tomasz Wereszka
+
+    feat(invoices): Swiss QR-bill block on PDF invoices
+
+    Adds the QR payment slip to the bottom of PDF invoices for clients
+    with a Swiss address.
+
+commit b9a2e6c
+Author: Odile Brandt
+
+    sync stuff
+
+commit 6c1d8a5
+Author: Marisol Quintero
+
+    feat(invoices): duplicate an invoice from the invoice menu
+
+    Copies line items, client and tax settings. The new invoice gets the
+    next invoice number and today's date.
+
+commit d57e3b1
+Author: Jun Okafor
+
+    perf(dashboard): faster invoice list for accounts with lots of invoices
+
+    Paginate server-side instead of fetching every invoice up front.
+
+commit 0ae4f9c
+Author: Odile Brandt
+
+    docs: fix typo in CONTRIBUTING
+
+commit f62c0d8
+Author: Marisol Quintero
+
+    fix(tax): VAT total off by a cent on long invoices
+
+    Totals on invoices with many line items could be off by 0.01 because
+    we rounded each line instead of the total.
+
+commit 19b7e44
+Author: Jun Okafor
+
+    chore: remove unused feature flag new_sidebar_v1
+
+commit ab30c6e
+Author: Tomasz Wereszka
+
+    fix(export): CSV export now includes the due date column
+
+commit 7e5d1f2
+Author: Odile Brandt
+
+    ci: run e2e suite on Node 22
+
+commit 2c94b8d
+Author: Jun Okafor
+
+    refactor(auth): extract session middleware
+
+commit 95f0a3c
+Author: Marisol Quintero
+
+    feat(portal): clients can download all their paid invoices as one ZIP
+
+    New "Download all" button on the client portal's invoice page.
+
+commit 4b6e7d9
+Author: Odile Brandt
+
+    chore(deps-dev): bump eslint to 9.12
+
+commit e8c2a05
+Author: Tomasz Wereszka
+
+    chore: fix typos in code comments

package/packs/dev-team/skills/release-notes/manifest.json

@@ -1,11 +1,11 @@
 {
   "name": "release-notes",
-  "version": "0.1.2",
-  "description": "Turn git history between two refs into release notes — a customer-facing version written in reader benefits and an internal Keep-a-Changelog version with commit traceability, breaking changes always first. Use when the user wants release notes, a changelog, \"what's in this release\", a deploy announcement, or to summarize changes since the last tag.",
+  "version": "0.2.0",
+  "description": "Write release notes from a git log, a list of merged PRs or rough notes about what changed — for the audience and in the format the user names (a customer changelog page, an email, an in-app \"what's new\", store text, a CHANGELOG.md entry). Says nothing the input doesn't, writes a vague change only as far as it was described, puts breaking changes and anything the reader must do first, and keeps internal noise and working notes out of the text. Use when the user wants release notes, a changelog entry, \"what's in this release\", a deploy announcement, or to summarise changes since the last tag.",
   "entrypoint": "SKILL.md",
   "deps": { "npm": [], "pip": [] },
   "env": { "required": [], "optional": [] },
   "related": ["devils-advocate", "exec-summary", "pr-description-writer", "sonar-review"],
   "tags": ["release-notes", "changelog", "git", "release", "deploy", "communication"],
-  "model": { "tier": "standard", "rationale": "Classifies commits and rewrites them for a reader. The audience translation needs judgment; escalate via large-input on very long ranges." }
+  "model": { "tier": "standard", "rationale": "Reads a range for what really shipped and rewrites it for one reader. The audience translation needs judgment; escalate via large-input on very long ranges." }
 }

package/packs/dev-team/skills/release-notes/templates/release-notes.md

@@ -1,83 +0,0 @@
-# Release-notes templates
-
-Two artifacts, two readers. Customer notes sell nothing and explain everything; the internal changelog trades polish for traceability.
-
-## Artifact 1 — customer-facing release notes
-
-> Note the deliberate rename for a non-technical reader: the Keep-a-Changelog categories **Added → New** and **Changed → Improved** here; Fixed/Security keep their names. The internal changelog (Artifact 2) uses the canonical Keep-a-Changelog headings.
-
-```markdown
-# {Product} {version} — {YYYY-MM-DD}
-
-> Covers {ref}..{ref} · {N} changes
-
-## ⚠ Breaking changes
-
-- **{What broke, in reader terms.}**
-  Action required: {the exact step — config to change, endpoint to migrate, deadline}.
-  (flagged by: {signal — `!:` marker / removed API / migration file})
-
-## New
-
-- {What the reader can now do. One sentence, benefit first.}
-
-## Improved
-
-- {What got better, measurably where possible: "export is ~3× faster on reports over 10k rows".}
-
-## Fixed
-
-- {The symptom the reader no longer suffers: "Fixed a crash when …", "Dates no longer shift by one day when …".}
-
-## Security
-
-- {Reader-relevant statement; severity and affected versions; no exploit detail.}
-```
-
-Rules: no class/function/service names, no ticket IDs, no author names, no "various improvements". A section with nothing in it is deleted, not left empty — except Breaking changes, which when empty becomes the single line "No breaking changes."
-
-## Artifact 2 — internal changelog (Keep a Changelog)
-
-```markdown
-## [{version}] - {YYYY-MM-DD}
-
-Range: `{ref}..{ref}` · {N} commits · {N} PRs
-
-### Added
-- {change} ({#PR}, `{hash}`)
-
-### Changed
-- **BREAKING:** {change + migration step} ({#PR}, `{hash}`)
-- {change} ({#PR}, `{hash}`)
-
-### Fixed
-- {change} ({#PR}, `{hash}`)
-
-### Deprecated / Removed / Security
-- {change} ({#PR}, `{hash}`)
-
-### Internal
-- {refactor / CI / deps / tests — one line each} (`{hash}`)
-```
-
-## Needs review (always emitted, both modes)
-
-```markdown
-## Needs review — {N} commits not classified
-
-| Hash | Subject | Why unclear |
-|---|---|---|
-| `{hash}` | {verbatim subject} | {vague / contradicts diff / can't tell if user-visible} |
-```
-
-## Classification cheat-sheet
-
-| Signal | Bucket |
-|---|---|
-| New endpoint, flag default-on, new UI surface | Added |
-| Behavior or default changed for existing users | Changed (BREAKING if old behavior unobtainable) |
-| `fix:`, bug ticket, regression revert | Fixed |
-| `chore:`, `refactor:`, `test:`, `ci:`, deps, lockfiles | Internal |
-| CVE, auth/authz, input validation, secrets | Security |
-| `feat!:`, `BREAKING CHANGE:`, removed public API, schema migration, major version bump of a public SDK | Breaking — hoist to top |
-| "wip", "fix stuff", "address comments", empty body + vague subject | Needs review |