core/agent
8
0
Fork 0
Code Issues 5 Pull requests Projects Releases Packages Wiki Activity Actions

fix(agentic): snapshot template content in versions

Browse source 
Co-Authored-By: Virgil <virgil@lethean.io>
This commit is contained in:
Virgil 2026-04-01 19:07:48 +00:00
parent 6adbf92277
commit 8fab46dcdc
2 changed files with 26 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
pkg/agentic/template.go
View file

@ -80,10 +80,11 @@ type TemplateCreatePlanOutput struct {
// version := agentic.PlanTemplateVersion{Slug: "new-feature", Version: 1, ContentHash: "..." }
type PlanTemplateVersion struct {
Slug string `json:"slug"`
Version int `json:"version"`
Name string `json:"name"`
ContentHash string `json:"content_hash"`
Slug string `json:"slug"`
Version int `json:"version"`
Name string `json:"name"`
Content planTemplateDefinition `json:"content,omitempty"`
ContentHash string `json:"content_hash"`
}
type planTemplateDefinition struct {
@ -361,10 +362,21 @@ func templateVersionFromContent(slug, name, content string) PlanTemplateVersion
sum := sha256.Sum256([]byte(content))
hash := hex.EncodeToString(sum[:])
version := templateVersionForHash(slug, hash)
var snapshot planTemplateDefinition
if err := yaml.Unmarshal([]byte(content), &snapshot); err != nil {
snapshot = planTemplateDefinition{}
}
snapshot.Slug = slug
if snapshot.Name == "" {
snapshot.Name = name
}
return PlanTemplateVersion{
Slug: slug,
Version: version,
Name: name,
Content: snapshot,
ContentHash: hash,
}
}

10
pkg/agentic/template_test.go
View file

@ -31,6 +31,7 @@ func TestTemplate_HandleTemplateList_Good(t *testing.T) {
assert.Equal(t, "development", summary.Category)
assert.Equal(t, 1, summary.Version.Version)
assert.NotEmpty(t, summary.Version.ContentHash)
assert.NotEmpty(t, summary.Version.Content.Name)
if summary.Slug == "bug-fix" {
found = true
assert.Equal(t, "Bug Fix", summary.Name)
@ -55,6 +56,8 @@ func TestTemplate_HandleTemplatePreview_Good(t *testing.T) {
assert.Equal(t, "new-feature", output.Template)
assert.Equal(t, 1, output.Version.Version)
assert.NotEmpty(t, output.Version.ContentHash)
assert.Equal(t, "new-feature", output.Version.Content.Slug)
assert.Equal(t, "New Feature", output.Version.Content.Name)
assert.Contains(t, output.Preview, "Authentication")
assert.Contains(t, output.Preview, "Phase 1")
}
@ -109,6 +112,8 @@ func TestTemplate_HandleTemplateCreatePlan_Good(t *testing.T) {
assert.Equal(t, "active", output.Plan.Status)
assert.Equal(t, 1, output.Version.Version)
assert.NotEmpty(t, output.Version.ContentHash)
assert.Equal(t, "new-feature", output.Version.Content.Slug)
assert.Equal(t, "New Feature", output.Version.Content.Name)
plan, err := readPlan(PlansRoot(), "auth-rollout")
require.NoError(t, err)
@ -138,6 +143,7 @@ func TestTemplate_HandleTemplateCreatePlan_Good_NoVariables(t *testing.T) {
assert.Equal(t, "draft", output.Plan.Status)
assert.Equal(t, 1, output.Version.Version)
assert.NotEmpty(t, output.Version.ContentHash)
assert.Equal(t, "api-consistency", output.Version.Content.Slug)
plan, err := readPlan(PlansRoot(), output.Plan.Slug)
require.NoError(t, err)
@ -190,6 +196,8 @@ func TestTemplate_TemplateVersionFromContent_Good_ReusesExistingVersion(t *testi
assert.Equal(t, 3, version.Version)
assert.Equal(t, hash, version.ContentHash)
assert.Equal(t, "new-feature", version.Content.Slug)
assert.Equal(t, "New Feature", version.Content.Name)
}
func TestTemplate_TemplateVersionFromContent_Bad_IncrementsOnChangedContent(t *testing.T) {
@ -218,6 +226,7 @@ func TestTemplate_TemplateVersionFromContent_Bad_IncrementsOnChangedContent(t *t
assert.Equal(t, 4, version.Version)
assert.NotEqual(t, hash, version.ContentHash)
assert.Equal(t, "new-feature", version.Content.Slug)
}
func TestTemplate_TemplateVersionFromContent_Ugly_IgnoresCorruptPlans(t *testing.T) {
@ -247,4 +256,5 @@ func TestTemplate_TemplateVersionFromContent_Ugly_IgnoresCorruptPlans(t *testing
version := templateVersionFromContent("new-feature", "New Feature", "name: New Feature\nphases:\n - name: Discovery\n")
assert.Equal(t, 4, version.Version)
assert.Equal(t, "new-feature", version.Content.Slug)
}