Skip to content

fix: Support writing keys with invalid chars#255

Merged
kkozik-amplify merged 1 commit intoamplify-education:tmccombs/key-outputfrom
tmccombs:key-output
Feb 19, 2026
Merged

fix: Support writing keys with invalid chars#255
kkozik-amplify merged 1 commit intoamplify-education:tmccombs/key-outputfrom
tmccombs:key-output

Conversation

@tmccombs
Copy link

@tmccombs tmccombs commented Feb 7, 2026

This actually fixes a few of things related to expressions for object keys:

  1. It allows they keys contain characters that aren't valid as identifiers, if the key is a plain string, for example: ":"
  2. It removes the need for superflouous parenthesis around expresions in the keys
  3. It no longer puts double quotes around an interpolated string in the key.

The second two were actually kind of side-affects of my fix for 1.

If we want to preserve the previous behavior for 2 and 3, I think it wouldn't be too hard to do though.

This actually fixes a few of things related to expressions for
object keys:
1. It allows they keys contain characters that aren't valid as identifiers, if the
   key is a plain string, for example: ":"
2. It removes the need for superflouous parenthesis around expresions in the keys
3. It no longer puts double quotes around an interpolated string in the key.

The second two were actually kind of side-affects of my fix for 1.

If we want to preserve the previous behavior for 2 and 3, I think it wouldn't be too hard to do
though.
@tmccombs tmccombs requested a review from a team as a code owner February 7, 2026 07:59
@kkozik-amplify kkozik-amplify changed the base branch from main to tmccombs/key-output February 19, 2026 11:48
@kkozik-amplify kkozik-amplify merged commit 24cda19 into amplify-education:tmccombs/key-output Feb 19, 2026
7 of 9 checks passed
@tmccombs tmccombs deleted the key-output branch February 19, 2026 15:39
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