chore: ignore auto-generated files (*.g.dart, *.mocks.dart)

[Catrya]

There was an inconsistency in how we handled generated files:

• CLAUDE.md documented "Generated files should never be committed"
• CI workflows already regenerate *.g.dart and *.mocks.dart files (since fix: add missing build_runner step to flutter.yml workflow #375)
• BUT these files were still tracked in git and committed in PRs
• This caused: noisy PRs, merge conflicts, and difficult code reviews

Example: A change to Settings class generates +30 lines in mocks.mocks.dart that pollute the PR diff.

This PR completes the work from #375 by adding generated files to .gitignore:

Before:

• Generated files committed to repo
• CI regenerates them (redundant)
• PRs show hundreds of lines of generated code

After:

• Generated files in .gitignore (not committed)
• CI regenerates them automatically
• PRs only show actual source code changes
• Developers generate locally with dart run build_runner build -d

This aligns with Flutter ecosystem standards (Flutter SDK, Riverpod, Provider all do this).

⚠️ Action Required

After pulling this PR, all developers must run:

dart run build_runner build -d                                                                                      

Git will delete the generated files from your workspace (since they're now untracked). This command regenerates them locally so the app compiles.

You only need to do this once after pulling this PR. The README and CONTRIBUTING.md now document this as a mandatory setup step for new contributors.

Summary by CodeRabbit

• Documentation

  • Enhanced setup instructions with emphasis on required code generation step
  • Added detailed guidance on generated files workflow and best practices
  • Updated contributor guidelines to clarify development setup process

• Chores

  • Configured git to ignore auto-generated files
  • Removed generated provider files for regeneration in CI/CD pipeline

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

[coderabbitai]

Walkthrough

This PR enforces a code generation workflow by adding generated file types to .gitignore, removing previously committed generated Riverpod providers, and updating documentation to clearly establish that generated files must be regenerated locally and in CI/CD rather than committed to the repository.

Changes

Cohort / File(s)	Summary
Configuration & Documentation .gitignore, CLAUDE.md, README.md, docs/CONTRIBUTING.md	Updated .gitignore to exclude *.g.dart and *.mocks.dart files. Expanded CLAUDE.md, README.md, and CONTRIBUTING.md with comprehensive guidance on generated file handling, CI/CD workflows, and regeneration requirements. Reorganized CONTRIBUTING.md steps to prioritize code generation and added dedicated "Generated Files & Git Workflow" section.
Generated Riverpod Provider Removals lib/features/order/providers/order_notifier_provider.g.dart, lib/services/event_bus.g.dart, lib/shared/providers/mostro_service_provider.g.dart	Removed committed generated Riverpod provider declarations, hash functions, and associated typedefs. Specifically: removed orderTypeNotifierProvider, eventBusProvider, eventStorageProvider, and mostroServiceProvider along with their ref typedefs and hash functions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix: add missing build_runner step to flutter.yml workflow #375 — Complements this PR by adding the CI step to run build_runner to regenerate the removed *.g.dart and *.mocks.dart files
• Implementation of Background Services & Notification System #74 — Previously modified the same Riverpod provider .g.dart files now being removed in this PR

Suggested reviewers

• grunch

Poem

🐰 Generated files, no more in git's sight,
Rebuild them fresh with each pull and write,
Clean diffs ahead, no conflicts in the way,
CI keeps us honest, hooray, hooray! 🎉

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and concisely summarizes the primary change: adding auto-generated files to .gitignore to prevent them from being committed.
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.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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