协作头脑风暴
brainstorming
在实现前澄清需求、探索设计选择,并形成清晰的开发方向。
lov-skill-optimizer【开发与工程】Audit and optimize one or more existing Agent Skills from a canonical source path, then bump semver, update README/SKILL.md/skill.yaml/CHANGELOG.md, and verify installed copies and catalog synchronization. Checks frontmatter, trigger quality, CLI hygiene, accurate and concise naming, portability, version drift, dirty worktrees, shared Skill feedback …
独立 GitHub 快照 · 自动分类 · 以原作者文档为准
name: lov-skill-optimizer category: Meta Skills tagline: "Audit an Agent Skill, apply focused fixes, bump semver, and verify every distribution layer." description: > Audit and optimize one or more existing Agent Skills from a canonical source path, then bump semver, update README/SKILL.md/skill.yaml/CHANGELOG.md, and verify installed copies and catalog synchronization. Checks frontmatter, trigger quality, CLI hygiene, accurate and concise naming, portability, version drift, dirty worktrees, shared Skill feedback policy, and compatibility contracts. Use when the user asks to optimize, refine, audit, polish, or update a Skill, or mentions "优化 skill", "skill 审计", "刷一遍 skill", "skill-optimizer", or "update skill changelog". license: MIT compatibility: > Requires Python 3.8+ (stdlib only). Git is optional for linting and required for source commit/push verification. Catalog synchronization is discovered from explicit paths, environment variables, or nearby checkouts; unavailable locations are reported rather than assumed. depends_on:
This is a non-interactive maintenance workflow. It infers the target and prioritizes issues from the current conversation, then supplements them with a generic lint pass. When several Skills are named in one request, process them in the order named and emit a separate result block for each Skill.
For a collection audit, generate a read-only inventory first:
python3 scripts/inspect_layout.py --all --root /absolute/path/to/skills --json
The inventory groups actual frontmatter IDs across source candidates, discovers
installation aliases by identity or resolved target, and records full payload
digests separately from catalog state. canonical_candidate is an inference
from a unique source or unique linked payload, never authorization to overwrite
another source. Ambiguous IDs stay unresolved. Generated output, templates and
paid src/public variants are not independent source entries; Kit modules are.
Lifecycle is disabled only for a disabled specification and otherwise remains
unclassified until its owner supplies a product decision.
Treat wrong_target, broken_link, unavailable digests and non-versioned
sources explicitly. A symlink is synchronized only when it resolves to the
selected installable payload; a paid repository root is not its public payload.
Missing public payloads must never fall back to distributing plaintext source.
Collection audit does not synchronize, delete, publish or rewrite any target.
Prefer an explicit canonical path whenever the Skill is outside a conventional skills repository:
python3 scripts/lint_skill.py --path /absolute/path/to/skill --json
python3 scripts/inspect_layout.py --path /absolute/path/to/skill --json
For a name, accept foo, lov-foo, or foo-skill. Resolution may find a
source checkout, an installed copy, or a catalog entry. Before editing:
synced
only when its content digest matches the source; a missing location is
not_discovered, never complete. A symlink is synced only when it
resolves to the canonical source.Paid/encrypted repositories may keep the authored specification at
src/SKILL.md and the installable payload under public/. Treat the repository
root as canonical, lint and version the authored spec, and compare or sync only
the public/ payload to installations. Never copy plaintext src/ content into
an installed paid Skill.
Do not absorb pre-existing edits into a maintenance commit. If a target file is already dirty, review the overlap before editing it and stage only the exact files changed by this optimization.
When the optimization is triggered by a user correction, classify it before choosing targets:
task-specific: applies only to the current artifact or one-off value. Do not edit a Skill.skill-specific reusable: applies to future runs of one domain or platform. Optimize the relevant
canonical Skill.global reusable: changes how every Skill should handle feedback, authorization, sequencing, or
handoff. Update the host's active user-level shared instruction artifact once (for example the
applicable user-level AGENTS.md); do not paste the same policy into every domain SKILL.md.If a global policy was previously placed in one domain Skill, move it to the shared layer and remove the domain duplicate while preserving any genuinely domain-specific rule learned in the same incident. A shared policy file may be outside the Skill repository; report its path and verification state explicitly instead of pretending it is part of the Skill package.
Any reusable correction invalidates prior terminal approval for the active task. Finish the policy/Skill
optimization and validation first, then apply the correction to the current artifact, report the new state,
and stop for the user's next instruction. Do not continue into publishing, submission, or another external
write using a pre-correction “confirm”, “continue”, or “go ahead”.
Normalize every explicitly named Skill and preserve the user's order. Strip the
lov- prefix only for lookup; keep the public Skill identifier in reports.
Collect the current-conversation fix list first: broken flags, trigger misses,
wrong paths, confusing output, missing modules, compatibility requirements, or
other concrete symptoms.
For every correction, distinguish four pieces of evidence: the observed defect, the latest user-approved baseline, the requested delta, and any remedy the user rejected. A rejected remedy is negative evidence, not a reusable best practice. Preserve the approved baseline outside the requested delta, and do not encode a failed over-correction into the target Skill.
For each target, run the linter against the resolved canonical path:
python3 scripts/lint_skill.py --path /absolute/path/to/skill --json
Prioritize findings in this order:
error findings;warn findings;info findings.The baseline must include version-source drift between README.md, SKILL.md
frontmatter, and skill.yaml. Portability findings are high priority for a
reusable Skill: move personal paths to flags, environment variables, or
references/user-config.md, or mark a genuinely author-only dependency in
compatibility.
Classify the Skill's normal output before editing. If it generates, edits,
reviews, renders, packages, or publishes text visible to an end user or reader,
top-level depends_on must contain lov-branding-consistency. This dependency
gates authored presentation copy only; it must not rewrite quotations,
transcripts, source data, legal text, identifiers, or code without permission.
For naming feedback or a collection naming audit, first read Skill naming review. Start from user-approved names and style examples, then review real capabilities and neighboring Skills for accuracy, brevity, memorability, elegance and consistency. Preserve product personality instead of forcing functional labels or banning role words. Record both changed and retained names. Distinguish display fields from compatibility identifiers, and do not infer permission to rename IDs, repositories or Profile keys from a display-name correction.
Edit only the canonical source. Keep the Skill's public trigger surface, compatibility aliases, storage contracts, and user-facing semantics explicit. Use progressive disclosure when SKILL.md grows beyond roughly 500 lines. Add a script or reference file only when it resolves a concrete audit finding or conversation issue.
When the user has already approved an artifact and then requests a localized fix, optimize the domain Skill around a minimal-delta contract: lock accepted layout, copy, data, and behavior as invariants; change only the defective region; verify the whole artifact afterward. A broad redesign requires an explicit new request, not an inferred opportunity.
The linter checks both root SKILL.md repositories and paid/encrypted
src/SKILL.md repositories:
metadata.version, README badge, and skill.yaml version consistency;lov-branding-consistency dependency for audience-visible text Skills.Use the path-aware version tool so all version surfaces move together:
python3 scripts/bump_version.py \
--path /absolute/path/to/skill \
--type minor \
--message "add guarded project rename workflow" \
--change "report source, installation, and catalog synchronization state"
Choose patch for bug, wording, frontmatter, or lint fixes; minor for a new
flag, reference, module, or expanded workflow; major for a breaking CLI or
removed behavior. Stay in 0.x unless the user explicitly requests otherwise.
The tool updates README.md, the authored SKILL.md, paid-skill public wrappers,
skill.yaml, and CHANGELOG.md and refuses to duplicate an existing changelog
version.
python3 scripts/lint_skill.py --path /absolute/path/to/skill --json
python3 scripts/inspect_layout.py --path /absolute/path/to/skill --json
Do not report remaining lint warnings: none unless the final JSON was read.
Do not report synchronization as complete unless every discovered distribution
copy and required catalog check has been verified after the source change. Keep
distribution_state, catalog_state, and the aggregate sync_state separate:
an installed copy can be complete while an undiscovered catalog keeps the
aggregate state partial. A discovered catalog is complete only when its
matching Skill payload digest is synced.
inspect_layout.py checks conventional and configured installation roots:
AGENT_SKILLS_DIR, CLAUDE_SKILLS_DIR, CODEX_SKILLS_DIR, SKILLS_DIR,
plus the host's agent-managed fallback roots. It also checks explicit
--install-root and --catalog-root values plus the nearby unified
lovstudio-skills catalog. Nearby checkouts named after the archived
general/dev split catalogs are reported as legacy and excluded from
catalog_state; pass one with --catalog-root only to compare it on purpose.
Use an environment variable or explicit flag when the installation root is
outside the conventional layout.
For a non-symlink installation copy, first run a read-only sync plan. Paid
repositories automatically use their public/ payload:
python3 scripts/sync_installation.py \
--source /absolute/path/to/canonical-skill \
--target /absolute/path/to/installed-skill \
--json
After reviewing missing, changed, and extra, apply the exact copy with:
python3 scripts/sync_installation.py \
--source /absolute/path/to/canonical-skill \
--target /absolute/path/to/installed-skill \
--apply --json
Use --prune only when removing extra files from the installation copy is
explicitly part of the task. Symlink installations are verified, not copied.
For a catalog, use its own scripts only when they are actually present:
python3 scripts/sync-skills.py
python3 scripts/render-marketplace.py
python3 scripts/render-readme.py
python3 scripts/validate_deps.py
Run only the commands that exist in that catalog checkout. If no catalog is
discovered, report not_discovered; if source and installation are updated but
the catalog is stale or unavailable, report partial. Never invent a catalog
path or claim a live-site update from a local source commit.
Inspect git diff --check, then stage the listed changed files explicitly:
git add -- SKILL.md README.md CHANGELOG.md skill.yaml scripts references
git diff --cached --name-only
git commit -m "fix(<skill-name>): <one-line summary>"
git push origin HEAD
Use feat for a minor feature and feat! for a breaking change. If the source
checkout has no remote, commit on its current branch and report push: not_configured; do not imply that a remote release happened. If a catalog is a
separate repository, commit and push it independently after its own validation.
Return one block per optimized target, with no trailing summary:
optimized: lov-<name>
version: <old> → <new>
source: <canonical path> (<clean|dirty>)
distribution:
- <path>: <synced|drifted|not_discovered>
catalog:
- <path>: <synced|partial|not_discovered|legacy>
fixes:
- <bullet 1>
- <bullet 2>
remaining lint warnings: <count> (or "none")
sync state: <complete|partial|not_discovered>
The keys stay stable for machine parsing; the values and fix bullets follow the user's language. A failed or skipped push, installation sync, or catalog sync must appear in the relevant state rather than being omitted.
Read this Skill's skill.yaml when the host supplies skill-runtime/v1.
Use only fields declared there. Profile data is for public identity facts and
preferences are for output language/timezone; neither replaces the canonical
source, installation, or catalog evidence collected by this workflow.
python3 scripts/lint_skill.py --path PATH [--json]
python3 scripts/lint_skill.py --all --root PATH [--json]
python3 scripts/bump_version.py --path PATH --type patch|minor|major -m MESSAGE
python3 scripts/inspect_layout.py --path PATH [--install-root PATH] [--catalog-root PATH] [--json]
python3 scripts/inspect_layout.py --all --root PATH [--install-root PATH] [--catalog-root PATH] --json
python3 scripts/sync_installation.py --source PATH --target PATH [--apply] [--prune] [--json]
All bundled tools use Python's standard library only.
Layout and synchronization regression checks:
PYTHONDONTWRITEBYTECODE=1 python3 -m unittest discover -s tests -v
用户在 Skill 驱动任务中提出修改意见时,继续当前产物前必须执行:
task-specific(仅本次)还是 reusable(可跨任务复用)。task-specific 只修改当前任务,不改 Skill。reusable 先确定作用域:领域规则先更新对应 canonical Skill;适用于所有 Skill 的规则先更新共享规范。reusable 修改会使此前的“确认”“继续”“发吧”失效;完成当前产物修改和回读后必须停下,等待用户下一步指示,不自动进入发布、提交或其他外部写入。