Export more package API endpoints.

Author: isoos

app/lib/package/api_export/api_exporter.dart

@@ -10,6 +10,7 @@ import 'package:gcloud/service_scope.dart' as ss;
 import 'package:gcloud/storage.dart';
 import 'package:logging/logging.dart';
 import 'package:pub_dev/frontend/handlers/atom_feed.dart';
+import 'package:pub_dev/scorecard/backend.dart';
 import 'package:pub_dev/service/security_advisories/backend.dart';
 import 'package:pub_dev/shared/exceptions.dart';
 import 'package:pub_dev/shared/parallel_foreach.dart';
@@ -244,6 +245,22 @@ final class ApiExporter {
           versionListing,
           forceWrite: forceWrite,
         );
+    await _api.package(package).likes.write(
+          await packageBackend.getPackageLikesCount(package),
+          forceWrite: forceWrite,
+        );
+    await _api.package(package).options.write(
+          await packageBackend.getPackageOptions(package),
+          forceWrite: forceWrite,
+        );
+    await _api.package(package).publisher.write(
+          await packageBackend.getPublisherInfo(package),
+          forceWrite: forceWrite,
+        );
+    await _api.package(package).score.write(
+          await scoreCardBackend.getVersionScore(package),
+          forceWrite: forceWrite,
+        );
     await _api.package(package).feedAtomFile.write(
           await buildPackageAtomFeedContent(package),
           forceWrite: forceWrite,

app/lib/package/api_export/exported_api.dart

@@ -6,6 +6,7 @@ import 'dart:async';
 import 'dart:convert';
 import 'dart:io';
 
+import 'package:_pub_shared/data/account_api.dart';
 import 'package:_pub_shared/data/advisories_api.dart';
 import 'package:_pub_shared/data/package_api.dart';
 import 'package:clock/clock.dart';
@@ -273,6 +274,20 @@ final class ExportedPackage {
   ExportedJsonFile<ListAdvisoriesResponse> get advisories =>
       _suffix<ListAdvisoriesResponse>('/advisories');
 
+  /// Interface for writing `/api/packages/<package>/likes`.
+  ExportedJsonFile<PackageLikesCount> get likes =>
+      _suffix<PackageLikesCount>('/likes');
+
+  /// Interface for writing `/api/packages/<package>/options`.
+  ExportedJsonFile<PkgOptions> get options => _suffix<PkgOptions>('/options');
+
+  /// Interface for writing `/api/packages/<package>/publisher`.
+  ExportedJsonFile<PackagePublisherInfo> get publisher =>
+      _suffix<PackagePublisherInfo>('/publisher');
+
+  /// Interface for writing `/api/packages/<package>/score`.
+  ExportedJsonFile<VersionScore> get score => _suffix<VersionScore>('/score');
+
   /// Interface for writing `/api/packages/<package>/feed.atom`
   ExportedAtomFeedFile get feedAtomFile => ExportedAtomFeedFile._(
         _owner,
@@ -406,6 +421,10 @@ final class ExportedPackage {
     await Future.wait([
       _owner._pool.withResource(() async => await versions.delete()),
       _owner._pool.withResource(() async => await advisories.delete()),
+      _owner._pool.withResource(() async => await likes.delete()),
+      _owner._pool.withResource(() async => await options.delete()),
+      _owner._pool.withResource(() async => await publisher.delete()),
+      _owner._pool.withResource(() async => await score.delete()),
       _owner._pool.withResource(() async => await feedAtomFile.delete()),
       ..._owner._prefixes.map((prefix) async {
         await _owner._listBucket(

app/lib/package/backend.dart

@@ -18,6 +18,7 @@ import 'package:meta/meta.dart';
 import 'package:pool/pool.dart';
 import 'package:pub_dev/package/api_export/api_exporter.dart';
 import 'package:pub_dev/package/tarball_storage.dart';
+import 'package:pub_dev/scorecard/backend.dart';
 import 'package:pub_dev/service/async_queue/async_queue.dart';
 import 'package:pub_dev/service/rate_limit/rate_limit.dart';
 import 'package:pub_dev/shared/versions.dart';
@@ -493,6 +494,7 @@ class PackageBackend {
     });
     await purgePackageCache(package);
     await taskBackend.trackPackage(package);
+    await apiExporter!.synchronizePackage(package);
   }
 
   /// Updates [options] on [package]/[version], assuming the current user
@@ -531,6 +533,9 @@ class PackageBackend {
       }
     });
     await purgePackageCache(package);
+    await purgeScorecardData(package, version,
+        isLatest: pkg.latestVersion == version);
+    await apiExporter!.synchronizePackage(package);
   }
 
   /// Verifies an update to the credential-less publishing settings and
@@ -795,7 +800,7 @@ class PackageBackend {
     if (currentPublisherId != null) {
       await purgePublisherCache(publisherId: currentPublisherId);
     }
-
+    await apiExporter!.synchronizePackage(packageName);
     return rs;
   }
 

app/lib/task/backend.dart

@@ -18,6 +18,7 @@ import 'package:indexed_blob/indexed_blob.dart' show BlobIndex, FileRange;
 import 'package:logging/logging.dart' show Logger;
 import 'package:pana/models.dart' show Summary;
 import 'package:pool/pool.dart' show Pool;
+import 'package:pub_dev/package/api_export/api_exporter.dart';
 import 'package:pub_dev/package/backend.dart';
 import 'package:pub_dev/package/models.dart';
 import 'package:pub_dev/package/upload_signer_service.dart';
@@ -721,6 +722,7 @@ class TaskBackend {
 
     // Clearing the state cache after the update.
     await _purgeCache(package, version);
+    await apiExporter!.synchronizePackage(package);
 
     // If nothing else is running on the instance, delete it!
     // We do this in a microtask after returning, so that it doesn't slow down

app/test/admin/exported_api_sync_test.dart

@@ -71,8 +71,8 @@ void main() {
     testWithProfile('baseline checks', fn: () async {
       await syncExportedApi();
       final data = await listExportedApi();
-      expect(data.keys, hasLength(greaterThan(20)));
-      expect(data.keys, hasLength(lessThan(40)));
+      expect(data.keys, hasLength(greaterThan(50)));
+      expect(data.keys, hasLength(lessThan(70)));
 
       final oxygenFiles = data.keys.where((k) => k.contains('oxygen')).toSet();
       expect(oxygenFiles, hasLength(greaterThan(5)));
@@ -82,12 +82,20 @@ void main() {
         '$runtimeVersion/api/archives/oxygen-2.0.0-dev.tar.gz',
         '$runtimeVersion/api/packages/oxygen',
         '$runtimeVersion/api/packages/oxygen/advisories',
+        '$runtimeVersion/api/packages/oxygen/likes',
+        '$runtimeVersion/api/packages/oxygen/options',
+        '$runtimeVersion/api/packages/oxygen/publisher',
+        '$runtimeVersion/api/packages/oxygen/score',
         '$runtimeVersion/api/packages/oxygen/feed.atom',
         'latest/api/archives/oxygen-1.0.0.tar.gz',
         'latest/api/archives/oxygen-1.2.0.tar.gz',
         'latest/api/archives/oxygen-2.0.0-dev.tar.gz',
         'latest/api/packages/oxygen',
         'latest/api/packages/oxygen/advisories',
+        'latest/api/packages/oxygen/likes',
+        'latest/api/packages/oxygen/options',
+        'latest/api/packages/oxygen/publisher',
+        'latest/api/packages/oxygen/score',
         'latest/api/packages/oxygen/feed.atom',
       });
 

app/test/package/api_export/api_exporter_test.dart

@@ -132,8 +132,36 @@ Future<void> _testExportedApiSynchronization(
       isNotNull,
     );
     expect(
-      await bucket.readBytes('$runtimeVersion/api/packages/foo/feed.atom'),
-      isNotNull,
+      await bucket.readGzippedJson('$runtimeVersion/api/packages/foo/likes'),
+      {
+        'package': 'foo',
+        'likes': 0,
+      },
+    );
+    expect(
+      await bucket.readGzippedJson('$runtimeVersion/api/packages/foo/options'),
+      {
+        'isDiscontinued': false,
+        'replacedBy': null,
+        'isUnlisted': false,
+      },
+    );
+    expect(
+      await bucket
+          .readGzippedJson('$runtimeVersion/api/packages/foo/publisher'),
+      {
+        'publisherId': null,
+      },
+    );
+    expect(
+      await bucket.readGzippedJson('$runtimeVersion/api/packages/foo/score'),
+      {
+        'grantedPoints': isNotNull,
+        'maxPoints': isNotNull,
+        'likeCount': isNotNull,
+        'tags': isNotEmpty,
+        'lastUpdated': isNotNull,
+      },
     );
     expect(
       await bucket.readGzippedJson('$runtimeVersion/api/packages/foo'),
@@ -143,6 +171,10 @@ Future<void> _testExportedApiSynchronization(
         'versions': hasLength(1),
       },
     );
+    expect(
+      await bucket.readString('$runtimeVersion/api/packages/foo/feed.atom'),
+      contains('v1.0.0 of foo'),
+    );
     expect(
       await bucket
           .readGzippedJson('$runtimeVersion/api/package-name-completion-data'),
@@ -156,10 +188,6 @@ Future<void> _testExportedApiSynchronization(
       await bucket.readString('$runtimeVersion/feed.atom'),
       contains('v1.0.0 of foo'),
     );
-    expect(
-      await bucket.readString('$runtimeVersion/api/packages/foo/feed.atom'),
-      contains('v1.0.0 of foo'),
-    );
   }
 
   _log.info('## New package');
@@ -197,6 +225,22 @@ Future<void> _testExportedApiSynchronization(
       await bucket.readString('latest/api/packages/foo/feed.atom'),
       contains('v1.0.0 of foo'),
     );
+    expect(
+      await bucket.readGzippedJson('latest/api/packages/foo/likes'),
+      isNotNull,
+    );
+    expect(
+      await bucket.readGzippedJson('latest/api/packages/foo/options'),
+      isNotNull,
+    );
+    expect(
+      await bucket.readGzippedJson('latest/api/packages/foo/publisher'),
+      isNotNull,
+    );
+    expect(
+      await bucket.readGzippedJson('latest/api/packages/foo/score'),
+      isNotNull,
+    );
     // Note. that name completion data won't be updated until search caches
     //       are purged, so we won't test that it is updated.
 
@@ -444,6 +488,10 @@ Future<void> _testExportedApiSynchronization(
       await bucket.readGzippedJson('latest/api/packages/bar'),
       isNull,
     );
+    expect(
+      await bucket.readGzippedJson('latest/api/packages/bar/options'),
+      isNull,
+    );
     expect(
       await bucket.readGzippedJson('latest/api/packages/feed.atom'),
       isNull,
@@ -484,6 +532,10 @@ Future<void> _testExportedApiSynchronization(
         'versions': hasLength(2),
       },
     );
+    expect(
+      await bucket.readGzippedJson('latest/api/packages/bar/options'),
+      isNotNull,
+    );
     expect(
       await bucket.readBytes('latest/api/archives/bar-2.0.0.tar.gz'),
       isNotNull,

app/test/task/task_test.dart

@@ -190,7 +190,9 @@ void main() {
     await fakeTime.elapse(minutes: 10);
   });
 
-  testWithFakeTime('failing instances will be retried', (fakeTime) async {
+  testWithFakeTime('failing instances will be retried', expectedLogMessages: [
+    'SHOUT [pub-notice:cached_value] Updating cached `thirtyDaysTotalDownloadCounts` value failed.',
+  ], (fakeTime) async {
     await taskBackend.backfillTrackingState();
     await fakeTime.elapse(minutes: 1);