Skip to content

chore: repository reorganization and cleanup#32

Merged
flyingrobots merged 6 commits intomainfrom
chore/cleaning-repo
Feb 15, 2026
Merged

chore: repository reorganization and cleanup#32
flyingrobots merged 6 commits intomainfrom
chore/cleaning-repo

Conversation

@flyingrobots
Copy link
Member

@flyingrobots flyingrobots commented Feb 15, 2026

This PR performs a repository reorganization and cleanup, including:

  • Reorganization: Moved Dockerfiles to docker/, documentation to docs/, and example scripts to examples/scripts/.
  • Hooks: Consolidated git hooks and updated HookInstaller to point to the new scripts/hooks/post-merge.sh.
  • Infrastructure: Added rationale for dependency patches in patches/README.md.
  • Build/Lint: Resolved ESLint parsing errors by correcting tsconfig inclusion patterns and updated package.json scripts.
  • Cleanup: Removed FIXUPS.md and pnpm-lock.yaml (migrated to npm).
  • Environment: Added docker/Dockerfile.node22-slim for optimized builds.

Summary by CodeRabbit

Release Notes

  • New Features

    • Added Node.js 22 Slim Docker image support for containerized environments
  • Chores

    • Bumped version to 11.0.0
    • Reorganized example scripts under new directory structure
    • Updated documentation and walkthrough references
    • Simplified post-merge hook behavior
    • Updated workspace dependency configuration
    • Added patch documentation for dependency modifications

- Moved hero.gif and hero.tape to docs/images/
- Moved TYPESCRIPT_ZERO.md to docs/
- Updated README.md hero image reference
- Updated docs/images/hero.tape output path
- Removed FIXUPS.md (completed checklist)
- Removed pnpm-lock.yaml and pnpm-workspace.yaml (switching to npm)
- Sync jsr.json version to 11.0.0
- Move root Dockerfiles to docker/ directory
- Create root symlink for Dockerfile -> docker/Dockerfile.node22-slim
- Update docker-compose.yml to use consolidated Dockerfiles
- Move example scripts to examples/scripts/
- Update package.json, WALKTHROUGH.md, and HTML examples to reflect new script paths
- Removed redundant src/hooks/ directory (scripts/hooks/ is source of truth)
- Moved types/git-cas.d.ts to src/domain/types/git-cas.d.ts
- Updated tsconfig.json and tsconfig.src.json to include src/**/*.d.ts
- Rename scripts/hooks/post-merge to post-merge.sh for consistency
- Update bin/cli/shared.js to use new templateDir path
- Update HookInstaller.test.js to reflect new source of truth
…atterns

- Removed explicit .d.ts glob inclusion from tsconfig files, allowing .js files to be correctly matched.
- Restored eslint.config.js to use the high-performance projectService.
@coderabbitai
Copy link

coderabbitai bot commented Feb 15, 2026

Caution

Review failed

Failed to post review comments

📝 Walkthrough

Walkthrough

This PR reorganizes the project structure by relocating hooks from src/hooks to scripts/hooks, moving example scripts under examples/scripts/, updating Docker infrastructure with a new Node.js 22 Slim image, bumping the version to 11.0.0, and updating related documentation and configuration files accordingly.

Changes

Cohort / File(s) Summary
Hook Script Reorganization
bin/cli/shared.js, scripts/hooks/post-merge, scripts/hooks/post-merge.sh, test/unit/domain/services/HookInstaller.test.js
Updated template search path from src/hooks to scripts/hooks in hook installer; post-merge script simplified and hook file removed; test paths updated to match new hook location.
Example Scripts Relocation
examples/WALKTHROUGH.md, examples/html/event-sourcing.html, examples/html/explore.html, examples/html/inspect-index.html, examples/html/lagrangian-path.html, examples/html/multi-writer.html, examples/html/setup.html, examples/html/streaming-benchmark.html, examples/html/traversal-benchmark.html, package.json
Updated all script path references from root examples directory to examples/scripts/ subdirectory; walkthrough documentation and demo npm scripts adjusted accordingly.
Docker Configuration
docker/Dockerfile.node22-slim, docker-compose.yml
Added new Node.js 22 Slim Dockerfile with build tools and git-warp integration; updated docker-compose services (test, benchmark) to reference new image paths under docker/.
Version & Configuration Updates
jsr.json, tsconfig.json, tsconfig.src.json, pnpm-workspace.yaml
Version bumped to 11.0.0; removed TypeScript declaration file glob patterns from tsconfig files; removed three entries (cbor-extract, esbuild, roaring) from pnpm workspace onlyBuiltDependencies.
Documentation & Assets
README.md, docs/images/hero.tape, patches/README.md, FIXUPS.md
Updated hero image path reference in README; updated output target in hero.tape build asset; added patches documentation explaining @mapbox/node-pre-gyp patch rationale; removed FIXUPS.md changelog file.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

🐰 The warren was reorganized with care,
Hooks moved to scripts, examples everywhere!
Docker images built, configs set just right,
Version eleven now shines so bright! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: a comprehensive repository reorganization and cleanup including file relocations, Docker refactoring, script consolidation, and dependency updates.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/cleaning-repo

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

Release Preflight

  • package version: 11.0.0
  • prerelease: false
  • npm dist-tag on release: latest
  • npm pack dry-run: passed
  • jsr publish dry-run: passed

If you tag this commit as v11.0.0, release workflow will publish.

@flyingrobots flyingrobots merged commit 6f27bc6 into main Feb 15, 2026
7 checks passed
@flyingrobots flyingrobots deleted the chore/cleaning-repo branch February 15, 2026 15:27
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