From abc55d7e0dffac430f08e0baeafd27307df5ba10 Mon Sep 17 00:00:00 2001 From: Tim 'mithro' Ansell Date: Mon, 8 Jun 2026 16:40:56 +0930 Subject: [PATCH] Add rule requiring confirmation before adding muffet exclusions CLAUDE.md now requires using AskUserQuestion before adding any muffet exclusions to preview-verification.yml. Exclusions hide potential problems and should be deliberate decisions, not automatic fixes: the agent must first investigate the root cause and attempt to fix the real problem (broken URL, missing anchor) before proposing an exclusion. Brings forward the policy from the closed PR #73 (commit f4b8312), placed under a new "Preview Verification" section since current main's CLAUDE.md has no verification section. Co-Authored-By: Claude Opus 4.8 (1M context) --- CLAUDE.md | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/CLAUDE.md b/CLAUDE.md index b3b1a5c5..6fbf60d6 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -129,3 +129,17 @@ Make sure you are on the right branch before doing any work. - Once the pull request has been created, wait for the GitHub Action checks to run and fix any issues they find. - Wait for GitHub Copilot to review your pull request and then use the GitHub GraphQL API to find all code suggestions, fix the issues and then resolve the threads. +## Preview Verification + +Pull requests are checked by the `preview-verification.yml` workflow, which runs the [muffet](https://github.com/raviqqe/muffet) link checker against the per-PR preview site. + +### NEVER Add Muffet Exclusions Without Confirmation + +**NEVER add exclusions to `.github/workflows/preview-verification.yml` without first asking the user for confirmation using the AskUserQuestion tool.** Exclusions hide potential problems and should be deliberate decisions, not automatic fixes. + +Before proposing an exclusion, you MUST: +1. Investigate the root cause of the failure +2. Attempt to fix the actual problem (broken URL, missing anchor, etc.) +3. If an exclusion is truly needed, use AskUserQuestion to explain the situation and get explicit approval +4. Document why the exclusion is necessary in the commit message +