Skip to content

feat: react native video skill#11

Open
santhoshvai wants to merge 10 commits into
mainfrom
feat/react-native-add-video
Open

feat: react native video skill#11
santhoshvai wants to merge 10 commits into
mainfrom
feat/react-native-add-video

Conversation

@santhoshvai
Copy link
Copy Markdown
Member

@santhoshvai santhoshvai commented May 15, 2026

Testing

  1. Symlink the skill into Claude Code's discovery path
ln -s /path/to/agent-skills/skills/stream-react-native ~/.claude/skills/stream-react-native
ln -s /path/to/agent-skills/skills/stream ~/.claude/skills/stream
ln -s /path/to/agent-skills/skills/stream-docs ~/.claude/skills/stream-docs
  1. Open a fresh Claude Code session inside any RN/Expo project (or just an empty directory for a new app)

  2. Run either of the prompts below

  1. /stream add a video call to my expo app — routes to stream-react-native, not stream-builder (peer-signal precedence).
  2. /stream-react-native wire stream video into a new expo app from scratch - generates a new app asking for credentials etc
  3. /stream-react-native I want both chat and video in my expo app — agent picks Chat + Video interop path

Out of scope

Feeds RN, livestream/audio-room/theming/component-override blueprints are out of scope (parity with PR #8)

@santhoshvai santhoshvai marked this pull request as ready for review May 18, 2026 13:50
# Conflicts:
#	skills/stream-react-native/RULES.md
#	skills/stream-react-native/builder.md
#	skills/stream-react-native/sdk.md
Copy link
Copy Markdown
Contributor

@isekovanic isekovanic left a comment

Choose a reason for hiding this comment

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

Some slight remarks, I want to triple check the removed snippets but otherwise looks great !

Comment thread skills/stream-react-native/builder.md

Required Android setup in the host app:

- `android/build.gradle`: `minSdkVersion = 24`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think specific hardcoded min/target versions might not be future proof, shall we omit these ? create-react-native-app and create-expo-app handle this themselves I believe.

Or is this some WebRCT specific thing ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

webrtc specific thing.. may get updated in the future.. but important to keep it here though, its a strict requirement for webrtc.. it is also in our video installtion docs

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

But I will update to make sure to keep it 24 only if the minSDK version is already 24+, this seems to hardcode to 24 always

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

sounds good, thanks !

Comment thread skills/stream-react-native/sdk.md
Copy link
Copy Markdown
Contributor

@isekovanic isekovanic left a comment

Choose a reason for hiding this comment

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

Double checked the snippets, they already belong to the blueprints spec ! They should've never been in builder.md/sdk.md at all. Thanks for the improvement !

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.

2 participants