copilot/frosty
1
0
Fork 0
Code Issues Pull requests Projects Releases 11 Packages Wiki Activity Actions

docs(agents): add fork-pin gotcha, test layout, pre-push check

Browse source 
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Tommy Chow 2026-07-02 18:42:48 -04:00
parent e2dff06f8d
commit 65974ef954
1 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

5
AGENTS.md
View file

@ -34,6 +34,8 @@ dart run build_runner build # Regenerate
dart run build_runner build --delete-conflicting-outputs # Same, but clears stale outputs dart run build_runner build --delete-conflicting-outputs # Same, but clears stale outputs
``` ```
Before pushing: `flutter analyze && flutter test`.
## Source Structure ## Source Structure
- `lib/apis/` — API services (all extend `BaseApiClient`) - `lib/apis/` — API services (all extend `BaseApiClient`)
@ -54,10 +56,11 @@ dart run build_runner build --delete-conflicting-outputs # Same, but
- After changing MobX stores or `@JsonSerializable` models, regenerate with `dart run build_runner build`. Never edit `.g.dart` files directly. Commit `.g.dart` files to source control. - After changing MobX stores or `@JsonSerializable` models, regenerate with `dart run build_runner build`. Never edit `.g.dart` files directly. Commit `.g.dart` files to source control.
- The secure storage cleanup in `main.dart` looks unnecessary but handles an Android/iOS edge case where secure storage persists after uninstall. Don't remove it. - The secure storage cleanup in `main.dart` looks unnecessary but handles an Android/iOS edge case where secure storage persists after uninstall. Don't remove it.
- Several deps in `pubspec.yaml` are git-pinned forks carrying compatibility patches (`extended_text_field`, `better_native_video_player`, `simple_pip_mode`). When upgrading dependencies, don't swap them back to pub.dev releases.
## Testing ## Testing
- `flutter test` runs all tests; `flutter test test/path/to/file.dart` for a single file - `flutter test` runs all tests; `flutter test test/path/to/file.dart` for a single file
- HTTP mocking: `http_mock_adapter` (`DioAdapter`) — use full URLs in `onGet`/`onPost` - HTTP mocking: `http_mock_adapter` (`DioAdapter`) — use full URLs in `onGet`/`onPost`
- General mocking: `mocktail` (no codegen required) - General mocking: `mocktail` (no codegen required)
- Fixtures live in `test/fixtures/` (e.g., `irc_messages.dart`, `api_responses.dart`) - Test layout mirrors `lib/` (`test/apis/`, `test/models/`, `test/screens/`); shared fixtures live in `test/fixtures/` (e.g., `irc_messages.dart`, `api_responses.dart`)