Transition barback's infrastructure to an aggregate-based model.

pkg/barback/lib/src/transformer/base_transform.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library barback.transformer.base_transform;
 
 import 'dart:async';
 
 import '../asset/asset_id.dart';
 import '../graph/transform_node.dart';
@@ skipping 29 matching lines @@
 
   /// A logger so that the [Transformer] can report build details.
   TransformLogger get logger => _logger;
   TransformLogger _logger;
 
   BaseTransform(this._node) {
     _logger = new TransformLogger((asset, level, message, span) {
       if (level == LogLevel.ERROR) _loggedError = true;
 
       // If the log isn't already associated with an asset, use the primary.
-      if (asset == null) asset = _node.primary.id;
+      if (asset == null) asset = _node.info.primaryId;

[Bob Nystrom, 2014/05/08 20:30:48]

This is a "key" now right? How about renaming "asset" -> "key" here and
elsewhere?

[nweiz, 2014/05/08 21:12:36]

See my comment on TransformInfo.primaryId.

       var entry = new LogEntry(_node.info, asset, level, message, span);
       _onLogController.add(entry);
     });
   }
 
   /// Consume a primary input so that it doesn't get processed by future
   /// phases or emitted once processing has finished.
   ///
   /// Normally each primary input will automatically be forwarded unless the
   /// transformer overwrites it by emitting an input with the same id. This
@@ skipping 16 matching lines @@
 
   /// The ids of primary inputs that should be consumed.
   Set<AssetId> get consumedPrimaries => transform._consumedPrimaries;
 
   /// Whether the transform logged an error.
   bool get loggedError => transform._loggedError;
 
   /// The stream of log entries emitted by the transformer during a run.
   Stream<LogEntry> get onLog => transform._onLogController.stream;
 
+  /// Whether the transform's input or id stream has been closed.
+  ///
+  /// See also [done].
+  bool get isDone;
+
   BaseTransformController(this.transform);
 
-  /// Notifies the [BaseTransform] that the transformation has finished being
-  /// applied.
+  /// Mark this transform as finished emitting new inputs or input ids.
+  ///
+  /// This is distinct from [cancel] in that it *doesn't* indicate that the
+  /// transform is finished being used entirely. The transformer may still log
+  /// messages and load secondary inputs. This just indicates that all the
+  /// primary inputs are accounted for.
+  void done();
+
+  /// Mark this transform as canceled.
   ///
   /// This will close any streams and release any resources that were allocated
-  /// for the duration of the transformation.
-  void close() {
+  /// for the duration of the transformation. Unlike [done], this indicates that
+  /// the transformation is no longer relevant; either it has returned, or
+  /// something external has preemptively invalidated its results.
+  void cancel() {
+    done();
     transform._onLogController.close();
   }
 }