From 94c534737e6d6a27071a3f7f2a3c55817b231415 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nathan=20=F0=9F=94=B6=20Tarbert?= <66887028+NathanTarbert@users.noreply.github.com> Date: Fri, 11 Sep 2026 10:41:45 -0400 Subject: [PATCH] chore: keep hand-maintained skill files out of the format check The format check added by #250 runs against the files a PR touches, which is the right shape -- but two kinds of file in this repo get worse when prettier formats them, and both are now failing PRs that have nothing wrong with them. .claude/skills/*.md are hand-maintained spec prose carrying wide reference tables. Prettier pads every cell out to the widest one, so a one-word edit rewrites the whole table and the real change disappears into the diff. On #238 that is 305 changed lines across two SKILL.md files. docs/community-signal/reddit-pulse-seen.json is operational dedup state, appended by the weekly-report routine rather than written by hand, and 2-space by construction. The repo's tabWidth is 4, so formatting it reindents all 412 lines -- and the next run rewrites it 2-space again, so the check would break again every week. Both patterns are anchored with a leading slash, for the reason the /templates/ entry above them records: unanchored, a gitignore-style pattern matches a directory of that name at any depth. Verified against the CI step's own command over #238's three changed files: fails before, passes after. Repo-wide `prettier --check .` is unchanged at 334 pre-existing warnings, and .claude/settings.json stays covered -- the only tracked files this exempts are the 13 markdown skill definitions. --- .prettierignore | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/.prettierignore b/.prettierignore index 72bb09d..e549bfa 100644 --- a/.prettierignore +++ b/.prettierignore @@ -21,3 +21,16 @@ packages/outpost/db/prisma/migrations/ # apps/web/src/app/**/templates/, exempting them from both the CI check and # `pnpm format` with no way to bring them back into compliance. /templates/ + +# Skill definitions under .claude/skills/ are hand-maintained spec prose, and +# several carry wide reference tables. Prettier pads every cell out to the +# widest one, so a one-word edit rewrites the whole table and the real change +# disappears into the diff. Anchored so it matches the directory at the repo +# root rather than any `skills` directory at any depth. +/.claude/skills/ + +# Operational dedup state for Reddit Pulse — appended by the weekly-report +# routine rather than written by hand, and 2-space by construction. Formatting +# it to the repo's 4-space width reindents all 412 lines and would do so again +# after every run. +/docs/community-signal/reddit-pulse-seen.json