AGENT SKILL

patina-mode

Patina Project's engineering style for concise, detailed responses, deliberate subagents, unslopped prose, simple code, and verified work. Use for /patina-mode or requests to work in this style.

License 许可
MIT
最近更新
2026-11-12
Stars
★ 1
Forks
1

01 前言 Front Matter

name
patina-mode
description
Patina Project's engineering style for concise, detailed responses, deliberate subagents, unslopped prose, simple code, and verified work. Use for /patina-mode or requests to work in this style.

02 资源文件 89

03 SKILL.md 正文

Patina mode

Platform Adaptation

These skills use Claude Code tool names (the Skill tool, the Agent tool, AskUserQuestion) and the Claude model names the Agent tool accepts, as each Models section lists them. On Claude Code they work as written. On Codex, read references/codex-tools.md for the Codex equivalent of a Claude tool, model, or skill named by these workflows. On Pi, read references/pi-tools.md instead. On GitHub Copilot, CLI or app, read references/copilot-tools.md instead. Other runtimes can discover the same Agent Skills tree, but they must use their own tool, model, and configuration equivalents. No one of these mappings is a cross-runtime map.

On Codex, patina-mode requires the multi_agent feature. Before selecting a playbook or changing anything, confirm that spawn_agent is available. If it is unavailable, stop and tell the user to set multi_agent = true under [features] in ~/.codex/config.toml, then restart Codex. Do not continue with a sequential substitute.

Use the session's task-tracking tools for the todolist. On Claude Code these are TaskCreate and TaskUpdate, or TodoWrite when configured. Availability depends on the model; CLAUDE_CODE_ENABLE_TODO_TOOLS=1 enables them through the environment or a project's .claude/settings.local.json env block. See the task-tool documentation. If no task-tracking tool is available, keep an uncommitted todo.md Markdown checklist in the work dir, next to the decision trail, with the playbook steps verbatim and each skip: <reason> line. When several sessions share the checkout, name it .audit/<task-slug>.todo.md, beside that session's .audit/<task-slug>.tsv trail, so they never write one file.

Playbooks

Route before you act. Match the task to a playbook below and open its file. Open a todolist whose first items are its steps, copied in verbatim, before any task-specific todos. Do this before the first file edit or commit. A step you choose not to do stays in the list with a one-line skip: <reason>.

A code change routes to its build playbook: Bug fix, Feature, Refactoring, Perf issue, Hillclimb, or Visual parity. A request to implement an issue and open a pull request is that build task when neither continuation case below holds. Opening a PR is the build playbook's last step, never the route for the code. Prototype sketches and Babysit repairs keep their own playbooks. Who writes the code follows A subagent writes the code in Subagents.

Issue-linked work that continues a branch routes through Session pickup. After working-on-issues passes, check the remote canonical branch it resolved and fetched. When that branch has an open pull request, Session pickup routes the work to the Babysit playbook. Babysit step 1 picks the mode, and an undeclared mode is drive. The unattended issue run counts as the request that starts babysitting, and the issue run's own thread runs Babysit, not a phase subagent. When the branch has commits beyond the default branch and no open pull request, Session pickup routes the work to the remaining steps of the build playbook that the work matches, then to Opening a PR. When the branch does not exist or has no commits beyond the default branch, the build route above applies unchanged.

Every repository change an issue run makes routes through a playbook, including docs, tests, config, and dependency updates. Route each by its effect:

  • An edit to a SKILL.md, a playbook, or another agent-facing doc goes to Authoring or modifying a skill.
  • A test or config change that keeps behavior goes to Refactoring. Its step 1 may take a skip: line when there is no behavior to pin.
  • A dependency update, or a config change that alters behavior, goes to Bug fix or Feature by its effect.
  • Anything else goes to figure-it-out.

A large or cross-cutting effort (a migration across many call sites, an ambitious multi-part change), or work the user steps away from to trust later, routes to the figure-it-out skill even when a narrower playbook like Feature fits. Use figure-it-out whenever no bundled playbook fits. It designs a bespoke, rigorous playbook for the task. A standing project-scale program (multi-day, many stacked PRs, a fleet of subagents under one coordinator) routes to Orchestrate instead. figure-it-out designs one bespoke run, orchestrate runs the program.

  • Investigation. Read-only question: how does X work, why was Y built this way, are we sure about Z, should we do X or Y. playbooks/investigation.md.
  • Triage. Unattended triage of one existing tracker issue to a ready label or one round of questions, without implementing it ("triage this issue", "Triage Linear issue … Do not implement it."). People triaging by hand use the Work plugin's triage skill instead. playbooks/triage.md.
  • Bug fix. A reported defect to reproduce, root-cause, and fix with runtime evidence. playbooks/bug-fix.md.
  • Perf issue. A measured slowness to trace and improve against a baseline. playbooks/perf-issue.md.
  • Hillclimb. Sustained, scientific improvement of one metric against a target: loop hypotheses with before/after measurement, a decision log, and one commit per accepted win. Distinct from Perf issue, which is a one-off fix. playbooks/hillclimb.md.
  • Runtime forensics. Diagnose a runtime symptom (leak, idle-CPU spin, glitch) from live instrumentation. The deliverable is a diagnosis, not a fix. playbooks/runtime-forensics.md.
  • Trace forensics. Diagnose a captured profiling artifact (cpuprofile, trace, spindump, heap snapshot) handed to you after the fact. The deliverable is a diagnosis, not a fix. playbooks/trace-forensics.md.
  • Feature. New or changed behavior, built from a named data shape. playbooks/feature.md.
  • Refactoring. A behavior-preserving change to structure or shape (rename, extract, inline, dedupe, move). playbooks/refactoring.md.
  • Prototype. A throwaway sketch to make a design or behavioral decision cheaply, or to settle an empirical fork by observing it instead of asking the human ("prototype", "mock it up", "try this layout", "sketch it to decide"). playbooks/prototype.md.
  • Visual parity. Pixel-exact UI equivalence: matching two implementations or migrating a styling system. playbooks/visual-parity.md.
  • Authoring or modifying a skill. Writing or editing a SKILL.md. playbooks/authoring-a-skill.md.
  • Eval. Testing how a skill, structure, or prompt change affects agent behavior before promoting it. playbooks/eval.md.
  • Babysit. Driving a PR or a stack to merge-ready: conflicts, review threads, CI. playbooks/babysit.md.
  • Shipping. The half after Babysit. Independently verifying a green stack, then landing the contiguous verified run bottom-up through gh by default or Origin when its CLI is available. playbooks/shipping.md.
  • Autonomous run. A long task to drive to completion without stopping ("run until done", "/loop until X"). playbooks/autonomous-run.md.
  • Orchestrate. A standing project handed to one coordinator chat: multi-day, many stacked PRs, dozens to hundreds of subagents, minimal human turns ("run this whole project", "own this migration until it lands"). Distinct from Autonomous run, which drives one task to a predicate. Work one agent could finish inside the session's budget routes there, not here, however program-shaped the phrasing sounds. playbooks/orchestrate.md.
  • Autopilot-full. A queue of independent PRs driven to merge-ready with full autonomy. One owner per PR carries build to merge-ready, the root swarm-verifies each PR, an owner runs pr-readiness publish only after the root's clean verdict, and the operator clicks the merge of each PR that publish does not arm ("autopilot this queue", "full autopilot", one-owner-per-PR programs). playbooks/autopilot-full.md.
  • Autopilot-stack. A queue of changes built and verified with full autonomy, delivered as one linear reviewed base-branch stack the operator lands ("autopilot-stack", "stack them, don't ship", "build the stack, I'll land it"). playbooks/autopilot-stack.md.
  • Session pickup. Resuming or taking over a prior agent's in-flight work from a transcript, cloud-agent URL, or pushed branch. playbooks/session-pickup.md.
  • Pause safely. Suspending in-flight work cleanly so it can be resumed, on an explicit pause, going offline, a session restart, or imminent context compaction. The complement to Session pickup. Full steps: playbooks/pause-safely.md.
  • Multi-phase or multi-PR plan. Work that spans phases or stacked PRs. playbooks/multi-phase-plan.md.
  • Worktree and simulator cleanup. Reclaiming local disk by pruning merged or abandoned git worktrees and stale iOS simulators ("what's using my disk", "clean up worktrees", "prune safe-to-prune worktrees", "free up space", "delete old simulators"). playbooks/worktree-cleanup.md.
  • Opening a PR. Invoked at the end of every other playbook. It routes alone only for work already built. playbooks/opening-a-pr.md.

Project playbooks. A repository can add playbooks of its own under .agents/playbooks/, one Markdown file each. Its frontmatter names extends, the bundled playbook stems it builds on (comma-separated, or empty for a standalone playbook), and when, one sentence naming the requests it serves. When the playbook you match is one a project playbook extends, or the task fits a project playbook's when, open that file as well. Copy the bundled steps into the todolist verbatim, then apply each change at the step its quoted text names, and stop where the project playbook says to stop. A change is a list item that starts with **After**, **Before**, **Replace** or **In** followed by a quoted run of the bundled step's own words. Before you apply a project playbook, run node <patina-mode-base>/scripts/check-playbooks.mjs from the repository root. It fails when a project playbook extends a playbook this version lacks, or quotes step text this version no longer has, which happens when a pstack upgrade rewrites a step. Report each line it prints to the user and do not guess where an unanchored change belongs.

Non-negotiables

The Principles section below grounds every trigger. Keep principle choices, routing decisions, skip reasons, and throughput notes in the private task record. Explain them to the user only when asked or when the reasoning affects a decision the user must make. Cite only principles whose leaf SKILL.md you read this session.

While patina-mode is active, this skill owns workflow routing. Use only the skills and playbooks named here. Do not substitute another development, repair, review, or pull request controller because it is installed.

Every skill named in this document and its playbooks is the Code plugin's. Invoke it by its bare name, such as patina-mode. An Agent Skills install registers bare names only, so a plugin:skill name for a Code skill does not resolve there. The exceptions are the references already carrying their own namespace, such as plugin-dev:skill-development, and the Claude Code built-in driver skills run and verify. On Codex, see references/codex-tools.md.

Every scripts/... and playbooks/... path in this document, its playbooks, and its references is relative to this skill's base directory, the absolute path your runtime states when it loads patina-mode. Join the two, then run or read the result. The bundled watch-pr watcher, orch store CLI, and worktree-audit.mjs live under scripts/ there. Two other anchors look plausible and both fail. CLAUDE_PLUGIN_ROOT is set only for hooks, so it is unset in any shell you run. The repository you are working in ships none of these files, so a repo-relative path resolves to nothing.

Remaining triggers:

  • Issue-linked work begins or resumes → follow references/issue-handoff.md. Its executable direct-work and Session-pickup entries call the same gate. Each entry requires a successful working-on-issues result before another playbook changes a branch, edits a file, creates a commit, changes a pull request, or starts an operational run. Its completed-issue, blocker, branch-ownership, and worktree gates override the general autonomy rules below.
  • An issue branch belongs to another worktree → let working-on-issues stop at its handoff gate. Run the bundled move-branch-here skill only when the operator approves the move.
  • A Claude session ID, codex://threads/<id>, or T3 Code thread handoff → run the bundled move-session-here skill, then route its recovered resume point through the Session pickup playbook. Session pickup enters the same issue handoff gate.
  • Nontrivial change, architecture decision, or "are we sure?" → the how skill.
  • About to AskUserQuestion on a "which approach", "how should I", or "what should this do" fork → classify it before you ask. If the answer is a fact you could observe by running something (behavior, timing, layout, output, perf, even whether an eval separates), it is not the human's to answer. Sketch it via the Prototype playbook (playbooks/prototype.md) and let the result decide. If the task is a read-only Investigation whose deliverable is a cited answer, stay in it and answer from the evidence rather than building a sketch. Reserve the question for a genuine product or preference call no experiment can settle. When you recommend or ask about design options, start from the real problem and its constraints. Present the best long-term solution first, with its cost stated up front. Offer a quick fix only as an optional first step, and label it as optional. This rule orders the options and picks the recommendation. The Laziness Protocol still sets how small the chosen option's diff is. Under a full-autonomy grant, decide a call that the grant covers, act on it, and report it, with no reply word and no offer. Under the grant, apply a default for a call that only the operator can make. For a design call, that default is the long-term option. Report it with its cost, the optional quick fix, and a full explanation, and say in plain words what the operator could tell you to do instead. The operator answers in their own words. Never give a shorthand token to type back. Gates that the operator named and the Always-pause list in Autonomy still need the operator.
  • Any code change → open its build playbook per Playbooks before the first source edit. In an issue run, a docs, test, config, or dependency change routes too, as Playbooks names. Name the data shape first, and choose its organizing structure per principle-model-the-domain.
  • A consequential choice about ownership or interfaces, or competing viable designs → the architect skill before implementing. A function boundary alone does not trigger Architect. When requirements and the implementation pattern are established, follow the matched playbook. Escalate if later evidence reveals unresolved risk, material uncertainty, or competing approaches.
  • Parallel fan-out → the swarm skill for coverage matrices, races, gauntlets, and exploration partitions. Use arena for design or code bakeoffs with base selection and grafting.
  • Stacked PRs, a GitHub stack suggestion, or a stack topology change → read references/github-stacks.md. It distinguishes dependent branches from native stack metadata and covers creation and verification.
  • Contested design → the interrogate skill (multi-model adversarial) before shipping.
  • Nontrivial multi-step → write the throughput checkpoint (Feature step 3) in the private task record.
  • Any prose surface → the unslop skill. Your reply is a prose surface. Write it per Writing the reply. Agent-facing prose also follows the plugin-dev:skill-development skill (Claude Code's authoring guidance for SKILL.md files).
  • Docs, RFCs, readmes, PR descriptions, commit messages → the technical-writing skill (/technical-writing) for structure and sentence discipline, on top of unslop.
  • A PR body is created or rewritten → pr (../pr/SKILL.md). It owns body structure, including when another writing skill applies.
  • Before commit → the deslop skill (/deslop).
  • Before review → the no-comments skill (/no-comments).
  • Running a benchmark, measuring perf yourself, or reporting a speedup or regression you measured → the benchmark-checklist skill before you report or act on the number.
  • Shipping UI / IDE / CLI → the project's driver skill, found per Project verification, for user-observable UI verification, or run for CLIs/TUIs. For bug fixes, reproduce first on the same surface yourself. Hand to the user only under the narrow Bug fix step 1 exception.
  • A human-authored change request or QA finding needs resolution → run the gather-evidence skill before editing or preparing a response. Route a confirmed case to the matching playbook, then run gather-evidence again on the changed target. Leave replies, thread resolution, and review state to the operator. Automated review uses the existing bot triage instead.
  • Work uses an Android emulator or iOS simulator → run the running-mobile-simulators skill before the first device state change. The agent that controls the device owns one exact device lease through app execution, automation, evidence capture, recovery, and cleanup.
  • Opening or preparing a PR for work already built → follow playbooks/opening-a-pr.md. Create the caller-selected mode, run /deslop, /no-comments, then code-review, resolve its findings on the current head, wait for a maintainer's approval at the head unless the repository is checks-only, and use pr-readiness publish for the only ready transition. Only publish arms auto-merge, or merges a PR GitHub already reports as mergeable, under the repository's autoMerge opt-in.
  • Any PR-status request → the Babysit playbook (playbooks/babysit.md), not the bundled babysit skill, whose description matches the same words. That includes "babysit this", "get it green", "address the review-bot comments", and the commonest phrasing, "check on PR X" / "anything outstanding on X". Never triggered by merely opening a PR. Declare its mode before polling. The playbook's step 1 owns the request-to-mode mapping. Reaching for drive inside a phase agent stops that agent finishing its turn.
  • A failed or reported CI check → identify its owner, reproduce it with the failure command, and route the repair to the fix-ci skill (../fix-ci/SKILL.md). Carry the owner, reproduction scope, command, and evidence into the routed brief, then return fix-ci's verification evidence to the active workflow. Each attempt follows Repair from evidence. Defect, review, and Babysit routing remain unchanged.
  • Asked to land or ship a green stack → the Shipping playbook (playbooks/shipping.md). Green is not safe. Nothing gets armed before an independent per-PR verdict, and only the contiguous verified run from the root lands.
  • An automated PR-review bot or the agentic security review commented → skeptical posture. They catch real bugs and also file non-issues and nitpicks, so assess each on its merits and dismiss noise with a concrete reason instead of churning code. Triage fix / dismiss / ask per references/bugbot-triage.md.
  • Deploying to a managed platform (Railway, Fly, Vercel, Heroku, and the like) → load that platform's skill, project-local or installed, before running its CLI, the same way Shipping step 1 resolves the forge before the first PR operation.
  • A defect found mid-task → severity decides its artifact, not where it turned up. A correctness or data gap gets a tracked issue even when it surfaces while writing a closure doc; a cosmetic margin can stay in the doc.
  • Broken skill mid-task → fix it in its own PR. Don't block. Don't silently work around it.
  • Long, autonomous, or multi-phase work, or any task the user steps away from to review later ("going to bed", "trust it when i'm back", "/loop until X") → a decision trail via the show-me-your-work skill. Commit it when stakes need an auditable record. Keep it local otherwise.

Project verification

When behavior verification starts, find the project's driver skill and load it before selecting a platform driver. The driver has no fixed name. Find it by name pattern and by what it does:

  1. A candidate is a skill in a project skill directory, such as .claude/skills/, .agents/skills/, .github/skills/, or .pi/skills/, at the repository root or in a monorepo package, whose directory name matches *-verify or verify-*, or equals verify. A built-in or plugin skill never counts, so Claude Code's bundled /verify is never a candidate.
  2. A candidate is the driver only when its SKILL.md tells an agent how to launch or drive the app. A skill named lint-verify that only runs a linter is not a driver.
  3. Count a skill once by its resolved real path. Installs symlink one skill into several skill directories.
  4. When several drivers remain, take the one in the package nearest the files being changed, or at the repository root when nothing has changed yet. Then prefer the one with a feature map, then the first by full path in byte order.

create-verification-skill writes the driver as project-verify. Older installs named it patinaproject-verify or verify, and the pattern still finds both.

Follow the selected skill's launch, doctor, affected-feature, evidence, and cleanup instructions. Use generic drivers only when no project skill exists, and record that coverage limitation.

Carry routing exceptions through the route

Pass each scoped routing exception in every executor brief. Require the executor to carry them into further delegation. Name the affected child requirement in the brief:

  • Skip the Feature or Refactoring Architect step only when no consequential ownership, interface, or competing-design decision remains. A straightforward change still follows every other matched playbook step, including a required configured implementation delegate, isolated worktree, and parent review.
  • Autopilot-full owners and supervisors inspect a slow worker before acting. Inspect process state, output, and task-relevant progress. Elapsed time or the lack of a side effect alone does not prove failure. Replace the worker only after recording failure evidence or when an explicit deadline contract requires replacement. Do not invent a deadline.
  • Reporting executors keep the outcome, relevant verification, and unresolved decisions. They omit required principle, routing, and throughput narration unless the user requests it or the reasoning affects the user's decision. Preserve any child reply item that communicates a result, evidence, risk, or decision.
  • Publication executors follow the repository's instructions for titles, bodies, references, links, and readiness. Under patina-mode, the caller selects the creation mode; /deslop, /no-comments, and code-review run in order; readiness counts only GitHub Actions checks and, in an approval-gated repository, a maintainer's approval read from the forge, which no agent can give; and pr-readiness publish is the sole ready transition and the only agent step that arms auto-merge or merges an already-mergeable PR, under the repository's autoMerge opt-in.

These exceptions apply only to work routed through patina-mode and only to the named child requirements. They do not override higher-priority user or repository instructions, and they do not change standalone child invocations. They never waive reproduction, evidence, issue ownership, authorization, configured model routing, writer isolation, a required independent review, or a current-head verification gate.

Repair from evidence

Every CI or pull request repair attempt starts from a diagnosis. Apply these rules before each edit and each push, including the first attempt after a resume.

  1. Read the owning job. Open the first failing job that owns the failure. Read its failing command and its first actionable error. An aggregate check such as Test Gate only reports that a required job did not succeed. It is never the root cause, so follow it to the owning job.
  2. Classify the failure. Name one class: PR code, test or fixture, base branch, workflow, environment, or flake. Only PR code and test or fixture failures get a commit on this branch. A base-branch failure needs an update from the base. A workflow or environment failure goes to its owner. A flake gets the one fresh build that step 7 of playbooks/babysit.md allows.
  3. Reproduce before you change code. Run the owning command locally on the current head and watch it fail. A local pass means the failure is not yet proven to be a code failure, so classify it again. When the command cannot run locally, name the reason and treat the next CI run as one experiment for one hypothesis.
  4. Wait for the current head. A pending or queued check on the current head is evidence to wait for. A new push cancels or supersedes that run and discards its result. Push again only after the decisive check finishes.
  5. Test one hypothesis per attempt. State the hypothesis, make the one focused change that tests it, and verify it locally before the push. When the result refutes the hypothesis, revert the change it motivated.
  6. Verify the requirement. Green checks prove only what the checks run. Verify the user-visible behavior on the surface that the requirement names, with the driver skill for that surface.
  7. Carry the attempt forward. A resumed or delegated repair brief names the previous attempt, the rejected hypotheses, and the known non-causes, so the next agent starts from them.
  8. Stop when the next action is a guess. Report the owning job, the class, the evidence you have, and the evidence that is missing. Resume when that evidence arrives.

Principles

Read the leaf skill in full for any principle you apply. Each entry names when it applies.

Core

  • Laziness Protocol (principle-laziness-protocol). Refactoring, sizing a diff, or tempted to add abstractions, layers, or signal threading. Bias to deletion and the smallest change that solves the problem.
  • Foundational Thinking (principle-foundational-thinking). Before writing logic: core types and data structures, scaffold-vs-feature sequencing, what concurrent actors share.
  • Redesign from First Principles (principle-redesign-from-first-principles). Integrating a new requirement into an existing design. Redesign as if it had been foundational from day one.
  • Attack the Premise (principle-attack-the-premise). Two or more fixes that share one premise have failed the same gate. Take a census of which actors hold the imbalance before the next fix, then question the premise instead of writing another fix that assumes it.
  • Subtract Before You Add (principle-subtract-before-you-add). Sequencing an addition, refactor, or rewrite. Remove dead weight first, then build on the simpler base.
  • Minimize Reader Load (principle-minimize-reader-load). Reviewing or shaping code that's hard to trace. Count layers and hidden state, collapse one-caller wrappers, shrink mutable scope.
  • Outcome-Oriented Execution (principle-outcome-oriented-execution). Planned rewrites and migrations with explicit phase boundaries. Converge on the target architecture, don't preserve throwaway compatibility states.
  • Experience First (principle-experience-first). Product, UX, or feature-scope tradeoffs. Choose user delight over implementation convenience.
  • Exhaust the Design Space (principle-exhaust-the-design-space). A novel interaction or architectural decision with no precedent. Build 2-3 competing prototypes and compare before committing.
  • Build the Lever (principle-build-the-lever). Any non-trivial work. Build the tool that does or proves it (codemod, script, generator), not by hand. The tool is the artifact a reviewer reruns.

Architecture

  • Model the Domain (principle-model-the-domain). Writing stateful logic, or code that branches a lot or repeats a shape assumption across files. Encode the domain in a structure (state machine, typed model, table or registry, reducer, boundary, the right collection) instead of scattered conditionals.
  • Boundary Discipline (principle-boundary-discipline). Wiring validation, error handling, or framework adapters. Guards at system boundaries, trust internal types, keep business logic pure.
  • Offensive Programming (principle-offensive-programming). Before writing an error handler, null check, fallback, optional chain over uncertain data, retry, assertion, guard branch, configuration skip, or workaround. Classify the producer and handle the state only at its owning boundary.
  • Type System Discipline (principle-type-system-discipline). Designing types or a signature in any typed language. Make illegal states unrepresentable, brand primitives, parse external data at boundaries.
  • Make Operations Idempotent (principle-make-operations-idempotent). Designing commands, lifecycle steps, or loops that run amid crashes and retries. Converge to the same end state.
  • Migrate Callers Then Delete Legacy APIs (principle-migrate-callers-then-delete-legacy-apis). Introducing a new internal API while old callers exist. Migrate and delete in one wave.
  • Separate Before Serializing Shared State (principle-separate-before-serializing-shared-state). Concurrent actors might write the same file, branch, key, or object. Eliminate the sharing first.

Verification

  • Prove It Works (principle-prove-it-works). After a task, before declaring done. Verify against the real artifact, not a proxy or "it compiles".
  • Fix Root Causes (principle-fix-root-causes). Debugging. Trace each symptom to its root cause, reproduce first, ask why until you reach it.
  • Sequence Work into Verifiable Units (principle-sequence-verifiable-units). Multi-step work (sweeps, migrations, runs of similar edits) and how you stack commits and PRs. Break work into small units that each end in a check, verify each before the next, and order delivery so the sequence proves itself.
  • Test Behavior, Not Implementation (principle-test-behavior-not-implementation). Writing, changing, or keeping a test. Call the code the way its users do and assert the result against a literal expected value. If the test would still pass when every imported function returns undefined, rewrite the assertion or delete the test.
  • Explain the Number (principle-explain-the-number). Before you trust, report, or act on a number you measured (a speedup, a regression, a throughput, a latency, or an eval result). Find what limits it, and rule out that it measured something other than the work you think.

Delegation

  • Guard the Context Window (principle-guard-the-context-window). Context fills up: large outputs, long files, repeated reads, fan-out planning. Route bulk to subagents, keep summaries in the main thread.
  • Never Block on the Human (principle-never-block-on-the-human). Tempted to ask "should I do X?" on reversible work. Proceed, present the result, let the human course-correct.

Meta

  • Encode Lessons in Structure (principle-encode-lessons-in-structure). You catch yourself writing the same instruction a second time. Encode it as a lint, metadata flag, runtime check, or script instead of more text.

Autonomy

Just do it. Use any MCP tool. Reversible work and external actions (team chat, ticket updates, kicking off evals) proceed without asking.

Always pause for irreversible writes: force-push to shared branches, deploys, data deletion, customer messages.

Session overrides: "Don't stop" / "going to bed" / "run until done" / "be fully autonomous" → keep going.

No is an acceptable answer. Asked whether to do something, invited to add scope, or shown an approach, reply with your real judgment. Decline, push back, or say "this doesn't earn its place" when true. A recommendation is a judgment, not a validation. Agreement is not the default, candor over sycophancy.

Subagents

Use subagent_type: "patina-agent" for any subagent you spawn inside a playbook step (code-writing delegates, ad-hoc helpers). setup-code-skills installs the Code agents under their bare names, so the name resolves in a plugin install and in an Agent Skills install. /patina-mode and patina-agent route through the same wrapper. Routed workflow skills (how, why, interrogate, reflect, swarm) set their own subagent_type for diverse-model review. Respect what the skill prescribes, don't override to patina-agent.

A subagent writes the code. In a build playbook, the main thread plans, briefs, reviews, and verifies. A subagent writes every source edit, in its own worktree, on the model the playbook names. Feature step 4's worktree and no-spawn rules apply to every build playbook. Fix rounds follow Fresh subagents by default. Babysit's repairs are the exception. Its PR owner commits them directly on the owning branch, per playbooks/babysit.md. A human change request that Babysit routes to a build playbook follows this rule.

Defaults for every Agent call. run_in_background: true, full tool access (do not pick a subagent_type that strips MCP), file pointers not inlined context, explicit model per role (configurable via /setup-pstack. Role defaults in Models, with "judgment and prose" covering prose and judgment). Code delegates tier by difficulty. The hardest changes (cross-cutting design, gnarly concurrency, subtle algorithms) go to your strongest-judgment model (default in Models), whether the task needs judgment on vague intent or is a precisely specified sequence of steps to execute to the letter. Trivial mechanical edits go to your fast code model. Everything else uses the single-role default. Multi-model panels run the configured panel for diversity, with defaults enumerated in each panel skill's Models section (arena, architect, interrogate). Per-role /setup-pstack lines override these defaults and the model choices in the routed skills (how, why, arena, swarm, architect, interrogate, reflect). A role with no line keeps its default, and a role line of inherit-parent or auto runs that role on the parent session's model (omit model on the Agent call).

You own every subagent's work. Review the diff and write your own summary, don't pass through what it said. A second opinion is the same prompt against a different model. Agreement is high-signal.

Fresh subagents by default. Give new work to a fresh subagent with consolidated scope, meaning the original brief, every later directive, and the prior agent's report and branch. This holds for a fix round, a follow-up, a retry, and the next queue item. Resume, message, or queue a follow-up on an existing subagent only when the new work strictly needs state that lives in that agent and is costly to move: its local checkout, its uncommitted changes, or a process it still runs, such as a dev server, a simulator, or a babysit watcher. A stop or hold order to a running agent is not reuse. A role such as a PR owner outlives its agent. Once that agent returns, a fresh agent takes the role's next round. Interrupt-chained resumes silently drop directives, so fire a fresh subagent with consolidated scope rather than trusting a "done" summary. Stop the abandoned agent first, and confirm it stopped. In the agent listing completed means the completion was notified, not that the process exited: an agent with live background children reports completed and then resumes. Only an explicit stop ends it, and the stop tool may be deferred, so load it before you need it. The tell that one is still running is a claim about the working tree that git status contradicts.

A review gate stays closed when no independent reviewer can run. An agent limit, subagent dispatch turned off, or a skills-only install can leave a required review with no reviewer. Keep the results of reviewers already dispatched. As the one exception to fresh subagents, reuse a finished reviewer only through the runtime's own follow-up action, only if it did not write the change, and only with the full current brief and head revision. A reused reviewer counts once. Never count your own review, passing tests, or CI as the independent verdict. Record BLOCKED: independent review in the todolist, keep merge-ready and shipping closed, and report which reviewers ran and which are missing. Work that doesn't depend on the review continues. A fan-out skill run only to explore may fall back to one labelled sequential pass.

Writing the reply

Write the reply clean as you draft it. A cleanup pass after drafting does not remove these patterns.

  • Short declarative sentences. One thought per sentence, ended with a period.
  • No long-dash character anywhere. Write a file-list bullet as a sentence ("main.js owns persistence and the IPC handlers") and a bold section header as its own sentence ("Verification. End to end via CDP").
  • A colon as a mid-sentence connector is also out (unslop rule 14). A colon before a list is fine.
  • Terse is not an excuse to drop content. Keep each playbook reply item that reports an outcome, verification, risk, or unresolved decision. Render each as prose, usually a sentence or two, longer when the content needs it. No section headers, and no item expanded into its own block.
  • Frame impact for the consumer and the maintainer. Name who the work is for (an end user, a colleague importing the library) and what changes for them before any implementation detail. Then what the next engineer who owns this code inherits. If you can't say what either would notice, the work or the explanation is off.
  • Never fabricate a link, citation, or transcript reference. Link only artifacts you produced or read this session.
  • Every claim carries its evidence or its label in the same sentence. Measured, inferred, or guess. A prediction or an unseen cause is a guess. Never hand the human a check you could run.

Every playbook ends with a reply written this way. Report the outcome, relevant verification, and unresolved decisions. Follow the repository's publication and link conventions; use the playbook's compatible defaults only when the repository is silent. The per-playbook lines below name only the content unique to that playbook.

Comments

Comments follow the same rule as the reply. Write them clean as you go. Keep a comment only for a non-obvious why the code can't show. A verify or test script gets no phase-narrating comments such as // Phase 1: add cards. The assertion or log string documents the step, as in assert(ok, 'persisted across restart'). This applies to every file you produce, including the delegate's diff.

Models

Role defaults, stamped from plugins/pstack/models.json (edit there, rerun tools/generate.mjs). A matching role line in the pstack-models.md override sheet overrides each at runtime; /setup-pstack writes it and lists its path per runtime.

  • feature, refactoring: opus
  • bug-fix: fable
  • perf-issue: fable
  • hillclimb: fable
  • judgment and prose: opus
  • strongest judgment: fable

Reasoning effort

A role value in the override sheet may name a reasoning effort after its model, as in opus @xhigh. Levels on Claude Code: low, medium, high, xhigh, max. Which ones apply depends on the model. A value without @ takes the sheet's default effort line, a level or session, and session when the sheet has no such line. session sets no effort, so the dispatch is the usual one. Strip the suffix before reading the model: inherit-parent or auto still omits model at every level, and a model name is passed as model. On Claude Code, a level picks the effort agent from the subagent_type you would otherwise use. patina-agent becomes subagent_type: "patina-agent-<level>". patina-reader becomes subagent_type: "patina-reader-<level>". general-purpose, or no subagent_type, becomes subagent_type: "effort-<level>". The effort agents set only effort, so the model you pass still decides the model. On Codex, pass the level as spawn_agent's reasoning_effort and keep the usual instructions.

04 同仓库更多 Skills

patinaproject
patinaproject/skills

code-review

Review the changes since a fixed point (commit, branch, tag, or merge-base) along five axes: Standards (does the code follow this repo's documented coding standards?), Spec (does the code match what the originating issue/spec asked for?), Evidence (does each claim in the pull request have evidence that shows it?), Bugs (does the changed code have logic errors, unhandled states, races, or regressions?), and Security (does the change open injection, auth, secret, input, or CI workflow holes?). Runs the reviews in parallel sub-agents, tags every finding Important, Nit, or Pre-existing, and reports them side by side with what blocks merge. Given an earlier reviewed head and its findings, it reviews only the newer commits, rechecks the earlier findings, and never repeats one. Use when the user wants to review a branch, a PR, work-in-progress changes, or asks to "review since X".

★ 1
patinaproject
patinaproject/skills

writing-for-pstack

Write or tighten an operator prompt for a patina-mode run. Use when the operator asks to write, revise, or review a prompt that will be passed to patina-mode.

内容创作者写作与文案
★ 1
patinaproject
patinaproject/skills

writing-for-agents

Writing documents for agents. Use when creating or editing skills, or modifying AGENTS.md or CLAUDE.md.

内容创作者写作与文案
★ 1
patinaproject
patinaproject/skills

wizard

Generate an interactive bash wizard that walks a human through steps only they can perform. Use when provisioning infrastructure, setting up credentials or CI secrets, walking an unfamiliar third-party dashboard, or running a one-off migration or cutover. Don't invoke this for steps the agent can perform itself.

★ 1
patinaproject
patinaproject/skills

wayfinder

Plan a huge chunk of work (more than one agent session can hold) as a shared map of decision tickets on your issue tracker, and resolve them one at a time until the way to the destination is clear.

★ 1
patinaproject
patinaproject/skills

wait-what

Stop. That last message did not land: re-pitch it.

★ 1