nil guard and comment update

Author: gregns1

base/bootstrap.go

@@ -52,7 +52,7 @@ type BootstrapConnection interface {
 	// for the given bucket; subsequent bootstrap reads for that bucket stop falling back to
 	// _default._default. Per-bucket so completing one bucket never disables another's fallback.
 	SetMigrationComplete(bucketName string)
-	// IsMigrationComplete returns true if bootstrap-metadata migration to _system._mobile has finished for the given bucket
+	// IsMigrationComplete reports whether this process has marked bootstrap-metadata migration complete for the given bucket (local cache).
 	IsMigrationComplete(bucketName string) bool
 	// GetMetadataMigrationStatus reads the bucket-level metadata-migration status doc directly from
 	// _system._mobile (never via the dual-collection wrapper). Returns ErrNotFound when the doc has

rest/server_context.go

@@ -2403,9 +2403,12 @@ func (sc *ServerContext) initializeBootstrapConnection(ctx context.Context) erro
 	return nil
 }
 
-// RecheckPendingBucketMetadataMigrations will iterate through all buckets and check if any bucket metadata migrations
-// still in pending state can now be moved into in_progress state.
+// RecheckPendingBucketMetadataMigrations iterates through all config buckets and re-evaluates
+// whether bucket-level metadata migration can now be completed.
 func (sc *ServerContext) RecheckPendingBucketMetadataMigrations(ctx context.Context) {
+	if sc.BootstrapContext == nil || sc.BootstrapContext.Connection == nil {
+		return
+	}
 	buckets, err := sc.BootstrapContext.Connection.GetConfigBuckets(ctx)
 	if err != nil {
 		base.WarnfCtx(ctx, "Couldn't list buckets to re-check bucket metadata migration: %v", err)