Skip to content

Conversation

@rossmanko
Copy link
Contributor

@rossmanko rossmanko commented Feb 1, 2026

Summary by CodeRabbit

  • Refactor
    • Removed file read and write capabilities from AI toolset
    • Removed pattern matching tool from AI toolset
    • Simplified message processing by removing model-specific reasoning handling
    • Updated internal module organization and imports

✏️ Tip: You can customize this high-level summary in your review settings.

rossmanko and others added 2 commits February 1, 2026 08:41
…easoning strip

- Add generationTimeMs and finishReason to fallback mode saveMessage call
- Remove stripReasoningFromMessagesForGemini logic (no longer needed)
- Remove unused stripReasoningForGemini functions from message-processor

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Deleted the read-file, write-file, and match tools as they are no longer needed.
- Updated the index file to reflect these removals and reorganized imports for clarity.
- Moved perplexity-related functions to a new utils file for better structure.
@vercel
Copy link

vercel bot commented Feb 1, 2026

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

Project Deployment Actions Updated (UTC)
hackerai Ready Ready Preview, Comment Feb 1, 2026 4:40pm

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

📝 Walkthrough

Walkthrough

This PR removes three AI tool implementations (match, read-file, write-file) from the codebase, unregisters them from the tools index, eliminates Gemini-specific reasoning-stripping utilities, refactors chat message handling to use summarizedMessages instead of finalMessages, and updates import paths for better module organization.

Changes

Cohort / File(s) Summary
Removed tool implementations
lib/ai/tools/match.ts, lib/ai/tools/read-file.ts, lib/ai/tools/write-file.ts
Deleted three complete tool definitions including sandbox-based search/file-read/file-write functionality with input validation, execution logic, error handling, and helper utilities.
Tools index update
lib/ai/tools/index.ts
Removed imports and tool registrations for read_file, write_file, and match tools from allTools and mode-specific tool subsets.
Perplexity module reorganization
lib/ai/tools/web-search.ts
Updated import path from ./perplexity to ./utils/perplexity without changing runtime behavior.
Gemini reasoning logic removal
lib/utils/message-processor.ts
Deleted exported functions stripReasoningForGemini, stripReasoningFromMessagesForGemini, and helper isReasoningPart that filtered reasoning parts for Gemini models.
Chat handler refactoring
lib/api/chat-handler.ts
Removed Gemini-specific reasoning-stripping invocations, replaced finalMessages with summarizedMessages in message streams, and added generationTimeMs and finishReason metadata to retry and message persistence paths.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Possibly related PRs

Poem

🐰 The tools hop away, reasoning fades from sight,
Gemini's thoughts take a quieter flight.
Files and matches vanish clean,
Leaving streamlined paths between.
A simpler toolbox, fresh and bright! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Daily branch 2026 02 01' is generic and does not clearly summarize the primary changes in the pull request, which involve removing AI tools and Gemini-specific reasoning logic. Consider using a more descriptive title that reflects the main objective, such as 'Remove deprecated AI tools and Gemini reasoning logic' or 'Clean up unused tool implementations and reasoning utilities'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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 daily-branch-2026-02-01

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

@rossmanko rossmanko merged commit 1376b16 into main Feb 1, 2026
4 checks passed
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