feat: implement global audio manager and default agent title fallback

[anthhub]

Summary by CodeRabbit

• New Features

  • Introduced audio playback coordination system to ensure only one audio plays simultaneously.
  • Added default agent title display with localization support (English and Chinese).

• Improvements

  • Enhanced workflow execution notifications—success/error alerts are now suppressed when resuming from saved URLs, showing only for new runs.
  • Refined UI scrolling behavior in results grid.
  • Improved fallback display for missing agent titles in workflow status UI.

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

[coderabbitai]

Walkthrough

This PR introduces a global audio manager for coordinating HTMLAudioElement playback, adds internationalization for a default agent title, suppresses workflow notifications on initial load with executionId, and adjusts UI scrolling behavior in the results grid.

Changes

Cohort / File(s)	Summary
Audio Management audio-manager.ts, audio.tsx	Introduces global AudioManager singleton to register and coordinate audio elements, ensuring only one plays simultaneously. Audio component integrates via useEffect and registers its ref with automatic cleanup.
Internationalization en-US/ui.ts, zh-Hans/ui.ts	Adds defaultAgentTitle translation key ('Agent' in English, '智能体' in Simplified Chinese) for fallback agent naming.
Layout selected-results-grid.tsx	Removes overflow-y-auto class from root container div, reverting to default scrolling behavior.
Workflow Application workflow-app/index.tsx	Tracks initial load state with executionId via isInitialLoadWithExecutionIdRef to conditionally suppress success/error notifications, reset flags after completion/failure, clear URL on E1021 error, and use defaultAgentTitle fallback for missing node titles.

Sequence Diagram(s)

sequenceDiagram
    participant Audio as HTMLAudioElement
    participant App as React Component
    participant Manager as AudioManager

    App->>Manager: register(audioElement)
    Manager->>Audio: attach play event listener
    Manager->>Audio: attach pause event listener
    Manager->>Audio: attach ended event listener
    
    Audio->>Manager: fire play event
    Manager->>Manager: stopAllExcept(current)
    Manager->>Manager: set currentAudio reference
    
    Audio->>Manager: fire ended event
    Manager->>Manager: clear currentAudio reference
    
    App->>Manager: unregister() on cleanup
    Manager->>Audio: detach all listeners
    Manager->>Manager: remove from registry

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

• packages/web-core/src/pages/workflow-app/index.tsx: Dense conditional logic across multiple handlers (finish, failed, onError, onSubmit) with state management tied to initial load semantics and executionId URL handling requires careful flow verification
• Interactions between audio manager and audio component: Verify ref lifecycle, listener cleanup, and unregister function timing
• Internationalization fallback integration: Confirm defaultAgentTitle is correctly applied as fallback for missing node titles in UI

Possibly related PRs

• feat(i18n): add agent translation key and use i18n for default agent … #1631: Adds same default agent translation key and implements UI fallback for missing node titles
• refactor(workflow-app): remove unused searchParams and related logic #1488: Modifies workflow-app/index.tsx for executionId handling and notification suppression logic
• feat(workflow-app): preserve workflow products when ending execution  #1854: Handles initial-load semantics for executionId from URL with conditional notification behavior

Suggested reviewers

• mrcfps
• pftom

Poem

🐰 A global tune coordinator hops in,
One audio plays whilst others spin,
Workflows quiet on first-load display,
Smart agents default when titles stray — ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title accurately captures the two main features added: a global audio manager and default agent title fallback, which align with the substantial changes across audio-related and workflow components.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/remain-bugs

---

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.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts (1)

11-11: Consider adding explicit return type annotation.

Per coding guidelines, functions should have explicit return types defined, especially for public APIs.

-  register(audioElement: HTMLAudioElement): () => void {
+  public register(audioElement: HTMLAudioElement): () => void {

The return type is already defined which is good. Optionally, you could also mark register and stopAll as public explicitly for clarity, though TypeScript defaults to public.

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a043e16 and 99ed41c.

📒 Files selected for processing (6)

• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts (1 hunks)
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx (1 hunks)
• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx (1 hunks)
• packages/i18n/src/en-US/ui.ts (1 hunks)
• packages/i18n/src/zh-Hans/ui.ts (1 hunks)
• packages/web-core/src/pages/workflow-app/index.tsx (4 hunks)

🧰 Additional context used

📓 Path-based instructions (23)

**/*.{jsx,tsx}

📄 CodeRabbit inference engine (.cursorrules)

**/*.{jsx,tsx}: Always use tailwind css to style the component
Always wrap pure components with React.memo to prevent unnecessary re-renders
Always use useMemo for expensive computations or complex object creation
Always use useCallback for function props to maintain referential equality
Always specify proper dependency arrays in useEffect to prevent infinite loops
Always avoid inline object/array creation in render to prevent unnecessary re-renders
Always use proper key props when rendering lists
Always split nested components with closures into separate components to avoid performance issues and improve code maintainability

**/*.{jsx,tsx}: Always wrap pure components with React.memo to prevent unnecessary re-renders
Always use useMemo for expensive computations or complex object creation in React
Always use useCallback for function props to maintain referential equality in React
Always specify proper dependency arrays in useEffect to prevent infinite loops in React
Always avoid inline object/array creation in render to prevent unnecessary re-renders in React
Always use proper key props when rendering lists in React (avoid using index when possible)
Always split nested components with closures into separate components in React
Use lazy loading for components that are not immediately needed in React
Debounce handlers for events that might fire rapidly (resize, scroll, input) in React
Implement fallback UI for components that might fail in React
Use error boundaries to catch and handle runtime errors in React

**/*.{jsx,tsx}: Place each attribute on a new line when a component has multiple attributes in JSX
Use self-closing tags for elements without children in JSX
Keep JSX expressions simple, extract complex logic to variables
Put closing brackets for multi-line JSX on a new line

**/*.{jsx,tsx}: Component file names should match the component name
Organize function components in order: imports, type definitions, constants, component function, hook calls, e...

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/web-core/src/pages/workflow-app/index.tsx

**/*.{js,ts,jsx,tsx}

📄 CodeRabbit inference engine (.cursorrules)

**/*.{js,ts,jsx,tsx}: Always use optional chaining (?.) when accessing object properties
Always use nullish coalescing (??) or default values for potentially undefined values
Always check array existence before using array methods
Always validate object properties before destructuring
Always use single quotes for string literals in JavaScript/TypeScript code

**/*.{js,ts,jsx,tsx}: Use semicolons at the end of statements
Include spaces around operators (e.g., a + b instead of a+b)
Always use curly braces for control statements
Place opening braces on the same line as their statement

**/*.{js,ts,jsx,tsx}: Group import statements in order: React/framework libraries, third-party libraries, internal modules, relative path imports, type imports, style imports
Sort imports alphabetically within each import group
Leave a blank line between import groups
Extract complex logic into custom hooks
Use functional updates for state (e.g., setCount(prev => prev + 1))
Split complex state into multiple state variables rather than single large objects
Use useReducer for complex state logic instead of multiple useState calls

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{js,ts,tsx,jsx,py,java,cpp,c,cs,rb,go,rs,php,swift,kt,scala,r,m,mm,sql}

📄 CodeRabbit inference engine (.cursor/rules/00-language-priority.mdc)

**/*.{js,ts,tsx,jsx,py,java,cpp,c,cs,rb,go,rs,php,swift,kt,scala,r,m,mm,sql}: All code comments MUST be written in English
All variable names, function names, class names, and other identifiers MUST use English words
Comments should be concise and explain 'why' rather than 'what'
Use proper grammar and punctuation in comments
Keep comments up-to-date when code changes
Document complex logic, edge cases, and important implementation details
Use clear, descriptive names that indicate purpose
Avoid abbreviations unless they are universally understood

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{js,ts,tsx,jsx}

📄 CodeRabbit inference engine (.cursor/rules/00-language-priority.mdc)

Use JSDoc style comments for functions and classes in JavaScript/TypeScript

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{js,jsx,ts,tsx}

📄 CodeRabbit inference engine (.cursor/rules/01-code-style.mdc)

**/*.{js,jsx,ts,tsx}: Use single quotes for string literals in TypeScript/JavaScript
Always use optional chaining (?.) when accessing object properties in TypeScript/JavaScript
Always use nullish coalescing (??) or default values for potentially undefined values in TypeScript/JavaScript
Always check array existence before using array methods in TypeScript/JavaScript
Validate object properties before destructuring in TypeScript/JavaScript
Use ES6+ features like arrow functions, destructuring, and spread operators in TypeScript/JavaScript
Avoid magic numbers and strings - use named constants in TypeScript/JavaScript
Use async/await instead of raw promises for asynchronous code in TypeScript/JavaScript

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{jsx,tsx,css}

📄 CodeRabbit inference engine (.cursor/rules/01-code-style.mdc)

**/*.{jsx,tsx,css}: Use Tailwind CSS for styling components
Follow the utility-first approach with Tailwind CSS
Group related utility classes together in Tailwind CSS
Prefer Tailwind utilities over custom CSS when possible

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/web-core/src/pages/workflow-app/index.tsx

**/*.{ts,tsx}

📄 CodeRabbit inference engine (.cursor/rules/03-typescript-guidelines.mdc)

**/*.{ts,tsx}: Avoid using any type whenever possible - use unknown type instead with proper type guards
Always define explicit return types for functions, especially for public APIs
Prefer extending existing types over creating entirely new types
Use TypeScript utility types (Partial<T>, Pick<T, K>, Omit<T, K>, Readonly<T>, Record<K, T>) to derive new types
Use union types and intersection types to combine existing types
Always import types explicitly using the import type syntax
Group type imports separately from value imports
Minimize creating local type aliases for imported types

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{js,ts,jsx,tsx,css,json}

📄 CodeRabbit inference engine (.cursor/rules/04-code-formatting.mdc)

Maximum line length of 100 characters

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{js,ts,jsx,tsx,css,json,yml,yaml}

📄 CodeRabbit inference engine (.cursor/rules/04-code-formatting.mdc)

Use 2 spaces for indentation, no tabs

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{js,ts,jsx,tsx,css,json,yml,yaml,md}

📄 CodeRabbit inference engine (.cursor/rules/04-code-formatting.mdc)

No trailing whitespace at the end of lines

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{jsx,tsx,js}

📄 CodeRabbit inference engine (.cursor/rules/05-code-organization.mdc)

Each component file should contain only one main component

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/web-core/src/pages/workflow-app/index.tsx

**/*.tsx

📄 CodeRabbit inference engine (.cursor/rules/05-code-organization.mdc)

Explicitly type props with interfaces or types in React components

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/web-core/src/pages/workflow-app/index.tsx

**/*.{css,scss,sass,less,js,jsx,ts,tsx}

📄 CodeRabbit inference engine (.cursor/rules/09-design-system.mdc)

**/*.{css,scss,sass,less,js,jsx,ts,tsx}: Primary color (#155EEF) should be used for main brand color in buttons, links, and accents
Error color (#F04438) should be used for error states and destructive actions
Success color (#12B76A) should be used for success states and confirmations
Warning color (#F79009) should be used for warnings and important notifications
Info color (#0BA5EC) should be used for informational elements

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{tsx,ts}

📄 CodeRabbit inference engine (.cursor/rules/09-i18n-guidelines.mdc)

**/*.{tsx,ts}: Use the translation wrapper component and useTranslation hook in components
Ensure all user-facing text is translatable

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{tsx,ts,json}

📄 CodeRabbit inference engine (.cursor/rules/09-i18n-guidelines.mdc)

Support dynamic content with placeholders in translations

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{tsx,ts,jsx,js,vue,css,scss,less}

📄 CodeRabbit inference engine (.cursor/rules/11-ui-design-patterns.mdc)

**/*.{tsx,ts,jsx,js,vue,css,scss,less}: Use the primary blue (#155EEF) for main UI elements, CTAs, and active states
Use red (#F04438) only for errors, warnings, and destructive actions
Use green (#12B76A) for success states and confirmations
Use orange (#F79009) for warning states and important notifications
Use blue (#0BA5EC) for informational elements
Primary buttons should be solid with the primary color
Secondary buttons should have a border with transparent or light background
Danger buttons should use the error color
Use consistent padding, border radius, and hover states for all buttons
Follow fixed button sizes based on their importance and context
Use consistent border radius (rounded-lg) for all cards
Apply light shadows (shadow-sm) for card elevation
Maintain consistent padding inside cards (p-4 or p-6)
Use subtle borders for card separation
Ensure proper spacing between card elements
Apply consistent styling to all form inputs
Use clear visual indicators for focus, hover, and error states in form elements
Apply proper spacing between elements using 8px, 16px, 24px increments
Ensure proper alignment of elements (left, center, or right)
Use responsive layouts that work across different device sizes
Maintain a minimum contrast ratio of 4.5:1 for text

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{tsx,ts,jsx,js,vue}

📄 CodeRabbit inference engine (.cursor/rules/11-ui-design-patterns.mdc)

**/*.{tsx,ts,jsx,js,vue}: Include appropriate loading states for async actions in buttons
Group related form elements with appropriate spacing
Provide clear validation feedback for forms
Ensure proper labeling and accessibility for form elements
Ensure all interactive elements are keyboard accessible
Include appropriate ARIA attributes for complex components
Provide alternative text for images and icons
Support screen readers with semantic HTML elements

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{ts,tsx,js,jsx}

📄 CodeRabbit inference engine (.cursor/rules/08-contributing-guidelines.mdc)

**/*.{ts,tsx,js,jsx}: Follow the TypeScript/JavaScript style guidelines
Ensure code is well-tested and documented

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/i18n/src/zh-Hans/ui.ts
• packages/web-core/src/pages/workflow-app/index.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts
• packages/i18n/src/en-US/ui.ts

**/*.{tsx,jsx}

📄 CodeRabbit inference engine (.cursor/rules/08-contributing-guidelines.mdc)

Use React best practices for frontend code

Files:

• packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx
• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx
• packages/web-core/src/pages/workflow-app/index.tsx

**/{i18n,locales,translations,lang}/**/*.{json,ts,tsx,js,jsx,properties}

📄 CodeRabbit inference engine (.cursor/rules/00-language-priority.mdc)

Use Chinese for all user-facing communication

Files:

• packages/i18n/src/zh-Hans/ui.ts
• packages/i18n/src/en-US/ui.ts

packages/i18n/src/{en-US,zh-Hans}/**

📄 CodeRabbit inference engine (.cursor/rules/09-i18n-guidelines.mdc)

Add new translation keys to both language files (en-US and zh-Hans)

Files:

• packages/i18n/src/zh-Hans/ui.ts
• packages/i18n/src/en-US/ui.ts

packages/i18n/src/**

📄 CodeRabbit inference engine (.cursor/rules/09-i18n-guidelines.mdc)

packages/i18n/src/**: Maintain consistency in translation key naming conventions
Use descriptive translation keys that reflect the content
Group related translation keys together in the translation file structure
Use namespaces for different sections of the application in translation keys
Follow a hierarchical structure for nested components in translation key organization

Files:

• packages/i18n/src/zh-Hans/ui.ts
• packages/i18n/src/en-US/ui.ts

**/index.{js,ts,jsx,tsx}

📄 CodeRabbit inference engine (.cursor/rules/05-code-organization.mdc)

Use index files to export multiple components from a directory

Files:

• packages/web-core/src/pages/workflow-app/index.tsx

🧠 Learnings (4)

📚 Learning: 2025-11-25T03:02:45.779Z

Learnt from: CR
Repo: refly-ai/refly PR: 0
File: .cursorrules:0-0
Timestamp: 2025-11-25T03:02:45.779Z
Learning: Applies to **/*.{jsx,tsx} : Always wrap pure components with React.memo to prevent unnecessary re-renders

Applied to files:

• packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx

📚 Learning: 2025-11-25T03:04:41.334Z

Learnt from: CR
Repo: refly-ai/refly PR: 0
File: .cursor/rules/09-i18n-guidelines.mdc:0-0
Timestamp: 2025-11-25T03:04:41.334Z
Learning: Applies to packages/i18n/src/{en-US,zh-Hans}/** : Add new translation keys to both language files (en-US and zh-Hans)

Applied to files:

• packages/i18n/src/zh-Hans/ui.ts
• packages/i18n/src/en-US/ui.ts

📚 Learning: 2025-11-25T03:03:19.158Z

Learnt from: CR
Repo: refly-ai/refly PR: 0
File: .cursor/rules/00-language-priority.mdc:0-0
Timestamp: 2025-11-25T03:03:19.158Z
Learning: Applies to **/{i18n,locales,translations,lang}/**/*.{json,ts,tsx,js,jsx,properties} : Use Chinese for all user-facing communication

Applied to files:

• packages/i18n/src/zh-Hans/ui.ts

📚 Learning: 2025-11-25T03:05:15.932Z

Learnt from: CR
Repo: refly-ai/refly PR: 0
File: .cursor/rules/16-language-guidelines.mdc:0-0
Timestamp: 2025-11-25T03:05:15.932Z
Learning: UI text should default to English with i18n support for Chinese

Applied to files:

• packages/i18n/src/zh-Hans/ui.ts

🧬 Code graph analysis (2)

packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx (2)

packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/types.ts (1)

• FileRendererProps (10-13)

packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts (1)

• audioManager (77-77)

packages/web-core/src/pages/workflow-app/index.tsx (4)

packages/openapi-schema/src/types.gen.ts (1)

• status (297-297)

packages/request/src/requests/types.gen.ts (1)

• status (297-297)

packages/ai-workspace-common/src/requests/types.gen.ts (1)

• status (297-297)

apps/api/src/modules/tool/sandbox/scalebox.dto.ts (1)

• error (214-220)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build / Build

🔇 Additional comments (13)

packages/ai-workspace-common/src/components/workflow-app/selected-results-grid.tsx (1)

129-129: LGTM: Scrolling behavior adjustment

Removing overflow-y-auto delegates scroll handling to the parent container, which aligns with the PR's goal of relying on default browser scrolling instead of constrained vertical overflow.

packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio.tsx (3)

1-3: LGTM: Necessary imports for audio manager integration

The added imports (useRef, useEffect, audioManager) are required for registering the audio element with the global audio manager.

---

9-18: LGTM: Proper audio manager registration with cleanup

The useEffect implementation correctly:

• Runs once on mount (empty dependency array)
• Performs null check before registration
• Returns the cleanup function to unregister on unmount

This ensures proper coordination of audio playback across the application and prevents memory leaks.

---

7-7: LGTM: Proper ref implementation for audio element

The audioRef is correctly typed and attached to the <audio> element, enabling the audio manager to coordinate playback with other audio instances in the application.

Also applies to: 23-23

packages/i18n/src/zh-Hans/ui.ts (1)

3827-3827: Translation keys are properly implemented in both language files.

Both defaultAgentTitle entries are present and correctly paired: 'Agent' in en-US/ui.ts (line 4089) and '智能体' in zh-Hans/ui.ts (line 3827). The i18n requirement is satisfied.

packages/i18n/src/en-US/ui.ts (1)

4089-4090: LGTM - Translation key added correctly.

The defaultAgentTitle key is appropriately placed within the workflowApp.run namespace. Based on learnings, ensure the corresponding key is also added to packages/i18n/src/zh-Hans/ui.ts (the AI summary indicates it was added with value '智能体').

packages/ai-workspace-common/src/components/canvas/canvas-resources/file-preview/audio-manager.ts (1)

1-77: Well-designed audio coordination singleton.

The AudioManager implementation is clean and handles the single-audio-at-a-time requirement effectively. Event listeners are properly attached and removed, preventing memory leaks.

packages/web-core/src/pages/workflow-app/index.tsx (6)

79-80: Good use of ref for tracking initial load state.

Using useRef to track whether the page loaded with an existing executionId is the correct approach - it avoids unnecessary re-renders while maintaining state across the component lifecycle.

---

203-222: Notification suppression logic is correctly implemented.

The conditional checks properly handle:

• Suppressing success notification on initial URL load with executionId
• Suppressing error notification when credit insufficient modal is visible
• Resetting the flag after first completion check to enable future notifications

---

239-242: Good error handling for E1021 error code.

Clearing the executionId from the URL when encountering error code E1021 (WorkflowExecutionNotFoundError based on context) is appropriate - it prevents the page from continuously trying to poll for a non-existent execution.

---

518-519: Correct flag reset when starting new execution.

Resetting isInitialLoadWithExecutionIdRef.current = false when the user manually starts a new workflow ensures that subsequent completion/error notifications will be displayed as expected.

---

957-960: Appropriate fallback for agent title.

Using the defaultAgentTitle translation key as a fallback when node.title is missing ensures consistent UI display across locales.

---

967-968: Consistent fallback pattern applied.

Same fallback pattern applied correctly for the single step display case.