Renamed files to kebab-case - core - services - part 2#8
Conversation
🤖 Augment PR SummarySummary: This PR continues the core/services rename effort by converting many server-side service modules to kebab-case filenames and updating all affected imports. Changes:
Technical Notes: The change is primarily mechanical but correctness depends on every updated import matching an on-disk filename (especially on case-sensitive filesystems). 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
| const EmailEventProcessor = require('../email-service/EmailEventProcessor'); | ||
| const MailgunProvider = require('./EmailAnalyticsProviderMailgun'); | ||
| const EmailAnalyticsService = require('./email-analytics-service'); | ||
| const EmailEventStorage = require('../email-service/email-event-storage'); |
There was a problem hiding this comment.
require('../email-service/email-event-storage') doesn’t match an existing file under services/email-service (it’s currently EmailEventStorage.js), so this will fail on case-sensitive filesystems. Same issue for email-event-processor.
Other Locations
ghost/core/core/server/services/email-analytics/email-analytics-service-wrapper.js:13
🤖 Was this useful? React with 👍 or 👎
| const WellKnownController = require('./controllers/well-known-controller'); | ||
|
|
||
| const {EmailSuppressedEvent} = require('../../email-suppression-list/email-suppression-list'); | ||
| const MagicLink = require('../../lib/magic-link/magic-link'); |
There was a problem hiding this comment.
require('../../lib/magic-link/magic-link') doesn’t correspond to a file in services/lib/magic-link (currently MagicLink.js), so this will throw "Cannot find module" on case-sensitive filesystems.
🤖 Was this useful? React with 👍 or 👎
2 participants
Benchmark PR from agentic-review-benchmarks#7