Skip to content

Commit

Permalink
feat: #2 improve pubdev scores (#16)
Browse files Browse the repository at this point in the history
* Improving pub.dev score for description

* Updated depdencies

* More package upgrades

* Package structure improvements

- Moved package source to `src`
- Exposed "public API" with `lib/raygun_cli.dart`
- Created empty example
- Updated `bin/raygun_cli.dart` to use the exposed public API

---------

Co-authored-by: Miguel Beltran <[email protected]>
  • Loading branch information
TheRealAgentK and miquelbeltran authored Jan 27, 2025
1 parent d3a9cc5 commit 0f49a33
Show file tree
Hide file tree
Showing 22 changed files with 127 additions and 98 deletions.
5 changes: 1 addition & 4 deletions bin/raygun_cli.dart
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
import 'package:args/args.dart';
import 'package:raygun_cli/sourcemap/sourcemap_command.dart';
import 'package:raygun_cli/symbols/flutter_symbols.dart';
import 'package:raygun_cli/deployments/deployments_command.dart';
import 'package:raygun_cli/proguard/proguard_command.dart';
import 'package:raygun_cli/raygun_cli.dart';

const String version = '0.0.2';

Expand Down
5 changes: 5 additions & 0 deletions example/main.dart
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
void main() {
// This package is not intended to be used as a library.
// It is a command line tool.
// See README.md for usage.
}
4 changes: 4 additions & 0 deletions lib/raygun_cli.dart
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
export 'src/deployments/deployments_command.dart';
export 'src/proguard/proguard_command.dart';
export 'src/sourcemap/sourcemap_command.dart';
export 'src/symbols/flutter_symbols.dart';
2 changes: 1 addition & 1 deletion lib/config_props.dart → lib/src/config_props.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import 'dart:io';

import 'package:args/args.dart';
import 'package:raygun_cli/environment.dart';
import 'package:raygun_cli/src/environment.dart';

/// A Config property is a value
/// that can be set via argument
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
import 'dart:io';

import 'package:raygun_cli/deployments/deployments_api.dart';
import 'package:raygun_cli/src/deployments/deployments_api.dart';
import 'package:args/args.dart';
import '../config_props.dart';

/// Deployments command
class Deployments {
final ArgResults command;
final bool verbose;
Expand All @@ -13,6 +14,7 @@ class Deployments {
required this.verbose,
});

/// Notifies Raygun that a new deployment has been made.
Future<void> notify() async {
if (!command.wasParsed('version')) {
print('Error: Missing "--version"');
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import 'dart:convert';
import 'package:http/http.dart' as http;

/// Creates a deployment in Raygun.
Future<bool> createDeployment({
required String token,
required String apiKey,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,10 +1,11 @@
import 'dart:io';

import 'package:args/args.dart';
import 'package:raygun_cli/deployments/deployments.dart';
import 'package:raygun_cli/src/deployments/deployments.dart';

const kDeploymentsCommand = 'deployments';

/// Builds the deployments command parser
ArgParser buildParserDeployments() {
return ArgParser()
..addFlag(
Expand Down Expand Up @@ -58,6 +59,7 @@ ArgParser buildParserDeployments() {
);
}

/// Parses the deployments command arguments
void parseDeploymentsCommand(ArgResults command, bool verbose) {
if (command.wasParsed('help')) {
print('Usage: raygun-cli deployments <arguments>');
Expand Down
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import 'dart:io';

import 'package:raygun_cli/proguard/proguard_api.dart';
import 'package:raygun_cli/src/proguard/proguard_api.dart';
import 'package:args/args.dart';
import '../config_props.dart';

Expand Down
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import 'dart:io';

import 'package:args/args.dart';
import 'package:raygun_cli/proguard/proguard.dart';
import 'package:raygun_cli/src/proguard/proguard.dart';

const kProguardCommand = 'proguard';

Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
import 'dart:io';

import 'package:raygun_cli/config_props.dart';
import 'package:raygun_cli/sourcemap/sourcemap_api.dart';
import 'package:raygun_cli/sourcemap/sourcemap_base.dart';
import 'package:raygun_cli/src/config_props.dart';
import 'package:raygun_cli/src/sourcemap/sourcemap_api.dart';
import 'package:raygun_cli/src/sourcemap/sourcemap_base.dart';

/// Uploads a Flutter web sourcemap file to Raygun.
class SourcemapFlutter extends SourcemapBase {
SourcemapFlutter({
required super.command,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
import 'dart:io';

import 'package:raygun_cli/sourcemap/sourcemap_base.dart';
import 'package:raygun_cli/src/sourcemap/sourcemap_base.dart';

/// Uploads a Node app sourcemap file to Raygun.
/// TODO: Not implemented yet.
class SourcemapNode extends SourcemapBase {
SourcemapNode({
required super.command,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
import 'dart:io';
import 'package:http/http.dart' as http;

/// Uploads a sourcemap file to Raygun.
/// returns true if the upload was successful.
Future<bool> uploadSourcemap({
required String appId,
required String token,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,13 +1,18 @@
import 'package:args/args.dart';

/// Base class for JavaScript sourcemap uploaders
abstract class SourcemapBase {
SourcemapBase({
required this.command,
required this.verbose,
});

/// Command line arguments
final ArgResults command;

/// Print verbose output
final bool verbose;

/// Uploads the sourcemap
Future<void> upload();
}
Original file line number Diff line number Diff line change
@@ -1,12 +1,13 @@
import 'dart:io';

import 'package:args/args.dart';
import 'package:raygun_cli/sourcemap/flutter/sourcemap_flutter.dart';
import 'package:raygun_cli/sourcemap/node/sourcemap_node.dart';
import 'package:raygun_cli/sourcemap/sourcemap_single_file.dart';
import 'package:raygun_cli/src/sourcemap/flutter/sourcemap_flutter.dart';
import 'package:raygun_cli/src/sourcemap/node/sourcemap_node.dart';
import 'package:raygun_cli/src/sourcemap/sourcemap_single_file.dart';

const kSourcemapCommand = 'sourcemap';

/// Creates a parser for the sourcemap command
ArgParser buildParserSourcemap() {
return ArgParser()
..addFlag(
Expand Down Expand Up @@ -48,6 +49,7 @@ ArgParser buildParserSourcemap() {
);
}

/// Parses the sourcemap command
void parseSourcemapCommand(ArgResults command, bool verbose) {
if (command.wasParsed('help')) {
print('Usage: raygun-cli sourcemap <arguments>');
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
import 'dart:io';

import 'package:raygun_cli/config_props.dart';
import 'package:raygun_cli/sourcemap/sourcemap_api.dart';
import 'package:raygun_cli/sourcemap/sourcemap_base.dart';
import 'package:raygun_cli/src/config_props.dart';
import 'package:raygun_cli/src/sourcemap/sourcemap_api.dart';
import 'package:raygun_cli/src/sourcemap/sourcemap_base.dart';

/// Uploads a single sourcemap file to Raygun.
class SourcemapSingleFile extends SourcemapBase {
SourcemapSingleFile({
required super.command,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
import 'dart:io';

import 'package:args/args.dart';
import 'package:raygun_cli/config_props.dart';
import 'package:raygun_cli/symbols/flutter_symbols_api.dart';
import 'package:raygun_cli/src/config_props.dart';
import 'package:raygun_cli/src/symbols/flutter_symbols_api.dart';

const kSymbolsCommand = 'symbols';

/// Parses the symbols command
void parseSymbolsCommand(ArgResults command, bool verbose) {
if (command.wasParsed('help')) {
print(
Expand Down Expand Up @@ -73,6 +74,7 @@ Future<bool> _run({
return false;
}

/// Builds the symbols command parser
ArgParser buildParserSymbols() {
return ArgParser()
..addFlag(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import 'dart:convert';
import 'dart:io';
import 'package:http/http.dart' as http;

/// Uploads Flutter symbols files to Raygun.
Future<bool> uploadSymbols({
required String appId,
required String token,
Expand Down Expand Up @@ -42,6 +43,7 @@ Future<bool> uploadSymbols({
}
}

/// Lists Flutter symbols files in Raygun.
Future<bool> listSymbols({
required String appId,
required String token,
Expand Down Expand Up @@ -72,6 +74,7 @@ Future<bool> listSymbols({
}
}

/// Deletes a Flutter symbols file in Raygun.
Future<bool> deleteSymbols({
required String appId,
required String token,
Expand Down
Loading

0 comments on commit 0f49a33

Please sign in to comment.