Skip to content

feat: create cli tool #608

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

Open
wants to merge 11 commits into
base: dev
Choose a base branch
from
Open

feat: create cli tool #608

wants to merge 11 commits into from

Conversation

MoKamall
Copy link
Collaborator

@MoKamall MoKamall commented Jul 23, 2025

Description of the change

Creates a CLI tool for instabug-flutter. Currently supports one command: upload-so-files

Usage

run dart run instabug_flutter:instabug --help to get started with the CLI.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

MOB-19407

Checklists

Development

  • Lint rules pass locally
  • The code changed/added as part of this pull request has been covered with tests

Code review

  • This pull request has a descriptive title and information useful to a reviewer
  • Issue from task tracker has a link to this pull request

MoKamall added 3 commits July 23, 2025 12:52
- Introduced a new command `upload-so-files` for uploading .so files with architecture and API key options.
- Enhanced the main CLI to support command registration and help functionality.
- Updated dependencies in `pubspec.yaml` for `args` and `http`.
- Added `android/app/.cxx/` to ignore C++ build artifacts for Android.
@MoKamall MoKamall changed the base branch from master to dev July 23, 2025 17:01
@MoKamall MoKamall self-assigned this Jul 23, 2025
@MoKamall MoKamall changed the title Feat/create cli tool feat: create cli tool Jul 23, 2025
@InstabugCI
Copy link
Collaborator

InstabugCI commented Jul 23, 2025

Coverage Report

Label Coverage Status
Dart 83.8%

Generated by 🚫 dangerJS against 8c55f00

MoKamall added 5 commits July 24, 2025 03:36
- Removed `http` dependency from `pubspec.yaml`.
- Added a new `makeHttpPostRequest` function in `bin/instabug.dart` for handling HTTP POST requests.
- Updated `upload_so_files.dart` to utilize the new HTTP request function for uploading .so files.
- Changed `args` dependency version from `^2.4.0` to `^2.3.0` for compatibility reasons.
- Add `http` package dependency, and use it instead of manually making the upload post request.
@MoKamall MoKamall marked this pull request as ready for review July 28, 2025 11:15
AyaMahmoud148
AyaMahmoud148 previously approved these changes Jul 29, 2025
flutter:
sdk: flutter
http: ^0.13.3
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should make the HTTP library version more flexible by not specifying a fixed version, to avoid potential conflicts with users' dependencies.

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.

5 participants