olblak · Sep 16, 2026 · Sep 15, 2026 · Sep 15, 2026 · Sep 15, 2026 · Sep 15, 2026
diff --git a/pkg/core/pipeline/actions.go b/pkg/core/pipeline/actions.go
@@ -188,7 +188,7 @@ func (p *Pipeline) RunActions(ctx context.Context) error {
 			pipelineName = p.Name
 		}
 
-		action.Report.ID = fmt.Sprintf("%x", sha256.Sum256([]byte(p.Name)))
+		action.Report.ID = p.actionReportID()
 		action.Report.Title = action.Title
 		action.Report.PipelineTitle = pipelineName
 
@@ -358,3 +358,15 @@ func (p *Pipeline) detectActionTitle(action *action.Action) {
 		}
 	}
 }
+
+// actionReportID returns the ID identifying the pipeline in action reports, such as pullrequest descriptions.
+// It relies on the unrendered pipeline name, so the ID doesn't change when the name references runtime values
+// such as {{ source "id" }}, and the report of an existing pullrequest keeps being merged instead of duplicated.
+func (p *Pipeline) actionReportID() string {
+	name := p.unrenderedName
+	// Pipelines not created by Init have no unrendered name
+	if name == "" {
+		name = p.Name
+	}
+	return fmt.Sprintf("%x", sha256.Sum256([]byte(name)))
+}
diff --git a/pkg/core/pipeline/main.go b/pkg/core/pipeline/main.go
@@ -53,6 +53,8 @@ type Pipeline struct {
 	// injected by the engine before the pipeline runs.
 	SourceCache *cache.SourceCache
 	tracer      trace.Tracer
+	// unrenderedName holds the pipeline name before runtime values, such as {{ source "id" }}, are rendered.
+	unrenderedName string
 }
 
 // Init initialize an updatecli context based on its configuration
@@ -62,6 +64,7 @@ func (p *Pipeline) Init(config *config.Config, options Options) error {
 	if len(config.Spec.Title) > 0 && p.Name == "" {
 		p.Name = config.Spec.Title
 	}
+	p.unrenderedName = p.Name
 
 	p.Options = options
 
@@ -80,7 +83,7 @@ func (p *Pipeline) Init(config *config.Config, options Options) error {
 	p.Report.Sources = make(map[string]*result.Source, len(config.Spec.Sources))
 	p.Report.Conditions = make(map[string]*result.Condition, len(config.Spec.Conditions))
 	p.Report.Targets = make(map[string]*result.Target, len(config.Spec.Targets))
-	p.Report.Name = config.Spec.Name
+	p.Report.Name = p.Name
 	p.Report.Result = result.SKIPPED
 	p.Report.PipelineID = config.Spec.PipelineID
 	p.Report.Labels = config.Spec.Labels
@@ -667,13 +670,26 @@ func (p *Pipeline) String() string {
 	return result
 }
 
+// refreshName sets the pipeline and report names from the rendered configuration.
+func (p *Pipeline) refreshName() {
+	p.Name = p.Config.Spec.Name
+	if len(p.Config.Spec.Title) > 0 && p.Config.Spec.Name == "" {
+		p.Name = p.Config.Spec.Title
+	}
+	p.Report.Name = p.Name
+}
+
 // Update updates the pipeline based on the latest configuration
 func (p *Pipeline) Update() error {
 	err := p.Config.Update(p)
 	if err != nil {
 		return err
 	}
 
+	// The pipeline name may reference runtime values, such as {{ source "id" }},
+	// so it's refreshed once the configuration is rendered.
+	p.refreshName()
+
 	// Reset scm
 	for id, scmConfig := range p.Config.Spec.SCMs {
 		var err error

diff --git a/pkg/core/pipeline/main_name_test.go b/pkg/core/pipeline/main_name_test.go
@@ -0,0 +1,111 @@
+package pipeline
+
+import (
+	"crypto/sha256"
+	"fmt"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+	"github.com/updatecli/updatecli/pkg/core/config"
+	"github.com/updatecli/updatecli/pkg/core/pipeline/source"
+	"github.com/updatecli/updatecli/pkg/core/result"
+)
+
+func TestUpdateRefreshesName(t *testing.T) {
+	testdata := []struct {
+		name         string
+		sourceResult string
+		expectedName string
+	}{
+		{
+			name:         "source succeeded",
+			sourceResult: result.SUCCESS,
+			expectedName: "deps(go): bump module golang.org/x/net to v0.38.0",
+		},
+		{
+			name:         "source skipped keeps the placeholder",
+			sourceResult: result.SKIPPED,
+			expectedName: `deps(go): bump module golang.org/x/net to {{ source "fixed" }}`,
+		},
+	}
+
+	for _, tt := range testdata {
+		t.Run(tt.name, func(t *testing.T) {
+			rawName := `deps(go): bump module golang.org/x/net to {{ source "fixed" }}`
+
+			p := Pipeline{
+				ID: "pipeline-id",
+				Config: &config.Config{
+					Spec: config.Spec{
+						Name:       rawName,
+						PipelineID: "pipeline-id",
+					},
+				},
+				Sources: map[string]source.Source{
+					"fixed": {
+						Result: &result.Source{Result: tt.sourceResult},
+						Output: "v0.38.0",
+					},
+				},
+			}
+			p.Report.Name = rawName
+
+			require.NoError(t, p.Update())
+
+			assert.Equal(t, tt.expectedName, p.Name)
+			assert.Equal(t, tt.expectedName, p.Report.Name)
+			assert.Equal(t, "pipeline-id", p.ID)
+			assert.Equal(t, "pipeline-id", p.Config.Spec.PipelineID)
+		})
+	}
+}
+
+// TestActionReportIDIgnoresRenderedName ensures the action report ID doesn't change when the pipeline name
+// is rendered, so pullrequest reports created before and after a version change are merged together.
+func TestActionReportIDIgnoresRenderedName(t *testing.T) {
+	rawName := `deps(go): bump module golang.org/x/net to {{ source "fixed" }}`
+
+	p := Pipeline{}
+	require.NoError(t, p.Init(&config.Config{
+		Spec: config.Spec{
+			Name:       rawName,
+			PipelineID: "pipeline-id",
+		},
+	}, Options{}))
+
+	p.Sources["fixed"] = source.Source{
+		Result: &result.Source{Result: result.SUCCESS},
+		Output: "v0.38.0",
+	}
+	require.NoError(t, p.Update())
+
+	require.Equal(t, "deps(go): bump module golang.org/x/net to v0.38.0", p.Name)
+	assert.Equal(t, fmt.Sprintf("%x", sha256.Sum256([]byte(rawName))), p.actionReportID())
+}
+
+func TestActionReportIDWithoutInit(t *testing.T) {
+	p := Pipeline{Name: "pipeline"}
+
+	assert.Equal(t, fmt.Sprintf("%x", sha256.Sum256([]byte("pipeline"))), p.actionReportID())
+}
+
+// TestInitNameFallsBackToTitle ensures a specification defining only a title exposes that title
+// as both the pipeline name and the report name, including before the configuration is rendered.
+func TestInitNameFallsBackToTitle(t *testing.T) {
+	p := Pipeline{}
+	require.NoError(t, p.Init(&config.Config{
+		Spec: config.Spec{
+			Title:      "Bump golang.org/x/net",
+			PipelineID: "pipeline-id",
+		},
+	}, Options{}))
+
+	assert.Equal(t, "Bump golang.org/x/net", p.Name)
+	assert.Equal(t, "Bump golang.org/x/net", p.Report.Name)
+
+	require.NoError(t, p.Update())
+
+	assert.Equal(t, "Bump golang.org/x/net", p.Name)
+	assert.Equal(t, "Bump golang.org/x/net", p.Report.Name)
+}