Skip to content

Conversation

@felarof99
Copy link
Contributor

  • fix: use source files for agent-sdk during development
  • feat: improve onboarding page

greptile-apps[bot]

This comment was marked as resolved.

@browseros-ai browseros-ai deleted a comment from greptile-apps bot Jan 22, 2026
@browseros-ai browseros-ai deleted a comment from claude bot Jan 22, 2026
@shadowfax92 shadowfax92 force-pushed the feat/onboarding-fixes-2 branch from 580abda to e579d05 Compare January 22, 2026 17:29
@shadowfax92
Copy link
Contributor

@greptileai

@shadowfax92 shadowfax92 changed the title feat: onboarding fixes (partial) feat: onboarding fixes Jan 22, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 22, 2026

Greptile Summary

This PR simplifies the onboarding features page UI and updates feature content. The Feature interface was streamlined by replacing nested highlight objects with a simple string array, and the dialog layout now uses a cleaner bullet list instead of a bento grid. Feature descriptions were rewritten to be more concise and compelling.

Major changes:

  • Simplified Feature interface: removed ReactNode support, nested highlight structure, and tip field
  • Replaced complex highlight cards with simple checkmark bullet list in dialog
  • Refactored all feature content with updated descriptions and highlights
  • Added "Explore Features" link to settings sidebar
  • Added three new media URL constants (marked with TODOs for placeholder URLs)

Issue found:

  • The AGENTIC_CODING_DEMO_URL constant points to cowork-hn-extract-demo.mp4, which appears to be mismatched with the feature name

Confidence Score: 4/5

  • This PR is safe to merge with one minor video URL mismatch
  • The changes are primarily UI improvements and content updates with no logic errors or security concerns. The only issue is a likely video URL mismatch where the Agentic Coding demo appears to point to a Cowork demo file. This won't cause runtime errors but may confuse users.
  • Check mediaUrls.ts to verify the video URL mappings are correct

Important Files Changed

Filename Overview
apps/agent/components/sidebar/SettingsSidebar.tsx Added "Explore Features" navigation item linking to /onboarding/features with Compass icon
apps/agent/entrypoints/onboarding/features/BentoCard.tsx Simplified Feature interface and dialog layout - removed nested highlight objects in favor of simple string array
apps/agent/entrypoints/onboarding/features/Features.tsx Refactored feature content with updated descriptions and highlights for all six features (agent, mcp-server, workflows, cowork, split-view, agentic-coding)
apps/agent/lib/constants/mediaUrls.ts Added three new media URL constants (COWORK_DEMO_URL, WORKFLOWS_DEMO_URL, AGENTIC_CODING_DEMO_URL) with TODO comments indicating placeholder URLs

Sequence Diagram

sequenceDiagram
    participant User
    participant SettingsSidebar
    participant FeaturesPage
    participant BentoCard
    participant Dialog
    participant VideoFrame

    User->>SettingsSidebar: Click "Explore Features"
    SettingsSidebar->>FeaturesPage: Navigate to /onboarding/features
    FeaturesPage->>FeaturesPage: Load features array (6 items)
    FeaturesPage->>User: Display hero section with intro video
    User->>User: Scroll to features section
    
    loop For each feature
        FeaturesPage->>BentoCard: Render feature card
        BentoCard->>User: Display simplified card (title, tag, description)
    end
    
    User->>BentoCard: Click feature card
    BentoCard->>Dialog: Open dialog
    Dialog->>VideoFrame: Load feature video/gif
    VideoFrame->>User: Autoplay video with controls
    Dialog->>User: Display detailedDescription
    Dialog->>User: Render highlights as bullet list with checkmarks
    
    User->>Dialog: Click close or backdrop
    Dialog->>FeaturesPage: Return to features grid
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment on lines +49 to +50
export const AGENTIC_CODING_DEMO_URL =
'https://pub-80f8a01e6e8b4239ae53a7652ef85877.r2.dev/resources/cowork-hn-extract-demo.mp4'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

logic: The URL for AGENTIC_CODING_DEMO_URL appears to be incorrectly mapped — it points to cowork-hn-extract-demo.mp4, which sounds like a Cowork demo, not an Agentic Coding demo.

Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/agent/lib/constants/mediaUrls.ts
Line: 49:50

Comment:
**logic:** The URL for `AGENTIC_CODING_DEMO_URL` appears to be incorrectly mapped — it points to `cowork-hn-extract-demo.mp4`, which sounds like a Cowork demo, not an Agentic Coding demo.

How can I resolve this? If you propose a fix, please make it concise.

@claude
Copy link

claude bot commented Jan 22, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@shadowfax92 shadowfax92 force-pushed the feat/onboarding-fixes-2 branch from ecef7d5 to 191ef2f Compare January 22, 2026 17:55
@shadowfax92 shadowfax92 merged commit ee29881 into main Jan 22, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants