Add issue links to TODO comments

packages/go_router/lib/src/information_provider.dart

@@ -33,6 +33,7 @@ class GoRouteInformationProvider extends RouteInformationProvider
             _valueInEngine.location == routeInformation.location);
     SystemNavigator.selectMultiEntryHistory();
     // TODO(chunhtai): report extra to browser through state if possible
+    // See https://github.com/flutter/flutter/issues/108142
     SystemNavigator.routeInformationUpdated(
       location: routeInformation.location!,
       replace: replace,

packages/go_router/lib/src/matching.dart

@@ -63,7 +63,6 @@ class RouteMatchList {
       _matches.isEmpty ? Uri() : Uri.parse(_matches.last.fullUriString);
 
   /// Pushes a match onto the list of matches.
-  // TODO(johnpryan): deprecate this API when new route types are added
   void push(RouteMatch match) {
     _matches.add(match);
   }

packages/go_router/lib/src/redirection.dart

@@ -8,7 +8,8 @@ import 'match.dart';
 import 'matching.dart';
 
 /// A GoRouter redirector function.
-// TODO(johnpryan): make redirector async (#105808)
+// TODO(johnpryan): make redirector async
+// See https://github.com/flutter/flutter/issues/105808
 typedef RouteRedirector = RouteMatchList Function(RouteMatchList matches,
     RouteConfiguration configuration, RouteMatcher matcher,
     {Object? extra});

packages/go_router/lib/src/router.dart

@@ -24,6 +24,7 @@ class GoRouter extends ChangeNotifier with NavigatorObserver {
   GoRouter({
     required List<GoRoute> routes,
     // TODO(johnpryan): Change to a route, improve error API
+    // See https://github.com/flutter/flutter/issues/108144
     GoRouterPageBuilder? errorPageBuilder,
     GoRouterWidgetBuilder? errorBuilder,
     GoRouterRedirect? redirect,
@@ -32,10 +33,12 @@ class GoRouter extends ChangeNotifier with NavigatorObserver {
     bool routerNeglect = false,
     String? initialLocation,
     // TODO(johnpryan): Deprecate this parameter
+    // See https://github.com/flutter/flutter/issues/108132
     UrlPathStrategy? urlPathStrategy,
     List<NavigatorObserver>? observers,
     bool debugLogDiagnostics = false,
     // TODO(johnpryan): Deprecate this parameter
+    // See https://github.com/flutter/flutter/issues/108145
     GoRouterNavigatorBuilder? navigatorBuilder,
     String? restorationScopeId,
   }) {
@@ -94,16 +97,13 @@ class GoRouter extends ChangeNotifier with NavigatorObserver {
 
   /// The router delegate. Provide this to the MaterialApp or CupertinoApp's
   /// `.router()` constructor
-  // TODO(johnpryan): change type to RouterDelegate
   GoRouterDelegate get routerDelegate => _routerDelegate;
 
   /// The route information provider used by [GoRouter].
-  // TODO(johnpryan): change type to RouteInformationProvider
   GoRouteInformationProvider get routeInformationProvider =>
       _routeInformationProvider;
 
   /// The route information parser used by [GoRouter].
-  // TODO(johnpryan): change type to RouteInformationParser
   GoRouteInformationParser get routeInformationParser =>
       _routeInformationParser;
 
@@ -119,6 +119,7 @@ class GoRouter extends ChangeNotifier with NavigatorObserver {
   /// Get a location from route name and parameters.
   /// This is useful for redirecting to a named location.
   // TODO(johnpryan): Deprecate this API
+  // See https://github.com/flutter/flutter/issues/107729
   String namedLocation(
     String name, {
     Map<String, String> params = const <String, String>{},