Skip to content
This repository was archived by the owner on Aug 2, 2026. It is now read-only.

chore(agents): clean up Claude instructions and rules - #87

Merged
vancura merged 1 commit into
mainfrom
claude-cleanup
Jul 29, 2026
Merged

vancura merged 1 commit into
mainfrom
claude-cleanup

Conversation

@vancura

@vancura vancura commented Jul 29, 2026 •

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Documentation
    • Reorganized and expanded the project guidance with a clearer overview of the scaffolding workflow.
    • Added documentation covering template generation, file ownership, synchronization, formatting, and project tooling.
    • Clarified kit content constraints, drift auditing practices, and review guidance for kit files.
    • Removed the dedicated American English spelling rules document.

Signed-off-by: Vaclav Vancura <commit@vancura.dev>
@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: d93a648c-6634-481c-a37a-56b8e7e191e0

📥 Commits

Reviewing files that changed from the base of the PR and between 4ca6dbc and 172fc66.

📒 Files selected for processing (2)
  • .claude/rules/american-english-spelling.md
  • CLAUDE.md
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • blit386/blit386 (auto-detected) → reviewed against open PR #437 claude-cleanup instead of the default branch
💤 Files with no reviewable changes (1)
  • .claude/rules/american-english-spelling.md

📝 Walkthrough

Walkthrough

CLAUDE.md was reorganized into a scaffold-flow guide covering generation, kit copying, ownership, drift auditing, repository rules, kit review points, and targeted information sources. The American English spelling rule file was removed.

Changes

Scaffolding documentation refresh

Layer / File(s) Summary
Scaffold flow and repository rules
CLAUDE.md
Documents template rendering, adapter outputs, verbatim kit copying, ownership manifests, generator reuse, Git conventions, MCP guidance, and agent drift checks.
Kit content and drift guidance
CLAUDE.md, .claude/rules/american-english-spelling.md
Clarifies engine-versus-kit boundaries, audit triggers, and review points for kit files; removes the American English spelling rule.
Repository information lookups
CLAUDE.md
Updates references for scaffolding, CLI behavior, migrations, ownership, hot reload, APIs, and release guidance.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning No PR description was provided, so the required Summary and Checklist sections are missing. Add a Summary plus the checklist items from the template, including DCO sign-off, preflight status, kit-doc updates, and AI trailer info if applicable.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise, conventional, and matches the PR's Claude instructions/rules cleanup.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude-cleanup

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

@vancura
vancura merged commit daaa693 into main Jul 29, 2026
9 checks passed
@vancura
vancura deleted the claude-cleanup branch July 29, 2026 19:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant