Skip to content

Add optional id field to widgets#24

Merged
terzioglub merged 4 commits into
mainfrom
terzioglub/widget-id
May 22, 2026
Merged

Add optional id field to widgets#24
terzioglub merged 4 commits into
mainfrom
terzioglub/widget-id

Conversation

@terzioglub
Copy link
Copy Markdown
Contributor

Summary

  • Adds optional id string field to Widget (model.go, json schema, TSX loader)
  • Pure addition: no validator rules, no existing fields touched, no frontend changes
  • Matches agent-runner's widget id (which is required there; we're keeping it optional in dac)
  • Enables cloud's selection/deep-link/data-fetch flows that key off widget.id

Test plan

  • `make test` green
  • `make build` green
  • All existing testdata/example dashboards still validate
  • 5 new test cases covering YAML/JSON round-trip + omitempty + optionality

🤖 Generated with Claude Code

terzioglub and others added 4 commits May 22, 2026 09:52
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@terzioglub terzioglub merged commit 462687f into main May 22, 2026
1 check passed
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