Skip to content

Add missing Title and Icons properties to ResourceLinkBlock#1320

Merged
stephentoub merged 3 commits intomainfrom
copilot/add-missing-properties-to-resourcelinkblock
Feb 20, 2026
Merged

Add missing Title and Icons properties to ResourceLinkBlock#1320
stephentoub merged 3 commits intomainfrom
copilot/add-missing-properties-to-resourcelinkblock

Conversation

Copy link
Contributor

Copilot AI commented Feb 20, 2026

ResourceLinkBlock was missing Title and Icons properties. Per the 2025-11-25 spec, ResourceLink extends Resource, and Resource extends BaseMetadata, Icons—so ResourceLink inherits both title and icons.

Changes

  • ResourceLinkBlock: Added Title (string?) and Icons (IList<Icon>?) properties
  • ContentBlock.Converter: Updated custom JSON read/write to handle title and icons for resource_link type, using AOT-compatible source-generated type info for Icon serialization
  • Tests: Updated existing roundtrip/minimal tests, added dedicated test for Title and Icons deserialization
var link = new ResourceLinkBlock
{
    Uri = "https://example.com/resource",
    Name = "my-resource",
    Title = "My Resource",
    Icons = [new Icon { Source = "https://example.com/icon.png", MimeType = "image/png" }]
};

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…5 spec

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
Copilot AI changed the title [WIP] Add missing properties to ResourceLinkBlock Add missing Title and Icons properties to ResourceLinkBlock Feb 20, 2026
Copilot AI requested a review from stephentoub February 20, 2026 02:01
@stephentoub stephentoub merged commit 54283a8 into main Feb 20, 2026
9 checks passed
@stephentoub stephentoub deleted the copilot/add-missing-properties-to-resourcelinkblock branch February 20, 2026 03:46
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.

3 participants

Comments