Skip to content

Remove dead code and update repository metadata to FalkorDB#33

Open
Claude wants to merge 5 commits into
mainfrom
claude/remove-dead-code-and-update-metadata
Open

Remove dead code and update repository metadata to FalkorDB#33
Claude wants to merge 5 commits into
mainfrom
claude/remove-dead-code-and-update-metadata

Conversation

@Claude

@Claude Claude AI commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

The repository contained unused legacy files and package.json metadata pointing to the previous owner's repository instead of the FalkorDB organization.

Changes

Dead code removal:

  • Removed src/utils/connection-parser.ts - unused connection string parser (config uses env vars directly)
  • Removed src/utils/connection-parser.test.ts - 21 tests for unused functionality
  • Removed src/models/mcp.types.ts - unused type definitions with no imports

Package.json metadata updates:

  • Updated repository, bugs, homepage URLs: SecKatie/falkordb-mcpserverFalkorDB/FalkorDB-MCPServer
  • Updated funding URL: sponsors/SecKatiesponsors/FalkorDB
Original prompt

This section details on the original issue you should resolve

<issue_title>Dead Code Removal & Metadata Correction</issue_title>
<issue_description>Description: The repository contains unused legacy files (connection-parser.ts, mcp.types.ts) and package.json metadata still points to the previous owner's repository.

Impact: Increases maintenance overhead and causes confusion for new contributors.

Recommended Fix: Delete identified unused files and update package.json with correct FalkorDB repository and homepage URLs.

Affected Files: package.json, src/utils/connection-parser.ts, src/models/mcp.types.ts.</issue_description>

Comments on the Issue (you are @claude[agent] in this section)

Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
@Claude Claude AI changed the title [WIP] Remove unused legacy files and update package metadata Remove dead code and update repository metadata to FalkorDB Feb 23, 2026
@Claude Claude AI requested a review from gkorland February 23, 2026 08:28
@gkorland gkorland marked this pull request as ready for review February 23, 2026 09:07
@codecov

codecov Bot commented Feb 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 44.49%. Comparing base (4db4fcd) to head (c3f25c7).
⚠️ Report is 9 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #33      +/-   ##
==========================================
- Coverage   48.55%   44.49%   -4.07%     
==========================================
  Files          11       10       -1     
  Lines         451      418      -33     
  Branches      105       90      -15     
==========================================
- Hits          219      186      -33     
  Misses        232      232              
Flag Coverage Δ
unittests 44.49% <ø> (-4.07%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

Dead Code Removal & Metadata Correction

2 participants