跳到正文
cccskills
免费在 GitHub 公开

modernize

For getting old prs up to date. Use when asked to rebase a pr or "modernize".

中文简介正在准备中。当前显示原文说明。

t3dotgg/tstack星标 543(GitHub 543 + cccskills 0)

实测评分 未实测

查看安装方法

包含的文件(2)

  • SKILL.md4.4 KB
  • agents/openai.yaml263 B

SKILL.md(原文)

安装前可以查看交给 AI 代理的指令内容。

Modernize

Modernize the idea, not just the branch. A clean rebase does not prove that the PR still belongs in the product.

Guardrails

  • Always check to make sure you have pulled latest origin for your branch, main branch, and any other branch related to your work before starting.
  • Treat existing worktree changes as user-owned. Use a disposable worktree when the active worktree is dirty or is being used for other work.
  • Modernizing authorizes rebasing and updating the PR branch. It does not authorize merging, closing, changing the base branch, or expanding scope.
  • Never use an unqualified force push. Update a rebased branch with --force-with-lease against the head SHA observed before the rebase.
  • Do not post a PR comment unless the user asked for one. Use the leaving-pr-comments skill for every comment posted on the user's behalf.

Establish the PR's intent

Resolve the PR and record its URL, state, base, head repository and branch, current head SHA, author, checks, and review state. Read the title, body, linked issues, commits, review threads, and complete diff. State the smallest coherent goal the PR was trying to accomplish before judging its implementation.

Fetch the latest remote base and PR head. Use the PR's actual base branch, usually main, rather than assuming the local branch is current. Record the merge base so the original PR and subsequent base-branch changes can be compared separately.

Check what changed around it

Inspect commits and diffs added to the base branch since the PR diverged. Search merged and open PRs when titles, code paths, or behavior suggest a replacement or competing implementation.

Look beyond overlapping lines:

  • Has the same user-facing problem already been solved?
  • Did APIs, types, schemas, dependencies, or architectural boundaries change?
  • Do both sides now claim ownership of the same behavior or invariant?
  • Was the code being changed deleted, moved, or intentionally simplified?
  • Would landing the PR restore an old behavior or create two ways to do the same thing?
  • Is the original benefit still large enough to justify the maintenance cost?

Do not describe a PR as current merely because Git reports no conflicts.

Choose a disposition before pushing

Use one of these recommendations:

  • MODERNIZE: the goal is still useful and the implementation remains the right shape. Rebase, resolve conflicts, test, and update the branch.
  • REWORK: the goal is still useful, but a mechanical rebase would preserve the wrong implementation. Explain the smallest credible rewrite and stop before pushing unless the user also asked for that rework.
  • CLOSE: the PR is superseded, obsolete, directionally wrong, or no longer worth its cost. Cite concrete evidence and stop. Do not spend time rebasing an obsolete branch, and do not close it without explicit permission.

Be willing to recommend CLOSE. Age alone is not evidence, and sunk effort is not a reason to keep a PR alive.

Rebase without losing intent

For a MODERNIZE disposition:

  1. Create a safe local branch or disposable worktree at the recorded PR head.
  2. Rebase its commits onto the freshly fetched base branch.
  3. Resolve each conflict according to the PR's current intent and the base branch's newer invariants. If a conflict reveals a product or architecture disagreement, stop and reconsider the disposition instead of guessing.
  4. Compare the pre-rebase and post-rebase commit series with git range-diff and inspect the final diff. Confirm that no intentional behavior disappeared and no already-landed behavior was reintroduced.
  5. Run the repository's focused checks for the touched area. Distinguish real failures from pre-existing or infrastructure failures.
  6. Push with --force-with-lease tied to the previously observed remote head. If the lease fails, fetch and reassess rather than overwriting new work.

Do not merge the PR.

Report

Lead with MODERNIZE, REWORK, or CLOSE and the reason. Then summarize:

  • meaningful changes on the base branch;
  • conceptual conflicts or confirmation that none were found;
  • the rebase and push result, if performed;
  • checks run and their results;
  • the remaining decision or work, with a link to the PR.

Separate observed facts from judgment. Say what could not be verified instead of manufacturing confidence.

评价

还没有评价。欢迎分享你的使用感受。

评论

无需登录即可发表。欢迎分享使用感受、提问,或说明在哪些环境下无法运行。

还没有评论。来写第一条吧。

提交后会立即公开。请勿填写邮箱、电话等个人信息。URL 最多 2 个。

0 / 1000 字

此表单受 reCAPTCHA 保护,适用 Google 的隐私权政策和服务条款。

同一仓库的技能

Monitor a pull request through review and CI. Use when the user asks to monitor, watch, or babysit a PR.

中文简介正在准备中。当前显示原文说明。

t3dotgg/tstack5432026年10月25日 更新

council

免费

Trigger this if the user says 'council'.

中文简介正在准备中。当前显示原文说明。

t3dotgg/tstack5432026年10月25日 更新

file-pr

免费

File a concise pull request. Use when the user asks to file, open, or create a PR.

中文简介正在准备中。当前显示原文说明。

t3dotgg/tstack5432026年10月25日 更新

full-send

免费

Trigger this if the user says 'full send'.

中文简介正在准备中。当前显示原文说明。

t3dotgg/tstack5432026年10月25日 更新

full-stop

免费

Trigger this if the user says 'full stop'.

中文简介正在准备中。当前显示原文说明。

t3dotgg/tstack5432026年10月25日 更新

Use whenever leaving comments on GitHub.

中文简介正在准备中。当前显示原文说明。

t3dotgg/tstack5432026年10月25日 更新

查看 t3dotgg 的全部技能

报告此技能的问题