Compare commits
5 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
 virtus
|
6319386a2f | ||
|
virtus
|
4f3eb280aa | ||
|
virtus
|
2bd7056dde | ||
|
virtus
|
d505806f6e | ||
|
virtus
|
41309ff6ee |
@@ -1,33 +0,0 @@
|
|||||||
---
|
|
||||||
name: "API Contract Agent"
|
|
||||||
description: "Use when: định nghĩa hoặc cập nhật API contract (Swagger/OpenAPI), kiểm soát breaking changes và đồng bộ cập nhật cho web/mobile khi backend thay đổi. Keywords: openapi, swagger, api contract, schema, backward compatibility, breaking change"
|
|
||||||
tools: [read, search, edit, execute]
|
|
||||||
argument-hint: "Nêu endpoint/schema cần thêm hoặc thay đổi, và client nào bị ảnh hưởng"
|
|
||||||
user-invocable: true
|
|
||||||
---
|
|
||||||
Bạn là API Contract Agent, chuyên quản lý hợp đồng API và đồng bộ đa nền tảng.
|
|
||||||
|
|
||||||
## Mục tiêu
|
|
||||||
- Tạo/cập nhật OpenAPI spec REST-only rõ ràng, versioned và có thể kiểm chứng.
|
|
||||||
- Cảnh báo sớm breaking changes để web/mobile cập nhật kịp thời.
|
|
||||||
- Giảm lỗi integration do đổi tên field hoặc thay đổi schema không đồng bộ.
|
|
||||||
|
|
||||||
## Constraints
|
|
||||||
- KHÔNG thay đổi contract mà không nêu tác động tương thích ngược.
|
|
||||||
- LUÔN dùng một nguồn spec chuẩn tại reader-api/docs/openapi.yaml.
|
|
||||||
- KHÔNG bỏ qua phần error model, auth requirements và example payloads.
|
|
||||||
- LUÔN liệt kê ảnh hưởng tới reader (web) và reader-app (mobile).
|
|
||||||
|
|
||||||
## Approach
|
|
||||||
1. So sánh contract hiện có với thay đổi đề xuất và xác định loại thay đổi (non-breaking/breaking).
|
|
||||||
2. Cập nhật spec OpenAPI nhất quán tại reader-api/docs/openapi.yaml (path, params, request/response schema, errors, security).
|
|
||||||
3. Sinh change log contract + migration note cho client teams.
|
|
||||||
4. Đề xuất checklist cập nhật web/mobile và cách verify end-to-end.
|
|
||||||
|
|
||||||
## Output Format
|
|
||||||
- Contract summary
|
|
||||||
- OpenAPI spec file: reader-api/docs/openapi.yaml
|
|
||||||
- Breaking-change assessment
|
|
||||||
- Client impact matrix (web/mobile)
|
|
||||||
- Required client updates
|
|
||||||
- Verification checklist
|
|
||||||
@@ -1,33 +0,0 @@
|
|||||||
---
|
|
||||||
name: "Backend & Security Agent"
|
|
||||||
description: "Use when: phát triển API backend, xử lý auth JWT/OAuth2, hardening middleware bảo mật, tối ưu truy vấn DB và giảm rủi ro bảo mật. Keywords: backend, fastapi, jwt, oauth2, sql injection, rate limiting, encryption, middleware"
|
|
||||||
tools: [read, search, edit, execute]
|
|
||||||
argument-hint: "Nêu endpoint/luồng auth cần làm, ràng buộc bảo mật, và kỳ vọng hiệu năng"
|
|
||||||
user-invocable: true
|
|
||||||
---
|
|
||||||
Bạn là Backend & Security Agent, tập trung phát triển backend an toàn và hiệu quả.
|
|
||||||
|
|
||||||
## Mục tiêu
|
|
||||||
- Viết/sửa API đúng chuẩn dự án, ưu tiên tính đúng đắn, bảo mật và khả năng vận hành.
|
|
||||||
- Thiết kế auth/authorization rõ ràng cho web và mobile clients.
|
|
||||||
- Tối ưu truy vấn và giảm bề mặt tấn công trong request path.
|
|
||||||
|
|
||||||
## Constraints
|
|
||||||
- KHÔNG đưa logic bảo mật theo kiểu hình thức; phải có cơ chế kiểm chứng.
|
|
||||||
- KHÔNG bỏ qua kiểm tra đầu vào, phân quyền, và xử lý lỗi có chủ đích.
|
|
||||||
- KHÔNG thực hiện thay đổi phá vỡ contract mà không mô tả migration path.
|
|
||||||
|
|
||||||
## Approach
|
|
||||||
1. Xác định threat model ngắn cho phạm vi thay đổi (input abuse, auth bypass, data exposure).
|
|
||||||
2. Thiết kế API/auth flow với validation và permission checks rõ ràng.
|
|
||||||
3. Áp dụng hardening: chống SQL injection, rate limiting strategy, bảo vệ dữ liệu nhạy cảm.
|
|
||||||
4. Tối ưu truy vấn và theo dõi tác động hiệu năng.
|
|
||||||
5. Đề xuất test bảo mật + regression checklist.
|
|
||||||
|
|
||||||
## Output Format
|
|
||||||
- Scope and threat model
|
|
||||||
- Files changed
|
|
||||||
- Security controls added/updated
|
|
||||||
- Query/performance notes
|
|
||||||
- Validation and auth checks
|
|
||||||
- Verification commands/tests
|
|
||||||
@@ -0,0 +1,20 @@
|
|||||||
|
---
|
||||||
|
name: "Cross Platform Delivery Agent"
|
||||||
|
description: "Use when: trien khai 1 tinh nang can dong bo web + mobile + api, can checklist contract, auth va rollout. Keywords: parity, rollout, cross-platform, sync, delivery"
|
||||||
|
tools: [read, search, edit, todo]
|
||||||
|
argument-hint: "Mo ta feature, endpoint lien quan, va impact len web/mobile"
|
||||||
|
user-invocable: true
|
||||||
|
---
|
||||||
|
Ban la Cross Platform Delivery Agent, dam bao moi thay doi feature co tinh lien mach giua 3 repo.
|
||||||
|
|
||||||
|
## Muc tieu
|
||||||
|
- Dong bo business flow giua `reader`, `reader-app`, `reader-api`.
|
||||||
|
- Tranh regression contract va auth behavior giua web/mobile.
|
||||||
|
- Tao checklist rollout theo thu tu API -> Web/Mobile -> QA.
|
||||||
|
|
||||||
|
## Workflow
|
||||||
|
1. Xac dinh API contract canonic va data ownership.
|
||||||
|
2. Liet ke delta can cap nhat cho tung repo.
|
||||||
|
3. Kiem tra auth matrix (web cookie, mobile JWT).
|
||||||
|
4. De xuat test plan E2E toi thieu cho 2 clients.
|
||||||
|
5. Tong hop release note ngan gon.
|
||||||
@@ -1,32 +0,0 @@
|
|||||||
---
|
|
||||||
name: "Mobile App Agent"
|
|
||||||
description: "Use when: phát triển tính năng mobile, xử lý local storage, lifecycle, push notifications và tối ưu hiệu năng thiết bị thật; ưu tiên Flutter, dùng Android/Kotlin module khi cần native. Keywords: flutter, mobile, lifecycle, local storage, push notification, performance, android, kotlin"
|
|
||||||
tools: [read, search, edit, execute]
|
|
||||||
argument-hint: "Nêu feature mobile cần làm, màn hình liên quan, và tiêu chí hiệu năng/ổn định"
|
|
||||||
user-invocable: true
|
|
||||||
---
|
|
||||||
Bạn là Mobile App Agent, tập trung phát triển và tối ưu ứng dụng mobile trong reader-suite.
|
|
||||||
|
|
||||||
## Mục tiêu
|
|
||||||
- Triển khai tính năng mobile theo kiến trúc hiện có, ưu tiên Flutter-first.
|
|
||||||
- Quản lý tốt local storage, lifecycle và thông báo đẩy.
|
|
||||||
- Tối ưu pin, bộ nhớ và độ mượt trên thiết bị thật.
|
|
||||||
|
|
||||||
## Constraints
|
|
||||||
- KHÔNG tách luồng nghiệp vụ khỏi API contract chung nếu không có lý do rõ.
|
|
||||||
- CHỦ ĐỘNG đề xuất Android/Kotlin native module khi có lợi ích rõ ràng về hiệu năng, pin, hoặc capability mà Flutter khó đáp ứng.
|
|
||||||
- KHÔNG bỏ qua kiểm tra hành vi offline/network fluctuation.
|
|
||||||
|
|
||||||
## Approach
|
|
||||||
1. Xác định user flow và dữ liệu cần lưu cục bộ (cache/session/preferences).
|
|
||||||
2. Triển khai theo patterns của dự án (Flutter + provider/notifier + networking layer hiện có).
|
|
||||||
3. Xử lý lifecycle, background/resume và push notifications có kiểm soát.
|
|
||||||
4. Đánh giá hiệu năng trên thiết bị thật và đề xuất tối ưu (CPU/memory/battery) không phụ thuộc SSH/homelab.
|
|
||||||
|
|
||||||
## Output Format
|
|
||||||
- Feature scope
|
|
||||||
- Files changed
|
|
||||||
- State/storage/lifecycle decisions
|
|
||||||
- Performance notes (real-device oriented)
|
|
||||||
- Verification steps
|
|
||||||
- Risks and follow-ups
|
|
||||||
@@ -1,35 +0,0 @@
|
|||||||
---
|
|
||||||
name: "SDET Agent"
|
|
||||||
description: "Use when: viết integration test API (ưu tiên Postman/Newman), viết E2E test web/mobile, chống regression sau khi sửa API, cần test coverage cho luồng đăng nhập/đọc truyện/tủ sách. Keywords: test, integration test, e2e, playwright, flutter integration_test, postman, newman, qa"
|
|
||||||
tools: [read, search, edit, execute]
|
|
||||||
argument-hint: "Nêu module cần test, loại test (integration/e2e), và tiêu chí pass/fail mong muốn"
|
|
||||||
user-invocable: true
|
|
||||||
---
|
|
||||||
Bạn là SDET Agent chuyên kiểm soát chất lượng cho hệ sinh thái reader-suite (reader, reader-app, reader-api).
|
|
||||||
|
|
||||||
## Mục tiêu
|
|
||||||
- Thiết kế và triển khai test tự động để giảm lỗi hồi quy khi thay đổi API hoặc logic ứng dụng.
|
|
||||||
- Ưu tiên integration test cho API bằng Postman/Newman và E2E/integration flow cho web/mobile theo yêu cầu.
|
|
||||||
|
|
||||||
## Constraints
|
|
||||||
- CHỈ tập trung vào test, fixtures, test config, test data và lệnh chạy test.
|
|
||||||
- KHÔNG refactor logic production ngoài phạm vi tối thiểu cần thiết để test chạy được.
|
|
||||||
- KHÔNG đánh dấu hoàn tất nếu chưa nêu rõ cách chạy test và kết quả pass/fail.
|
|
||||||
|
|
||||||
## Approach
|
|
||||||
1. Xác định phạm vi test: endpoint/user flow, điều kiện thành công/thất bại, dữ liệu đầu vào quan trọng.
|
|
||||||
2. Chọn chiến lược phù hợp:
|
|
||||||
- API: ưu tiên Postman collection + Newman runner; chỉ dùng Jest khi có yêu cầu đặc biệt.
|
|
||||||
- Web: ưu tiên Playwright E2E cho luồng người dùng chính.
|
|
||||||
- Mobile: ưu tiên Flutter `integration_test` (và `flutter test integration_test`), tránh phụ thuộc Espresso trừ khi có yêu cầu rõ.
|
|
||||||
3. Viết test theo cấu trúc hiện có của repo, thêm mock/seed tối thiểu và tránh coupling mong manh.
|
|
||||||
4. Chạy test hoặc hướng dẫn lệnh chạy chuẩn trong repo hiện hành.
|
|
||||||
5. Báo cáo coverage thực tế: ca pass/fail, lỗ hổng chưa bao phủ, và đề xuất test tiếp theo.
|
|
||||||
|
|
||||||
## Output Format
|
|
||||||
- Test scope
|
|
||||||
- Files created/updated
|
|
||||||
- Commands run
|
|
||||||
- Results (pass/fail + lỗi chính nếu có)
|
|
||||||
- Residual risks
|
|
||||||
- Next tests to add
|
|
||||||
@@ -1,32 +0,0 @@
|
|||||||
---
|
|
||||||
name: "Sleuth Debugger Agent"
|
|
||||||
description: "Use when: debug lỗi runtime từ log server hoặc log mobile (Flutter/Logcat), phân tích stack trace, truy ra root cause, đề xuất patch sửa nhanh. Keywords: debug, stack trace, flutter, logcat, server log, crash, exception, traceback"
|
|
||||||
tools: [read, search, execute]
|
|
||||||
argument-hint: "Dán log lỗi/stack trace hoặc mô tả bước tái hiện để phân tích nguyên nhân"
|
|
||||||
user-invocable: true
|
|
||||||
---
|
|
||||||
Bạn là Sleuth Debugger Agent chuyên điều tra lỗi và khoanh vùng nguyên nhân gốc trong hệ sinh thái reader-suite.
|
|
||||||
|
|
||||||
## Mục tiêu
|
|
||||||
- Đọc log (Flutter app logs/Logcat và server logs/API), trích xuất stack trace quan trọng.
|
|
||||||
- Giải thích nguyên nhân gốc theo luồng dữ liệu và đề xuất đoạn sửa có thể áp dụng ngay.
|
|
||||||
|
|
||||||
## Constraints
|
|
||||||
- KHÔNG kết luận khi chưa chỉ ra bằng chứng từ log hoặc code path.
|
|
||||||
- KHÔNG đề xuất sửa mơ hồ; phải nêu file/khối code liên quan và lý do.
|
|
||||||
- KHÔNG mở rộng sang refactor lớn nếu không cần để xử lý lỗi hiện tại.
|
|
||||||
|
|
||||||
## Approach
|
|
||||||
1. Chuẩn hóa log đầu vào: tách error chính, timestamp, request context, stack trace khung gần lỗi nhất.
|
|
||||||
2. Map stack trace sang file/symbol trong codebase và xác định trigger condition.
|
|
||||||
3. Nêu root cause theo chuỗi nhân quả (input -> xử lý -> điểm nổ).
|
|
||||||
4. Đưa fix proposal ngắn gọn, ưu tiên thay đổi nhỏ và an toàn.
|
|
||||||
5. Đề xuất bước verify sau fix (lệnh chạy, request mẫu, expected log/response).
|
|
||||||
|
|
||||||
## Output Format
|
|
||||||
- Error signature
|
|
||||||
- Reproduction assumptions
|
|
||||||
- Root cause
|
|
||||||
- Proposed fix snippet
|
|
||||||
- Verification steps
|
|
||||||
- Follow-up guardrails (logging/test cần thêm)
|
|
||||||
@@ -1,33 +0,0 @@
|
|||||||
---
|
|
||||||
name: "System Architect Agent"
|
|
||||||
description: "Use when: thiết kế kiến trúc hệ thống, database schema, API endpoint strategy (REST), chọn tech stack và giữ single source of truth giữa web/mobile/api. Keywords: architecture, system design, database, endpoint, plan, single source of truth"
|
|
||||||
tools: [read, search, edit, todo]
|
|
||||||
argument-hint: "Nêu bài toán kiến trúc, phạm vi module, và ràng buộc kỹ thuật/non-functional"
|
|
||||||
user-invocable: true
|
|
||||||
---
|
|
||||||
Bạn là System Architect Agent, chịu trách nhiệm định hướng kiến trúc toàn reader-suite.
|
|
||||||
|
|
||||||
## Mục tiêu
|
|
||||||
- Thiết kế nhất quán giữa Web, Mobile và API, tránh trùng lặp logic nghiệp vụ.
|
|
||||||
- Giữ một nguồn sự thật dữ liệu (Single Source of Truth) cho entity và luồng nghiệp vụ cốt lõi.
|
|
||||||
- Đưa ra blueprint có thể triển khai dần theo milestone.
|
|
||||||
|
|
||||||
## Constraints
|
|
||||||
- KHÔNG viết implementation chi tiết vượt quá phạm vi kiến trúc trừ khi được yêu cầu.
|
|
||||||
- KHÔNG đề xuất kiến trúc mâu thuẫn conventions đã có trong workspace.
|
|
||||||
- LUÔN nêu trade-off và lý do chọn phương án.
|
|
||||||
|
|
||||||
## Approach
|
|
||||||
1. Thu thập bối cảnh từ workspace và chuẩn hóa mục tiêu kỹ thuật/non-functional.
|
|
||||||
2. Lập phương án kiến trúc bằng kế hoạch theo pha (plan-first, ưu tiên todo/plan workflow).
|
|
||||||
3. Xác định ranh giới domain, nguồn dữ liệu chuẩn, ownership của từng layer.
|
|
||||||
4. Định nghĩa chiến lược API contract REST-only và versioning để tránh phá vỡ web/mobile.
|
|
||||||
5. Đề xuất roadmap triển khai + kiểm soát rủi ro kỹ thuật.
|
|
||||||
|
|
||||||
## Output Format
|
|
||||||
- Problem framing
|
|
||||||
- Architecture decision
|
|
||||||
- Data model and API boundary
|
|
||||||
- Single source of truth mapping
|
|
||||||
- Migration/rollout plan
|
|
||||||
- Risks and mitigations
|
|
||||||
@@ -1,32 +0,0 @@
|
|||||||
---
|
|
||||||
name: "Web Frontend Agent"
|
|
||||||
description: "Use when: xây dựng hoặc tối ưu giao diện web React/Next.js, cải thiện SEO và Core Web Vitals, đồng bộ UI với API contract. Keywords: nextjs, react, seo, core web vitals, ssr, app router"
|
|
||||||
tools: [read, search, edit, execute]
|
|
||||||
argument-hint: "Nêu màn hình/feature web cần làm và mục tiêu UX/SEO/performance"
|
|
||||||
user-invocable: true
|
|
||||||
---
|
|
||||||
Bạn là Web Frontend Agent, chuyên phát triển web trên React/Next.js cho reader-suite.
|
|
||||||
|
|
||||||
## Mục tiêu
|
|
||||||
- Xây dựng UI/UX web nhất quán, hiệu năng tốt và thân thiện SEO.
|
|
||||||
- Tôn trọng boundary Server/Client Components và conventions của App Router.
|
|
||||||
- Đảm bảo web đồng bộ contract với backend, giảm lỗi runtime phía client.
|
|
||||||
|
|
||||||
## Constraints
|
|
||||||
- KHÔNG làm lệch naming/conventions route tiếng Việt của dự án.
|
|
||||||
- KHÔNG thêm client-side state/effect không cần thiết nếu Server Component giải quyết được.
|
|
||||||
- KHÔNG đánh đổi SEO/performance cho giải pháp nhanh tạm bợ.
|
|
||||||
|
|
||||||
## Approach
|
|
||||||
1. Phân tích yêu cầu giao diện + dữ liệu và chọn rendering strategy phù hợp.
|
|
||||||
2. Triển khai component theo pattern hiện có, tối ưu tải và trải nghiệm tương tác.
|
|
||||||
3. Kiểm tra SEO metadata, semantics và Core Web Vitals impact.
|
|
||||||
4. Xác nhận hành vi với API contract hiện tại.
|
|
||||||
|
|
||||||
## Output Format
|
|
||||||
- Feature scope
|
|
||||||
- Files changed
|
|
||||||
- Rendering/data strategy
|
|
||||||
- SEO/Core Web Vitals notes
|
|
||||||
- QA checklist
|
|
||||||
- Follow-up improvements
|
|
||||||
@@ -1,42 +0,0 @@
|
|||||||
# Project Guidelines
|
|
||||||
|
|
||||||
## Code Style
|
|
||||||
- Dùng Dart/Flutter theo lint mặc định trong `analysis_options.yaml` (flutter_lints).
|
|
||||||
- Tổ chức theo feature-first: code mới đặt đúng module trong `lib/features/**`, phần dùng chung đặt ở `lib/core/**` hoặc `lib/shared/**`.
|
|
||||||
- Tránh logic nghiệp vụ trong widget build; chuyển sang provider/notifier.
|
|
||||||
- Giữ naming nhất quán: file snake_case, class PascalCase, provider rõ nghĩa theo tính năng.
|
|
||||||
|
|
||||||
## Architecture
|
|
||||||
- `lib/main.dart`: bootstrap app + ProviderScope.
|
|
||||||
- `lib/app/`: app shell, router và route names.
|
|
||||||
- `lib/core/`: config, network, storage, theme, models dùng toàn app.
|
|
||||||
- `lib/features/`: từng domain (auth, home, search, novel, reader, bookshelf, comments, profile, settings...).
|
|
||||||
- `lib/shared/`: widgets dùng chung.
|
|
||||||
- State management chính: Riverpod; networking: Dio; routing: go_router.
|
|
||||||
|
|
||||||
## Build and Test
|
|
||||||
- Cài dependencies: `flutter pub get`
|
|
||||||
- Chạy app: `flutter run`
|
|
||||||
- Khuyến nghị local: `bash scripts/flutter_run_with_env.sh`
|
|
||||||
- Phân tích lint/static: `flutter analyze`
|
|
||||||
- Chạy test hiện có: `flutter test`
|
|
||||||
|
|
||||||
## Conventions
|
|
||||||
- Cấu hình API base URL lấy từ AppConfig/env; không hardcode URL trực tiếp trong feature code.
|
|
||||||
- Token/session xử lý qua tầng storage/network ở `lib/core`, tránh duplicate auth flow trong từng feature.
|
|
||||||
- Khi thêm màn hình mới, cập nhật router tập trung ở `lib/app/router/app_router.dart`.
|
|
||||||
- Ưu tiên tái sử dụng models trong `lib/core/models` thay vì tạo kiểu dữ liệu rời rạc.
|
|
||||||
|
|
||||||
## Pitfalls
|
|
||||||
- Android emulator phải dùng `10.0.2.2` để gọi localhost backend; iOS simulator/web thường dùng `localhost`.
|
|
||||||
- Google Sign-In Android dễ lỗi `ApiException: 10` nếu cấu hình SHA/OAuth client không khớp.
|
|
||||||
- Thiếu `.env.mobile` hoặc thiếu `GOOGLE_SERVER_CLIENT_ID` có thể làm luồng đăng nhập thất bại.
|
|
||||||
- Thiết bị thật cần LAN IP đúng mạng nội bộ; không dùng VPN IP nếu điện thoại không cùng tunnel.
|
|
||||||
|
|
||||||
## Key References
|
|
||||||
- Tổng quan setup + Google Sign-In: `README.md`
|
|
||||||
- Lint rules: `analysis_options.yaml`
|
|
||||||
- App config/env: `lib/core/config/app_config.dart`
|
|
||||||
- API client + interceptor: `lib/core/network/api_client.dart`
|
|
||||||
- Router trung tâm: `lib/app/router/app_router.dart`
|
|
||||||
- Script chạy với env: `scripts/flutter_run_with_env.sh`
|
|
||||||
@@ -1,22 +0,0 @@
|
|||||||
---
|
|
||||||
name: "Debug Triage Checklist"
|
|
||||||
description: "Use when: cần phân tích nhanh lỗi runtime từ logcat/server logs, xác định root cause và đề xuất patch an toàn"
|
|
||||||
argument-hint: "Dán log, stack trace, bước tái hiện, và môi trường bị lỗi"
|
|
||||||
agent: "Sleuth Debugger Agent"
|
|
||||||
---
|
|
||||||
Thực hiện debug triage theo checklist chuẩn cho lỗi runtime được cung cấp.
|
|
||||||
|
|
||||||
Checklist bắt buộc:
|
|
||||||
- Trích xuất error signature chính và stack trace quan trọng nhất.
|
|
||||||
- Xác định giả định tái hiện lỗi và điều kiện kích hoạt.
|
|
||||||
- Khoanh vùng root cause bằng bằng chứng từ log + code path.
|
|
||||||
- Đề xuất patch nhỏ nhất có thể áp dụng ngay.
|
|
||||||
- Đưa kế hoạch verify sau fix với expected behavior rõ ràng.
|
|
||||||
|
|
||||||
Kết quả bắt buộc:
|
|
||||||
- Error signature.
|
|
||||||
- Root cause theo chuỗi nhân quả.
|
|
||||||
- File/symbol bị ảnh hưởng.
|
|
||||||
- Patch proposal cụ thể.
|
|
||||||
- Verification checklist.
|
|
||||||
- Guardrails phòng tái phát (test/logging/alerts).
|
|
||||||
@@ -1,19 +0,0 @@
|
|||||||
---
|
|
||||||
name: "Flutter Integration Test Flow"
|
|
||||||
description: "Use when: cần viết hoặc mở rộng Flutter integration_test cho user flow quan trọng và chống regression mobile"
|
|
||||||
argument-hint: "Nêu flow cần test, dữ liệu đầu vào, và tiêu chí thành công"
|
|
||||||
agent: "SDET Agent"
|
|
||||||
---
|
|
||||||
Tạo hoặc cập nhật Flutter integration_test cho user flow được chỉ định.
|
|
||||||
|
|
||||||
Yêu cầu thực hiện:
|
|
||||||
- Ưu tiên `integration_test` theo cấu trúc hiện có của dự án.
|
|
||||||
- Thiết kế test theo hành vi người dùng thật (đăng nhập, điều hướng, thao tác chính, trạng thái mong đợi).
|
|
||||||
- Bao gồm ít nhất một nhánh lỗi hoặc edge case có giá trị.
|
|
||||||
- Tránh test phụ thuộc dữ liệu không ổn định; thêm setup/teardown cần thiết.
|
|
||||||
|
|
||||||
Kết quả bắt buộc:
|
|
||||||
- Danh sách file test đã tạo/cập nhật.
|
|
||||||
- Lệnh chạy cụ thể (`flutter test integration_test` hoặc lệnh tương đương của repo).
|
|
||||||
- Kết quả pass/fail và nguyên nhân nếu fail.
|
|
||||||
- Đề xuất mở rộng coverage cho vòng kế tiếp.
|
|
||||||
@@ -1,27 +0,0 @@
|
|||||||
---
|
|
||||||
name: "Multi-Agent Workflow"
|
|
||||||
description: "Use when: vận hành chu kỳ Planner -> Coder -> Tester -> Debugger cho tính năng mới hoặc bugfix"
|
|
||||||
argument-hint: "Nêu tính năng/bug cần xử lý, phạm vi web-mobile-api, và tiêu chí hoàn tất"
|
|
||||||
agent: "System Architect Agent"
|
|
||||||
---
|
|
||||||
Điều phối quy trình Multi-Agent theo chu kỳ sau, bám sát phạm vi người dùng cung cấp.
|
|
||||||
|
|
||||||
Quy trình bắt buộc:
|
|
||||||
1. Planner: dùng tư duy @workspace + plan-first để thiết kế feature/bugfix scope và các mốc triển khai.
|
|
||||||
2. Coder: triển khai đồng bộ phần Backend API và phần Web/Mobile liên quan theo contract hiện hành.
|
|
||||||
3. Tester: gọi SDET Agent để tạo/chạy test tự động.
|
|
||||||
- API: ưu tiên Postman/Newman.
|
|
||||||
- Mobile: ưu tiên Flutter integration_test.
|
|
||||||
4. Debugger: nếu crash/lỗi runtime, gọi Sleuth Debugger Agent phân tích log và đề xuất patch nhỏ nhất.
|
|
||||||
|
|
||||||
Ràng buộc:
|
|
||||||
- API phải REST-only.
|
|
||||||
- Contract chuẩn dùng duy nhất reader-api/docs/openapi.yaml.
|
|
||||||
- Báo cáo rõ tác động tới web và mobile sau mỗi thay đổi contract.
|
|
||||||
|
|
||||||
Kết quả bắt buộc:
|
|
||||||
- Kế hoạch triển khai theo bước.
|
|
||||||
- Danh sách file thay đổi theo từng pha.
|
|
||||||
- Kết quả test pass/fail.
|
|
||||||
- Root cause + patch nếu có lỗi.
|
|
||||||
- Danh sách việc còn lại trước khi merge.
|
|
||||||
@@ -1,19 +0,0 @@
|
|||||||
---
|
|
||||||
name: "Regression API Newman"
|
|
||||||
description: "Use when: cần tạo hoặc cập nhật regression suite cho API bằng Postman/Newman sau khi sửa endpoint hoặc auth flow"
|
|
||||||
argument-hint: "Nêu endpoint/flow cần cover, env chạy test, và tiêu chí pass/fail"
|
|
||||||
agent: "SDET Agent"
|
|
||||||
---
|
|
||||||
Thiết kế và triển khai regression API test theo chuẩn Postman/Newman cho phạm vi người dùng cung cấp.
|
|
||||||
|
|
||||||
Yêu cầu thực hiện:
|
|
||||||
- Ưu tiên Postman collection + environment + Newman command.
|
|
||||||
- Bám theo cấu trúc endpoint và auth flow hiện có của workspace đang mở.
|
|
||||||
- Bao phủ cả happy path, validation errors, auth errors và regression cases đã biết.
|
|
||||||
- Nếu thiếu dữ liệu test, đề xuất seed/mock tối thiểu và cách chạy lại ổn định.
|
|
||||||
|
|
||||||
Kết quả bắt buộc:
|
|
||||||
- Danh sách file đã tạo/cập nhật cho Postman/Newman.
|
|
||||||
- Lệnh chạy Newman cụ thể.
|
|
||||||
- Bảng pass/fail theo từng test case.
|
|
||||||
- Rủi ro còn lại và test nên thêm tiếp.
|
|
||||||
@@ -0,0 +1,45 @@
|
|||||||
|
## Summary
|
||||||
|
- What problem does this PR solve?
|
||||||
|
- Why now?
|
||||||
|
|
||||||
|
## Scope
|
||||||
|
- [ ] Web (`reader`)
|
||||||
|
- [ ] API (`reader-api`)
|
||||||
|
- [ ] Mobile (`reader-app`)
|
||||||
|
|
||||||
|
## API Contract Impact
|
||||||
|
- [ ] No API changes
|
||||||
|
- [ ] Backward-compatible API changes
|
||||||
|
- [ ] Breaking API changes (must include migration plan)
|
||||||
|
|
||||||
|
Changed endpoints:
|
||||||
|
- `...`
|
||||||
|
|
||||||
|
## Auth Impact
|
||||||
|
- [ ] None
|
||||||
|
- [ ] Web session (NextAuth cookie)
|
||||||
|
- [ ] Mobile JWT
|
||||||
|
|
||||||
|
## Cross-Platform Parity Checklist
|
||||||
|
- [ ] Web behavior aligned with API response
|
||||||
|
- [ ] Mobile behavior aligned with API response
|
||||||
|
- [ ] Error states consistent (401/403/4xx/5xx)
|
||||||
|
- [ ] Pagination behavior consistent (`page`, `limit`)
|
||||||
|
|
||||||
|
## Test Evidence
|
||||||
|
- [ ] Unit tests
|
||||||
|
- [ ] Integration/API tests
|
||||||
|
- [ ] Manual smoke test Web
|
||||||
|
- [ ] Manual smoke test Mobile
|
||||||
|
|
||||||
|
Commands / notes:
|
||||||
|
```bash
|
||||||
|
# paste test/build commands executed
|
||||||
|
```
|
||||||
|
|
||||||
|
## Rollout & Rollback
|
||||||
|
- Rollout order: API -> Web/Mobile -> QA
|
||||||
|
- Rollback strategy:
|
||||||
|
- API:
|
||||||
|
- Web:
|
||||||
|
- Mobile:
|
||||||
@@ -0,0 +1,38 @@
|
|||||||
|
# Reader Suite Architecture (Android/Flutter)
|
||||||
|
|
||||||
|
Tai lieu nay mo ta vai tro `reader-app` trong he sinh thai doc truyen gom web + mobile + backend.
|
||||||
|
|
||||||
|
## Vai tro trong he thong
|
||||||
|
|
||||||
|
- `reader-app` la mobile client cho end-user feature.
|
||||||
|
- KHONG chua logic nghiep vu canonic; backend (`reader-api`) la noi quyet dinh rule.
|
||||||
|
- Muc tieu: feature parity voi web user-facing, tru workflow MOD/ADMIN.
|
||||||
|
|
||||||
|
## Kien truc app
|
||||||
|
|
||||||
|
- `lib/core`: config, networking, storage, app-level services.
|
||||||
|
- `lib/features`: module theo domain (auth, browse, novel-detail, reader, bookshelf...).
|
||||||
|
- `lib/shared`: widget dung chung.
|
||||||
|
|
||||||
|
## Nguyen tac dong bo voi web + backend
|
||||||
|
|
||||||
|
- Dung chung endpoint contract voi web, khong tao API rieng cho mobile neu khong can thiet.
|
||||||
|
- Dung chung semantic cho state:
|
||||||
|
- bookmark
|
||||||
|
- reading-progress
|
||||||
|
- recommendation
|
||||||
|
- user-settings
|
||||||
|
- Auth mobile qua JWT; backend phai map cung identity voi web.
|
||||||
|
|
||||||
|
## Environment va ket noi
|
||||||
|
|
||||||
|
- Khong co `--dart-define`: Android native mac dinh toi `https://reader-api.fevirtus.dev`; cac platform khac `http://localhost:8000` (`lib/core/config/app_config.dart`).
|
||||||
|
- Dev local Android emulator: dat `BASE_URL=http://10.0.2.2:8000` trong `.env.mobile` va chay qua `scripts/flutter_run_with_env.sh`.
|
||||||
|
- `BASE_URL`, `GOOGLE_SERVER_CLIENT_ID`, `GOOGLE_CLIENT_ID` duoc truyen qua `--dart-define`.
|
||||||
|
|
||||||
|
## Definition of Done (Mobile)
|
||||||
|
|
||||||
|
- API calls tuan thu contract `reader-api` (status code + error format).
|
||||||
|
- UX/state nhat quan voi web cho luong user-facing.
|
||||||
|
- Da test tren it nhat Android emulator + 1 moi truong khac (iOS simulator hoac device).
|
||||||
|
- Cac huong dan env/auth trong README van dung sau thay doi.
|
||||||
+65
@@ -0,0 +1,65 @@
|
|||||||
|
# API Contract Standard (Reader Suite)
|
||||||
|
|
||||||
|
Tai lieu contract chung cho `reader`, `reader-app`, `reader-api`.
|
||||||
|
|
||||||
|
## Base and Versioning
|
||||||
|
|
||||||
|
- Base path: `/api/*`
|
||||||
|
- Current mode: unversioned with backward-compatible evolution.
|
||||||
|
- Breaking changes bat buoc qua ke hoach migration va release note.
|
||||||
|
|
||||||
|
## Response Convention
|
||||||
|
|
||||||
|
- Success:
|
||||||
|
- `200/201`: tra JSON payload domain data.
|
||||||
|
- `204`: khong co body.
|
||||||
|
- Error (standardized):
|
||||||
|
|
||||||
|
```json
|
||||||
|
{
|
||||||
|
"code": "string",
|
||||||
|
"message": "human readable",
|
||||||
|
"details": {}
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
## HTTP Status Usage
|
||||||
|
|
||||||
|
- `400`: input/validation khong hop le.
|
||||||
|
- `401`: chua dang nhap hoặc token/session invalid.
|
||||||
|
- `403`: da dang nhap nhung khong du quyen.
|
||||||
|
- `404`: resource khong ton tai.
|
||||||
|
- `409`: xung dot du lieu.
|
||||||
|
- `422`: payload format dung JSON nhung khong dat rule nghiep vu.
|
||||||
|
- `500`: loi he thong.
|
||||||
|
- `410`: (du tru) tai nguyen da go bo hoac khong con ho tro.
|
||||||
|
|
||||||
|
## Pagination Convention
|
||||||
|
|
||||||
|
- Query params: `page`, `limit`.
|
||||||
|
- Response envelope:
|
||||||
|
|
||||||
|
```json
|
||||||
|
{
|
||||||
|
"items": [],
|
||||||
|
"pagination": {
|
||||||
|
"page": 1,
|
||||||
|
"limit": 20,
|
||||||
|
"total": 0,
|
||||||
|
"totalPages": 0
|
||||||
|
}
|
||||||
|
}
|
||||||
|
```
|
||||||
|
|
||||||
|
## Auth Matrix
|
||||||
|
|
||||||
|
- Web: NextAuth session cookie.
|
||||||
|
- Mobile: Bearer JWT.
|
||||||
|
- Backend phai map ca 2 vao cung user identity + role.
|
||||||
|
|
||||||
|
## Compatibility Rules
|
||||||
|
|
||||||
|
- Khong xoa hoac doi y nghia field dang duoc client su dung.
|
||||||
|
- Field moi phai optional theo default trong giai doan rollout.
|
||||||
|
- Doi ten field => tao migration layer hoac add field moi, deprecate sau.
|
||||||
|
- Endpoint moi can update README + mapping matrix.
|
||||||
@@ -0,0 +1,37 @@
|
|||||||
|
# Cross-Repo Endpoint Mapping Matrix
|
||||||
|
|
||||||
|
Muc tieu: map 1-1 giua API backend, Web va Mobile cho user-facing flows.
|
||||||
|
|
||||||
|
Legend:
|
||||||
|
- `Y`: da tich hop
|
||||||
|
- `P`: partial / can verify them
|
||||||
|
- `N`: chua tich hop
|
||||||
|
|
||||||
|
| Domain | Endpoint | API | Web | Mobile | Notes |
|
||||||
|
|---|---|---|---|---|---|
|
||||||
|
| Health | `GET /api/health` | Y | P | P | Dung cho monitor, khong phai main UI flow |
|
||||||
|
| Auth | `POST /api/auth/mobile-login` | Y | Y | Y | Web dung route adapter login, mobile dung JWT |
|
||||||
|
| User | `GET /api/user/profile` | Y | P | Y | Web dang goi qua user route proxy |
|
||||||
|
| User | `GET/POST /api/user/bookmarks` | Y | Y | Y | Parity can test theo tabs bookshelf |
|
||||||
|
| User | `DELETE /api/user/bookmarks/{novelId}` | Y | P | Y | Web co remove flow can verify UX parity |
|
||||||
|
| User | `POST /api/user/reading-progress` | Y | P | Y | Can doi chieu dong bo chapter progress |
|
||||||
|
| User | `GET/POST /api/user/settings` | Y | Y | N | Mobile chua thay call settings ro rang |
|
||||||
|
| User | `GET/POST/DELETE /api/user/recommendations` | Y | Y | N | Mobile chua thay provider recommendation |
|
||||||
|
| Catalog | `GET /api/genres` | Y | Y | Y | |
|
||||||
|
| Catalog | `GET /api/novels/browse` | Y | Y | Y | |
|
||||||
|
| Catalog | `GET /api/novels/{idOrSlug}` | Y | Y | Y | |
|
||||||
|
| Novel | `GET /api/truyen/{id}/chapters` | Y | Y | Y | |
|
||||||
|
| Novel | `GET /api/truyen/{id}/chapters/by-number/{n}` | Y | Y | N | Mobile doc chapter theo chapterId endpoint |
|
||||||
|
| Chapter | `GET /api/chapters/{chapterId}` | Y | N | Y | Web doc chapter qua truyen/by-number |
|
||||||
|
| Comment | `GET/POST /api/truyen/{id}/comments` | Y | Y | Y | |
|
||||||
|
| Rating | `POST /api/truyen/{id}/rate` | Y | Y | N | Mobile chua thay rating flow |
|
||||||
|
| Search | `GET /api/truyen/suggest` | Y | Y | N | Mobile search suggest can bo sung |
|
||||||
|
| Import | `POST /api/import/uploads/preview` | Y | Y | N | Upload EPUB multipart (preview) |
|
||||||
|
| Import | `POST /api/mod/epub`, `POST /api/mod/epub/ai-suggest` | Y | Y | N | Luong `/mod/import` |
|
||||||
|
| Import | `GET/POST/PUT/DELETE /api/mod/the-loai` | Y | Y | N | MOD quan ly the loai trong wizard |
|
||||||
|
|
||||||
|
## Priority gaps de dong bo tiep
|
||||||
|
|
||||||
|
1. Mobile: `user/settings`, `recommendations`, `rate`, `suggest`.
|
||||||
|
2. Web/Mobile chapter-read strategy can unify (`chapters/{id}` vs `by-number`).
|
||||||
|
3. Chuan hoa error contract implementation theo `CONTRACT.md`.
|
||||||
+34
@@ -0,0 +1,34 @@
|
|||||||
|
# Features - Reader Android App
|
||||||
|
|
||||||
|
Trang thai tinh nang mobile `reader-app` theo parity voi web.
|
||||||
|
|
||||||
|
## Guest/User-facing
|
||||||
|
|
||||||
|
| Feature | Status | Notes |
|
||||||
|
|---|---|---|
|
||||||
|
| Google login | done | Qua `/api/auth/mobile-login` |
|
||||||
|
| Home/browse boards | done | `/api/novels/browse` |
|
||||||
|
| Genre list | done | `/api/genres` |
|
||||||
|
| Novel detail + chapter list | done | `/api/novels/{idOrSlug}`, `/api/truyen/{id}/chapters` |
|
||||||
|
| Reader chapter detail | done | `/api/chapters/{chapterId}` |
|
||||||
|
| Bookmark | done | `/api/user/bookmarks` |
|
||||||
|
| Reading progress sync | done | `/api/user/reading-progress` |
|
||||||
|
| Comments | done | `/api/truyen/{id}/comments` |
|
||||||
|
|
||||||
|
## Parity Gaps
|
||||||
|
|
||||||
|
| Feature | Status | Notes |
|
||||||
|
|---|---|---|
|
||||||
|
| User settings sync | planned | `/api/user/settings` |
|
||||||
|
| User recommendations | planned | `/api/user/recommendations` |
|
||||||
|
| Rating | planned | `/api/truyen/{id}/rate` |
|
||||||
|
| Search suggest | planned | `/api/truyen/suggest` |
|
||||||
|
|
||||||
|
## Dependencies
|
||||||
|
|
||||||
|
- Contract: `reader-app/CONTRACT.md`
|
||||||
|
- Mapping: `reader-app/CROSS_REPO_ENDPOINT_MATRIX.md`
|
||||||
|
|
||||||
|
## Note
|
||||||
|
|
||||||
|
- EPUB import flow is currently MOD-only on web (`reader /mod/import`), not in mobile scope.
|
||||||
@@ -0,0 +1,52 @@
|
|||||||
|
# Flows - Reader Android App
|
||||||
|
|
||||||
|
Muc tieu: mobile follow cung business behavior voi web.
|
||||||
|
|
||||||
|
## Flow 1: Mobile Login and Token
|
||||||
|
|
||||||
|
- Trigger: user dang nhap Google tren app.
|
||||||
|
- Steps:
|
||||||
|
1. Lay Google token.
|
||||||
|
2. Goi `/api/auth/mobile-login` doi lay app JWT.
|
||||||
|
3. Luu JWT, goi `/api/user/profile` hydrate session.
|
||||||
|
- Failure:
|
||||||
|
- token invalid -> `401`
|
||||||
|
- profile unavailable -> retry + user notice
|
||||||
|
|
||||||
|
## Flow 2: Discover and Read
|
||||||
|
|
||||||
|
- Discover:
|
||||||
|
- `/api/genres`
|
||||||
|
- `/api/novels/browse`
|
||||||
|
- `/api/novels/{idOrSlug}`
|
||||||
|
- Read:
|
||||||
|
- `/api/truyen/{id}/chapters`
|
||||||
|
- `/api/chapters/{chapterId}`
|
||||||
|
- Expected: state chapter/current progress dong bo server.
|
||||||
|
|
||||||
|
## Flow 3: Bookmark and Progress
|
||||||
|
|
||||||
|
- Bookmark add/remove/sync:
|
||||||
|
- `GET/POST/DELETE /api/user/bookmarks`
|
||||||
|
- Progress sync:
|
||||||
|
- `POST /api/user/reading-progress`
|
||||||
|
- Rule: optimistic UI + rollback neu API fail.
|
||||||
|
|
||||||
|
## Flow 4: Comment Interaction
|
||||||
|
|
||||||
|
- Load comments:
|
||||||
|
- `GET /api/truyen/{id}/comments`
|
||||||
|
- Post comment:
|
||||||
|
- `POST /api/truyen/{id}/comments`
|
||||||
|
- Rule: require login, follow error contract chung.
|
||||||
|
|
||||||
|
## Planned Flows (Parity)
|
||||||
|
|
||||||
|
- Settings sync flow (`/api/user/settings`)
|
||||||
|
- Recommendation flow (`/api/user/recommendations`)
|
||||||
|
- Rating flow (`/api/truyen/{id}/rate`)
|
||||||
|
- Search suggest flow (`/api/truyen/suggest`)
|
||||||
|
|
||||||
|
## Out of Scope (Current)
|
||||||
|
|
||||||
|
- MOD EPUB import chi tren web (`/mod/import`: `/api/mod/epub`, `POST /api/import/uploads/preview`, …). Mobile khong co wizard import.
|
||||||
@@ -7,16 +7,11 @@ Flutter mobile app for reading novels, synced with the existing web platform.
|
|||||||
- Full end-user feature parity with the current web app.
|
- Full end-user feature parity with the current web app.
|
||||||
- Excludes all moderator/admin workflows.
|
- Excludes all moderator/admin workflows.
|
||||||
|
|
||||||
## Planned Feature Set
|
## Implemented vs planned
|
||||||
|
|
||||||
- Google login and authenticated user session.
|
Đã có trong code (theo `lib/features`): đăng nhập Google, home/browse, genres, tìm kiếm, chi tiết truyện + danh sách chương, reader (kèm TTS), bookshelf, bookmark/progress, bình luận, splash/settings.
|
||||||
- Home feed, hot boards, and recommendations.
|
|
||||||
- Search with suggestions, genre, status, and sort filters.
|
Còn thiếu hoặc mới dạng placeholder so với web: gợi ý tìm kiếm (`/api/truyen/suggest`), đánh giá truyện (`/api/truyen/{id}/rate`), đồng bộ settings và đề cử người dùng (`/api/user/settings`, `/api/user/recommendations`). Chi tiết parity xem `FEATURES.md` và `CROSS_REPO_ENDPOINT_MATRIX.md`.
|
||||||
- Novel detail with chapter list and series metadata.
|
|
||||||
- Reader with TOC, reading preferences, and progress sync.
|
|
||||||
- Bookshelf tabs: Dang doc, Danh dau, Da doc, De cu.
|
|
||||||
- Comments, ratings, and user recommendations.
|
|
||||||
- Native TTS and offline reading support.
|
|
||||||
|
|
||||||
## Architecture
|
## Architecture
|
||||||
|
|
||||||
@@ -51,12 +46,14 @@ This script reads `.env.mobile` and automatically passes:
|
|||||||
- `GOOGLE_SERVER_CLIENT_ID`
|
- `GOOGLE_SERVER_CLIENT_ID`
|
||||||
- optional `GOOGLE_CLIENT_ID`
|
- optional `GOOGLE_CLIENT_ID`
|
||||||
|
|
||||||
Default `BASE_URL` behavior:
|
Default `BASE_URL` khi **không** truyền `--dart-define` (xem `lib/core/config/app_config.dart`):
|
||||||
|
|
||||||
- Android emulator: `http://10.0.2.2:8000`
|
- Android (native, không phải web build): `https://reader-api.fevirtus.dev`
|
||||||
- Others (iOS simulator, desktop, web): `http://localhost:8000`
|
- Các nền tảng khác (iOS, desktop, web build, v.v.): `http://localhost:8000`
|
||||||
|
|
||||||
If needed, you can still override explicitly:
|
Để dev local trên Android emulator, luôn set rõ qua `--dart-define` hoặc file `.env.mobile` + `scripts/flutter_run_with_env.sh`, ví dụ `http://10.0.2.2:8000`.
|
||||||
|
|
||||||
|
Override trực tiếp:
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
flutter run --dart-define=BASE_URL=http://localhost:8000
|
flutter run --dart-define=BASE_URL=http://localhost:8000
|
||||||
|
|||||||
@@ -0,0 +1,23 @@
|
|||||||
|
# Shared Development Skills (Reader Suite)
|
||||||
|
|
||||||
|
Tai lieu mo ta bo skill de phat trien thong nhat giua web/app/api.
|
||||||
|
|
||||||
|
## skill: feature-parity-check
|
||||||
|
- Muc dich: so sanh 1 feature giua web va mobile theo cung API contract.
|
||||||
|
- Input: ten feature, endpoint list, expected UX behavior.
|
||||||
|
- Output: parity report (done/missing/risk) cho 3 repo.
|
||||||
|
|
||||||
|
## skill: api-contract-guard
|
||||||
|
- Muc dich: review thay doi API de tranh breaking web/mobile.
|
||||||
|
- Input: diff endpoint, request/response schema, auth requirement.
|
||||||
|
- Output: compatibility checklist + migration huong dan.
|
||||||
|
|
||||||
|
## skill: auth-flow-verifier
|
||||||
|
- Muc dich: verify luong dang nhap web cookie va mobile JWT map cung identity.
|
||||||
|
- Input: login flow, token/session behavior, protected endpoints.
|
||||||
|
- Output: matrix pass/fail + fix recommendation.
|
||||||
|
|
||||||
|
## skill: release-readiness
|
||||||
|
- Muc dich: chuan bi release dong bo 3 repo.
|
||||||
|
- Input: danh sach thay doi theo repo.
|
||||||
|
- Output: rollout order, smoke-test list, rollback note.
|
||||||
+18
-6
@@ -24,15 +24,27 @@ class _ReaderAppState extends ConsumerState<ReaderApp> {
|
|||||||
final _scaffoldMessengerKey = GlobalKey<ScaffoldMessengerState>();
|
final _scaffoldMessengerKey = GlobalKey<ScaffoldMessengerState>();
|
||||||
ProviderSubscription<int>? _sessionExpirySub;
|
ProviderSubscription<int>? _sessionExpirySub;
|
||||||
late final GoRouter _router;
|
late final GoRouter _router;
|
||||||
|
String? _previousPath;
|
||||||
|
|
||||||
void _persistRouteForRestore() {
|
void _persistRouteForRestore() {
|
||||||
if (!mounted) return;
|
if (!mounted) return;
|
||||||
unawaited(() async {
|
final uri = _router.state.uri;
|
||||||
final uri = _router.state.uri;
|
final fullPath = uri.hasQuery ? '${uri.path}?${uri.query}' : uri.path;
|
||||||
final fullPath = uri.hasQuery ? '${uri.path}?${uri.query}' : uri.path;
|
if (fullPath == RouteNames.splash) return;
|
||||||
if (fullPath == RouteNames.splash) return;
|
|
||||||
await ref.read(localStoreProvider).saveLastRoutePath(fullPath);
|
// When navigating into reader from a novel page, save "novelPath|readerPath"
|
||||||
}());
|
// so the splash screen can reconstruct the full back stack on restore.
|
||||||
|
final String pathToSave;
|
||||||
|
if (fullPath.startsWith('/reader/') &&
|
||||||
|
_previousPath != null &&
|
||||||
|
_previousPath!.startsWith('/novel/')) {
|
||||||
|
pathToSave = '$_previousPath|$fullPath';
|
||||||
|
} else {
|
||||||
|
pathToSave = fullPath;
|
||||||
|
}
|
||||||
|
_previousPath = fullPath;
|
||||||
|
|
||||||
|
unawaited(ref.read(localStoreProvider).saveLastRoutePath(pathToSave));
|
||||||
}
|
}
|
||||||
|
|
||||||
@override
|
@override
|
||||||
|
|||||||
@@ -220,7 +220,7 @@ final chapterListProvider =
|
|||||||
try {
|
try {
|
||||||
return await fetchAllChapters(novelId);
|
return await fetchAllChapters(novelId);
|
||||||
} catch (_) {
|
} catch (_) {
|
||||||
// Backend stores chapters by novel id in MongoDB; if route opened by slug,
|
// If route opened by slug/id mismatch, resolve canonical novel id and retry once.
|
||||||
// first request can return empty list. Resolve canonical id and retry once.
|
// first request can return empty list. Resolve canonical id and retry once.
|
||||||
try {
|
try {
|
||||||
final novelRes = await client.dio.get('/api/novels/$novelId');
|
final novelRes = await client.dio.get('/api/novels/$novelId');
|
||||||
|
|||||||
@@ -19,17 +19,19 @@ class _SplashScreenState extends ConsumerState<SplashScreen> {
|
|||||||
|
|
||||||
bool _isRestorableRoute(String path) {
|
bool _isRestorableRoute(String path) {
|
||||||
if (path.isEmpty || path == RouteNames.splash) return false;
|
if (path.isEmpty || path == RouteNames.splash) return false;
|
||||||
return path == RouteNames.home ||
|
// Composite "parentPath|deepPath" — validate the deep path portion
|
||||||
path == RouteNames.login ||
|
final checkPath = path.contains('|') ? path.substring(path.indexOf('|') + 1) : path;
|
||||||
path == RouteNames.search ||
|
return checkPath == RouteNames.home ||
|
||||||
path.startsWith('${RouteNames.search}?') ||
|
checkPath == RouteNames.login ||
|
||||||
path == RouteNames.genres ||
|
checkPath == RouteNames.search ||
|
||||||
path == RouteNames.bookshelf ||
|
checkPath.startsWith('${RouteNames.search}?') ||
|
||||||
path == RouteNames.profile ||
|
checkPath == RouteNames.genres ||
|
||||||
path == RouteNames.settings ||
|
checkPath == RouteNames.bookshelf ||
|
||||||
path.startsWith('/novel/') ||
|
checkPath == RouteNames.profile ||
|
||||||
path.startsWith('/reader/') ||
|
checkPath == RouteNames.settings ||
|
||||||
path.startsWith('/comments/');
|
checkPath.startsWith('/novel/') ||
|
||||||
|
checkPath.startsWith('/reader/') ||
|
||||||
|
checkPath.startsWith('/comments/');
|
||||||
}
|
}
|
||||||
|
|
||||||
@override
|
@override
|
||||||
@@ -40,7 +42,34 @@ class _SplashScreenState extends ConsumerState<SplashScreen> {
|
|||||||
final lastPath = await ref.read(localStoreProvider).loadLastRoutePath();
|
final lastPath = await ref.read(localStoreProvider).loadLastRoutePath();
|
||||||
if (!mounted) return;
|
if (!mounted) return;
|
||||||
if (lastPath != null && _isRestorableRoute(lastPath)) {
|
if (lastPath != null && _isRestorableRoute(lastPath)) {
|
||||||
context.go(lastPath);
|
if (lastPath.contains('|')) {
|
||||||
|
// Composite "parentPath|deepPath" e.g. "/novel/123|/reader/abc"
|
||||||
|
// Restore full stack: Home → Novel Detail → Reader
|
||||||
|
final sep = lastPath.indexOf('|');
|
||||||
|
final parentPath = lastPath.substring(0, sep);
|
||||||
|
final deepPath = lastPath.substring(sep + 1);
|
||||||
|
context.go(RouteNames.home);
|
||||||
|
WidgetsBinding.instance.addPostFrameCallback((_) {
|
||||||
|
if (!mounted) return;
|
||||||
|
context.push(parentPath);
|
||||||
|
WidgetsBinding.instance.addPostFrameCallback((_) {
|
||||||
|
if (mounted) context.push(deepPath);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
} else {
|
||||||
|
// Single deep route (novel, comments) outside ShellRoute: push on Home
|
||||||
|
final isDeepRoute = lastPath.startsWith('/reader/') ||
|
||||||
|
lastPath.startsWith('/novel/') ||
|
||||||
|
lastPath.startsWith('/comments/');
|
||||||
|
if (isDeepRoute) {
|
||||||
|
context.go(RouteNames.home);
|
||||||
|
WidgetsBinding.instance.addPostFrameCallback((_) {
|
||||||
|
if (mounted) context.push(lastPath);
|
||||||
|
});
|
||||||
|
} else {
|
||||||
|
context.go(lastPath);
|
||||||
|
}
|
||||||
|
}
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
context.go(RouteNames.home);
|
context.go(RouteNames.home);
|
||||||
|
|||||||
+1
-1
@@ -16,7 +16,7 @@ publish_to: 'none' # Remove this line if you wish to publish to pub.dev
|
|||||||
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
|
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
|
||||||
# In Windows, build-name is used as the major, minor, and patch parts
|
# In Windows, build-name is used as the major, minor, and patch parts
|
||||||
# of the product and file versions while build-number is used as the build suffix.
|
# of the product and file versions while build-number is used as the build suffix.
|
||||||
version: 1.0.3+5
|
version: 1.0.3+6
|
||||||
|
|
||||||
environment:
|
environment:
|
||||||
sdk: ^3.11.3
|
sdk: ^3.11.3
|
||||||
|
|||||||
Reference in New Issue
Block a user