Skip to content

Commit 6882bd7

Browse files
ecPabloecPablo
committed
fix: make newlines consistent and fix default value for format flag
1 parent 909e6f4 commit 6882bd7

23 files changed

Lines changed: 63 additions & 40 deletions

‎engine/cld/legacy/cli/mcmsv2/mcms_v2.go‎

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -733,7 +733,10 @@ func buildMCMSv2AnalyzeProposalCmd(
733733
}
734734

735735
// Set renderer based on format flag
736-
renderer := createRendererFromFormat(format)
736+
renderer, err := createRendererFromFormat(format)
737+
if err != nil {
738+
return fmt.Errorf("failed to create renderer: %w", err)
739+
}
737740
cfgv2.proposalCtx.SetRenderer(renderer)
738741

739742
var analyzedProposal string
@@ -1611,14 +1614,16 @@ func addCallProxyOption(
16111614

16121615
// createRendererFromFormat creates an appropriate renderer based on the format string.
16131616
// Defaults to markdown renderer for unknown formats.
1614-
func createRendererFromFormat(format string) analyzer.Renderer {
1617+
func createRendererFromFormat(format string) (analyzer.Renderer, error) {
16151618
switch format {
16161619
case "text", "txt":
1617-
return analyzer.NewTextRenderer()
1620+
return analyzer.NewTextRenderer(), nil
16181621
case "markdown", "md":
1619-
return analyzer.NewMarkdownRenderer()
1622+
return analyzer.NewMarkdownRenderer(), nil
1623+
case "":
1624+
return analyzer.NewMarkdownRenderer(), nil
16201625
default:
16211626
// Default to markdown if format is not specified or invalid
1622-
return analyzer.NewMarkdownRenderer()
1627+
return nil, fmt.Errorf("unknown format '%s'", format)
16231628
}
16241629
}

‎experimental/analyzer/decoded_call_test.go‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,6 @@ Method: complexCall
114114
Inputs:
115115
address: 0x0000000000000000000000000000000000000001
116116
chain: ` + "`" + `1 (<chain unknown>)` + "`" + `
117-
118117
data: 0x010203
119118
120119
Outputs:

‎experimental/analyzer/describe_test.go‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ func TestDescribeProposal(t *testing.T) {
3030
name: "Empty proposal",
3131
operations: []types.Operation{},
3232
expectError: false,
33-
outputContains: []string{""},
33+
outputContains: []string{"\n"},
3434
},
3535
{
3636
name: "Single operation - unsupported chain",
@@ -128,7 +128,7 @@ func TestDescribeTimelockProposal(t *testing.T) {
128128
name: "Empty proposal",
129129
operations: []types.BatchOperation{},
130130
expectError: false,
131-
outputContains: []string{""},
131+
outputContains: []string{"\n"},
132132
},
133133
{
134134
name: "Single batch - unsupported chain",

‎experimental/analyzer/renderer_markdown.go‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -608,7 +608,12 @@ func (r *MarkdownRenderer) renderTemplate(tmpl *template.Template, data interfac
608608
return fmt.Sprintf("Error rendering template: %v", err)
609609
}
610610

611-
return buf.String()
611+
out := buf.String()
612+
if !strings.HasSuffix(out, "\n\n") { // assuming double newline means intentional spacing
613+
out = strings.TrimSuffix(out, "\n")
614+
}
615+
return out
616+
612617
}
613618

614619
// isSimpleValue determines if a string represents a simple value that should be displayed without backticks.

‎experimental/analyzer/renderer_text.go‎

Lines changed: 28 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import (
44
"bytes"
55
"embed"
66
"fmt"
7+
"strings"
78
"text/template"
89

910
"github.com/ethereum/go-ethereum/common/hexutil"
@@ -132,8 +133,9 @@ func (r *TextRenderer) RenderDecodedCall(d *DecodedCall, ctx *FieldContext) stri
132133
if err := r.decodedCallTmpl.Execute(&buf, data); err != nil {
133134
return fmt.Sprintf("Error rendering decoded call: %v", err)
134135
}
135-
136-
return buf.String()
136+
out := buf.String()
137+
out = strings.TrimSuffix(out, "\n")
138+
return out
137139
}
138140

139141
// RenderProposal renders a ProposalReport as plain text using templates
@@ -156,7 +158,11 @@ func (r *TextRenderer) RenderProposal(rep *ProposalReport, ctx *FieldContext) st
156158
return fmt.Sprintf("Error rendering proposal: %v", err)
157159
}
158160

159-
return buf.String()
161+
out := buf.String()
162+
if !strings.HasSuffix(out, "\n\n") { // assuming double newline means intentional spacing
163+
out = strings.TrimSuffix(out, "\n")
164+
}
165+
return out
160166
}
161167

162168
// RenderTimelockProposal renders a Timelock ProposalReport as plain text using templates
@@ -188,7 +194,11 @@ func (r *TextRenderer) RenderTimelockProposal(rep *ProposalReport, ctx *FieldCon
188194
return fmt.Sprintf("Error rendering timelock: %v", err)
189195
}
190196

191-
return buf.String()
197+
out := buf.String()
198+
if !strings.HasSuffix(out, "\n\n") { // assuming double newline means intentional spacing
199+
out = strings.TrimSuffix(out, "\n")
200+
}
201+
return out
192202
}
193203

194204
// RenderField renders a NamedField as plain text
@@ -225,66 +235,71 @@ func (r *TextRenderer) getChainNameOrEmpty(selector uint64) string {
225235
// renderFieldValue renders any field value as plain text using templates
226236
func (r *TextRenderer) renderFieldValue(field FieldValue) string {
227237
var buf bytes.Buffer
228-
238+
var out string
229239
switch f := field.(type) {
230240
case AddressField:
231241
data := AddressFieldData{Value: f.GetValue()}
232242
if err := r.addressFieldTmpl.Execute(&buf, data); err != nil {
233243
return fmt.Sprintf("Error rendering address field: %v", err)
234244
}
235245

236-
return buf.String()
246+
out = buf.String()
237247

238248
case ChainSelectorField:
239249
if err := r.chainSelectorFieldTmpl.Execute(&buf, f); err != nil {
240250
return fmt.Sprintf("Error rendering chain selector field: %v", err)
241251
}
242252

243-
return buf.String()
253+
out = buf.String()
244254

245255
case BytesField:
246256
if err := r.bytesFieldTmpl.Execute(&buf, f); err != nil {
247257
return fmt.Sprintf("Error rendering bytes field: %v", err)
248258
}
249259

250-
return buf.String()
260+
out = buf.String()
251261

252262
case ArrayField:
253263
if err := r.arrayFieldTmpl.Execute(&buf, f); err != nil {
254264
return fmt.Sprintf("Error rendering array field: %v", err)
255265
}
256266

257-
return buf.String()
267+
out = buf.String()
258268

259269
case StructField:
260270
if err := r.structFieldTmpl.Execute(&buf, f); err != nil {
261271
return fmt.Sprintf("Error rendering struct field: %v", err)
262272
}
263273

264-
return buf.String()
274+
out = buf.String()
265275

266276
case SimpleField:
267277
if err := r.simpleFieldTmpl.Execute(&buf, f); err != nil {
268278
return fmt.Sprintf("Error rendering simple field: %v", err)
269279
}
270280

271-
return buf.String()
281+
out = buf.String()
272282

273283
case YamlField:
274284
if err := r.yamlFieldTmpl.Execute(&buf, f); err != nil {
275285
return fmt.Sprintf("Error rendering yaml field: %v", err)
276286
}
277287

278-
return buf.String()
288+
out = buf.String()
279289

280290
case NamedField:
281291
if err := r.namedFieldTmpl.Execute(&buf, f); err != nil {
282292
return fmt.Sprintf("Error rendering named field: %v", err)
283293
}
284294

285-
return buf.String()
295+
out = buf.String()
286296

287297
default:
288298
return fmt.Sprintf("<unknown field type: %s>", field.GetType())
289299
}
300+
301+
if !strings.HasSuffix(out, "\n\n") { // assuming double newline means intentional spacing
302+
out = strings.TrimSuffix(out, "\n")
303+
}
304+
return out
290305
}

‎experimental/analyzer/renderer_text_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ func TestTextRenderer_RenderField_ChainSelectorField(t *testing.T) {
142142
field := ChainSelectorField{Value: 999999}
143143
result := renderer.RenderField(NamedField{Name: "chain", Value: field}, ctx)
144144

145-
assert.Equal(t, "chain: `999999 (<chain unknown>)`\n", result)
145+
assert.Equal(t, "chain: `999999 (<chain unknown>)`", result)
146146
})
147147
}
148148

‎experimental/analyzer/templates/markdown/address_field.tmpl‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
`{{.Value}}`
1+
`{{.Value}}`

‎experimental/analyzer/templates/markdown/array_field.tmpl‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
array[{{.Length}}]: {{if eq .Length 0}}[]{{else}}[{{range $i, $elem := .Elements}}{{if $i}}, {{end}}{{compactValue $elem $.Context}}{{end}}{{if gt .Length 3}}, … (+{{sub .Length 3}}){{end}}]{{end}}
1+
array[{{.Length}}]: {{if eq .Length 0}}[]{{else}}[{{range $i, $elem := .Elements}}{{if $i}}, {{end}}{{compactValue $elem $.Context}}{{end}}{{if gt .Length 3}}, … (+{{sub .Length 3}}){{end}}]{{end}}

‎experimental/analyzer/templates/markdown/bytes_field.tmpl‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
bytes(len={{.Length}}): {{hexPreview .Value 16}}
1+
bytes(len={{.Length}}): {{hexPreview .Value 16}}

‎experimental/analyzer/templates/markdown/proposal.tmpl‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
{{range $i, $op := .Operations}}Operation #{{$i}}
22
Chain selector: {{$op.ChainSelector}} ({{$op.ChainName}})
33
{{range $call := $op.Calls}}{{indent (renderCall $call $.Context)}}{{end}}
4-
{{end}}
4+
{{end}}

0 commit comments

Comments
 (0)