-
Notifications
You must be signed in to change notification settings - Fork 36
update main to dev #416
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update main to dev #416
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces a new CLI package for the Reown AppKit React Native toolkit. The CLI provides an easy way for developers to scaffold new React Native projects using the AppKit template.
- Adds a new
@reown/appkit-react-native-clipackage with initialization functionality - Creates a command-line interface that uses Expo's create-expo tool with a predefined template
- Includes proper TypeScript configuration and build setup for the CLI package
Reviewed Changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| packages/cli/package.json | Package configuration for the new CLI with dependencies and build scripts |
| packages/cli/src/index.ts | Main CLI entry point that spawns expo create with template URL |
| packages/cli/src/utils.ts | ASCII art banner for the CLI |
| packages/cli/tsconfig.json | TypeScript configuration extending root config |
| packages/cli/bob.config.js | Build configuration for CommonJS, ES modules, and TypeScript outputs |
| packages/cli/CHANGELOG.md | Changelog documenting the initial CLI feature |
| packages/cli/.npmignore | NPM ignore file excluding source files from published package |
| package.json | Root package.json updated to include the new CLI package in workspaces |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| await runExpoCreate(); | ||
| } catch (error: any) { | ||
| // eslint-disable-next-line no-console | ||
| console.error('Failed to run Expo initializer:', error?.message || error); |
Copilot
AI
Aug 21, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using 'any' type defeats the purpose of TypeScript's type safety. Consider using 'unknown' type instead and properly type guard the error, or create a specific error type.
| console.error('Failed to run Expo initializer:', error?.message || error); | |
| } catch (error: unknown) { | |
| // eslint-disable-next-line no-console | |
| let errorMessage: string; | |
| if (typeof error === 'object' && error !== null && 'message' in error && typeof (error as any).message === 'string') { | |
| errorMessage = (error as { message: string }).message; | |
| } else { | |
| errorMessage = String(error); | |
| } | |
| console.error('Failed to run Expo initializer:', errorMessage); |
|



No description provided.