Skip to content

chore(repo): clean up repo root and consolidate editor rules - #7758

Merged
waleedlatif1 merged 4 commits into
stagingfrom
chore/root-cleanup
Sep 11, 2026
Merged

chore(repo): clean up repo root and consolidate editor rules#7758
waleedlatif1 merged 4 commits into
stagingfrom
chore/root-cleanup

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Remove .devcontainer/ and its CONTRIBUTING section; the Dev Containers setup was unmaintained
  • Remove renovate.json; Renovate has never opened a PR or dashboard issue here, Dependabot is what runs
  • Remove ci-cache-cleanup.yml; it only drained legacy Next.js build-cache sticky disks from PRs open before perf(ci): disable the Turbopack persistent build cache (3.2x faster builds) #6080, and Blacksmith's 7-day idle eviction has already reclaimed them
  • Move vitest.scripts.config.ts to scripts/vitest.config.ts with a pinned root so the repo root holds no test config
  • Drop the duplicate husky prepare in apps/sim; the root prepare installs the hooks
  • Make .claude/rules/<name>.md the single source for editor rules and generate .cursor/rules/<name>.mdc from it via skills:sync; the Cursor copies had drifted since July and five newer rules never reached Cursor. Pre-commit regenerates on staged rule edits, check:skills verifies
  • Move the Cursor-only testing rule to .claude/rules/sim-testing.md
  • AGENTS.md is now a symlink to CLAUDE.md; it was a stale subset

Type of Change

  • Chore

Testing

  • bun run test:scripts (18 files, 298 tests) and bun run check:script-tests pass from the root and from scripts/
  • bun run skills:sync is idempotent and bun run check:skills passes
  • bun run lint, bun run check:audits (46 audits), docs-manifest:check, block-registry check all pass

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

🤖 Generated with Claude Code

https://claude.ai/code/session_01WibCGEe4oUWD59QuMayCKe

…cleanup workflow

- Delete .devcontainer/ and its CONTRIBUTING section; the Dev Containers
  setup was undocumented elsewhere and no longer maintained
- Delete renovate.json: Renovate has never opened a PR or dashboard issue
  on this repo; Dependabot security updates are what actually run
- Delete ci-cache-cleanup.yml: it drained legacy Next.js build-cache sticky
  disks left by PRs open before #6080 (2026-07-30); Blacksmith evicts sticky
  disks after 7 days idle, so the backlog is gone and the job is a no-op
- Move vitest.scripts.config.ts to scripts/vitest.config.ts with a pinned
  root so the repo root holds no test config; update test:scripts and
  check:script-tests
- Drop the duplicate husky `prepare` in apps/sim; the root prepare installs
  the hooks
…GENTS.md

- .claude/rules/<name>.md is now the single source for editor rules; the
  sync script projects .cursor/rules/<name>.mdc from it (description +
  globs, or alwaysApply when a rule has no paths). The Cursor copies had
  drifted since July and five newer rules never reached Cursor
- Move the Cursor-only testing rule to .claude/rules/sim-testing.md and add
  a description to every canonical rule
- Pre-commit regenerates on staged rule edits; check:skills verifies
- AGENTS.md becomes a symlink to CLAUDE.md; it was a stale subset
@waleedlatif1
waleedlatif1 requested a review from a team as a code owner September 11, 2026 02:35
@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 11, 2026 2:49am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge with no outstanding actionable findings.

Summary

  • Removes the unused Dev Container, Renovate configuration, and legacy cache-cleanup workflow.
  • Generates Cursor rules from .claude/rules and integrates synchronization into pre-commit and CI checks.
  • Relocates the scripts Vitest configuration while preserving root-independent test discovery.
  • Replaces the stale AGENTS.md copy with a symlink to CLAUDE.md.

Reviews (3) · Last reviewed commit: "fix(agents): reject implicitly typed YAM..."

Comment thread scripts/sync-skills.ts Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

Comment thread scripts/sync-skills.ts Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1
waleedlatif1 merged commit 3b8ad1a into staging Sep 11, 2026
30 of 31 checks passed
@waleedlatif1
waleedlatif1 deleted the chore/root-cleanup branch September 11, 2026 02:49
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