Skip to content

chore(repo): clean repo root (.playwright-mcp, screenshots) - #712

Open
ajitpratap0 wants to merge 1 commit into
mainfrom
chore/repo-root-cleanup
Open

ajitpratap0 wants to merge 1 commit into
mainfrom
chore/repo-root-cleanup

Conversation

@ajitpratap0

Copy link
Copy Markdown
Owner

Summary

  • Stop tracking .playwright-mcp/ (git rm --cached; the files stay on disk) and add .playwright-mcp/ and /*.png to .gitignore, so local screenshots no longer appear in git status.
  • Decision on .claude/settings.json: keep it tracked. It only enables two shared plugins (qodo-skills, pagerduty) and contains no secrets.
  • render.yaml and glama.json are untouched.
  • ACTION_README.md is not moved. This keeps the PR focused; the move can follow separately.

Test Plan

  • task check passes locally (fmt, vet, lint, race tests).
  • git check-ignore confirms .playwright-mcp/x.png and a root foo.png are ignored.
  • git ls-files shows no .playwright-mcp/ entries after the change.

Breaking Changes

None. The .playwright-mcp/ files are removed from the index only.

Closes #670
Part of #634

🤖 Generated with Claude Code

https://claude.ai/code/session_01RkzVu3JY9Y6BRWfjesWts9

Stop tracking the Playwright MCP session output under .playwright-mcp/
(files stay on disk) and ignore that directory plus root-level PNGs so
local screenshots no longer show in git status.

Tracked .claude/settings.json is kept: it only enables two shared
plugins and holds no secrets. render.yaml and glama.json are unchanged.

Closes #670
Part of #634

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RkzVu3JY9Y6BRWfjesWts9
@vercel

vercel Bot commented Oct 14, 2026

Copy link
Copy Markdown

Deployment failed for project gosqlx-website with the following error:

Resource is limited - try again in 1 day (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/coolajitpratapsingh0-9932s-projects?upgradeToPro=build-rate-limit

@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Claude finished @ajitpratap0's task in 12s —— View job


PR Review: chore(repo): clean repo root

Verdict: LGTM.

  • .gitignore adds .playwright-mcp/ and /*.png. The / anchors the png rule to the repo root, so PNGs under website/, docs/ and similar directories are unaffected.
  • No root-level tracked .png files exist, so /*.png won't hide anything already committed. The earlier gosqlx.png and gosqlx_text.png entries were already ignored.
  • After this change git ls-files shows 0 .playwright-mcp/ entries.
  • The six deleted PNGs are removed from the index only, so there's no runtime, security or test impact. There's no code to cover with tests.

Nits (non-blocking)

  • The new block sits next to an existing comment block about local UI audit screenshots. Merging the two would be a little tidier, but it isn't necessary.
  • The old screenshots remain in git history. If repo size matters, a history rewrite would be a separate decision, and it's probably not worth it for six images.
  • Keeping .claude/settings.json tracked is reasonable, since it contains no secrets.

I didn't run task check. The PR only changes ignore rules and deletes images, and I checked the diff and the tracked files directly.

This branch has not been deployed

No deployments
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.

chore(repo): clean repo root (.playwright-mcp, screenshots) and decide tracked .claude/settings.json [3.5]

1 participant