Skip to content

chore: gitignore Maina local artifacts under .maina/#117

Closed
beeeku wants to merge 1 commit into
masterfrom
claude/affectionate-shannon-auX08
Closed

chore: gitignore Maina local artifacts under .maina/#117
beeeku wants to merge 1 commit into
masterfrom
claude/affectionate-shannon-auX08

Conversation

@beeeku
Copy link
Copy Markdown
Owner

@beeeku beeeku commented May 21, 2026

Summary

  • Ignore per-developer Maina MCP tooling state (sqlite DBs + on-disk caches) generated under .maina/: .maina/cache/, .maina/context/, .maina/*.db.
  • Tracked Maina content (constitution.md, decisions/, features/, prompts/) is unaffected.

Why this supersedes #114

PR #114 had the right intent but added .main/ patterns (missing the a) — those match nothing, so the artifacts kept showing up as untracked. CodeRabbit's pre-merge title check flagged the same mismatch. This PR uses the correct .maina/ paths. Recommend closing #114 in favor of this.

Test plan

  • Patterns target the real .maina/ directory (confirmed .maina/ is the tracked dir; git ls-files .maina/ = 32 tracked files, all under constitution.md/decisions/features/prompts).
  • Confirm Maina cache/db regenerate locally on first use after a fresh checkout.

Generated by Claude Code

Ignore per-developer Maina MCP tooling state (sqlite DBs + on-disk
caches) generated under .maina/. Supersedes #114, which mistakenly
targeted .main/ (missing the 'a') and so matched nothing.

https://claude.ai/code/session_01SMSKutPGegZz7PiPwfMrXW
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 21, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 687e2b97-c77e-4e2a-a3a1-56c840d7d784

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/affectionate-shannon-auX08

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

Bundle Size Report

Package Base PR Delta
@workkit/agent 80KiB 80KiB no change
@workkit/ai 93KiB 93KiB no change
@workkit/ai-gateway 260KiB 260KiB no change
@workkit/api 119KiB 119KiB no change
@workkit/approval 106KiB 106KiB no change
@workkit/astro 18KiB 18KiB no change
@workkit/auth 48KiB 48KiB no change
@workkit/browser 32KiB 32KiB no change
@workkit/cache 39KiB 39KiB no change
@workkit/chat 46KiB 46KiB no change
@workkit/cli 312KiB 312KiB no change
@workkit/cron 68KiB 68KiB no change
@workkit/crypto 40KiB 40KiB no change
@workkit/d1 90KiB 90KiB no change
@workkit/do 58KiB 58KiB no change
@workkit/env 43KiB 43KiB no change
@workkit/errors 40KiB 40KiB no change
@workkit/features 23KiB 23KiB no change
@workkit/health 19KiB 19KiB no change
@workkit/hono 36KiB 36KiB no change
@workkit/kv 52KiB 52KiB no change
@workkit/logger 24KiB 24KiB no change
@workkit/mail 54KiB 54KiB no change
@workkit/mcp 127KiB 127KiB no change
@workkit/memory 41KiB 41KiB no change
@workkit/notify 290KiB 290KiB no change
@workkit/pdf 36KiB 36KiB no change
@workkit/queue 54KiB 54KiB no change
@workkit/r2 57KiB 57KiB no change
@workkit/ratelimit 51KiB 51KiB no change
@workkit/realtime 44KiB 44KiB no change
@workkit/remix 34KiB 34KiB no change
@workkit/testing 114KiB 114KiB no change
@workkit/turnstile 14KiB 14KiB no change
@workkit/types 21KiB 21KiB no change
@workkit/workflow 46KiB 46KiB no change

Copy link
Copy Markdown
Owner Author

beeeku commented May 23, 2026

Branch audit: this PR's diff and #114's diff are byte-identical — both add:

+.maina/cache/
+.maina/context/
+.maina/*.db

#114 does not contain the .main/ (missing a) typo described in this PR's body. That claim originated from CodeRabbit's pre-merge "Title check" on #114, which hallucinated .main/ even though the actual hunk used .maina/. The CodeRabbit walkthrough in the same comment further refers to the patterns as .main/cache/ etc. — so the bot confused itself and the title-mismatch warning followed from the bad summary, not from the diff.

Net: #114 is correct on its own and has been sitting green (verify, Type Check, Test (Node 22), Lint, Bundle Size, Constitution all ✅) for ~22 days. Suggest closing this PR as a duplicate and merging #114 directly. Happy to close this one on your signal.


Generated by Claude Code

@beeeku beeeku closed this May 23, 2026
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