Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
There was a problem hiding this comment.
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); |
There was a problem hiding this comment.
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.