Skip to content

chore: gitignore playwright cache and root-level screenshots#12

Merged
Navi Bot (project-navi-bot) merged 1 commit into
mainfrom
chore/gitignore-cleanup
Apr 30, 2026
Merged

chore: gitignore playwright cache and root-level screenshots#12
Navi Bot (project-navi-bot) merged 1 commit into
mainfrom
chore/gitignore-cleanup

Conversation

@Fieldnote-Echo
Copy link
Copy Markdown
Member

Summary

  • Add /.playwright-mcp — playwright MCP browser cache (transient screenshots and console logs) generated during /loop browser-check sessions
  • Add /*.png (root-anchored) — root-level browser-check screenshots like landing-3x3-full.png, takens-*-full.png, delay-check-viewport.png
  • Tracked image assets in docs/assets/ (favicon, logo) are unaffected — the leading-slash anchor scopes the pattern to repo root only

Discovered during the CF-1 takens local audit (2026-04-30): git status was returning ten untracked artifacts cluttering the working tree. The .gitignore was the only outstanding hygiene item; the build, axiom dashboard, and headline declarations were all clean (audit memo lives in the consumer repo, reccs-formalization).

Test plan

  • git diff main..HEAD shows only the .gitignore addition (+6 lines)
  • git status silent after the change — no ?? lines for the ten transient artifacts
  • git ls-files 'docs/assets/*.png' still lists favicon.png and logo.png
  • No *.lean files touched; build state unchanged
  • CI green on this branch

Out of scope

  • No proof-side or docs-side changes. Audit memo for the CF-1 closure is a separate cross-repo deliverable in reccs-formalization.

Coding-Agent: claude-code
Model: claude-opus-4-7

…shots

The /loop browser-check sessions during docs-site iteration generated
playwright cache (.playwright-mcp/) and root-level screenshot PNGs
(landing-3x3-full.png, takens-*-full.png, delay-check-viewport.png).
These are transient artifacts that should not enter the tree.

The /*.png pattern is anchored to repo root only — docs/assets/favicon.png
and docs/assets/logo.png remain tracked.

Coding-Agent: claude-code
Model: claude-opus-4-7
@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

chore: gitignore playwright cache and root-level screenshots

📦 Other

Grey Divider

Walkthroughs

Description
• Add /.playwright-mcp to .gitignore to exclude playwright MCP browser cache (transient
  screenshots and console logs) generated during /loop browser-check sessions
• Add /*.png (root-anchored) to .gitignore to exclude root-level browser-check screenshots like
  landing-3x3-full.png, takens-*-full.png, delay-check-viewport.png
• Tracked image assets in docs/assets/ (favicon, logo) remain unaffected due to leading-slash
  anchor scoping the pattern to repo root only
• Resolves ten untracked artifacts cluttering the working tree discovered during CF-1 takens local
  audit
Diagram
flowchart LR
  gitignore[".gitignore"]
  playwright["/.playwright-mcp<br/>(transient cache)"]
  screenshots["/*.png<br/>(root-level)"]
  assets["docs/assets/*.png<br/>(tracked)"]
  gitignore -- "add pattern" --> playwright
  gitignore -- "add pattern" --> screenshots
  assets -- "unaffected" --> gitignore
Loading

Grey Divider

File Changes

Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

Code Review by Qodo

Grey Divider

No Changes in PR

Qodo reviewed your PR and found no changes in the code

Grey Divider

Qodo Logo

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.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@project-navi-bot Navi Bot (project-navi-bot) merged commit 8bca202 into main Apr 30, 2026
4 checks passed
@project-navi-bot Navi Bot (project-navi-bot) deleted the chore/gitignore-cleanup branch April 30, 2026 17:54
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.

3 participants