fix(mcms/analyzer/renderer): inline complex parameter values in markdown for new proposal analysis framework#858
Conversation
🦋 Changeset detectedLatest commit: 319e513 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Pull request overview
Updates the MCMS proposal analysis markdown renderer to display multiline/complex parameter values as indented fenced blocks (closer to legacy output), and expands the golden fixtures to cover the new formatting.
Changes:
- Render parameters whose formatted value contains newlines as indented fenced
textblocks in markdown. - Add template helper functions to detect multiline params and indent block content.
- Extend the markdown golden test/fixture with complex parameter examples and add a changeset entry.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| engine/cld/mcms/proposalanalysis/renderer/templates/markdown/parameter.tmpl | Adds conditional block rendering for multiline parameter values. |
| engine/cld/mcms/proposalanalysis/renderer/funcmap.go | Adds paramNeedsBlock and indentLines helpers to support block rendering. |
| engine/cld/mcms/proposalanalysis/renderer/golden_test.go | Expands the golden proposal test data with complex params to exercise new rendering. |
| engine/cld/mcms/proposalanalysis/renderer/testdata/golden_markdown.md | Updates expected markdown output to include fenced blocks for multiline params. |
| .changeset/five-colts-yawn.md | Adds a patch changeset entry for the markdown formatting fix. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
engine/cld/mcms/proposalanalysis/renderer/templates/markdown/parameter.tmpl
Outdated
Show resolved
Hide resolved
92bc46b to
f01cb71
Compare
There was a problem hiding this comment.
Pull request overview
Updates the proposal analysis Markdown renderer to display complex/multiline parameter values inline as indented fenced blocks (matching legacy rendering), and extends the golden fixture/test to cover these cases.
Changes:
- Render multiline parameter values as indented ```text blocks under the parameter bullet.
- Add template helpers (
hasNewline,indentLines) to support the new Markdown formatting. - Expand golden test data to include bytes and structured/multiline inputs, plus a changeset entry.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| engine/cld/mcms/proposalanalysis/renderer/testdata/golden_markdown.md | Updates expected Markdown output to include inline fenced blocks for complex inputs. |
| engine/cld/mcms/proposalanalysis/renderer/templates/markdown/parameter.tmpl | Changes parameter rendering to switch between inline vs fenced-block formatting based on newlines. |
| engine/cld/mcms/proposalanalysis/renderer/golden_test.go | Extends the golden proposal inputs to exercise multiline/structured params and bytes formatting. |
| engine/cld/mcms/proposalanalysis/renderer/funcmap.go | Adds helper funcs for detecting newlines and indenting multiline values in templates. |
| .changeset/five-colts-yawn.md | Records the patch-level change for release tooling. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
f01cb71 to
2d70a6a
Compare
2d70a6a to
6c7c821
Compare
There was a problem hiding this comment.
Pull request overview
This PR improves the Markdown renderer used by the MCMS proposal analysis framework so that complex (multi-line) parameter values are displayed inline as indented fenced blocks, matching the legacy/expected presentation and updating golden outputs accordingly.
Changes:
- Render parameter values containing newlines as indented fenced
textblocks under the parameter bullet. - Add
hasNewlineandindentLinestemplate helpers to support multi-line formatting. - Expand/update golden fixtures and tests to cover complex parameter rendering (including nested JSON-like structures and bytes).
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| engine/cld/mcms/proposalanalysis/renderer/templates/markdown/parameter.tmpl | Detects multi-line formatted values and renders them as indented fenced blocks. |
| engine/cld/mcms/proposalanalysis/renderer/funcmap.go | Adds hasNewline and indentLines helpers to the template func map. |
| engine/cld/mcms/proposalanalysis/renderer/golden_test.go | Extends the golden proposal to include complex params (bytes + structured slice) to exercise the new rendering. |
| engine/cld/mcms/proposalanalysis/renderer/testdata/golden_markdown.md | Updates renderer golden output to reflect the new multi-line parameter formatting. |
| engine/cld/mcms/proposalanalysis/examples/ccip/analyzers/tokenpool/testdata/golden_markdown.md | Updates example analyzer golden output to match the renderer’s updated formatting/spacing. |
| .changeset/five-colts-yawn.md | Adds a patch changeset entry for the Markdown rendering fix. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
6c7c821 to
4c70573
Compare
Render multiline parameter values as indented text blocks within input bullets and expand golden test coverage.
4c70573 to
319e513
Compare
There was a problem hiding this comment.
Pull request overview
Updates the markdown renderer in the proposal analysis framework to render multiline/complex parameter values in a readable, legacy-like format, and extends golden coverage to validate the new rendering behavior.
Changes:
- Render parameters with multiline formatted values as fenced
textblocks within the input bullet. - Add template helper functions (
hasNewline,indentLines) to support the new markdown formatting. - Expand the markdown golden fixture and its generating test proposal to include complex, multiline parameters.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| engine/cld/mcms/proposalanalysis/renderer/templates/markdown/parameter.tmpl | Switches multiline parameter rendering to an indented fenced block under the parameter bullet. |
| engine/cld/mcms/proposalanalysis/renderer/funcmap.go | Adds hasNewline and indentLines template helpers and exposes them via the func map. |
| engine/cld/mcms/proposalanalysis/renderer/golden_test.go | Extends the golden proposal with bytes and a complex slice/struct parameter to exercise multiline rendering. |
| engine/cld/mcms/proposalanalysis/renderer/testdata/golden_markdown.md | Updates expected markdown output to include the new multiline parameter block rendering. |
| .changeset/five-colts-yawn.md | Adds a patch changeset entry for the renderer output fix. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
engine/cld/mcms/proposalanalysis/renderer/templates/markdown/parameter.tmpl
Show resolved
Hide resolved
|
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## chainlink-deployments-framework@0.88.1 ### Patch Changes - [#858](#858) [`b05a8f2`](b05a8f2) Thanks [@graham-chainlink](https://github.com/graham-chainlink)! - fix(mcms/analyzer/renderer): inline complex parameter values in markdown - [#853](#853) [`7b0467c`](7b0467c) Thanks [@finleydecker](https://github.com/finleydecker)! - fix: make RPC health check timeout configurable Co-authored-by: Graham Goh <graham.goh@smartcontract.com>




Render multiline parameter values as indented text blocks within input bullets and expand golden test coverage.
Currently:

Should be similar to the legacy behaviour:
