Skip to content

commit-count-change#46

Merged
TonyCasey merged 2 commits intomainfrom
commit-count-change
Feb 13, 2026
Merged

commit-count-change#46
TonyCasey merged 2 commits intomainfrom
commit-count-change

Conversation

@TonyCasey
Copy link
Copy Markdown
Owner

@TonyCasey TonyCasey commented Feb 13, 2026

Summary by CodeRabbit

  • Documentation

    • Redesigned getting-started guide with an interactive quick-start setup approach, replacing detailed command-line instructions for improved user onboarding.
  • Chores

    • Updated project configuration and gitignore setup patterns.
    • Optimized initialization defaults for faster setup.

TonyCasey and others added 2 commits February 13, 2026 14:29
)

* docs: rewrite getting-started.md with current CLI and trailers info

- Full rewrite to match current codebase (8 CLI commands, unified init)
- Add Claude Code hooks section with .git-mem.json config example
- Add Git Trailers subsection to "How it works" (AI-* trailer details)
- Fix sessionStop config key in JSON example
- Remove outdated init-mcp standalone section and architecture diagrams
- Reorganize: Quick Start → Claude hooks → How it works → MCP → CLI

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* docs: address review - markdown lint, --no-trailers flag, init default

- Add blank lines around headings (MD022)
- Add text language tag to fenced code blocks (MD040)
- Document --no-trailers flag on remember command
- Fix init --commit-count default to 100 (matches src/cli.ts)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
AI-Fact: Changed initial commit count
AI-Confidence: high
AI-Memory-Id: a0b9475f-6014-477e-ad81-584f7c72fd61
Copilot AI review requested due to automatic review settings February 13, 2026 14:29
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 13, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR updates the git-mem tool's configuration and documentation. It adds .git-mem.json to the gitignore file, substantially rewrites the getting-started documentation to emphasize a guided setup-first approach instead of detailed CLI commands, and reduces the default commit count in the init command from 100 to 30.

Changes

Cohort / File(s) Summary
Configuration & Defaults
.gitignore, src/commands/init.ts
Added .git-mem.json to gitignore patterns and changed default commitCount from 100 to 30 in init command.
Documentation
docs/getting-started.md
Rewrote documentation from CLI-focused command reference to a guided quick-start flow, emphasizing interactive setup (init), simplified workflows, and high-level concepts instead of granular options and technical command details.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

✨ 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 commit-count-change

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

@TonyCasey TonyCasey merged commit 275ec5b into main Feb 13, 2026
6 of 7 checks passed
@TonyCasey TonyCasey deleted the commit-count-change branch February 13, 2026 14:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR changes the default commit count for the git-mem init command from 100 to 30. The change affects how many commits are extracted by default during initialization, making the default behavior process fewer commits.

Changes:

  • Reduced default commit count from 100 to 30 in the init command implementation
  • Added .git-mem.json to the repository's .gitignore (completing an already-existing programmatic addition)
  • Restructured and condensed the getting-started documentation

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/commands/init.ts Updated default commit count from 100 to 30 in three locations (initial value, error message, fallback)
docs/getting-started.md Major documentation restructure with condensed content, but still shows old default of 100 for --commit-count
.gitignore Added .git-mem.json to repository gitignore (aligns with programmatic gitignore entry in init command)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/getting-started.md
- **Non-invasive** — doesn't modify commits, branches, or working tree
Options:
- `-y, --yes` — Accept defaults without prompting
- `--commit-count <n>` — Number of commits to extract (default: 100)
Copy link

Copilot AI Feb 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The documentation states the default for --commit-count is 100, but the implementation in src/commands/init.ts has been changed to 30. This needs to be updated to 30 to match the implementation.

Copilot uses AI. Check for mistakes.
@TonyCasey TonyCasey restored the commit-count-change branch February 14, 2026 15:10
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