hide github token

For github.rest, use vscode settings (user setting, not
.vscode/settings, so it won't tracked by source control).
For dart source, use dotenv to save token in untracked .env, packages it
as asset, and read the token at runtime.

Author: michael-wang

.gitignore

@@ -44,3 +44,5 @@ app.*.map.json
 /android/app/debug
 /android/app/profile
 /android/app/release
+
+*.env

.vscode/github.rest

@@ -1,17 +1,18 @@
 @baseUrl = https://api.github.com
 
-@token = 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
+@token = {{githubToken}}
 
 @acceptJson = application/vnd.github.v3+json
 
 @acceptHTML = application/vnd.github.v3.html+json
 
 GET {{baseUrl}}/user/starred
     ?sort=stars
-    ?order=desc
-    ?page=1
+    &order=desc
+    &page=1
 Authorization: token {{token}}
 Accept: {{acceptJson}}
+#If-None-Match: W/"fc39e9c03134895d01a6b94e1cb27b1d805e339a1a8d41230c8ab02c239f112e"
 
 ###
 
@@ -20,7 +21,7 @@ GET {{baseUrl}}/search/repositories
     &sort=stars
     &order=desc
     &page=1
-Authorization: token {{token}}
+# Authorization: token {{token}}
 Accept: {{acceptJson}}
 
 ###

lib/core/domain/fresh.dart

@@ -0,0 +1,12 @@
+import 'package:freezed_annotation/freezed_annotation.dart';
+
+part 'fresh.freezed.dart';
+
+@freezed
+class Fresh<T> with _$Fresh<T> {
+  const Fresh._();
+  const factory Fresh({
+    required T entity,
+    required bool isFresh,
+  }) = _Fresh<T>;
+}

lib/core/domain/fresh.freezed.dart

@@ -0,0 +1,159 @@
+// coverage:ignore-file
+// GENERATED CODE - DO NOT MODIFY BY HAND
+// ignore_for_file: unused_element, deprecated_member_use, deprecated_member_use_from_same_package, use_function_type_syntax_for_parameters, unnecessary_const, avoid_init_to_null, invalid_override_different_default_values_named, prefer_expression_function_bodies, annotate_overrides, invalid_annotation_target
+
+part of 'fresh.dart';
+
+// **************************************************************************
+// FreezedGenerator
+// **************************************************************************
+
+T _$identity<T>(T value) => value;
+
+final _privateConstructorUsedError = UnsupportedError(
+    'It seems like you constructed your class using `MyClass._()`. This constructor is only meant to be used by freezed and you are not supposed to need it nor use it.\nPlease check the documentation here for more informations: https://github.com/rrousselGit/freezed#custom-getters-and-methods');
+
+/// @nodoc
+class _$FreshTearOff {
+  const _$FreshTearOff();
+
+  _Fresh<T> call<T>({required T entity, required bool isFresh}) {
+    return _Fresh<T>(
+      entity: entity,
+      isFresh: isFresh,
+    );
+  }
+}
+
+/// @nodoc
+const $Fresh = _$FreshTearOff();
+
+/// @nodoc
+mixin _$Fresh<T> {
+  T get entity => throw _privateConstructorUsedError;
+  bool get isFresh => throw _privateConstructorUsedError;
+
+  @JsonKey(ignore: true)
+  $FreshCopyWith<T, Fresh<T>> get copyWith =>
+      throw _privateConstructorUsedError;
+}
+
+/// @nodoc
+abstract class $FreshCopyWith<T, $Res> {
+  factory $FreshCopyWith(Fresh<T> value, $Res Function(Fresh<T>) then) =
+      _$FreshCopyWithImpl<T, $Res>;
+  $Res call({T entity, bool isFresh});
+}
+
+/// @nodoc
+class _$FreshCopyWithImpl<T, $Res> implements $FreshCopyWith<T, $Res> {
+  _$FreshCopyWithImpl(this._value, this._then);
+
+  final Fresh<T> _value;
+  // ignore: unused_field
+  final $Res Function(Fresh<T>) _then;
+
+  @override
+  $Res call({
+    Object? entity = freezed,
+    Object? isFresh = freezed,
+  }) {
+    return _then(_value.copyWith(
+      entity: entity == freezed
+          ? _value.entity
+          : entity // ignore: cast_nullable_to_non_nullable
+              as T,
+      isFresh: isFresh == freezed
+          ? _value.isFresh
+          : isFresh // ignore: cast_nullable_to_non_nullable
+              as bool,
+    ));
+  }
+}
+
+/// @nodoc
+abstract class _$FreshCopyWith<T, $Res> implements $FreshCopyWith<T, $Res> {
+  factory _$FreshCopyWith(_Fresh<T> value, $Res Function(_Fresh<T>) then) =
+      __$FreshCopyWithImpl<T, $Res>;
+  @override
+  $Res call({T entity, bool isFresh});
+}
+
+/// @nodoc
+class __$FreshCopyWithImpl<T, $Res> extends _$FreshCopyWithImpl<T, $Res>
+    implements _$FreshCopyWith<T, $Res> {
+  __$FreshCopyWithImpl(_Fresh<T> _value, $Res Function(_Fresh<T>) _then)
+      : super(_value, (v) => _then(v as _Fresh<T>));
+
+  @override
+  _Fresh<T> get _value => super._value as _Fresh<T>;
+
+  @override
+  $Res call({
+    Object? entity = freezed,
+    Object? isFresh = freezed,
+  }) {
+    return _then(_Fresh<T>(
+      entity: entity == freezed
+          ? _value.entity
+          : entity // ignore: cast_nullable_to_non_nullable
+              as T,
+      isFresh: isFresh == freezed
+          ? _value.isFresh
+          : isFresh // ignore: cast_nullable_to_non_nullable
+              as bool,
+    ));
+  }
+}
+
+/// @nodoc
+
+class _$_Fresh<T> extends _Fresh<T> {
+  const _$_Fresh({required this.entity, required this.isFresh}) : super._();
+
+  @override
+  final T entity;
+  @override
+  final bool isFresh;
+
+  @override
+  String toString() {
+    return 'Fresh<$T>(entity: $entity, isFresh: $isFresh)';
+  }
+
+  @override
+  bool operator ==(dynamic other) {
+    return identical(this, other) ||
+        (other is _Fresh<T> &&
+            (identical(other.entity, entity) ||
+                const DeepCollectionEquality().equals(other.entity, entity)) &&
+            (identical(other.isFresh, isFresh) ||
+                const DeepCollectionEquality().equals(other.isFresh, isFresh)));
+  }
+
+  @override
+  int get hashCode =>
+      runtimeType.hashCode ^
+      const DeepCollectionEquality().hash(entity) ^
+      const DeepCollectionEquality().hash(isFresh);
+
+  @JsonKey(ignore: true)
+  @override
+  _$FreshCopyWith<T, _Fresh<T>> get copyWith =>
+      __$FreshCopyWithImpl<T, _Fresh<T>>(this, _$identity);
+}
+
+abstract class _Fresh<T> extends Fresh<T> {
+  const factory _Fresh({required T entity, required bool isFresh}) =
+      _$_Fresh<T>;
+  const _Fresh._() : super._();
+
+  @override
+  T get entity => throw _privateConstructorUsedError;
+  @override
+  bool get isFresh => throw _privateConstructorUsedError;
+  @override
+  @JsonKey(ignore: true)
+  _$FreshCopyWith<T, _Fresh<T>> get copyWith =>
+      throw _privateConstructorUsedError;
+}

lib/github/repos/starred/infra/starred_repos_remote_service.dart

@@ -1,6 +1,7 @@
 import 'dart:io';
 
 import 'package:dio/dio.dart';
+import 'package:flutter_dotenv/flutter_dotenv.dart';
 import 'package:repo_viewer/core/infra/dio_extensions.dart';
 import 'package:repo_viewer/core/infra/network_exceptions.dart';
 import 'package:repo_viewer/core/infra/remote_response.dart';
@@ -20,7 +21,7 @@ class StarredReposRemoteService {
 
   Future<RemoteResponse<List<GithubRepoDTO>>> getStarredReposPage(
       int page) async {
-    const token = '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';
+    final token = dotenv.env['GITHUB_TOKEN'];
     const accept = 'application/vnd.github.v3+json';
     final reqURL = Uri.https(
       'api.github.com',

lib/main.dart

@@ -1,9 +1,13 @@
 import 'package:flutter/material.dart';
+import 'package:flutter_dotenv/flutter_dotenv.dart';
 import 'package:hooks_riverpod/hooks_riverpod.dart';
 import 'package:repo_viewer/core/presentation/app_widget.dart';
 
-void main() => runApp(
-      ProviderScope(
-        child: AppWidget(),
-      ),
-    );
+void main() async {
+  await dotenv.load(fileName: ".env");
+  runApp(
+    ProviderScope(
+      child: AppWidget(),
+    ),
+  );
+}

pubspec.lock

@@ -237,6 +237,13 @@ packages:
     description: flutter
     source: sdk
     version: "0.0.0"
+  flutter_dotenv:
+    dependency: "direct main"
+    description:
+      name: flutter_dotenv
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "5.0.2"
   flutter_hooks:
     dependency: "direct main"
     description:
@@ -400,6 +407,13 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "0.12.11"
+  material_color_utilities:
+    dependency: transitive
+    description:
+      name: material_color_utilities
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "0.1.3"
   material_design_icons_flutter:
     dependency: "direct main"
     description:
@@ -644,7 +658,7 @@ packages:
       name: test_api
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "0.4.3"
+    version: "0.4.8"
   timing:
     dependency: transitive
     description:

pubspec.yaml

@@ -15,6 +15,7 @@ dependencies:
   flash: ^2.0.3
   flutter:
     sdk: flutter
+  flutter_dotenv: ^5.0.2
   flutter_hooks: ^0.16.0
   flutter_secure_storage: ^4.2.1
   freezed_annotation: ^0.14.2
@@ -55,3 +56,6 @@ dependency_overrides:
 
 flutter:
   uses-material-design: true
+
+assets:
+  - .env