CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

when-to-act-without-approval

Use when deciding whether a change is yours to ship or needs the maintainer's approval first, and to look up what to verify before shipping it — the per-domain carve-out details behind `.claude/rules/workflow.md` rule 10 (front-end, backend, migrations, paid production runs, dead-code removal, concurrent-session safety) plus the catalogue of handbacks that produced each grant. Invoke before opening a PR, when tempted to ask "want me to ship this?", and before committing or merging while other sessions are running.

Quality

79%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.claude/skills/when-to-act-without-approval/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

63%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

The body is a dense, highly specific decision reference with genuinely concrete verification commands and well-observed operational traps — its guidance is actionable and its validations explicit. It is held back by verbosity (long nested sentences, inline incident narratives) and by keeping catalogue/history material inline in SKILL.md instead of splitting it into a reference file.

Suggestions

Move the handback catalogue table and the '*(Origin: ...)*' incident narratives into a one-level-deep reference file (e.g. references/handback-log.md), keeping a one-line summary per grant in SKILL.md — the intro already says this material need not be resident.

Tighten the longest passages: the migrations 'traps' and 'Solve the broader problem' sections run 100+ words per sentence; convert each trap/trigger into a short bullet that preserves the command and the evidence citation.

Make the per-domain 'Verify before shipping' steps uniform — a short checklist or table per section — so the front-end, backend, and migration lanes read with the same scannable structure.

DimensionReasoningScore

Conciseness

Nearly all content is repo-specific knowledge Claude could not know (precedents, deploy traps, incident history), so it does not explain known concepts — but the prose is noticeably dense: 100+ word sentences with nested parentheticals ('*(Origin: shipping #571's root-word search fix, I scoped down to inflectional matching, filed the broader fuzzy/typo fix as #573, and stopped...)*') and long narrative asides that could be tightened without losing information. Mostly efficient but with real tightening headroom, matching the 3 anchor.

3 / 5

Actionability

Concrete, executable guidance throughout: '`tsc --noEmit`, `prettier --check .`', '`just worktree <branch>`', '`git fetch origin main`, then `git diff origin/main -- <file>`', 'upgrade→downgrade→upgrade round-trip against real Postgres', the 32-char `alembic_version` limit with the `NNNN_short_slug` convention. Falls short of 5 because sections like 'Sweep the pattern' and 'Solve the broader problem' are principle-narrative rather than concrete procedure, leaving minor gaps.

4 / 5

Workflow Clarity

Each domain carries explicit validation checkpoints ('Verify before shipping: tsc, prettier, build ... manually verified on the dev server'), the paid-run flow is a numbered order of operations with a dry-run + scoped-live-check gate, and error-recovery guidance exists (post-merge `alembic_version` query, the `StringDataRightTruncation` failure mode, 'testing the downgrade' resolving doubt). Validation is present so no cap applies, but there is no single coherent top-level sequence — it is per-domain procedures — and the front-end section has no explicit failure-recovery loop, placing it at 4 rather than 5.

4 / 5

Progressive Disclosure

Section headers are clear and per-domain organization is sensible, but this is a ~103-line single-file body with no bundle files at all, and the handback catalogue table plus the incident-history narratives (the #489/#486 migration handbacks, the #571/#573 origin story) are precisely the non-resident material that could live in a one-level-deep reference file. This matches the 3 anchor: 'some structure ... content that should be separate is inline'.

3 / 5

Total

14

/

20

Passed

Description

87%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A strong description: it states what the skill contains (per-domain carve-out details, verification lookups, handback catalogue), when to invoke it with natural, concrete trigger phrases, and is tightly scoped to avoid conflicts. Its only weakness is mild verbosity and a coverage-area enumeration where a more action-oriented list could sit.

DimensionReasoningScore

Specificity

Names concrete capabilities — 'deciding whether a change is yours to ship or needs the maintainer's approval first', 'look up what to verify before shipping it' — and enumerates its coverage domains ('front-end, backend, migrations, paid production runs, dead-code removal, concurrent-session safety'). This matches the 'several specific actions; minor gaps' anchor rather than 5, since the enumerated items are coverage areas rather than a comprehensive list of concrete actions.

4 / 5

Completeness

Explicitly answers both questions: the 'what' ('the per-domain carve-out details behind `.claude/rules/workflow.md` rule 10 ... plus the catalogue of handbacks that produced each grant') and the 'when' with multiple concrete trigger phrases ('Use when deciding whether a change is yours to ship', 'Invoke before opening a PR', 'before committing or merging while other sessions are running'). Clearly matches the top anchor.

5 / 5

Trigger Term Quality

Natural trigger phrases are present and idiomatic: 'before opening a PR', 'when tempted to ask "want me to ship this?"', 'before committing or merging while other sessions are running'. Good keyword coverage (ship, PR, commit, merge, approval, verify), but a few natural variants users might say ('ask permission', 'deploy', 'can I merge') are missing, so it sits at 4 rather than the comprehensive-synonyms anchor at 5.

4 / 5

Distinctiveness Conflict Risk

It occupies a clear niche — ship-vs-ask approval decisions and pre-ship verification for this repo's workflow rule — with distinct triggers (PR, merge, commit, concurrent sessions, maintainer approval) that no generic skill would claim. Minimal conflict risk.

5 / 5

Total

18

/

20

Passed

Validation

93%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 1 suspicious

Warning

Total

15

/

16

Passed

Repository

alethical-org/alethical

Path

.claude/skills/when-to-act-without-approval/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.