fix(logbook): commit timestamps overwrite run timestamps in logs

cmd/cmd_test.go

@@ -482,8 +482,8 @@ func TestSaveTransformModifiedButSameBody(t *testing.T) {
     created dataset from tf_123.star
 
 `, map[string]string{
-		"commit1": "/ipfs/Qmb18FaCYQMRbE4vH7G1sMnBr32opTBfKKeE6s7AYyR9YY",
-		"commit2": "/ipfs/QmSPaMdKzpEmodF7R6LzkTGHKtk4hvN6e6PsNKX1A6yW3g",
+		"commit1": "/ipfs/QmYSxB13nw1PWk6Y2itXxRiE42cvK5zzC9v6saGcDu8PFw",
+		"commit2": "/ipfs/QmZx6mYranv8ZFgwq7QVFoVVje2vPueC3MJYPy1LVnANmJ",
 	})
 	if diff := cmp.Diff(expect, output); diff != "" {
 		t.Errorf("log (-want +got):\n%s", diff)

cmd/test_runner_test.go

@@ -29,7 +29,7 @@ import (
 	"github.com/qri-io/qri/repo/gen"
 	reporef "github.com/qri-io/qri/repo/ref"
 	repotest "github.com/qri-io/qri/repo/test"
-	"github.com/qri-io/qri/transform/run"
+	tfrun "github.com/qri-io/qri/transform/run"
 	"github.com/qri-io/qri/transform/startf"
 	"github.com/spf13/cobra"
 )
@@ -131,7 +131,7 @@ func NewTestRunnerWithTempRegistry(t *testing.T, peerName, testName string) *Tes
 
 func useConsistentRunIDs() {
 	source := strings.NewReader(strings.Repeat("OmgZombies!?!?!", 200))
-	run.SetIDRand(source)
+	tfrun.SetIDRand(source)
 }
 
 func newTestRunnerFromRoot(root *repotest.TempRepo) *TestRunner {
@@ -192,7 +192,7 @@ func (run *TestRunner) Delete() {
 		os.RemoveAll(run.TmpDir)
 	}
 	// restore random RunID generator
-	run.SetIDRand(nil)
+	tfrun.SetIDRand(nil)
 	dsfs.Timestamp = run.DsfsTsFunc
 	logbook.NewTimestamp = run.LogbookTsFunc
 	StringerLocation = run.LocOrig

logbook/logbook.go

@@ -1132,6 +1132,7 @@ func runItemFromOp(ref dsref.Ref, op oplog.Op) dsref.VersionInfo {
 }
 
 func addCommitDetailsToRunItem(li dsref.VersionInfo, op oplog.Op) dsref.VersionInfo {
+	li.CommitTime = time.Unix(0, op.Timestamp)
 	li.CommitTitle = op.Note
 	li.BodySize = int(op.Size)
 	li.Path = op.Ref