This file contains instructions for AI coding assistants working on the Forem codebase (the platform that powers dev.to). Follow these rules and patterns to ensure high-quality contributions.
- Follow Patterns: Application consistency is key. Mimic existing patterns for controllers, services, and specs.
- Smaller is Better: Prefer atomic, focused modifications over sprawling refactors.
- Re-usability: Changes should strengthen Forem as a platform, avoiding DEV-specific hardcoding unless absolutely necessary.
- Regression Tests are Mandatory: specific regression tests to verify your code works are required for almost all PRs.
- Follow Test Patterns: Use
create(:factory)syntax (FactoryBot) and standard RSpec expectations. - Frontend vs Backend: We are currently focused on robust backend regression tests. Frontend changes require more manual user review, so clear descriptions of UI changes are vital.
- Update All Locales: If your change involves new or modified text, you MUST update the corresponding i18n files for ALL supported languages found in
config/locales. - Supported Languages:
en(English)fr(French)pt(Portuguese)- And any others present in
config/locales.
- Avoid
current_userin Cache: Never usecurrent_userobjects in cached pages or partials to prevent cache leaks and private data exposure. - Database Indexes:
- Add indexes concurrently using
algorithm: :concurrently. - Use
disable_ddl_transaction!in the migration class. - Ideally, place index additions in their own separate migration files.
- Add indexes concurrently using
- We are transitioning to a Preact-first frontend.
- Use
app/javascriptandapp/assetspatterns as established.
- If you find documentation that contradicts the codebase, trust the codebase patterns but note the discrepancy.
If the schema.rb file is modified with extra changes not tied to the migration due to branch collision, etc, fix the schema.rb file to match the work being done in the migration.