Skip to content

docs(assets): Add pics and videos for mothership#4216

Merged
TheodoreSpeaks merged 2 commits intostagingfrom
doc/update-docs
Apr 17, 2026
Merged

docs(assets): Add pics and videos for mothership#4216
TheodoreSpeaks merged 2 commits intostagingfrom
doc/update-docs

Conversation

@TheodoreSpeaks
Copy link
Copy Markdown
Collaborator

Summary

Add videos and pictures describing different mothership capabilities.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Other: ___________

Testing

  • Rendered locally.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Apr 17, 2026 8:24pm

Request Review

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 17, 2026

PR Summary

Low Risk
Documentation-only content updates plus a small, backwards-compatible UI prop addition to Video; low risk aside from potential layout/asset path issues in docs rendering.

Overview
Adds optional width/height props to the docs Video component and passes them through to the underlying <video> element.

Updates multiple English docs pages (Copilot and Mothership sections) to embed new demo videos and screenshots at the top of pages/sections, and removes the workflow-deletion warning callout from the Mothership Workflows doc.

Reviewed by Cursor Bugbot for commit e822678. Bugbot is set up for automated code reviews on this repo. Configure here.

@TheodoreSpeaks TheodoreSpeaks changed the base branch from main to staging April 17, 2026 20:06
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 17, 2026

Greptile Summary

This PR replaces TODO placeholder comments across Mothership and Copilot docs with real screenshots and videos, and extends the Video component with optional width/height props to support them. Most pages are well filled-in, but three minor issues are worth addressing: a destructive-action warning callout was removed from the workflows page, scheduled-task.png is added but never referenced, and create-workflow.mp4 is used as the hero on both the overview and workflows subpages.

Confidence Score: 5/5

Safe to merge — all findings are P2 documentation quality issues that do not block functionality.

All three findings are P2: a removed callout (info preserved in FAQ), an unused asset, and a duplicate video. None affects runtime behavior or user safety in a blocking way.

apps/docs/content/docs/en/mothership/workflows.mdx (removed deletion warning); apps/docs/public/static/mothership/scheduled-task.png (orphaned asset)

Important Files Changed

Filename Overview
apps/docs/components/ui/video.tsx Adds optional width and height props to the Video component and passes them through to the underlying <video> element — clean, backward-compatible change.
apps/docs/content/docs/en/mothership/workflows.mdx Replaces several TODO placeholders with videos, but also removes the prominent deletion-warning <Callout> and reuses the same create-workflow.mp4 as the hero that the overview page already uses.
apps/docs/content/docs/en/mothership/index.mdx Adds hero video and context-menu / split-view clips; two TODO placeholder comments remain unfilled.
apps/docs/content/docs/en/mothership/tasks.mdx Adds hero video; five TODO placeholder comments remain; scheduled-task.png asset was added to the repo but is never referenced here.
apps/docs/content/docs/en/mothership/files.mdx Replaces all TODO placeholders with real assets (video + three screenshots); clean improvement.
apps/docs/content/docs/en/mothership/knowledge.mdx Swaps TODO placeholder for a kb.mp4 hero video; no issues.
apps/docs/content/docs/en/mothership/research.mdx Swaps TODO placeholder for a research-agent.mp4 hero video; no issues.
apps/docs/content/docs/en/mothership/tables.mdx Replaces TODO placeholder with table-example.png screenshot; no issues.
apps/docs/content/docs/en/copilot/index.mdx Swaps Copilot TODO placeholder for a copilot.mp4 video; no issues.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Video Component] -->|width/height props added| B[video element]
    B --> C{enableLightbox?}
    C -->|Yes| D[Lightbox overlay]
    C -->|No| E[Inline playback only]

    subgraph MDX Pages
        P1[mothership/index.mdx] -->|create-workflow.mp4 + context-menu.mp4 + split-view.mp4| A
        P2[mothership/workflows.mdx] -->|create-workflow.mp4 + run-workflow.mp4| A
        P3[mothership/files.mdx] -->|files-pipeline-deals-summarizer.mp4 + toggle-file-view.mp4| A
        P4[mothership/knowledge.mdx] -->|kb.mp4| A
        P5[mothership/research.mdx] -->|research-agent.mp4| A
        P6[mothership/tasks.mdx] -->|job-create.mp4| A
        P7[copilot/index.mdx] -->|copilot.mp4| A
    end

    subgraph Static Images
        I1[pptx-example.png]
        I2[image-example.png]
        I3[chart-example.png]
        I4[table-example.png]
        I5[scheduled-task.png - unreferenced]
    end
Loading

Comments Outside Diff (2)

  1. apps/docs/content/docs/en/mothership/workflows.mdx, line 108-111 (link)

    P2 Deletion warning callout removed

    The PR removes the <Callout type="warn"> block that explicitly warned users that workflow deletion is permanent and removes deployed versions. The FAQ at the bottom now contains equivalent text, but a collapsible FAQ entry is far less visible than an inline warning callout for a destructive, irreversible action. Users who don't read the FAQ may delete a deployed workflow without understanding the consequences.

    Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

  2. apps/docs/public/static/mothership/scheduled-task.png, line 1 (link)

    P2 Orphaned static asset

    scheduled-task.png is added to the repository but is not referenced anywhere in the MDX files changed in this PR (including tasks.mdx). This asset will never be served to users and just adds dead weight to the docs bundle.

Reviews (1): Last reviewed commit: "Add pics and videos for mothership" | Re-trigger Greptile

Comment thread apps/docs/content/docs/en/mothership/workflows.mdx
@TheodoreSpeaks TheodoreSpeaks merged commit 5e716d7 into staging Apr 17, 2026
14 checks passed
@TheodoreSpeaks TheodoreSpeaks deleted the doc/update-docs branch April 17, 2026 20:31
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