Back to Plate

rewrite performance skill

docs/plans/2026-06-02-rewrite-performance-skill.md

53.0.810.5 KB
Original Source

rewrite performance skill

Objective: Rewrite the performance skill into a docs-creator-shaped operational review contract and regenerate the generated skill copy.

Goal plan: docs/plans/2026-06-02-rewrite-performance-skill.md

Template: docs/plans/templates/docs.md

Docs source:

  • type: agent rule
  • id / link: .agents/rules/performance.mdc
  • title: Performance
  • acceptance criteria: source rule has owner map, required output, quick pass, blockers, unique rule map, Plate example, generated skill sync, and no stale Vercel catalog dump.

Docs lane:

  • lane: spec/law / behavior
  • target docs: .agents/rules/performance.mdc, .agents/skills/performance/SKILL.md
  • documented source owner: .agents/rules/performance.mdc
  • nearest sibling docs: .agents/rules/docs-creator.mdc
  • plugin page: N/A: skill rule, not a public plugin page.

Completion threshold:

  • Source rule is rewritten around the working review path.
  • Generated skill copy is regenerated by pnpm install.
  • The generated skill contains the rewritten sections.
  • No placeholder or changelog-style language remains in the touched skill docs.
  • node .agents/skills/autogoal/scripts/check-complete.mjs docs/plans/2026-06-02-rewrite-performance-skill.md passes.

Verification surface:

  • pnpm install
  • sed -n '1,260p' .agents/skills/performance/SKILL.md
  • rg -n "Vercel React Rules Already Covered|Required Output|Plate Example: Huge Document 10k|previously|now supports|has been removed" .agents/rules/performance.mdc .agents/skills/performance/SKILL.md
  • node .agents/skills/autogoal/scripts/check-complete.mjs docs/plans/2026-06-02-rewrite-performance-skill.md

Constraints:

  • Edit source rule first.
  • Regenerate generated skill copy with pnpm install.
  • Keep the skill operational, not public-doc decorative.
  • Preserve links to the unique performance/rules/* files.
  • Do not keep a giant stale Vercel rule catalog in this skill.

Boundaries:

  • Source of truth: .agents/rules/performance.mdc.
  • Allowed edit scope: performance rule, generated performance skill, and this goal plan.
  • Browser surface: N/A: agent skill text only, no rendered route.
  • Tracker sync: N/A: no external tracker.
  • Non-goals: rewriting individual performance rule files, changing public docs, changing package code, or changing Vercel rule docs.

Blocked condition:

  • Blocked only if pnpm install fails to regenerate the skill or the generated skill does not reflect .agents/rules/performance.mdc.

Docs state:

  • task_type: docs
  • task_complexity: moderate
  • current_phase: complete
  • current_phase_status: complete
  • next_phase: final response
  • goal_status: complete

Current verdict:

  • verdict: pass
  • confidence: high
  • next owner: none
  • reason: source and generated skill are synchronized and the rule is now organized around the performance review workflow.

Start Gates:

GateAppliesEvidence
docs-creator loadedyesRead .agents/rules/docs-creator.mdc and used owner map / quick path / anti-slop rules.
Active goal checked or createdyesCreated this docs goal plan with create-goal-scratchpad.mjs.
Docs lane selectedyesClassified as spec/law / behavior for an agent skill contract.
Target docs readyesRead .agents/rules/performance.mdc.
Nearest sibling docs readyesRead .agents/rules/docs-creator.mdc.
Docs style doctrine readyesApplied docs-creator structure and anti-slop rules.
Documented source code readN/ASkill text has no package code API surface.
Ownership map draftedyesAdded ## Owner Map.
Plugin-page rules decisionN/ANot a plugin page.
Browser/render proof decisionN/AAgent skill text only.
PR/tracker expectation decisionyesNo tracker or PR action requested for this follow-up.

Work Checklist:

  • Short objective plus outcome, completion threshold, verification surface, constraints, boundaries, and blocked condition are concrete.
  • Docs lane is classified as install, guide/system, plugin/feature, serialization/conversion, workflow/AI, API reference, or spec/law.
  • Target docs and nearest sibling docs were read before writing.
  • Docs style doctrine in docs-creator was read before writing.
  • Documented behavior or API was verified against current source.
  • Ownership map records core runtime, package, kit, registry, and app-local ownership where relevant.
  • Fastest success path appears before deeper mechanics or API reference.
  • Opening is three sentences or fewer and avoids generic fluff.
  • Named APIs, options, transforms, components, imports, routes, and package specifiers are exact and current.
  • Plugin docs, if applicable, satisfy kit/manual/API ordering and headless package ownership.
  • Serialization docs, if applicable, split directions and state environment constraints before examples.
  • API reference docs, if applicable, use exact contracts and avoid tutorial filler.
  • Spec/law docs, if applicable, record owner map, evidence, and explicit gaps.
  • Demos/previews are real registry entries or marked N/A with reason.
  • Links target real leaf pages and do not reinforce pages being displaced.
  • Anti-slop audit passed: no changelog voice, no fake APIs, no placeholder comments, no unresolved task markers, no dead anchors, no redundant summary section.
  • Workspace authority recorded: every proof command names the cwd/tool that owns the changed docs.
  • Review/autoreview target selected for non-trivial docs work, or marked N/A with reason.

Completion Gates:

GateAppliesRequired actionEvidence
Named verification thresholdyesRun source audit and sync proofpnpm install; sed; rg audit.
Docs lane shape satisfiedyesCheck against docs-creatorOwner map, quick pass, blockers, rule map, example, no stale catalog.
Source-backed claim audityesVerify referenced rule names and source/generated syncExisting performance/rules/* links preserved; generated skill inspected.
Ownership map verifiedyesConfirm ownership claims against loaded skill list and rule sourceOwner map delegates to existing skills and keeps this skill's unique scope.
MDX/content parserN/ARecord N/AAgent skill text only, no MDX route.
Links/routes/previews verifiedyesCheck touched skill links are rule-relativePreserved ./rules/*.md links in generated skill.
Plugin page specificsN/ARecord N/ANot a plugin page.
Browser/render surface changedN/ARecord N/ANo public route or UI changed.
Package/API behavior changedN/ARecord N/ANo package/API code changed.
Agent rules or skills changedyesRun pnpm install and verify generated skill syncpnpm install completed; generated skill inspected.
Autoreview for non-trivial docs changesN/ARecord N/ALocal skill doc rewrite, verified by sync and anti-slop audit.
Final lintN/ARecord N/AMarkdown skill text only; pnpm install generated copies.
Goal plan completeyesRun check-completenode .agents/skills/autogoal/scripts/check-complete.mjs docs/plans/2026-06-02-rewrite-performance-skill.md passed.

Phase / pass table:

PhaseStatusEvidenceNext
Intake and source readcompleteRead performance and docs-creator rules.writing
WritingcompleteRewrote .agents/rules/performance.mdc.verification
VerificationcompleteRan pnpm install; inspected generated skill; ran anti-slop grep.closeout
PR / tracker syncN/AUser asked local follow-up only.final response
CloseoutcompleteThis plan records final evidence.final response

Findings:

  • The old performance skill buried its unique performance-lane contract under a full Vercel rule catalog.
  • The useful surface is owner map, required output, quick pass, blockers, unique rule map, and a Plate-shaped example.

Decisions and tradeoffs:

  • Kept Vercel rule selection as a compact symptom table instead of a long catalog.
  • Kept the unique performance/rules/* links because those are this skill's real source value.
  • Added a Huge Document 10k example because it matches the PR lane and gives agents a copyable review shape.

Implementation notes:

  • Source rule rewritten.
  • Generated skill regenerated by pnpm install.
  • No public docs route changed.

Review fixes:

  • N/A.

Error attempts:

Error / failed attemptCountNext different moveResolution
None0N/AN/A

Verification evidence:

  • pnpm install completed and skiller applied rules for Claude Code and Codex.
  • sed -n '1,260p' .agents/skills/performance/SKILL.md showed the generated skill contains ## Owner Map, ## Required Output, ## Quick Pass, ## Blockers, ## Vercel Rule Selection, ## Extra Rules Owned Here, and ## Plate Example: Huge Document 10k.
  • rg anti-slop audit found expected section headings and no stale Vercel catalog heading or changelog voice in touched skill files.
  • node .agents/skills/autogoal/scripts/check-complete.mjs docs/plans/2026-06-02-rewrite-performance-skill.md passed.

Final handoff contract:

  • PR line: N/A.
  • Issue / tracker line: N/A.
  • Confidence line: high.
  • Docs lane: spec/law / behavior.
  • Source-backed claims: source/generated skill sync verified.
  • Content build / parser: N/A for skill docs.
  • Links / demos / previews: rule links preserved; demos/previews N/A.
  • Browser check: N/A for agent skill text.
  • Outcome: performance skill is rewritten and regenerated.
  • Caveat: no individual performance/rules/* files were rewritten.
  • Verified: pnpm install, generated skill inspection, anti-slop grep, goal check.

Final handoff / sync:

  • PR: N/A.
  • Issue / tracker: N/A.
  • Browser proof: N/A.
  • Caveats: individual rule files unchanged.

Timeline:

  • 2026-06-02T11:25:46.930Z Docs goal plan created.
  • 2026-06-02T11:25Z Read performance and docs-creator rules.
  • 2026-06-02T11:26Z Rewrote source performance rule.
  • 2026-06-02T11:27Z Ran pnpm install and inspected generated skill.

Reboot status:

QuestionAnswer
Where am I?Closeout complete.
Where am I going?Final response.
What is the goal?Rewrite and regenerate the performance skill.
What have I learned?The skill needed an operational contract, not a stale rule catalog.
What have I done?Rewrote source, regenerated generated copy, recorded verification.

Open risks:

  • None for the skill rewrite. Individual rule files may still be old prose, but they were outside this request.