@mohammadhprp/system-prompt 0.12.3 → 0.12.5

package/framework/commands/review.md

@@ -1,24 +1,9 @@
 ---
-description: Perform comprehensive code quality review
+description: Review local, GitHub, or GitLab changes and write review.json
 agent: plan
 ---
 
-Review $ARGUMENTS
-
-Perform comprehensive code quality review.
-
-## Process
-
-1. **Review conversation and diff** - Read conversation history, run `git diff` for staged/unstaged changes, identify the problem being solved and the behavior being modified. Read related standards: [`references/standards/naming.md`](../references/standards/naming.md), [`references/standards/testing.md`](../references/standards/testing.md), [`references/standards/security.md`](../references/standards/security.md), [`references/standards/performance.md`](../references/standards/performance.md).
-
-2. **Check correctness** - Edge cases, concurrency, error handling, state transitions, backward compatibility. Read contracts and interfaces before implementation.
-
-3. **Check maintainability** - Naming reflects business meaning, structure matches project conventions, comments explain why not what.
-
-4. **Check testing** - Do tests prove the behavior change? Missing edge cases or failure paths? Tests should verify behavior, not mirror implementation.
-
-5. **Check performance** - N+1 queries, unbounded loops, unnecessary allocations, caching opportunities.
-
-6. **Check security** - Input validation, authentication enforcement, secrets exposure, least privilege.
-
-7. **Present findings** - Distinguish blockers from suggestions, explain reasoning for each, summarize overall risk and production readiness.
+Review `$ARGUMENTS` using the `review` skill. Select local changes, a GitHub
+pull request, or a GitLab merge request from the arguments. Use `gh` or `glab`
+for read-only remote inspection, never publish comments, and write the required
+`review.json` artifact.

package/framework/skills/README.md

@@ -9,7 +9,12 @@ This catalog is framework-agnostic: each skill defines when to activate, a step-
 | Skill | Purpose | Best fit |
 | --- | --- | --- |
 | [agent-browser](./agent-browser/SKILL.md) | Automate browser and Electron workflows for navigation, testing, screenshots, and data extraction. | Website interaction, browser automation, exploratory testing, QA, and Electron desktop app workflows. |
+| [architect](./architect/SKILL.md) | Sketch architecture and module boundaries before implementation. | Non-trivial design and implementation work. |
+| [arena](./arena/SKILL.md) | Compare parallel candidate solutions and synthesize the strongest result. | Non-trivial artifacts and design alternatives. |
 | [backend-best-practices](./backend-best-practices/SKILL.md) | Consolidated backend engineering practices for API design, architecture, data, security, testing, observability, performance, debugging, and refactoring. | Any backend task that benefits from structured domain guidance across multiple backend disciplines. |
+| [bro](./bro/SKILL.md) | Restate the last message in plain human language without jargon. | Simplifying technical explanations. |
+| [changelog](./changelog/SKILL.md) | Create or update `CHANGELOG.md` entries. | Release notes and unreleased changes. |
+| [commit](./commit/SKILL.md) | Create atomic Git commits with conventional messages. | Committing reviewed changes. |
 | [diagram-design](./diagram-design/SKILL.md) | Create technical and product diagrams as standalone HTML files with inline SVG and an opinionated editorial design system. | Architecture, data, process, and other diagrams that communicate more clearly than prose or tables. |
 | [docs-writer](./docs-writer/SKILL.md) | Write, review, and edit documentation files with consistent structure, tone, and technical accuracy. | Creating docs, reviewing markdown files, writing READMEs, or updating `/docs` directories. |
 | [effective-html](./effective-html/SKILL.md) | Create self-contained HTML artifacts with routed guidance for design, wireframes, prototypes, plans, and diagrams. | Standalone HTML reports, explainers, interfaces, wireframes, prototypes, plans, and diagrams. |
@@ -18,7 +23,14 @@ This catalog is framework-agnostic: each skill defines when to activate, a step-
 | [glab](./glab/SKILL.md) | Use the GitLab CLI (glab) to manage merge requests, issues, pipelines, and repositories from the command line. | Any project hosted on GitLab (SaaS or self-hosted). |
 | [humanizer](./humanizer/SKILL.md) | Remove signs of AI-generated writing from text — inflated importance, promotional language, em dash overuse, rule of three, AI vocabulary, and filler phrases. | Editing or reviewing prose to make it sound more natural and human-written. |
 | [improve](./improve/SKILL.md) | Audit repositories as a read-only senior advisor and produce prioritized implementation plans for another agent. | Repository-wide audits, improvement roadmaps, and implementation handoffs. |
+| [how](./how/SKILL.md) | Explain subsystem architecture, runtime flow, ownership, and layering. | Code walkthroughs and architecture questions. |
 | [jira-cli](./jira-cli/SKILL.md) | Use the Jira CLI (jira) to manage issues, sprints, epics, comments, transitions, and worklogs from the command line. | Any project using Jira (Cloud or self-hosted) for issue tracking. |
+| [commit](./commit/SKILL.md) | Create atomic Git commits with conventional messages. | Committing reviewed changes. |
+| [pull-request](./pull-request/SKILL.md) | Create or update GitHub pull requests (PRs). | GitHub contribution workflows. |
+| [merge-request](./merge-request/SKILL.md) | Create GitLab merge requests (MRs). | GitLab contribution workflows. |
+| [review](./review/SKILL.md) | Perform comprehensive code quality reviews. | Diff reviews and production-readiness checks. |
+| [changelog](./changelog/SKILL.md) | Create or update `CHANGELOG.md` entries. | Release notes and unreleased changes. |
+| [release](./release/SKILL.md) | Prepare and tag semantic-versioned releases. | Release management. |
 | [laravel-best-practices](./laravel-best-practices/SKILL.md) | Apply Laravel PHP patterns for Eloquent, validation, security, testing, caching, and architecture. | Writing, reviewing, or refactoring Laravel PHP code. |
 | [perf-web-optimization](./perf-web-optimization/SKILL.md) | Optimize web performance: bundle size, images, caching, lazy loading, and overall page speed. | A slow site, large bundles, layout shifts, poor Time to Interactive, or low Lighthouse scores. |
 | [security-best-practices](./security-best-practices/SKILL.md) | Perform language- and framework-specific security best-practice reviews and suggest improvements, with support for Python, JavaScript/TypeScript, and Go. | Explicit security reviews or reports, secure-by-default coding, or auditing existing code for major vulnerabilities. |
@@ -27,6 +39,9 @@ This catalog is framework-agnostic: each skill defines when to activate, a step-
 | [skill-creator](./skill-creator/SKILL.md) | Create new skills, modify and improve existing skills, and measure skill performance. | Designing new skills, optimizing existing skills, or running evals. |
 | [tlc-spec-driven](./tlc-spec-driven/SKILL.md) | Feature planning and implementation with adaptive phases, EARS testable requirements, atomic Conventional Commits, and independent verification. | Planning features, implementing with verification and atomic commits, or validating an implementation against a spec. |
 | [taste](./taste/SKILL.md) | Unified design-quality skill spanning anti-slop frontend design, image generation and image-to-code, brand kits, minimalist and industrial UI, redesigns, and Google Stitch design systems. | Landing pages, portfolios, redesigns, website/mobile/brand imagery, or any UI that must not look AI-generated. |
+| [tdd](./tdd/SKILL.md) | Drive clear bug fixes with focused failing regression tests. | Explicit TDD and regression-test workflows. |
+| [unslop](./unslop/SKILL.md) | Remove AI writing patterns and restore a natural human voice. | Editing prose to remove generic or artificial phrasing. |
+| [why](./why/SKILL.md) | Investigate the evidence and rationale behind code and design decisions. | Historical design-reasoning and motivation investigations. |
 
 ## How Skills Are Organized
 

package/framework/skills/architect/SKILL.md

@@ -0,0 +1,83 @@
+---
+name: architect
+description: "Sketch types, signatures, and module structure before code, then stay in the loop while implementation fills in. Use for /architect, 'architect this', 'design this', or non-trivial work where jumping to code would lock in the wrong shape."
+disable-model-invocation: true
+---
+
+# Architect
+
+Design before implementing. Sketch types, function signatures, class shapes, and module boundaries with `not implemented` bodies and pseudocode. Synthesize across multiple model perspectives, then fill in code against the chosen sketch. If implementation proves the sketch wrong, throw it out and redesign.
+
+## Start
+
+Open a todolist with one entry per phase before starting. Autonomous mode without checkpoints needs the list to show phase position and keep phases from silently disappearing.
+
+1. Ground
+2. Sketch
+3. Agree
+4. Implement
+5. Scrap
+
+## Phase A: Ground the problem
+
+Build a real mental model of every system the new code touches. Run the **how** skill over the relevant subsystems. Critique mode if existing structure is the constraint or the design must push back on it.
+
+Naming a file isn't grounding. Produce the traced model `how` prescribes. If the design redefines ownership or layering, also run the **why** skill on the existing shape so the rationale becomes a constraint, not a guess.
+
+Skip Phase A only when the work is genuinely greenfield with no surrounding system to integrate.
+
+## Phase B: Sketch
+
+Run the **arena** skill with the design-sketch task and the Phase A grounding artifacts. Pass `references/runner-prompt.md` as each runner's prompt. Each candidate produces a design package shaped per `references/rationale-template.md`: the caller's usage written first, then the type sketch, function signatures, module map, and prose rationale derived from it.
+
+Use your configured architect runners (defaults `claude-fable-5-1-thinking-max`, `gpt-5.6-sol-max`, `grok-4.6-fast-xhigh`, `claude-opus-5-thinking-xhigh`).
+
+Design it twice. Require at least two structurally distinct candidates before synthesis, even when the first looks sufficient. This is the **exhaust-the-design-space** principle skill made concrete. Whole-shape alternatives, not point fixes inside one shape.
+
+Screen every candidate against [`references/design-red-flags.md`](references/design-red-flags.md) before synthesis. Reject or revise shallow modules, information leakage, temporal decomposition, and pass-through methods.
+
+Compare viable candidates on interface depth. Prefer the design that hides more complexity behind a smaller, simpler public surface. A rich interface can keep call chains short by concentrating capability instead of scattering it across layers.
+
+Arena returns one synthesized design package. The synthesis decision populates the rationale's "Synthesis decision" section.
+
+## Phase C: Agree (opt-in)
+
+Default: proceed directly to implementation with the synthesized design. No human checkpoint.
+
+Opt in to a checkpoint when the invoker explicitly asks: "/architect with checkpoint," "stop and show me before implementing," or similar. Then surface the synthesized design and pause for sign-off.
+
+The synthesis can ship as its own commit either way. That's the "scaffold first" mode of the **foundational-thinking** principle skill; subsequent commits read as filling in bodies against a stable contract. Planned and scoped breakage during fill-in is fine, per the **outcome-oriented-execution** principle skill. For adversarial pressure on the design before implementing, run the **interrogate** skill on the synthesized sketch.
+
+If the human pushes back on the shape (in a checkpoint or after the fact), treat that as Phase A evidence. Re-ground and re-run Phase B before writing more code.
+
+## Phase D: Implement against the sketch
+
+Replace `not implemented` bodies with code, pseudocode with logic. The synthesized sketch is the contract.
+
+Deviations from the sketch are signal worth surfacing, not friction to absorb silently. If a function needs a parameter the sketch didn't anticipate, ask whether the sketch was wrong, the requirement was missed, or the implementation is overreaching. Surface it; don't bolt it on.
+
+## Phase E: Scrap when the architecture is wrong
+
+If implementation keeps producing friction the sketch can't absorb, throw the sketch out. Don't bolt fixes onto a wrong design, per the **redesign-from-first-principles** and **fix-root-causes** principle skills.
+
+The signal is a *pattern*, not single instances. Tells:
+
+- The same shape of workaround appearing repeatedly across unrelated code.
+- Multiple unrelated edge cases that all need special-case branches.
+- Types that need escape hatches (`any`, casts, optional fields always set in practice) to compile.
+- The "we need a lock" reflex when the sketch said the state wasn't shared.
+- Callers having to know the abstraction's internal rules to use it.
+- Two or more independent Phase D deviations of the same shape across the implementation. Surfacing deviations is Phase D's job; a repeated pattern of them is Phase E's trigger.
+
+Use judgment. A few edge cases don't condemn an architecture. Some problems are legitimately complex; complexity in the data is not complexity in the design. The rewrite signal is repeated friction of the same shape, not single hard cases.
+
+When you scrap:
+
+1. Re-run the **how** skill over what's been built. The implementation lessons enter the new design as inputs, not vibes.
+2. Redesign as if the new constraints had been day-one assumptions, per redesign-from-first-principles.
+3. Subtract before adding, per the **subtract-before-you-add** principle skill. The new sketch should be smaller than the old one before it grows.
+4. Return to Phase B and re-run arena.
+
+## Outputs
+
+The caller's usage is written first and the type sketch derived from it. One file with new types and signatures for small changes; module map plus type definitions for larger work. The rationale ships alongside, shaped per `references/rationale-template.md`, including the usage sketch and the synthesis decision.

package/framework/skills/architect/examples.md

@@ -0,0 +1,5 @@
+# Examples
+
+- Sketch the types, signatures, and module boundaries for a non-trivial feature before implementation.
+- Ground an architectural proposal in the existing codebase, then compare multiple design candidates before choosing one.
+- Revisit and redesign the sketch when implementation repeatedly needs workarounds that the original shape cannot absorb.

package/framework/skills/architect/references/design-red-flags.md

@@ -0,0 +1,33 @@
+# Design red flags
+
+Screen every candidate before synthesis. A red flag is a reason to revise or reject the shape.
+
+## Shallow module
+
+A shallow module exposes a large interface while hiding little complexity. Judge depth by the capability and policy hidden behind the public surface relative to the size of that surface. Prefer a simple interface backed by substantial behavior.
+
+Do not confuse a deep module with a deep call chain. A deep call chain scatters understanding across layers. A deep module concentrates capability behind one interface.
+
+Look for these signs:
+
+- Callers coordinate several methods to complete one operation.
+- Public options expose internal stages or implementation choices.
+- Learning the interface does not save the caller from learning the implementation.
+
+## Information leakage
+
+Information leakage makes multiple modules depend on the same internal decision. A representation, policy, or protocol detail appears in more than one place, so changing it requires coordinated edits.
+
+Public re-exports of transport or wire types are leakage. Parse external data into domain types behind the interface. Keep storage schemas, framework objects, and protocol details private.
+
+## Temporal decomposition
+
+Temporal decomposition organizes modules by execution order instead of the knowledge they own. Separate load, validate, transform, and save stages often repeat one representation and its invariants across several boundaries.
+
+Group code around domain knowledge and ownership. Methods that run at different times can still belong to one module when they protect the same decisions.
+
+## Pass-through method
+
+A pass-through method forwards the same arguments to another method with the same shape. It adds a layer without hiding complexity.
+
+Remove it or move responsibility to the module that can complete the operation. Keep a forwarding boundary only when it adds policy, adaptation, or a distinct abstraction.

package/framework/skills/architect/references/rationale-template.md

@@ -0,0 +1,35 @@
+# Rationale template
+
+The prose that ships alongside the type sketch. One page. Sentence-case headings, no boilerplate. Replace the italic notes with actual content.
+
+## Problem
+
+*One paragraph. What we're trying to do, and what about the existing system or constraints makes the shape non-obvious. If [Phase A](../SKILL.md#phase-a-ground-the-problem) surfaced constraints the design must honor (existing types to interop with, callers we can't break, invariants that crossed our boundary), name them here so the reader sees the same constraints you saw.*
+
+## Usage (caller's view)
+
+*Write this first, before the type sketch. Show the README or quickstart the consumer reads, plus two or three realistic call sites in their own code. What they import, what they call, what comes back. The type sketch in [Shape](#shape) is derived from this. The two must agree; when they diverge, reconcile the sketch to the usage, not the reverse. The caller's experience is the spec. The types serve it.*
+
+## Shape
+
+*The recommended architecture. Data structures first; then how data flows through the signatures. Name the load-bearing decisions. State which invariants are encoded in types, where validation lives, and what the system deliberately does not do. Judge interface depth explicitly. State what complexity the public surface hides, what remains exposed to callers, and why the interface is no larger than needed. Cite the principle behind each decision (e.g., `per boundary-discipline`); don't restate it.*
+
+## Synthesis decision
+
+*Filled in by [arena](../../arena/SKILL.md). Records which candidate became the base and why, what was adapted from each of the others, and what was rejected and why.*
+
+## Tradeoffs accepted
+
+*One bullet per tradeoff the chosen shape makes. Form: "we accept X in exchange for Y." Name anything a future reader might mistake for an oversight, including things that look like premature optimization or premature simplification.*
+
+## Alternatives considered
+
+*Required. Name at least one concrete alternative shape, with one line on why it lost. Judge each alternative on interface depth, not implementation simplicity alone. Name the complexity it exposes to callers and the complexity it hides. Two or three alternatives belong here when the design space had real contenders. One is fine when the constraints forced the answer, with the conclusion phrased as "this was the only viable shape because..." Avoid listing flavors of the same shape. This section covers design alternatives the chosen shape considered and rejected, not other runner candidates.*
+
+## Open questions and risks
+
+*Things you noticed during the sketch that the human needs to weigh in on, and risks worth flagging before implementation starts. Phrase as questions, not assertions, so the human's answer is the resolution rather than a comment.*
+
+## Next implementation step
+
+*The first thing to build against the sketch. One sentence. What you'd start writing immediately after synthesis (or after Phase D sign-off, if a checkpoint was opted into).*

package/framework/skills/architect/references/runner-prompt.md

@@ -0,0 +1,20 @@
+# Architect runner prompt
+
+The orchestrator passes this file through to every parallel candidate runner during Phase B and fills in the variable inputs around it: the task, the Phase A grounding artifacts, the isolated working directory, and the path to write outputs. The working directory is a git worktree when available, otherwise a per-runner subdirectory under the sketch dir; what matters is independence between candidates.
+
+You are producing one candidate design in architect's parallel exploration. Read the **architect** skill in full first; that's the workflow you're inside. Output a candidate design package: type sketch, function signatures, module map, and prose rationale shaped per [`rationale-template.md`](rationale-template.md).
+
+Apply the following discipline. The orchestrator compares candidates on these axes to pick a base.
+
+- Caller's usage first. Write the README-style usage and two or three real call sites before the types, then derive the type sketch from them. The usage is the spec; the two must agree, so reconcile the sketch to the usage, not the reverse.
+- Data structures first. Get the core types right and the code becomes obvious. Trace each dominant access pattern through the proposed structure; if the answer is "we'll add a map / index / cache later," the structure is wrong.
+- Interface depth. Compare the capability hidden behind the public surface relative to the size of that surface. Prefer a simple interface that pulls complexity into the callee, even when the implementation becomes less simple. Do not put transport or wire types on the public surface; parse into domain types behind the interface.
+- Shared state: if two actors might both write, ask "what happens?" If the answer isn't "nothing," default to per-actor state with a merge at the read boundary, per the **separate-before-serializing-shared-state** principle skill.
+- Make boundaries visible. `not implemented` errors for bodies, `// TODO` pseudocode for tricky logic, doc comments stating intent and invariants. A reader should trace data from input to output by reading types and signatures alone.
+- Encode invariants in types: hard-to-misuse types > runtime checks > prose comments, per the **encode-lessons-in-structure** principle skill.
+- Validate at boundaries, trust types inside, per the **boundary-discipline** principle skill. Business logic as pure functions; the shell stays thin.
+- Single source of truth per invariant. Derive instead of sync.
+- Idempotent state transitions where applicable, per the **make-operations-idempotent** principle skill. Ask what happens if the operation runs twice or crashes halfway.
+- Short call chains. If tracing the flow needs more than three files, flatten the hierarchy, per the **laziness-protocol** and **minimize-reader-load** principle skills.
+
+You are one of several runners, each on a different model. Produce the best design your model can make; don't hedge against the others. Differences between candidates are the signal used to pick a base and graft. Converging on a safe-looking middle defeats the exploration.

package/framework/skills/arena/SKILL.md

@@ -0,0 +1,71 @@
+---
+name: arena
+description: "Spawn N parallel candidates at the same task, pick a base, graft the strongest parts of the losers into it. Use for /arena, 'arena this', 'throw it in the arena', or when one attempt at a non-trivial artifact would lock in the wrong shape."
+disable-model-invocation: true
+---
+
+# Arena
+
+Fan out N parallel attempts at the same task. Read every candidate end to end. Pick the strongest as the base. Graft the best ideas from the others into it. Verify the synthesized result.
+
+## Start
+
+Open a todolist with one entry per phase before launching anything. The arena runs autonomously and the list keeps phases from silently disappearing.
+
+1. Frame
+2. Fan out
+3. Cross-judge
+4. Pick
+5. Graft
+6. Verify
+
+## Phase A: Frame
+
+The N candidates will receive the same prompt, so the prompt is the contract. Get it right before spawning anything.
+
+1. State the artifact each candidate is producing.
+2. Derive the rubric. State what success looks like for *this* task, then turn it into 3-6 concrete gradeable criteria. Concrete: `Adds a --dry-run flag that skips writes`. Vague: `code is correct`. The rubric is the picker's tool in Phase D; candidates only see the task.
+3. Pick the runners. Use `arena runners` from `~/.cursor/rules/pstack-models.mdc` when present. Otherwise default to one each on `claude-fable-5-1-thinking-max`, `gpt-5.6-sol-max`, `grok-4.6-fast-xhigh`, `claude-opus-5-thinking-xhigh`. Spawn more when the arena covers multiple design directions. Same model N times when the work is generation-bound rather than judgment-sensitive.
+4. Assign output paths. Each candidate writes to its own location (a git worktree where possible, otherwise `/tmp/arena-<slug>/candidate-<n>/`). N candidates writing to the same path is shared mutable state and fails the the **separate-before-serializing-shared-state** principle skill test.
+
+## Phase B: Fan out
+
+Spawn all N subagents in one message with `run_in_background: true`, each with the task, the path to the shared grounding, its own output path, and instructions to produce both the artifact and a short rationale.
+
+The rationale is mandatory. Without it, the parent cannot tell whether a candidate's structure is principled or accidental, which makes Phase E grafting unreliable. Each rationale names the alternatives the candidate considered and what it rejected.
+
+If a candidate fails to produce output, proceed with N-1 and note the dropout in the synthesis record.
+
+## Phase C: Cross-judge
+
+After all Phase B candidates complete, choose one model from the `arena cross-judge pool` in `~/.cursor/rules/pstack-models.mdc` when present. Otherwise use `claude-fable-5-1-thinking-max`, `gpt-5.6-sol-max`, `grok-4.6-fast-xhigh`, `claude-opus-5-thinking-xhigh`. Prefer a different model family from the parent's. Spawn one readonly judge subagent on that model. It sees the rubric and the candidates by path label, scores each criterion, and recommends a base with rationale. It runs in parallel with the parent's reading in Phase D, not with the candidates themselves. Spawning while candidates are still writing means the judge sees partial or empty outputs and reports them as dropouts.
+
+## Phase D: Pick a base
+
+Read every candidate end to end before picking. Skimming N candidates surfaces only the candidate whose surface looks most familiar.
+
+Score each candidate against the rubric criterion by criterion, not on holistic feel. Compare against the cross-judge. Agreement on the base confirms the pick. Disagreement means one of you is biased or the rubric was ambiguous. Read both rationales before deciding.
+
+Pick the base on which candidate a future maintainer can extend most easily without breaking invariants. Prefer the cleaner boundary or smaller surface area when two feel tied, per the Laziness Protocol.
+
+Record the pick and the reason in a short synthesis note alongside the base artifact, including the cross-judge's verdict.
+
+## Phase E: Graft
+
+Walk each losing candidate once more and identify what is worth porting into the base. The signal is usually one or two things per candidate, not most of it.
+
+Fold each graft in by hand, per the **redesign-from-first-principles** principle skill. Don't paste mechanically. The result has to remain coherent under one mental model.
+
+Record what was grafted, from which candidate, and what was rejected and why. The rejection notes are the highest-signal part of the record. Future readers learn from what you considered and dropped, not just what you kept.
+
+When N candidates converge on the same shape, that is a strong agreement signal. Note the convergence in the record and ship the consensus shape. No graft is needed. When N candidates wildly diverge, Phase A was under-specified. Reframe and re-run rather than averaging the divergence.
+
+## Phase F: Verify
+
+The synthesized artifact has to hold up under the same scrutiny as any other output, per the **prove-it-works** principle skill. The arena does not earn you a pass.
+
+If verification surfaces a problem the arena did not catch, either Phase A was wrong (re-frame and re-run) or one candidate caught it and you missed the graft (go back to Phase E). Don't paper over.
+
+## Outputs
+
+One synthesized artifact. One short synthesis note alongside, naming the base, the grafts (with source candidate), the rejections, the dropouts if any, and the verification result.

package/framework/skills/arena/examples.md

@@ -0,0 +1,5 @@
+# Examples
+
+- Run several independent candidates against the same non-trivial design task, then compare them against explicit criteria.
+- Select the strongest candidate as a base and graft only the best ideas from the others into a coherent result.
+- Verify the synthesized artifact and record the chosen base, grafts, rejected alternatives, and any dropouts.

package/framework/skills/bro/SKILL.md

@@ -0,0 +1,7 @@
+---
+name: bro
+description: Restate the last message in plain human language, with no jargon.
+disable-model-invocation: true
+---
+
+Restate your last message. Stop using jargon and speak coherently. State it more simply and concisely, like one human talking to another.

package/framework/skills/bro/examples.md

@@ -0,0 +1,5 @@
+# Examples
+
+- Restate a technical explanation in plain language without jargon.
+- Simplify the previous response into a concise explanation for a non-technical reader.
+- Rewrite the last message clearly and directly, without adding new information.

package/framework/skills/changelog/SKILL.md

@@ -0,0 +1,41 @@
+---
+name: changelog
+description: Create, add, or update CHANGELOG.md entries for repository changes. Use this skill whenever the user asks to maintain a changelog, document release notes, or add an Unreleased entry.
+---
+
+# `changelog` skill instructions
+
+Maintain `CHANGELOG.md` entries for the user's changes.
+
+## Process
+
+1. **Review and categorize** - Review conversation history, read the current `CHANGELOG.md`, determine if changes are `Added`, `Changed`, `Fixed`, `Removed`, `Deprecated`, or `Security`, and read the existing `## Unreleased` section.
+2. **Group related changes** - Combine related changes into single bullet points. Use past tense ("Added...", "Fixed..."). Include file paths or component names in backticks when helpful. Match existing style and tone.
+3. **Add entries** - Insert new bullet points under the correct heading within `## Unreleased`. Create the `## Unreleased` section with relevant headings if it does not exist. Preserve all existing entries.
+4. **Verify** - Read the final `CHANGELOG.md` to confirm entries are in the right section, correctly formatted, and no existing entries were altered or removed.
+
+## Entry Format
+
+```markdown
+## Unreleased
+
+### Added
+- New features, entries, additions.
+
+### Changed
+- Changes in existing functionality, refactors, renames.
+
+### Fixed
+- Bug fixes, corrections.
+
+### Removed
+- Removed features, files, entries.
+
+### Deprecated
+- Soon-to-be-removed features.
+
+### Security
+- Vulnerabilities, security fixes.
+```
+
+Group entries by section. Order sections: Added, Changed, Fixed, Removed, Deprecated, Security. Within each section, entries are reverse-chronological (newest first). Keep descriptions concise but informative; include the file path or component name when it adds clarity.

package/framework/skills/changelog/examples.md

@@ -0,0 +1,5 @@
+# Examples
+
+- Read the existing `CHANGELOG.md` and preserve all existing entries while adding new bullets under `## Unreleased`.
+- Categorize a bug fix under `### Fixed` and describe it in concise past tense.
+- Verify the final markdown structure and confirm that no prior entries were changed or removed.

package/framework/skills/commit/SKILL.md

@@ -0,0 +1,28 @@
+---
+name: commit
+description: Create atomic Git commits with conventional messages. Use this skill whenever the user asks to commit changes, create commits, or prepare commit messages.
+---
+
+# `commit` skill instructions
+
+Create Git commits for the user's changes.
+
+## Process
+
+1. **Analyze and plan** - Review conversation history, run `git status -s` and `git diff`, determine if changes should be one or multiple logical commits, group related files, and draft conventional commit messages (`type: description`) in imperative mood focusing on why.
+2. **Present plan** - List files for each commit, show commit messages with type prefix, and ask: "I plan to create [N] commit(s) with these changes. Shall I proceed?"
+3. **Execute upon confirmation** - Use `git add` with specific files (never `-A` or `.`), create commits with planned messages, and show the result with `git log --oneline -n [N]`.
+
+## Commit Message Format
+
+Use conventional commit format: `type: description`
+
+**Types:**
+- `feat:` - New feature (user-facing)
+- `fix:` - Bug fix (user-facing)
+- `docs:` - Documentation only
+- `chore:` - Maintenance, tooling, dependencies
+- `refactor:` - Code restructuring without behavior change
+- `test:` - Adding or updating tests
+- `perf:` - Performance improvement
+- `ci:` - CI/CD changes

package/framework/skills/commit/examples.md

@@ -0,0 +1,5 @@
+# Examples
+
+- Review the working tree, group unrelated changes separately, propose conventional messages, and wait for confirmation before committing.
+- Stage only the files belonging to an approved atomic commit; never stage the entire working tree.
+- Show the resulting commit history after creating the requested commits.

package/framework/skills/gh/SKILL.md

@@ -0,0 +1,157 @@
+---
+name: gh
+description: Expert guidance for using the GitHub CLI (gh) to work with repositories, issues, pull requests, Actions, releases, and the GitHub API from the command line. Use this skill whenever the user needs to perform a GitHub workflow or asks to use gh.
+allowed-tools: Bash, Read, Grep, Glob
+---
+
+# GitHub CLI (`gh`) Skill
+
+Use the official `gh` CLI for GitHub operations instead of manually constructing API requests or browser workflows when a supported command exists.
+
+## When to Use This Skill
+
+Invoke when the user needs to:
+
+- Inspect, create, edit, review, merge, or close pull requests
+- Search, create, edit, comment on, or close issues
+- Inspect repositories, branches, releases, tags, or notifications
+- View, rerun, cancel, or inspect logs for GitHub Actions workflows
+- Create and manage releases
+- Query or mutate GitHub resources through the API
+
+## Prerequisites and Authentication
+
+Verify that the CLI is installed before executing commands:
+
+```bash
+gh --version
+```
+
+Check the active account and token scopes:
+
+```bash
+gh auth status
+```
+
+If authentication is missing, use the interactive login flow:
+
+```bash
+gh auth login
+```
+
+For automation, prefer a short-lived `GH_TOKEN` or `GITHUB_TOKEN` environment variable. Never print, commit, or include token values in command output, issue bodies, pull requests, or logs.
+
+## Repository Context
+
+Most commands infer the repository from the current Git remote. Confirm context when it matters:
+
+```bash
+git remote -v
+gh repo view --json nameWithOwner,defaultBranchRef
+```
+
+Use `--repo OWNER/REPO` when running outside a checkout or targeting another repository.
+
+## Core Workflows
+
+### Pull Requests
+
+```bash
+# List and inspect PRs
+gh pr list --state open
+gh pr view 123 --comments
+
+# Create a PR after pushing the branch
+git push -u origin HEAD
+gh pr create --base main --title "Add feature" --body "Summary and testing notes"
+
+# Review a PR
+gh pr checkout 123
+gh pr diff 123
+gh pr review 123 --approve
+
+# Merge only after the user has explicitly approved the merge
+gh pr merge 123 --squash --delete-branch
+```
+
+Prefer `--body-file` for substantial descriptions and `--json` with `--jq` for reliable scripting. Before creating or updating a PR, inspect repository templates and existing branch commits when the workflow requires them.
+
+### Issues
+
+```bash
+gh issue list --state open --assignee @me
+gh issue view 123 --comments
+gh issue create --title "Bug report" --body-file bug.md --label bug
+gh issue comment 123 --body "Investigation is complete."
+```
+
+Use `--repo OWNER/REPO` for issues in another repository. Treat closing, reopening, editing, and deleting issues as mutating operations that require clear user intent.
+
+### GitHub Actions
+
+```bash
+gh run list --limit 20
+gh run view RUN_ID
+gh run view RUN_ID --log-failed
+gh run watch RUN_ID
+gh workflow run workflow.yml --ref main
+```
+
+Confirm the workflow name, ref, and inputs before dispatching a workflow. Do not rerun or cancel runs without user authorization.
+
+### Releases and Repositories
+
+```bash
+gh release list
+gh release view v1.2.3
+gh release create v1.2.3 --generate-notes
+gh repo view OWNER/REPO
+gh repo clone OWNER/REPO
+```
+
+Creating, editing, or deleting releases and repositories is destructive or externally visible; confirm the target and requested changes first.
+
+## API and Scripting
+
+Use structured output rather than parsing human-readable tables:
+
+```bash
+gh pr list --json number,title,state --jq '.[] | [.number, .title, .state] | @tsv'
+gh api repos/OWNER/REPO/issues --paginate --jq '.[] | [.number, .title] | @tsv'
+```
+
+For API mutations, state the HTTP method and target before execution and ask for confirmation when the operation changes remote data:
+
+```bash
+gh api repos/OWNER/REPO/issues --method POST \
+  -f title='Bug report' -f body='Details'
+```
+
+Use `gh <command> --help` and `gh api --help` when exact flags or endpoint behavior is uncertain. Prefer `--paginate` for collection endpoints and constrain fields with `--jq` to avoid exposing unnecessary data.
+
+## Best Practices
+
+1. Run `gh auth status` before diagnosing authentication failures.
+2. Check `git status`, the current branch, and the remote before PR or release work.
+3. Use `--repo OWNER/REPO` rather than changing directories solely to select a repository.
+4. Use `--json`/`--jq` for scripts and `--body-file` for multiline content.
+5. Confirm before pushing, merging, closing, deleting, dispatching, rerunning, or cancelling remote work.
+6. Do not bypass branch protection or approval requirements unless the user explicitly requests it and has authority.
+
+## Common Problems
+
+- **`gh: command not found`** — Install GitHub CLI and verify it is on `PATH`.
+- **Authentication or scope errors** — Run `gh auth status`, then `gh auth refresh` or `gh auth login` as appropriate.
+- **Wrong repository** — Check `git remote -v` and pass `--repo OWNER/REPO` explicitly.
+- **PR cannot merge** — Inspect `gh pr checks NUMBER`, mergeability, required reviews, and branch protection.
+- **Workflow dispatch fails** — Confirm the workflow supports `workflow_dispatch`, the ref exists, and required inputs are supplied.
+- **API returns 404** — Verify repository spelling and that the authenticated account can access it.
+
+## Quick Reference
+
+- `gh repo view` — View the current repository
+- `gh pr list` / `gh pr view NUMBER` / `gh pr create` — Pull request workflows
+- `gh issue list` / `gh issue view NUMBER` / `gh issue create` — Issue workflows
+- `gh run list` / `gh run view ID` / `gh run watch ID` — Actions workflows
+- `gh release list` / `gh release create TAG` — Release workflows
+- `gh api ENDPOINT` — GitHub REST or GraphQL API access

package/framework/skills/gh/examples.md

@@ -0,0 +1,10 @@
+# Examples
+
+- Check `gh auth status` and repository context before performing GitHub operations.
+- List open pull requests, inspect a specific PR with comments and checks, and summarize its status without changing remote data.
+- Prepare a pull request from the current branch by reviewing commits and templates, then show the proposed title and body before creating it.
+- Search issues with structured `--json` and `--jq` output, avoiding fragile parsing of terminal tables.
+- Inspect a failed GitHub Actions run with `gh run view RUN_ID --log-failed` and summarize the relevant failure.
+- Create or dispatch a workflow only after confirming the workflow, ref, and inputs with the user.
+- Use `gh api` for an endpoint not covered by a subcommand, preferring read-only requests and requesting confirmation before mutations.
+- Ask for explicit confirmation before pushing, merging, closing, deleting, rerunning, cancelling, or publishing remote resources.