Skip to content

docs: regenerate the skill-triggers block - #458

Merged
mergify[bot] merged 1 commit into
mainfrom
plan/skill-triggers-regenerate
Sep 12, 2026
Merged

docs: regenerate the skill-triggers block#458
mergify[bot] merged 1 commit into
mainfrom
plan/skill-triggers-regenerate

Conversation

@EdbertChan

@EdbertChan EdbertChan commented Sep 12, 2026

Copy link
Copy Markdown
Owner

Summary

Every PR's preflight was failing on a generated docs block that had fallen behind the skills on disk.

Regenerating the block clears that failure, for any PR, whether or not it touches skills.

Nothing else changes.

Review Claim

docs/skill-triggers.md holds the current output of scripts/check_skill_trigger_policy.py --write, so that gate passes.

Review Lane

docs

Review Unit

docs

Safety Invariant

One generated file, written by the repo's own --write flag. No skill, script, or hook changes.

Slice Rationale

Repo debt found while writing an unrelated rule. It lands on its own so the rule's PR does not carry a docs file, and so any other PR blocked by the same gate is unblocked by one merge.

Non-goals

  • No change to the checker or to any skill's triggers.
  • No hand edits inside the generated block.

Test Plan

Test Plan
  • Before, on a clean checkout of main at 329c844: python3 scripts/check_skill_trigger_policy.py printed fail docs/skill-triggers.md: generated block is stale; run python3 scripts/check_skill_trigger_policy.py --write, exit 1
  • After --write on this branch: ok skill trigger policy, exit 0
  • The diff is two lines in the generated block and nothing else: docs/skill-triggers.md | 4 ++--

Revert Plan

Revert Plan
  • Safe to revert? Yes.
  • Revert command: git revert <merge-sha>
  • Post-revert steps: None. The gate goes back to failing on main.
  • Data migration? No

🤖 Generated with Claude Code

https://claude.ai/code/session_01Gnua5cuAAF1ey1b7zRqJCC


Note

Low Risk
Single generated docs file sync; no runtime, auth, or policy script changes in this diff.

Overview
Refreshes the generated inventory in docs/skill-triggers.md via check_skill_trigger_policy.py --write so the preflight gate stops reporting a stale block.

The only inventory delta is phrase-judge joining the auto-fire list (count 21 → 22). No checker, skill frontmatter, or hand edits inside the block—docs catch up to skills already on disk.

Reviewed by Cursor Bugbot for commit 24add77. Bugbot is set up for automated code reviews on this repo. Configure here.

check_skill_trigger_policy.py failed on main because the generated block in
docs/skill-triggers.md lagged the skills on disk, so every PR's make-pr
preflight failed on a file the PR never touched. This is the output of
scripts/check_skill_trigger_policy.py --write, nothing else.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Gnua5cuAAF1ey1b7zRqJCC
Change-Id: I5462317f830a12287fbc5cfd3dff4be8b52e2945
@cursor

cursor Bot commented Sep 12, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_544d4ea2-6a6b-47f1-8706-531b11de8a04)

@mergify

mergify Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Queued — the merge queue status continues in this comment ↓.

@EdbertChan

Copy link
Copy Markdown
Owner Author

@Mergifyio queue

@mergify

mergify Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 7 minutes 3 seconds in the queue, including 5 minutes 51 seconds running CI.

Required conditions to merge
  • check-success = lint
  • check-success = test

@mergify mergify Bot added the queued label Sep 12, 2026
@mergify
mergify Bot merged commit 0abdc94 into main Sep 12, 2026
4 checks passed
@mergify mergify Bot removed the queued label Sep 12, 2026
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