Skip to content

chore(gitignore): ignore L3 self-dogfood artifacts (.githooks, l3-stability.yml)#66

Merged
broomva merged 1 commit into
mainfrom
chore/gitignore-l3-dogfood-artifacts
May 29, 2026
Merged

chore(gitignore): ignore L3 self-dogfood artifacts (.githooks, l3-stability.yml)#66
broomva merged 1 commit into
mainfrom
chore/gitignore-l3-dogfood-artifacts

Conversation

@broomva
Copy link
Copy Markdown
Owner

@broomva broomva commented May 29, 2026

What

Adds .githooks/ and .github/workflows/l3-stability.yml to the bstack repo's self-dogfood .gitignore block (alongside the already-ignored .control/ + .claude/).

Why

Surfaced while validating v0.22.0 (#65) on the bstack repo itself: running bstack bootstrap / install-rcs-stability against the source repo scaffolds .githooks/pre-commit + .github/workflows/l3-stability.yml. The source repo uses .github/workflows/ci.yml + .git/hooks (its core.hooksPath), so those are generated artifacts that should never be committed — same class as .control/ and .claude/. Before this, every self-dogfood run left two untracked files polluting git status.

Verification (P11)

  • .githooks/pre-commit + .github/workflows/l3-stability.yml → now ignored.
  • ✅ Real tracked CI (ci.yml, release.yml, validate-release.yml) → not ignored (the rule is scoped to l3-stability.yml specifically, not all of .github/workflows/).
  • ✅ Fresh bstack bootstrap on the bstack repo that creates both artifacts → git status shows only the .gitignore change, no pollution.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated internal configuration to ignore locally generated development artifacts from bootstrap and installation workflows.

Review Change Stack

…bility.yml)

Running `bstack bootstrap`/install-rcs against the bstack repo itself (self-dogfood,
e.g. validating v0.22.0) scaffolds .githooks/pre-commit + .github/workflows/
l3-stability.yml. The source repo uses .github/workflows/ci.yml + .git/hooks, so
these are generated artifacts that should never be committed — same class as the
already-ignored .control/ and .claude/. Adds them to the self-dogfood ignore block.

Precise: only l3-stability.yml is ignored under .github/workflows/, so the real
tracked CI (ci.yml, release.yml, validate-release.yml) is unaffected — verified.
Verified bootstrap now leaves a clean tree on the bstack repo (only .gitignore M).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 14195f60-82b6-4846-83fc-97ac085caaf5

📥 Commits

Reviewing files that changed from the base of the PR and between ccbf530 and 314a3b8.

📒 Files selected for processing (1)
  • .gitignore

📝 Walkthrough

Walkthrough

This PR extends .gitignore to ignore locally generated scaffolding artifacts from bstack's bootstrap/doctor/install-rcs self-dogfood workflow, including .control/, .claude/, .githooks/ directories and the generated .github/workflows/l3-stability.yml file.

Changes

Installer scaffolding ignore

Layer / File(s) Summary
Installer artifacts in gitignore
.gitignore
New entries and comments ignore the .control/, .claude/, .githooks/ directories and .github/workflows/l3-stability.yml file that are generated locally by the bstack self-dogfood installer workflow.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A tidy .gitignore, oh what a sight,
Scaffolding artifacts bundled just right,
.control/, .claude/, and .githooks/ too,
Gone from the repo, kept out of view! 📦✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically summarizes the main change: adding L3 self-dogfood artifacts to gitignore, with concrete examples (.githooks, l3-stability.yml) that match the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/gitignore-l3-dogfood-artifacts

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.

@broomva broomva merged commit ca46568 into main May 29, 2026
5 checks passed
@broomva broomva deleted the chore/gitignore-l3-dogfood-artifacts branch May 29, 2026 14:00
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.

1 participant