Skip to content

Slim copilot-instructions globals to bot-enforceable rules#60

Merged
jamditis merged 1 commit into
masterfrom
chore/slim-copilot-instructions
May 2, 2026
Merged

Slim copilot-instructions globals to bot-enforceable rules#60
jamditis merged 1 commit into
masterfrom
chore/slim-copilot-instructions

Conversation

@jamditis
Copy link
Copy Markdown
Owner

@jamditis jamditis commented May 2, 2026

Why

Pivot from 2026-05-02: Copilot's PR review bot is a bug finder, not a style linter. The globals block in .github/copilot-instructions.md (added in PR #57) was asking the bot to enforce English-prose rules it doesn't actually act on. Slim the section to only what the bot can pattern-match.

Kept

  • AI-authorship attribution. Commit-trailer / PR-body strings like Co-Authored-By: Claude and "Generated with Claude Code" are pattern-matchable, and Copilot has flagged them in past reviews.
  • HTML favicon + OG/Twitter meta tags. Concrete tag/attribute checklist the bot can diff against new HTML files.

Removed

  • Sentence case rule (prose-style judgment, not bot-shaped).
  • Banned-words / anti-slop list (same reason).
  • Broad emoji ban (the AI-authorship rule already covers the high-leverage case; the bot won't usefully police every emoji).

Char count

Before: 3723. After: 2886. Project-specific bug classes section untouched.

Refs #59.

Per 2026-05-02 pivot: Copilot's PR review bot enforces code-bug-shaped
rules but won't flag English-prose conventions like sentence case or
banned words.

Kept: AI-authorship attribution; public-HTML favicon and OG/Twitter
meta tags. Both pattern-matchable.

Removed: sentence case, banned words, broad emoji ban.

Refs #59.
Copilot AI review requested due to automatic review settings May 2, 2026 23:46
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR trims .github/copilot-instructions.md so the Copilot review guidance focuses on rules that are concrete and realistically enforceable by the bot, while leaving the repository-specific bug classes intact.

Changes:

  • Removes prose-style global rules that are not reliably bot-enforceable, such as sentence case, emoji bans, and anti-slop wording guidance.
  • Keeps the AI-authorship attribution rule, but rewrites it more concisely around commit messages, PR bodies, and committed docs.
  • Keeps the favicon/OG/Twitter metadata requirement, also in a shorter, more checklist-like form.

@jamditis jamditis merged commit 755048d into master May 2, 2026
4 checks passed
@jamditis jamditis deleted the chore/slim-copilot-instructions branch May 2, 2026 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants