Skip to content

agents: allow composition refs on BYO image Agents - #624

Open
xytian315 wants to merge 4 commits into
mainfrom
xytian315/composition-gate-lift-image-only
Open

agents: allow composition refs on BYO image Agents#624
xytian315 wants to merge 4 commits into
mainfrom
xytian315/composition-gate-lift-image-only

Conversation

@xytian315

Copy link
Copy Markdown
Collaborator

Description

Motivation:

This lifts the gate for BYO image Agents: plugins, skills, and instructions are accepted when the Agent has a non-blank source image.

Change Type

/kind feature

Changelog

Image Agents can now declare plugins, skills, and instructions without compatibleHarnesses; a non-blank source image satisfies the composition gate.

Additional Notes

Copilot AI lite review requested due to automatic review settings August 18, 2026 05:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the v1alpha1 Agent API validation rules to allow composition references (plugins, skills, instructions) on “bring-your-own image” Agents when spec.source.image is non-blank, aligning with the stated goal of lifting the previous compatibleHarnesses gate for image-based Agents.

Changes:

  • Relaxed AgentSpec validation so plugins/skills/instructions are allowed when either compatibleHarnesses is set or a non-blank spec.source.image is provided.
  • Updated API comments/documentation to reflect the new composition delivery rules for image Agents.
  • Expanded unit tests to cover the new acceptance/rejection matrix (image vs harness vs repository source).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
pkg/api/v1alpha1/agent.go Updates AgentSpec composition docs to reflect that image Agents can receive composed plugins/skills/instructions.
pkg/api/v1alpha1/agent_validate.go Adjusts validation gate to permit composition refs when spec.source.image is non-blank.
pkg/api/v1alpha1/agent_harness_validate_test.go Adds coverage for image Agents using plugins/skills/instructions and keeps rejection cases for missing image+harness.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread pkg/api/v1alpha1/agent_validate.go
@xytian315
xytian315 marked this pull request as draft August 18, 2026 06:21
@xytian315
xytian315 marked this pull request as ready for review August 20, 2026 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants