Skip to content

Add second-take skill - #5172

Open
flashfrogluo wants to merge 1 commit into
github:mainfrom
flashfrogluo:add-second-take-skill
Open

flashfrogluo wants to merge 1 commit into
github:mainfrom
flashfrogluo:add-second-take-skill

Conversation

@flashfrogluo

Copy link
Copy Markdown

What this skill does

second-take reviews the reasoning behind an AI answer (six check items, thirteen defect subclasses, four evaluation standards) and produces a retake note — a self-contained instruction packet the user pastes back into the original conversation so that the original AI gives a better answer. It never rewrites the answer itself; the deliverable is the instruction.

It is a meta-skill: it targets another agent's output rather than generating content directly.

Files

Self-contained as required by CONTRIBUTING:

  • skills/second-take/SKILL.md
  • skills/second-take/references/ — 11 reference files (hard constraints, decision rules, caption spec, rubric, examples)
  • skills/second-take/prompts/ — 4 standalone fallbacks (CN/EN × generic/DeepSeek) usable without install

Checklist

  • Frontmatter has name, description, license, compatibility, metadata.version (same shape as existing skills)
  • node eng/validate-skills.mjs → all 436 skills are valid
  • npm start run; docs/README.skills.md updated with the generated row (one line only)
  • No bundled asset exceeds 5 MB
  • Documentation and examples included

Notes

Upstream source of truth: https://github.com/flashfrogluo/second-take (MIT). If this is accepted, I will keep the copy here in sync with upstream releases.

Happy to adjust naming, description length, or folder placement.

Reviews an AI answer's reasoning and produces a self-contained retake note the
user pastes back into the original conversation, so that AI answers better.

Self-contained: SKILL.md + references/ (11 files) + prompts/ (4 fallbacks).
Validator: node eng/validate-skills.mjs -> all 436 skills valid.
@github-actions

Copy link
Copy Markdown
Contributor

🚦 Submission status: ⏳ Awaiting automation

Risk tier: merge-risk:medium — New or substantially changed resource without privileged execution
Required to merge: passing submission-gate checks plus 1 approval from reviewers with write access.

Why this tier
  • Adds 16 file(s), e.g. skills/second-take/SKILL.md
  • Changes 2364 lines (low-risk limit is 40)

Automated checks

Check Status Details
Line endings 🔧 Infrastructure failure Waiting for a maintainer to approve workflow runs for this PR · logs
Spelling 🔧 Infrastructure failure Waiting for a maintainer to approve workflow runs for this PR · logs
Generated README consistency 🔧 Infrastructure failure Waiting for a maintainer to approve workflow runs for this PR · logs
Skill validation 🔧 Infrastructure failure Waiting for a maintainer to approve workflow runs for this PR · logs
Skill lint (vally) 🔧 Infrastructure failure Waiting for a maintainer to approve workflow runs for this PR · logs
Risk scan 🔧 Infrastructure failure Waiting for a maintainer to approve workflow runs for this PR · logs
Contributor reputation 🔧 Infrastructure failure Waiting for a maintainer to approve workflow runs for this PR · logs
Duplicate resource scan ⚠️ Failed (advisory, non-blocking) Waiting for a maintainer to approve workflow runs for this PR · logs
PR quality signal ⚠️ Failed (advisory, non-blocking) Waiting for a maintainer to approve workflow runs for this PR · logs

Action needed

  • 🔧 Line endings, Spelling, Generated README consistency, Skill validation, Skill lint (vally), Risk scan, Contributor reputation hit an automation problem that is not caused by your contribution. Comment /rerun-checks to retry; maintainers are notified if it keeps failing.
  • ⚠️ Advisory checks did not complete (Duplicate resource scan, PR quality signal). This does not block the PR.

Review

  • Approvals: 0/1
  • Assigned reviewer: not assigned yet — comment /request-review to ask for one
  • Review target date: not set
  • Still needed: 1 more approval(s)
  • No staffed reviewer pool owns these files yet; any reviewer with write access counts as the domain reviewer.

Commands

Command Who What it does
/rerun-checks PR author, maintainers Re-runs failed or incomplete checks and re-evaluates this gate
/request-review PR author, maintainers Asks the review rotation to assign a reviewer (adds needs-reviewer)

Updated for 6cc44ec · gate run · This comment is maintained automatically — see submission gate docs.

@flashfrogluo

Copy link
Copy Markdown
Author

/rerun-checks

The submission gate reports infrastructure failures (line endings, spelling, generated README consistency, skill validation, skill lint, risk scan, contributor reputation) rather than findings about the contribution. Locally the same checks pass:

  • node eng/validate-skills.mjs → all 436 skills are valid
  • node eng/update-readme.mjs → docs/README.skills.md gains exactly one line
  • the diff contains only skills/second-take/** plus that one README line

Happy to fix anything the re-run surfaces.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant