refactor: remove liberate backward compat code

[TonyCasey]

Summary

• Remove deprecated liberate CLI alias
• Remove liberate from SourceType union, SOURCE_VALUES, DEFAULT_CONFIDENCE
• Remove autoLiberate → autoExtract migration in loadHookConfig and deepMergeGitMemConfig
• Remove 4 backward compat tests

Pre-release project — no migration shims needed.

Closes GIT-72 (cleanup follow-up)

Test plan

• npm run type-check passes
• npm run lint passes
• All 268 unit tests pass

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Removed the deprecated liberate CLI command; use extract command instead.
  • Removed backward-compatibility migrations for deprecated liberate configuration options.
  • Removed associated tests for deprecated functionality.

[chatgpt-codex-connector]

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]

📝 Walkthrough

Walkthrough

This PR removes deprecated "liberate" command alias, related configuration migration logic, and associated type definitions throughout the codebase. The changes consolidate functionality around the "extract" command by eliminating backward-compatibility code that previously migrated autoLiberate to autoExtract configuration.

Changes

Cohort / File(s)	Summary
CLI Command src/cli.ts	Removed deprecated liberate top-level command including all options (--since, --commit-count, --dry-run, --threshold, --enrich) and action handler that forwarded to extractCommand.
Configuration & Migration Logic src/commands/init-hooks.ts, src/hooks/utils/config.ts	Removed backward-compatibility migration code that previously converted sessionStop.autoLiberate to sessionStop.autoExtract; migration no longer occurs.
Type Definitions src/domain/types/IMemoryQuality.ts	Removed liberate from SourceType union, SOURCE_VALUES array, and DEFAULT_CONFIDENCE mapping; removed associated deprecation comments.
Unit Tests tests/unit/commands/init-hooks.test.ts, tests/unit/hooks/utils/config.test.ts	Removed tests covering autoLiberate migration and precedence logic; remaining tests for defaults and merging behavior preserved.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• refactor: rename liberate to extract across codebase (GIT-72) #37: Refactored liberate to extract command, and this PR completes the cleanup by removing the deprecated alias and migration logic.
• test: integration tests for hook entry points (GIT-58) #27: Added integration tests using autoLiberate and session-stop hooks that will need alignment with the removed backward-compatibility migration.
• feat: unified git-mem hook command + safe settings.json merge (GIT-60) #25: Modified deepMergeGitMemConfig and hook enablement logic in the same code paths affected by the migration removals.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and concisely describes the main change: removing deprecated liberate backward compatibility code across multiple files and tests.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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 git-72-cleanup

---

No actionable comments were generated in the recent review. 🎉

---

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

[Copilot]

Pull request overview

This PR removes all backward compatibility code for the deprecated "liberate" terminology, which has been replaced with "extract". As noted in the description, since this is a pre-release project, no migration shims are needed. The changes cleanly remove deprecated aliases, type definitions, and migration logic.

Changes:

• Removed deprecated liberate CLI command alias
• Removed liberate from SourceType union type, SOURCE_VALUES array, and DEFAULT_CONFIDENCE mapping
• Removed autoLiberate → autoExtract migration logic from configuration loading functions
• Removed 4 backward compatibility tests

Reviewed changes

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

Show a summary per file

File	Description
src/cli.ts	Removed deprecated liberate CLI command that was an alias for extract
src/domain/types/IMemoryQuality.ts	Removed liberate from source type definitions and related constants
src/hooks/utils/config.ts	Removed autoLiberate → autoExtract migration logic from config loader
src/commands/init-hooks.ts	Removed autoLiberate → autoExtract migration logic from deep merge function
tests/unit/hooks/utils/config.test.ts	Removed 2 backward compatibility tests for autoLiberate migration
tests/unit/commands/init-hooks.test.ts	Removed 2 backward compatibility tests for autoLiberate migration

---

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

[Copilot]

The rawStop variable is no longer needed after removing the migration code. For consistency with sessionStart and promptSubmit configuration handling (lines 39-42 and 47-50), consider using rawHooks.sessionStop directly in the spread on line 45 instead of this intermediate variable.