Skip to content

Renamed files to kebab-case - core - services - part 2#9

Open
tomerqodo wants to merge 2 commits intocursor_full_base_renamed_files_to_kebab-case_-_core_-_services__-_part_2_pr7from
cursor_full_head_renamed_files_to_kebab-case_-_core_-_services__-_part_2_pr7
Open

Renamed files to kebab-case - core - services - part 2#9
tomerqodo wants to merge 2 commits intocursor_full_base_renamed_files_to_kebab-case_-_core_-_services__-_part_2_pr7from
cursor_full_head_renamed_files_to_kebab-case_-_core_-_services__-_part_2_pr7

Conversation

@tomerqodo
Copy link
Copy Markdown

@tomerqodo tomerqodo commented Jan 25, 2026

Benchmark PR from agentic-review-benchmarks#7


Note

Standardizes file naming across core to kebab-case for consistency and case-sensitive platforms.

  • Updates require/import paths throughout API endpoints, services (members, email, stats, comments, identity-tokens, activitypub, etc.), and tests
  • Minor whitespace/comment tweaks; no functional logic changes intended

Written by Cursor Bugbot for commit 092595d. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

const MailgunProvider = require('./EmailAnalyticsProviderMailgun');
const EmailAnalyticsService = require('./email-analytics-service');
const EmailEventStorage = require('../email-service/email-event-storage');
const EmailEventProcessor = require('../email-service/email-event-processor');
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Import paths reference non-existent kebab-case files

High Severity

The import paths for EmailEventStorage and EmailEventProcessor were changed to kebab-case (email-event-storage and email-event-processor), but the actual source files remain named in PascalCase (EmailEventStorage.js and EmailEventProcessor.js). This mismatch causes a MODULE_NOT_FOUND error at runtime because the files at the new paths don't exist.

Fix in Cursor Fix in Web

const WellKnownController = require('./controllers/well-known-controller');

const {EmailSuppressedEvent} = require('../../email-suppression-list/email-suppression-list');
const MagicLink = require('../../lib/magic-link/magic-link');
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

MagicLink import path references non-existent kebab-case file

High Severity

The import path for MagicLink was changed to kebab-case (../../lib/magic-link/magic-link), but the actual source file remains named in PascalCase (MagicLink.js). This mismatch causes a MODULE_NOT_FOUND error at runtime because the file magic-link.js doesn't exist.

Fix in Cursor Fix in Web

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