Skip to content

Spell parameter default values per language instead of pasting text - #332

Merged
matt-edmondson merged 2 commits into
mainfrom
claude/project-thread-kddtpg-182
Oct 20, 2026
Merged

matt-edmondson merged 2 commits into
mainfrom
claude/project-thread-kddtpg-182

Conversation

@matt-edmondson

Copy link
Copy Markdown
Contributor

@/home/claude/body-182.md


Generated by Claude Code

…patch]

Parameter.DefaultValue was target-language source text written out verbatim, so the
README's own DefaultValue = "False" produced C#, C++ and JavaScript that failed to
compile or threw, and no spelling of a flag worked in every target. It is now an
Expression, written by each generator's own literal writer; C, Go and Rust note it
in a comment with any */ broken apart. Parameter.ParseDefaultValue and
FormatDefaultValue are the text form YAML and the editor field use, so documents
written while it was text still read.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018tZnrna9oooMZHDaLyXumk
Comment thread Coder.Test/Ast/ParameterDefaultValueTests.cs Fixed
Comment thread Coder.Test/Ast/ParameterDefaultValueTests.cs Fixed
Comment thread Coder.Test/Ast/ParameterDefaultValueTests.cs Fixed
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018tZnrna9oooMZHDaLyXumk
@sonarqubecloud

Copy link
Copy Markdown

@matt-edmondson
matt-edmondson merged commit 108d466 into main Oct 20, 2026
16 checks passed
@matt-edmondson
matt-edmondson deleted the claude/project-thread-kddtpg-182 branch October 20, 2026 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants