Skip to content

feat: add _meta provider metadata to AgentCustomization across clients#185

Merged
DonJayamanne merged 1 commit into
mainfrom
don/far-tiger
Jun 3, 2026
Merged

feat: add _meta provider metadata to AgentCustomization across clients#185
DonJayamanne merged 1 commit into
mainfrom
don/far-tiger

Conversation

@DonJayamanne

Copy link
Copy Markdown
Contributor

No description provided.

@DonJayamanne DonJayamanne marked this pull request as ready for review June 3, 2026 02:21
@DonJayamanne DonJayamanne requested a review from connor4312 June 3, 2026 02:21
@DonJayamanne DonJayamanne enabled auto-merge June 3, 2026 02:21
@connor4312

Copy link
Copy Markdown
Member

@DonJayamanne not opposed to this in principal but I'm curious why we need this already -- is there a non-standard field we want to talk about?

@DonJayamanne

Copy link
Copy Markdown
Contributor Author

y -- is there a non-standard field we want to talk about?

Had a chat with @aeschli and almost all of the fields are non standard or we have different interpretations
E.g. event the prompt isn't a standard field.
We treat model as an array, others & spec treats it as a single value
Tools have different meanings as well.

TLDR: Based on my discussions with martin, there aren't any, hence his suggestion was to not add any property to this spec

Phase 1

  • So i opted to go with the simplest to unblock from the perf issue of VS Code reading the file everytime
    I want to get this , as we have a perf issue with VS Code (client) reading the files everytime (& lots of AHP messages sent around)

Phase 2

  • Figure out what are the standard properties, we must have some, I'd leave phase 2 as a debt issue to remove this content property

@DonJayamanne DonJayamanne merged commit fa66dc4 into main Jun 3, 2026
8 checks passed
@DonJayamanne DonJayamanne deleted the don/far-tiger branch June 3, 2026 04:52
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