Skip to content

修改README跟吉祥物#16

Merged
Retsomm merged 1 commit into
mainfrom
dev
May 21, 2026
Merged

修改README跟吉祥物#16
Retsomm merged 1 commit into
mainfrom
dev

Conversation

@Retsomm
Copy link
Copy Markdown
Owner

@Retsomm Retsomm commented May 21, 2026

Summary by CodeRabbit

  • Documentation

    • Updated project documentation with comprehensive setup and development guidance.
  • New Features

    • Added greeting mascot overlay that displays during audio teaching sessions.
    • Improved live caption display behavior based on active call status.
  • Style

    • Updated mascot and lesson artwork assets throughout the app.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 21, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d1d1afa8-632a-480b-84e4-9246cdf5d391

📥 Commits

Reviewing files that changed from the base of the PR and between f358e16 and db68b64.

⛔ Files ignored due to path filters (2)
  • assets/images/lesson-cafe-scene-mascot.png is excluded by !**/*.png
  • assets/images/mascot-pink-creature.png is excluded by !**/*.png
📒 Files selected for processing (3)
  • README.md
  • components/audio-teacher-session.tsx
  • constants/images.ts

📝 Walkthrough

Walkthrough

Image assets are consolidated from multiple mascot sources to a single pink creature reference. The audio session UI now conditionally renders a greeting overlay with mascot when live captions are inactive, controlled by a call-in-progress flag. Project documentation is rewritten to describe the Lingua app structure and setup.

Changes

Greeting Overlay and Asset Consolidation

Layer / File(s) Summary
Image asset consolidation
constants/images.ts
Multiple mascot imports (mascotLogo, mascotAuth, mascotWelcome, aiTeacherFoxSweater) and lesson scene assets are consolidated: mascot references now point to mascotPinkCreature, teacherPortrait uses mascotPinkCreature, and lessonArt maps cafe/cafeScene to lessonCafeSceneMascot with completed using mascotPinkCreature.
Greeting overlay UI and live caption gating
components/audio-teacher-session.tsx
Audio session introduces shouldShowLiveCaptions (set when call is in progress) to conditionally render either the live caption transcript with auto-scroll or a centered greeting overlay displaying mascot and text. Greeting mascot size is computed from viewport width, and a new greetingBubble style provides shadow/z-index styling.
Project documentation
README.md
README is rewritten from Expo starter template to a Lingua-specific guide covering app concept, features, technical stack, folder structure, environment variable setup (Clerk, Stream, Vision Agent, PostHog), Vision Agent service running, app routing/user flow, local data model conventions, styling rules, API route descriptions, quality-check commands, and development/security guidance.

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly Related PRs

  • Retsomm/react-native-lingua#12: Refactors audio-teacher-session.tsx to introduce live caption-driven transcript UI and call/speaking flow, directly overlapping with this PR's live caption state gating and greeting overlay conditional rendering.
  • Retsomm/react-native-lingua#7: Modifies the teacherPortrait entry in constants/images.ts; this PR remaps it to mascotPinkCreature while the retrieved PR maps it to mascotAuth.
  • Retsomm/react-native-lingua#9: Also touches components/audio-teacher-session.tsx to introduce/rework the AudioTeacherSession screen itself, providing context for this PR's greeting overlay and live caption behavior updates.

Poem

A pink creature greets with mascot cheer,
When lessons pause and captions disappear,
One image consolidated, clean and bright,
The UI blooms—a greeting just right! 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: updating the README with project-specific documentation and consolidating mascot asset references.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

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

@Retsomm Retsomm merged commit 1579b06 into main May 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant