Skip to content

fix(web): enable JSON_OBJECT type support in console UI#37

Open
tomerqodo wants to merge 5 commits intoaugment_combined_20260121_augment_sentry_coderabbit_1_base_fixweb_enable_json_object_type_support_in_console_ui_pr439from
augment_combined_20260121_augment_sentry_coderabbit_1_head_fixweb_enable_json_object_type_support_in_console_ui_pr439
Open

fix(web): enable JSON_OBJECT type support in console UI#37
tomerqodo wants to merge 5 commits intoaugment_combined_20260121_augment_sentry_coderabbit_1_base_fixweb_enable_json_object_type_support_in_console_ui_pr439from
augment_combined_20260121_augment_sentry_coderabbit_1_head_fixweb_enable_json_object_type_support_in_console_ui_pr439

Conversation

@tomerqodo
Copy link

Benchmark PR from qodo-benchmark#439

@augmentcode
Copy link

augmentcode bot commented Jan 22, 2026

🤖 Augment PR Summary

Summary: This PR enables editing/support for JSON_OBJECT input variables in the console UI.

Changes:

  • Updates the JSON schema placeholder to a full JSON Schema object (type/properties) rather than just a properties map.
  • Stores and displays the full json_schema content in the config modal (instead of only .properties).
  • Enables the JSON_OBJECT type option for non-basic apps and normalizes empty schemas by removing json_schema from the saved payload.
  • Adds confirm-time validation + localized error messages when the schema is invalid JSON or not an object schema.

Technical Notes: The modal now persists json_schema as the raw JSON string and validates schema.type === "object" before saving.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 2 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

return ''
try {
return JSON.stringify(JSON.parse(tempPayload.json_schema).properties, null, 2)
return JSON.stringify(JSON.parse(tempPayload.json_schema), null, 2)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jsonSchemaStr’s useMemo reads type, but the dependency list only includes tempPayload.json_schema, so the memoized value can become stale if the user switches the input type away from JSON and then back.

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎

}
handlePayloadChange('json_schema')(JSON.stringify(res, null, 2))
handlePayloadChange('json_schema')(value)
return True
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

return True will throw at runtime because True isn’t a defined identifier in JS/TS, which would break JSON schema editing as soon as onChange fires.

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎

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